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

[Patrick W. Healy (phealy)]

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.

[Copilot]

Copilot review overview

🟡 Changes recommended

Final review comments remain unresolved.

Get a fresh assessment by requesting another Copilot review.

Review effort: Lite
Findings: 1

Open (1)

• Update linked networking configuration reference · New

What changed in this PR

Activates summary-only network status publication, retires heavy connectivity broadcasts, and updates related documentation and lifecycle handling.

Changes:

• Enables summary defaults in code, tests, and rendered configuration.
• Removes connectivity-matrix generation and detailed broadcast history.
• Updates operational, configuration, troubleshooting, and architecture documentation.

File	Summary
internal/​net/​config/​status_detail_test.go	Updates default-mode tests.
internal/​net/​config/​runtime_config.go	Sets summary as the default mode.
docs/​net/​troubleshooting.md	Adds summary and diagnostic guidance.
docs/​net/​operations.md	Documents explicit diagnostics and operations.
docs/​net/​configuration.md	Updates runtime configuration guidance.
docs/​net/​architecture.md	Describes lightweight status architecture.
deploy/​net/​01-configmap.yaml.tmpl	Enables summary publication by default.
cmd/​unbounded-net-node/​status_publication_test.go	Tests summary publication defaults.
cmd/​unbounded-net-node/​status_detail_config_test.go	Updates configuration expectations.
cmd/​unbounded-net-controller/​websocket.go	Retires heavy broadcasts.
cmd/​unbounded-net-controller/​websocket_retention_test.go	Tests summary-only retention and shutdown cleanup.
cmd/​unbounded-net-controller/​status_types.go	Removes connectivity-matrix types and fields.
cmd/​unbounded-net-controller/​server.go	Uses cancellable server context for background loops.
cmd/​unbounded-net-controller/​memory_bench_test.go	Removes obsolete matrix benchmarks.
cmd/​unbounded-net-controller/​matrix_memory_test.go	Removes obsolete matrix tests.
cmd/​unbounded-net-controller/​cluster_status.go	Removes matrix construction.
cmd/​unbounded-net-controller/​cluster_status_test.go	Removes matrix test coverage.
AGENTS.md	Adds branch and PR approval policy.

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.