@opensip-cli/checks-universal 0.1.0

package/dist/checks/quality/no-markdown-references.js

@@ -0,0 +1,145 @@
+// @fitness-ignore-file unused-config-options -- Config options reserved for future use or environment-specific
+// @fitness-ignore-file no-markdown-references -- Check's own JSDoc contains example .md references to document what it detects
+/**
+ * @fileoverview Detect markdown file references in code comments
+ *
+ * File path references to markdown documents become stale when documents are
+ * moved, renamed, or deleted. Use stable identifiers instead.
+ */
+import { defineCheck } from '@opensip-cli/fitness';
+/**
+ * Pattern to match .md file references in comments
+ * Matches patterns like:
+ *   - docs/adr/052-something.md
+ *   - ./README.md
+ *   - ../docs/guide.md
+ *   - CHANGELOG.md
+ */
+const MARKDOWN_REFERENCE_PATTERN = /(?:^|[\s(])([./\w-]+\.md)(?:[\s)]|$)/g;
+/**
+ * Pattern to detect if a line is likely a comment (not a template literal)
+ */
+const COMMENT_PATTERNS = [
+    /^\s*\/\//, // Single-line comment
+    /^\s*\/\*/, // Multi-line comment start (/**)
+    /^\s*\*(?!\*)/, // JSDoc continuation (* text) but NOT markdown bold (**text**)
+];
+/**
+ * Files/patterns to exclude from this check
+ */
+const EXCLUDED_PATTERNS = [
+    /\.md$/, // Don't check markdown files themselves
+    /CHANGELOG/, // Changelog files often reference other changelogs
+    /README/, // README files reference other docs
+    /__tests__/, // Test files may legitimately reference test fixtures
+    /\.test\.ts$/, // Test files
+    /\.spec\.ts$/, // Spec files
+];
+/**
+ * Well-known markdown file names that are stable references (not stale paths).
+ * These are root-level or conventionally-named files that don't become stale.
+ */
+const STABLE_REFERENCE_PATTERNS = [
+    /^CLAUDE\.md$/i, // Project-level AI guidance file
+    /^README\.md$/i, // Standard readme files
+    /^CHANGELOG\.md$/i, // Standard changelog files
+    /^CONTRIBUTING\.md$/i, // Contribution guidelines
+    /^LICENSE\.md$/i, // License file
+    /^CODE_OF_CONDUCT\.md$/i, // Code of conduct
+    /^SECURITY\.md$/i, // Security policy
+];
+/**
+ * Check if a line is a comment line
+ */
+function isCommentLine(line) {
+    return COMMENT_PATTERNS.some((pattern) => pattern.test(line));
+}
+/**
+ * Check if a file should be excluded
+ */
+function shouldExcludeFile(filePath) {
+    return EXCLUDED_PATTERNS.some((pattern) => pattern.test(filePath));
+}
+/**
+ * Find markdown references in a file
+ */
+function findMarkdownReferences(content) {
+    const references = [];
+    const lines = content.split('\n');
+    for (const [i, line] of lines.entries()) {
+        /* v8 ignore next -- defensive: lines.entries() never yields undefined */
+        if (line === undefined)
+            continue;
+        // Only check comment lines
+        if (!isCommentLine(line))
+            continue;
+        // Find all .md references in this line
+        let match;
+        MARKDOWN_REFERENCE_PATTERN.lastIndex = 0; // Reset regex state
+        while ((match = MARKDOWN_REFERENCE_PATTERN.exec(line)) !== null) {
+            const reference = match[1];
+            // Skip if it's just mentioning the file extension generically or undefined
+            if (!reference || reference === '.md')
+                continue;
+            // Skip well-known stable markdown file names (e.g., CLAUDE.md, README.md)
+            /* v8 ignore next -- defensive: split + pop on a non-empty string never returns undefined */
+            const basename = reference.split('/').pop() ?? reference;
+            if (STABLE_REFERENCE_PATTERNS.some((pattern) => pattern.test(basename)))
+                continue;
+            references.push({
+                lineNumber: i + 1,
+                line,
+                reference,
+            });
+        }
+    }
+    return references;
+}
+/**
+ * Check: quality/no-markdown-references
+ *
+ * Detects markdown file path references in code comments. These references
+ * become stale when documents are moved, renamed, or deleted.
+ *
+ * Use stable identifiers instead of file paths to avoid stale references.
+ */
+export const noMarkdownReferences = defineCheck({
+    id: '8be86917-6908-49e5-a185-a6bd18045b31',
+    slug: 'no-markdown-references',
+    scope: { languages: ['typescript'], concerns: ['backend', 'frontend', 'cli'] },
+    contentFilter: 'raw',
+    confidence: 'medium',
+    description: 'Detect markdown file references in code comments that may become stale',
+    longDescription: `**Purpose:** Detects markdown file path references in code comments that become stale when documents are moved, renamed, or deleted.
+
+**Detects:**
+- References matching \`[./\\w-]+\\.md\` in comment lines (single-line \`//\`, multi-line \`/* */\`, and JSDoc \`/** */\`)
+- Examples: \`docs/adr/052-something.md\`, \`./README.md\`, \`../docs/guide.md\`, \`CHANGELOG.md\`
+
+**Why it matters:** File path references silently break when documents are reorganized. Stable identifiers (e.g., document titles or IDs) remain valid regardless of file location.
+
+**Scope:** General best practice. Analyzes each file individually (\`analyze\`). Targets production files, excluding markdown, test, and changelog files.`,
+    tags: ['documentation', 'maintainability', 'quality'],
+    fileTypes: ['ts', 'tsx'],
+    analyze(content, filePath) {
+        // Skip excluded files
+        if (shouldExcludeFile(filePath)) {
+            return [];
+        }
+        const violations = [];
+        const references = findMarkdownReferences(content);
+        for (const { lineNumber, reference } of references) {
+            const suggestion = `Remove or replace the markdown file reference '${reference}'. Use stable identifiers (e.g., document titles or IDs) instead of file paths that break when documents are moved.`;
+            violations.push({
+                line: lineNumber,
+                column: 0,
+                message: `Markdown file reference '${reference}' may become stale`,
+                severity: 'warning',
+                suggestion,
+                match: reference,
+            });
+        }
+        return violations;
+    },
+});
+//# sourceMappingURL=no-markdown-references.js.map

