Skip to content

Upgrade markup to Calcit 0.19.1 - #10

Merged
tiye merged 1 commit into
mainfrom
upgrade/calcit-0.19.1
Sep 23, 2026
Merged

tiye merged 1 commit into
mainfrom
upgrade/calcit-0.19.1

Conversation

@tiye

@tiye tiye commented Sep 23, 2026 •

Copy link
Copy Markdown
Member

Summary

  • Upgrade Markup to Calcit and @calcit/procs 0.19.1, module 0.0.4, Reel 0.6.32, Respo UI 0.7.31, and Respo core 0.16.113.
  • Use Reel's typed devtools component for the already typed app state. This resolves the 0.19 strict check warning.
  • Make the saved-store decoder's contract generic: it validates any input before returning Option<Store>. The project now has zero unresolved Dynamic schema/code occurrences and zero dynamic method findings.
  • Update the CI dynamic-method check for the current Calcit CLI. Add a Node rendering regression for the open Reel devtools, including Merge and Reset controls.

Verification

  • caps --strict --ci, caps verify --toolchain, Yarn 4.18 immutable install.
  • Strict entry check, deprecated and dynamic-method reports, and syntax-fix preview (no suggestions).
  • Node rendering regression and Calcit JS codegen plus Vite production build pass locally.

The workflow already deploys pull requests under /pr/<number>/ and retains that path.

Summary by CodeRabbit

  • Improvements
    • Development mode now renders the typed Reel component, including its content placeholder and the Merge and Reset controls. This provides a preview of the typed component’s rendered output during development. The store’s decoded result remains unchanged.

@coderabbitai

coderabbitai Bot commented Sep 23, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Advanced

Run ID: 0e7ac1bb-05fa-4419-89ae-fae50cf08a16

📥 Commits

Reviewing files that changed from the base of the PR and between 86e5ab6 and 6974e71.

⛔ Files ignored due to path filters (1)
  • yarn.lock is excluded by !**/yarn.lock, !**/*.lock
📒 Files selected for processing (5)
  • .github/workflows/upload.yaml
  • calcit.cirru
  • deps.cirru
  • package.json
  • scripts/typed-reel-render.test.mjs

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.


Walkthrough

Development-mode container rendering now uses the typed Reel component. The decoder schema accepts a generic input type. A new test checks rendered content and controls, and CI runs the test after compilation. The workflow also checks dynamic-method findings. Project and tool versions are updated.

Changes

Typed Reel development rendering

Layer / File(s) Summary
Toolchain version updates
deps.cirru, package.json
Updates the Calcit, project, and Respo dependency versions.
Typed Reel wiring and decoder schema
calcit.cirru
The decoder schema uses a generic input type. The development container imports and renders comp-typed-reel instead of comp-reel.
Typed Reel render validation
scripts/typed-reel-render.test.mjs, .github/workflows/upload.yaml
Adds a test that checks the typed Reel's rendered placeholder and controls. The workflow runs the test after compilation and checks that the dynamic-method analyzer reports zero findings.

Estimated code review effort: 2 (Simple) | ~10 minutes

Merge Risk: ⚪ Minimal · up to 6974e

No concrete user-facing or build regression is established in the reviewed changes; merge readiness is minimal risk, subject to the ordinary workflow checks.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and concisely identifies the primary change: upgrading Markup to Calcit 0.19.1 and related dependency versions.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 1…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
📝 Generate docstrings
  • Commit to this branch
  • Create a new PR
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@tiye
tiye merged commit 40bfb95 into main Sep 23, 2026
1 check passed
@tiye
tiye deleted the upgrade/calcit-0.19.1 branch September 23, 2026 17:36
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