Skip to content

fix(cli): print all MCP text blocks from tool results#277

Closed
ScriptedAlchemy wants to merge 3 commits into
masterfrom
codex/cli-multiblock-output
Closed

fix(cli): print all MCP text blocks from tool results#277
ScriptedAlchemy wants to merge 3 commits into
masterfrom
codex/cli-multiblock-output

Conversation

@ScriptedAlchemy

@ScriptedAlchemy ScriptedAlchemy commented Jul 4, 2026

Copy link
Copy Markdown
Owner

tracedecay tool <name> printed only content[0].text; when a handler prepends a warning block (stale-index banner, automation nag), the payload and metrics were silently discarded — stale-index projects got warnings instead of results, exit 0. Now joins every text block, blank-line separated.

Status: COMPLETE — all checks green, ready for review

Found by the 74-tool markdown-output audit (P0 finding); the identical fix was independently produced by a Codex session and a workflow agent — this lands the Codex single-commit version; the duplicate hunk in feat/plugin-suite-improvements falls away on rebase.

Recovery context

  • Orchestrating Claude session: 2c51d204-3565-4a10-833d-d8fbd51620c3tracedecay tool message_search --query 'multi-block CLI payload'
  • Durable fact Fix MCP protocol error semantics #24 (markdown audit incl. this P0) — tracedecay tool fact_store --action get --fact-id 24

🤖 Generated with Claude Code

@changeset-bot

changeset-bot Bot commented Jul 4, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 16e84c1

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, add credits to your account and enable them for code reviews in your settings.

@ScriptedAlchemy

ScriptedAlchemy commented Jul 4, 2026

Copy link
Copy Markdown
Owner Author

Full recovery manifest (all session + workflow ids)

Main orchestrating session: 2c51d204-3565-4a10-833d-d8fbd51620c3 (claude, /home/zack/projects/tracedecay) — tracedecay tool lcm_load_session --provider claude --session-id 2c51d204-3565-4a10-833d-d8fbd51620c3

Workflow runs (transcripts: session dir → subagents/workflows/<run_id>/):

Run Purpose Status
wf_785c5851-aee plugin-suite wave 1 (grep, telemetry, Claude surfaces, steering) completed; E/F/H died on session limit
wf_7f23bed7-803 triple-plugin completion (all 8 tasks + review + verify) wave 1 cached; resumed
wf_bcfa3aa1-ac9 / wf_2ede3e1d-75a / wf_d002080a-6e2 auto-sync D1–D7 (attempts 1/2/3) 3rd attempt active
wf_d0bf6fa4-48f 36-scenario eval corpus (sonnet+codex) done → fact 28
wf_d0ae2099-bfe obscure-tool evals + missed-opportunity hunt done → fact 33
wf_f46d3a1c-ccb session-git-correlation build (this feature) active

Research agents (task transcripts in the session's tasks/ dir): a24542ee4f08eee98 markdown audit→fact 24 · a143ce24edd3525f7 adoption research→fact 21 · a07f2677c93ebc313 rstest diagnosis→fact 23 · af4d3dac14e17f59c eval baseline+opus rerun→fact 25 · a3cfc66f0ca02f263 freshness design→fact 22 · ab49d9ae2d3a5f72d skill-pattern study (children af2210c6ae0c73588, af50cc51138da9bb2)→fact 29 · a546e5bb4d17b8ac6 hermetic harness

Durable decisions: facts 17–39 in this project's fact store (tracedecay tool fact_store --action search --query 'recovery manifest' → fact 39 has this table).

@ScriptedAlchemy

Copy link
Copy Markdown
Owner Author

Superseded — safe to close. The CLI multi-block fix here (join_content_text, joining all content[*].text blocks) is now in master via the merged #280 (verified: 12 matches in src/tool_command.rs on master). This PR is fully redundant. Recommend closing.

@ScriptedAlchemy

Copy link
Copy Markdown
Owner Author

Closing as redundant: the CLI multi-block fix (join_content_text) merged into master via #280 (verified: present in src/tool_command.rs on master). No unique changes remain here.

@ScriptedAlchemy ScriptedAlchemy deleted the codex/cli-multiblock-output branch July 4, 2026 20:31
@ScriptedAlchemy ScriptedAlchemy restored the codex/cli-multiblock-output branch July 4, 2026 21:54
@ScriptedAlchemy ScriptedAlchemy deleted the codex/cli-multiblock-output branch July 4, 2026 21:58
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