[jimcadden]

Summary

• pr-review-fixer.sh footer prints the github-pr-review attribution line for the review body and its Slack post. Inline comments stay without it. The scanner and fixer pin _meta on startup: write it only after the SHA resolves, never overwrite a valid file, and fall back to blob/main when resolution fails.
• weekly-report.sh appends the github-weekly-report line to the report that gets posted.
• The agent instruction to append that line lives in the repoman skill (feat: Show a REPOMAN banner on interactive sessions repoman#9). Standing orders stay with the run-path migration in Migrate standing-orders off $ORG/core-repos.txt onto the RepoMan run path #118.

Refs rossoctl/repoman#10

Test plan

• bash tests/test-attribution-consumers.sh
• bash tests/test-weekly-report.sh
• bash tests/test-skill-meta.sh
• CI on the previous head (shellcheck + the new test). This docs commit re-runs it.

Assisted-By: Claude Code

[clawgenti]

Adds the pinned-SHA attribution footer to the pr-review flow (new footer command plus startup _meta pinning in the fixer/scanner) and to weekly-report (footer appended to the posted report); the new hermetic test covers create-only _meta, no-refresh of a valid pin, and the blob/main fallback, and all three claimed tests pass locally with CI green and DCO signed.

Findings:

• suggestion — standing-orders/pr-review.md still documents the fixer as "Two modes" (begin/finalize); document the new footer command there so the review workflow actually invokes it (see inline comment).

---

Reviewed by clawgenti using the github-pr-review skill

[rubambiza]

Human review is done. I'll follow up with an AI review for anything I may have missed.

Update: LiteLLM is down. So, the AI-assisted review is skipped.

[jimcadden]

Addressed the standing-orders suggestion in 1bb5706.

pr-review.md now documents footer and says to append that exact line to the review body and its Slack post. Inline comments stay without one. The matching skill text is on rossoctl/repoman#9.

Assisted-By: Claude Code

[clawgenti]

The follow-up commit addresses the prior docs gap — standing-orders/pr-review.md now documents the fixer's footer command and where the pinned-SHA line goes — with CI green on both legs, DCO signed, and the new hermetic attribution test wired into both the shellcheck and run lists.

All checks pass. Ready for human review.

---

Reviewed by clawgenti using the github-pr-review skill

[jimcadden]

Took the human review in 5982a35.

• Removed the 6a / Phase 6 labels from the fixer, scanner, weekly-report, and the new test. The comments now state the rules: write _meta only after the SHA resolves, never overwrite a valid file, and fall back to blob/main when resolution fails.
• Blank line after the shebang in tests/test-attribution-consumers.sh.
• Reverted the standing-orders edit. Documenting footer there belongs with Migrate standing-orders off $ORG/core-repos.txt onto the RepoMan run path #118. The agent instruction stays on feat: Show a REPOMAN banner on interactive sessions repoman#9.
• Left the footer sentence as Generated by RepoMan <skill>@<sha> .... The suggested printf patch adds a skill argument the format string never prints, and it would leave the fallback line unchanged.

Assisted-By: Claude Code

[clawgenti]

Follow-up on 5982a35: this docs-only commit removes the 6a phase tags from the provenance comments, states the footer rules directly in the new test's header, and reverts the standing-orders edit in favor of #118 as requested. The reviewed behavior is unchanged, DCO is signed on all three commits, and CI is green on both legs.

All checks pass. Ready for human review.

---

Reviewed by clawgenti using the github-pr-review skill

[rubambiza]

LGTM

[clawgenti]

Follow-up on 5982a35: docs-only push restating the footer rules without phase labels in the script comments and the new test header; reviewed behavior is unchanged, CI is green on both legs, and DCO is signed on all three commits.

All checks pass. Ready for human review.

---

Reviewed by clawgenti using the github-pr-review skill