Skip to content

fix(music-data): name the mismatch when an engine package is rejected - #23

Merged
nichinichisou0609 merged 1 commit into
mainfrom
fix/replay-package-errors
Oct 4, 2026
Merged

nichinichisou0609 merged 1 commit into
mainfrom
fix/replay-package-errors

Conversation

@nichinichisou0609

Copy link
Copy Markdown
Collaborator

--replay-engine and --recommend-engine check a package's build.json against the pinned deck model. Both a wrong format and a wrong commit used to raise build.json commit differs from the pinned deck model. Now:

  • a wrong format: recommend engine: build.json format is 'ournotes.replay-engine/1', not ournotes.recommend-engine/1
  • a wrong commit: replay engine: build.json commit '<commit>' differs from the pinned deck model <pinned commit>
  • a missing model identity is reported before the package files are read

Tests: tests/test_replaydata.py covers each message.

🤖 Generated with Claude Code

A replay or recommendation engine package whose build.json has another format now says so, with the format it has and the one expected, instead of reporting a commit difference. A commit difference names both commits. A missing deck model identity is reported before the package files are read.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@nichinichisou0609
nichinichisou0609 force-pushed the fix/replay-package-errors branch from e05937a to d98a850 Compare October 4, 2026 16:21
@nichinichisou0609
nichinichisou0609 merged commit e7fdacd into main Oct 4, 2026
6 checks passed
@nichinichisou0609
nichinichisou0609 deleted the fix/replay-package-errors branch October 4, 2026 16:55
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