Skip to content

Add CLJS animation, tracking, and speech planners#45

Open
meekmachine wants to merge 6 commits into
staging/cljs-worker-agenciesfrom
codex/cljs-hair-agency
Open

Add CLJS animation, tracking, and speech planners#45
meekmachine wants to merge 6 commits into
staging/cljs-worker-agenciesfrom
codex/cljs-hair-agency

Conversation

@meekmachine

@meekmachine meekmachine commented May 26, 2026

Copy link
Copy Markdown
Owner

Summary

  • add a CLJS Animation boundary agency that owns serializable snippet registry/schedule state and emits typed main-thread animation operations
  • add a CLJS Eye/head tracking facade over the CLJS Gaze planner, preserving stable AU pair snippets, inherited first frames, reset/stop, and pause/resume host effects without browser/webcam/direct-engine code in worker scope
  • add CLJS Prosodic, Vocal, and Lipsync planners for speech gesture state, phoneme/viseme timelines, word-boundary drift decisions, Azure/word snippet planning, neutral returns, and cleanup plans without browser/audio/engine dependencies
  • add a CLJS Hair agency that owns serializable hair color, eyebrow color, outline, part visibility/scale/position, and physics state
  • wire Animation, Eye/head tracking, Prosodic, Vocal, Lipsync, and Hair through CLJS npm exports, worker dispatch, runtime host callbacks, and declaration types
  • expand the CLJS smoke test to cover Animation, Blink, Gaze, Eye/head tracking, Prosodic, Vocal, Lipsync, and Hair

Refs #30.
Refs #33.
Refs #34.
Refs #35.
Refs #36.
Refs #40.

Validation

  • npm run build
  • npm run typecheck
  • npm test
  • npm run test:cljs

CI/CD integration note

  • package exports stay in this Latticework PR via ./cljs and ./cljs-worker
  • LoomLarge CI/deploy already resolves linked Latticework PRs from PR text/comments and can install the PR SHA without committing dependency changes
  • after release to npm, LoomLarge CI/deploy refreshes @lovelace_lol/latticework@latest in the CI checkout, so no separate app-side package wiring PR is required for package consumption

Follow-up

  • continue agency-by-agency with TTS, transcription, and conversation orchestration
  • emotionExpression still needs a real agency scaffold before conversion

Copy link
Copy Markdown
Owner Author

Coordinator queue packet:

Status checked this run:

  • Non-draft, open PR targeting staging/cljs-worker-agencies.
  • Mergeability: MERGEABLE / CLEAN.
  • Checks: none reported on codex/cljs-hair-agency via gh pr checks.
  • Review threads: none unresolved.
  • Scope: CLJS hair agency planner/runtime surface plus npm/runtime worker exports and smoke script.

Coordinator classification: needs-review. This is mechanically mergeable, but it should get explicit architecture/runtime review and either CI coverage or a documented local verification packet before being treated as ready for human merge. I did not merge the PR.

@meekmachine

Copy link
Copy Markdown
Owner Author

Reviewed the PR diff and ran local verification against an isolated export of aeb7da7325f1d41092a08878f229e6fac6efcec7 so the checkout stayed untouched.

Verification:

  • npm ci --ignore-scripts
  • npm run typecheck
  • npm test - 13 files / 148 tests passed
  • npm run test:cljs - CLJS npm + worker builds passed, smoke passed with hair coverage
  • npm run build - passed

No blocking findings from review. Residual note: GitHub reports no checks for this branch, so this local packet is the current verification evidence.

@meekmachine meekmachine changed the title Add CLJS hair agency state planner Add CLJS animation boundary and hair agency May 27, 2026
@meekmachine meekmachine changed the title Add CLJS animation boundary and hair agency Add CLJS animation, prosodic, and hair agencies May 27, 2026
@meekmachine

Copy link
Copy Markdown
Owner Author

Coordinator update: #45 has new commits after the earlier local verification packet (b1e2f1b animation agency and 1d871f7 prosodic agency). Current state is non-draft, clean/mergeable, no unresolved review threads, and still no GitHub checks reported for codex/cljs-hair-agency. Classification remains needs-review / needs refreshed verification before human merge; expected verification is npm run build, npm run typecheck, npm test, and npm run test:cljs against the current head.

@meekmachine meekmachine changed the title Add CLJS animation, prosodic, and hair agencies Add CLJS animation, vocal, prosodic, and hair agencies May 27, 2026
@meekmachine

Copy link
Copy Markdown
Owner Author

Coordinator update: #45 has another new commit after the prior verification note: c0e6c2e adds the CLJS vocal agency planner, expanding scope to Animation, Vocal, Prosodic, and Hair. Current state: non-draft, clean/mergeable, no unresolved review threads, and no GitHub checks reported. Classification remains needs-review / needs refreshed verification before human merge. Expected verification against current head: npm run build, npm run typecheck, npm test, npm run test:cljs.

@meekmachine meekmachine changed the title Add CLJS animation, vocal, prosodic, and hair agencies Add CLJS animation and speech agency planners May 27, 2026
@meekmachine meekmachine changed the title Add CLJS animation and speech agency planners Add CLJS animation, tracking, and speech planners May 27, 2026
@meekmachine

Copy link
Copy Markdown
Owner Author

Coordinator update: #45 added another commit after the previous verification note: efa253f adds the CLJS lipsync agency planner. Current scope now includes hair, animation, prosodic, vocal, and lipsync agency planners. State remains non-draft, clean/mergeable, no unresolved review threads, and no GitHub checks reported. Classification remains needs-review / needs refreshed verification before human merge. Expected verification against current head: npm run build, npm run typecheck, npm test, npm run test:cljs.

@meekmachine

Copy link
Copy Markdown
Owner Author

Coordinator update: #45 added another commit after the previous verification note: 9b4e6a5 adds the CLJS eye/head tracking facade. Current scope now includes hair, animation, prosodic, vocal, lipsync, and eye/head tracking planners/facades. State remains non-draft, clean/mergeable, no unresolved review threads, and no GitHub checks reported. Classification remains needs-review pending refreshed verification. I assigned an isolated worktree verification against current head with: npm run build, npm run typecheck, npm test, npm run test:cljs.

@meekmachine

Copy link
Copy Markdown
Owner Author

Verification complete at head 9b4e6a5fcc9d4d9d8de5830bf2dea37e439df4d3 in isolated worktree /Users/jonathan/Enero/.worktrees/latticework-pr-45-verify.

Commands:

  • npm ci passed
  • npm run build passed
  • npm run typecheck passed
  • npm test passed: 13 files / 148 tests
  • npm run test:cljs passed: CLJS smoke covered animation, gaze, eye/head, prosodic, vocal, lipsync, and hair states

Notes:

  • No source changes were made during verification.
  • npm ci reported existing audit findings: 3 vulnerabilities, 2 moderate and 1 high.
  • Java/Closure emitted a terminal deprecation warning from sun.misc.Unsafe; it did not fail the build.

Coordinator classification: ready-for-human-merge into staging/cljs-worker-agencies, with the audit findings recorded as residual risk.

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