agentic-qe 3.11.4 → 3.11.5

package/dist/planning/actions/qe-action-library.js

@@ -31,6 +31,12 @@ const coverageActions = [
         successRate: 0.95,
         category: 'coverage',
         qeDomain: 'coverage-analysis',
+        // A14: coordinator.ts:525 analyze() — real, but expects pre-collected
+        // coverageData a GOAP plan step doesn't carry; will error on missing
+        // required params rather than being given a fabricated value.
+        method: 'analyze',
+        params: {},
+        implemented: true,
     },
     {
         name: 'analyze-coverage-gaps',
@@ -43,6 +49,9 @@ const coverageActions = [
         successRate: 0.9,
         category: 'coverage',
         qeDomain: 'coverage-analysis',
+        method: 'detectGaps', // coordinator.ts:555
+        params: {},
+        implemented: true,
     },
     {
         name: 'generate-coverage-tests',
@@ -55,6 +64,9 @@ const coverageActions = [
         successRate: 0.8,
         category: 'coverage',
         qeDomain: 'test-generation',
+        method: 'generateTests', // coordinator.ts:458
+        params: {},
+        implemented: true,
     },
     {
         name: 'run-mutation-testing',
@@ -67,6 +79,9 @@ const coverageActions = [
         successRate: 0.85,
         category: 'coverage',
         qeDomain: 'coverage-analysis',
+        // A14: no mutation-testing capability exists anywhere in src/domains —
+        // do not fake it. GOAPExecutor must report this as not implemented.
+        implemented: false,
     },
     {
         name: 'prioritize-uncovered-paths',
@@ -79,6 +94,9 @@ const coverageActions = [
         successRate: 0.92,
         category: 'coverage',
         qeDomain: 'coverage-analysis',
+        method: 'calculateRisk', // coordinator.ts:607
+        params: {},
+        implemented: true,
     },
     {
         name: 'generate-branch-tests',
@@ -94,6 +112,9 @@ const coverageActions = [
         successRate: 0.78,
         category: 'coverage',
         qeDomain: 'test-generation',
+        method: 'generateTests', // coordinator.ts:458 — no branch-target param; same caveat as generate-coverage-tests
+        params: {},
+        implemented: true,
     },
 ];
 // ============================================================================
@@ -111,6 +132,9 @@ const testActions = [
         successRate: 0.95,
         category: 'test',
         qeDomain: 'test-execution',
+        method: 'runTests', // plugin.ts:51
+        params: {},
+        implemented: true,
     },
     {
         name: 'run-integration-tests',
@@ -123,6 +147,9 @@ const testActions = [
         successRate: 0.85,
         category: 'test',
         qeDomain: 'test-execution',
+        method: 'execute', // plugin.ts:52 — framework/env selection is caller's job
+        params: {},
+        implemented: true,
     },
     {
         name: 'run-e2e-tests',
@@ -135,6 +162,9 @@ const testActions = [
         successRate: 0.8,
         category: 'test',
         qeDomain: 'test-execution',
+        method: 'executeE2ETestSuite', // plugin.ts:57
+        params: {},
+        implemented: true,
     },
     {
         name: 'fix-failing-tests',
@@ -147,6 +177,9 @@ const testActions = [
         successRate: 0.75,
         category: 'test',
         qeDomain: 'test-generation',
+        // A14: test-generation only generates; there's no "repair a failing
+        // test" capability anywhere in src/domains.
+        implemented: false,
     },
     {
         name: 'fix-flaky-tests',
@@ -159,6 +192,12 @@ const testActions = [
         successRate: 0.7,
         category: 'test',
         qeDomain: 'test-execution',
+        // A14: detectFlaky() only detects, doesn't stabilize — name/effects
+        // ("fix"/-3 flaky count) overstate what this binding actually does;
+        // still real, non-fabricated detection work, so kept implemented.
+        method: 'detectFlaky', // plugin.ts:54
+        params: {},
+        implemented: true,
     },
     {
         name: 'generate-property-tests',
@@ -171,6 +210,9 @@ const testActions = [
         successRate: 0.82,
         category: 'test',
         qeDomain: 'test-generation',
+        method: 'generatePropertyTests', // coordinator.ts:759
+        params: {},
+        implemented: true,
     },
     {
         name: 'run-contract-tests',
@@ -183,6 +225,9 @@ const testActions = [
         successRate: 0.88,
         category: 'test',
         qeDomain: 'contract-testing',
+        method: 'verifyAllConsumers', // plugin.ts:163
+        params: {},
+        implemented: true,
     },
 ];
 // ============================================================================
@@ -200,6 +245,9 @@ const securityActions = [
         successRate: 0.95,
         category: 'security',
         qeDomain: 'security-compliance',
+        method: 'runSecurityAudit', // plugin.ts:142
+        params: {},
+        implemented: true,
     },
     {
         name: 'fix-vulnerabilities',
@@ -218,6 +266,9 @@ const securityActions = [
         successRate: 0.7,
         category: 'security',
         qeDomain: 'security-compliance',
+        // A14: only triageVulnerabilities() (identification) exists — no
+        // remediation/dependency-update capability anywhere in src/domains.
+        implemented: false,
     },
     {
         name: 'owasp-audit',
@@ -230,6 +281,9 @@ const securityActions = [
         successRate: 0.85,
         category: 'security',
         qeDomain: 'security-compliance',
+        method: 'runComplianceCheck', // plugin.ts:147
+        params: { standard: 'owasp' },
+        implemented: true,
     },
     {
         name: 'scan-secrets',
@@ -242,6 +296,11 @@ const securityActions = [
         successRate: 0.98,
         category: 'security',
         qeDomain: 'security-compliance',
+        // A14: no dedicated secrets-scan method — includeSecrets is a flag on
+        // runSecurityAudit (plugin.ts:179), not a standalone capability.
+        method: 'runSecurityAudit',
+        params: { includeSecrets: true },
+        implemented: true,
     },
     {
         name: 'run-dast',
@@ -254,6 +313,9 @@ const securityActions = [
         successRate: 0.8,
         category: 'security',
         qeDomain: 'security-compliance',
+        method: 'runDASTScan', // plugin.ts:145
+        params: {},
+        implemented: true,
     },
 ];
 // ============================================================================
@@ -271,6 +333,9 @@ const performanceActions = [
         successRate: 0.9,
         category: 'performance',
         qeDomain: 'chaos-resilience',
+        // A14: no baseline-benchmark concept exists — load testing is
+        // semantically different (see load-test below), not a substitute.
+        implemented: false,
     },
     {
         name: 'load-test',
@@ -283,6 +348,9 @@ const performanceActions = [
         successRate: 0.85,
         category: 'performance',
         qeDomain: 'chaos-resilience',
+        method: 'runLoadTestSuite', // plugin.ts:168
+        params: {},
+        implemented: true,
     },
     {
         name: 'stress-test',
@@ -298,6 +366,12 @@ const performanceActions = [
         successRate: 0.8,
         category: 'performance',
         qeDomain: 'chaos-resilience',
+        // A14: real chaos-resilience test path (plugin.ts runTest), typed as
+        // 'stress' — createTest() is a separate prep step this binding doesn't
+        // perform, so this may error without a pre-created test id.
+        method: 'runTest',
+        params: { type: 'stress' },
+        implemented: true,
     },
     {
         name: 'profile-memory',
@@ -310,6 +384,8 @@ const performanceActions = [
         successRate: 0.88,
         category: 'performance',
         qeDomain: 'chaos-resilience',
+        // A14: no memory-profiling capability exists anywhere in src/domains.
+        implemented: false,
     },
     {
         name: 'optimize-slow-tests',
@@ -322,6 +398,8 @@ const performanceActions = [
         successRate: 0.82,
         category: 'performance',
         qeDomain: 'test-execution',
+        // A14: getStats() reports timing only — no "optimize" action exists.
+        implemented: false,
     },
 ];
 // ============================================================================
@@ -338,7 +416,13 @@ const analysisActions = [
         estimatedDurationMs: 15000,
         successRate: 0.95,
         category: 'analysis',
-        qeDomain: 'code-intelligence',
+        // A14: fixed domain-assignment bug — analyzeComplexity() actually lives
+        // on quality-assessment (plugin.ts:393), not code-intelligence, which
+        // has no complexity/smell/technical-debt methods at all.
+        qeDomain: 'quality-assessment',
+        method: 'analyzeComplexity',
+        params: {},
+        implemented: true,
     },
     {
         name: 'detect-code-smells',
@@ -351,6 +435,8 @@ const analysisActions = [
         successRate: 0.9,
         category: 'analysis',
         qeDomain: 'code-intelligence',
+        // A14: no code-smell/anti-pattern detector exists anywhere in src/domains.
+        implemented: false,
     },
     {
         name: 'analyze-dependencies',
@@ -363,6 +449,9 @@ const analysisActions = [
         successRate: 0.92,
         category: 'analysis',
         qeDomain: 'code-intelligence',
+        method: 'mapDependencies', // plugin.ts:373
+        params: {},
+        implemented: true,
     },
     {
         name: 'measure-technical-debt',
@@ -378,6 +467,8 @@ const analysisActions = [
         successRate: 0.88,
         category: 'analysis',
         qeDomain: 'code-intelligence',
+        // A14: no technical-debt measurement capability exists anywhere in src/domains.
+        implemented: false,
     },
     {
         name: 'build-knowledge-graph',
@@ -390,6 +481,9 @@ const analysisActions = [
         successRate: 0.85,
         category: 'analysis',
         qeDomain: 'code-intelligence',
+        method: 'index', // plugin.ts:317 — real KG/semantic index build
+        params: {},
+        implemented: true,
     },
 ];
 // ============================================================================
@@ -407,6 +501,9 @@ const fleetActions = [
         successRate: 0.95,
         category: 'fleet',
         qeDomain: 'test-execution',
+        // A14: agent spawning is a Queen/fleet coordinator concern, not exposed
+        // via any domain API method — no real binding exists for this today.
+        implemented: false,
     },
     {
         name: 'optimize-fleet-topology',
@@ -418,6 +515,8 @@ const fleetActions = [
         estimatedDurationMs: 10000,
         successRate: 0.85,
         category: 'fleet',
+        // A14: no qeDomain — a pure fleet/Queen concern with no domain API to bind to.
+        implemented: false,
     },
     {
         name: 'scale-down-fleet',
@@ -429,6 +528,7 @@ const fleetActions = [
         estimatedDurationMs: 3000,
         successRate: 0.98,
         category: 'fleet',
+        implemented: false,
     },
     {
         name: 'spawn-specialist-agent',
@@ -446,6 +546,7 @@ const fleetActions = [
         estimatedDurationMs: 8000,
         successRate: 0.9,
         category: 'fleet',
+        implemented: false,
     },
     {
         name: 'rebalance-workload',
@@ -457,6 +558,7 @@ const fleetActions = [
         estimatedDurationMs: 5000,
         successRate: 0.92,
         category: 'fleet',
+        implemented: false,
     },
 ];
 // ============================================================================
@@ -477,6 +579,9 @@ const qualityActions = [
         successRate: 0.95,
         category: 'quality',
         qeDomain: 'quality-assessment',
+        method: 'analyzeQuality', // plugin.ts:353
+        params: {},
+        implemented: true,
     },
     {
         name: 'enforce-quality-gate',
@@ -489,6 +594,9 @@ const qualityActions = [
         successRate: 0.9,
         category: 'quality',
         qeDomain: 'quality-assessment',
+        method: 'evaluateGate', // plugin.ts:333
+        params: {},
+        implemented: true,
     },
     {
         name: 'validate-requirements',
@@ -501,6 +609,9 @@ const qualityActions = [
         successRate: 0.88,
         category: 'quality',
         qeDomain: 'requirements-validation',
+        method: 'validate', // plugin.ts:427
+        params: {},
+        implemented: true,
     },
     {
         name: 'predict-defects',
@@ -516,6 +627,9 @@ const qualityActions = [
         successRate: 0.82,
         category: 'quality',
         qeDomain: 'defect-intelligence',
+        method: 'predictDefects', // plugin.ts:366
+        params: {},
+        implemented: true,
     },
     {
         name: 'generate-deployment-report',
@@ -531,6 +645,9 @@ const qualityActions = [
         successRate: 0.9,
         category: 'quality',
         qeDomain: 'quality-assessment',
+        method: 'getDeploymentAdvice', // plugin.ts:373
+        params: {},
+        implemented: true,
     },
 ];
 // ============================================================================
@@ -548,6 +665,9 @@ const accessibilityActions = [
         successRate: 0.9,
         category: 'quality',
         qeDomain: 'visual-accessibility',
+        method: 'runAccessibilityAudit', // plugin.ts:143
+        params: {},
+        implemented: true,
     },
     {
         name: 'run-visual-regression',
@@ -560,6 +680,9 @@ const accessibilityActions = [
         successRate: 0.85,
         category: 'quality',
         qeDomain: 'visual-accessibility',
+        method: 'runVisualTests', // plugin.ts:142
+        params: {},
+        implemented: true,
     },
 ];
 // ============================================================================

package/dist/planning/goap-planner.d.ts

@@ -63,6 +63,18 @@ export declare class GOAPPlanner {
      * Uses direct DB insertion to avoid recursive initialize() calls
      */
     private seedDefaultActions;
+    /**
+     * A14: backfill method/params/implemented onto already-seeded action rows,
+     * matched by `name` against the current qe-action-library.ts definitions.
+     *
+     * Action ids are randomly generated at seed time (`action-<timestamp>-
+     * <uuid>`), not stable across the library source, so this can't match by
+     * id. Only touches rows currently missing a binding the library now has
+     * (`method IS NULL AND library.implemented`), so it's cheap and
+     * idempotent on every initialize() call — and picks up newly-added real
+     * bindings on already-seeded databases without requiring a full re-seed.
+     */
+    private backfillActionMethodBindings;
     /**
      * Find an optimal plan using A* search
      *
@@ -146,7 +158,7 @@ export declare class GOAPPlanner {
     /**
      * Update action statistics after execution
      */
-    updateActionStats(actionId: string, success: boolean, durationMs: number): Promise<void>;
+    updateActionStats(actionId: string, success: boolean, _durationMs: number): Promise<void>;
     /**
      * Get actions by category
      */
@@ -166,7 +178,7 @@ export declare class GOAPPlanner {
     /**
      * Find a similar plan by goal conditions
      */
-    findSimilarPlan(goal: StateConditions, similarityThreshold?: number): Promise<GOAPPlan | null>;
+    findSimilarPlan(goal: StateConditions, _similarityThreshold?: number): Promise<GOAPPlan | null>;
     /**
      * Store plan signature for future reuse
      */

package/dist/planning/goap-planner.js

@@ -149,6 +149,13 @@ export class GOAPPlanner {
         if (actionCount.count === 0) {
             this.seedDefaultActions();
         }
+        // A14: backfill real method/params/implemented bindings onto
+        // already-seeded rows (matched by name — seed-time ids are random, not
+        // stable across the qe-action-library.ts source) BEFORE loading into
+        // the in-memory cache, so this.actions reflects current bindings even
+        // for databases seeded before this feature existed, or before a given
+        // action's binding was added to the library.
+        this.backfillActionMethodBindings();
         await this.loadActions();
         this.initialized = true;
         console.log(`[GOAPPlanner] Initialized: ${this.persistence.getDbPath()}`);
@@ -162,12 +169,12 @@ export class GOAPPlanner {
         const allActions = getAllQEActions();
         const db = this.ensureDb();
         const insertAction = db.prepare(`
-      INSERT INTO goap_actions (id, name, description, category, preconditions, effects, cost, qe_domain, agent_type)
-      VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?)
+      INSERT INTO goap_actions (id, name, description, category, preconditions, effects, cost, qe_domain, agent_type, method, params, implemented)
+      VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?)
     `);
         for (const action of allActions) {
             const id = `action-${Date.now()}-${randomUUID().slice(0, 8)}`;
-            insertAction.run(id, action.name, action.description, action.category, JSON.stringify(action.preconditions), JSON.stringify(action.effects), action.cost, action.qeDomain ?? null, action.agentType ?? null);
+            insertAction.run(id, action.name, action.description, action.category, JSON.stringify(action.preconditions), JSON.stringify(action.effects), action.cost, action.qeDomain ?? null, action.agentType ?? null, action.method ?? null, action.params ? JSON.stringify(action.params) : null, action.implemented ? 1 : 0);
         }
         // Seed QE goals using direct insertion
         const insertGoal = db.prepare(`
@@ -181,6 +188,29 @@ export class GOAPPlanner {
         // eslint-disable-next-line no-console
         console.log(`[GOAPPlanner] Seeded ${allActions.length} actions and ${QE_GOALS.length} goals`);
     }
+    /**
+     * A14: backfill method/params/implemented onto already-seeded action rows,
+     * matched by `name` against the current qe-action-library.ts definitions.
+     *
+     * Action ids are randomly generated at seed time (`action-<timestamp>-
+     * <uuid>`), not stable across the library source, so this can't match by
+     * id. Only touches rows currently missing a binding the library now has
+     * (`method IS NULL AND library.implemented`), so it's cheap and
+     * idempotent on every initialize() call — and picks up newly-added real
+     * bindings on already-seeded databases without requiring a full re-seed.
+     */
+    backfillActionMethodBindings() {
+        const db = this.ensureDb();
+        const update = db.prepare(`
+      UPDATE goap_actions SET method = ?, params = ?, implemented = 1
+      WHERE name = ? AND (method IS NULL OR implemented = 0)
+    `);
+        for (const action of getAllQEActions()) {
+            if (!action.implemented || !action.method)
+                continue;
+            update.run(action.method, action.params ? JSON.stringify(action.params) : '{}', action.name);
+        }
+    }
     // ==========================================================================
     // Core Planning - A* Search
     // ==========================================================================
@@ -201,13 +231,18 @@ export class GOAPPlanner {
             if (reusedPlan && this.validatePlanForState(reusedPlan, currentState)) {
                 // Update reuse stats
                 this.recordPlanReuse(reusedPlan.id, true);
-                return {
+                const clonedPlan = {
                     ...reusedPlan,
                     id: `plan-${Date.now()}-${randomUUID().slice(0, 8)}`,
                     initialState: this.cloneState(currentState),
                     reusedFrom: reusedPlan.id,
                     status: 'pending',
                 };
+                // A14: persist so goap_execute can find this plan by id — findPlan()
+                // previously only ever stored a reuse-lookup signature, never the
+                // plan itself, so every returned planId 404'd on execute.
+                await this.savePlan(clonedPlan);
+                return clonedPlan;
             }
         }
         // Get available actions
@@ -227,6 +262,9 @@ export class GOAPPlanner {
             estimatedDurationMs: actionSequence.reduce((sum, a) => sum + (a.estimatedDurationMs ?? 0), 0),
             status: 'pending',
         };
+        // A14: persist the plan itself (not just its reuse signature) so
+        // goap_execute can find it by id — see savePlan() below.
+        await this.savePlan(plan);
         // Store plan signature for future reuse
         await this.storePlanSignature(plan);
         const elapsedMs = Date.now() - startTime;
@@ -629,6 +667,9 @@ export class GOAPPlanner {
                 executionCount: row.execution_count,
                 category: row.category,
                 qeDomain: row.qe_domain,
+                method: row.method ?? undefined,
+                params: row.params ? safeJsonParse(row.params) : undefined,
+                implemented: row.implemented === 1,
             };
             this.actions.set(action.id, action);
         }
@@ -643,10 +684,11 @@ export class GOAPPlanner {
             .prepare(`
       INSERT INTO goap_actions (
         id, name, description, agent_type, preconditions, effects,
-        cost, estimated_duration_ms, success_rate, execution_count, category, qe_domain
-      ) VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?, 0, ?, ?)
+        cost, estimated_duration_ms, success_rate, execution_count, category, qe_domain,
+        method, params, implemented
+      ) VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?, 0, ?, ?, ?, ?, ?)
     `)
-            .run(id, action.name, action.description ?? null, action.agentType, JSON.stringify(action.preconditions), JSON.stringify(action.effects), action.cost, action.estimatedDurationMs ?? null, action.successRate, action.category, action.qeDomain ?? null);
+            .run(id, action.name, action.description ?? null, action.agentType, JSON.stringify(action.preconditions), JSON.stringify(action.effects), action.cost, action.estimatedDurationMs ?? null, action.successRate, action.category, action.qeDomain ?? null, action.method ?? null, action.params ? JSON.stringify(action.params) : null, action.implemented ? 1 : 0);
         // Update in-memory cache
         const fullAction = {
             ...action,
@@ -659,7 +701,7 @@ export class GOAPPlanner {
     /**
      * Update action statistics after execution
      */
-    async updateActionStats(actionId, success, durationMs) {
+    async updateActionStats(actionId, success, _durationMs) {
         await this.initialize();
         const action = this.actions.get(actionId);
         if (!action)
@@ -745,7 +787,7 @@ export class GOAPPlanner {
     /**
      * Find a similar plan by goal conditions
      */
-    async findSimilarPlan(goal, similarityThreshold = 0.75) {
+    async findSimilarPlan(goal, _similarityThreshold = 0.75) {
         await this.initialize();
         const goalHash = this.hashGoalConditions(goal);
         // Look for exact goal match first

package/dist/planning/index.d.ts

@@ -12,5 +12,5 @@ export { QE_ACTIONS, QE_GOALS, getAllQEActions, getActionsByCategory, getActions
 export type { QEActionTemplate, QEGoalTemplate } from './actions/qe-action-library.js';
 export { GOAPPlanner, getSharedGOAPPlanner, resetSharedGOAPPlanner, } from './goap-planner.js';
 export { PlanExecutor, MockAgentSpawner, ClaudeFlowSpawner, createMockExecutor, createClaudeFlowExecutor, } from './plan-executor.js';
-export type { ExecutionConfig, ExecutionResult, ExecutedStep, AgentSpawner, AgentSpawnResult, } from './plan-executor.js';
+export type { ExecutionConfig, ExecutionResult, ExecutedStep, AgentSpawner, AgentSpawnResult, GetDomainAPIFn, } from './plan-executor.js';
 //# sourceMappingURL=index.d.ts.map