Skip to content

Conversation

@Spiev
Copy link
Owner

@Spiev Spiev commented Feb 12, 2026

Summary

  • Add system prompt enforcing brevity (no filler, no praise, no pleasantries)
  • Dependabot PRs: replace verbose text with compact table format (Type/Risk/Change/Security/Action)
  • Feature PRs: reduce 6-category checklist to focused bullet points (Summary/Security/Issues/Suggestions)
  • Add mandatory Security section to both review types — always explicitly shown
  • Lower max_tokens from 1500 → 300 (Dependabot) / 800 (Feature)

Test plan

  • Trigger review on a Dependabot PR and verify table format
  • Trigger review on a feature PR and verify concise output with Security section

🤖 Generated with Claude Code

- Add system prompt enforcing brevity (no filler, no praise)
- Dependabot: replace verbose text with compact table format
- Feature PRs: reduce 6-category checklist to focused bullet points
- Add mandatory Security section to both review types
- Lower max_tokens (1500 → 300 dependabot, 800 feature)

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@Spiev Spiev deployed to pr-review February 12, 2026 12:15 — with GitHub Actions Active
@github-actions
Copy link

🔍 Code Review (by Claude)

Issues

  • Incomplete diff: The provided code change is truncated mid-line (Descr). Cannot verify the full feature PR prompt structure.

Security Review

  • No issues in the visible code (system prompt definition, table format changes, prompt restructuring).

Notes

  • Token reduction (1500→300 for Dependabot, 1500→800 for feature) may be too aggressive. 300 tokens could truncate table responses or complex security findings.
  • Mandatory Security section is good practice.
  • Table format for Dependabot reviews improves readability.
  • Missing test plan checkbox completion - ensure actual testing happens.

Review generated by Claude AI. Please use your judgment for final approval.

@Spiev Spiev merged commit eddef96 into main Feb 12, 2026
5 checks passed
@Spiev Spiev deleted the refactor/concise-pr-reviews branch February 12, 2026 12:16
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