chore(orb): add Amp setup scripts for the Rust toolchain - #186
Merged
Merged
Conversation
Install rustup/stable, rustfmt, and clippy so orbs can run the crate without a preinstalled toolchain. Warm setup is a no-op fetch. Amp-Thread-ID: https://ampcode.com/threads/T-01a0a5f1-67fb-76fe-ac75-eae1b8f3a08a Co-authored-by: Max Carter <undivisible@vk.com>
|
Capy couldn't review this pull request because Max's workspace is out of credits, add credits or enable auto-reload to resume automatic reviews. |
Bugbot couldn't run - usage limit reachedBugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit. A user or team admin can review and increase usage limits in the Cursor dashboard. (requestId: serverGenReqId_4a5407e0-b25b-4ca0-acba-c2d4ca21bf37) |
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
2 participants
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.
Why
Fresh Amp orbs for this repo had no
rustc/cargo. Agents cannot run the AGENTS.md validation commands without installing the toolchain every thread.What
.agents/setupinstalls rustup (stable + rustfmt/clippy) andcargo fetch --locked..agents/resumechecks the toolchain is on PATH..amp/portals/*.Verified in this orb: cold setup ~23s, warm setup ~0.6s, resume <1s, login shell sees
rustc 1.98.1.Note
Low Risk
Dev-environment and gitignore-only changes; no application runtime, auth, or data-path behavior.
Overview
Adds Amp orb bootstrap scripts so fresh environments get a working Rust toolchain without reinstalling every agent thread.
.agents/setupis an idempotent bootstrap: installs missing apt build deps, rustup (stable +rustfmt/clippy), appends a one-time.bash_profilehook to source~/.cargo/env, and runscargo fetch(with--lockedwhenCargo.lockexists)..agents/resumeis a fast wake check that sources cargo env and fails with a pointer to setup ifrustcorcargoare missing..gitignorenow ignores.amp/portals/*so local Amp portal artifacts stay untracked.Reviewed by Cursor Bugbot for commit 8ef0a4c. Configure here.