README: name the reflect enforcement setting and its four hooks - #501
Conversation
Owner
Author
|
This pull request is part of a Mergify stack:
|
11 tasks
Bugbot couldn't run - usage limit reachedBugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit. A user or team admin can review and increase usage limits in the Cursor dashboard. (requestId: serverGenReqId_240fdc7e-5508-41a6-a97e-d27365d541f6) |
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_011vy9sMw49j6PqS2NLRBYhs Change-Id: Id9143a48f8bdc165254f60e9f6016f62acde2049
EdbertChan
force-pushed
the
stack/EdbertChan/docs/reflect-enforcement-flag-readme/readme-names-reflect-enforcement-flag-four-hooks--d9143a48
branch
from
September 12, 2026 19:28
c48f2c0 to
24f54ea
Compare
EdbertChan
force-pushed
the
stack/EdbertChan/docs/reflect-enforcement-flag-readme/reflect-automate-me-enforcement-opt-behind-one--82a63478
branch
from
September 12, 2026 19:28
0c00aa6 to
710df82
Compare
Bugbot couldn't run - usage limit reachedBugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit. A user or team admin can review and increase usage limits in the Cursor dashboard. (requestId: serverGenReqId_88765792-d043-4680-b3ec-c68b45a4b8ee) |
Owner
Author
Revision history
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
This kit has a front page that lists its checkers. People read it to learn what each checker does and how to set it up.
The problem: four checkers that push you to review your work became opt-in, but the front page did not say so.
The cause: the change that added the switch was kept to code and tests. The front page is a separate kind of file.
The fix: the front page now names the switch, lists the four checkers it controls, and shows the one line that turns them on.
Review Claim
The front page tells a reader that four review-pushing checkers are off until one setting turns them on, and how to set it.
Review Lane
docs
Review Unit
docs
Safety Invariant
Documentation only: one table row gains a note and one new section is added to
README.md. No code, test, or install behavior changes.Slice Rationale
Stacked on the PR that adds
CATSTACK_REFLECT_ENFORCEMENT, because the lane rules keep root docs out of a behavior PR. It describes that PR's behavior, so it merges after it.The first draft carried this README change inside #500. The validator refused it:
Non-goals
Test Plan
Test Plan
bash scripts/run_all_tests.shon this branch's exact tree:EXIT=0, 1932 ok, 0 failed, 1 skipped (CONSUMER_REPO_ROOTnot set; unrelated)git diff --stat <tested-head> HEADis empty: this branch is the tree the full suite ran onRevert Plan
Revert Plan
git revert <merge-sha>🤖 Generated with Claude Code
https://claude.ai/code/session_011vy9sMw49j6PqS2NLRBYhs
Note
Low Risk
README-only documentation; no runtime, install, or hook behavior changes.
Overview
Documents the opt-in reflect enforcement switch on the main hooks page so readers know four
/reflect-pushing checkers stay off until they enable it.The
reflect-on-thrashrow now notes it is inactive unlessCATSTACK_REFLECT_ENFORCEMENT=1, with a pointer to the new section below.A new Reflect enforcement (opt-in) subsection lists
scope-lock,reflect-on-thrash,wrong-check-reflect, andverdict-flip-watch, shows the~/.catstack.envone-liner to turn them on, explains env file precedence, and clarifies thatfrustration-watchdogis outside this group—with a link toengine/hooks/_flags/README.md.Reviewed by Cursor Bugbot for commit 24f54ea. Bugbot is set up for automated code reviews on this repo. Configure here.