Skip to content

net (12/12): Activate summary defaults and retire heavy broadcasts - #748

Draft
Patrick W. Healy (phealy) wants to merge 1 commit into
phealy/lightweight-status-thin-clusterfrom
phealy/lightweight-status-remove-matrix
Draft

Patrick W. Healy (phealy) wants to merge 1 commit into
phealy/lightweight-status-thin-clusterfrom
phealy/lightweight-status-remove-matrix

Conversation

@phealy

@phealy Patrick W. Healy (phealy) commented Sep 16, 2026

Copy link
Copy Markdown
Contributor

Consolidated lightweight-status stack: 12/12

Based on #788. This existing draft PR is reused for the user-approved consolidation, replacing unnecessarily fragmented PRs rather than creating another set.

Scope

Remove controller connectivity matrix and full-detail broadcast history, enable summary defaults, finish operations/configuration/architecture/troubleshooting docs including both independent CLI fixes, and preserve the exact root AGENTS.md approval-policy update.

Original work represented: #748, #790, #792, #793, #795. Cross-cutting originals may also be represented by adjacent functional layers.

Size

+498 / -830 = 1,328 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

  • Scoped make fmt across all changed Go packages (tmp/layer12-fmt.log, tmp/final-cli-fix-fmt.log, tmp/final-info-fix-fmt.log)
  • Final relevant Go suites (tmp/final-tests.log, tmp/final-cli-fix-tests.log, tmp/final-info-fix-tests.log)
  • Complete relevant race suite passes on rerun (tmp/final-race-rerun.log) and after each CLI fix integration (tmp/final-cli-fix-race.log, tmp/final-info-fix-race.log)
  • Controller, node, and kubectl binary builds
  • Final 24 frontend unit tests, tsc, Vite production build, actual Chromium regression (tmp/final-frontend-tests.log, tmp/final-frontend-build.log, tmp/final-browser.log); frontend unchanged by CLI integration
  • Final source-target Go suites/race and all three binary builds (tmp/final-allocation-fix-tests.log, tmp/final-allocation-fix-race.log)

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

Validation caveat: TestDetailWebSocketCommandAndResponse intermittently reported EOF under race instrumentation; the full relevant race suite passed on rerun without source changes. No data race was reported.

This was referenced Sep 16, 2026
This was referenced Sep 16, 2026
@phealy
Patrick W. Healy (phealy) force-pushed the phealy/lightweight-status-remove-matrix branch from bd2c6e1 to 0c18ea6 Compare September 17, 2026 14:46
@phealy Patrick W. Healy (phealy) changed the title net: remove connectivity matrix computation and contracts net (12/12): Activate summary defaults and retire heavy broadcasts Sep 17, 2026
@phealy
Patrick W. Healy (phealy) changed the base branch from phealy/lightweight-status-remove-topology to phealy/lightweight-status-thin-cluster September 17, 2026 14:49
@phealy
Patrick W. Healy (phealy) added this pull request to stack #813 September 17, 2026 14:59
@phealy
Patrick W. Healy (phealy) force-pushed the phealy/lightweight-status-remove-matrix branch from 0c18ea6 to cca7a22 Compare September 17, 2026 21:01
…il broadcasts

Includes the approved local-branch and PR publication policy.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session: d2243398-6c36-4c3d-969e-7ed7bfb5b459
@phealy
Patrick W. Healy (phealy) force-pushed the phealy/lightweight-status-remove-matrix branch from cca7a22 to 10d261d Compare September 18, 2026 14:31
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