@codeyam/codeyam-cli 0.1.21 → 0.1.23

package/codeyam-cli/src/commands/__tests__/editor.analyzeImportsArgs.test.js

@@ -0,0 +1,47 @@
+/**
+ * Tests that `codeyam editor analyze-imports` correctly parses optional
+ * file path arguments to scope analysis to specific files.
+ *
+ * Without file paths, analyze-imports analyzes all glossary entities.
+ * With file paths, it scopes analysis to only those files.
+ */
+import { parseAnalyzeImportsArgs } from "../editorAnalyzeImportsArgs.js";
+describe('parseAnalyzeImportsArgs', () => {
+    it('should return empty array when no args provided', () => {
+        expect(parseAnalyzeImportsArgs(undefined, [])).toEqual([]);
+        expect(parseAnalyzeImportsArgs('', [])).toEqual([]);
+    });
+    it('should parse a single file path from json arg', () => {
+        // codeyam editor analyze-imports app/components/Foo.tsx
+        expect(parseAnalyzeImportsArgs('app/components/Foo.tsx', [])).toEqual([
+            'app/components/Foo.tsx',
+        ]);
+    });
+    it('should collect extra positional args from argv._', () => {
+        // codeyam editor analyze-imports app/components/Foo.tsx app/components/Bar.tsx
+        // yargs: json="app/components/Foo.tsx", argv._=["editor", "app/components/Bar.tsx"]
+        expect(parseAnalyzeImportsArgs('app/components/Foo.tsx', [
+            'editor',
+            'app/components/Bar.tsx',
+        ])).toEqual(['app/components/Foo.tsx', 'app/components/Bar.tsx']);
+    });
+    it('should filter out "editor" from argv._', () => {
+        expect(parseAnalyzeImportsArgs(undefined, [
+            'editor',
+            'app/components/Foo.tsx',
+            'app/lib/utils.ts',
+        ])).toEqual(['app/components/Foo.tsx', 'app/lib/utils.ts']);
+    });
+    it('should deduplicate file paths', () => {
+        expect(parseAnalyzeImportsArgs('app/components/Foo.tsx', [
+            'editor',
+            'app/components/Foo.tsx',
+            'app/components/Bar.tsx',
+        ])).toEqual(['app/components/Foo.tsx', 'app/components/Bar.tsx']);
+    });
+    it('should handle numeric extras gracefully', () => {
+        // yargs can put numbers in argv._
+        expect(parseAnalyzeImportsArgs('app/components/Foo.tsx', [42, 'editor'])).toEqual(['app/components/Foo.tsx']);
+    });
+});
+//# sourceMappingURL=editor.analyzeImportsArgs.test.js.map

