Conversation
Rename packages/cli -> packages/kernel-cli (@ocap/cli -> @metamask/kernel-cli) and packages/nodejs -> packages/kernel-node-runtime (@ocap/nodejs -> @metamask/kernel-node-runtime), adding all required public package fields (license, keywords, publishConfig, module/main/types, changelog scripts, README, LICENSE files). Move bundle-vats vite plugins from @ocap/repo-tools into @metamask/kernel-utils under a new ./vite-plugins export subpath, so kernel-cli can depend on a public package for that functionality. Update all monorepo-wide references: kernel-test, nodejs-test-workers, extension, omnium-gatherum, root tsconfig/eslint config, yarn.config.cjs (add kernel-cli to exportsExceptions and buildExceptions; rename noBuild -> buildExceptions). Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
7500c9a to
eb9660e
Compare
eb9660e to
3b2b0d1
Compare
Coverage Report
File Coverage
|
||||||||||||||||||||||||||||||||||||||||||||||||||
matrix.package values are scoped package names (e.g. @metamask/kernel-node-runtime) but the artifact path needs the directory name (kernel-node-runtime). Add matrix include entries to map each package to its directory, and use matrix.directory for artifact name and path. Also adds the missing packages/kernel-cli/CHANGELOG.md. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.
| "libp2p": "2.10.0", | ||
| "setimmediate": "^1.0.5" | ||
| "setimmediate": "^1.0.5", | ||
| "vite": "^7.3.0" |
There was a problem hiding this comment.
vite as production dependency bloats all consumers
Medium Severity
Moving vite (and acorn) from devDependencies to dependencies in the public @metamask/kernel-utils package means every consumer installing kernel-utils will pull in vite and its full dependency tree, even if they only use core utilities and never touch the ./vite-plugins subpath export. The standard pattern for vite plugins is to declare vite as a peerDependency, letting consumers provide their own version and avoiding unnecessary installation for non-plugin users.


packages/cli→packages/kernel-cli(@ocap/cli→@metamask/kernel-cli) and make it publicpackages/nodejs→packages/kernel-node-runtime(@ocap/nodejs→@metamask/kernel-node-runtime) and make it publicbundle-vatsvite plugins from@ocap/repo-toolsinto@metamask/kernel-utils/vite-pluginssokernel-clican depend on a public package for that functionalitynoBuild→buildExceptionsinyarn.config.cjs; addkernel-clitoexportsExceptionsandbuildExceptionsNote
Medium Risk
Primarily packaging/renaming, but it changes published artifacts, workspace dependency graph, and CI/test wiring; mistakes could break builds or consumers of the newly public packages.
Overview
This PR renames and makes public the former
@ocap/cliand@ocap/nodejsworkspaces as@metamask/kernel-cliand@metamask/kernel-node-runtime, adding publicpublishConfig, dual MIT/Apache licensing, metadata, and updating build/changelog scripts accordingly.It moves vat bundling Vite plugin APIs (
bundleVat/bundleVatsandVatBundle) out of@ocap/repo-toolsand into@metamask/kernel-utils/vite-plugins, then rewires the CLI, extension, andomnium-gatherumconfigs to import from the new public location.Monorepo plumbing is updated to match the renames: root/workspace deps and LavaMoat allowlist entries, TS project references, ESLint file globs, Yarn constraints (
noBuild→buildExceptionsplus new exceptions), CI E2E workspace matrix/artifact paths, and test packages (kernel-test,nodejs-test-workers) now depend on@metamask/kernel-node-runtime/@metamask/kernel-cliinstead of@ocap/*.Written by Cursor Bugbot for commit 5e04b83. This will update automatically on new commits. Configure here.