fix(memory): governance module ate all @memory commands — standard handle_command signature, declines non-governance commands (Track 2 follow-up). Search verified live, 1011 green
This commit is contained in:
@@ -71,6 +71,18 @@ PyPI version — not the changelog header.
|
||||
auto-trigger swept a pre-existing backlog file on its first run. 730 flow +
|
||||
878 drone tests green, seedgo 100%.
|
||||
|
||||
### Fixed
|
||||
|
||||
- **@memory command routing eaten by the new governance module.** The
|
||||
governance module shipped in Track 2 had the wrong `handle_command`
|
||||
signature (`args: list` instead of `command: str, args: list`) and always
|
||||
returned True, so auto-discovery routed EVERY @memory command through it
|
||||
first — `drone @memory search` answered "governance: unknown command 's'".
|
||||
Fixed to the standard signature returning False for commands not its own;
|
||||
search verified live (135 results). Library modules must decline commands
|
||||
they don't own or they silently hijack the whole CLI. 1011 memory tests
|
||||
green, seedgo 31/31.
|
||||
|
||||
## [2026-07-15]
|
||||
|
||||
### Fixed
|
||||
|
||||
@@ -189,18 +189,21 @@ def print_introspection() -> None:
|
||||
console.print("[dim]Library module — import from: aipass.memory.apps.modules.governance[/dim]")
|
||||
|
||||
|
||||
def handle_command(args: list, _remaining: list | None = None) -> bool:
|
||||
def handle_command(command: str, args: list) -> bool:
|
||||
"""Entry point for drone module discovery — governance has no CLI surface."""
|
||||
if command != "governance":
|
||||
return False
|
||||
|
||||
if not args:
|
||||
print_introspection()
|
||||
return True
|
||||
|
||||
from aipass.cli.apps.modules import warning
|
||||
|
||||
if args[0] in ("--help", "-h", "help"):
|
||||
print_introspection()
|
||||
return True
|
||||
|
||||
warning(f"governance: unknown command '{args[0]}'")
|
||||
from aipass.cli.apps.modules import warning
|
||||
|
||||
warning(f"governance: unknown subcommand '{args[0]}'")
|
||||
print_introspection()
|
||||
return True
|
||||
|
||||
Reference in New Issue
Block a user