Skip to content

MCP surface: slim payloads, pinned workspaces, costs - #63

Merged
dkackman merged 8 commits into
masterfrom
field-report-mcp-surface
Sep 11, 2026
Merged

MCP surface: slim payloads, pinned workspaces, costs#63
dkackman merged 8 commits into
masterfrom
field-report-mcp-surface

Conversation

@dkackman

Copy link
Copy Markdown
Owner

Items #4, #5, #9 and #10 of the 2026-09-10 MCP field report (/Users/don/testing/10/dw-improvements.md), executed from docs/superpowers/plans/2026-09-10-mcp-field-report-followups.md (PR-2 section).

#4 Tool payloads

  • export_job no longer inlines workflow, manifest and job — they are in the zip and served individually by get_job_workflow / get_job. A music-video export was 55 KB of repeated JSON.
  • wait_for_job returns a slim job: status, warnings, error, queue_position, and the manifest once terminal — no arguments (4-6k tokens of prompt text per poll on an H3 workflow) or traceback. get_job has those.

#5 Workspace sharp edges

  • run_workflow(workspace=...) / validate_workflow(workspace=...) pin one call to a named workspace without switching the session — a restarted session forgets use_workspace, and a job then resolved output: references in the wrong root.
  • create_workspace(use=False): pass use=true to switch as well; otherwise the result says where the session still is.

#9 Catalog costs

Measured RTX 3090 cost arrays on music-video (35 min), dialogue-short (42 min), assemble-and-score and dissolve-between-shots (0.2 min).

#10 The last mile

Step 7 of the guide's agent loop: fetching outputs from a remote server with the DW_API_TOKEN bearer; pointer from docs/MCP.md.

Also: black --check was red on master from #62 (two files) — fixed here.

Full suite: 3314 passed, 5 skipped. black --check dw dw_mcp tests clean.

🤖 Generated with Claude Code

dkackman and others added 8 commits September 10, 2026 23:05
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Copy exact docstring wording from dw_mcp/server.py into docs/MCP.md for
export_job, validate_workflow, run_workflow, create_workspace and
wait_for_job; note the workspace-override exception in dw_mcp/CLAUDE.md;
fix the stale $DW_TOKEN/$DW_SERVER example in WORKFLOW_GUIDE.md; reformat
four templates' cost arrays to match the established three-line form; and
add queue_position to wait_for_job's slim keys (TDD: failing test first)
plus a get_job assertion on the terminal-wait test.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@dkackman
dkackman merged commit c5d5867 into master Sep 11, 2026
9 checks passed
@dkackman
dkackman deleted the field-report-mcp-surface branch September 11, 2026 13:34
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