Skip to content

feat(desktop): show fleet coordinator status - #100

Merged
ctwoodwa merged 2 commits into
feat/daemon-visibilityfrom
feat/coordinator-visibility
Jul 24, 2026
Merged

feat(desktop): show fleet coordinator status#100
ctwoodwa merged 2 commits into
feat/daemon-visibilityfrom
feat/coordinator-visibility

Conversation

@ctwoodwa

Copy link
Copy Markdown
Contributor

What changed

  • show the Fleet Coordinator at the top of Console → Coordination Daemons
  • report authority, epoch, revision, assignment/attempt counts, and reporting nodes
  • expose online, unreachable, token-required, degraded, and not-configured states without relying
    on color alone
  • add a Dock Settings connection field that writes the shared Ordinance coordinator config
  • keep the bearer token in the native process and preserve its local file reference on URL edits
  • remove device assumptions: no customer host, address, user path, or credential is compiled in

Why

Tender is the fleet cockpit. Operators need to see whether the single assignment authority is
reachable and authenticated, while connection targets remain install- and customer-specific
settings rather than hard-coded developer-machine values.

Security and behavior

  • accepts only credential-free HTTP(S) origins
  • reads the token locally and never returns it through Tauri IPC
  • uses atomic mode-0600 config replacement on Unix
  • preserves unknown config fields and token references
  • serializes status polling so a slow probe cannot create overlapping request bursts
  • treats every non-online authority state as requiring attention

Validation

  • npm run build
  • cargo test --lib (124 passing, 3 ignored)
  • Impeccable detector: no findings in either changed TSX surface
  • browser QA at the product's fixed 360px tray width
  • keyboard focus/ordering verified for both connection fields
  • no Tender-origin console errors observed

Stacking

This draft targets feat/daemon-visibility so its diff contains only coordinator visibility and
configuration. Retarget to main after the prerequisite daemon-visibility PR lands.

@coderabbitai

coderabbitai Bot commented Jul 20, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 04d2a604-f79c-49d7-87a4-fca94471a474

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/coordinator-visibility

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

…#102)

* feat(desktop): checkpoint Windows tray seams [wip]

* docs(tender): describe Windows tray support and gate a Unix-only test

Harborline Toolbox now ships as a Windows tray app (platform.rs opener/home_dir
seam, tauri-plugin-autostart integration) from an earlier checkpoint, but the
README and product spec still described macOS only. Document both platforms
end to end: install/build steps, the platform-seam table, start-at-login
mechanism differences, and the per-user MSI/NSIS installer nuance (Tauri's
WiX bundler has no perUser install-scope option yet -- upstream feature
request tauri-apps/tauri issue 13792 -- so the NSIS installer is the
per-user, no-elevation-required path today; the MSI is per-machine).

Also gate stop_by_pattern_terminates_a_live_process_then_reports_none as
Unix-only: it spawns bash and relies on pkill -f, neither of which exist on
Windows, and stop_by_pattern itself has no Windows backend yet (pre-existing
gap, unrelated to the tray work). Verified clean on a Windows x64 build: MSI
builds without remediation, cargo test --lib is 124/0/3 (was 123/1/3),
frontend build passes, and the installed app launches in the logged-on
desktop session (not Session 0) with a real notification-area icon that
opens the panel on click.

Refs: tender#2956
@ctwoodwa
ctwoodwa marked this pull request as ready for review July 24, 2026 10:01
@ctwoodwa
ctwoodwa merged commit 289c86b into feat/daemon-visibility Jul 24, 2026
3 checks passed
@ctwoodwa
ctwoodwa deleted the feat/coordinator-visibility branch July 24, 2026 10:01
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