@opensip-cli/fitness 0.1.0

package/dist/framework/check-types.d.ts

@@ -0,0 +1,82 @@
+/**
+ * @fileoverview Core type definitions for fitness checks
+ *
+ * Check interface is the return type of defineCheck.
+ * CheckConfig represents the internal configuration structure.
+ * CheckResult carries Signal[].
+ */
+import type { CheckScope, ResolvedScope } from './check-config.js';
+import type { ExecutionContext, RunOptions } from './execution-context.js';
+import type { PathMatcher } from './path-matcher.js';
+import type { CheckResult, ItemType } from '../types/findings.js';
+/**
+ * Check configuration options.
+ */
+export interface CheckConfig {
+    readonly id: string;
+    readonly slug: string;
+    readonly tags: readonly string[];
+    readonly description: string;
+    readonly longDescription?: string | undefined;
+    readonly analysisMode: 'analyze' | 'analyzeAll' | 'command';
+    readonly scope: ResolvedScope;
+    readonly itemType: ItemType;
+    readonly unit?: string | undefined;
+    readonly additionalExcludes?: readonly string[] | undefined;
+    readonly docs?: string | undefined;
+    readonly disabled?: boolean | undefined;
+    readonly confidence?: 'high' | 'medium' | 'low' | undefined;
+    readonly timeout?: number | undefined;
+    readonly scansFiles?: boolean | undefined;
+    readonly fileTypes?: readonly string[] | undefined;
+    /** Portable scope declaration for marketplace-ready target matching. */
+    readonly checkScope?: CheckScope | undefined;
+    /**
+     * Display icon (an emoji) for CLI/dashboard output. Travels WITH the check
+     * (§5.3 separate-domains fold) — there is no separate per-process display
+     * sidecar map. Absent → consumers fall back to a default icon.
+     */
+    readonly icon?: string | undefined;
+    /**
+     * Human-readable display name for CLI/dashboard output. Absent → consumers
+     * fall back to kebab-to-title-case of the slug.
+     */
+    readonly displayName?: string | undefined;
+    readonly execute: (ctx: ExecutionContext) => Promise<CheckResult>;
+}
+/**
+ * A defined check, ready to run.
+ */
+export interface Check {
+    readonly config: CheckConfig;
+    readonly run: (cwd: string, options?: RunOptions) => Promise<CheckResult>;
+    readonly getScope: () => ResolvedScope;
+    readonly getMatcher: (cwd: string) => PathMatcher;
+}
+/**
+ * Type guard: is this value a Check object?
+ *
+ * Checks for the shape of a Check object:
+ * - Has a `config` property that is an object
+ * - config has an `id` property that is a string
+ * - config has a `slug` property that is a string
+ * - config has an `execute` property that is a function
+ * - Has a `run` property that is a function
+ */
+export declare function isCheck(value: unknown): value is Check;
+/**
+ * Walk a barrel-export object and collect every Check it contains,
+ * deduplicated by `config.id`. Recurses into nested object exports;
+ * arrays are not traversed (they are typically the `checks` re-export
+ * the loader handles separately).
+ *
+ * Pack authors call this in their package's `index.ts`:
+ *
+ *     import * as allChecks from './checks/index.js'
+ *     export const checks = collectCheckObjects(allChecks)
+ *
+ * `seen` is exposed for callers that need to merge multiple barrels.
+ */
+export declare function collectCheckObjects(obj: Record<string, unknown>, seen?: Set<string>): Check[];
+export { type ResolvedScope } from './check-config.js';
+//# sourceMappingURL=check-types.d.ts.map

