net (08/12): Add dashboard summary and expiring diagnostic state - #764
Open
Patrick W. Healy (phealy) wants to merge 2 commits into
Conversation
This was referenced Sep 16, 2026
Patrick W. Healy (phealy)
changed the base branch from
phealy/lightweight-status-detail-responses
to
phealy/lightweight-status-node-publication
September 17, 2026 14:48
Patrick W. Healy (phealy)
added this pull request to stack #813
September 17, 2026 14:59
Patrick W. Healy (phealy)
force-pushed
the
phealy/lightweight-status-dashboard-contract
branch
from
September 17, 2026 21:01
0a2adda to
86f1d28
Compare
Patrick W. Healy (phealy)
force-pushed
the
phealy/lightweight-status-dashboard-contract
branch
from
September 18, 2026 14:31
86f1d28 to
0b9601f
Compare
Patrick W. Healy (phealy)
force-pushed
the
phealy/lightweight-status-dashboard-contract
branch
from
September 22, 2026 16:44
0b9601f to
0ddecc6
Compare
Patrick W. Healy (phealy)
removed this pull request from stack #813
September 22, 2026 16:47
Patrick W. Healy (phealy)
added this pull request to stack #827
September 22, 2026 16:47
Patrick W. Healy (phealy)
force-pushed
the
phealy/lightweight-status-dashboard-contract
branch
from
September 22, 2026 17:09
0ddecc6 to
5adcf1d
Compare
Copilot started reviewing on behalf of
Patrick W. Healy (phealy)
September 22, 2026 17:15
View session
Patrick W. Healy (phealy)
force-pushed
the
phealy/lightweight-status-dashboard-contract
branch
from
September 22, 2026 17:22
5adcf1d to
a6a7de5
Compare
Contributor
There was a problem hiding this comment.
Copilot review overview
🟡 Changes recommended
Unresolved issues remain in diagnostic expiry handling and summary metadata/projection behavior.
Get a fresh assessment by requesting another Copilot review.
Review effort: Lite
Findings: 2
Open (2)
What changed in this PR
Adds frontend dashboard summary projection and cancellable, expiring node diagnostic state.
Changes:
- Adds summary projection and delta handling.
- Adds diagnostic polling, cancellation, expiry, and React hook support.
- Adds API/types and lifecycle tests.
| File | Summary |
|---|---|
frontend/tests/nodeDetails.test.ts |
Diagnostic lifecycle and expiry tests |
frontend/tests/clusterSummary.test.ts |
Summary projection tests |
frontend/src/types.ts |
Summary and diagnostic result types |
frontend/src/state/nodeDetails.ts |
Diagnostic lifecycle state |
frontend/src/state/clusterSummary.ts |
Summary projection and delta merging |
frontend/src/hooks/useNodeDetails.ts |
React diagnostic hook |
frontend/src/api.ts |
Diagnostic HTTP transport |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Patrick W. Healy (phealy)
force-pushed
the
phealy/lightweight-status-dashboard-contract
branch
from
September 22, 2026 17:32
a6a7de5 to
5e098dc
Compare
Patrick W. Healy (phealy)
force-pushed
the
phealy/lightweight-status-dashboard-contract
branch
from
September 22, 2026 17:58
5e098dc to
dab6d5d
Compare
Patrick W. Healy (phealy)
marked this pull request as ready for review
September 22, 2026 18:06
Patrick W. Healy (phealy)
force-pushed
the
phealy/lightweight-status-dashboard-contract
branch
from
September 22, 2026 19:06
dab6d5d to
9acac96
Compare
Cameron Childress (cchildress)
force-pushed
the
phealy/lightweight-status-dashboard-contract
branch
from
September 22, 2026 19:39
9acac96 to
67dd001
Compare
Patrick W. Healy (phealy)
force-pushed
the
phealy/lightweight-status-dashboard-contract
branch
from
September 22, 2026 20:43
67dd001 to
fe5f9b2
Compare
…equests Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: d2243398-6c36-4c3d-969e-7ed7bfb5b459
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: d2243398-6c36-4c3d-969e-7ed7bfb5b459
Patrick W. Healy (phealy)
force-pushed
the
phealy/lightweight-status-dashboard-contract
branch
from
September 22, 2026 21:56
fe5f9b2 to
e2c35c6
Compare
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Consolidated lightweight-status stack: 8/12
Based on #774. This existing draft PR is reused for the user-approved consolidation, replacing unnecessarily fragmented PRs rather than creating another set.
Scope
Final frontend summary projection and diagnostic client state, request cancellation/polling/expiry, hook, metadata/freshness preservation, and browser-safe timer implementation. Actual dialog/browser fixtures remain in layer 9.
Original work represented: #764, #765, #766, #782, #783, #784, #796, #797. Cross-cutting originals may also be represented by adjacent functional layers.
Size
+794 / -1 = 795 non-generated changed lines, measured against the immediate parent. Generated changes: 0 lines, excluded from the approved 2,000-line ceiling. About 1,000 is a guideline, not a requirement to split coherent functionality.
Validation
The final stack preserves the validated implementation and all three CLI corrections exactly; its only additional tracked difference is the requested AGENTS.md approval/size policy. No PR is merged into main.
Approved 12-PR draft stack
Base: #745. Each PR targets its preceding layer.
#746 -> #812 -> #760 -> #762 -> #755 -> #757 -> #774 -> #764 -> #766 -> #767 -> #788 -> #748