fix: #505 duplicate MCP tools, grok-4.6 500k, TUI/worker vision, #504 GIT_DIR - #507
Open
justrach wants to merge 10 commits into
Open
fix: #505 duplicate MCP tools, grok-4.6 500k, TUI/worker vision, #504 GIT_DIR#507justrach wants to merge 10 commits into
justrach wants to merge 10 commits into
Conversation
--yolo starts codedbpro from .mcp.json, then connectCompanion starts it again. load_tool_schemas unfolded both copies and xAI/Kimi rejected the next turn: Duplicate function definition provided: mcp__codedbpro__read. Claim the pending server name during the deferred handshake, skip a second addServer, and emit each qualified name only once in the catalog.
xAI's /v1/models list is often id-only, so contextFor fell back to 200k. Bake the official 500k windows and read the context_window field when a row actually declares one.
The hosted TUI path copied @[png] as text, so grok-4.6 never got pixels. Workers seated by vision-ask had the same hole: the path was in the brief, not on the wire. Both now go through imageMessage when the file is readable.
tier-1 already unsets the hook-exported git keys, but a direct GIT_DIR=… zig build test still let fixture git init retarget this repo (core.bare=true). fixtureCmd now wraps git with env -u, and stripGitHookEnv is the reusable map helper.
Bake grok-4.5 and grok-4.6 into the generated model lists.
This was referenced Aug 15, 2026
Closed
The CI fixture still required a single tools/list. First launch overlaps discover + tools/list + at most one initialize. A well-behaved server no longer 500s the rest of the handshake (which was what produced the second initialize).
prepareInput tests now expect std.fs.path.resolve output, not a hard-coded slash. adopt builds and compares paths with std.fs.path.join / either separator, writes with read=true, and alreadyAdopted falls back to a path-based read when File.stat fails on a write-only handle.
…-*.md (#470) Outcome.describe already named the miss. The tool result and the inspect record now carry that same string; honoured pins stay quiet.
renameSession deleted the old .session.json but left g_transcript_note pointing at it. rearmAfterRename rewrites the armed line to the live name.
This was referenced Aug 15, 2026
Owner
Author
|
Also on this branch now:
|
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.
Ports the uncommitted hover-tree fixes onto
release/v0.0.258and lands the remaining small follow-ups.#505 — Duplicate
mcp__codedbpro__read--yolostarts codedbpro from.mcp.json, then companion auto-connect starts it again. Afterload_tool_schemasboth copies unfolded and the Responses wire 400'd (Duplicate function definition provided). Pending-name claim + catalog name-dedup.grok-4.6 window
Bake grok-4.5/4.6 at 500k (id-only
/v1/modelsno longer falls back to 200k). Honorcontext_windowwhen a row declares it.TUI
@[path]+ #385Hosted TUI history now stages a real
image_url. Vision-ask workers get the named local image on their first message (missing files stay text).#504
fixtureCmdwraps git withenv -u GIT_*so a hook-inheritedGIT_DIRcannot mark the checkout bare.stripGitHookEnvis the map helper.#489
Already on this branch (
bccd9cc). Not re-implemented.Hover
1003his not in this PR — already on 258 via the kitty overhaul.