Skip to content

[codex] Fix planning closeout summary routing#1223

Draft
rickardvh wants to merge 1 commit into
masterfrom
codex/1218-closeout-summary-command
Draft

[codex] Fix planning closeout summary routing#1223
rickardvh wants to merge 1 commit into
masterfrom
codex/1218-closeout-summary-command

Conversation

@rickardvh
Copy link
Copy Markdown
Owner

Summary

  • Reuse the workspace command rewriter for planning front-door output instead of blanket-mapping every agentic-planning command under agentic-workspace planning.
  • Add a regression that closes a plan through agentic-workspace planning closeout and verifies the next safe action points to top-level agentic-workspace summary.

Closes #1218.

Validation

  • uv run pytest tests/test_workspace_planning_help_cli.py::test_planning_front_door_rewrites_closeout_summary_action_to_top_level -q
  • uv run pytest tests/test_workspace_planning_help_cli.py -q
  • uv run agentic-workspace implement --changed src/agentic_workspace/workspace_runtime_primitives.py tests/test_workspace_planning_help_cli.py --task "Implement GitHub #1218: closeout suggests invalid planning summary command" --format json
  • make test-workspace
  • make lint-workspace
  • uv run agentic-workspace proof --verbose --changed src/agentic_workspace/workspace_runtime_primitives.py tests/test_workspace_planning_help_cli.py --format json
  • Dogfooding checks: uv run agentic-workspace report --target . --section workflow_obligations --format json, uv run agentic-workspace report --target . --section closeout_trust --format json, uv run agentic-workspace reconcile --format json, uv run agentic-workspace memory promotion-report --target . --mode remediation --format json

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.

Closeout suggests invalid planning summary command

1 participant