Skip to content

[Fix] Guard likely owner commit links#52

Open
samzong wants to merge 1 commit intoopenclaw:mainfrom
samzong:fix/likely-owner-commit-links
Open

[Fix] Guard likely owner commit links#52
samzong wants to merge 1 commit intoopenclaw:mainfrom
samzong:fix/likely-owner-commit-links

Conversation

@samzong
Copy link
Copy Markdown

@samzong samzong commented May 7, 2026

What's changed?

  • Trim likely owner commit entries before rendering them.
  • Render only SHA-like values as commit links in Likely related people.
  • Add unit coverage for mixed provenance where a PR URL is present alongside a valid commit SHA.

Why

ClawSweeper public comments can receive non-SHA provenance strings in likelyOwners[].commits. Before this change, those values were passed directly to the commit-link renderer, producing invalid GitHub commit URLs in visible comments.

Evidence from existing public ClawSweeper comments:

Validation

  • pnpm run check
  • /pre-ship --committed: 0 MUST-FIX findings, no deferred findings

Signed-off-by: samzong <samzong.lu@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant