@syntesseraai/opencode-feature-factory 0.2.7 → 0.2.8

package/dist/discovery.js

@@ -1,4 +1,7 @@
-import { DEFAULT_QUALITY_GATE, fileExists, hasConfiguredCommands, readJsonFile, } from './quality-gate-config';
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.resolveCommands = resolveCommands;
+const quality_gate_config_1 = require("./quality-gate-config");
 // ============================================================================
 // Package Manager Detection
 // ============================================================================
@@ -11,15 +14,15 @@ async function detectPackageManager($, directory, override) {
         return override;
     }
     // Priority order: pnpm > bun > yarn > npm
-    if (await fileExists($, `${directory}/pnpm-lock.yaml`))
+    if (await (0, quality_gate_config_1.fileExists)($, `${directory}/pnpm-lock.yaml`))
         return 'pnpm';
-    if (await fileExists($, `${directory}/bun.lockb`))
+    if (await (0, quality_gate_config_1.fileExists)($, `${directory}/bun.lockb`))
         return 'bun';
-    if (await fileExists($, `${directory}/bun.lock`))
+    if (await (0, quality_gate_config_1.fileExists)($, `${directory}/bun.lock`))
         return 'bun';
-    if (await fileExists($, `${directory}/yarn.lock`))
+    if (await (0, quality_gate_config_1.fileExists)($, `${directory}/yarn.lock`))
         return 'yarn';
-    if (await fileExists($, `${directory}/package-lock.json`))
+    if (await (0, quality_gate_config_1.fileExists)($, `${directory}/package-lock.json`))
         return 'npm';
     return 'npm'; // fallback
 }
