mirror of
https://github.com/proffesor-for-testing/agentic-qe.git
synced 2026-09-19 08:45:47 +08:00
Dragan Spiridonov
112fffc466
Devil's-advocate review of today's branch surfaced three real gaps: C1 (init-handler.ts): bare `aqe init --no-claude` (no --with-<platform>, no --no-database) fell through to runStandardInit and installed the FULL Claude surface it was meant to suppress — `noClaude` was only honored inside the modular path. Add `options.claude === false` to the modular routing guard. Verified: `aqe init --no-claude --minimal` now writes no .claude/, .mcp.json, or CLAUDE.md (and the empty-install warning fires). C2 (continuedev-installer.ts): the mergeExistingYamlConfig branch (existing `mcpServers:` + --overwrite) appended a hardcoded AQE_MEMORY_PATH entry, ignoring memoryBackend — re-introducing the #533 db leak in db-free installs over an existing Continue config. Honor memoryBackend on this path too. Adds a regression test. H3 (entry.ts): main() ran fire-and-forget, so a startup rejection that escaped its internal try hit the unhandledRejection handler (which intentionally keeps the server alive at runtime) → on the in-process `agentic-qe mcp` path the CLI wrapper would hang on its never-resolving promise with a zero exit. main() now `.catch`es and exits non-zero on a failed startup. Re-verified #528/#535 integration tests still green (normal startup unaffected). Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>