Default TraceDecay tool output to markdown - #262
Merged
Conversation
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
|
All MCP tool handlers now route through render::finalize, so the default output is token-efficient markdown and format:"json" returns compact JSON for programmatic consumers. Converted the JSON-only paths: message_search, all lcm_* tools, fact_store/fact_feedback, skill_list/skill_view/ automation_run_artifact_view/hermes_skill_bridge (previously pretty-printed), all edit tools, branch_list, active_project, dashboard, run_affected_tests error paths, git error results, dsm, and tracedecay_retrieve (markdown default returns stored content verbatim; never re-truncates). All converted tools joined FORMAT_CAPABLE_TOOL_NAMES, so their schemas advertise the format param and the generated Hermes bridge plugin keeps injecting format:"json" for its machine contract. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
- message_search default output is now a compact markdown list (provider, session+title, role, timestamp, score, readable snippet) instead of generic_md dumping raw metadata_json/source_path/transcript_path blobs into table cells. format:"json" still returns full structured records. - Fix regression: lcm_preflight markdown-default truncation now threads the project root so an oversized payload stores a retrieval handle instead of clipping irreversibly (matches lcm_expand_query). - Fix metrics: JSON truncation envelope reports reversible=handle.is_some() instead of hardcoded true, so failed/absent-handle truncations aren't miscounted as reversible. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Address truncation-audit findings so every markdown-default response stays bounded and recoverable: - S4: tracedecay_files and tracedecay_type_hierarchy no longer clip oversized output irreversibly. New render::truncate_text_with_handle stores the full body and returns the markdown truncation envelope with an rh_ handle (delegates to truncated_markdown_with_handle; no duplicated logic). - S3: lcm_expand_query_tool_json's format:"json"+needs_synthesis path could emit an unbounded payload when even the Minimal compaction tier overflowed. New bounded_lcm_expand_query_floor_text enforces a floor: stores a retrieval handle (surfaced as response_handle for the Hermes bridge) and keeps the contract keys (status, needs_synthesis, bounded answer/prompt, rebuilt synthesis_prompt) while zeroing unbounded arrays with *_truncated markers. - S5: LCM preflight/expand_query compaction tiers now call observe_response_truncation so those truncations show up in metrics. - Drop unused LCM_GREP_SNIPPET_CHARS (lcm_grep hits are already bounded). Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
ScriptedAlchemy
force-pushed
the
explore/markdown-default-output
branch
from
July 3, 2026 23:40
ce133df to
a705263
Compare
ScriptedAlchemy
force-pushed
the
explore/markdown-default-output
branch
from
July 3, 2026 23:47
a705263 to
9e97516
Compare
This was referenced Jul 3, 2026
Merged
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
format: "json"is requestedtracedecay_message_searchwith compact markdown rows instead of raw JSON blobsVerification
Claude Session Recall
tracedecay_message_searchproviderclaude: sessiond37694b3-62ad-470e-8121-932bf9491045createdexplore/markdown-default-outputfromorigin/masterand recorded the message-search markdown issue.tracedecay_message_searchproviderclaude: sessiona62b3a59-fe48-4cec-8b89-148f295c9050pushed84a28f8toorigin/explore/markdown-default-output.