Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion .claude/policy-brief.md
Original file line number Diff line number Diff line change
Expand Up @@ -62,7 +62,7 @@ Every agent-authored or materially agent-modified diff, as a final pass before t
- Remove generated boilerplate, needless wrappers and abstractions, and other local-style drift.

Preserve required validation, security, cancellation, provenance, and failure handling.
Re-run the owning issue's normal verification after the pass, and record a 1-3 sentence anti-slop summary in the handoff.
Re-run the owning issue's normal verification after the pass, and record a 1-3 sentence quality summary in the handoff.
The pass is review judgment on the diff; do not add brittle automated style metrics.
Canonical policy: Berry Studio BSP-002 搂4.2 and 搂4.3 (https://app.notion.com/p/3b39cb079ddb811d8527ed129c3ac511).

Expand Down
2 changes: 1 addition & 1 deletion .cursor/agent-policy.md
Original file line number Diff line number Diff line change
Expand Up @@ -62,7 +62,7 @@ Every agent-authored or materially agent-modified diff, as a final pass before t
- Remove generated boilerplate, needless wrappers and abstractions, and other local-style drift.

Preserve required validation, security, cancellation, provenance, and failure handling.
Re-run the owning issue's normal verification after the pass, and record a 1-3 sentence anti-slop summary in the handoff.
Re-run the owning issue's normal verification after the pass, and record a 1-3 sentence quality summary in the handoff.
The pass is review judgment on the diff; do not add brittle automated style metrics.
Canonical policy: Berry Studio BSP-002 搂4.2 and 搂4.3 (https://app.notion.com/p/3b39cb079ddb811d8527ed129c3ac511).

Expand Down
4 changes: 2 additions & 2 deletions .github/pull_request_template.md
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@
- [ ] Invalid state stops before partial mutation or publication and returns a descriptive error or result
- [ ] Names carry the domain intent, and comments explain rationale rather than restating the code

## Anti-slop pass
## Quality pass

- [ ] Redundant or explanatory comments that do not match the file's style removed
- [ ] Abnormal defensive checks and broad try/catch blocks removed where a trusted upstream boundary already guarantees the invariant, with real boundary and safety checks kept
Expand All @@ -31,7 +31,7 @@
- [ ] Generated boilerplate, needless wrappers, and local-style drift removed
- [ ] Validation, security, cancellation, provenance, and failure handling preserved

Anti-slop summary (1-3 sentences):
Quality summary (1-3 sentences):

<!-- What was removed, and what was kept on purpose. -->

Expand Down
2 changes: 1 addition & 1 deletion AGENTS.md
Original file line number Diff line number Diff line change
Expand Up @@ -62,7 +62,7 @@ Every agent-authored or materially agent-modified diff, as a final pass before t
- Remove generated boilerplate, needless wrappers and abstractions, and other local-style drift.

Preserve required validation, security, cancellation, provenance, and failure handling.
Re-run the owning issue's normal verification after the pass, and record a 1-3 sentence anti-slop summary in the handoff.
Re-run the owning issue's normal verification after the pass, and record a 1-3 sentence quality summary in the handoff.
The pass is review judgment on the diff; do not add brittle automated style metrics.
Canonical policy: Berry Studio BSP-002 搂4.2 and 搂4.3 (https://app.notion.com/p/3b39cb079ddb811d8527ed129c3ac511).

Expand Down
2 changes: 1 addition & 1 deletion agent-policy.json
Original file line number Diff line number Diff line change
Expand Up @@ -63,7 +63,7 @@
],
"preserve": "Preserve required validation, security, cancellation, provenance, and failure handling.",
"closing": [
"Re-run the owning issue's normal verification after the pass, and record a 1-3 sentence anti-slop summary in the handoff.",
"Re-run the owning issue's normal verification after the pass, and record a 1-3 sentence quality summary in the handoff.",
"The pass is review judgment on the diff; do not add brittle automated style metrics.",
"Canonical policy: Berry Studio BSP-002 搂4.2 and 搂4.3 (https://app.notion.com/p/3b39cb079ddb811d8527ed129c3ac511)."
]
Expand Down
12 changes: 12 additions & 0 deletions changes/cc-pr-template-quality-pass.evidence.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,12 @@
format_version: 1
kind: evidence
claims:
- id: policy-handoff-quality-summary
evidence:
- unit:scripts/agent/test_architecture_contracts.py
- architecture:scripts/agent/check-architecture.py
- architecture:scripts/agent/generate-adapters.py
- id: pr-template-quality-pass
evidence:
- architecture:scripts/generate-architecture-catalogs.py
unresolved: []
4 changes: 4 additions & 0 deletions changes/cc-pr-template-quality-pass.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,4 @@
Category: internal
Audience: contributors
Breaking-Change: no
Summary: Rename the pull request template's "Anti-slop pass" section and summary to "Quality pass" and "Quality summary", and ask for a "quality summary" in the agent policy handoff; checklist items and the agent-policy.json key names are unchanged.
Loading