Skip to content

chore(vst-host): delete monorepo package, repoint at published PyPI package - #2604

Closed
vanceingalls wants to merge 1 commit into
vo-carve-fallow-cleanupfrom
vo-carve-repo-split
Closed

chore(vst-host): delete monorepo package, repoint at published PyPI package#2604
vanceingalls wants to merge 1 commit into
vo-carve-fallow-cleanupfrom
vo-carve-repo-split

Conversation

@vanceingalls

@vanceingalls vanceingalls commented Jul 17, 2026

Copy link
Copy Markdown
Collaborator

What

Deletes packages/vst-host from the monorepo — it now lives as a standalone GPL-3.0 repo (heygen-com/hyperframes-vst-host), published to PyPI as hyperframes-vst-host (uv tool install hyperframes-vst-host puts hyperframes-vst on PATH). Arm's-length IPC (spawn/socket) keeps the GPL/Apache license boundary intact either way.

Simplifies resolveVstHostCommand in both vstBounce.ts (engine) and vstSidecar.ts (studio-server) from a 3-step precedence (env override → monorepo uv run --project → bare PATH) down to 2 steps (env override → bare PATH), since the monorepo path can never resolve again post-deletion. Updates vstRenderParity.test.ts's skip condition to probe for the installed hyperframes-vst CLI instead of a local packages/vst-host checkout, so the integration test keeps running wherever the package is installed rather than silently skipping forever. Repoints doc-comments that pointed at packages/vst-host source paths to the new repo.

Why

Finishes the repo split: the sidecar's GPL dependency (pedalboard) and its own release/versioning cadence don't belong bundled in this Apache-2.0 monorepo.

Test plan

  • fallow audit passes clean
  • Full monorepo build succeeds
  • Full test suite passes across studio/engine/cli/studio-server/producer (vstRenderParity.test.ts correctly skips without the sidecar on PATH, and was separately verified to pass for real against the installed PyPI package)

🤖 Generated with Claude Code

…ackage

Deletes packages/vst-host — it now lives as a standalone GPL-3.0 repo
(heygen-com/hyperframes-vst-host), published to PyPI as
hyperframes-vst-host (uv tool install hyperframes-vst-host puts
`hyperframes-vst` on PATH). Arm's-length IPC (spawn/socket) keeps the
GPL/Apache license boundary intact.

Simplifies resolveVstHostCommand in both vstBounce.ts and vstSidecar.ts
(duplicated between engine and studio-server) from a 3-step precedence
(env override -> monorepo uv run --project -> bare PATH) down to 2 steps
(env override -> bare PATH), since the monorepo path can never resolve
again. Updates vstRenderParity.test.ts's skip condition to probe for the
installed hyperframes-vst CLI instead of a local packages/vst-host
checkout, so the integration test still runs wherever the package is
installed rather than silently skipping forever. Repoints doc-comments
in types.ts/vstHostWire.ts/useVstPreviewHelpers.ts that pointed at
packages/vst-host source paths to the new repo.

Verified: fallow audit passes clean, full monorepo build succeeds, and
the full test suite passes across studio/engine/cli/studio-server/producer
(vstRenderParity.test.ts correctly skips without the sidecar on PATH).

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>

Copy link
Copy Markdown
Collaborator Author

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

This stack of pull requests is managed by Graphite. Learn more about stacking.

@vanceingalls

Copy link
Copy Markdown
Collaborator Author

Superseded by a recut stack: #2919#2920#2921#2922#2923.

Same net result (68 files, +8,632/-491) with the add-then-delete churn removed and rebased onto current main (this stack's merge-base was 519 commits behind). Leaving these open until the new stack is reviewed.

Per-PR deletions now: 122 (a net-negative refactor), 1, 297, 15, 54 — all real edits against main, none of it code an earlier PR in the stack had added.

@vanceingalls

Copy link
Copy Markdown
Collaborator Author

Closing in favor of the recut stack: #2919#2920#2921#2922#2923.

Same net result (68 files, +8,632/-491), rebased onto current main, with the add-then-delete churn removed — packages/vst-host/ is never added and then deleted, files arrive pre-split, and the feature flag lands with the first studio commit instead of trailing the stack.

Branches are left in place; nothing is deleted.

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