package/dist/checks/quality/no-markdown-references.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"no-markdown-references.js","sourceRoot":"","sources":["../../../src/checks/quality/no-markdown-references.ts"],"names":[],"mappings":"AAAA,+GAA+G;AAC/G,+HAA+H;AAC/H;;;;;GAKG;AAEH,OAAO,EAAE,WAAW,EAAuB,MAAM,sBAAsB,CAAC;AAExE;;;;;;;GAOG;AACH,MAAM,0BAA0B,GAAG,uCAAuC,CAAC;AAE3E;;GAEG;AACH,MAAM,gBAAgB,GAAG;IACvB,UAAU,EAAE,sBAAsB;IAClC,UAAU,EAAE,iCAAiC;IAC7C,cAAc,EAAE,+DAA+D;CAChF,CAAC;AAEF;;GAEG;AACH,MAAM,iBAAiB,GAAG;IACxB,OAAO,EAAE,wCAAwC;IACjD,WAAW,EAAE,mDAAmD;IAChE,QAAQ,EAAE,oCAAoC;IAC9C,WAAW,EAAE,sDAAsD;IACnE,aAAa,EAAE,aAAa;IAC5B,aAAa,EAAE,aAAa;CAC7B,CAAC;AAEF;;;GAGG;AACH,MAAM,yBAAyB,GAAG;IAChC,eAAe,EAAE,iCAAiC;IAClD,eAAe,EAAE,wBAAwB;IACzC,kBAAkB,EAAE,2BAA2B;IAC/C,qBAAqB,EAAE,0BAA0B;IACjD,gBAAgB,EAAE,eAAe;IACjC,wBAAwB,EAAE,kBAAkB;IAC5C,iBAAiB,EAAE,kBAAkB;CACtC,CAAC;AAEF;;GAEG;AACH,SAAS,aAAa,CAAC,IAAY;IACjC,OAAO,gBAAgB,CAAC,IAAI,CAAC,CAAC,OAAO,EAAE,EAAE,CAAC,OAAO,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC,CAAC;AAChE,CAAC;AAED;;GAEG;AACH,SAAS,iBAAiB,CAAC,QAAgB;IACzC,OAAO,iBAAiB,CAAC,IAAI,CAAC,CAAC,OAAO,EAAE,EAAE,CAAC,OAAO,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC,CAAC;AACrE,CAAC;AAED;;GAEG;AACH,SAAS,sBAAsB,CAC7B,OAAe;IAEf,MAAM,UAAU,GAA8D,EAAE,CAAC;IACjF,MAAM,KAAK,GAAG,OAAO,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC;IAElC,KAAK,MAAM,CAAC,CAAC,EAAE,IAAI,CAAC,IAAI,KAAK,CAAC,OAAO,EAAE,EAAE,CAAC;QACxC,yEAAyE;QACzE,IAAI,IAAI,KAAK,SAAS;YAAE,SAAS;QAEjC,2BAA2B;QAC3B,IAAI,CAAC,aAAa,CAAC,IAAI,CAAC;YAAE,SAAS;QAEnC,uCAAuC;QACvC,IAAI,KAAK,CAAC;QACV,0BAA0B,CAAC,SAAS,GAAG,CAAC,CAAC,CAAC,oBAAoB;QAC9D,OAAO,CAAC,KAAK,GAAG,0BAA0B,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC,KAAK,IAAI,EAAE,CAAC;YAChE,MAAM,SAAS,GAAG,KAAK,CAAC,CAAC,CAAC,CAAC;YAE3B,2EAA2E;YAC3E,IAAI,CAAC,SAAS,IAAI,SAAS,KAAK,KAAK;gBAAE,SAAS;YAEhD,0EAA0E;YAC1E,4FAA4F;YAC5F,MAAM,QAAQ,GAAG,SAAS,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC,GAAG,EAAE,IAAI,SAAS,CAAC;YACzD,IAAI,yBAAyB,CAAC,IAAI,CAAC,CAAC,OAAO,EAAE,EAAE,CAAC,OAAO,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC;gBAAE,SAAS;YAElF,UAAU,CAAC,IAAI,CAAC;gBACd,UAAU,EAAE,CAAC,GAAG,CAAC;gBACjB,IAAI;gBACJ,SAAS;aACV,CAAC,CAAC;QACL,CAAC;IACH,CAAC;IAED,OAAO,UAAU,CAAC;AACpB,CAAC;AAED;;;;;;;GAOG;AACH,MAAM,CAAC,MAAM,oBAAoB,GAAG,WAAW,CAAC;IAC9C,EAAE,EAAE,sCAAsC;IAC1C,IAAI,EAAE,wBAAwB;IAC9B,KAAK,EAAE,EAAE,SAAS,EAAE,CAAC,YAAY,CAAC,EAAE,QAAQ,EAAE,CAAC,SAAS,EAAE,UAAU,EAAE,KAAK,CAAC,EAAE;IAC9E,aAAa,EAAE,KAAK;IAEpB,UAAU,EAAE,QAAQ;IACpB,WAAW,EAAE,wEAAwE;IACrF,eAAe,EAAE;;;;;;;;0JAQuI;IACxJ,IAAI,EAAE,CAAC,eAAe,EAAE,iBAAiB,EAAE,SAAS,CAAC;IACrD,SAAS,EAAE,CAAC,IAAI,EAAE,KAAK,CAAC;IAExB,OAAO,CAAC,OAAO,EAAE,QAAQ;QACvB,sBAAsB;QACtB,IAAI,iBAAiB,CAAC,QAAQ,CAAC,EAAE,CAAC;YAChC,OAAO,EAAE,CAAC;QACZ,CAAC;QAED,MAAM,UAAU,GAAqB,EAAE,CAAC;QACxC,MAAM,UAAU,GAAG,sBAAsB,CAAC,OAAO,CAAC,CAAC;QAEnD,KAAK,MAAM,EAAE,UAAU,EAAE,SAAS,EAAE,IAAI,UAAU,EAAE,CAAC;YACnD,MAAM,UAAU,GAAG,kDAAkD,SAAS,qHAAqH,CAAC;YAEpM,UAAU,CAAC,IAAI,CAAC;gBACd,IAAI,EAAE,UAAU;gBAChB,MAAM,EAAE,CAAC;gBACT,OAAO,EAAE,4BAA4B,SAAS,oBAAoB;gBAClE,QAAQ,EAAE,SAAS;gBACnB,UAAU;gBACV,KAAK,EAAE,SAAS;aACjB,CAAC,CAAC;QACL,CAAC;QAED,OAAO,UAAU,CAAC;IACpB,CAAC;CACF,CAAC,CAAC"}

