@syntesseraai/opencode-feature-factory 0.2.3 → 0.2.4

package/src/output.ts

@@ -1,55 +0,0 @@
-/**
- * Regex pattern for detecting error-like lines in command output
- */
-const ERROR_PATTERNS =
-  /(error|failed|failure|panic|assert|exception|traceback|TypeError|ReferenceError|SyntaxError|FAILED|FAIL|ERR!)/i;
-
-/**
- * Extract lines that likely contain error information from command output.
- * Includes 1-2 lines of context after each error line.
- *
- * @param output - The command output to search
- * @param maxLines - Maximum number of error lines to extract
- * @returns Array of error-like lines (up to maxLines)
- */
-export function extractErrorLines(output: string, maxLines: number): string[] {
-  const lines = output.split('\n');
-  const errorLines: string[] = [];
-
-  for (let i = 0; i < lines.length && errorLines.length < maxLines; i++) {
-    const line = lines[i];
-    if (line && ERROR_PATTERNS.test(line)) {
-      errorLines.push(line);
-      // Include 1-2 lines of context after the error
-      const nextLine = lines[i + 1];
-      const nextNextLine = lines[i + 2];
-      if (nextLine !== undefined) {
-        errorLines.push(nextLine);
-      }
-      if (nextNextLine !== undefined) {
-        errorLines.push(nextNextLine);
-      }
-    }
-  }
-
-  return errorLines.slice(0, maxLines);
-}
-
-/**
- * Return the last N lines of output, with a truncation notice if needed.
- *
- * @param output - The full command output
- * @param maxLines - Maximum number of lines to return
- * @returns Truncated output with notice, or full output if short enough
- */
-export function tailLines(output: string, maxLines: number): string {
-  const lines = output.split('\n');
-  if (lines.length <= maxLines) {
-    return output;
-  }
-
-  const truncatedCount = lines.length - maxLines;
-  const tail = lines.slice(-maxLines).join('\n');
-
-  return `... (${truncatedCount} lines truncated)\n${tail}`;
-}

package/src/quality-gate-config.test.ts

