Skip to content

chore(internal): add contributors update skill#1932

Merged
julienld merged 6 commits into
masterfrom
restore-contributors-update-skill
Jul 19, 2026
Merged

chore(internal): add contributors update skill#1932
julienld merged 6 commits into
masterfrom
restore-contributors-update-skill

Conversation

@julienld

@julienld julienld commented Jul 18, 2026

Copy link
Copy Markdown
Member

Summary

  • add a /contributors-update skill to identify merged PR authors missing from the README
  • require a preview and explicit approval before editing or pushing
  • allow approved documentation-only (*.md) adjustments to be pushed directly to master without a PR
  • require a clean, current master, an exact diff preview, and explicit approval for both the change and direct push
  • keep code, configuration, generated artifacts, and workflow files on the normal worktree and draft-PR path
  • document the new skill in AGENTS.md

Testing

  • git diff --check
  • documentation-only change; no runtime tests required

🤖 Generated with Claude Code

Co-Authored-By: Claude <noreply@anthropic.com>
@ghhamcp

ghhamcp commented Jul 18, 2026

Copy link
Copy Markdown
Contributor

@codex review — apply the review criteria in .gemini/styleguide.md in addition to AGENTS.md guidance

@julienld julienld changed the title chore(internal): restore contributors update skill chore(internal): add contributors update skill Jul 18, 2026

@chatgpt-codex-connector chatgpt-codex-connector Bot 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: d3f8e9c3be

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread .claude/skills/contributors-update/SKILL.md Outdated
Comment thread .claude/skills/contributors-update/SKILL.md Outdated
Comment thread .claude/skills/contributors-update/SKILL.md Outdated
Comment thread .claude/skills/contributors-update/SKILL.md Outdated
Comment thread .claude/skills/contributors-update/SKILL.md
Co-Authored-By: Claude <noreply@anthropic.com>

@julienld julienld left a comment

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

✅ Addressed all five Codex review findings in cef4981: primary-checkout worktree discovery, merged-history marker lookup, server-side merge-date filtering, untrusted PR metadata handling, and post-PR CI/review validation.

julienld and others added 4 commits July 18, 2026 11:36
Co-Authored-By: Claude <noreply@anthropic.com>
Co-Authored-By: Claude <noreply@anthropic.com>
Co-Authored-By: Claude <noreply@anthropic.com>
Co-Authored-By: Claude <noreply@anthropic.com>
@julienld
julienld marked this pull request as ready for review July 19, 2026 00:00
@julienld
julienld requested review from a team and kingpanther13 July 19, 2026 00:00
@julienld
julienld merged commit 177dae1 into master Jul 19, 2026
28 of 29 checks passed
@julienld
julienld deleted the restore-contributors-update-skill branch July 19, 2026 00:00
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.

2 participants