package/dist/checks/quality/no-raw-regex-on-code.d.ts

@@ -0,0 +1,9 @@
+/**
+ * @fileoverview Advisory meta-check for content filter usage
+ *
+ * Detects fitness checks that use regex pattern matching on file content
+ * without declaring contentFilter: 'strip-strings', which may lead to
+ * false positives from string literal and comment content.
+ */
+export declare const noRawRegexOnCode: import("@opensip-cli/fitness").Check;
+//# sourceMappingURL=no-raw-regex-on-code.d.ts.map

package/dist/checks/quality/no-raw-regex-on-code.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"no-raw-regex-on-code.d.ts","sourceRoot":"","sources":["../../../src/checks/quality/no-raw-regex-on-code.ts"],"names":[],"mappings":"AAAA;;;;;;GAMG;AASH,eAAO,MAAM,gBAAgB,sCAoD3B,CAAC"}

package/dist/checks/quality/no-raw-regex-on-code.js

@@ -0,0 +1,61 @@
+/**
+ * @fileoverview Advisory meta-check for content filter usage
+ *
+ * Detects fitness checks that use regex pattern matching on file content
+ * without declaring contentFilter: 'strip-strings', which may lead to
+ * false positives from string literal and comment content.
+ */
+import { defineCheck } from '@opensip-cli/fitness';
+const REGEX_USAGE_PATTERN = /\.(test|match|exec|search)\s*\(/;
+const CONTENT_FILTER_PATTERN = /contentFilter\s*:\s*['"](raw|strip-strings|strip-strings-and-comments)['"]/;
+const DEFINE_CHECK_PATTERN = /defineCheck\s*\(/;
+export const noRawRegexOnCode = defineCheck({
+    id: '7a0f6bc1-f4dd-4e55-9628-d797c877c6e0',
+    slug: 'no-raw-regex-on-code',
+    scope: { languages: ['typescript'], concerns: ['fitness'] },
+    confidence: 'medium',
+    description: 'Detect regex checks that should use contentFilter: strip-strings',
+    longDescription: `**Purpose:** Advisory meta-check that identifies fitness checks using regex pattern matching without declaring \`contentFilter: 'strip-strings'\`. Such checks may produce false positives when patterns match inside string literals or comments.
+
+**Detects:**
+- Check files that call \`.test()\`, \`.match()\`, \`.exec()\`, or \`.search()\` on content
+- That do NOT declare \`contentFilter: 'strip-strings'\` or \`contentFilter: 'raw'\`
+
+**Why it matters:** Regex checks without content filtering match patterns inside string literals and documentation, producing false positives. Adding \`contentFilter: 'strip-strings'\` to the check config eliminates this class of false positives.
+
+**Scope:** Fitness check files only.`,
+    tags: ['quality', 'internal', 'meta', 'content-filter'],
+    fileTypes: ['ts'],
+    analyze(content, filePath) {
+        // Only analyze fitness check files
+        if (!filePath.includes('fitness/src/checks/'))
+            return [];
+        // Must be a defineCheck file
+        if (!DEFINE_CHECK_PATTERN.test(content))
+            return [];
+        // Check if it uses regex methods
+        if (!REGEX_USAGE_PATTERN.test(content))
+            return [];
+        // Check if contentFilter is already declared (either value)
+        if (CONTENT_FILTER_PATTERN.test(content))
+            return [];
+        // Find the line with defineCheck to report the violation
+        const lines = content.split('\n');
+        for (const [i, line] of lines.entries()) {
+            if (line && DEFINE_CHECK_PATTERN.test(line)) {
+                return [
+                    {
+                        line: i + 1,
+                        column: 0,
+                        message: 'Check uses regex pattern matching without contentFilter declaration',
+                        severity: 'warning',
+                        type: 'missing-content-filter',
+                        suggestion: "Add contentFilter: 'strip-strings' to skip string literals, or contentFilter: 'raw' to explicitly opt out (e.g., for secret detection checks).",
+                    },
+                ];
+            }
+        }
+        return [];
+    },
+});
+//# sourceMappingURL=no-raw-regex-on-code.js.map

package/dist/checks/quality/no-raw-regex-on-code.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"no-raw-regex-on-code.js","sourceRoot":"","sources":["../../../src/checks/quality/no-raw-regex-on-code.ts"],"names":[],"mappings":"AAAA;;;;;;GAMG;AAEH,OAAO,EAAE,WAAW,EAAuB,MAAM,sBAAsB,CAAC;AAExE,MAAM,mBAAmB,GAAG,iCAAiC,CAAC;AAC9D,MAAM,sBAAsB,GAC1B,4EAA4E,CAAC;AAC/E,MAAM,oBAAoB,GAAG,kBAAkB,CAAC;AAEhD,MAAM,CAAC,MAAM,gBAAgB,GAAG,WAAW,CAAC;IAC1C,EAAE,EAAE,sCAAsC;IAC1C,IAAI,EAAE,sBAAsB;IAC5B,KAAK,EAAE,EAAE,SAAS,EAAE,CAAC,YAAY,CAAC,EAAE,QAAQ,EAAE,CAAC,SAAS,CAAC,EAAE;IAE3D,UAAU,EAAE,QAAQ;IACpB,WAAW,EAAE,kEAAkE;IAC/E,eAAe,EAAE;;;;;;;;qCAQkB;IACnC,IAAI,EAAE,CAAC,SAAS,EAAE,UAAU,EAAE,MAAM,EAAE,gBAAgB,CAAC;IACvD,SAAS,EAAE,CAAC,IAAI,CAAC;IAEjB,OAAO,CAAC,OAAO,EAAE,QAAQ;QACvB,mCAAmC;QACnC,IAAI,CAAC,QAAQ,CAAC,QAAQ,CAAC,qBAAqB,CAAC;YAAE,OAAO,EAAE,CAAC;QAEzD,6BAA6B;QAC7B,IAAI,CAAC,oBAAoB,CAAC,IAAI,CAAC,OAAO,CAAC;YAAE,OAAO,EAAE,CAAC;QAEnD,iCAAiC;QACjC,IAAI,CAAC,mBAAmB,CAAC,IAAI,CAAC,OAAO,CAAC;YAAE,OAAO,EAAE,CAAC;QAElD,4DAA4D;QAC5D,IAAI,sBAAsB,CAAC,IAAI,CAAC,OAAO,CAAC;YAAE,OAAO,EAAE,CAAC;QAEpD,yDAAyD;QACzD,MAAM,KAAK,GAAG,OAAO,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC;QAClC,KAAK,MAAM,CAAC,CAAC,EAAE,IAAI,CAAC,IAAI,KAAK,CAAC,OAAO,EAAE,EAAE,CAAC;YACxC,IAAI,IAAI,IAAI,oBAAoB,CAAC,IAAI,CAAC,IAAI,CAAC,EAAE,CAAC;gBAC5C,OAAO;oBACL;wBACE,IAAI,EAAE,CAAC,GAAG,CAAC;wBACX,MAAM,EAAE,CAAC;wBACT,OAAO,EAAE,qEAAqE;wBAC9E,QAAQ,EAAE,SAAS;wBACnB,IAAI,EAAE,wBAAwB;wBAC9B,UAAU,EACR,gJAAgJ;qBACnJ;iBACF,CAAC;YACJ,CAAC;QACH,CAAC;QAED,OAAO,EAAE,CAAC;IACZ,CAAC;CACF,CAAC,CAAC"}

package/dist/checks/quality/no-temporary-workarounds.d.ts

@@ -0,0 +1,11 @@
+/**
+ * @fileoverview Detects temporary-workaround comments.
+ *
+ * Catches `HACK`/`FIXME` lines that explicitly describe themselves
+ * as `temporary`, a `workaround`, or "before launch" — these
+ * announcements are signs that a permanent fix was deferred.
+ *
+ * Extracted from the former `no-legacy-code` umbrella in Phase C4.
+ */
+export declare const noTemporaryWorkarounds: import("@opensip-cli/fitness").Check;
+//# sourceMappingURL=no-temporary-workarounds.d.ts.map

package/dist/checks/quality/no-temporary-workarounds.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"no-temporary-workarounds.d.ts","sourceRoot":"","sources":["../../../src/checks/quality/no-temporary-workarounds.ts"],"names":[],"mappings":"AAEA;;;;;;;;GAQG;AAmBH,eAAO,MAAM,sBAAsB,sCA6CjC,CAAC"}

package/dist/checks/quality/no-temporary-workarounds.js

@@ -0,0 +1,69 @@
+/* eslint-disable sonarjs/fixme-tag -- this file's documentation references the FIXME marker by design */
+// @fitness-ignore-file no-todo-comments -- this file's job is to detect FIXME/HACK workaround markers; the words appear in JSDoc by design
+/**
+ * @fileoverview Detects temporary-workaround comments.
+ *
+ * Catches `HACK`/`FIXME` lines that explicitly describe themselves
+ * as `temporary`, a `workaround`, or "before launch" — these
+ * announcements are signs that a permanent fix was deferred.
+ *
+ * Extracted from the former `no-legacy-code` umbrella in Phase C4.
+ */
+import { defineCheck } from '@opensip-cli/fitness';
+const EXCLUDE_PATTERNS = [/fitness/, /test/, /spec/, /docs/, /reports/, /versioning/];
+function shouldExcludeFile(relativePath) {
+    return EXCLUDE_PATTERNS.some((pattern) => pattern.test(relativePath));
+}
+const MARKER_NEEDLES = ['hack', 'fixme'];
+const QUALIFIER_NEEDLES = ['before launch', 'temporary', 'workaround'];
+function matchTemporaryWorkaround(line) {
+    const lower = line.toLowerCase();
+    if (!MARKER_NEEDLES.some((m) => lower.includes(m)))
+        return false;
+    return QUALIFIER_NEEDLES.some((q) => lower.includes(q));
+}
+export const noTemporaryWorkarounds = defineCheck({
+    id: '09a93ec8-7b08-47b2-946a-c635e135b67b',
+    slug: 'no-temporary-workarounds',
+    scope: { languages: ['typescript'], concerns: ['backend', 'frontend', 'cli'] },
+    contentFilter: 'raw',
+    confidence: 'medium',
+    description: 'Detects HACK/FIXME comments that describe themselves as temporary',
+    longDescription: `**Purpose:** Flags \`HACK\`/\`FIXME\` comments that explicitly mark themselves as temporary, "before launch", or workarounds. These announcements are reliable signals that a permanent fix was deferred.
+
+**Detects:**
+- Lines containing \`HACK\` or \`FIXME\` *and* one of \`temporary\`, \`workaround\`, \`before launch\`
+
+**Why it matters:** Temporary workarounds outlive their context. Surfacing them on every fitness run keeps the conversation about the permanent fix open.
+
+**Scope:** General best practice. Analyzes each file individually; excludes test/docs/versioning paths.`,
+    tags: ['code-quality', 'compliance', 'quality'],
+    fileTypes: ['ts', 'tsx'],
+    analyze(content, filePath) {
+        if (shouldExcludeFile(filePath))
+            return [];
+        const lower = content.toLowerCase();
+        const hasMarker = MARKER_NEEDLES.some((m) => lower.includes(m));
+        if (!hasMarker)
+            return [];
+        const violations = [];
+        const lines = content.split('\n');
+        for (const [i, line] of lines.entries()) {
+            if (!line)
+                continue;
+            if (matchTemporaryWorkaround(line)) {
+                violations.push({
+                    line: i + 1,
+                    column: 0,
+                    message: 'Found temporary workaround - implement permanent solution before launch',
+                    severity: 'error',
+                    type: 'temporary-workaround',
+                    suggestion: 'Replace temporary workaround with a permanent, production-ready solution',
+                    match: line.trim(),
+                });
+            }
+        }
+        return violations;
+    },
+});
+//# sourceMappingURL=no-temporary-workarounds.js.map

package/dist/checks/quality/no-temporary-workarounds.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"no-temporary-workarounds.js","sourceRoot":"","sources":["../../../src/checks/quality/no-temporary-workarounds.ts"],"names":[],"mappings":"AAAA,yGAAyG;AACzG,2IAA2I;AAC3I;;;;;;;;GAQG;AAEH,OAAO,EAAE,WAAW,EAAuB,MAAM,sBAAsB,CAAC;AAExE,MAAM,gBAAgB,GAAG,CAAC,SAAS,EAAE,MAAM,EAAE,MAAM,EAAE,MAAM,EAAE,SAAS,EAAE,YAAY,CAAC,CAAC;AAEtF,SAAS,iBAAiB,CAAC,YAAoB;IAC7C,OAAO,gBAAgB,CAAC,IAAI,CAAC,CAAC,OAAO,EAAE,EAAE,CAAC,OAAO,CAAC,IAAI,CAAC,YAAY,CAAC,CAAC,CAAC;AACxE,CAAC;AAED,MAAM,cAAc,GAAG,CAAC,MAAM,EAAE,OAAO,CAAU,CAAC;AAClD,MAAM,iBAAiB,GAAG,CAAC,eAAe,EAAE,WAAW,EAAE,YAAY,CAAU,CAAC;AAEhF,SAAS,wBAAwB,CAAC,IAAY;IAC5C,MAAM,KAAK,GAAG,IAAI,CAAC,WAAW,EAAE,CAAC;IACjC,IAAI,CAAC,cAAc,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,KAAK,CAAC,QAAQ,CAAC,CAAC,CAAC,CAAC;QAAE,OAAO,KAAK,CAAC;IACjE,OAAO,iBAAiB,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,KAAK,CAAC,QAAQ,CAAC,CAAC,CAAC,CAAC,CAAC;AAC1D,CAAC;AAED,MAAM,CAAC,MAAM,sBAAsB,GAAG,WAAW,CAAC;IAChD,EAAE,EAAE,sCAAsC;IAC1C,IAAI,EAAE,0BAA0B;IAChC,KAAK,EAAE,EAAE,SAAS,EAAE,CAAC,YAAY,CAAC,EAAE,QAAQ,EAAE,CAAC,SAAS,EAAE,UAAU,EAAE,KAAK,CAAC,EAAE;IAC9E,aAAa,EAAE,KAAK;IACpB,UAAU,EAAE,QAAQ;IACpB,WAAW,EAAE,mEAAmE;IAChF,eAAe,EAAE;;;;;;;wGAOqF;IACtG,IAAI,EAAE,CAAC,cAAc,EAAE,YAAY,EAAE,SAAS,CAAC;IAC/C,SAAS,EAAE,CAAC,IAAI,EAAE,KAAK,CAAC;IAExB,OAAO,CAAC,OAAO,EAAE,QAAQ;QACvB,IAAI,iBAAiB,CAAC,QAAQ,CAAC;YAAE,OAAO,EAAE,CAAC;QAE3C,MAAM,KAAK,GAAG,OAAO,CAAC,WAAW,EAAE,CAAC;QACpC,MAAM,SAAS,GAAG,cAAc,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,KAAK,CAAC,QAAQ,CAAC,CAAC,CAAC,CAAC,CAAC;QAChE,IAAI,CAAC,SAAS;YAAE,OAAO,EAAE,CAAC;QAE1B,MAAM,UAAU,GAAqB,EAAE,CAAC;QACxC,MAAM,KAAK,GAAG,OAAO,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC;QAElC,KAAK,MAAM,CAAC,CAAC,EAAE,IAAI,CAAC,IAAI,KAAK,CAAC,OAAO,EAAE,EAAE,CAAC;YACxC,IAAI,CAAC,IAAI;gBAAE,SAAS;YACpB,IAAI,wBAAwB,CAAC,IAAI,CAAC,EAAE,CAAC;gBACnC,UAAU,CAAC,IAAI,CAAC;oBACd,IAAI,EAAE,CAAC,GAAG,CAAC;oBACX,MAAM,EAAE,CAAC;oBACT,OAAO,EAAE,yEAAyE;oBAClF,QAAQ,EAAE,OAAO;oBACjB,IAAI,EAAE,sBAAsB;oBAC5B,UAAU,EAAE,0EAA0E;oBACtF,KAAK,EAAE,IAAI,CAAC,IAAI,EAAE;iBACnB,CAAC,CAAC;YACL,CAAC;QACH,CAAC;QAED,OAAO,UAAU,CAAC;IACpB,CAAC;CACF,CAAC,CAAC"}

