Skip to content

parity instrument: dump all gone-repo rows on failure - #165

Merged
m4ttheweric merged 1 commit into
mainfrom
test/parity-flake-instrument
Sep 1, 2026
Merged

m4ttheweric merged 1 commit into
mainfrom
test/parity-flake-instrument

Conversation

@m4ttheweric

Copy link
Copy Markdown
Collaborator

One-line layer on a2c0a4a: the failure dump now shows every gone-repo row instead of the first, so a live/scanned duplicate shadowing the lost row (the other standing theory) is visible in the same firing.

🤖 Generated with Claude Code

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Sep 1, 2026

Copy link
Copy Markdown

Warning

Review limit reached

Next included review available in 57 seconds.

Check out review usage here.

View limit details

Limit details: You’ve used all 6 included reviews currently available. Your 45 included PR review attempts over the past 7 days set your current allowance at 6 reviews 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: Team

Run ID: 266d2d63-8251-423b-8cf3-3f72e86d20a5

📥 Commits

Reviewing files that changed from the base of the PR and between a2c0a4a and c84b79d.

📒 Files selected for processing (1)
  • lib/__tests__/repo-index-async.test.ts

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

@m4ttheweric
m4ttheweric merged commit fdb7c38 into main Sep 1, 2026
4 checks passed
@m4ttheweric
m4ttheweric deleted the test/parity-flake-instrument branch September 1, 2026 20:02
m4ttheweric added a commit that referenced this pull request Sep 17, 2026
Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
m4ttheweric added a commit that referenced this pull request Sep 26, 2026
… its service (RT-318) (#165)

* board: the badge and the decision queue build one gate set (RT-317)

decisionGates attaches each visible MR's gates, its runs' gates included,
and builds the queue's extras; /data.json serves it and /api/badge counts
it through decisionBadge, so the badge cannot drop the run links the queue
shows.

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

* deck: its own row shows the launchd job running it (RT-318)

The app bundle's helper keeps its plist in the bundle, so no LaunchAgent
names it and the exact-label join left deck's row without a service,
which hid its pid and restart button. buildStatus now gives the self row
the label deckOwner.runningLabel() reports and this process's pid; with
no launchd job running deck the row borrows nothing.

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

* deck: the self row asks launchd for this pid's own job, once

DeckOwner.selfLabel() names a label only when a helper job's pid is this
process's, so a hand-started or worktree deck no longer shows the helper's
label (whose restart would kickstart the other deck). startApi caches it:
a hit for the life of the process, a miss rechecked at most every 60s, so
board polls stop spawning launchctl print. A hand agent that is the
running job keeps its own launchd reading on the row.

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

---------

Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
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