Profa da09bc2622 fix: ReasoningBank learning pipeline — 6 issues fixed with 25 tests ...

After brutal honesty review exposed fragile implementations and zero
test coverage, reworked the critical issues properly:

N1: Replace fire-and-forget polling IIFE with event-driven trajectory
    close via CrossDomainEventRouter TaskCompleted/TaskFailed events.
    Subscribed during fleet init, unsubscribed on dispose.

N2: Clean up generate-tests payload mapping — filePaths and codeContext
    already exist on TaskOrchestrateParams (lines 265-267).

N3: Add queryCount() to UnifiedMemoryManager with SQL injection-safe
    allowlist. Wire into fleet_status learning metrics response.

N4: Fix memory_usage to return real vector/namespace counts from
    HybridMemoryBackend instead of hardcoded zeros.

N5: Simplify memory_store persistence check — HybridMemoryBackend
    always persists to SQLite when initialized. Remove duck-typing.

N6: Fix object return type assertion ordering in all 5 test generators
    (7 locations). Check for '{' before primitive type matching to
    prevent misclassifying { valid: boolean } as boolean.

Also: consolidate ReasoningBank/PatternStore embeddings from 768-dim
to 384-dim (all-MiniLM-L6-v2). Coverage-analysis 768-dim indices are
intentionally separate and unchanged.

25 new tests across 5 test files. 157/157 passing.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

2026-02-22 12:31:13 +00:00

..

fleet-code-intelligence-integration.md

feat(v3): add TinyDancer routing, consensus engine, and documentation updates

2026-01-18 17:33:22 +00:00

reasoningbank-learning-system.md

fix: ReasoningBank learning pipeline — 6 issues fixed with 25 tests

2026-02-22 12:31:13 +00:00

skill-validation.md

refactor(skills): remove all validate.sh scripts, use cross-platform Node.js validators

2026-02-11 09:29:43 +00:00