package/dist/framework/check-types.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"check-types.d.ts","sourceRoot":"","sources":["../../src/framework/check-types.ts"],"names":[],"mappings":"AAAA;;;;;;GAMG;AAEH,OAAO,KAAK,EAAE,UAAU,EAAE,aAAa,EAAE,MAAM,mBAAmB,CAAC;AACnE,OAAO,KAAK,EAAE,gBAAgB,EAAE,UAAU,EAAE,MAAM,wBAAwB,CAAC;AAC3E,OAAO,KAAK,EAAE,WAAW,EAAE,MAAM,mBAAmB,CAAC;AACrD,OAAO,KAAK,EAAE,WAAW,EAAE,QAAQ,EAAE,MAAM,sBAAsB,CAAC;AAElE;;GAEG;AACH,MAAM,WAAW,WAAW;IAC1B,QAAQ,CAAC,EAAE,EAAE,MAAM,CAAC;IACpB,QAAQ,CAAC,IAAI,EAAE,MAAM,CAAC;IACtB,QAAQ,CAAC,IAAI,EAAE,SAAS,MAAM,EAAE,CAAC;IACjC,QAAQ,CAAC,WAAW,EAAE,MAAM,CAAC;IAC7B,QAAQ,CAAC,eAAe,CAAC,EAAE,MAAM,GAAG,SAAS,CAAC;IAC9C,QAAQ,CAAC,YAAY,EAAE,SAAS,GAAG,YAAY,GAAG,SAAS,CAAC;IAC5D,QAAQ,CAAC,KAAK,EAAE,aAAa,CAAC;IAC9B,QAAQ,CAAC,QAAQ,EAAE,QAAQ,CAAC;IAC5B,QAAQ,CAAC,IAAI,CAAC,EAAE,MAAM,GAAG,SAAS,CAAC;IACnC,QAAQ,CAAC,kBAAkB,CAAC,EAAE,SAAS,MAAM,EAAE,GAAG,SAAS,CAAC;IAC5D,QAAQ,CAAC,IAAI,CAAC,EAAE,MAAM,GAAG,SAAS,CAAC;IACnC,QAAQ,CAAC,QAAQ,CAAC,EAAE,OAAO,GAAG,SAAS,CAAC;IACxC,QAAQ,CAAC,UAAU,CAAC,EAAE,MAAM,GAAG,QAAQ,GAAG,KAAK,GAAG,SAAS,CAAC;IAC5D,QAAQ,CAAC,OAAO,CAAC,EAAE,MAAM,GAAG,SAAS,CAAC;IACtC,QAAQ,CAAC,UAAU,CAAC,EAAE,OAAO,GAAG,SAAS,CAAC;IAC1C,QAAQ,CAAC,SAAS,CAAC,EAAE,SAAS,MAAM,EAAE,GAAG,SAAS,CAAC;IACnD,wEAAwE;IACxE,QAAQ,CAAC,UAAU,CAAC,EAAE,UAAU,GAAG,SAAS,CAAC;IAC7C;;;;OAIG;IACH,QAAQ,CAAC,IAAI,CAAC,EAAE,MAAM,GAAG,SAAS,CAAC;IACnC;;;OAGG;IACH,QAAQ,CAAC,WAAW,CAAC,EAAE,MAAM,GAAG,SAAS,CAAC;IAC1C,QAAQ,CAAC,OAAO,EAAE,CAAC,GAAG,EAAE,gBAAgB,KAAK,OAAO,CAAC,WAAW,CAAC,CAAC;CACnE;AAED;;GAEG;AACH,MAAM,WAAW,KAAK;IACpB,QAAQ,CAAC,MAAM,EAAE,WAAW,CAAC;IAC7B,QAAQ,CAAC,GAAG,EAAE,CAAC,GAAG,EAAE,MAAM,EAAE,OAAO,CAAC,EAAE,UAAU,KAAK,OAAO,CAAC,WAAW,CAAC,CAAC;IAC1E,QAAQ,CAAC,QAAQ,EAAE,MAAM,aAAa,CAAC;IACvC,QAAQ,CAAC,UAAU,EAAE,CAAC,GAAG,EAAE,MAAM,KAAK,WAAW,CAAC;CACnD;AAED;;;;;;;;;GASG;AACH,wBAAgB,OAAO,CAAC,KAAK,EAAE,OAAO,GAAG,KAAK,IAAI,KAAK,CAatD;AAED;;;;;;;;;;;;GAYG;AACH,wBAAgB,mBAAmB,CACjC,GAAG,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,EAC5B,IAAI,GAAE,GAAG,CAAC,MAAM,CAAqB,GACpC,KAAK,EAAE,CAaT;AAED,OAAO,EAAE,KAAK,aAAa,EAAE,MAAM,mBAAmB,CAAC"}

package/dist/framework/check-types.js

@@ -0,0 +1,63 @@
+/**
+ * @fileoverview Core type definitions for fitness checks
+ *
+ * Check interface is the return type of defineCheck.
+ * CheckConfig represents the internal configuration structure.
+ * CheckResult carries Signal[].
+ */
+/**
+ * Type guard: is this value a Check object?
+ *
+ * Checks for the shape of a Check object:
+ * - Has a `config` property that is an object
+ * - config has an `id` property that is a string
+ * - config has a `slug` property that is a string
+ * - config has an `execute` property that is a function
+ * - Has a `run` property that is a function
+ */
+export function isCheck(value) {
+    if (value === null || typeof value !== 'object')
+        return false;
+    const obj = value;
+    if (!obj.config || typeof obj.config !== 'object')
+        return false;
+    const config = obj.config;
+    if (typeof config.id !== 'string')
+        return false;
+    if (typeof config.slug !== 'string')
+        return false;
+    if (typeof config.execute !== 'function')
+        return false;
+    if (typeof obj.run !== 'function')
+        return false;
+    return true;
+}
+/**
+ * Walk a barrel-export object and collect every Check it contains,
+ * deduplicated by `config.id`. Recurses into nested object exports;
+ * arrays are not traversed (they are typically the `checks` re-export
+ * the loader handles separately).
+ *
+ * Pack authors call this in their package's `index.ts`:
+ *
+ *     import * as allChecks from './checks/index.js'
+ *     export const checks = collectCheckObjects(allChecks)
+ *
+ * `seen` is exposed for callers that need to merge multiple barrels.
+ */
+export function collectCheckObjects(obj, seen = new Set()) {
+    const result = [];
+    for (const value of Object.values(obj)) {
+        if (isCheck(value)) {
+            if (!seen.has(value.config.id)) {
+                seen.add(value.config.id);
+                result.push(value);
+            }
+        }
+        else if (typeof value === 'object' && value !== null && !Array.isArray(value)) {
+            result.push(...collectCheckObjects(value, seen));
+        }
+    }
+    return result;
+}
+//# sourceMappingURL=check-types.js.map