@@ -1,186 +0,0 @@
-/**
- * Unit tests for quality-gate-config module
- *
- * Tests focus on pure functions:
- * - mergeQualityGateConfig: merges two config objects with deep merge for 'include'
- * - hasConfiguredCommands: checks if explicit commands are configured
- * - DEFAULT_QUALITY_GATE: verify default values
- */
-
-import {
-  mergeQualityGateConfig,
-  hasConfiguredCommands,
-  DEFAULT_QUALITY_GATE,
-} from './quality-gate-config';
-import type { QualityGateConfig } from './types';
-
-describe('DEFAULT_QUALITY_GATE', () => {
-  it('should have correct default values', () => {
-    expect(DEFAULT_QUALITY_GATE.steps).toEqual(['lint', 'build', 'test']);
-    expect(DEFAULT_QUALITY_GATE.useCiSh).toBe('auto');
-    expect(DEFAULT_QUALITY_GATE.packageManager).toBe('auto');
-    expect(DEFAULT_QUALITY_GATE.cacheSeconds).toBe(30);
-    expect(DEFAULT_QUALITY_GATE.maxOutputLines).toBe(160);
-    expect(DEFAULT_QUALITY_GATE.maxErrorLines).toBe(60);
-    expect(DEFAULT_QUALITY_GATE.include.rustClippy).toBe(true);
-  });
-});
-
-describe('mergeQualityGateConfig', () => {
-  it('should return empty config when both inputs are undefined', () => {
-    const result = mergeQualityGateConfig(undefined, undefined);
-    expect(result).toEqual({ include: {} });
-  });
-
-  it('should return root config when dotOpencode is undefined', () => {
-    const root: QualityGateConfig = {
-      lint: 'pnpm lint',
-      cacheSeconds: 60,
-    };
-    const result = mergeQualityGateConfig(root, undefined);
-    expect(result).toEqual({ lint: 'pnpm lint', cacheSeconds: 60, include: {} });
-  });
-
-  it('should return dotOpencode config when root is undefined', () => {
-    const dotOpencode: QualityGateConfig = {
-      build: 'npm run build',
-      maxOutputLines: 200,
-    };
-    const result = mergeQualityGateConfig(undefined, dotOpencode);
-    expect(result).toEqual({ build: 'npm run build', maxOutputLines: 200, include: {} });
-  });
-
-  it('should override root values with dotOpencode values', () => {
-    const root: QualityGateConfig = {
-      lint: 'pnpm lint',
-      build: 'pnpm build',
-      cacheSeconds: 30,
-    };
-    const dotOpencode: QualityGateConfig = {
-      lint: 'npm run lint:strict',
-      cacheSeconds: 60,
-    };
-    const result = mergeQualityGateConfig(root, dotOpencode);
-    expect(result.lint).toBe('npm run lint:strict');
-    expect(result.build).toBe('pnpm build');
-    expect(result.cacheSeconds).toBe(60);
-  });
-
-  it('should deep merge include objects', () => {
-    const root: QualityGateConfig = {
-      include: {
-        rustClippy: false,
-      },
-    };
-    const dotOpencode: QualityGateConfig = {
-      include: {
-        rustClippy: true,
-      },
-    };
-    const result = mergeQualityGateConfig(root, dotOpencode);
-    expect(result.include?.rustClippy).toBe(true);
-  });
-
-  it('should preserve root include values not overridden by dotOpencode', () => {
-    const root: QualityGateConfig = {
-      include: {
-        rustClippy: true,
-      },
-    };
-    const dotOpencode: QualityGateConfig = {
-      lint: 'custom lint',
-      // No include specified
-    };
-    const result = mergeQualityGateConfig(root, dotOpencode);
-    expect(result.include?.rustClippy).toBe(true);
-    expect(result.lint).toBe('custom lint');
-  });
-
-  it('should handle all config properties', () => {
-    const root: QualityGateConfig = {
-      lint: 'lint-root',
-      build: 'build-root',
-      test: 'test-root',
-      cwd: './root',
-      steps: ['lint'],
-      useCiSh: 'auto',
-      packageManager: 'npm',
-      cacheSeconds: 30,
-      maxOutputLines: 100,
-      maxErrorLines: 50,
-      include: { rustClippy: false },
-    };
-    const dotOpencode: QualityGateConfig = {
-      lint: 'lint-override',
-      steps: ['lint', 'test'],
-      useCiSh: 'never',
-      include: { rustClippy: true },
-    };
-    const result = mergeQualityGateConfig(root, dotOpencode);
-
-    expect(result.lint).toBe('lint-override');
-    expect(result.build).toBe('build-root');
-    expect(result.test).toBe('test-root');
-    expect(result.cwd).toBe('./root');
-    expect(result.steps).toEqual(['lint', 'test']);
-    expect(result.useCiSh).toBe('never');
-    expect(result.packageManager).toBe('npm');
-    expect(result.cacheSeconds).toBe(30);
-    expect(result.maxOutputLines).toBe(100);
-    expect(result.maxErrorLines).toBe(50);
-    expect(result.include?.rustClippy).toBe(true);
-  });
-});
-
-describe('hasConfiguredCommands', () => {
-  it('should return false for empty config', () => {
-    expect(hasConfiguredCommands({})).toBe(false);
-  });
-
-  it('should return false when only non-command properties are set', () => {
-    const config: QualityGateConfig = {
-      cacheSeconds: 60,
-      maxOutputLines: 200,
-      useCiSh: 'always',
-      packageManager: 'pnpm',
-    };
-    expect(hasConfiguredCommands(config)).toBe(false);
-  });
-
-  it('should return true when lint is configured', () => {
-    const config: QualityGateConfig = {
-      lint: 'pnpm lint',
-    };
-    expect(hasConfiguredCommands(config)).toBe(true);
-  });
-
-  it('should return true when build is configured', () => {
-    const config: QualityGateConfig = {
-      build: 'pnpm build',
-    };
-    expect(hasConfiguredCommands(config)).toBe(true);
-  });
-
-  it('should return true when test is configured', () => {
-    const config: QualityGateConfig = {
-      test: 'pnpm test',
-    };
-    expect(hasConfiguredCommands(config)).toBe(true);
-  });
-
-  it('should return true when multiple commands are configured', () => {
-    const config: QualityGateConfig = {
-      lint: 'pnpm lint',
-      build: 'pnpm build',
-      test: 'pnpm test',
-    };
-    expect(hasConfiguredCommands(config)).toBe(true);
-  });
-
-  it('should return false for empty string commands', () => {
-    const config: QualityGateConfig = {
-      lint: '',
-    };
-    expect(hasConfiguredCommands(config)).toBe(false);
-  });
-});

