Skip to content

docs: permission-to-document anti-rationalization row (P6 reflex, BRO-1288)#7

Merged
broomva merged 1 commit into
mainfrom
feature/bro-1292-proactive-doc-reflex
Jun 1, 2026
Merged

docs: permission-to-document anti-rationalization row (P6 reflex, BRO-1288)#7
broomva merged 1 commit into
mainfrom
feature/bro-1292-proactive-doc-reflex

Conversation

@broomva
Copy link
Copy Markdown
Owner

@broomva broomva commented Jun 1, 2026

What

Adds the permission-to-document anti-rationalization row to the autonomous skill — propagating the BRO-1288 P6 reflex tightening ("documentation is a reflex, not a request — and never a question") into the canonical source.

| "I'll ask the user whether to file this into the knowledge graph" | Documentation is a reflex, not a request — and **never a question**. File proactively …, then report what you filed in one line. Asking permission to document is the permission-to-document anti-pattern; the user vetoes after, never gates before. |

Why this PR exists

During BRO-1288 the row was added to the workspace's gitignored .agents/ snapshot of this skill — which can't be committed and gets overwritten on npx skills update. The canonical broomva/autonomous repo (what installs actually pull) never received it. This closes that gap.

Scope notes

  • One row + one CHANGELOG line. The canonical repo is already ahead of the stale local snapshot (it has the P20 integration the snapshot lacks), so this adds only the missing row — no snapshot sync-back.
  • Content is identical to the row already cross-reviewed (P20, 8/10) and merged in BRO-1288 across broomva/bookkeeping, broomva/bstack (v0.23.1), and broomva/workspace. Below the substantive P20 threshold (single file, doc-only) → CI + CodeRabbit are the gate.
  • validate.yml structure checks pass locally (frontmatter, name, description "Use when…", LICENSE).

Closes BRO-1292 · follows BRO-1288.

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Documentation
    • Clarified system documentation regarding documentation and knowledge-graph filing procedures. Updated guidelines specify these operations should function as automatic reflex actions, handled proactively and reported in standardized format, rather than requiring user permission. Changes ensure consistent behavior across the platform.

… BRO-1288)

Propagates the "documentation is a reflex, not a request — and never a question"
rule into the canonical autonomous skill. The row was added to the workspace's
gitignored .agents/ snapshot during BRO-1288 but the canonical source (what
`npx skills add broomva/autonomous` installs) lacked it.

Forbids: "I'll ask the user whether to file this into the knowledge graph."
File proactively, report after; user vetoes after, never gates before.

Content identical to the row already cross-reviewed (P20, 8/10) and merged in
BRO-1288 across broomva/bookkeeping, broomva/bstack (v0.23.1), broomva/workspace.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@linear
Copy link
Copy Markdown

linear Bot commented Jun 1, 2026

BRO-1292

BRO-1288

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Jun 1, 2026

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: e08d87de-4275-4390-aeb8-6a5bca5a576c

📥 Commits

Reviewing files that changed from the base of the PR and between 91fd104 and d40bfd2.

📒 Files selected for processing (2)
  • CHANGELOG.md
  • SKILL.md

📝 Walkthrough

Walkthrough

This PR adds a new "permission-to-document" reflex policy across two documentation files: a changelog entry describing the P6 reflex tightening (BRO-1288), and a corresponding anti-rationalization rule in SKILL.md specifying that documentation filing is proactive, unrequested, and reported in a single line.

Changes

Permission-to-document reflex policy

Layer / File(s) Summary
Permission-to-document anti-rationalization policy
CHANGELOG.md, SKILL.md
Defines the "permission-to-document" P6 reflex policy: documentation and knowledge-graph updates must be filed proactively without asking permission, then reported in a single line. CHANGELOG documents the policy change under Unreleased/Added; SKILL.md adds the corresponding anti-rationalization rule.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

  • broomva/autonomous#1: Both PRs modify SKILL.md's anti-rationalization/reflex policy table—this PR adds the "permission-to-document" reflex row, while the related PR expands the same anti-rationalization battery and tightens the surrounding discipline model.

Poem

🐰 A rabbit hops forth with a scroll in its paws,
"No asking, just file! That's the reflex we cause,"
Documentation blooms without hesitation or pause,
Then report it one line—blessed anti-rationalize clause!

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'docs: permission-to-document anti-rationalization row (P6 reflex, BRO-1288)' clearly and specifically summarizes the main change: adding a documentation anti-rationalization row. It matches the changeset which adds this row to both CHANGELOG.md and SKILL.md.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ 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 feature/bro-1292-proactive-doc-reflex

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@broomva broomva merged commit 33fc625 into main Jun 1, 2026
2 checks passed
@broomva broomva deleted the feature/bro-1292-proactive-doc-reflex branch June 1, 2026 19:40
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