Skip to content

chore: update package-lock.json peer flags - #60

Merged
veillette merged 2 commits into
mainfrom
claude/review-codebase-design-9CbJk
Feb 27, 2026
Merged

chore: update package-lock.json peer flags#60
veillette merged 2 commits into
mainfrom
claude/review-codebase-design-9CbJk

Conversation

@veillette

Copy link
Copy Markdown
Collaborator

Remove stale "peer": true annotations from several devDependencies in
package-lock.json, generated automatically by npm install in the
session startup hook.

https://claude.ai/code/session_01AkJKY77taZh12TtxiEJ81S

Remove stale "peer": true annotations from several devDependencies in
package-lock.json, generated automatically by npm install in the
session startup hook.

https://claude.ai/code/session_01AkJKY77taZh12TtxiEJ81S
…idate icons

SimModel scope creep (High):
- Extract VideoPlaybackModel: timing, frame rate, playback speed, video dimensions/transform
- Extract VideoSourceModel: webcam recordings, uploaded videos, active blob, isWebcamVideo flag
- Extract TrackingModel: particle tracks, kinematics cache with explicit invariant docs, OpenCV tracker facade
- SimModel becomes a thin coordinator (model.playback.*, model.sources.*, model.tracking.*)
- Re-export constants and types from SimModel for backward compatibility
- Update all 12 view files to use the new sub-model property paths

Kinematics cache implicit invariant (Medium):
- Add detailed comment in TrackingModel explaining identity-based cache invalidation:
  cached.points === track.points is valid because all mutation paths replace the
  entire Track object immutably; track IDs are never reused (A–Z monotonic sequence)

Scattered icon definitions (Medium):
- Create src/TrackLabIcons.ts with makeDownloadIcon, makeUploadIcon, makePlusIcon, makeTrashIcon
- TrackLabButton.ts re-exports from TrackLabIcons.ts (backward-compatible)
- TrackListPanel.ts imports from TrackLabIcons.ts and removes local icon definitions

https://claude.ai/code/session_01AkJKY77taZh12TtxiEJ81S
@veillette
veillette merged commit 16fe641 into main Feb 27, 2026
1 check passed
@veillette
veillette deleted the claude/review-codebase-design-9CbJk branch March 1, 2026 01:13
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