threadlines 0.1.39 → 0.1.40

package/dist/commands/check.js

@@ -50,13 +50,30 @@ const git_diff_executor_1 = require("../utils/git-diff-executor");
 const fs = __importStar(require("fs"));
 const path = __importStar(require("path"));
 const chalk_1 = __importDefault(require("chalk"));
+const simple_git_1 = __importDefault(require("simple-git"));
 // Get CLI version from package.json
 const packageJsonPath = path.join(__dirname, '../../package.json');
 const packageJson = JSON.parse(fs.readFileSync(packageJsonPath, 'utf-8'));
 const CLI_VERSION = packageJson.version;
 async function checkCommand(options) {
-    const repoRoot = process.cwd();
+    const cwd = process.cwd();
+    const repoRoot = cwd; // Keep for backward compatibility with rest of function
     console.log(chalk_1.default.blue(`🔍 Threadline CLI v${CLI_VERSION}: Checking code against your threadlines...\n`));
+    // Get git root for consistent file paths across monorepo
+    const git = (0, simple_git_1.default)(cwd);
+    let gitRoot;
+    try {
+        const isRepo = await git.checkIsRepo();
+        if (!isRepo) {
+            console.error(chalk_1.default.red('❌ Error: Not a git repository. Threadline requires a git repository.'));
+            process.exit(1);
+        }
+        gitRoot = (await git.revparse(['--show-toplevel'])).trim();
+    }
+    catch {
+        console.error(chalk_1.default.red('❌ Error: Failed to get git root. Make sure you are in a git repository.'));
+        process.exit(1);
+    }
     // Pre-flight check: Validate ALL required environment variables at once
     const apiKey = (0, config_1.getThreadlineApiKey)();
     const account = (0, config_1.getThreadlineAccount)();
@@ -96,7 +113,7 @@ async function checkCommand(options) {
     try {
         // 1. Find and validate threadlines
         console.log(chalk_1.default.gray('📋 Finding threadlines...'));
-        const threadlines = await (0, experts_1.findThreadlines)(repoRoot);
+        const threadlines = await (0, experts_1.findThreadlines)(cwd, gitRoot);
         console.log(chalk_1.default.green(`✓ Found ${threadlines.length} threadline(s)\n`));
         if (threadlines.length === 0) {
             console.log(chalk_1.default.yellow('⚠️  No valid threadlines found.'));

package/dist/validators/experts.js

@@ -39,8 +39,13 @@ const fs = __importStar(require("fs"));
 const path = __importStar(require("path"));
 const yaml = __importStar(require("js-yaml"));
 const REQUIRED_FIELDS = ['id', 'version', 'patterns'];
-async function findThreadlines(repoRoot) {
-    const expertsDir = path.join(repoRoot, 'threadlines');
+/**
+ * Find and validate all threadlines in the threadlines folder.
+ * @param searchRoot - Where to look for threadlines (usually cwd)
+ * @param gitRoot - Git repository root (for consistent filePath across monorepo)
+ */
+async function findThreadlines(searchRoot, gitRoot) {
+    const expertsDir = path.join(searchRoot, 'threadlines');
     if (!fs.existsSync(expertsDir)) {
         throw new Error('No /threadlines folder found. Run `npx threadlines init` to create your first threadline.');
     }
@@ -51,7 +56,7 @@ async function findThreadlines(repoRoot) {
     }
     const threadlines = [];
     for (const file of expertFiles) {
-        const result = await validateThreadline(path.join(expertsDir, file), repoRoot);
+        const result = await validateThreadline(path.join(expertsDir, file), searchRoot, gitRoot);
         if (result.valid && result.threadline) {
             threadlines.push(result.threadline);
         }
@@ -61,7 +66,13 @@ async function findThreadlines(repoRoot) {
     }
     return threadlines;
 }
-async function validateThreadline(filePath, repoRoot) {
+/**
+ * Validate a threadline file and extract its configuration.
+ * @param filePath - Absolute path to the threadline file
+ * @param searchRoot - Where threadlines were searched from (for context file resolution)
+ * @param gitRoot - Git repository root (for consistent filePath in database)
+ */
+async function validateThreadline(filePath, searchRoot, gitRoot) {
     try {
         const content = fs.readFileSync(filePath, 'utf-8');
         // Parse YAML frontmatter
@@ -94,10 +105,10 @@ async function validateThreadline(filePath, repoRoot) {
                 errors.push('context_files must be an array');
             }
             else {
-                // Check if context files exist
+                // Check if context files exist (relative to searchRoot)
                 for (const contextFile of frontmatter.context_files) {
                     if (typeof contextFile === 'string') {
-                        const fullPath = path.join(repoRoot, contextFile);
+                        const fullPath = path.join(searchRoot, contextFile);
                         if (!fs.existsSync(fullPath)) {
                             errors.push(`Context file not found: ${contextFile}`);
                         }
@@ -117,13 +128,14 @@ async function validateThreadline(filePath, repoRoot) {
             return { valid: false, errors };
         }
         // Type assertions for required fields (already validated above)
+        // filePath is relative to gitRoot for consistent identification across monorepo
         const threadline = {
             id: frontmatter.id,
             version: frontmatter.version,
             patterns: frontmatter.patterns,
             contextFiles: (Array.isArray(frontmatter.context_files) ? frontmatter.context_files.filter((f) => typeof f === 'string') : []),
             content: body,
-            filePath: path.relative(repoRoot, filePath)
+            filePath: path.relative(gitRoot, filePath)
         };
         return { valid: true, threadline };
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "threadlines",
-  "version": "0.1.39",
+  "version": "0.1.40",
   "description": "Threadline CLI - AI-powered linter based on your natural language documentation",
   "main": "dist/index.js",
   "bin": {