package/dist/framework/check-types.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"check-types.js","sourceRoot":"","sources":["../../src/framework/check-types.ts"],"names":[],"mappings":"AAAA;;;;;;GAMG;AAqDH;;;;;;;;;GASG;AACH,MAAM,UAAU,OAAO,CAAC,KAAc;IACpC,IAAI,KAAK,KAAK,IAAI,IAAI,OAAO,KAAK,KAAK,QAAQ;QAAE,OAAO,KAAK,CAAC;IAE9D,MAAM,GAAG,GAAG,KAAgC,CAAC;IAC7C,IAAI,CAAC,GAAG,CAAC,MAAM,IAAI,OAAO,GAAG,CAAC,MAAM,KAAK,QAAQ;QAAE,OAAO,KAAK,CAAC;IAEhE,MAAM,MAAM,GAAG,GAAG,CAAC,MAAiC,CAAC;IACrD,IAAI,OAAO,MAAM,CAAC,EAAE,KAAK,QAAQ;QAAE,OAAO,KAAK,CAAC;IAChD,IAAI,OAAO,MAAM,CAAC,IAAI,KAAK,QAAQ;QAAE,OAAO,KAAK,CAAC;IAClD,IAAI,OAAO,MAAM,CAAC,OAAO,KAAK,UAAU;QAAE,OAAO,KAAK,CAAC;IACvD,IAAI,OAAO,GAAG,CAAC,GAAG,KAAK,UAAU;QAAE,OAAO,KAAK,CAAC;IAEhD,OAAO,IAAI,CAAC;AACd,CAAC;AAED;;;;;;;;;;;;GAYG;AACH,MAAM,UAAU,mBAAmB,CACjC,GAA4B,EAC5B,OAAoB,IAAI,GAAG,EAAU;IAErC,MAAM,MAAM,GAAY,EAAE,CAAC;IAC3B,KAAK,MAAM,KAAK,IAAI,MAAM,CAAC,MAAM,CAAC,GAAG,CAAC,EAAE,CAAC;QACvC,IAAI,OAAO,CAAC,KAAK,CAAC,EAAE,CAAC;YACnB,IAAI,CAAC,IAAI,CAAC,GAAG,CAAC,KAAK,CAAC,MAAM,CAAC,EAAE,CAAC,EAAE,CAAC;gBAC/B,IAAI,CAAC,GAAG,CAAC,KAAK,CAAC,MAAM,CAAC,EAAE,CAAC,CAAC;gBAC1B,MAAM,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;YACrB,CAAC;QACH,CAAC;aAAM,IAAI,OAAO,KAAK,KAAK,QAAQ,IAAI,KAAK,KAAK,IAAI,IAAI,CAAC,KAAK,CAAC,OAAO,CAAC,KAAK,CAAC,EAAE,CAAC;YAChF,MAAM,CAAC,IAAI,CAAC,GAAG,mBAAmB,CAAC,KAAgC,EAAE,IAAI,CAAC,CAAC,CAAC;QAC9E,CAAC;IACH,CAAC;IACD,OAAO,MAAM,CAAC;AAChB,CAAC"}

package/dist/framework/command-executor.d.ts

@@ -0,0 +1,25 @@
+/**
+ * @fileoverview Command executor for external tool checks
+ *
+ * Executes external commands (eslint, prettier, etc.) with abort
+ * and timeout support, then parses output into violations.
+ */
+import type { CheckViolation, CommandConfig } from './check-config.js';
+/** Options for executing an external command. */
+export interface CommandExecutorOptions {
+    readonly cwd: string;
+    readonly signal?: AbortSignal | undefined;
+    readonly timeout?: number | undefined;
+}
+/** Result of executing an external command, including parsed violations. */
+export interface CommandExecutionResult {
+    readonly violations: CheckViolation[];
+    readonly aborted: boolean;
+    readonly exitCode: number | null;
+    readonly error?: string;
+}
+/**
+ * Execute an external command and parse its output into violations.
+ */
+export declare function executeCommand(config: CommandConfig, files: readonly string[], options: CommandExecutorOptions): Promise<CommandExecutionResult>;
+//# sourceMappingURL=command-executor.d.ts.map

package/dist/framework/command-executor.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"command-executor.d.ts","sourceRoot":"","sources":["../../src/framework/command-executor.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAIH,OAAO,KAAK,EAAE,cAAc,EAAE,aAAa,EAAE,MAAM,mBAAmB,CAAC;AAMvE,iDAAiD;AACjD,MAAM,WAAW,sBAAsB;IACrC,QAAQ,CAAC,GAAG,EAAE,MAAM,CAAC;IACrB,QAAQ,CAAC,MAAM,CAAC,EAAE,WAAW,GAAG,SAAS,CAAC;IAC1C,QAAQ,CAAC,OAAO,CAAC,EAAE,MAAM,GAAG,SAAS,CAAC;CACvC;AAED,4EAA4E;AAC5E,MAAM,WAAW,sBAAsB;IACrC,QAAQ,CAAC,UAAU,EAAE,cAAc,EAAE,CAAC;IACtC,QAAQ,CAAC,OAAO,EAAE,OAAO,CAAC;IAC1B,QAAQ,CAAC,QAAQ,EAAE,MAAM,GAAG,IAAI,CAAC;IACjC,QAAQ,CAAC,KAAK,CAAC,EAAE,MAAM,CAAC;CACzB;AAQD;;GAEG;AACH,wBAAsB,cAAc,CAClC,MAAM,EAAE,aAAa,EACrB,KAAK,EAAE,SAAS,MAAM,EAAE,EACxB,OAAO,EAAE,sBAAsB,GAC9B,OAAO,CAAC,sBAAsB,CAAC,CA6DjC"}

