Skip to content

docs(agent-rules): require --bug-hunt on manual specfact code review runs#209

Merged
djm81 merged 1 commit intodevfrom
chore/agent-rules-specfact-bug-hunt
Apr 16, 2026
Merged

docs(agent-rules): require --bug-hunt on manual specfact code review runs#209
djm81 merged 1 commit intodevfrom
chore/agent-rules-specfact-bug-hunt

Conversation

@djm81
Copy link
Copy Markdown
Contributor

@djm81 djm81 commented Apr 16, 2026

Updates the canonical manual specfact code review run examples in docs/agent-rules/50-quality-gates-and-review.md and docs/agent-rules/20-repository-context.md to include --bug-hunt, with short rationale (CrossHair bug-hunt budgets) and a note that pre-commit keeps its own faster invocation via scripts/pre_commit_code_review.py.

Bumps last_reviewed on the touched rule pages.

Made with Cursor

…runs.

Document the flag in the quality gate checklist, repository command block, and
clean-code gate notes so agents default to CrossHair bug-hunt budgets outside
pre-commit.

Made-with: Cursor
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Apr 16, 2026

Caution

Review failed

Pull request was closed or merged during review

📝 Walkthrough

Walkthrough

Documentation updates to agent rules clarifying SpecFact code review command invocation. Modified command examples to include the --bug-hunt flag and updated last_reviewed metadata to 2026-04-16 across both documentation files.

Changes

Cohort / File(s) Summary
Agent Rules Documentation
docs/agent-rules/20-repository-context.md, docs/agent-rules/50-quality-gates-and-review.md
Updated SpecFact clean-code review commands to include --bug-hunt flag for bug-hunt timeout behavior. Clarified that manual invocations must pass this flag. Refreshed last_reviewed dates and expanded documentation to specify --bug-hunt requirements when running reviews outside pre-commit helpers.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The pull request description is largely incomplete relative to the repository template. It lacks most required sections including Scope, Bundle Impact, Validation Evidence, CI checks, and signature integrity checklist. Complete the standard PR template by filling in Scope (documentation changes), Bundle Impact assessment, and required validation evidence sections before merge.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title follows the Conventional Commits style with 'docs:' prefix and clearly describes the documentation change to require --bug-hunt flag on manual specfact code review runs.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/agent-rules-specfact-bug-hunt

Comment @coderabbitai help to get the list of available commands and usage tips.

@djm81 djm81 self-assigned this Apr 16, 2026
@djm81 djm81 added enhancement New feature or request codebase Specfact codebase related topic dependencies Pull requests that update a dependency file labels Apr 16, 2026
@djm81 djm81 moved this to In Progress in SpecFact CLI Apr 16, 2026
@djm81 djm81 linked an issue Apr 16, 2026 that may be closed by this pull request
@djm81 djm81 merged commit 05b982b into dev Apr 16, 2026
6 of 7 checks passed
@github-project-automation github-project-automation bot moved this from In Progress to Done in SpecFact CLI Apr 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

codebase Specfact codebase related topic dependencies Pull requests that update a dependency file enhancement New feature or request

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

[Change] Code review bug-finding mode and sidecar venv self-scan fix

1 participant