package/dist/checks/quality/no-window-alert.d.ts

@@ -0,0 +1,19 @@
+/**
+ * @fileoverview No window.alert/confirm/prompt Check
+ *
+ * Detects usage of window.alert(), window.confirm(), and window.prompt()
+ * in frontend code. These native browser dialogs provide poor UX and
+ * should be replaced with modal components or toast notifications.
+ *
+ * Migrated to defineRegexListCheck (Layer 4 Phase C6). The original
+ * site's `break // Only report once per line` is preserved via the
+ * helper's `oneViolationPerLine: true` option, and the import-skip is
+ * preserved via `skipLine`.
+ */
+/**
+ * Check: quality/no-window-alert
+ *
+ * Prevents usage of native browser dialogs in frontend code.
+ */
+export declare const noWindowAlert: import("@opensip-cli/fitness").Check;
+//# sourceMappingURL=no-window-alert.d.ts.map

package/dist/checks/quality/no-window-alert.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"no-window-alert.d.ts","sourceRoot":"","sources":["../../../src/checks/quality/no-window-alert.ts"],"names":[],"mappings":"AACA;;;;;;;;;;;GAWG;AAIH;;;;GAIG;AACH,eAAO,MAAM,aAAa,sCAsDxB,CAAC"}

package/dist/checks/quality/no-window-alert.js

