Skip to content

[codex] link fixing PRs in close comments#38

Merged
Patrick-Erichsen merged 1 commit into
mainfrom
codex/link-fix-pr-in-close-comments
May 4, 2026
Merged

[codex] link fixing PRs in close comments#38
Patrick-Erichsen merged 1 commit into
mainfrom
codex/link-fix-pr-in-close-comments

Conversation

@Patrick-Erichsen
Copy link
Copy Markdown
Contributor

Summary

  • Resolve implemented-on-main fix SHAs back to merged PRs when applying ClawSweeper close proposals.
  • Add a Fixed by: block to durable close comments before manual issue closure, with commit fallback for direct-push fixes.
  • Cover PR-linked and commit-only rendering in unit tests.

Why

When ClawSweeper manually closes an issue as already implemented on main, GitHub only shows a generic closed this as completed timeline event. This preserves a clear audit trail in the bot comment by naming the PR or commit that actually landed the fix.

Verification

  • pnpm run check

@Patrick-Erichsen Patrick-Erichsen marked this pull request as ready for review May 4, 2026 22:46
@Patrick-Erichsen Patrick-Erichsen force-pushed the codex/link-fix-pr-in-close-comments branch from 855f6b5 to 0761da3 Compare May 4, 2026 22:52
@Patrick-Erichsen Patrick-Erichsen merged commit 86376d8 into main May 4, 2026
5 checks passed
@Patrick-Erichsen Patrick-Erichsen deleted the codex/link-fix-pr-in-close-comments branch May 4, 2026 23:09
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