package/codeyam-cli/src/commands/__tests__/editor.analyzeImportsArgs.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"editor.analyzeImportsArgs.test.js","sourceRoot":"","sources":["../../../../../src/commands/__tests__/editor.analyzeImportsArgs.test.ts"],"names":[],"mappings":"AAAA;;;;;;GAMG;AAEH,OAAO,EAAE,uBAAuB,EAAE,MAAM,6BAA6B,CAAC;AAEtE,QAAQ,CAAC,yBAAyB,EAAE,GAAG,EAAE;IACvC,EAAE,CAAC,iDAAiD,EAAE,GAAG,EAAE;QACzD,MAAM,CAAC,uBAAuB,CAAC,SAAS,EAAE,EAAE,CAAC,CAAC,CAAC,OAAO,CAAC,EAAE,CAAC,CAAC;QAC3D,MAAM,CAAC,uBAAuB,CAAC,EAAE,EAAE,EAAE,CAAC,CAAC,CAAC,OAAO,CAAC,EAAE,CAAC,CAAC;IACtD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,+CAA+C,EAAE,GAAG,EAAE;QACvD,wDAAwD;QACxD,MAAM,CAAC,uBAAuB,CAAC,wBAAwB,EAAE,EAAE,CAAC,CAAC,CAAC,OAAO,CAAC;YACpE,wBAAwB;SACzB,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,kDAAkD,EAAE,GAAG,EAAE;QAC1D,+EAA+E;QAC/E,oFAAoF;QACpF,MAAM,CACJ,uBAAuB,CAAC,wBAAwB,EAAE;YAChD,QAAQ;YACR,wBAAwB;SACzB,CAAC,CACH,CAAC,OAAO,CAAC,CAAC,wBAAwB,EAAE,wBAAwB,CAAC,CAAC,CAAC;IAClE,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,wCAAwC,EAAE,GAAG,EAAE;QAChD,MAAM,CACJ,uBAAuB,CAAC,SAAS,EAAE;YACjC,QAAQ;YACR,wBAAwB;YACxB,kBAAkB;SACnB,CAAC,CACH,CAAC,OAAO,CAAC,CAAC,wBAAwB,EAAE,kBAAkB,CAAC,CAAC,CAAC;IAC5D,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,+BAA+B,EAAE,GAAG,EAAE;QACvC,MAAM,CACJ,uBAAuB,CAAC,wBAAwB,EAAE;YAChD,QAAQ;YACR,wBAAwB;YACxB,wBAAwB;SACzB,CAAC,CACH,CAAC,OAAO,CAAC,CAAC,wBAAwB,EAAE,wBAAwB,CAAC,CAAC,CAAC;IAClE,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,yCAAyC,EAAE,GAAG,EAAE;QACjD,kCAAkC;QAClC,MAAM,CACJ,uBAAuB,CAAC,wBAAwB,EAAE,CAAC,EAAE,EAAE,QAAQ,CAAC,CAAC,CAClE,CAAC,OAAO,CAAC,CAAC,wBAAwB,CAAC,CAAC,CAAC;IACxC,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}

package/codeyam-cli/src/commands/__tests__/editor.auditNoAutoAnalysis.test.js

