@chemx/starter-kit 26.10.4-1258 → 26.10.4-235

package/cli/audit/ai-slop-detector.js

@@ -73,38 +73,10 @@ const normalizeWords = (text) => {
     .filter((w) => w.length > 1 && !STOP_WORDS.has(w));
 };
 
-// A line-based scan cannot tell a comment from a string that quotes one. Walk the line
-// up to the match and track quote state: if a quote is still open at that offset, the
-// match sits inside a string literal and is describing the pattern, not committing it.
-// Lines that are wholly a comment short-circuit, so apostrophes in prose cannot open a
-// phantom string and suppress a real finding.
-const COMMENT_LINE_START = /^\s*(?:\/\/|#|\*|--)/;
-
-const isInsideStringLiteral = (lineText, matchIndex) => {
-  if (matchIndex <= 0) return false;
-  if (COMMENT_LINE_START.test(lineText)) return false;
-
-  let quote = null;
-  for (let i = 0; i < matchIndex; i += 1) {
-    const ch = lineText[i];
-    if (ch === '\\') {
-      i += 1;
-      continue;
-    }
-    if (quote) {
-      if (ch === quote) quote = null;
-    } else if (ch === '"' || ch === "'" || ch === '`') {
-      quote = ch;
-    }
-  }
-  return quote !== null;
-};
-
 export const checkSlopTextPatterns = (content, lines, relativePath, violations) => {
   lines.forEach((lineText, idx) => {
     for (const pat of CONVERSATIONAL_PATTERNS) {
-      const hit = pat.regex.exec(lineText);
-      if (hit && !isInsideStringLiteral(lineText, hit.index)) {
+      if (pat.regex.test(lineText)) {
         const meta = RULE_REGISTRY[pat.rule];
         violations.push({
           filePath: relativePath,

package/cli/reader.js

@@ -4,8 +4,6 @@ import { parse } from '@babel/parser';
 import traverseModule from '@babel/traverse';
 import { ANSI } from './theme.js';
 import { resolveSafePath } from './path-scope.js';
-import { isBabelParsable } from './languages.js';
-import { extractAstMetadata } from './search-ast.js';
 
 import {
   stripCodeComments,
@@ -36,12 +34,6 @@ const traverse = traverseModule.default || traverseModule;
  * @param {string} filePath File path for parser context.
  * @returns {string} Compressed structural outline.
  */
-const OUTLINE_KIND_LABELS = {
-  class: 'class',
-  function: 'function',
-  symbol: 'symbol'
-};
-
 export const generateAstOutline = (code, filePath) => {
   const isVue = filePath.endsWith('.vue');
   const isSvelte = filePath.endsWith('.svelte');
@@ -60,21 +52,6 @@ export const generateAstOutline = (code, filePath) => {
   const lines = [];
   lines.push(`// Outline: ${filePath}`);
 
-  // Babel cannot parse C/C++, Python, Go, Rust, Java, C# or Kotlin. It also does not
-  // throw on them, because errorRecovery swallows the failure and yields an empty AST,
-  // so the catch-block fallback below never fires for these files. Route them to the
-  // polyglot extractor instead.
-  if (!isBabelParsable(filePath)) {
-    const meta = extractAstMetadata(scriptContent, filePath);
-    const seen = new Set();
-    for (const sym of meta.symbols) {
-      if (!sym?.name || seen.has(sym.name)) continue;
-      seen.add(sym.name);
-      lines.push(`${OUTLINE_KIND_LABELS[sym.kind] || 'symbol'} ${sym.name}`);
-    }
-    return lines.join('\n');
-  }
-
   try {
     const ast = parse(scriptContent, {
       sourceType: 'module',

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@chemx/starter-kit",
-  "version": "26.10.4-1258",
+  "version": "26.10.4-235",
   "description": "Chemical X Protocol: Private drop-in architecture starter kit and capsule generator",
   "type": "module",
   "bin": {

package/cli/audit/slop-string-literal.spec.js

@@ -1,34 +0,0 @@
-import test from 'node:test';
-import assert from 'node:assert/strict';
-import { auditCode } from './rules.js';
-
-const fired = (code, file = 'src/thing.js') =>
-  auditCode(code, file, file).some((v) => v.rule === 'AI_SLOP_LAZY_PLACEHOLDER');
-
-test('slop: real truncation placeholder in a comment still fires', () => {
-  assert.equal(fired('function a() {\n  // ... rest of code\n}\n'), true);
-});
-
-test('slop: trailing truncation comment after code still fires', () => {
-  assert.equal(fired('doThing();  // ... remaining implementation\n'), true);
-});
-
-test('slop: placeholder text inside a double-quoted string does not fire', () => {
-  const code = 'const copy = "Blocks lazy \'// ...rest of code\' truncation placeholders";\n';
-  assert.equal(fired(code), false, 'marketing copy describing the pattern must not be flagged');
-});
-
-test('slop: placeholder text inside a single-quoted string does not fire', () => {
-  const code = "const copy = 'we detect // ... existing code markers';\n";
-  assert.equal(fired(code), false);
-});
-
-test('slop: placeholder inside a template literal does not fire', () => {
-  const code = 'const copy = `catches // ... remaining logic in your diff`;\n';
-  assert.equal(fired(code), false);
-});
-
-test('slop: string-literal guard applies to non-JS languages too', () => {
-  const py = 'DESCRIPTION = "flags // ... rest of code placeholders"\n';
-  assert.equal(fired(py, 'services/detect.py'), false);
-});

package/cli/reader-polyglot.spec.js

@@ -1,47 +0,0 @@
-import test from 'node:test';
-import assert from 'node:assert/strict';
-import { generateAstOutline } from './reader.js';
-
-const CPP = `
-#include <memory>
-namespace audio {
-struct FrameHeader { uint32_t size; };
-class StreamDecoder : public IDecoder {
-public:
-    bool DecodeFrame(const FrameHeader& header);
-};
-}
-`;
-
-const PY = `
-import os
-
-class ConsentService:
-    def revoke(self, user_id):
-        return True
-
-def helper(x):
-    return x
-`;
-
-test('outline: renders C++ declarations instead of an empty header', () => {
-  const out = generateAstOutline(CPP, 'src/audio/StreamDecoder.cpp');
-  const body = out.split('\n').filter((l) => !l.startsWith('// Outline:'));
-  assert.ok(body.length > 0, `expected symbols, got only the header:\n${out}`);
-  assert.ok(out.includes('StreamDecoder'), `expected StreamDecoder in:\n${out}`);
-  assert.ok(out.includes('FrameHeader'), `expected FrameHeader in:\n${out}`);
-});
-
-test('outline: renders Python declarations instead of an empty header', () => {
-  const out = generateAstOutline(PY, 'services/consent.py');
-  const body = out.split('\n').filter((l) => !l.startsWith('// Outline:'));
-  assert.ok(body.length > 0, `expected symbols, got only the header:\n${out}`);
-  assert.ok(out.includes('ConsentService'), `expected ConsentService in:\n${out}`);
-});
-
-test('outline: still renders JavaScript exports unchanged', () => {
-  const js = `export const FOO = 1;\nexport function bar(a) { return a; }\n`;
-  const out = generateAstOutline(js, 'cli/thing.js');
-  assert.ok(out.includes('FOO'), out);
-  assert.ok(out.includes('bar'), out);
-});