package/dist/framework/command-executor.js

@@ -0,0 +1,63 @@
+/**
+ * @fileoverview Command executor for external tool checks
+ *
+ * Executes external commands (eslint, prettier, etc.) with abort
+ * and timeout support, then parses output into violations.
+ */
+import { execAbortable } from './abortable-exec.js';
+// =============================================================================
+// EXECUTOR
+// =============================================================================
+const DEFAULT_EXPECTED_EXIT_CODES = [0, 1];
+/**
+ * Execute an external command and parse its output into violations.
+ */
+export async function executeCommand(config, files, options) {
+    const { cwd, signal, timeout } = options;
+    const args = typeof config.args === 'function' ? config.args(files) : config.args;
+    const command = [config.bin, ...args];
+    const execOptions = { cwd, signal, timeout };
+    let result;
+    try {
+        result = await execAbortable(command, execOptions);
+    }
+    catch (error) {
+        // ENOENT = tool not installed (spawn fails before the process starts)
+        const message = error instanceof Error ? error.message : String(error);
+        if (message.includes('ENOENT') || message.includes('not found')) {
+            return {
+                violations: [],
+                aborted: false,
+                exitCode: null,
+                error: `${config.bin} is not installed. Install it to enable this check.`,
+            };
+        }
+        throw error;
+    }
+    if (result.aborted) {
+        return { violations: [], aborted: true, exitCode: result.exitCode };
+    }
+    // Exit code 127 = command not found (shell mode)
+    if (result.exitCode === 127) {
+        return {
+            violations: [],
+            aborted: false,
+            exitCode: 127,
+            error: `${config.bin} is not installed. Install it to enable this check.`,
+        };
+    }
+    const expectedCodes = config.expectedExitCodes ?? DEFAULT_EXPECTED_EXIT_CODES;
+    if (result.exitCode !== null && !expectedCodes.includes(result.exitCode)) {
+        return {
+            violations: [],
+            aborted: false,
+            exitCode: result.exitCode,
+            error: `Command exited with unexpected code ${result.exitCode}. ` +
+                `Expected one of: ${expectedCodes.join(', ')}. ` +
+                `stderr: ${result.stderr.slice(0, 500)}${result.stderr.length > 500 ? ' [truncated]' : ''}`,
+        };
+    }
+    const violations = config.parseOutput(result.stdout, result.stderr, result.exitCode ?? 0, files, cwd);
+    return { violations, aborted: false, exitCode: result.exitCode };
+}
+//# sourceMappingURL=command-executor.js.map

package/dist/framework/command-executor.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"command-executor.js","sourceRoot":"","sources":["../../src/framework/command-executor.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAEH,OAAO,EAAE,aAAa,EAA6B,MAAM,qBAAqB,CAAC;AAuB/E,gFAAgF;AAChF,WAAW;AACX,gFAAgF;AAEhF,MAAM,2BAA2B,GAAsB,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC;AAE9D;;GAEG;AACH,MAAM,CAAC,KAAK,UAAU,cAAc,CAClC,MAAqB,EACrB,KAAwB,EACxB,OAA+B;IAE/B,MAAM,EAAE,GAAG,EAAE,MAAM,EAAE,OAAO,EAAE,GAAG,OAAO,CAAC;IAEzC,MAAM,IAAI,GAAG,OAAO,MAAM,CAAC,IAAI,KAAK,UAAU,CAAC,CAAC,CAAC,MAAM,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,MAAM,CAAC,IAAI,CAAC;IAClF,MAAM,OAAO,GAAG,CAAC,MAAM,CAAC,GAAG,EAAE,GAAG,IAAI,CAAC,CAAC;IAEtC,MAAM,WAAW,GAAyB,EAAE,GAAG,EAAE,MAAM,EAAE,OAAO,EAAE,CAAC;IAEnE,IAAI,MAAiD,CAAC;IACtD,IAAI,CAAC;QACH,MAAM,GAAG,MAAM,aAAa,CAAC,OAAO,EAAE,WAAW,CAAC,CAAC;IACrD,CAAC;IAAC,OAAO,KAAK,EAAE,CAAC;QACf,sEAAsE;QACtE,MAAM,OAAO,GAAG,KAAK,YAAY,KAAK,CAAC,CAAC,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC;QACvE,IAAI,OAAO,CAAC,QAAQ,CAAC,QAAQ,CAAC,IAAI,OAAO,CAAC,QAAQ,CAAC,WAAW,CAAC,EAAE,CAAC;YAChE,OAAO;gBACL,UAAU,EAAE,EAAE;gBACd,OAAO,EAAE,KAAK;gBACd,QAAQ,EAAE,IAAI;gBACd,KAAK,EAAE,GAAG,MAAM,CAAC,GAAG,qDAAqD;aAC1E,CAAC;QACJ,CAAC;QACD,MAAM,KAAK,CAAC;IACd,CAAC;IAED,IAAI,MAAM,CAAC,OAAO,EAAE,CAAC;QACnB,OAAO,EAAE,UAAU,EAAE,EAAE,EAAE,OAAO,EAAE,IAAI,EAAE,QAAQ,EAAE,MAAM,CAAC,QAAQ,EAAE,CAAC;IACtE,CAAC;IAED,iDAAiD;IACjD,IAAI,MAAM,CAAC,QAAQ,KAAK,GAAG,EAAE,CAAC;QAC5B,OAAO;YACL,UAAU,EAAE,EAAE;YACd,OAAO,EAAE,KAAK;YACd,QAAQ,EAAE,GAAG;YACb,KAAK,EAAE,GAAG,MAAM,CAAC,GAAG,qDAAqD;SAC1E,CAAC;IACJ,CAAC;IAED,MAAM,aAAa,GAAG,MAAM,CAAC,iBAAiB,IAAI,2BAA2B,CAAC;IAC9E,IAAI,MAAM,CAAC,QAAQ,KAAK,IAAI,IAAI,CAAC,aAAa,CAAC,QAAQ,CAAC,MAAM,CAAC,QAAQ,CAAC,EAAE,CAAC;QACzE,OAAO;YACL,UAAU,EAAE,EAAE;YACd,OAAO,EAAE,KAAK;YACd,QAAQ,EAAE,MAAM,CAAC,QAAQ;YACzB,KAAK,EACH,uCAAuC,MAAM,CAAC,QAAQ,IAAI;gBAC1D,oBAAoB,aAAa,CAAC,IAAI,CAAC,IAAI,CAAC,IAAI;gBAChD,WAAW,MAAM,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC,EAAE,GAAG,CAAC,GAAG,MAAM,CAAC,MAAM,CAAC,MAAM,GAAG,GAAG,CAAC,CAAC,CAAC,cAAc,CAAC,CAAC,CAAC,EAAE,EAAE;SAC9F,CAAC;IACJ,CAAC;IAED,MAAM,UAAU,GAAG,MAAM,CAAC,WAAW,CACnC,MAAM,CAAC,MAAM,EACb,MAAM,CAAC,MAAM,EACb,MAAM,CAAC,QAAQ,IAAI,CAAC,EACpB,KAAK,EACL,GAAG,CACJ,CAAC;IAEF,OAAO,EAAE,UAAU,EAAE,OAAO,EAAE,KAAK,EAAE,QAAQ,EAAE,MAAM,CAAC,QAAQ,EAAE,CAAC;AACnE,CAAC"}

