principle-subagent-inherits-scope: a brief carries decisions, not just facts and a question - #788
Conversation
|
This pull request is part of a Mergify stack:
|
83d83f9 to
843cbad
Compare
Revision history
|
843cbad to
350ee29
Compare
fab6704 to
1937775
Compare
3ed5352 to
1fb69a0
Compare
350ee29 to
1984ddf
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_7e2c1b74-2ce0-46c4-9194-fd92803fec92) |
1fb69a0 to
a8e7f75
Compare
1984ddf to
de107a7
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_57e573f5-63a4-489d-8b8a-def8e0084b09) |
de107a7 to
d7173e7
Compare
a8e7f75 to
2e69d29
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_22c5d3f9-d56a-4781-aec3-4e1d9c772d57) |
|
Queued — the merge queue status continues in this comment ↓. |
|
@Mergifyio queue |
Merge Queue Status🛑 Queue command has been cancelled |
…t facts and a question The parent's side of the skill offered two slots: facts and the question. A decision the user already made is neither, so it gets filed under the question -- and the "don't prime the answer" rule then rewards re-opening it, because that reads as rigour. Third slot added: decisions travel as constraint sentences marked settled. "Don't prime" is scoped to findings, never decisions; leaving a decision out is dropping a constraint, not staying neutral. The skill says plainly that repeating the user's words is not enough: a delegation can contain the requirement verbatim, pass a containment check, and still lose it by appending one open question beside it. No mechanical catch is claimed, and the fires fixture records why -- the containment check returns PASS on the exact call that drifted. Prior art and its caveats are in the skill: Gotel & Finkelstein on pre-requirements-specification traceability, marked single-source because Crossref's issued field is null and its author rendering differs, with no publisher page read. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Change-Id: I370384357ee26e01c4b4a2d8ad1d65808c496ab3
d7173e7 to
5fe7b3c
Compare
|
@Mergifyio queue |
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_3a54a118-c79c-4eeb-a702-06ade81d917e) |
☑️ Command
|
|
@Mergifyio queue |
Merge Queue Status
This pull request spent 18 minutes 27 seconds in the queue, including 17 minutes 43 seconds running CI. Required conditions to merge
|
Summary
When handing work to a helper, the guidance names two things to pass on: the facts, and the question.
A decision the person already made is neither, so it gets written as a question again. The rule against leading the helper then rewards re-opening it, because that looks like fairness.
In one measured case the requirement was passed on word for word and still got re-litigated; the helper ranked it fourth of six and argued the premise away.
This adds a third thing to pass on: decisions, stated as constraints.
Review Claim
Guidance for handing off work names decisions as their own category, not to be re-opened.
Review Lane
behavior
Review Unit
corpus-lesson
Safety Invariant
The rule against leading a helper still governs findings; it is narrowed so it no longer applies to choices the person has already made. Nothing becomes un-askable that was open.
Slice Rationale
Prose plus its fires and stays-silent examples. No code changes, so it carries no runtime risk and can land independently of the rest of this stack.
Non-goals
Does not add a check that inspects handoff text before it is sent. The obvious mechanical version was tested and does not work: the requirement was present word for word in the case that went wrong.
Test Plan
Test Plan
Run at this commit:
The two example files are the fires and stays-silent pair for the guidance. No detector is claimed here, because the measurable one was shown not to fire on the real case.
Revert Plan
Revert Plan
Revert this commit. The guidance returns to two categories. No behaviour changes either way.
Note
Low Risk
Corpus guidance and example prose only; no executable behavior or enforcement logic changes.
Overview
Extends
principle-subagent-inherits-scopeso parent handoffs must pass three items—facts, the question, and settled user decisions—not just facts plus a question.The skill now tells parents to phrase decisions as explicit constraints (not open questions), clarifies that anti-priming applies to findings only (omitting a decision is not neutral), and adds requirements-traceability prior art.
fires_example.mdadds a third drift case where verbatim requirements still get re-litigated when filed under the question, with an explicit note that mechanical containment checks do not catch it.stays_silent_example.mdadds the matching good handoff where the decision travels as a non-reopenable constraint.Documentation and scenario examples only; no runtime or detector changes.
Reviewed by Cursor Bugbot for commit 5fe7b3c. Bugbot is set up for automated code reviews on this repo. Configure here.