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

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

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