Skip to content

Feat: Add a link back to the skill when opening a link issue (breadcrumb) - #41

Draft
esnible wants to merge 1 commit into
rossoctl:mainfrom
esnible:breadcrumb-link-checker
Draft

Feat: Add a link back to the skill when opening a link issue (breadcrumb)#41
esnible wants to merge 1 commit into
rossoctl:mainfrom
esnible:breadcrumb-link-checker

Conversation

@esnible

@esnible esnible commented Jul 29, 2026

Copy link
Copy Markdown
Member

Summary

I'd like the issue opened by the link checker to include a 'breadcrumb' back to the link checker itself. This will help the developer find the skill that generated the report. The developer can then decide if the skill itself could be improved, in addition to fixing the reported problem.

I noticed there were only 2 forks of this repo, suggesting that Rossoctl developers are unaware of the potential of contributing automation to increase our work velocity.

Signed-off-by: Ed Snible <snible@us.ibm.com>
@esnible esnible added the ready-for-ai-review Request automated AI code review from clawgenti label Jul 29, 2026

@clawgenti clawgenti left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Docs-only change that adds a breadcrumb link back to the scanner skill and a note about auto-close behavior — clear and useful context for issue recipients. All checks pass. Ready for human review.


Reviewed by clawgenti using github:pr-review

@rubambiza rubambiza left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change to reconsider the assertion that the automation does not create PRs with fixes.

Detected by: OpenClaw Link Health Scanner (cron: link-health-scanner)"
Detected by: [Rossoctl Automation Link Health Scanner](https://github.com/rossoctl/automation/blob/main/skills/link-health/SKILL.md) (cron: link-health-scanner)"

Note that [Rossoctl Automation](https://github.com/rossoctl/automation) will auto-close this issue if fixed, but will not create a PR with fixes.

@rubambiza rubambiza Jul 29, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@esnible I agree that the automation auto-closes the issues. However, at least for internal links, the automation (i.e., link-health-fixer) does create commits

# Apply each fix

and push PRs

with fixes.

@rubambiza rubambiza added ready-for-human-review AI review passed, ready for human reviewer and removed ready-for-ai-review Request automated AI code review from clawgenti labels Jul 29, 2026
@esnible
esnible marked this pull request as draft July 29, 2026 19:45
@esnible

esnible commented Jul 29, 2026

Copy link
Copy Markdown
Member Author

Converted to draft while I add references to https://github.com/rossoctl/automation/blob/main/scripts/link-health-fixer.sh

@rubambiza

rubambiza commented Jul 29, 2026

Copy link
Copy Markdown
Contributor

@esnible If you are referring to particular lines in link-health-fixer.sh, let me know because I am working toward a refactor that makes all of the scripts more portable beyond the rossoctl org. The spec for it can be found at https://github.com/rossoctl/automation/blob/main/docs/specs/2026-07-29-org-portability-design.md

@rubambiza

Copy link
Copy Markdown
Contributor

Nice breadcrumb idea — we just adopted the same "link back to the skill" pattern for the pr-review program's review footer. One portability caveat worth surfacing here:

The (cron: link-health-scanner) part of the breadcrumb is only accurate when the skill runs under our OpenClaw setup (clawgenti on the automation host). Once a skill is importable via /plugin and run in someone else's environment, there is no link-health-scanner cron and no standing-orders/... to point at — so a hardcoded cron/standing-order breadcrumb becomes wrong for every external importer.

For pr-review (whose skill now lives in rossoctl/agent-skills, importable) we resolved this by making the footer skill-link only — it names the one thing true in any environment:

*Reviewed by clawgenti using the [github-pr-review skill](https://github.com/rossoctl/agent-skills/blob/main/skills/github-pr-review/SKILL.md)*

The OpenClaw-specific provenance (cron job, schedule, scope) stays documented in the automation standing order (standing-orders/pr-review.md), where it's accurate and version-controlled, rather than baked into the portable skill's output.

link-health's skill still lives in automation today, so the cron breadcrumb is fine as-is here. But if/when it migrates to agent-skills (like pr-review did), the same skill-link-only (or env-aware) treatment would keep the breadcrumb correct for importers.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready-for-human-review AI review passed, ready for human reviewer

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants