Skip to content

升级 Touch Control 到 Calcit 0.41.1 - #53

Merged
tiye merged 1 commit into
mainfrom
migrate/touch-control-calcit-0411
Nov 12, 2026
Merged

tiye merged 1 commit into
mainfrom
migrate/touch-control-calcit-0411

Conversation

@tiye

@tiye tiye commented Nov 11, 2026 •

Copy link
Copy Markdown
Contributor

用户影响

触控和方向盘回调沿用当前用法。开发环境请使用配对的 Calcit CLI 与 JS 运行时 0.41.1,并重新生成 JS。

修改与风险

同步工具链和锁文件,按编译器规范简化九处类型引用。主要风险在重新生成的 JS 与运行时配对。

验证

工具链、不可变安装、全部定义、完整 strict、动态方法和弃用检查、原坐标单测及 Vite 构建通过。

Summary by CodeRabbit

  • Documentation
    • Updated the documented Calcit version for the Touch Control 0.0.40 candidate to 0.41.1.
    • Clarified that movement fields use fixed-length two-number tuples in type declarations, while runtime values remain two-number lists. Direct construction must meet the tuple length requirement.
    • The note about projects using Calcit 0.37.0 remaining on Touch Control 0.0.39 is unchanged.

@coderabbitai

coderabbitai Bot commented Nov 11, 2026 •

Copy link
Copy Markdown

Explain Change →

No actionable comments were generated in the recent review.

Walkthrough

The project updates its Calcit and @calcit/procs versions from 0.41.0 to 0.41.1. Pointer-event type hints now use the browser alias. The README compatibility note reflects the updated version.

Changes: Calcit toolchain update

Layer / File(s) Summary
Update toolchain and event type hints
deps.cirru, package.json, calcit.cirru, README.md
The Calcit and @calcit/procs versions change to 0.41.1. Event callback type hints use the browser alias. The README compatibility note reflects the updated version.

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

Change: Other

Possibly related PRs

  • Triadica/touch-control#51: PR #51 changes movement fields and &c- to Tuple Number Number. PR #53 documents the same tuple contract.
  • Triadica/touch-control#52: PR #52 upgrades the project to Calcit 0.41.0 and changes host type references to browser aliases. PR #53 updates the toolchain to 0.41.1 and applies the alias to pointer-event type hints.
Merge Risk: Minimal · up to 1322f

This is a toolchain version bump with type hint alias simplifications and no behavior change. No merge-blocking risk was identified.

Pre-merge checks: Passed 5

Check name Status Explanation
Description Check Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check Passed The title clearly and concisely describes the main change: upgrading Touch Control to Calcit 0.41.1.
Docstring Coverage Passed No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentat…
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.
Full details: Docstring Coverage

Explanation

No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentation-only edits and deletions within surviving functions are included; unchanged diff context and fully deleted functions are excluded. Analyzed 0 functions across 0 files. (4 skipped: 4 unsupported.)

Finishing Touches

Generate unit tests (beta)

  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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

Recent review info

Run configuration

  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 6d03f0b6-55ea-486b-af84-7d9de659b127
Commits

Reviewing files that changed from the base of the PR and between 7ef34dc and 1322f0f.

Files ignored due to path filters: 1

  • yarn.lock is excluded by !**/yarn.lock, !**/*.lock
Files selected for processing: 4

  • README.md
  • calcit.cirru
  • deps.cirru
  • package.json

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

@tiye
tiye merged commit 7f0d5c8 into main Nov 12, 2026
2 checks passed
@tiye
tiye deleted the migrate/touch-control-calcit-0411 branch November 12, 2026 06:22
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