Skip to content

fix(cli): respect registry compatibility metadata#1251

Open
kiyeonjeon21 wants to merge 1 commit into
heygen-com:mainfrom
kiyeonjeon21:kj/registry-metadata-compat
Open

fix(cli): respect registry compatibility metadata#1251
kiyeonjeon21 wants to merge 1 commit into
heygen-com:mainfrom
kiyeonjeon21:kj/registry-metadata-compat

Conversation

@kiyeonjeon21

Copy link
Copy Markdown
Contributor

What

Adds CLI-side handling for registry item compatibility metadata that is already part of the registry item schema:

  • warn when installing deprecated registry items
  • stop before writing files when an item requires a newer CLI via minCliVersion
  • surface compatibility warnings from hyperframes add and the interactive catalog install path

Why

Registry items can declare deprecated and minCliVersion, but hyperframes add currently installs them without checking those fields. That can lead to confusing installs when an item depends on newer CLI behavior or has been superseded.

How

  • Added a small compatibility helper for deprecated and minCliVersion checks
  • Run the check after registry item resolution and before target remapping / file writes
  • Keep source/dev CLI builds (0.0.0-dev) unblocked for local development
  • Left transitive registryDependencies resolution out of scope for this change

Test plan

  • Unit tests added/updated
  • Manual testing performed
  • Documentation updated (if applicable)

Commands run:

  • bun run --filter @hyperframes/cli test src/registry/compatibility.test.ts src/commands/add.test.ts
  • bun run --filter @hyperframes/cli typecheck
  • bunx oxlint packages/cli/src/registry/compatibility.ts packages/cli/src/registry/compatibility.test.ts packages/cli/src/commands/add.ts packages/cli/src/commands/add.test.ts packages/cli/src/commands/catalog.ts
  • bunx oxfmt --check packages/cli/src/registry/compatibility.ts packages/cli/src/registry/compatibility.test.ts packages/cli/src/commands/add.ts packages/cli/src/commands/add.test.ts packages/cli/src/commands/catalog.ts
  • bunx fallow audit --base upstream/main --fail-on-issues

@miguel-heygen

Copy link
Copy Markdown
Collaborator

sign the commit pls to merge it

@kiyeonjeon21 kiyeonjeon21 force-pushed the kj/registry-metadata-compat branch from 838f612 to f24c54f Compare June 7, 2026 08:11
@kiyeonjeon21

Copy link
Copy Markdown
Contributor Author

Done, signed and force-pushed. Thanks!

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