@claudetools/tools 0.3.1 → 0.3.3

package/dist/setup.js

@@ -7,8 +7,9 @@ import chalk from 'chalk';
 import ora from 'ora';
 import { homedir, hostname, platform } from 'os';
 import { join, basename } from 'path';
-import { existsSync, readFileSync, writeFileSync, mkdirSync, copyFileSync } from 'fs';
+import { existsSync, readFileSync, writeFileSync, mkdirSync, copyFileSync, realpathSync } from 'fs';
 import { randomUUID } from 'crypto';
+import { execSync } from 'child_process';
 import { loadConfigFromFile, saveConfig, ensureConfigDir, getConfigPath, DEFAULT_CONFIG, } from './helpers/config-manager.js';
 import { GLOBAL_TEMPLATE, SECTION_START, SECTION_END, getProjectTemplate, } from './templates/claude-md.js';
 // -----------------------------------------------------------------------------
@@ -110,6 +111,29 @@ function initializeProjectsFile() {
         }, null, 2));
     }
 }
+/**
+ * Detect git remote for a directory (secure implementation)
+ */
+function detectGitRemote(localPath) {
+    try {
+        // Resolve symlinks and validate path to prevent path traversal attacks
+        const resolvedPath = realpathSync(localPath);
+        // Ensure path is absolute and doesn't contain path traversal
+        if (!resolvedPath.startsWith('/') || resolvedPath.includes('..')) {
+            return undefined;
+        }
+        const gitRemote = execSync('git config --get remote.origin.url', {
+            cwd: resolvedPath,
+            encoding: 'utf-8',
+            stdio: ['pipe', 'pipe', 'ignore'],
+        }).trim();
+        return gitRemote || undefined;
+    }
+    catch {
+        // Not a git repo or no remote configured
+        return undefined;
+    }
+}
 // -----------------------------------------------------------------------------
 // Authentication
 // -----------------------------------------------------------------------------
@@ -480,9 +504,11 @@ if [ -f "$WATCHER_PID_FILE" ]; then
 fi
 
 # Start watcher in background if not running
+# The watcher manages its own PID file, so don't write it here
 if [ ! -f "$WATCHER_PID_FILE" ] && command -v claudetools &> /dev/null; then
-  nohup claudetools watch > /tmp/claudetools-watcher.log 2>&1 &
-  echo $! > "$WATCHER_PID_FILE"
+  nohup claudetools watch >> /tmp/claudetools-watcher.log 2>&1 &
+  # Give the watcher a moment to start and write its PID file
+  sleep 1
 fi
 
 # --- Context Injection ---
@@ -1088,58 +1114,106 @@ export async function runInit() {
         error('ClaudeTools not configured. Run "claudetools --setup" first.');
         process.exit(1);
     }
+    // Load system info
+    const systemInfo = loadSystemInfo();
+    if (!systemInfo?.system_id) {
+        error('System not registered. Run "claudetools --setup" first.');
+        process.exit(1);
+    }
+    // Detect git remote for this directory
+    const gitRemote = detectGitRemote(cwd);
     // Try to register project with API
     const spinner = ora('Registering project...').start();
     let projectId;