@@ -0,0 +1,71 @@
+import * as fs from 'fs';
+import * as path from 'path';
+/**
+ * Structural test: handleAudit must NOT call runAnalysisForEntities.
+ *
+ * Previously, handleAudit auto-ran full analysis for incomplete entities,
+ * which was slow (analyzer startup + AST scan for all glossary entries).
+ *
+ * Targeted analysis (with filePaths for 1-3 specific files) is acceptable,
+ * but a full scan (without filePaths) must never be used here.
+ * This test ensures the full auto-analysis doesn't creep back in.
+ */
+describe('handleAudit - no auto-analysis', () => {
+    it('should not call runAnalysisForEntities inside handleAudit', () => {
+        const editorTsPath = path.join(__dirname, '..', 'editor.ts');
+        const source = fs.readFileSync(editorTsPath, 'utf8');
+        // Extract the handleAudit function body.
+        // It starts with "async function handleAudit" and ends at the next
+        // top-level "async function" or "function" declaration (unindented).
+        const handleAuditStart = source.indexOf('async function handleAudit');
+        expect(handleAuditStart).toBeGreaterThan(-1);
+        // Find the next top-level function declaration after handleAudit starts.
+        // Top-level functions start at the beginning of a line (no indentation).
+        const afterStart = source.slice(handleAuditStart + 1);
+        const nextFnMatch = afterStart.match(/\n(?:async )?function [a-zA-Z]/);
+        const handleAuditEnd = nextFnMatch
+            ? handleAuditStart + 1 + nextFnMatch.index
+            : source.length;
+        const handleAuditBody = source.slice(handleAuditStart, handleAuditEnd);
+        // The function body must NOT contain runAnalysisForEntities
+        expect(handleAuditBody).not.toContain('runAnalysisForEntities');
+    });
+    it('should report incomplete entities with guidance instead of auto-fixing', () => {
+        const editorTsPath = path.join(__dirname, '..', 'editor.ts');
+        const source = fs.readFileSync(editorTsPath, 'utf8');
+        const handleAuditStart = source.indexOf('async function handleAudit');
+        const afterStart = source.slice(handleAuditStart + 1);
+        const nextFnMatch = afterStart.match(/\n(?:async )?function [a-zA-Z]/);
+        const handleAuditEnd = nextFnMatch
+            ? handleAuditStart + 1 + nextFnMatch.index
+            : source.length;
+        const handleAuditBody = source.slice(handleAuditStart, handleAuditEnd);
+        // Should still report incomplete entities
+        expect(handleAuditBody).toContain('incompleteEntities');
+        expect(handleAuditBody).toContain('formatIncompleteEntityGuidance');
+    });
+    it('should only call handleAnalyzeImports with filePaths (targeted, not full scan)', () => {
+        const editorTsPath = path.join(__dirname, '..', 'editor.ts');
+        const source = fs.readFileSync(editorTsPath, 'utf8');
+        const handleAuditStart = source.indexOf('async function handleAudit');
+        expect(handleAuditStart).toBeGreaterThan(-1);
+        const afterStart = source.slice(handleAuditStart + 1);
+        const nextFnMatch = afterStart.match(/\n(?:async )?function [a-zA-Z]/);
+        const handleAuditEnd = nextFnMatch
+            ? handleAuditStart + 1 + nextFnMatch.index
+            : source.length;
+        const handleAuditBody = source.slice(handleAuditStart, handleAuditEnd);
+        // Targeted handleAnalyzeImports (with filePaths) is OK — it only
+        // analyzes 1-3 specific files. A full scan (without filePaths) takes
+        // minutes on large projects and must never be used here.
+        const calls = [...handleAuditBody.matchAll(/handleAnalyzeImports\s*\(/g)];
+        for (const call of calls) {
+            const afterCall = handleAuditBody.slice(call.index + call[0].length);
+            // Each call must include filePaths in its arguments
+            const closingParen = afterCall.indexOf(')');
+            const callArgs = afterCall.slice(0, closingParen);
+            expect(callArgs).toContain('filePaths');
+        }
+    });
+});
+//# sourceMappingURL=editor.auditNoAutoAnalysis.test.js.map

package/codeyam-cli/src/commands/__tests__/editor.auditNoAutoAnalysis.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"editor.auditNoAutoAnalysis.test.js","sourceRoot":"","sources":["../../../../../src/commands/__tests__/editor.auditNoAutoAnalysis.test.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,MAAM,IAAI,CAAC;AACzB,OAAO,KAAK,IAAI,MAAM,MAAM,CAAC;AAE7B;;;;;;;;;GASG;AACH,QAAQ,CAAC,gCAAgC,EAAE,GAAG,EAAE;IAC9C,EAAE,CAAC,2DAA2D,EAAE,GAAG,EAAE;QACnE,MAAM,YAAY,GAAG,IAAI,CAAC,IAAI,CAAC,SAAS,EAAE,IAAI,EAAE,WAAW,CAAC,CAAC;QAC7D,MAAM,MAAM,GAAG,EAAE,CAAC,YAAY,CAAC,YAAY,EAAE,MAAM,CAAC,CAAC;QAErD,yCAAyC;QACzC,mEAAmE;QACnE,qEAAqE;QACrE,MAAM,gBAAgB,GAAG,MAAM,CAAC,OAAO,CAAC,4BAA4B,CAAC,CAAC;QACtE,MAAM,CAAC,gBAAgB,CAAC,CAAC,eAAe,CAAC,CAAC,CAAC,CAAC,CAAC;QAE7C,yEAAyE;QACzE,yEAAyE;QACzE,MAAM,UAAU,GAAG,MAAM,CAAC,KAAK,CAAC,gBAAgB,GAAG,CAAC,CAAC,CAAC;QACtD,MAAM,WAAW,GAAG,UAAU,CAAC,KAAK,CAAC,gCAAgC,CAAC,CAAC;QACvE,MAAM,cAAc,GAAG,WAAW;YAChC,CAAC,CAAC,gBAAgB,GAAG,CAAC,GAAG,WAAW,CAAC,KAAM;YAC3C,CAAC,CAAC,MAAM,CAAC,MAAM,CAAC;QAElB,MAAM,eAAe,GAAG,MAAM,CAAC,KAAK,CAAC,gBAAgB,EAAE,cAAc,CAAC,CAAC;QAEvE,4DAA4D;QAC5D,MAAM,CAAC,eAAe,CAAC,CAAC,GAAG,CAAC,SAAS,CAAC,wBAAwB,CAAC,CAAC;IAClE,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,wEAAwE,EAAE,GAAG,EAAE;QAChF,MAAM,YAAY,GAAG,IAAI,CAAC,IAAI,CAAC,SAAS,EAAE,IAAI,EAAE,WAAW,CAAC,CAAC;QAC7D,MAAM,MAAM,GAAG,EAAE,CAAC,YAAY,CAAC,YAAY,EAAE,MAAM,CAAC,CAAC;QAErD,MAAM,gBAAgB,GAAG,MAAM,CAAC,OAAO,CAAC,4BAA4B,CAAC,CAAC;QACtE,MAAM,UAAU,GAAG,MAAM,CAAC,KAAK,CAAC,gBAAgB,GAAG,CAAC,CAAC,CAAC;QACtD,MAAM,WAAW,GAAG,UAAU,CAAC,KAAK,CAAC,gCAAgC,CAAC,CAAC;QACvE,MAAM,cAAc,GAAG,WAAW;YAChC,CAAC,CAAC,gBAAgB,GAAG,CAAC,GAAG,WAAW,CAAC,KAAM;YAC3C,CAAC,CAAC,MAAM,CAAC,MAAM,CAAC;QAElB,MAAM,eAAe,GAAG,MAAM,CAAC,KAAK,CAAC,gBAAgB,EAAE,cAAc,CAAC,CAAC;QAEvE,0CAA0C;QAC1C,MAAM,CAAC,eAAe,CAAC,CAAC,SAAS,CAAC,oBAAoB,CAAC,CAAC;QACxD,MAAM,CAAC,eAAe,CAAC,CAAC,SAAS,CAAC,gCAAgC,CAAC,CAAC;IACtE,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,gFAAgF,EAAE,GAAG,EAAE;QACxF,MAAM,YAAY,GAAG,IAAI,CAAC,IAAI,CAAC,SAAS,EAAE,IAAI,EAAE,WAAW,CAAC,CAAC;QAC7D,MAAM,MAAM,GAAG,EAAE,CAAC,YAAY,CAAC,YAAY,EAAE,MAAM,CAAC,CAAC;QAErD,MAAM,gBAAgB,GAAG,MAAM,CAAC,OAAO,CAAC,4BAA4B,CAAC,CAAC;QACtE,MAAM,CAAC,gBAAgB,CAAC,CAAC,eAAe,CAAC,CAAC,CAAC,CAAC,CAAC;QAE7C,MAAM,UAAU,GAAG,MAAM,CAAC,KAAK,CAAC,gBAAgB,GAAG,CAAC,CAAC,CAAC;QACtD,MAAM,WAAW,GAAG,UAAU,CAAC,KAAK,CAAC,gCAAgC,CAAC,CAAC;QACvE,MAAM,cAAc,GAAG,WAAW;YAChC,CAAC,CAAC,gBAAgB,GAAG,CAAC,GAAG,WAAW,CAAC,KAAM;YAC3C,CAAC,CAAC,MAAM,CAAC,MAAM,CAAC;QAElB,MAAM,eAAe,GAAG,MAAM,CAAC,KAAK,CAAC,gBAAgB,EAAE,cAAc,CAAC,CAAC;QAEvE,iEAAiE;QACjE,qEAAqE;QACrE,yDAAyD;QACzD,MAAM,KAAK,GAAG,CAAC,GAAG,eAAe,CAAC,QAAQ,CAAC,4BAA4B,CAAC,CAAC,CAAC;QAC1E,KAAK,MAAM,IAAI,IAAI,KAAK,EAAE,CAAC;YACzB,MAAM,SAAS,GAAG,eAAe,CAAC,KAAK,CAAC,IAAI,CAAC,KAAM,GAAG,IAAI,CAAC,CAAC,CAAC,CAAC,MAAM,CAAC,CAAC;YACtE,oDAAoD;YACpD,MAAM,YAAY,GAAG,SAAS,CAAC,OAAO,CAAC,GAAG,CAAC,CAAC;YAC5C,MAAM,QAAQ,GAAG,SAAS,CAAC,KAAK,CAAC,CAAC,EAAE,YAAY,CAAC,CAAC;YAClD,MAAM,CAAC,QAAQ,CAAC,CAAC,SAAS,CAAC,WAAW,CAAC,CAAC;QAC1C,CAAC;IACH,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}