package/dist/framework/comment-openers.d.ts

@@ -0,0 +1,11 @@
+/**
+ * @fileoverview Comment-opener table — re-exported from the kernel.
+ *
+ * The canonical table now lives in `@opensip-cli/core`
+ * (`signals/comment-openers.ts`, ADR-0014) so the suppression scanner and the
+ * fitness directive inventory share a single source of truth (they
+ * historically drifted). This module preserves the local import path for
+ * fitness consumers (`directive-inventory.ts`).
+ */
+export { COMMENT_OPENERS, stripCommentOpener } from '@opensip-cli/core';
+//# sourceMappingURL=comment-openers.d.ts.map

package/dist/framework/comment-openers.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"comment-openers.d.ts","sourceRoot":"","sources":["../../src/framework/comment-openers.ts"],"names":[],"mappings":"AAAA;;;;;;;;GAQG;AAEH,OAAO,EAAE,eAAe,EAAE,kBAAkB,EAAE,MAAM,mBAAmB,CAAC"}

package/dist/framework/comment-openers.js

@@ -0,0 +1,11 @@
+/**
+ * @fileoverview Comment-opener table — re-exported from the kernel.
+ *
+ * The canonical table now lives in `@opensip-cli/core`
+ * (`signals/comment-openers.ts`, ADR-0014) so the suppression scanner and the
+ * fitness directive inventory share a single source of truth (they
+ * historically drifted). This module preserves the local import path for
+ * fitness consumers (`directive-inventory.ts`).
+ */
+export { COMMENT_OPENERS, stripCommentOpener } from '@opensip-cli/core';
+//# sourceMappingURL=comment-openers.js.map

package/dist/framework/comment-openers.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"comment-openers.js","sourceRoot":"","sources":["../../src/framework/comment-openers.ts"],"names":[],"mappings":"AAAA;;;;;;;;GAQG;AAEH,OAAO,EAAE,eAAe,EAAE,kBAAkB,EAAE,MAAM,mBAAmB,CAAC"}

package/dist/framework/constants.d.ts

@@ -0,0 +1,9 @@
+/**
+ * @fileoverview Shared constants for the fitness framework
+ */
+/**
+ * Standard exclusion patterns for file matching across the framework.
+ * Used by execution context, directive inventory, and file cache.
+ */
+export declare const DEFAULT_EXCLUSION_PATTERNS: readonly ["**/node_modules/**", "**/dist/**", "**/.git/**", "**/coverage/**", "**/*.tsbuildinfo"];
+//# sourceMappingURL=constants.d.ts.map

package/dist/framework/constants.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"constants.d.ts","sourceRoot":"","sources":["../../src/framework/constants.ts"],"names":[],"mappings":"AACA;;GAEG;AAEH;;;GAGG;AACH,eAAO,MAAM,0BAA0B,mGAM7B,CAAC"}

package/dist/framework/constants.js

