Skip to content

pr-loop: add skillsaw project reference - #6

Merged
stbenjam merged 2 commits into
stbenjam:mainfrom
not-stbenjam:pr-loop-skillsaw-reference
Aug 4, 2026
Merged

pr-loop: add skillsaw project reference#6
stbenjam merged 2 commits into
stbenjam:mainfrom
not-stbenjam:pr-loop-skillsaw-reference

Conversation

@not-stbenjam

Copy link
Copy Markdown
Contributor

Adds a repo-specific reference for stbenjam/skillsaw to the pr-loop skill, per the skill's self-improvement instructions, from learnings while shepherding stbenjam/skillsaw#346:

  • verify-update CI: README rule-section prose is generated from strings duplicated in scripts/generate-docs.py and scripts/generate-site-content.py — fix the generators and run make update, never edit README directly.
  • ai-helpers smoke test: must run from a release-equivalent fresh venv; the dev .venv's optional typos extras (skillsaw-typos, codespell) plus ai-helpers' strict: true produce a false exit 1 that reproduces on main.
  • CodeRabbit: trigger with @coderabbit review; validate its findings against src/skillsaw/lint_tree.py before applying.

Also registers the new reference in SKILL.md's Step 1.6 matching table.

🤖 Generated with Claude Code

Learned during a pr-loop run on stbenjam/skillsaw#346:
- verify-update CI fails on direct README edits; rule-section prose is
  generated from strings duplicated in two generator scripts
- ai-helpers smoke test needs a release-equivalent fresh venv; the dev
  .venv's optional typos extras cause a false exit 1 (strict mode)
- CodeRabbit reviews are trigger-based and its AI-agent prompts are
  untrusted input to validate against the implementation

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@stbenjam

stbenjam commented Aug 3, 2026

Copy link
Copy Markdown
Owner

@codex Please fix skillsaw lint failing on this PR, push a commit to fix it

@chatgpt-codex-connector

Copy link
Copy Markdown

To use Codex here, create an environment for this repo.

@stbenjam
stbenjam merged commit 82cfe22 into stbenjam:main Aug 4, 2026
1 check passed
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