Feature/zoo 645 terminalregistry race condition guard#111
Merged
krzychdre merged 4 commits intoJun 17, 2026
Conversation
…#555)
Port of Zoo-Code PR #555. Registers `claude-fable-5` across the Anthropic,
Bedrock, Vertex, OpenRouter, Requesty and Vercel AI Gateway providers
(context 1M, direct maxTokens 128k / 8k on Bedrock+Vertex, adaptive-binary
reasoning, temperature unsupported), and routes the model through adaptive
thinking.
Fork-specific: our Vertex handler never received the adaptive-binary reasoning
routing that Zoo already had, so it emitted `{type:"enabled",budget_tokens}` for
Opus 4.7/4.8 and Fable 5. Recompute `thinking` via getAnthropicProviderReasoning
in AnthropicVertexHandler.getModel so Vertex matches the direct provider.
Co-authored-by: T <taltas@users.noreply.github.com>
Co-authored-by: Elliott de Launay <edelauna@gmail.com>
…ng streams (zoo #588) Replace the duplicated inline reasoning_content extraction block in the OpenAI-compatible streaming providers (deepseek, openai, qwen-code, requesty, unbound) with the shared extractReasoningFromDelta helper, which prefers delta.reasoning_content and falls back to delta.reasoning (OpenRouter style). Adds streaming coverage for both paths. Fork adaptations: the helper already existed here; mimo and opencode-go do not exist in this fork and are skipped; unbound has no spec file here so its new streaming tests are skipped (the helper has its own unit tests). Co-authored-by: dw <41457565+daewoongoh@users.noreply.github.com> Co-authored-by: Oh Daewoong <dw.oh@samsung.com>
Adds the glm-5.2 flagship model (1M context) to both Z.ai model maps with a new Max reasoning-effort tier; High is the default and Max is opt-in. Persisted reasoning effort not offered by the current model now falls back to the model default instead of silently disabling reasoning, and the streaming create() is wrapped in handleOpenAIError for parity with the base class. GLM-5.2 pricing is provisional (mirrors GLM-5.1). Adds Max translations across all 18 locales. Co-authored-by: Mob Code 100 <66469454+MobCode100@users.noreply.github.com>
…of running=true (zoo #645) A fast command's onDidEndTerminalShellExecution can arrive before setActiveStream() flips terminal.running to true. The old guard logged an error, set busy=false, and returned without signalling completion, so a TerminalProcess awaiting shell_execution_complete hung forever. Now, when running is still false but a process exists, deliver shellExecutionComplete(exitDetails) so the waiter unblocks; fall back to clearing busy only when no process exists. Also drops the redundant trailing busy=false after the normal completion path, since shellExecutionComplete already clears it. Co-authored-by: edelauna <54631123+edelauna@users.noreply.github.com>
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.
No description provided.