ropilot 0.1.42 → 0.1.44

package/lib/proxy.js

@@ -122,13 +122,49 @@ async function handleToolsList(apiKey, request) {
 }
 
 /**
- * Write large script content to a temp file and return the path
+ * Track script path -> local file mappings for sourcecontrol
+ */
+const scriptLocalFiles = new Map();
+
+/**
+ * Convert Studio path to local file path in src/ folder
+ * e.g., "game.ServerScriptService.MyScript" -> "src/ServerScriptService/MyScript.lua"
+ */
+function studioPathToLocalPath(studioPath) {
+  const parts = studioPath.split('.');
+  // Remove "game" prefix
+  if (parts[0] === 'game') {
+    parts.shift();
+  }
+  // Join with / and add .lua extension
+  return join('src', ...parts) + '.lua';
+}
+
+/**
+ * Write script content to local file and track the mapping
+ */
+function writeScriptToLocalFile(scriptPath, content) {
+  const localPath = studioPathToLocalPath(scriptPath);
+  const dir = join(process.cwd(), localPath.split('/').slice(0, -1).join('/'));
+
+  mkdirSync(dir, { recursive: true });
+
+  const fullPath = join(process.cwd(), localPath);
+  writeFileSync(fullPath, content);
+
+  // Track mapping for commitscript
+  scriptLocalFiles.set(scriptPath, fullPath);
+
+  return fullPath;
+}
+
+/**
+ * Write large script content to a temp file (for onboard_action readScript)
  */
 function writeScriptToTempFile(scriptPath, content) {
   const tempDir = join(tmpdir(), 'ropilot-scripts');
   mkdirSync(tempDir, { recursive: true });
 
-  // Create a filename from the script path
   const safeName = scriptPath.replace(/[^a-zA-Z0-9]/g, '_');
   const tempFile = join(tempDir, `${safeName}_${randomUUID().slice(0, 8)}.lua`);
 
@@ -136,6 +172,13 @@ function writeScriptToTempFile(scriptPath, content) {
   return tempFile;
 }
 
+/**
+ * Get local file path for a script path
+ */
+function getLocalFileForScript(scriptPath) {
+  return scriptLocalFiles.get(scriptPath);
+}
+
 /**
  * Handle MCP tools/call request
  */
@@ -151,6 +194,38 @@ async function handleToolsCall(apiKey, request) {
     };
   }
 
+  // Handle sourcecontrol_commitscript - inject file content before forwarding
+  if (request.params?.name === 'ropilot_sourcecontrol_commitscript') {
+    const scriptPath = request.params?.arguments?.path;
+    const localFile = getLocalFileForScript(scriptPath);
+
+    if (!localFile) {
+      return {
+        jsonrpc: '2.0',
+        id: request.id,
+        result: {
+          content: [{ type: 'text', text: `Error: No local file found for "${scriptPath}". Did you call ropilot_sourcecontrol_requestscript first?` }],
+          isError: true
+        }
+      };
+    }
+
+    if (!existsSync(localFile)) {
+      return {
+        jsonrpc: '2.0',
+        id: request.id,
+        result: {
+          content: [{ type: 'text', text: `Error: Local file not found: ${localFile}` }],
+          isError: true
+        }
+      };
+    }
+
+    // Read the file and inject content into the request
+    const content = readFileSync(localFile, 'utf-8');
+    request.params.arguments.content = content;
+  }
+
   // Forward to edge
   const response = await forwardToEdge(apiKey, request);
 
@@ -173,6 +248,48 @@ async function handleToolsCall(apiKey, request) {
     }
   }
 
+  // Handle sourcecontrol_requestscript - write to src/ folder
+  if (request.params?.name === 'ropilot_sourcecontrol_requestscript') {
+    const content = response?.result?.content;
+    if (content && Array.isArray(content) && content[0]?.text) {
+      const scriptContent = content[0].text;
+      const scriptPath = request.params.arguments.path || 'unknown';
+
+      // Check for error responses
+      if (scriptContent.startsWith('Error:')) {
+        return response;
+      }
+
+      // Write to src/ folder
+      const localFile = writeScriptToLocalFile(scriptPath, scriptContent);
+      response.result.content = [{
+        type: 'text',
+        text: `Script saved to:\n${localFile}\n\nEdit this file, then call ropilot_sourcecontrol_commitscript with path: "${scriptPath}"`
+      }];
+    }
+  }
+
+  // Handle sourcecontrol_createscript - auto-write to src/ folder
+  if (request.params?.name === 'ropilot_sourcecontrol_createscript') {
+    const content = response?.result?.content;
+    if (content && Array.isArray(content) && content[0]?.text) {
+      const responseText = content[0].text;
+
+      // Check for CREATED: prefix
+      if (responseText.startsWith('CREATED:')) {
+        const scriptPath = responseText.replace('CREATED:', '').trim();
+        const source = request.params.arguments.source || '';
+
+        // Write to src/ folder
+        const localFile = writeScriptToLocalFile(scriptPath, source);
+        response.result.content = [{
+          type: 'text',
+          text: `Created script: ${scriptPath}\nLocal file: ${localFile}\n\nEdit this file, then call ropilot_sourcecontrol_commitscript with path: "${scriptPath}"`
+        }];
+      }
+    }
+  }
+
   return response;
 }
 
@@ -269,37 +386,6 @@ async function checkPluginUpdate() {
   }
 }
 
-/**
- * Try to start rojo serve in background (non-blocking, silent on failure)
- */
-function tryStartRojoServe() {
-  try {
-    // Check if default.project.json exists (indicates a Rojo project)
-    if (!existsSync(join(process.cwd(), 'default.project.json'))) {
-      return;  // Not a Rojo project
-    }
-
-    // Check if rojo serve is already running
-    const check = spawn('pgrep', ['-f', 'rojo serve'], { stdio: 'pipe' });
-    check.on('close', (code) => {
-      // code 0 = found running process, code 1 = not found
-      if (code === 0) {
-        return;  // Rojo already running
-      }
-
-      // Start rojo serve in background
-      const child = spawn('rojo', ['serve'], {
-        cwd: process.cwd(),
-        detached: true,
-        stdio: 'ignore',
-        shell: true
-      });
-      child.unref();
-    });
-  } catch {
-    // Silently ignore
-  }
-}
 
 /**
  * Run the stdio MCP server
@@ -319,9 +405,6 @@ export async function serve() {
   // Check for plugin updates in background (non-blocking)
   checkPluginUpdate();
 
-  // Start rojo serve in background if available (non-blocking, silent on failure)
-  tryStartRojoServe();
-
   // Read JSON-RPC messages from stdin
   // Supports both Content-Length framing (LSP-style) and newline-delimited JSON
   let buffer = '';

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ropilot",
-  "version": "0.1.42",
+  "version": "0.1.44",
   "description": "AI-powered Roblox development assistant - MCP CLI",
   "author": "whut",
   "license": "MIT",