@@ -0,0 +1,16 @@
+// @fitness-ignore-file project-readme-existence -- internal module, not a package root
+/**
+ * @fileoverview Shared constants for the fitness framework
+ */
+/**
+ * Standard exclusion patterns for file matching across the framework.
+ * Used by execution context, directive inventory, and file cache.
+ */
+export const DEFAULT_EXCLUSION_PATTERNS = [
+    '**/node_modules/**',
+    '**/dist/**',
+    '**/.git/**',
+    '**/coverage/**',
+    '**/*.tsbuildinfo',
+];
+//# sourceMappingURL=constants.js.map

package/dist/framework/constants.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"constants.js","sourceRoot":"","sources":["../../src/framework/constants.ts"],"names":[],"mappings":"AAAA,uFAAuF;AACvF;;GAEG;AAEH;;;GAGG;AACH,MAAM,CAAC,MAAM,0BAA0B,GAAG;IACxC,oBAAoB;IACpB,YAAY;IACZ,YAAY;IACZ,gBAAgB;IAChB,kBAAkB;CACV,CAAC"}

package/dist/framework/define-check.d.ts

@@ -0,0 +1,38 @@
+/**
+ * @fileoverview defineCheck - Unified check definition API
+ *
+ * The main API for creating fitness checks. Supports three modes:
+ * - analyze: Per-file analysis with content and path
+ * - analyzeAll: Multi-file analysis with lazy loading FileAccessor
+ * - command: External tool execution with output parsing
+ *
+ * Check authors return CheckViolation[]. The framework converts each
+ * CheckViolation into a universal Signal via createSignal().
+ */
+import type { UnifiedCheckConfig } from './check-config.js';
+import type { Check } from './check-types.js';
+/**
+ * Define a fitness check using the unified API.
+ *
+ * @example
+ * ```typescript
+ * export const noConsoleLog = defineCheck({
+ *   id: 'a1b2c3d4-e5f6-7890-abcd-ef1234567890',
+ *   slug: 'no-console-log',
+ *   category: 'quality',
+ *   description: 'Disallow console.log in production code',
+ *   analyze: (content, filePath) => {
+ *     const violations: CheckViolation[] = [];
+ *     content.split('\n').forEach((line, idx) => {
+ *       if (line.includes('console.log')) {
+ *         violations.push({ line: idx + 1, message: 'No console.log', severity: 'error' });
+ *       }
+ *     });
+ *     return violations;
+ *   },
+ * });
+ * ```
+ * @throws {ValidationError} When the check config is invalid
+ */
+export declare function defineCheck(config: UnifiedCheckConfig): Check;
+//# sourceMappingURL=define-check.d.ts.map

package/dist/framework/define-check.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"define-check.d.ts","sourceRoot":"","sources":["../../src/framework/define-check.ts"],"names":[],"mappings":"AAGA;;;;;;;;;;GAUG;AA0BH,OAAO,KAAK,EACV,kBAAkB,EAKnB,MAAM,mBAAmB,CAAC;AAC3B,OAAO,KAAK,EAAE,KAAK,EAAE,MAAM,kBAAkB,CAAC;AA0K9C;;;;;;;;;;;;;;;;;;;;;;GAsBG;AACH,wBAAgB,WAAW,CAAC,MAAM,EAAE,kBAAkB,GAAG,KAAK,CAqH7D"}

package/dist/framework/define-check.js

