Skip to content

docs: improve Claude rules based on eval results#1851

Merged
KRRT7 merged 2 commits intomainfrom
cf-update-claude-rules
Mar 17, 2026
Merged

docs: improve Claude rules based on eval results#1851
KRRT7 merged 2 commits intomainfrom
cf-update-claude-rules

Conversation

@KRRT7
Copy link
Copy Markdown
Collaborator

@KRRT7 KRRT7 commented Mar 17, 2026

Summary

  • Strengthened docstring, verification, tmp_path, and .resolve() rules based on eval runs comparing with-rules vs without-rules behavior
  • Expanded git commit/PR conventions and bug fix workflow instructions
  • Added type annotation and verification rules that were missing

Eval findings that drove changes

  • Docstrings: Both with/without rules added docstrings — stronger phrasing needed
  • tmp_path: With-rules agent copied old tempfile.mkdtemp() from existing code — explicitly banned alternatives
  • .resolve(): Never followed — added concrete example
  • prek: Baseline ran ruff/mypy separately — clarified prek is the single command

KRRT7 added 2 commits March 16, 2026 20:45
Expanded bug fix workflow to explicit 5-step sequence with subagent delegation, aligned type annotation rule with codebase conventions, simplified verification rule to reference prek, and expanded git/PR guidelines.
Eval showed docstrings, tmp_path, and .resolve() rules were being
ignored. Made phrasing more explicit and added concrete examples.
@claude
Copy link
Copy Markdown
Contributor

claude bot commented Mar 17, 2026

Claude finished @KRRT7's task in 15s —— View job


No substantive code changes to review.

@KRRT7 KRRT7 merged commit 4822d1a into main Mar 17, 2026
26 checks passed
@KRRT7 KRRT7 deleted the cf-update-claude-rules branch March 17, 2026 04:05
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