-    try {
-        const response = await fetch(`${config.apiUrl || DEFAULT_CONFIG.apiUrl}/api/v1/projects/register`, {
-            method: 'POST',
-            headers: {
-                'Content-Type': 'application/json',
-                'Authorization': `Bearer ${config.apiKey}`,
-            },
-            body: JSON.stringify({
-                name: projectName,
-                local_path: cwd,
-            }),
-        });
-        if (response.ok) {
-            const data = await response.json();
-            projectId = data.project_id;
-            spinner.succeed(`Registered project: ${projectId}`);
-            // Update local projects.json
-            try {
-                const projectsData = existsSync(PROJECTS_FILE)
-                    ? JSON.parse(readFileSync(PROJECTS_FILE, 'utf-8'))
-                    : { bindings: [] };
-                // Check if already in bindings
-                const existingIdx = projectsData.bindings.findIndex((b) => b.local_path === cwd);
-                if (existingIdx >= 0) {
-                    projectsData.bindings[existingIdx].project_id = projectId;
+    // Use auto-bind endpoint if git remote exists
+    if (gitRemote) {
+        try {
+            const response = await fetch(`${config.apiUrl || DEFAULT_CONFIG.apiUrl}/api/v1/systems/${systemInfo.system_id}/bindings/auto`, {
+                method: 'POST',
+                headers: {
+                    'Content-Type': 'application/json',
+                    'Authorization': `Bearer ${config.apiKey}`,
+                },
+                body: JSON.stringify({
+                    local_path: cwd,
+                    git_remote: gitRemote,
+                    project_name: projectName,
+                }),
+            });
+            if (response.ok) {
+                const data = await response.json();
+                projectId = data.data?.project_id || `local_${projectName.toLowerCase().replace(/[^a-z0-9]/g, '_')}`;
+                const registeredName = data.data?.project_name || projectName;
+                spinner.succeed(`Registered project: ${registeredName} (${projectId})`);
+                // Update local projects.json
+                try {
+                    const projectsData = existsSync(PROJECTS_FILE)
+                        ? JSON.parse(readFileSync(PROJECTS_FILE, 'utf-8'))
+                        : { bindings: [], system_id: systemInfo.system_id };
+                    // Check if already in bindings
+                    const existingIdx = projectsData.bindings.findIndex((b) => b.local_path === cwd);
+                    if (existingIdx >= 0) {
+                        projectsData.bindings[existingIdx].project_id = projectId;
+                        projectsData.bindings[existingIdx].project_name = registeredName;
+                        projectsData.bindings[existingIdx].git_remote = gitRemote;
+                    }
+                    else {
+                        projectsData.bindings.push({
+                            project_id: projectId,
+                            local_path: cwd,
+                            project_name: registeredName,
+                            git_remote: gitRemote,
+                            system_id: systemInfo.system_id,
+                            cached_at: new Date().toISOString(),
+                        });
+                    }
+                    projectsData.last_sync = new Date().toISOString();
+                    writeFileSync(PROJECTS_FILE, JSON.stringify(projectsData, null, 2));
                 }
-                else {
-                    projectsData.bindings.push({
-                        project_id: projectId,
-                        local_path: cwd,
-                        project_name: projectName,
-                        cached_at: new Date().toISOString(),
-                    });
+                catch {
+                    // Non-fatal
                 }
-                projectsData.last_sync = new Date().toISOString();
-                writeFileSync(PROJECTS_FILE, JSON.stringify(projectsData, null, 2));
             }
-            catch {
-                // Non-fatal
+            else {
+                const errorData = await response.json().catch(() => ({}));
+                spinner.warn(`Could not register with API: ${errorData.error || response.status}`);
+                projectId = `local_${projectName.toLowerCase().replace(/[^a-z0-9]/g, '_')}`;
             }
         }
-        else {
-            spinner.warn('Could not register with API, using local ID');
+        catch (err) {
+            spinner.warn('Could not reach API, using local ID');
             projectId = `local_${projectName.toLowerCase().replace(/[^a-z0-9]/g, '_')}`;
         }
     }
-    catch {
-        spinner.warn('Could not reach API, using local ID');
+    else {
+        // No git remote - use local ID
+        spinner.info('No git remote detected, using local project ID');
         projectId = `local_${projectName.toLowerCase().replace(/[^a-z0-9]/g, '_')}`;
+        // Still update local projects.json for session context
+        try {
+            const projectsData = existsSync(PROJECTS_FILE)
+                ? JSON.parse(readFileSync(PROJECTS_FILE, 'utf-8'))
+                : { bindings: [], system_id: systemInfo.system_id };
+            const existingIdx = projectsData.bindings.findIndex((b) => b.local_path === cwd);
+            if (existingIdx >= 0) {
+                projectsData.bindings[existingIdx].project_id = projectId;
+                projectsData.bindings[existingIdx].project_name = projectName;
+            }
+            else {
+                projectsData.bindings.push({
+                    project_id: projectId,
+                    local_path: cwd,
+                    project_name: projectName,
+                    system_id: systemInfo.system_id,
+                    cached_at: new Date().toISOString(),
+                });
+            }
+            projectsData.last_sync = new Date().toISOString();
+            writeFileSync(PROJECTS_FILE, JSON.stringify(projectsData, null, 2));
+        }
+        catch {
+            // Non-fatal
+        }
     }
     // Create .claude directory if needed
     if (!existsSync(projectClaudeDir)) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@claudetools/tools",
-  "version": "0.3.1",
+  "version": "0.3.3",
   "description": "Persistent AI memory, task management, and codebase intelligence for Claude Code",
   "type": "module",
   "main": "dist/index.js",