Skip to content

dev app: a local build fetches helper deps a fresh worktree lacks - #421

Merged
m4ttheweric merged 2 commits into
mainfrom
dev-app-stage-fetch-deps
Sep 24, 2026
Merged

m4ttheweric merged 2 commits into
mainfrom
dev-app-stage-fetch-deps

Conversation

@m4ttheweric

Copy link
Copy Markdown
Collaborator
  • stageLocalDevApp fetches deps into the scratch copy when rt-tray/deps/arm64 or rt-tray/deps/tools is missing, even after copying a partial source deps dir (a fresh worktree made "Rebuild from" fail with deps/arm64 missing)
  • runs bash scripts/fetch-deps.sh arm64 in the scratch dir with a 30 min timeout; a failed fetch ends the log with ✗ ... naming the step
  • build-dev-app.ts streams the fetch like the build

4 new tests in dev-app-stage.test.ts (no deps, partial deps, timeout, failed fetch); 12/12, tsc clean.

🤖 Generated with Claude Code

m4ttheweric and others added 2 commits September 24, 2026 15:05
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Sep 24, 2026

Copy link
Copy Markdown

Warning

Review limit reached

Next included review available in 24 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available. Your 83 included PR review attempts over the past 7 days set your current allowance at 1 review per hour.

This review is too large to run within your organization's remaining usage spending cap. Raise or remove your spending cap in the billing tab, then retry.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Advanced

Run ID: b006102a-3847-40e8-98f7-7193a09ffc43

📥 Commits

Reviewing files that changed from the base of the PR and between 2520166 and a5837df.

📒 Files selected for processing (3)
  • lib/release/__tests__/dev-app-stage.test.ts
  • lib/release/dev-app-stage.ts
  • scripts/build-dev-app.ts

Comment @coderabbitai help to get the list of available commands.

@m4ttheweric
m4ttheweric merged commit a1ff23b into main Sep 24, 2026
6 checks passed
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.

1 participant