Skip to content

tray: Discard New Build, cached instead of wasted - #483

Merged
m4ttheweric merged 2 commits into
mainfrom
tray-discard-staged-build
Sep 26, 2026
Merged

m4ttheweric merged 2 commits into
mainfrom
tray-discard-staged-build

Conversation

@m4ttheweric

Copy link
Copy Markdown
Collaborator

Adds a way to clear the dev tray's "new build ready" state without restarting into the build. The discarded build goes into the build cache first, so rebuilding the same tree state later is a copy, not a build.

What changed

Menu (AppDelegate.swift)

  • Adds Discard New Build under the Restart item

Discard (DevBuild.swift, DevBuildWatcher.swift)

  • Adds DevBuild.discardScript: files the staged bundle under its cache key, then clears staged/
  • An unkeyed build, or a cache that can't be written, is just deleted
  • Splits the handoff's filing and tidying (sweep, drop gone trees, keep four) so both scripts share them

Verification

Two new checks in DevBuildChecks.swift; swift run mattstack-checks 546/546 green. Not yet exercised in a running dev app.

🤖 Generated with Claude Code

…ars it

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

coderabbitai Bot commented Sep 25, 2026 •

Copy link
Copy Markdown

Warning

Review limit reached

Next included review available in 49 minutes.

Check out review usage here.

View limit details

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

Your organization has reached its usage spending cap. Adjust your spending cap in the billing tab.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Advanced

Run ID: ba60d929-e9a8-4ce4-9cfd-4a2875ed3385

📥 Commits

Reviewing files that changed from the base of the PR and between 86f2936 and c394c98.

📒 Files selected for processing (5)
  • rt-tray/Sources-core/Services/DevBuild.swift
  • rt-tray/Sources/AccessibilityIDs.swift
  • rt-tray/Sources/AppDelegate.swift
  • rt-tray/Sources/DevBuildWatcher.swift
  • rt-tray/Tests/MattstackCoreChecks/DevBuildChecks.swift

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

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@m4ttheweric
m4ttheweric merged commit db26dfc into main Sep 26, 2026
12 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