Skip to content

Migrate Calcit callbacks and validate canonical frontend CDN builds - #30

Merged
tiye merged 4 commits into
mainfrom
codex/guard-retired-snapshots
Oct 1, 2026
Merged

tiye merged 4 commits into
mainfrom
codex/guard-retired-snapshots

Conversation

@tiye

@tiye tiye commented Sep 30, 2026 •

Copy link
Copy Markdown
Member

Summary

  • Keep calcit.cirru and deps.cirru as canonical Calcit 0.27.0 project files; ignore and reject compact.cirru/package.cirru in CI.
  • Give each PR its own COS preview prefix and concurrency group; fail shell validation immediately.
  • Preserve the canonical Snapshot, production COS prefix and server rsync paths. Update Reel and Respo UI to published compatible versions and remove unused Markdown.

Validation

  • Calcit 0.27.0 check-only, strict workflow verification, codegen and README snippet validation passed.
  • Eight Node tests cover generated JavaScript behavior and the built HTML asset URLs/files; Vite CDN build passed.
  • No production deployment or strict Caps conflict resolution is claimed.

@tiye tiye changed the title Prevent retired Calcit snapshots from returning Migrate Calcit callbacks and validate canonical frontend CDN builds Sep 30, 2026
@tiye

tiye commented Sep 30, 2026

Copy link
Copy Markdown
Member Author

Extended the migration with real runtime coverage: both input handlers now accept the actual two Respo callback arguments, read nominal RespoEvent fields and dispatch one Enum. The tests render the actual component, invoke real handlers, run the real updater and rerender input values (no dependency-export mocks). Removed only unused Markdown; retained core matching/visual rendering and the used typed browser adapters. Updated published Reel/UI versions, declared the browser entry target, and added a 20/20 all-local-public gate.

Normal yarn build explicitly generates JS and honors VITE_BASE_URL; CI checks actual HTML JS/CSS prefixes and preserves original shared font/icon URLs. Local Calcit 0.27.0 check-only/strict workflow, 2 native tests, 20 public definitions, immutable install, build and 9 Node tests passed. Wrong-prefix negative check also passed. Existing .DS_Store changes were preserved and excluded.

Strict Caps still reports the direct js-ffi alpha.10 versus Respo alpha.4 request conflict; no dependency downgrade or strict success claim was made. Original storage key and server rsync paths remain unchanged. Latest Actions verification pending.

@tiye

tiye commented Sep 30, 2026

Copy link
Copy Markdown
Member Author

Latest Actions run 36788904689 passed at commit 9c3013a. COS Action public verification confirmed 3 actual files: HTML 702 bytes, CSS 451 bytes, JS 376199 bytes. All validation, runtime/CDN tests and frontend build steps passed; server production deployment was correctly skipped for this unmerged PR.

Copilot AI balanced review requested due to automatic review settings October 1, 2026 02:50

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.

Copilot review overview

🟡 Changes recommended

Shared workflow concurrency can cancel unrelated PR checks, and the dependency changes conflict with the stated preservation scope.

Review effort: Balanced
Findings: 2 Medium severity · 2 Low severity

Open (4)
What changed in this PR

Migrates Calcit callbacks and strengthens canonical CDN build validation.

Changes:

  • Updates Calcit configuration, dependencies, and callback tests.
  • Adds Vite CDN-base configuration and build scripts.
  • Tightens CI validation and serializes COS uploads.
File Description
.github/​workflows/​upload.yaml Expands validation, build, and upload handling.
.gitignore Ignores legacy Calcit files.
README.md Documents validation and CDN builds.
calcit.cirru Migrates browser entry and input callbacks.
deps.cirru Updates the dependency graph.
package.json Adds development and build scripts.
scripts/​fuzzy-filter.test.mjs Adds callback/rendering regression tests.
vite.config.mjs Configures the frontend base URL.

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

Comment thread .github/workflows/upload.yaml Outdated
Comment thread scripts/fuzzy-filter.test.mjs
Comment thread README.md Outdated
Comment thread deps.cirru
@tiye
tiye merged commit 663ce79 into main Oct 1, 2026
1 check passed
@tiye
tiye deleted the codex/guard-retired-snapshots branch October 1, 2026 05:02
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