fix: M1-M11 medium-severity issues from prior code review - #53

Merged

Patel230 merged 14 commits into

mainfrom

fix/m1-m11-medium-severity-2026-06

Jun 17, 2026

Commits on Jun 17, 2026

fix(cmd): M1 part 1 - pinSubcommand, GLM thinking, status use accessors
Patel230
committed
2deeacf
fix(cmd): M1 part 2 - Autonomy, ContainerExecutor, ConvoDAG accessors
Patel230
committed
1abfb17
fix(cmd): M1 part 3 - remaining cmd/ Cost/Mode/ContextWindow accessors
Patel230
committed
4e4867e
fix(engine): M2 - AddUser/AddAssistant use MemorySvc() and Persistence().DAG()
Patel230
committed
a1fbace
fix(engine): M3 - deprecation comments point at real accessors
Patel230
committed
d952fe7
fix(engine): M4 - PermissionService constructor no longer pre-sets mode
Patel230
committed
3da81c4
fix(cmd): M5 - SubcommandRegistry.Register detects alias collision
Patel230
committed
31d8928
fix(cmd): M6 - sessionSubcommand.Handle prepends command name to args
Patel230
committed
2a4faa9
fix(multiagent): M7 - WaitForLock no longer busy-spins
Patel230
committed
26ffba6
fix(multiagent): M8 - logDroppedMessage moved out of mb.mu critical section
Patel230
committed
78ca51d
fix(multiagent): M9 - extract dropLogEveryN as a named constant
Patel230
committed
3a4f1cb
test(multiagent): M10 - rename test + add real waiter-assertion test
Patel230
committed
a01da92
test(multiagent): M11 - join the WaitForResponse goroutine via a done channel
Patel230
committed
1d06b3d
fix(engine): M2 - make AddUser/AddAssistant/ForkConversation/etc nil-safe
Patel230
committed
f1ddebb