Skip to content

fix: resolve state-backed strategy work items - #692

Open
ApexWorm wants to merge 1 commit into
peteromallet:mainfrom
ApexWorm:fix/resolve-synthetic-strategy-state
Open

fix: resolve state-backed strategy work items#692
ApexWorm wants to merge 1 commit into
peteromallet:mainfrom
ApexWorm:fix/resolve-synthetic-strategy-state

Conversation

@ApexWorm

@ApexWorm ApexWorm commented Aug 1, 2026

Copy link
Copy Markdown

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 prints Resolved, but does not resolve the backing state work item, so next can 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.py
  • ruff check desloppify/app/commands/plan/override/resolve_helpers.py desloppify/tests/commands/plan/test_plan_overrides_direct.py
  • git diff --check

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