Repository navigation
Conversation
…y work items Source: peteromallet#692
|
Independently reproduced and verified this fix against persisted CLI fixtures (related reports: #747 and #773). The fixtures create real strategy work items with the strategist's production helper, save state/plan to a temporary project, invoke
Results: nine CLI cases before = 8 failed / 1 passed; this PR alone = 8 passed / 1 unchanged cluster-guard failure; combined queue fix = 9 passed. Existing nearby resolve/workflow/strategist/queue tests: 84 passed. Full combined upstream suite: 6829 passed, 18 skipped, 2 failed; both failures are the duplicated historical No application sources, original project scanner state, or installed CLI were changed during verification. The global synthetic-prefix semantics remain untouched; the correction is local to resolution routing. |
Problem
desloppify plan resolve strategy::<id>classifies the strategy ID as a plan-only synthetic item. The workflow resolver removes it from the plan and printsResolved, but does not resolve the backing state work item, sonextcan enqueue it again.Fix
Route
strategy::items through the normal state-backed resolution path. Workflow, triage, and subjective synthetic items retain their existing plan-only handling.Verification
python3 -m pytest -q -p no:cacheprovider desloppify/tests/commands/plan/test_plan_overrides_direct.py desloppify/tests/commands/plan/test_workflow_gates.py desloppify/tests/commands/plan/test_strategist.py desloppify/tests/commands/test_queue_order_guard.py desloppify/tests/commands/resolve/test_cmd_resolve.pyruff check desloppify/app/commands/plan/override/resolve_helpers.py desloppify/tests/commands/plan/test_plan_overrides_direct.pygit diff --check