feat(frontend)add derive-animation task post - #365
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Codecov Report✅ All modified and coverable lines are covered by tests.
Flags with carried forward coverage won't be shown. Click here to find out more.
... and 1 file with indirect coverage changes 🚀 New features to boost your workflow:
|
|
@cqhasy enhance test cove |
There was a problem hiding this comment.
Reviewed the complete fixed diff for the derive-animation flow, including direction-group filtering, request serialization, generation-run restoration, candidate merging, and editor review/apply behavior. The frontend request shape matches the Core derivation contract, and the changed paths are covered by the added tests.
Verification: repository diff checks passed. Automated frontend tests could not be rerun in this environment because the available Node.js is v20.20.2 while the configured pnpm 11.5.0 requires Node.js >=22.13 and imports node:sqlite.
Change Description
derive_animationgeneration task.Implementation Approach
derive_animation.Related Issue
Part of #345
Testing
cd frontend && pnpm format:check- passed.cd frontend && pnpm lint- passed with one existingimport(no-duplicates)warning insrc/model/asset/record/types/asset-snapshot.ts.cd frontend && pnpm test- passed: 113 test files and 600 tests.cd frontend && pnpm build- passed (tsc -b && vite build).Screenshots or Recordings
Not included.
Risks and Follow-ups
derive_animationtasks. Output quality issues in the derivation pipeline, such as non-symmetric generated directions or occasional green edges, are outside this frontend change and should be addressed separately.Checklist