@@ -0,0 +1,74 @@
+// @fitness-ignore-file no-window-alert -- Fitness check definition references window.alert/confirm/prompt in string literals and regex patterns, not actual usage
+/**
+ * @fileoverview No window.alert/confirm/prompt Check
+ *
+ * Detects usage of window.alert(), window.confirm(), and window.prompt()
+ * in frontend code. These native browser dialogs provide poor UX and
+ * should be replaced with modal components or toast notifications.
+ *
+ * Migrated to defineRegexListCheck (Layer 4 Phase C6). The original
+ * site's `break // Only report once per line` is preserved via the
+ * helper's `oneViolationPerLine: true` option, and the import-skip is
+ * preserved via `skipLine`.
+ */
+import { defineRegexListCheck } from '@opensip-cli/fitness';
+/**
+ * Check: quality/no-window-alert
+ *
+ * Prevents usage of native browser dialogs in frontend code.
+ */
+export const noWindowAlert = defineRegexListCheck({
+    id: '170b156b-a45d-4f1a-af7a-a40ed507afe0',
+    slug: 'no-window-alert',
+    scope: { languages: ['typescript', 'tsx'], concerns: ['frontend', 'ui'] },
+    contentFilter: 'strip-strings',
+    confidence: 'medium',
+    description: 'Disallows window.alert(), window.confirm(), and window.prompt() — use proper UI components',
+    longDescription: `**Purpose:** Prevents usage of native browser dialog APIs in frontend code, enforcing proper UI components instead.
+
+**Detects:**
+- \`window.alert()\` calls -- should use toast notifications or modal dialogs
+- \`window.confirm()\` calls -- should use confirmation modal dialogs
+- \`window.prompt()\` calls -- should use form inputs in modal dialogs
+
+**Why it matters:** Native browser dialogs block the main thread, cannot be styled, and provide a jarring, inconsistent user experience compared to in-app UI components.
+
+**Scope:** General best practice. Analyzes each file individually (\`analyze\`). Targets frontend files (preset: \`frontend\`), excluding tests.`,
+    tags: ['frontend', 'ux', 'quality', 'best-practices'],
+    fileTypes: ['ts', 'tsx'],
+    options: {
+        // Original site skipped lines starting with `import ` (defensive
+        // against lines like `import { alert } from "..."`).
+        skipLine: (trimmed) => trimmed.startsWith('import '),
+        // Original site `break`s after the first matching pattern to emit
+        // only one violation per line.
+        oneViolationPerLine: true,
+    },
+    patterns: [
+        {
+            id: '7a0f6bc1-f4dd-4e55-9628-d797c877c6e0',
+            slug: 'window-alert',
+            regex: /window\.alert\s*\(/,
+            message: 'window.alert() provides poor UX — replace with a proper UI component',
+            severity: 'error',
+            suggestion: 'Use a toast notification or modal dialog instead of window.alert()',
+        },
+        {
+            id: '09a93ec8-7b08-47b2-946a-c635e135b67b',
+            slug: 'window-confirm',
+            regex: /window\.confirm\s*\(/,
+            message: 'window.confirm() provides poor UX — replace with a proper UI component',
+            severity: 'error',
+            suggestion: 'Use a confirmation modal dialog instead of window.confirm()',
+        },
+        {
+            id: 'e39edca8-ee4d-4de8-9a39-655f4d0eb86d',
+            slug: 'window-prompt',
+            regex: /window\.prompt\s*\(/,
+            message: 'window.prompt() provides poor UX — replace with a proper UI component',
+            severity: 'error',
+            suggestion: 'Use a form input in a modal dialog instead of window.prompt()',
+        },
+    ],
+});
+//# sourceMappingURL=no-window-alert.js.map

package/dist/checks/quality/no-window-alert.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"no-window-alert.js","sourceRoot":"","sources":["../../../src/checks/quality/no-window-alert.ts"],"names":[],"mappings":"AAAA,kKAAkK;AAClK;;;;;;;;;;;GAWG;AAEH,OAAO,EAAE,oBAAoB,EAAE,MAAM,sBAAsB,CAAC;AAE5D;;;;GAIG;AACH,MAAM,CAAC,MAAM,aAAa,GAAG,oBAAoB,CAAC;IAChD,EAAE,EAAE,sCAAsC;IAC1C,IAAI,EAAE,iBAAiB;IACvB,KAAK,EAAE,EAAE,SAAS,EAAE,CAAC,YAAY,EAAE,KAAK,CAAC,EAAE,QAAQ,EAAE,CAAC,UAAU,EAAE,IAAI,CAAC,EAAE;IACzE,aAAa,EAAE,eAAe;IAC9B,UAAU,EAAE,QAAQ;IACpB,WAAW,EACT,4FAA4F;IAC9F,eAAe,EAAE;;;;;;;;;iJAS8H;IAC/I,IAAI,EAAE,CAAC,UAAU,EAAE,IAAI,EAAE,SAAS,EAAE,gBAAgB,CAAC;IACrD,SAAS,EAAE,CAAC,IAAI,EAAE,KAAK,CAAC;IACxB,OAAO,EAAE;QACP,iEAAiE;QACjE,qDAAqD;QACrD,QAAQ,EAAE,CAAC,OAAO,EAAE,EAAE,CAAC,OAAO,CAAC,UAAU,CAAC,SAAS,CAAC;QACpD,kEAAkE;QAClE,+BAA+B;QAC/B,mBAAmB,EAAE,IAAI;KAC1B;IACD,QAAQ,EAAE;QACR;YACE,EAAE,EAAE,sCAAsC;YAC1C,IAAI,EAAE,cAAc;YACpB,KAAK,EAAE,oBAAoB;YAC3B,OAAO,EAAE,sEAAsE;YAC/E,QAAQ,EAAE,OAAO;YACjB,UAAU,EAAE,oEAAoE;SACjF;QACD;YACE,EAAE,EAAE,sCAAsC;YAC1C,IAAI,EAAE,gBAAgB;YACtB,KAAK,EAAE,sBAAsB;YAC7B,OAAO,EAAE,wEAAwE;YACjF,QAAQ,EAAE,OAAO;YACjB,UAAU,EAAE,6DAA6D;SAC1E;QACD;YACE,EAAE,EAAE,sCAAsC;YAC1C,IAAI,EAAE,eAAe;YACrB,KAAK,EAAE,qBAAqB;YAC5B,OAAO,EAAE,uEAAuE;YAChF,QAAQ,EAAE,OAAO;YACjB,UAAU,EAAE,+DAA+D;SAC5E;KACF;CACF,CAAC,CAAC"}

package/dist/checks/quality/observability/index.d.ts

@@ -0,0 +1,2 @@
+export * from './pino-serializer-coverage.js';
+//# sourceMappingURL=index.d.ts.map

package/dist/checks/quality/observability/index.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../../../src/checks/quality/observability/index.ts"],"names":[],"mappings":"AAAA,cAAc,+BAA+B,CAAC"}

package/dist/checks/quality/observability/index.js

@@ -0,0 +1,2 @@
+export * from './pino-serializer-coverage.js';
+//# sourceMappingURL=index.js.map

package/dist/checks/quality/observability/index.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"index.js","sourceRoot":"","sources":["../../../../src/checks/quality/observability/index.ts"],"names":[],"mappings":"AAAA,cAAc,+BAA+B,CAAC"}

package/dist/checks/quality/observability/pino-serializer-coverage.d.ts

@@ -0,0 +1,15 @@
+/**
+ * @fileoverview Pino Serializer Coverage Check
+ *
+ * Validates that complex objects logged with Pino have serializers:
+ * - Objects without registered serializers break structured logging
+ * - Circular references cause logging failures
+ * - Large objects need truncation
+ */
+/**
+ * Check: quality/pino-serializer-coverage
+ *
+ * Validates that complex objects logged have proper Pino serializers.
+ */
+export declare const pinoSerializerCoverage: import("@opensip-cli/fitness").Check;
+//# sourceMappingURL=pino-serializer-coverage.d.ts.map

package/dist/checks/quality/observability/pino-serializer-coverage.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"pino-serializer-coverage.d.ts","sourceRoot":"","sources":["../../../../src/checks/quality/observability/pino-serializer-coverage.ts"],"names":[],"mappings":"AAAA;;;;;;;GAOG;AA2NH;;;;GAIG;AACH,eAAO,MAAM,sBAAsB,sCAwBjC,CAAC"}