Skip to content

Create odin2.json - #461

Merged
sd88me merged 4 commits into
sd88me:mainfrom
3dthc:patch-9
Oct 15, 2026
Merged

sd88me merged 4 commits into
sd88me:mainfrom
3dthc:patch-9

Conversation

@3dthc

@3dthc 3dthc commented Oct 15, 2026

Copy link
Copy Markdown
Contributor

What and why

Checks

  • Offline first: tools/test_port.sh <vst.json> (x86, ASan) and/or python3 tools/test_catalog.py; skin changes previewed offline
  • Device-verified, or marked "offline only" here and in NOTES (device, MPC OS version, date)
  • Catalog-conformant if it is a port or release (catalog_check.py --catalog OK)
  • New or changed catalog entry or skin: no manufacturer names or logos in the skin art, screenshot or plugin name (descriptive "X-style" in summary/tags is fine)
  • New catalog entry: it starts as Listed (or Experimental if untested bulk work); add a tested.json entry in the plugin repo, with device and MPC OS version, to be Verified
  • Docs synced (map in .claude/skills/mpc-vst-plugin/SKILL.md, "Docs sync"): NOTES / PORTING / RELEASING / README / ROADMAP / SKILL / docstrings, or "no docs needed: "
  • Repo stays device-generic (no IPs, serials, Akai stock skin files or Steinberg SDK)

sd88me commented Oct 15, 2026

Copy link
Copy Markdown
Owner

Entry is in the right place (catalog/plugins/odin2.json). Before merging:

  • The summary is over 350 characters and contains a non-ASCII apostrophe (Odin’s). Shorten it to 350 characters or fewer and use a plain '.
  • It says "ported to the Akai MPC Live II". If that is the only device tested, say so as tested-on, so it doesn't read as device support.
  • Duplicate of Create odin2.json #454; please keep one of the two and close the other.

Generated by Claude Code

@sd88me sd88me mentioned this pull request Oct 15, 2026
7 tasks
@sd88me
sd88me merged commit 12b8daa into sd88me:main Oct 15, 2026
2 checks passed
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