Skip to content

Docs: the DSP hooks guide says the engine header carries the price - #1520

Merged
repeat98 merged 3 commits into
repeat98:mainfrom
twwwy:docs/engine-price-hooks-doc
Nov 13, 2026
Merged

repeat98 merged 3 commits into
repeat98:mainfrom
twwwy:docs/engine-price-hooks-doc

Conversation

@twwwy

@twwwy twwwy commented Nov 13, 2026 •

Copy link
Copy Markdown
Contributor

Docs: follow-up to #1487 (merged in #1502), item 1 of the review there.

  • What was wrong: docs/OCTATRACK_PACKAGE_DSP_HOOKS.md (Trust, lines 74-76) still said the price (411 a voice, 3 a core) stays in the machine record and is a controller change until Phase 2.
  • What this changes: a new Price bullet: the engine header's cycles and kind carry the price per voice, per-core cost folded in (Analog BD 414, executed), outside the controller digest, so a new price is a new package under the same trust row; the record is 0/0; what the loader, abi9.engine(), the site and machine_registry.c refuse; pointer to the loader README's "Price". Changelog entry.
  • How I checked it: read against main: Analog BD 0.1.7's header (414, kind 1) and its machine-ports.json record (0/0, inEngine). Docs only, no code.
  • What the guides lacked or got wrong, and where I fixed it: the hooks guide still named the machine record as the price's home; fixed in docs/OCTATRACK_PACKAGE_DSP_HOOKS.md, as a pointer to the loader README's "Price" so the guide stays under its size limit.

🤖 Generated with Claude Code

repeat98's review of repeat98#1487: docs/OCTATRACK_PACKAGE_DSP_HOOKS.md still said
the price (411 a voice, 3 a core) stays in the machine record until Phase 2.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@github-actions github-actions Bot added the documentation Improvements or additions to documentation label Nov 13, 2026
@repeat98
repeat98 merged commit a086523 into repeat98:main Nov 13, 2026
13 of 14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants