Skip to content

Show the Fleet in the Console Inbox - #1826

Merged
JacobStephens2 merged 4 commits into
issue-1142from
issue-1763
Nov 6, 2026
Merged

JacobStephens2 merged 4 commits into
issue-1142from
issue-1763

Conversation

@JacobStephens2

@JacobStephens2 JacobStephens2 commented Nov 6, 2026 •

Copy link
Copy Markdown
Owner

Summary

open page subscription
  poll each live Fleet machine independently every 15 seconds
  retain last good Inbox and last error in memory
  merge groups, Needs you counts, notifications and active runs
  ?machine=<name> selects one machine
  60 seconds without a good answer → unreachable, grey retained rows
close last page subscription → stop polling

Machine tags, answer ages, refused/update diagnostics and Console links fit desktop and phone widths. The three hand-written site pages describe the Fleet Inbox.

Closes #1763

Evidence

Execution evidence

  • Before: node --test console/tests/fleet.test.cjs failed with Timeout 5000ms exceeded, waiting for the remote machine's row.
    After: the same command passes 2/2, covering two Consoles, remote Needs you notifications, filtering, stale/refused rows, no disk copy, idle request counts and slow-machine isolation.
  • Review reproductions initially failed for Custom process names, duplicate dispatched rows, a lost machine filter and the regression suite's working directory. Each exact reproduction now passes; permanent tests remain in the Console suite.
  • scripts/affected-tests issue-1142: public site PASS (100.89s); Console browser suite PASS (143 tests, 227.26s). Its Rust step hit a transient systemd scope collision; the exact affected architecture test passed on its focused rerun. The subsequent local full retry was stopped after the same head passed CI, reusing CI as the repository instructs.
  • Full Linux Rust CI: 3,856 passed, 91 skipped, 1,281.93s test execution, at 06f222cbab84c0216f629fea5856a2d9a0440bf1.
  • cargo fmt --check and cargo clippy --all-targets -- -D warnings: PASS.
  • Site CI retry: PASS. Its initial Chromium installation consumed 9m19s of the 10-minute job budget, leaving 40s for tests. The complete site suite also passes locally. All applicable CI checks now pass.

Visual evidence

Desktop, 1440 × 900 viewport, Overview:

Before
After

Phone, 390 × 844 viewport, Overview:

Before
After

Test seams

  • Two running Consoles through public HTTP and scoped token CLI commands.
  • Inbox browser interactions at desktop and phone widths, with fixture HTTP and Notification adapters.

Review coverage

Reviewed against issue-1142. Standards: 2 findings, both fixed. Spec: 2 findings, both fixed. Each axis read all 16 changed files. Unread changed files: none. Generated-output exclusions: none.

Unaddressed findings

Standards

None.

Spec

None.

Security

None.

Security review outcome

No unaddressed introduced findings.

Merge Danger

Door: two-way

Reverting restores the local Inbox. Remote answers and errors have no persistent store or migration.

Blast Radius: Console

Inbox rendering, page subscription polling and browser notifications.

Built with codex codex-cli 0.164.0 · gpt-6.1-sol · xhigh

A paired machine's status document can set issue.url independently of
the repository and number that the Inbox shows as the link text. Drop
that href unless it is the same GitHub issue or pull request.
@JacobStephens2
JacobStephens2 merged commit 947713b into issue-1142 Nov 6, 2026
16 checks passed
@JacobStephens2
JacobStephens2 deleted the issue-1763 branch November 6, 2026 17:17
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