@@ -45,7 +48,7 @@ function buildRunCommand(pm, script) {
  * Discover Node.js lint/build/test commands from package.json scripts
  */
 async function discoverNodeCommands($, directory, pm) {
-    const pkgJson = await readJsonFile($, `${directory}/package.json`);
+    const pkgJson = await (0, quality_gate_config_1.readJsonFile)($, `${directory}/package.json`);
     if (!pkgJson?.scripts)
         return [];
     const scripts = pkgJson.scripts;
@@ -74,7 +77,7 @@ async function discoverNodeCommands($, directory, pm) {
  * Discover Rust commands from Cargo.toml presence
  */
 async function discoverRustCommands($, directory, includeClippy) {
-    if (!(await fileExists($, `${directory}/Cargo.toml`)))
+    if (!(await (0, quality_gate_config_1.fileExists)($, `${directory}/Cargo.toml`)))
         return [];
     const steps = [{ step: 'lint (fmt)', cmd: 'cargo fmt --check' }];
     if (includeClippy) {
@@ -94,7 +97,7 @@ async function discoverRustCommands($, directory, includeClippy) {
  * Discover Go commands from go.mod presence
  */
 async function discoverGoCommands($, directory) {
-    if (!(await fileExists($, `${directory}/go.mod`)))
+    if (!(await (0, quality_gate_config_1.fileExists)($, `${directory}/go.mod`)))
         return [];
     return [{ step: 'test', cmd: 'go test ./...' }];
 }
@@ -106,8 +109,8 @@ async function discoverGoCommands($, directory) {
  */
 async function discoverPythonCommands($, directory) {
     // Only add pytest if we have strong signal
-    const hasPytestIni = await fileExists($, `${directory}/pytest.ini`);
-    const hasPyproject = await fileExists($, `${directory}/pyproject.toml`);
+    const hasPytestIni = await (0, quality_gate_config_1.fileExists)($, `${directory}/pytest.ini`);
+    const hasPyproject = await (0, quality_gate_config_1.fileExists)($, `${directory}/pyproject.toml`);
     if (!hasPytestIni && !hasPyproject)
         return [];
     // pytest.ini is strong signal
@@ -142,11 +145,11 @@ async function discoverPythonCommands($, directory) {
  *
  * @returns Array of command steps to execute, or empty if nothing found
  */
-export async function resolveCommands(args) {
+async function resolveCommands(args) {
     const { $, directory, config } = args;
-    const mergedConfig = { ...DEFAULT_QUALITY_GATE, ...config };
+    const mergedConfig = { ...quality_gate_config_1.DEFAULT_QUALITY_GATE, ...config };
     // 1. Configured commands take priority (do NOT run ci.sh if these exist)
-    if (hasConfiguredCommands(config)) {
+    if ((0, quality_gate_config_1.hasConfiguredCommands)(config)) {
         const steps = [];
         const order = mergedConfig.steps;
         for (const stepName of order) {
@@ -160,14 +163,14 @@ export async function resolveCommands(args) {
     // 2. Feature Factory CI script (only if no configured commands)
     if (mergedConfig.useCiSh !== 'never') {
         const ciShPath = `${directory}/management/ci.sh`;
-        if (await fileExists($, ciShPath)) {
+        if (await (0, quality_gate_config_1.fileExists)($, ciShPath)) {
             return [{ step: 'ci', cmd: `bash ${ciShPath}` }];
         }
     }
     // 3. Conventional discovery (only if no ci.sh)
     const pm = await detectPackageManager($, directory, mergedConfig.packageManager);
     // Try Node first (most common)
-    if (await fileExists($, `${directory}/package.json`)) {
+    if (await (0, quality_gate_config_1.fileExists)($, `${directory}/package.json`)) {
         const nodeSteps = await discoverNodeCommands($, directory, pm);
         if (nodeSteps.length > 0)
             return nodeSteps;

package/dist/discovery.test.js

@@ -1,3 +1,4 @@
+"use strict";
 /**
  * Unit tests for discovery module
  *
@@ -7,6 +8,7 @@
  * Note: Most functions in discovery.ts require shell access ($) so they're
  * integration-tested elsewhere. This file tests the pure utility functions.
  */
+Object.defineProperty(exports, "__esModule", { value: true });
 // Re-implement buildRunCommand for testing since it's not exported
 function buildRunCommand(pm, script) {
     switch (pm) {
@@ -92,4 +94,3 @@ describe('PackageManager type', () => {
         });
     });
 });
-export {};

package/dist/index.d.ts

@@ -1,4 +1,6 @@
-import type { Plugin } from '@opencode-ai/plugin';
+import type { Plugin } from '@opencode-ai/plugin' with {
+    'resolution-mode': 'import'
+};
 export declare const SKILL_PATHS: {
     'ff-mini-plan': string;
     'ff-todo-management': string;

package/dist/index.js

@@ -1,17 +1,20 @@
-import { createQualityGateHooks } from './stop-quality-gate';
+'use strict';
+Object.defineProperty(exports, '__esModule', { value: true });
+exports.StopQualityGatePlugin = exports.AGENT_PATHS = exports.SKILL_PATHS = void 0;
+const stop_quality_gate_1 = require('./stop-quality-gate');
 // Export skill and agent paths for programmatic access
-export const SKILL_PATHS = {
-    'ff-mini-plan': './skills/ff-mini-plan/SKILL.md',
-    'ff-todo-management': './skills/ff-todo-management/SKILL.md',
-    'ff-severity-classification': './skills/ff-severity-classification/SKILL.md',
-    'ff-report-templates': './skills/ff-report-templates/SKILL.md',
+exports.SKILL_PATHS = {
+  'ff-mini-plan': './skills/ff-mini-plan/SKILL.md',
+  'ff-todo-management': './skills/ff-todo-management/SKILL.md',
+  'ff-severity-classification': './skills/ff-severity-classification/SKILL.md',
+  'ff-report-templates': './skills/ff-report-templates/SKILL.md',
 };
-export const AGENT_PATHS = {
-    'ff-acceptance': './agents/ff-acceptance.md',
-    'ff-review': './agents/ff-review.md',
-    'ff-security': './agents/ff-security.md',
-    'ff-well-architected': './agents/ff-well-architected.md',
-    'ff-validate': './agents/ff-validate.md',
+exports.AGENT_PATHS = {
+  'ff-acceptance': './agents/ff-acceptance.md',
+  'ff-review': './agents/ff-review.md',
+  'ff-security': './agents/ff-security.md',
+  'ff-well-architected': './agents/ff-well-architected.md',
+  'ff-validate': './agents/ff-validate.md',
 };
 const SERVICE_NAME = 'feature-factory';
 /**
@@ -19,29 +22,27 @@ const SERVICE_NAME = 'feature-factory';
  * Silently fails if logging is unavailable.
  */
 async function log(client, level, message, extra) {
-    try {
-        await client.app.log({
-            body: {
-                service: SERVICE_NAME,
-                level,
-                message,
-                extra,
-            },
-        });
-    }
-    catch {
-        // Logging failure should not affect plugin operation
-    }
+  try {
+    await client.app.log({
+      body: {
+        service: SERVICE_NAME,
+        level,
+        message,
+        extra,
+      },
+    });
+  } catch {
+    // Logging failure should not affect plugin operation
+  }
 }
 /**
  * Determine the project root directory.
  * Prefer worktree (git root) if available, otherwise use directory.
  */
 function resolveRootDir(input) {
-    const worktree = (input.worktree ?? '').trim();
-    if (worktree.length > 0)
-        return worktree;
-    return input.directory;
+  const worktree = (input.worktree ?? '').trim();
+  if (worktree.length > 0) return worktree;
+  return input.directory;
 }
 /**
  * Stop Quality Gate Plugin
@@ -56,26 +57,28 @@ function resolveRootDir(input) {
  * - On failure: passes full CI output to LLM for fix instructions
  * - If management/ci.sh does not exist, quality gate does not run
  */
-export const StopQualityGatePlugin = async (input) => {
-    const { worktree, directory, client } = input;
-    const rootDir = resolveRootDir({ worktree, directory });
-    // Skip quality gate if no valid directory (e.g., global config with no project)
-    if (!rootDir || rootDir === '' || rootDir === '/') {
-        return {};
-    }
-    // Create quality gate hooks
-    let qualityGateHooks = {};
-    try {
-        qualityGateHooks = await createQualityGateHooks(input);
-    }
-    catch (error) {
-        await log(client, 'error', 'quality-gate.init-error', {
-            error: String(error),
-        });
-    }
-    return {
-        ...qualityGateHooks,
-    };
+const StopQualityGatePlugin = async (input) => {
+  const { worktree, directory, client } = input;
+  const rootDir = resolveRootDir({ worktree, directory });
+  // Skip quality gate if no valid directory (e.g., global config with no project)
+  if (!rootDir || rootDir === '' || rootDir === '/') {
+    return {};
+  }
+  // Create quality gate hooks
+  let qualityGateHooks = {};
+  try {
+    qualityGateHooks = await (0, stop_quality_gate_1.createQualityGateHooks)(input);
+  } catch (error) {
+    await log(client, 'error', 'quality-gate.init-error', {
+      error: String(error),
+    });
+  }
+  return {
+    ...qualityGateHooks,
+  };
 };
+exports.StopQualityGatePlugin = StopQualityGatePlugin;
 // Default export for OpenCode plugin discovery
-export default StopQualityGatePlugin;
+exports.default = exports.StopQualityGatePlugin;
+// Make module directly callable for CommonJS compatibility
+module.exports = StopQualityGatePlugin;

package/dist/output.js

@@ -1,3 +1,7 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.extractErrorLines = extractErrorLines;
+exports.tailLines = tailLines;
 /**
  * Regex pattern for detecting error-like lines in command output
  */
@@ -10,7 +14,7 @@ const ERROR_PATTERNS = /(error|failed|failure|panic|assert|exception|traceback|T
  * @param maxLines - Maximum number of error lines to extract
  * @returns Array of error-like lines (up to maxLines)
  */
-export function extractErrorLines(output, maxLines) {
+function extractErrorLines(output, maxLines) {
     const lines = output.split('\n');
     const errorLines = [];
     for (let i = 0; i < lines.length && errorLines.length < maxLines; i++) {
@@ -37,7 +41,7 @@ export function extractErrorLines(output, maxLines) {
  * @param maxLines - Maximum number of lines to return
  * @returns Truncated output with notice, or full output if short enough
  */
-export function tailLines(output, maxLines) {
+function tailLines(output, maxLines) {
     const lines = output.split('\n');
     if (lines.length <= maxLines) {
         return output;

package/dist/output.test.js

@@ -1,3 +1,4 @@
+"use strict";
 /**
  * Unit tests for output module
  *
@@ -5,10 +6,11 @@
  * - extractErrorLines: extracts error-like lines from command output
  * - tailLines: returns last N lines with truncation notice
  */
-import { extractErrorLines, tailLines } from './output';
+Object.defineProperty(exports, "__esModule", { value: true });
+const output_1 = require("./output");
 describe('extractErrorLines', () => {
     it('should return empty array for empty input', () => {
-        expect(extractErrorLines('', 10)).toEqual([]);
+        expect((0, output_1.extractErrorLines)('', 10)).toEqual([]);
     });
     it('should return empty array when no errors found', () => {
         const output = `
@@ -16,7 +18,7 @@ Build started
 Compiling modules...
 Build completed successfully
 `;
-        expect(extractErrorLines(output, 10)).toEqual([]);
+        expect((0, output_1.extractErrorLines)(output, 10)).toEqual([]);
     });
     it('should extract lines containing "error"', () => {
         const output = `
@@ -24,7 +26,7 @@ Starting build
 error: Cannot find module 'missing'
 Build failed
 `;
-        const result = extractErrorLines(output, 10);
+        const result = (0, output_1.extractErrorLines)(output, 10);
         expect(result.some((line) => line.includes('error:'))).toBe(true);
     });
     it('should extract lines containing "Error" (case insensitive)', () => {
@@ -33,7 +35,7 @@ Running tests
 TypeError: undefined is not a function
   at test.js:10:5
 `;
-        const result = extractErrorLines(output, 10);
+        const result = (0, output_1.extractErrorLines)(output, 10);
         expect(result.some((line) => line.includes('TypeError'))).toBe(true);
     });
     it('should extract lines containing "failed"', () => {
@@ -42,7 +44,7 @@ Test suite: MyTests
 Test failed: should work correctly
 1 test failed
 `;
-        const result = extractErrorLines(output, 10);
+        const result = (0, output_1.extractErrorLines)(output, 10);
         expect(result.some((line) => line.toLowerCase().includes('failed'))).toBe(true);
     });
     it('should extract lines containing "FAILED"', () => {
@@ -50,7 +52,7 @@ Test failed: should work correctly
 Running: test_function
 FAILED test_function - assertion error
 `;
-        const result = extractErrorLines(output, 10);
+        const result = (0, output_1.extractErrorLines)(output, 10);
         expect(result.some((line) => line.includes('FAILED'))).toBe(true);
     });
     it('should extract lines containing "panic"', () => {
@@ -58,7 +60,7 @@ FAILED test_function - assertion error
 thread 'main' panicked at 'assertion failed'
 note: run with RUST_BACKTRACE=1
 `;
-        const result = extractErrorLines(output, 10);
+        const result = (0, output_1.extractErrorLines)(output, 10);
         expect(result.some((line) => line.includes('panic'))).toBe(true);
     });
     it('should extract lines containing "exception"', () => {
@@ -66,7 +68,7 @@ note: run with RUST_BACKTRACE=1
 Exception in thread "main" java.lang.NullPointerException
   at Main.main(Main.java:5)
 `;
-        const result = extractErrorLines(output, 10);
+        const result = (0, output_1.extractErrorLines)(output, 10);
         expect(result.some((line) => line.includes('Exception'))).toBe(true);
     });
     it('should extract lines containing "traceback"', () => {
@@ -75,7 +77,7 @@ Traceback (most recent call last):
   File "test.py", line 10
 NameError: name 'x' is not defined
 `;
-        const result = extractErrorLines(output, 10);
+        const result = (0, output_1.extractErrorLines)(output, 10);
         expect(result.some((line) => line.includes('Traceback'))).toBe(true);
     });
     it('should include 2 lines of context after error', () => {
@@ -84,7 +86,7 @@ error: something went wrong
 context line 1
 context line 2
 line5`;
-        const result = extractErrorLines(output, 10);
+        const result = (0, output_1.extractErrorLines)(output, 10);
         expect(result).toContain('error: something went wrong');
         expect(result).toContain('context line 1');
         expect(result).toContain('context line 2');
@@ -101,36 +103,36 @@ error: third error
 context5
 context6
 `;
-        const result = extractErrorLines(output, 5);
+        const result = (0, output_1.extractErrorLines)(output, 5);
         expect(result.length).toBeLessThanOrEqual(5);
     });
     it('should handle error at end of output without enough context lines', () => {
         const output = `line1
 line2
 error: final error`;
-        const result = extractErrorLines(output, 10);
+        const result = (0, output_1.extractErrorLines)(output, 10);
         expect(result).toContain('error: final error');
     });
     it('should extract ReferenceError', () => {
         const output = `ReferenceError: x is not defined`;
-        const result = extractErrorLines(output, 10);
+        const result = (0, output_1.extractErrorLines)(output, 10);
         expect(result).toContain('ReferenceError: x is not defined');
     });
     it('should extract SyntaxError', () => {
         const output = `SyntaxError: Unexpected token '}'`;
-        const result = extractErrorLines(output, 10);
+        const result = (0, output_1.extractErrorLines)(output, 10);
         expect(result).toContain("SyntaxError: Unexpected token '}'");
     });
     it('should extract ERR! (npm style errors)', () => {
         const output = `npm ERR! code ENOENT
 npm ERR! path /app/package.json`;
-        const result = extractErrorLines(output, 10);
+        const result = (0, output_1.extractErrorLines)(output, 10);
         expect(result.some((line) => line.includes('ERR!'))).toBe(true);
     });
     it('should extract FAIL (test runner style)', () => {
         const output = `FAIL src/test.ts
   Test suite failed to run`;
-        const result = extractErrorLines(output, 10);
+        const result = (0, output_1.extractErrorLines)(output, 10);
         expect(result.some((line) => line.includes('FAIL'))).toBe(true);
     });
     it('should extract multiple error types in same output', () => {
@@ -141,7 +143,7 @@ TypeError: Cannot read property 'x'
 FAILED: test_something
 Build process ended
 `;
-        const result = extractErrorLines(output, 20);
+        const result = (0, output_1.extractErrorLines)(output, 20);
         expect(result.some((line) => line.includes('error:'))).toBe(true);
         expect(result.some((line) => line.includes('TypeError'))).toBe(true);
         expect(result.some((line) => line.includes('FAILED'))).toBe(true);
@@ -150,15 +152,15 @@ Build process ended
 describe('tailLines', () => {
     it('should return full output when lines count is less than maxLines', () => {
         const output = 'line1\nline2\nline3';
-        expect(tailLines(output, 10)).toBe(output);
+        expect((0, output_1.tailLines)(output, 10)).toBe(output);
     });
     it('should return full output when lines count equals maxLines', () => {
         const output = 'line1\nline2\nline3';
-        expect(tailLines(output, 3)).toBe(output);
+        expect((0, output_1.tailLines)(output, 3)).toBe(output);
     });
     it('should truncate and add notice when output exceeds maxLines', () => {
         const output = 'line1\nline2\nline3\nline4\nline5';
-        const result = tailLines(output, 3);
+        const result = (0, output_1.tailLines)(output, 3);
         expect(result).toContain('... (2 lines truncated)');
         expect(result).toContain('line3');
         expect(result).toContain('line4');
@@ -168,38 +170,38 @@ describe('tailLines', () => {
     });
     it('should handle single line output', () => {
         const output = 'single line';
-        expect(tailLines(output, 5)).toBe('single line');
+        expect((0, output_1.tailLines)(output, 5)).toBe('single line');
     });
     it('should handle empty output', () => {
-        expect(tailLines('', 5)).toBe('');
+        expect((0, output_1.tailLines)('', 5)).toBe('');
     });
     it('should handle maxLines of 1', () => {
         const output = 'line1\nline2\nline3';
-        const result = tailLines(output, 1);
+        const result = (0, output_1.tailLines)(output, 1);
         expect(result).toContain('... (2 lines truncated)');
         expect(result).toContain('line3');
     });
     it('should correctly count truncated lines', () => {
         const lines = Array.from({ length: 100 }, (_, i) => `line${i + 1}`);
         const output = lines.join('\n');
-        const result = tailLines(output, 10);
+        const result = (0, output_1.tailLines)(output, 10);
         expect(result).toContain('... (90 lines truncated)');
     });
     it('should preserve line content in tail', () => {
         const output = 'first\nsecond\nthird\nfourth\nfifth';
-        const result = tailLines(output, 2);
+        const result = (0, output_1.tailLines)(output, 2);
         const resultLines = result.split('\n');
         expect(resultLines[resultLines.length - 1]).toBe('fifth');
         expect(resultLines[resultLines.length - 2]).toBe('fourth');
     });
     it('should handle output with empty lines', () => {
         const output = 'line1\n\nline3\n\nline5';
-        const result = tailLines(output, 3);
+        const result = (0, output_1.tailLines)(output, 3);
         expect(result).toContain('... (2 lines truncated)');
     });
     it('should handle output with only newlines', () => {
         const output = '\n\n\n\n';
-        const result = tailLines(output, 2);
+        const result = (0, output_1.tailLines)(output, 2);
         expect(result).toContain('... (3 lines truncated)');
     });
 });

package/dist/quality-gate-config.js

@@ -1,7 +1,15 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.DEFAULT_QUALITY_GATE = void 0;
+exports.readJsonFile = readJsonFile;
+exports.fileExists = fileExists;
+exports.mergeQualityGateConfig = mergeQualityGateConfig;
+exports.loadQualityGateConfig = loadQualityGateConfig;
+exports.hasConfiguredCommands = hasConfiguredCommands;
 /**
  * Default configuration values for StopQualityGate
  */
-export const DEFAULT_QUALITY_GATE = {
+exports.DEFAULT_QUALITY_GATE = {
     steps: ['lint', 'build', 'test'],
     useCiSh: 'auto',
     packageManager: 'auto',
@@ -15,7 +23,7 @@ export const DEFAULT_QUALITY_GATE = {
 /**
  * Read a JSON file safely, returning null if not found or invalid
  */
-export async function readJsonFile($, path) {
+async function readJsonFile($, path) {
     try {
         const result = await $ `cat ${path}`.quiet();
         return JSON.parse(result.text());
@@ -27,7 +35,7 @@ export async function readJsonFile($, path) {
 /**
  * Check if a file exists
  */
-export async function fileExists($, path) {
+async function fileExists($, path) {
     try {
         await $ `test -f ${path}`.quiet();
         return true;
@@ -42,7 +50,7 @@ export async function fileExists($, path) {
  * Precedence: `.opencode/opencode.json` overrides `opencode.json` (root).
  * Deep-merges nested objects like `include`.
  */
-export function mergeQualityGateConfig(root, dotOpencode) {
+function mergeQualityGateConfig(root, dotOpencode) {
     const base = root ?? {};
     const override = dotOpencode ?? {};
     return {
@@ -64,7 +72,7 @@ export function mergeQualityGateConfig(root, dotOpencode) {
  *
  * Merges ONLY the `qualityGate` key. `.opencode/opencode.json` takes precedence.
  */
-export async function loadQualityGateConfig($, directory) {
+async function loadQualityGateConfig($, directory) {
     const rootConfigPath = `${directory}/opencode.json`;
     const dotOpencodeConfigPath = `${directory}/.opencode/opencode.json`;
     const [rootJson, dotOpencodeJson] = await Promise.all([
@@ -79,6 +87,6 @@ export async function loadQualityGateConfig($, directory) {
  * Check if any explicit commands are configured (lint/build/test).
  * If so, these take priority over ci.sh and discovery.
  */
-export function hasConfiguredCommands(config) {
+function hasConfiguredCommands(config) {
     return !!(config.lint || config.build || config.test);
 }