scai 0.1.107 → 0.1.109

package/README.md

@@ -559,7 +559,7 @@ You can run it in two ways:
 
 ---
 
-### Backup only of `~/.scai`:**
+### Backup only of `~/.scai`:
 
 Creates a backup of the .scai folder in the user root dir. 
 
@@ -569,7 +569,8 @@ Creates a backup of the .scai folder in the user root dir.
 
 Results in this:
 ~/.scai_backup_2025-08-12T06-58-00-227Z/
----
+
+</br>
 
 ## 🔐 License & Fair Use
 

package/dist/CHANGELOG.md

@@ -162,4 +162,13 @@ Type handling with the module pipeline
 ## 2025-08-31
 
 • Update Spinner class to correctly display text and frames
-• Update CLI help to reflect new agent workflow examples.
+• Update CLI help to reflect new agent workflow examples.
+
+## 2025-09-01
+
+* Improve handling of GitHub repository URLs and paths by extracting the owner and name separately
+
+## 2025-09-02
+
+• Added test configuration for project and generated tests
+• Add runTestsModule and repairTestsModule for testing pipeline

package/dist/__tests__/example.test.js

@@ -0,0 +1,10 @@
+import { SCAI_HOME } from "../constants"; // example constant
+// cli/src/__tests__/example.test.ts
+describe("CLI src basic test", () => {
+    it("should pass a simple truthy test", () => {
+        expect(true).toBe(true);
+    });
+    it("should import a constant from cli/src/constants.ts", () => {
+        expect(typeof SCAI_HOME).not.toBe("undefined");
+    });
+});

package/dist/agent/agentManager.js

