Rename the package to Browser Loop - #90
Merged
Merged
Conversation
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes using high effort and found 1 potential issue.
Bugbot Autofix prepared a fix for the issue found in the latest run.
- ✅ Fixed: Stale screenshot path after rename
- Updated the fallback fixture lookup path in native-action-probe to
packages/browser-loop/examples/screenshot.pngso root-invoked probes find the screenshot after the rename.
- Updated the fallback fixture lookup path in native-action-probe to
Or push these changes by commenting:
@cursor push c82dce3134
Preview (c82dce3134)
diff --git a/packages/browser-loop/scripts/native-action-probe.ts b/packages/browser-loop/scripts/native-action-probe.ts
--- a/packages/browser-loop/scripts/native-action-probe.ts
+++ b/packages/browser-loop/scripts/native-action-probe.ts
@@ -246,7 +246,7 @@
async function readFixtureScreenshot(): Promise<Buffer> {
const path = [
join(process.cwd(), "examples", "screenshot.png"),
- join(process.cwd(), "packages", "loop", "examples", "screenshot.png"),
+ join(process.cwd(), "packages", "browser-loop", "examples", "screenshot.png"),
].find(existsSync);
if (!path) throw new Error("could not find packages/browser-loop/examples/screenshot.png");
return readFile(path);You can send follow-ups to the cloud agent here.
Reviewed by Cursor Bugbot for commit 7b1ecf3. Configure here.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.


Summary
Prepare the project for the new Browser Loop name before the first npm publish and GitHub repository rename:
@onkernel/loopto@onkernel/browser-loopand its workspace directory topackages/browser-loopkernel/browser-loop(repository, homepage, issues), rename the private root workspace tobrowser-loop, and update the lockfile/project referencesloopnamespace,Loop*TypeScript symbols, andkloop.*.v1tool identitiesrelease-browser-loop.yml, tags tobrowser-loop/v*, tarball smokes toonkernel-browser-loop-*.tgz, and all CI workspace targetskernel/browser-loop, and the new trusted-publisher tupleThe historical
@onkernel/cua-*names remain unchanged in the changelog.Verification
npm installnpm run typechecknpm run build --workspace @onkernel/browser-loopnpm test --workspace @onkernel/browser-loop— 34 files, 429 testsonkernel-browser-loop-0.11.0.tgz, installed it in a clean project, and imported both@onkernel/browser-loopand@onkernel/browser-loop/piNote
Medium Risk
Wide rename across lockfile, workflows, and every import alias risks a missed
@onkernel/loopreference or broken release tag/workflow pairing until the repo rename and first npm publish are completed.Overview
Rebrands the project from Loop /
@onkernel/loopto Browser Loop and@onkernel/browser-loop, aligning the workspace directory (packages/browser-loop), root package name (browser-loop), and planned GitHub repo (kernel/browser-loop) before the first npm publish.Consumer-facing API stays stable: the
loopnamespace,Loop*TypeScript types, andkloop.*.v1tool identities are unchanged; only npm package names, install/import paths (@onkernel/browser-loop,@onkernel/browser-loop/pi), and user-facing prose move to Browser Loop. The tool spec kind string updates to@onkernel/browser-loop-tool-spec/v1.Release and CI switch to
release-browser-loop.yml, tagsbrowser-loop/v*, tarballonkernel-browser-loop-*.tgz, and workspace-scoped npm commands across CI and agent skills. Docs and runbooks (docs/npm-releases.md, release/update-docs skills) describe manual first publish, trusted publishing for the new repo/workflow pair, and avoiding republishing0.11.0after a manual release.Internal pi-extension and test aliases, tsconfig paths, and vitest resolves now point at
@onkernel/browser-loopso jiti-loaded extension code matches the published package name.Reviewed by Cursor Bugbot for commit f72a0f2. Bugbot is set up for automated code reviews on this repo. Configure here.