feat: short PR refs + harvest merged PR by commit (0.26.6)#54
Merged
Conversation
Two refinements to the close-time ledger: - artifact extractor now reads "PR #51" / "PR#51" / "pull request #51" from event text (normalised to "PR #51"), anchored to the PR keyword so a bare "#3" in prose is not captured. - harvest falls back to GitHub commit search (gh pr list --search <HEAD-sha> --state merged) when the branch's open PR is gone — the common case of closing a task on main after the branch was deleted. Best-effort. Bump 0.26.6. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Two refinements to the close-time artifact ledger (follow-up to #53).
PR #51/PR#51/pull request #51from event text (normalised toPR #51), not just full/pull/NURLs. Anchored to the PR keyword so a bare#3in prose is notmistaken for a PR.
main(branch already gone, so
gh pr viewfinds nothing), the harvest falls backto GitHub commit search (
gh pr list --search <HEAD-sha> --state merged) torecover the PR that introduced HEAD. Best-effort.
Type of change
Test plan
captures_short_pr_reference_but_not_bare_hash(PR #N yes,#3no)harvest::build+ artifact tests still greencargo fmt/clippy -D warnings/cargo test --workspace/cargo docCHANGELOG
## [0.26.6].