Problem
-
.specs/PAN-COMMANDS.md contains draft reasoning text at lines 45-58 ("Wait: /export, /share...", "Actually the code comment...") that was not cleaned up before writing.
-
Five dispatch tuning parameters (timeout, maxDepth, concurrency, heartbeatInterval, workerTimeout) are admin-only and have no slash commands. This is arguably correct for admin-only params, but /doctor cannot suggest parameter adjustments that the user can apply directly.
-
The CONFIG_CHANGED handler uses a switch/case on event.key with an empty default branch. Adding new config params to the schema does not produce a compile-time error if the handler is not updated.
-
The sharedBus uses string-typed channel names at runtime. BusEventMap provides type documentation but emit/on accept plain strings. A typo in a channel name creates a silent no-op.
Audit Reference
AUDIT-ISSUES.md: C9, E7, F4, F5
Objective
Clean up documentation and address minor architecture hygiene.
Deliverables
Problem
.specs/PAN-COMMANDS.mdcontains draft reasoning text at lines 45-58 ("Wait: /export, /share...", "Actually the code comment...") that was not cleaned up before writing.Five dispatch tuning parameters (timeout, maxDepth, concurrency, heartbeatInterval, workerTimeout) are admin-only and have no slash commands. This is arguably correct for admin-only params, but /doctor cannot suggest parameter adjustments that the user can apply directly.
The CONFIG_CHANGED handler uses a switch/case on event.key with an empty default branch. Adding new config params to the schema does not produce a compile-time error if the handler is not updated.
The sharedBus uses string-typed channel names at runtime. BusEventMap provides type documentation but emit/on accept plain strings. A typo in a channel name creates a silent no-op.
Audit Reference
AUDIT-ISSUES.md: C9, E7, F4, F5
Objective
Clean up documentation and address minor architecture hygiene.
Deliverables