Skip to content

config phase-set + full --json error chain (rc.13, TASK-390)#324

Merged
Shooksie merged 2 commits into
release/0.7from
feat/config-phase-set-and-json-error-chain
Jul 14, 2026
Merged

config phase-set + full --json error chain (rc.13, TASK-390)#324
Shooksie merged 2 commits into
release/0.7from
feat/config-phase-set-and-json-error-chain

Conversation

@Shooksie

Copy link
Copy Markdown
Contributor

Two workflow-authoring tooling fixes (portal REQ). A: --json errors now carry the full anyhow chain ({err:#}) instead of only the top context — so workflow config workflow-set surfaces the real cause (e.g. 'references unknown phase X'). B: new animus workflow config phase-set --id --input-json writes WorkflowConfig.phase_definitions on the config_source base (not the agent-runtime overlay phases upsert uses), so a workflow referencing the phase validates — enables full custom-workflow authoring via MCP. Tests + codex self-vet clean; fmt/clippy/check green.

Shooksie added 2 commits July 13, 2026 21:38
…hase-set`

Fix A: emit_cli_error now builds the --json error.message via anyhow's
alternate Display (`{:#}`) instead of `err.to_string()`, so the message
carries the full context chain (outer .context() + every underlying source)
matching the human path. Previously only the top-level context surfaced,
dropping the real cause (e.g. the "references unknown phase" detail behind
"the config to write is invalid...").

Fix B: add `animus workflow config phase-set --id <id> --input-json <json>`,
mirroring agent-set/workflow-set. It read-modify-writes the RAW config_source
base's `WorkflowConfig.phase_definitions` (NOT the agent-runtime overlay that
`workflow phases upsert` writes), so a subsequently-set workflow that
references the phase resolves at post-pack-merge validation instead of failing
"references unknown phase". Case-insensitive upsert to match phase-id
resolution elsewhere.

Tests: json_error_message chain assertion; phase-set JSON-error handler;
phase-authored-on-base lets a referencing workflow validate (+ control).
Docs: docs/reference/cli/index.md.
@vercel

vercel Bot commented Jul 14, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
animus-docs Ready Ready Preview, Comment Jul 14, 2026 3:42am

Request Review

@Shooksie Shooksie merged commit 3a4dced into release/0.7 Jul 14, 2026
7 of 10 checks passed
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