@syntesseraai/opencode-feature-factory 0.1.15 → 0.1.17

package/package.json

@@ -1,7 +1,7 @@
 {
   "$schema": "https://json.schemastore.org/package.json",
   "name": "@syntesseraai/opencode-feature-factory",
-  "version": "0.1.15",
+  "version": "0.1.17",
   "description": "OpenCode plugin for Feature Factory agents - provides planning, implementation, review, testing, and validation agents",
   "type": "module",
   "license": "MIT",

package/src/discovery.ts

@@ -6,8 +6,6 @@ import {
   readJsonFile,
 } from './quality-gate-config';
 
-// Shell type - uses `any` to be compatible with OpenCode's BunShell
-// eslint-disable-next-line @typescript-eslint/no-explicit-any
 type BunShell = any;
 
 // ============================================================================

package/src/index.ts

@@ -100,17 +100,11 @@ function mergeHooks(...hookSets: Partial<Hooks>[]): Hooks {
 
       const existingHandler = merged[key as keyof Hooks];
       if (existingHandler) {
-        // Chain handlers - run existing first, then new
-        // eslint-disable-next-line @typescript-eslint/no-explicit-any
         merged[key as keyof Hooks] = (async (...args: any[]) => {
-          // eslint-disable-next-line @typescript-eslint/no-explicit-any
           await (existingHandler as (...args: any[]) => Promise<void>)(...args);
-          // eslint-disable-next-line @typescript-eslint/no-explicit-any
           await (handler as (...args: any[]) => Promise<void>)(...args);
-          // eslint-disable-next-line @typescript-eslint/no-explicit-any
         }) as any;
       } else {
-        // eslint-disable-next-line @typescript-eslint/no-explicit-any
         merged[key as keyof Hooks] = handler as any;
       }
     }

package/src/quality-gate-config.ts

@@ -17,8 +17,6 @@ export const DEFAULT_QUALITY_GATE: Required<
   },
 };
 
-// Shell type - uses `any` to be compatible with OpenCode's BunShell
-// eslint-disable-next-line @typescript-eslint/no-explicit-any
 type BunShell = any;
 
 /**

package/src/stop-quality-gate.ts

@@ -8,6 +8,7 @@ interface SessionState {
   dirty: boolean;
   qualityGatePassed: boolean;
   idleDebounce: ReturnType<typeof setTimeout> | null;
+  running: boolean;
   parentID?: string;
   isReadOnly?: boolean;
 }
@@ -23,6 +24,7 @@ function getSessionState(sessionId: string): SessionState {
     dirty: true,
     qualityGatePassed: false,
     idleDebounce: null,
+    running: false,
   };
   sessions.set(sessionId, state);
   return state;
@@ -75,7 +77,7 @@ export async function createQualityGateHooks(input: PluginInput): Promise<Partia
     }
   }
 
-  async function ensureSessionMetadata(sessionId: string, state: SessionState): Promise<void> {
+  async function _ensureSessionMetadata(sessionId: string, state: SessionState): Promise<void> {
     if (state.isReadOnly !== undefined) return;
 
     try {
@@ -104,6 +106,11 @@ export async function createQualityGateHooks(input: PluginInput): Promise<Partia
       return;
     }
 
+    if (state.running) {
+      await log(client, 'debug', 'quality-gate.skipped (already running)', { sessionId });
+      return;
+    }
+
     const now = Date.now();
     const cacheExpired = now - state.lastRunAt > 30 * 1000;
 
@@ -119,19 +126,9 @@ export async function createQualityGateHooks(input: PluginInput): Promise<Partia
       return;
     }
 
-    await log(client, 'info', 'quality-gate.started', { sessionId, directory });
+    state.running = true;
 
-    await client.session.prompt({
-      path: { id: sessionId },
-      body: {
-        parts: [
-          {
-            type: 'text',
-            text: `🔄 Quality gate is running, please stand-by for results ...`,
-          },
-        ],
-      },
-    });
+    await log(client, 'info', 'quality-gate.started', { sessionId, directory });
 
     let ciOutput = '';
     let ciPassed = false;
@@ -150,21 +147,9 @@ export async function createQualityGateHooks(input: PluginInput): Promise<Partia
     state.lastRunAt = Date.now();
     state.dirty = false;
     state.qualityGatePassed = ciPassed;
+    state.running = false;
 
-    if (ciPassed) {
-      await client.session.prompt({
-        path: { id: sessionId },
-        body: {
-          parts: [
-            {
-              type: 'text',
-              text: `✅ Quality gate passed`,
-            },
-          ],
-        },
-      });
-      await log(client, 'debug', 'quality-gate.passed', { sessionId });
-    } else {
+    if (!ciPassed) {
       await client.session.prompt({
         path: { id: sessionId },
         body: {
@@ -201,6 +186,7 @@ export async function createQualityGateHooks(input: PluginInput): Promise<Partia
             dirty: true,
             qualityGatePassed: false,
             idleDebounce: null,
+            running: false,
             parentID: sessionInfo?.parentID,
             isReadOnly,
           });