@@ -18,8 +18,20 @@ export class Agent {
         // Resolve modules (with before/after dependencies)
         const modules = this.resolveModules(this.goals);
         console.log(chalk.green("📋 Modules to run:"), modules.map((m) => m.name).join(" → "));
-        // Read file content (optional, could be used by modules in workflow)
-        await fs.readFile(filepath, "utf-8");
+        try {
+            // Check that the file exists before trying to read it
+            await fs.access(filepath);
+            // Read file content (optional, could be used by modules in workflow)
+            await fs.readFile(filepath, "utf-8");
+        }
+        catch (err) {
+            if (err.code === "ENOENT") {
+                console.error(chalk.redBright("❌ Error:"), `File not found: ${chalk.yellow(filepath)}`);
+                console.error(`Make sure the path is correct. (cwd: ${chalk.gray(process.cwd())})`);
+                process.exit(1);
+            }
+            throw err; // rethrow for unexpected errors
+        }
         // Delegate everything to handleAgentRun (like CLI commands do)
         await handleAgentRun(filepath, modules);
         console.log(chalk.green("✅ Agent finished!"));

package/dist/agent/workflowManager.js

@@ -76,6 +76,11 @@ export async function handleAgentRun(filepath, modules) {
                     baseChunks.length = 0;
                     baseChunks.push(...reset);
                     break;
+                case 'skip':
+                    console.log(chalk.gray(`⏭️ Skipped writing for module ${mod.name}`));
+                    // don’t touch files, but keep chunks flowing
+                    workingChunks = processed;
+                    break;
                 default:
                     console.log(chalk.yellow(`⚠️ Unknown mode; skipping write`));
                     // still move pipeline forward with processed

package/dist/github/repo.js

@@ -15,12 +15,19 @@ function runGitCommand(cmd, cwd) {
  */
 function getRepoOwnerAndNameFromGit(indexDir) {
     try {
-        const originUrl = runGitCommand('git config --get remote.origin.url', indexDir);
+        const originUrl = runGitCommand('git config --get remote.origin.url', indexDir).trim();
         console.log(`🔗 Git origin URL from '${indexDir}': ${originUrl}`);
-        const match = originUrl.match(/github[^:/]*[:/](.+?)(?:\.git)?$/);
+        // Handle both SSH and HTTPS formats, with or without extra slashes
+        // Examples:
+        // - git@github.com:owner/repo.git
+        // - git@github.com:/owner/repo.git
+        // - https://github.com/owner/repo.git
+        const match = originUrl.match(/github[^:/]*[:/]\/?(.+?)(?:\.git)?$/);
         if (!match)
             throw new Error("❌ Could not parse GitHub repo from origin URL.");
         const [owner, repo] = match[1].split('/');
+        if (!owner || !repo)
+            throw new Error("❌ Failed to extract owner or repo name from URL.");
         console.log(`✅ Parsed from Git: owner='${owner}', repo='${repo}'`);
         return { owner, repo };
     }

package/dist/pipeline/modules/cleanGeneratedTestsModule.js

@@ -7,16 +7,27 @@ export const cleanGeneratedTestsModule = {
         // normalize + strip markdown
         const normalized = normalizeText(content);
         const stripped = stripMarkdownFences(normalized);
-        // filter non-code lines
-        const lines = stripped.split("\n");
         // filter non-code lines, but keep blank ones
+        const lines = stripped.split("\n");
         const codeLines = lines.filter(line => line.trim() === "" || isCodeLike(line));
-        const cleanedCode = codeLines.join("\n");
+        // remove duplicate imports (normalize spacing/semicolon)
+        const seenImports = new Set();
+        const dedupedLines = codeLines.filter(line => {
+            if (line.trim().startsWith("import")) {
+                const key = line.trim().replace(/;$/, "");
+                if (seenImports.has(key)) {
+                    return false;
+                }
+                seenImports.add(key);
+            }
+            return true;
+        });
+        const cleanedCode = dedupedLines.join("\n").trimEnd();
         return {
             originalContent: content,
-            content: cleanedCode, // cleaned code for pipeline
-            filepath, // original file path
-            mode: "overwrite", // indicates overwrite existing file
+            content: cleanedCode,
+            filepath,
+            mode: "overwrite",
         };
     }
 };

package/dist/pipeline/modules/repairTestsModule.js

@@ -0,0 +1,40 @@
+import { generate } from "../../lib/generate.js";
+import { Config } from "../../config.js";
+export const repairTestsModule = {
+    name: "repairTestsModule",
+    description: "Fix failing Jest tests using AI",
+    async run({ content, filepath, summary }) {
+        const model = Config.getModel();
+        const prompt = `
+You are a senior engineer tasked with repairing Jest tests.
+
+The following test file failed:
+
+--- BEGIN TEST FILE ---
+${content}
+--- END TEST FILE ---
+
+Failure summary:
+${summary || "No summary provided."}
+
+Instructions:
+- Keep the overall structure, imports, and test cases.
+- Only fix syntax errors, invalid Jest matchers, or broken references.
+- Do NOT remove or replace entire test suites unless strictly necessary.
+- Do NOT generate trivial placeholder tests (like add(2,3) examples).
+- Only return valid Jest code, no explanations, no markdown fences.
+
+Output the repaired test file:
+`.trim();
+        const response = await generate({ content: prompt }, model);
+        if (!response)
+            throw new Error("⚠️ No repaired test code returned from model");
+        return {
+            originalContent: content,
+            content: response.content, // repaired test code
+            filepath, // repair in-place
+            summary: `Repaired tests based on failure: ${summary || "unknown error"}`,
+            mode: "overwrite" // signal to overwrite existing test file
+        };
+    }
+};

package/dist/pipeline/modules/runTestsModule.js

@@ -0,0 +1,37 @@
+import { execSync } from "child_process";
+import path from "path";
+export const runTestsModule = {
+    name: "runTestsModule",
+    description: "Runs generated Jest tests with safety checks for a single file",
+    async run(input) {
+        const { filepath } = input;
+        if (!filepath) {
+            throw new Error("No filepath provided to runTestsModule.");
+        }
+        const absoluteFilePath = path.resolve(filepath);
+        try {
+            // Step 1: TypeScript syntax check for this file only using tsconfig.test.json
+            // Step 1: TypeScript syntax check for this file only
+            execSync(`npx tsc --noEmit --project tsconfig.test.json`, { stdio: "inherit" });
+            // Step 2: Dry-run Jest for this file only
+            execSync(`npx jest --config ${path.resolve("jest.config.ts")} --dryRun ${absoluteFilePath}`, { stdio: "inherit" });
+            // Step 3: Full Jest run for this file only
+            execSync(`npx jest --config ${path.resolve("jest.config.ts")} ${absoluteFilePath}`, { stdio: "inherit" });
+            return {
+                content: "✅ Tests ran successfully",
+                filepath,
+                mode: "skip",
+                summary: "All tests passed successfully."
+            };
+        }
+        catch (error) {
+            const errorMessage = error?.message || String(error);
+            return {
+                content: `❌ Test run failed:\n${errorMessage}`,
+                filepath,
+                mode: "skip",
+                summary: errorMessage // provides failure context for repair
+            };
+        }
+    }
+};

package/dist/pipeline/registry/moduleRegistry.js

@@ -6,6 +6,8 @@ import { commitSuggesterModule } from '../modules/commitSuggesterModule.js';
 import { changelogModule } from '../modules/changeLogModule.js';
 import { cleanGeneratedTestsModule } from '../modules/cleanGeneratedTestsModule.js';
 import { preserveCodeModule } from '../modules/preserveCodeModule.js';
+import { runTestsModule } from '../modules/runTestsModule.js';
+import { repairTestsModule } from '../modules/repairTestsModule.js';
 // Built-in modules with metadata
 export const builtInModules = {
     comments: {
@@ -38,6 +40,21 @@ export const builtInModules = {
         ...cleanGeneratedTestsModule,
         group: 'testing',
     },
+    runTests: {
+        ...runTestsModule,
+        group: 'testing',
+        dependencies: {
+            before: ['tests'], // must exist after tests are generated
+            after: ['cleanTests'], // run after cleaning
+        },
+    },
+    repairTests: {
+        ...repairTestsModule,
+        group: 'testing',
+        dependencies: {
+            after: ['runTests'], // repair runs after tests have been executed
+        },
+    },
     suggest: {
         ...commitSuggesterModule,
         group: 'git',

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "scai",
-  "version": "0.1.107",
+  "version": "0.1.109",
   "type": "module",
   "bin": {
     "scai": "./dist/index.js"

package/dist/jest.config.js

@@ -1,11 +0,0 @@
-const config = {
-    preset: 'ts-jest',
-    testEnvironment: 'node',
-    setupFilesAfterEnv: ['./jest.setup.ts'],
-    transform: {
-        '^.+\\.ts$': 'ts-jest',
-    },
-    moduleFileExtensions: ['ts', 'js', 'json', 'node'],
-    testPathIgnorePatterns: ['/node_modules/', '/dist/'],
-};
-export default config;

package/dist/jest.setup.js

@@ -1,14 +0,0 @@
-import { jest } from '@jest/globals';
-// jest.setup.ts
-// Mock the global fetch function in Jest for Node.js 18+
-global.fetch = jest.fn(() => Promise.resolve({
-    json: () => Promise.resolve({ response: 'Mocked Commit Message' }),
-    ok: true,
-    status: 200,
-    statusText: 'OK',
-    headers: new Headers(),
-    redirected: false,
-    type: 'default',
-    url: 'https://mocked-url.com',
-}) // Type assertion for `Response` object
-);

package/dist/utils/runTests.js

@@ -1,11 +0,0 @@
-import { execSync } from 'child_process';
-export function runJestTest(testFile) {
-    try {
-        execSync(`npx jest ${testFile} --silent`, { stdio: 'inherit' });
-        return true;
-    }
-    catch (err) {
-        console.error(`❌ Tests failed for ${testFile}`);
-        return false;
-    }
-}