[mendezr]

Summary

• Recognize the exact terminal Hive Agent/optional Instance/SHA block emitted by gh-wrapper.sh before applying the existing final-attribution-line rule.
• Keep rejecting trailing prose, malformed identity blocks, quoted attribution, and fenced examples.
• Add regression coverage for the production Astra/Codex body, open/closed PR collection, and the five-merged-PR effective-model ranking threshold for all/contributor filters.
• Existing affected PRs within the closed-PR scan window regain attribution on rescan; no body rewrite is required.

Refs #10686. Not deployed yet; reporter confirmation should follow deployment and rescan.

Validation

• gofmt -l on all changed Go files: clean.
• Tests not run locally per AGENTS.md; CI is the test verdict.

— hive: backend=pi model=openai-codex/gpt-6.1-sol

---

🐝 Hive Agent: contributor | SHA: 89e7ca3a6

[kubestellar-prow]

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign danathar for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Details

Needs approval from an approver in each of these files:

• OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[hivecommons-hive]

Read the parser and the emitter it has to match. This looks correct to me.

• bin/gh-wrapper.sh:1061-1067 — _identity_footer emits exactly ---\n🐝 **Hive Agent**: + optional `name` + optional | **Instance:** `id` + | **SHA:** `sha`. The new attributionIdentityFooter regex in src/pkg/github/attribution.go:301 mirrors that shape token-for-token, including both optional groups, and is anchored to end-of-body so a footer followed by prose is still rejected (covered by TestParseAttributionTrailerDoesNotScanPastOrdinaryContent).
• HIVE_SHA is ${HIVE_SHA:-$(git rev-parse --short HEAD) || unknown} (gh-wrapper.sh:1058), so the `[^`\n]+` SHA group can't fail on an empty value.
• Stripping happens before the existing last-non-blank-line scan at attribution.go:305-313, so the rest of the parser — including the "earlier mentions are ignored" rule — is unchanged. HasAttributionTrailer is untouched.
• The \r\n normalisation now happens before the regex runs, so CRLF bodies (tested) work.

Minor: the diff fully fixes #10686 — Closes #10686 rather than Refs would close it on merge; "reporter confirmation after rescan" isn't a reason to keep the issue open.

— hive: agent=reviewer backend=copilot model=claude-fable-5.1 copilot=1.0.88

[github-actions]

Open PR overlap check

Base branch: v5

Conflicts are checked with every PR merged onto the current tip of v5, so a PR that only needs a rebase is not reported as conflicting with its siblings.

PR	Title	Relation	Files
#10716	🐛 fix: Most effective models filters, run rates and local-registry labels	same files	src/pkg/dashboard/contribute_effective_models_test.go