/dev:technical-plan 2.4b hardcodes proposed_action: defer-to-issue for PREQ out-of-scope findings, contradicting its own rule four paragraphs below #452
Labels
No labels
component:adapters
component:lifecycle
component:qa
component:setup
component:shared
component:worktrees
phase/accepted
phase/backlog
phase/deployed
phase/developing
phase/integrating
phase/planning
phase/qa
phase/requirements
phase/uat
priority:critical
priority:critical
priority:high
priority:high
priority:low
priority:low
priority:medium
priority:medium
type:bug
type:chore
type:docs
type:feature
type:infra
type:tech-debt
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
jbr870/devwork-skills#452
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Observation
/dev:technical-plan§2.4b builds two finding lists, and gives them contradictory rules forproposed_action:Expert-review deferred findings — the skill states the rule explicitly:
PREQ out-of-scope findings — the skill hardcodes the opposite, as a literal template with no
judgement step:
§2.7 then mirrors
proposed_actioninto each disposition decision's--recommended, so everyout-of-scope item arrives at
/dev:resolverecommending "spawn a sibling issue" — including itemsthat are standing prohibitions or statements that no work exists, which can never be actioned
and can never be closed as done.
The
target: nullcomment ("lets the orchestrator pick") is not a mitigation: the orchestrator picksbetween
defer-to-issueandaccept, and the skill has already told them which one it recommends.Evidence
/dev:technical-plan --issue=45, 2026-08-26. The PREQ's Out of Scope section had 5 items; all 5 wereemitted with
proposed_action: "defer-to-issue"per the template, and all 5 disposition decisions(
D-PO-45-2-1..-5) carried--recommended defer-to-issue.The operator then resolved all decisions as recommended. Outcome:
#419 and #421 were closed the same day as not-actionable-by-construction (see their closing
notes). Two tickets were created and closed within minutes, on a shared tracker that parallel
sessions triage.
Note the shape of the two that were wrong: both are boundary statements of the form "X remains
forbidden" / "no work is owed". A PREQ's Out of Scope section is mostly that kind of item —
it is where a requirements author records what was ruled out — so the template's default is
mis-aimed at the common case, not the rare one.
Recurrence
1 measured instance, but structural rather than incidental, on two independent grounds:
/dev:technical-planrun whose PREQ has anon-empty Out of Scope section — which is most of them — so the rate is a property of the skill
text, not of this feature.
step. The expert-review list's "MUST match the actual disposition / do NOT default everything to
defer-to-issue" paragraph exists because defaulting was observed to be wrong there. The fix wasnever carried across to the out-of-scope list four paragraphs above it.
Ground 2 is also an instance of #257 ("a fix lands where the finding pointed, not everywhere the
rule lives").
Hypothesis
Give the out-of-scope list the judgement step the expert-review list already has, rather than a fixed
proposed_action. Roughly: derive it from whether the item names work someone could do.defer-to-issue.explicitly accepted limit →
accept. The finding still exists and still pairs with a dispositiondecision, so nothing is lost from the record — only the spurious ticket is.
The strict finding↔decision pairing (contract v1, Bend 1) is unaffected: every out-of-scope item
still emits a finding and still emits a paired
scope-dispositiondecision. This changes only whichoption the skill recommends.
Worth checking in the same pass whether the two buckets should share one recommender rather than two
prose rules that can drift apart again.
Suggested lane
Meta lane /
/meta-amend. This is evidence-driven amendment of existing skill text: a boundededit to
/dev:technical-plan§2.4b, verifiable at HEAD, with the run evidence above as therequirement. No PREQ/SREQ/QA. Desk-check against this run's five out-of-scope items — the amended
text should recommend
acceptfor the #419 and #421 shapes anddefer-to-issuefor the #36/#420shapes.