Skip to content

chore(fork): carry the launcher-skip-backup patch - #135

Merged
fireboltdude1357 merged 1 commit into
patch/releasefrom
t3/list-launcher-skip-backup
Oct 29, 2026
Merged

fireboltdude1357 merged 1 commit into
patch/releasefrom
t3/list-launcher-skip-backup

Conversation

@fireboltdude1357

Copy link
Copy Markdown
Owner

Problem

The fork nightly doesn't include the launcher fix in #134 yet. Without it, every service update on stl-wsl copies the 43 GB database while the server is down. That took about 8 minutes on 2026-10-28.

Change

Adds launcher-skip-backup patch/launcher-skip-backup to the end of .github/fork-patches.txt. The branch holds #134 once that merges. Until then it matches upstream main and adds nothing to the build.

Scope and approval

Fork-only, per docs/operations/fork-nightly.md. Requested by Tanner.

Verification

  • None of the 14 listed patches changes serviceLauncher.ts, its test, servicePreflight.ts or serviceProtocol.ts relative to upstream main (git diff --name-only from each branch's merge base). The new patch should merge without conflicts.
  • I did not run scripts/fork-sync.ts. The build runs the full typecheck and the patch's test files.

Review tier: 7/10. Once this merges, nightlies include the patch, and it skips the update backup.

Two-way door: removing the line drops the patch from the next build

Blast radius: which patches the fork nightly merges

Changes made by Claude Opus 5.5 in Claude Code, running in T3 Code.

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Oct 29, 2026

Copy link
Copy Markdown

Review in Change Stack →

No actionable comments were generated in the recent review.

Recent review info
Run configuration
Commits

Reviewing files that changed from the base of the PR and between 60b01e9 and 7c39047.

Files selected for processing: 1
  • .github/fork-patches.txt

Included review availability: This review used your included allowance. 0 included reviews remain after this review. Your included PR review attempts over the past 7 days set your current allowance at 1 review per hour.

Walkthrough

The nightly fork patch list now includes launcher-skip-backup with the fork branch patch/launcher-skip-backup.

Changes: Nightly fork patch list

Layer / File(s) Summary
Add launcher-skip-backup branch
.github/fork-patches.txt
Adds launcher-skip-backup with fork branch patch/launcher-skip-backup to the patch list.

Priority: Normal

Estimated code review effort: 1 (Trivial) | ~3 minutes

Change: Other

Suggested labels: coderabbit-queue

Merge Risk: Minimal · up to 7c390

The nightly remains mergeable; the backup change takes effect after the patch branch is updated.

Pre-merge checks: Passed 4
Check name Status Explanation
Description check Passed The description explains the problem, change, scope, and verification. It identifies the fork-only behavior and states the checks not run. It does not provide a link to a triaged issue or explicit mai…
Title check Passed The title clearly and concisely identifies the main change: carrying the launcher-skip-backup patch in the fork.
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.
Full details: Description check

Explanation

The description explains the problem, change, scope, and verification. It identifies the fork-only behavior and states the checks not run. It does not provide a link to a triaged issue or explicit maintainer approval, but it gives a focused-scope justification and identifies the relevant operations document.

Finishing Touches
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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

@fireboltdude1357
fireboltdude1357 merged commit 6b41bf1 into patch/release Oct 29, 2026
2 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