Skip to content

refactor(controller): generic pkg/registry.Store[K, V] (ADR-0925, audit #15) - #425

Merged
lusoris merged 1 commit into
masterfrom
chore/go-generic-registry
May 31, 2026
Merged

lusoris merged 1 commit into
masterfrom
chore/go-generic-registry

Conversation

@lusoris

@lusoris lusoris commented May 31, 2026 •

Copy link
Copy Markdown
Contributor

Refactor controller node registry to generic pkg/registry.Store[K comparable, V any] interface. Replaces map+lock pattern with type-safe generic collection; nodes/registry.go consumes it. Audit #15.

no state delta: per CLAUDE.md §12 r8; no bug tracker entry.

ADR-0108 deliverables checklist

  • Research digest — no digest needed: scope-bounded change.
  • Decision matrix — no alternatives: only-one-way fix.
  • AGENTS.md invariant note — no rebase-sensitive invariants.
  • Reproducer / smoke-test command — branch CI run on this PR.
  • CHANGELOG fragment — no changelog needed: routine refactor.
  • Rebase note — docs/rebase-notes.md.

🤖 Generated with Claude Code

@lusoris
lusoris marked this pull request as ready for review May 31, 2026 03:33
@lusoris
lusoris enabled auto-merge (squash) May 31, 2026 03:33
@lusoris
lusoris force-pushed the chore/go-generic-registry branch from 5e04b50 to ed51696 Compare May 31, 2026 07:28
@lusoris
lusoris marked this pull request as draft May 31, 2026 09:02
auto-merge was automatically disabled May 31, 2026 09:02

Pull request was converted to draft

@lusoris
lusoris marked this pull request as ready for review May 31, 2026 12:09
@lusoris
lusoris force-pushed the chore/go-generic-registry branch from ed51696 to ac8db18 Compare May 31, 2026 12:11
@lusoris
lusoris merged commit 940e255 into master May 31, 2026
1 check failed
@lusoris
lusoris deleted the chore/go-generic-registry branch May 31, 2026 12:11
@lusoris lusoris added this to the 1.0.0 — First release milestone Sep 4, 2026
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