You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
This pull request updates the sandbox setup and dependency installation steps to leverage a pre-configured snapshot, streamlining the environment initialization process and reducing redundant configuration. The main improvements are the switch to a snapshot-based sandbox, removal of manual Git and global tool setup, and simplification of dependency installation.
Sandbox initialization improvements:
The create-sandbox.ts step now uses a pre-configured snapshot for environment setup, eliminating the need for manual Git user configuration and global tool installation. The repository is cloned using the authenticated URL after snapshot creation.
Git configuration simplification:
The configure-git.ts step removes manual configuration of user.name, user.email, and core.hooksPath, as these are already set in the snapshot. Only the remote URL is updated per run. [1][2]
Dependency installation simplification:
The install-dependencies.ts step removes installation of global tools (ni, claude-code, and package managers), as these are pre-installed in the snapshot. Now, only project dependencies are installed using ni.
Note
Medium Risk
Changes the sandbox bootstrap path (snapshot + manual clone) and git/dep setup used by automated workflows; failures would break PR review/lint automation though scope is contained to sandbox orchestration.
Overview
Switches linting/PR-review workflows to create sandboxes from a pre-baked snapshot, then explicitly git clone the target repo into the sandbox instead of having Sandbox.create pull from Git.
Simplifies per-run setup by removing global tool installation from installDependencies and dropping git user configuration from configureGit (now just sets authenticated origin and forces local hooks off). Docs are updated to describe the snapshot environment, and the lockfile bumps the CLI/docs toolchain versions.
Written by Cursor Bugbot for commit a498cea. This will update automatically on new commits. Configure here.
Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.
This PR includes no changesets
When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types
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
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.
This pull request updates the sandbox setup and dependency installation steps to leverage a pre-configured snapshot, streamlining the environment initialization process and reducing redundant configuration. The main improvements are the switch to a snapshot-based sandbox, removal of manual Git and global tool setup, and simplification of dependency installation.
Sandbox initialization improvements:
create-sandbox.tsstep now uses a pre-configured snapshot for environment setup, eliminating the need for manual Git user configuration and global tool installation. The repository is cloned using the authenticated URL after snapshot creation.Git configuration simplification:
configure-git.tsstep removes manual configuration ofuser.name,user.email, andcore.hooksPath, as these are already set in the snapshot. Only the remote URL is updated per run. [1] [2]Dependency installation simplification:
install-dependencies.tsstep removes installation of global tools (ni,claude-code, and package managers), as these are pre-installed in the snapshot. Now, only project dependencies are installed usingni.Note
Medium Risk
Changes the sandbox bootstrap path (snapshot + manual clone) and git/dep setup used by automated workflows; failures would break PR review/lint automation though scope is contained to sandbox orchestration.
Overview
Switches linting/PR-review workflows to create sandboxes from a pre-baked snapshot, then explicitly
git clonethe target repo into the sandbox instead of havingSandbox.createpull from Git.Simplifies per-run setup by removing global tool installation from
installDependenciesand dropping git user configuration fromconfigureGit(now just sets authenticatedoriginand forces local hooks off). Docs are updated to describe the snapshot environment, and the lockfile bumps the CLI/docs toolchain versions.Written by Cursor Bugbot for commit a498cea. This will update automatically on new commits. Configure here.