Skip to content

fix: upgrade Calcit messaging runtime - #33

Merged
tiye merged 4 commits into
mainfrom
fix/calcit-alerts-0.10.26
Sep 1, 2026
Merged

fix: upgrade Calcit messaging runtime#33
tiye merged 4 commits into
mainfrom
fix/calcit-alerts-0.10.26

Conversation

@tiye

@tiye tiye commented Sep 1, 2026

Copy link
Copy Markdown
Member

Summary

  • upgrade Calcit/@calcit/procs and the Respo dependency chain, including alerts.calcit 0.10.26
  • replace nullable chat text/thinking state with concrete strings and normalize legacy persisted state
  • isolate GenAI stream decoding behind a typed StreamChunk adapter and guard non-string FFI values
  • preserve Reply submission with the released alerts callback behavior
  • replace unsupported native prompt() API-key collection with an alerts prompt, validate blank keys, and map each provider model to its storage key
  • route the existing Claude Sonnet 4.5 menu item through OpenRouter instead of the Gemini fallback

Validation

  • caps --strict
  • yarn install --immutable
  • calcit edit format
  • calcit --check-only
  • calcit test (4 passed)
  • calcit analyze quality --baseline config/calcit-quality.json
    • codeNil: 41 -> 21
    • unresolved: 106 -> 85
    • schemaDynamic: 65 -> 64
  • calcit js
  • yarn vite build --base=./
  • browser smoke: no initial undefined; Reply opens the API-key prompt when the provider key is absent; blank Save shows validation; no new native-prompt error is appended

Copilot AI lite review requested due to automatic review settings September 1, 2026 09:37

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟡 Changes recommended

The new GenAI chunk decoder still performs unconditional nested property/index reads that can throw on partial chunk shapes, and the legacy :done? default in normalization can mis-render the UI as “Streaming...” on load.

Once you've addressed the issues Copilot identified, you can request another Copilot review.

Pull request overview

Upgrades the Calcit messaging/runtime and Respo dependency chain, while tightening the chat state and streaming pipeline by replacing nullable fields with concrete strings and centralizing GenAI stream chunk decoding behind a typed adapter.

Changes:

  • Bump @calcit/procs to 0.13.72 and update Calcit/Respo Calcits in deps.cirru.
  • Refactor streaming handling: introduce StreamChunk + decode-genai-chunk, and enforce string-only streaming text via stream-text.
  • Normalize legacy persisted ChatState into the new non-nullable schema (answer/thinking as strings; model as tag) and update UI call sites accordingly.
File summaries
File Description
yarn.lock Locks the upgraded @calcit/procs version.
package.json Updates the direct @calcit/procs dependency to match the runtime upgrade.
deps.cirru Updates Calcit + Respo dependency versions used by the Calcit build.
calcit.cirru Implements new typed streaming decode + chat-state normalization and updates schema/call sites to remove nullable text fields.
Review details
  • Files reviewed: 3/4 changed files
  • Comments generated: 2
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread calcit.cirru
Comment thread calcit.cirru
@tiye
tiye merged commit 91b5794 into main Sep 1, 2026
1 check passed
@tiye
tiye deleted the fix/calcit-alerts-0.10.26 branch September 1, 2026 10:28
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.

2 participants