package/src/quality-gate-config.ts

@@ -1,106 +0,0 @@
-import type { QualityGateConfig } from './types';
-
-/**
- * Default configuration values for StopQualityGate
- */
-export const DEFAULT_QUALITY_GATE: Required<
-  Omit<QualityGateConfig, 'lint' | 'build' | 'test' | 'cwd'>
-> = {
-  steps: ['lint', 'build', 'test'],
-  useCiSh: 'auto',
-  packageManager: 'auto',
-  cacheSeconds: 30,
-  maxOutputLines: 160,
-  maxErrorLines: 60,
-  include: {
-    rustClippy: true,
-  },
-};
-
-type BunShell = any;
-
-/**
- * Read a JSON file safely, returning null if not found or invalid
- */
-export async function readJsonFile(
-  $: BunShell,
-  path: string
-): Promise<Record<string, unknown> | null> {
-  try {
-    const result = await $`cat ${path}`.quiet();
-    return JSON.parse(result.text()) as Record<string, unknown>;
-  } catch {
-    return null;
-  }
-}
-
-/**
- * Check if a file exists
- */
-export async function fileExists($: BunShell, path: string): Promise<boolean> {
-  try {
-    await $`test -f ${path}`.quiet();
-    return true;
-  } catch {
-    return false;
-  }
-}
-
-/**
- * Merge two qualityGate config objects.
- *
- * Precedence: `.opencode/opencode.json` overrides `opencode.json` (root).
- * Deep-merges nested objects like `include`.
- */
-export function mergeQualityGateConfig(
-  root: QualityGateConfig | undefined,
-  dotOpencode: QualityGateConfig | undefined
-): QualityGateConfig {
-  const base = root ?? {};
-  const override = dotOpencode ?? {};
-
-  return {
-    ...base,
-    ...override,
-    // Deep merge the `include` object
-    include: {
-      ...(base.include ?? {}),
-      ...(override.include ?? {}),
-    },
-  };
-}
-
-/**
- * Load and merge qualityGate config from both config file locations.
- *
- * Reads from:
- * - `${directory}/opencode.json`
- * - `${directory}/.opencode/opencode.json`
- *
- * Merges ONLY the `qualityGate` key. `.opencode/opencode.json` takes precedence.
- */
-export async function loadQualityGateConfig(
-  $: BunShell,
-  directory: string
-): Promise<QualityGateConfig> {
-  const rootConfigPath = `${directory}/opencode.json`;
-  const dotOpencodeConfigPath = `${directory}/.opencode/opencode.json`;
-
-  const [rootJson, dotOpencodeJson] = await Promise.all([
-    readJsonFile($, rootConfigPath),
-    readJsonFile($, dotOpencodeConfigPath),
-  ]);
-
-  const rootQualityGate = rootJson?.qualityGate as QualityGateConfig | undefined;
-  const dotOpencodeQualityGate = dotOpencodeJson?.qualityGate as QualityGateConfig | undefined;
-
-  return mergeQualityGateConfig(rootQualityGate, dotOpencodeQualityGate);
-}
-
-/**
- * Check if any explicit commands are configured (lint/build/test).
- * If so, these take priority over ci.sh and discovery.
- */
-export function hasConfiguredCommands(config: QualityGateConfig): boolean {
-  return !!(config.lint || config.build || config.test);
-}