@@ -0,0 +1,296 @@
+// @fitness-ignore-file module-coupling-metrics -- central orchestration module with necessary coupling
+// @fitness-ignore-file null-safety -- ResultBuilder.create() returns a fluent builder; .totalItems().filesScanned() chain is always safe
+// @fitness-ignore-file unbounded-memory -- reads single source files at a time via FileAccessor; per-file memory bounded by source size
+/**
+ * @fileoverview defineCheck - Unified check definition API
+ *
+ * The main API for creating fitness checks. Supports three modes:
+ * - analyze: Per-file analysis with content and path
+ * - analyzeAll: Multi-file analysis with lazy loading FileAccessor
+ * - command: External tool execution with output parsing
+ *
+ * Check authors return CheckViolation[]. The framework converts each
+ * CheckViolation into a universal Signal via createSignal().
+ */
+import { logger, SystemError, createSignal, applyContentFilter, currentScope, } from '@opensip-cli/core';
+import { getAnalysisMode, isAnalyzeConfig, isAnalyzeAllConfig, isCommandConfig, validateCheckConfig, } from './check-config.js';
+import { executeCommand } from './command-executor.js';
+import { CheckAbortedError, createExecutionContext } from './execution-context.js';
+import { createFileAccessor } from './file-accessor.js';
+import { filterFilesByType } from './file-type-filter.js';
+import { filterSignalsByDirectives, buildFilteredResult } from './ignore-processing.js';
+import { PathMatcher } from './path-matcher.js';
+import { ResultBuilder } from './result-builder.js';
+import { mapFindingSeverity, mapTagsToSignalCategory } from './severity-mapping.js';
+// =============================================================================
+// VIOLATION → SIGNAL CONVERSION
+// =============================================================================
+function toSignal(violation, checkSlug, checkTags, defaultFilePath, provider = 'opensip') {
+    const filePath = violation.filePath ?? defaultFilePath ?? '';
+    return createSignal({
+        source: 'fitness',
+        provider,
+        severity: mapFindingSeverity(violation.severity),
+        category: mapTagsToSignalCategory(checkTags),
+        ruleId: `fit:${checkSlug}`,
+        message: violation.message,
+        suggestion: violation.suggestion,
+        code: { file: filePath, line: violation.line, column: violation.column },
+        fix: violation.fix ??
+            (violation.suggestion ? { action: 'refactor', confidence: 0.5 } : undefined),
+        metadata: Object.fromEntries(Object.entries({
+            match: violation.match,
+            type: violation.type,
+            checkSlug,
+            checkTags: checkTags.length > 0 ? checkTags.join(',') : undefined,
+        }).filter(([, v]) => v != null && v !== '')),
+    });
+}
+// =============================================================================
+// ANALYSIS MODE EXECUTORS
+// =============================================================================
+/** @throws {CheckAbortedError} When the check is aborted via AbortSignal */
+async function executeAnalyzeMode(config, files, ctx) {
+    const builder = ResultBuilder.create({
+        checkId: config.id,
+        itemType: config.itemType ?? 'files',
+    })
+        .totalItems(files.length)
+        .filesScanned(files.length);
+    for (const filePath of files) {
+        if (ctx.signal?.aborted) {
+            throw new CheckAbortedError(config.slug);
+        }
+        try {
+            const rawContent = await ctx.readFile(filePath);
+            // Dispatch the content filter through the LanguageAdapter for the
+            // file's extension. Falls back to raw content when no adapter is
+            // registered. See languages/content-filter-dispatch.ts.
+            const content = applyContentFilter(filePath, rawContent, config.contentFilter ?? 'none');
+            const violations = config.analyze(content, filePath);
+            for (const violation of violations) {
+                void builder.addSignal(toSignal(violation, config.slug, config.tags ?? [], filePath, config.provider));
+            }
+        }
+        catch (error) {
+            if (error instanceof CheckAbortedError)
+                throw error;
+            logger.debug('Skipping unreadable file', {
+                evt: 'fitness.check.file.skip',
+                module: 'fitness:framework',
+                filePath,
+                checkSlug: config.slug,
+            });
+        }
+    }
+    return builder.build();
+}
+/** @throws {CheckAbortedError} When the check is aborted via AbortSignal */
+async function executeAnalyzeAllMode(config, files, ctx) {
+    if (ctx.signal?.aborted) {
+        throw new CheckAbortedError(config.slug);
+    }
+    const fileAccessor = createFileAccessor(files, {
+        signal: ctx.signal,
+        contentFilter: config.contentFilter,
+    });
+    const violations = await config.analyzeAll(fileAccessor);
+    if (ctx.signal?.aborted) {
+        throw new CheckAbortedError(config.slug);
+    }
+    const builder = ResultBuilder.create({
+        checkId: config.id,
+        itemType: config.itemType ?? 'files',
+    })
+        .totalItems(files.length)
+        .filesScanned(files.length);
+    for (const violation of violations) {
+        if (!violation.filePath) {
+            ctx.log(`Warning: violation missing filePath in analyzeAll mode`);
+        }
+        void builder.addSignal(toSignal(violation, config.slug, config.tags ?? [], undefined, config.provider));
+    }
+    return builder.build();
+}
+/** @throws {CheckAbortedError} When the check is aborted via AbortSignal */
+async function executeCommandMode(config, files, ctx) {
+    const result = await executeCommand(config.command, files, {
+        cwd: ctx.cwd,
+        signal: ctx.signal,
+        timeout: config.timeout,
+    });
+    /* v8 ignore start -- defensive: command-mode tests cover the non-aborted path; abort during external command execution requires a long-running subprocess that's intentionally not unit-testable */
+    if (result.aborted) {
+        throw new CheckAbortedError(config.slug);
+    }
+    /* v8 ignore stop */
+    const builder = ResultBuilder.create({
+        checkId: config.id,
+        itemType: config.itemType ?? 'files',
+    })
+        .totalItems(files.length)
+        .filesScanned(0);
+    if (result.error) {
+        return builder.buildError(result.error);
+    }
+    for (const violation of result.violations) {
+        void builder.addSignal(toSignal(violation, config.slug, config.tags ?? [], undefined, config.provider));
+    }
+    return builder.build();
+}
+// =============================================================================
+// MAIN EXPORT
+// =============================================================================
+/**
+ * Define a fitness check using the unified API.
+ *
+ * @example
+ * ```typescript
+ * export const noConsoleLog = defineCheck({
+ *   id: 'a1b2c3d4-e5f6-7890-abcd-ef1234567890',
+ *   slug: 'no-console-log',
+ *   category: 'quality',
+ *   description: 'Disallow console.log in production code',
+ *   analyze: (content, filePath) => {
+ *     const violations: CheckViolation[] = [];
+ *     content.split('\n').forEach((line, idx) => {
+ *       if (line.includes('console.log')) {
+ *         violations.push({ line: idx + 1, message: 'No console.log', severity: 'error' });
+ *       }
+ *     });
+ *     return violations;
+ *   },
+ * });
+ * ```
+ * @throws {ValidationError} When the check config is invalid
+ */
+export function defineCheck(config) {
+    validateCheckConfig(config);
+    // Canonicalise scope languages through the current scope's language
+    // registry so a check declared with `scope: { languages: ['c'] }` is
+    // indexed under the canonical id `'cpp'`. Unknown languages pass
+    // through unchanged (with a debug log) — they may resolve later if a
+    // custom adapter ships, and dropping them here would silently break
+    // checks. When `defineCheck` runs at module-load time before a scope
+    // is bound (the typical case for top-level `export const x =
+    // defineCheck(...)`), we cannot canonicalise — just lowercase.
+    //
+    // Recovery: the engine *always* re-canonicalizes check-declared languages
+    // at scope-match / file-resolution time (see scope-resolver.ts: liveScopeLangs
+    // map + target-registry.ts:toCanonical + findByScope). This makes define-time
+    // canonicalization best-effort only; execution-time canonicalization against
+    // the entered RunScope's LanguageRegistry is the source of truth for matching.
+    const scope = currentScope();
+    const canonicalLanguages = config.scope
+        ? config.scope.languages.map((lang) => {
+            const canonical = scope?.languages.canonicalize(lang);
+            if (canonical === undefined) {
+                logger.debug({
+                    evt: 'fitness.check.scope.unknown_language',
+                    module: 'fitness:framework',
+                    checkSlug: config.slug,
+                    language: lang,
+                    msg: `Check ${config.slug} declared scope language ${lang} which is not registered (or no scope at definition time)`,
+                });
+                return lang.toLowerCase();
+            }
+            return canonical;
+        })
+        : undefined;
+    const check = {
+        config: {
+            id: config.id,
+            slug: config.slug,
+            tags: config.tags ? [...config.tags] : [],
+            description: config.description,
+            longDescription: config.longDescription,
+            analysisMode: getAnalysisMode(config),
+            scope: { include: [], exclude: [], description: '' },
+            itemType: config.itemType ?? 'files',
+            docs: config.docs,
+            disabled: config.disabled,
+            confidence: config.confidence,
+            timeout: config.timeout,
+            scansFiles: !isCommandConfig(config),
+            fileTypes: config.fileTypes ? [...config.fileTypes] : undefined,
+            checkScope: config.scope && canonicalLanguages
+                ? { languages: canonicalLanguages, concerns: [...config.scope.concerns] }
+                : undefined,
+            // Display metadata travels WITH the check (§5.3 fold) — no separate
+            // per-process display sidecar/singleton. Authors set these inline, or a
+            // pack's display map is applied via applyCheckDisplay().
+            icon: config.icon,
+            displayName: config.displayName,
+            // @fitness-ignore-next-line concurrency-safety -- async arrow delegates to executeUnifiedCheck which is async; needed for type compatibility
+            execute: async (ctx) => executeUnifiedCheck(config, ctx),
+        },
+        getScope() {
+            return { include: [], exclude: [], description: 'target-based scope' };
+        },
+        getMatcher(cwd) {
+            return PathMatcher.create({
+                include: [],
+                exclude: [],
+                cwd,
+            });
+        },
+        async run(cwd, options) {
+            const start = Date.now();
+            const matcher = PathMatcher.create({
+                include: [],
+                exclude: [],
+                cwd,
+            });
+            const executionConfig = {
+                id: config.id,
+                slug: config.slug,
+                itemType: config.itemType ?? 'files',
+            };
+            const ctx = createExecutionContext(executionConfig, cwd, matcher, options);
+            try {
+                const result = await executeUnifiedCheck(config, ctx);
+                const { filteredSignals, ignoredCount, appliedDirectives } = await filterSignalsByDirectives(result.signals, config.slug, result.ignoredCount ?? 0);
+                const filtered = buildFilteredResult(result, filteredSignals, ignoredCount, start);
+                return appliedDirectives.length > 0 ? { ...filtered, appliedDirectives } : filtered;
+            }
+            catch (error) {
+                if (error instanceof CheckAbortedError)
+                    throw error;
+                const builder = ResultBuilder.create({
+                    checkId: config.id,
+                    itemType: config.itemType ?? 'files',
+                });
+                return builder.buildError(`Check ${config.slug} threw an error: ${error instanceof Error ? error.message : String(error)}`, error instanceof Error ? error : undefined);
+            }
+        },
+    };
+    return check;
+}
+/**
+ * Internal: Execute a check based on its analysis mode (analyze /
+ * analyzeAll / command).
+ * @throws {CheckAbortedError} When the check is aborted via AbortSignal
+ * @throws {SystemError} When an unknown analysis mode is encountered
+ */
+async function executeUnifiedCheck(config, ctx) {
+    const matchedFiles = await ctx.matchFiles();
+    // Filter by check's declared file types
+    const files = filterFilesByType(matchedFiles, config.fileTypes);
+    ctx.log(`Matched ${files.length} files`);
+    if (isAnalyzeConfig(config)) {
+        return executeAnalyzeMode(config, files, ctx);
+    }
+    else if (isAnalyzeAllConfig(config)) {
+        return executeAnalyzeAllMode(config, files, ctx);
+    }
+    else if (isCommandConfig(config)) {
+        return executeCommandMode(config, files, ctx);
+    }
+    /* v8 ignore start -- exhaustive check: all UnifiedCheckConfig variants are handled above; this throw fires only if someone introduces a new variant without updating this switch */
+    const _exhaustiveCheck = config;
+    throw new SystemError(`Unknown analysis mode: ${JSON.stringify(_exhaustiveCheck)}`, {
+        code: 'SYSTEM.FITNESS.UNKNOWN_MODE',
+    });
+    /* v8 ignore stop */
+}
+//# sourceMappingURL=define-check.js.map