Skip to content

test: guard CLJS mixer boundary#26

Draft
meekmachine wants to merge 1 commit into
mainfrom
codex/guard-cljs-mixer-boundary
Draft

test: guard CLJS mixer boundary#26
meekmachine wants to merge 1 commit into
mainfrom
codex/guard-cljs-mixer-boundary

Conversation

@meekmachine

@meekmachine meekmachine commented May 30, 2026

Copy link
Copy Markdown
Owner

Summary

  • replace the stale mixer migration note with the current Polyester/Loom3 mixer ownership boundary
  • add a CLJS mixer-boundary check that rejects frame-loop/runtime terms in CLJS source and verifies runtime host control effects stay exposed
  • wire the boundary check into npm run test:cljs
  • tighten the CLJS smoke test so animation schedule/update/seek/pause/resume/parameter operations are forwarded as host control effects
  • update AGENTS.md with mixer-boundary guidance plus GitHub CLI REST/Markdown quoting fallbacks

Why

Issue #16 is mostly about preventing the wrong architecture from creeping back in. Polyester CLJS should plan and emit scheduled snippets/control effects; Loom3/Three should own clip construction and AnimationMixer.update(delta). This PR makes that boundary executable in CI and removes the stale doc that still described a reverted legacy baseline.

Validation

  • git diff --check
  • node scripts/check-cljs-mixer-boundary.mjs
  • npm run typecheck
  • npm test
  • npm run test:cljs
  • npm run build

Refs #16

@meekmachine meekmachine force-pushed the codex/guard-cljs-mixer-boundary branch from 51e1bf2 to e28772f Compare May 30, 2026 00:46
@meekmachine meekmachine marked this pull request as draft May 30, 2026 01: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.

1 participant