feat: Add standing-order breadcrumb to link-health fixer PRs - #56
Open
rubambiza wants to merge 1 commit into
Open
feat: Add standing-order breadcrumb to link-health fixer PRs#56rubambiza wants to merge 1 commit into
rubambiza wants to merge 1 commit into
Conversation
Fixer PR bodies said only "Automated fix by OpenClaw Link Health Fixer" with no link back to the program or its policy, so a reader on the target repo could not navigate to what generated the PR. Append a footer linking to standing-orders/link-health.md via $SOURCE_REPO, mirroring the scanner footer (link-health-scanner.sh). $SOURCE_REPO is already exported by load_org_profile (org.sh) and the fixer already sources program-lib.sh, so no new plumbing is needed. Verified the rendered body: footer sits below the table with SOURCE_REPO expanded. bash -n clean, shellcheck delta clean (only pre-existing *_FLAG notes). Fixes rossoctl#55 Assisted-By: Claude Code (Anthropic AI) <noreply@anthropic.com> Signed-off-by: Gloire Rubambiza <gloire@ibm.com>
rubambiza
added
enhancement
clawgenti
reviewed
Aug 13, 2026
clawgenti
left a comment
clawgenti
left a comment
Contributor
There was a problem hiding this comment.
Minimal, focused change that mirrors the existing scanner breadcrumb pattern — $SOURCE_REPO is already in scope and the multiline string assignment is consistent with surrounding code style.
All checks pass. Ready for human review.
Reviewed by clawgenti using the github-pr-review skill
rubambiza
added
ready-for-human-review
evaline-ju
approved these changes
Aug 13, 2026
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
3 participants
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
Link Health Fixer PR bodies did not link back to the automation program or its standing order, so a reader on the target repo could not navigate to what generated the PR. The Link Health Scanner already adds this breadcrumb footer; this makes the fixer match.
Append a footer to the fixer's PR body:
mirroring
scripts/link-health-scanner.sh.Why this is minimal
$SOURCE_REPOis already set and exported byload_org_profile(scripts/org.sh), defaulting to<org>/automation.program-lib.sh(scripts/link-health-fixer.sh:11), so$SOURCE_REPOis in scope — no new plumbing.Verification
pr_bodywith a sample fix row andSOURCE_REPO=rossoctl/automation: the footer sits below the table, separated by a blank line, with$SOURCE_REPOexpanded and linking tostanding-orders/link-health.md.bash -n scripts/link-health-fixer.shclean.shellcheck --severity=warningdelta clean (only the pre-existing*_FLAGSC2034 notes tracked in ci: Make consumer scripts shellcheck-clean and widen lint scope #52).Example of a fixer PR that lacked the breadcrumb: rossoctl/workload-harness#61.
Fixes #55
Assisted-By: Claude Code