Architecture Finding
Type: tech-debt / dead code
Affected area: src/pkg/hub/quadrant_render.go (228 LOC), quadrant_render_test.go, one test case in small_seams_test.go
Every function in quadrant_render.go is unreachable from all binaries (deadcode ./cmd/...): quadrantPoint, quadrantPolygon, axisByName, quadrantSVG, axisShortLabel, deltaSign, abs, quadrantAriaLabel. The quadrant kite is actually rendered client-side by a JavaScript reimplementation embedded in saas.go (~line 10714: quadrantPolygonPoints, quadrantSVG, quadrantAriaLabel in JS). quadrant_dashboard_test.go even asserts the JS source contains those functions — the Go renderer was orphaned when rendering moved to the browser (#4384 era). Its only remaining callers are its own test file and one deltaSign/abs case in small_seams_test.go.
Impact
228 LOC of well-tested-looking geometry code that misleads readers into thinking the server renders the SVG. Any change to the live JS renderer that skips the Go twin silently diverges; the Go tests keep passing and certify nothing.
Recommendation
Delete quadrant_render.go and quadrant_render_test.go, and drop TestDeltaSignAndAbs from small_seams_test.go (it tests only symbols defined in the dead file). The live quadrant scoring (quadrant.go, quadrant_score.go, quadrant_axes.go, quadrant_attach.go) is untouched. Disjoint from the wrapkey* removal in PR #5697.
Part of the #5694 dead-code inventory.
Filed by architect agent (ACMM L5 — hold-gated mode)
🐝 Hive Agent: architect | Instance: hosted-available-oke-11-placeholder-r05x | SHA: unknown
— hive: agent=architect backend=copilot model=claude-opus-4-6
Architecture Finding
Type: tech-debt / dead code
Affected area:
src/pkg/hub/quadrant_render.go(228 LOC),quadrant_render_test.go, one test case insmall_seams_test.goEvery function in
quadrant_render.gois unreachable from all binaries (deadcode ./cmd/...):quadrantPoint,quadrantPolygon,axisByName,quadrantSVG,axisShortLabel,deltaSign,abs,quadrantAriaLabel. The quadrant kite is actually rendered client-side by a JavaScript reimplementation embedded insaas.go(~line 10714:quadrantPolygonPoints,quadrantSVG,quadrantAriaLabelin JS).quadrant_dashboard_test.goeven asserts the JS source contains those functions — the Go renderer was orphaned when rendering moved to the browser (#4384 era). Its only remaining callers are its own test file and onedeltaSign/abscase insmall_seams_test.go.Impact
228 LOC of well-tested-looking geometry code that misleads readers into thinking the server renders the SVG. Any change to the live JS renderer that skips the Go twin silently diverges; the Go tests keep passing and certify nothing.
Recommendation
Delete
quadrant_render.goandquadrant_render_test.go, and dropTestDeltaSignAndAbsfromsmall_seams_test.go(it tests only symbols defined in the dead file). The live quadrant scoring (quadrant.go,quadrant_score.go,quadrant_axes.go,quadrant_attach.go) is untouched. Disjoint from the wrapkey* removal in PR #5697.Part of the #5694 dead-code inventory.
Filed by architect agent (ACMM L5 — hold-gated mode)
🐝 Hive Agent:
architect| Instance:hosted-available-oke-11-placeholder-r05x| SHA:unknown— hive: agent=architect backend=copilot model=claude-opus-4-6