mcp-inflight 0.2.4 → 0.3.1

package/README.md

@@ -83,9 +83,9 @@ The server automatically patches configuration files for CodeSandbox compatibili
 
 Projects over 3MB are automatically chunked into smaller uploads to handle Vercel's serverless function body size limits.
 
-## Slash Commands (Optional)
+## Slash Commands (Recommended)
 
-For a better experience, install the included slash commands. These provide `/share` and `/inflight` commands in Claude Code.
+For the best experience, install the included slash commands. The `/share` command enables **automated AI feedback guide generation** - Claude analyzes your git changes and generates product-focused review questions that appear on your InFlight version.
 
 ### Installation
 
@@ -109,7 +109,7 @@ cp packages/mcp-inflight/commands/*.md ~/.claude/commands/
 
 | Command | Description |
 |---------|-------------|
-| `/share` | Share a project to InFlight (shortcut for the share tool) |
+| `/share` | Share a project to InFlight with AI-generated feedback guide |
 | `/inflight` | Menu for managing prototypes: list, sync, delete, login, logout |
 
 ## Troubleshooting

package/commands/inflight.md

@@ -0,0 +1,70 @@
+# InFlight
+
+Manage InFlight prototypes and authentication.
+
+## Arguments
+
+$ARGUMENTS - Optional action: `login`, `logout`, `list`, `sync`, `delete`. If not provided, shows menu.
+
+## Instructions
+
+### If no argument provided, show menu
+
+Ask the user what they'd like to do using the AskUserQuestion tool with these options:
+- **Check login status** - See who's logged in
+- **List prototypes** - Show all shared prototypes
+- **Sync prototype** - Update files in an existing prototype
+- **Delete prototype** - Remove a prototype
+- **Logout** - Log out of InFlight
+
+Then execute the selected action below.
+
+### Action: login (or "Check login status")
+
+Call the `login` MCP tool. It will either:
+- Show current user info if already logged in
+- Open browser for authentication if not logged in
+
+Display the result to the user.
+
+### Action: logout
+
+Call the `logout` MCP tool to clear stored credentials.
+Confirm to the user they've been logged out.
+
+### Action: list (or "List prototypes")
+
+Call the `prototype_list` MCP tool.
+
+Display results in a table:
+- Prototype ID
+- Project Name
+- Type
+- Status
+- InFlight URL
+- Created
+
+If no prototypes found, inform the user.
+
+### Action: sync (or "Sync prototype")
+
+1. If a prototype ID is provided in arguments, use that
+2. Otherwise, call `prototype_list` and ask user which prototype to sync
+3. Ask for the project path (or use current directory)
+4. Check for .env files - only ask about including them if they exist
+5. Call `prototype_sync` with:
+   - `prototypeId`: The selected ID
+   - `path`: The project path
+   - `includeEnvFiles`: Based on user response (default: false)
+
+### Action: delete (or "Delete prototype")
+
+1. If a prototype ID is provided in arguments, use that
+2. Otherwise, call `prototype_list` and ask user which prototype to delete
+3. Confirm: "Delete prototype [ID]? This will stop it and make it unavailable."
+4. Call `prototype_delete` with the prototype ID
+5. Confirm deletion to user
+
+## Related Commands
+
+- `/share` - Share a project to InFlight

package/commands/share.md

@@ -0,0 +1,75 @@
+# Share Project
+
+Share a local project to InFlight for feedback and collaboration.
+
+## Arguments
+
+$ARGUMENTS - Optional path to the project directory. If not provided, uses current working directory.
+
+## Instructions
+
+### 1. Determine project path
+
+Use the provided path argument or current working directory.
+
+### 2. Check for .env files (STOP only if found)
+
+Look for `.env`, `.env.local`, `.env.development`, `.env.production` in the project directory.
+
+**Only if .env files exist, ASK:** "Found .env files. Include them in deployment? (May contain sensitive API keys)"
+- Yes: set `includeEnvFiles: true`
+- No: set `includeEnvFiles: false` (default)
+
+If no .env files found, skip this step entirely.
+
+### 3. Analyze git changes (if on feature branch)
+
+Check if there are git changes to generate review questions:
+
+1. **Check for commits on branch:** Run `git log main..HEAD --oneline 2>/dev/null || git log master..HEAD --oneline 2>/dev/null`
+2. **If commits exist**, analyze the changes:
+   - Run `git diff main...HEAD 2>/dev/null || git diff master...HEAD 2>/dev/null` to see what changed
+   - **Generate a product-focused summary:** Write 2-3 sentences about user-visible changes (not internal code details)
+   - **List key changes:** Identify main user-facing changes (e.g., "Added dark mode toggle", "Redesigned checkout flow")
+   - **List affected areas:** Identify UI/feature areas (e.g., "Settings page", "Navigation", "Forms")
+
+**Skip this step if:**
+- On main/master branch
+- No commits found on the branch
+- Git commands fail (not a git repo)
+
+### 4. Call the `share` MCP tool
+
+Call the `share` tool with:
+- `path`: the project path
+- `includeEnvFiles`: based on user response (default: false)
+- `diffSummary` (only if git changes were analyzed in step 3):
+  - `summary`: your product-focused summary
+  - `keyChanges`: array of key user-visible changes
+  - `affectedAreas`: array of affected UI areas
+
+The MCP tool handles everything:
+- Project analysis and type detection
+- Package manager detection
+- VM tier selection
+- Creating prototype and InFlight version
+- Generating review questions from your summary (automatic)
+- Opening InFlight in browser
+
+### 5. Display the result
+
+Show the InFlight URL to the user:
+
+```
+Shared!
+
+InFlight: https://www.inflight.co/v/[version-id]
+```
+
+## Error Handling
+
+If the tool returns an error, display it to the user.
+
+## Related Commands
+
+- `/inflight` - Manage prototypes, login/logout

package/dist/index.js

@@ -986,7 +986,8 @@ async function deployProject(files, options, log) {
       vmTier: options.vmTier || "Micro",
       projectName: options.projectName,
       workspaceId: options.workspaceId || auth.defaultWorkspaceId,
-      gitInfo: options.gitInfo
+      gitInfo: options.gitInfo,
+      diffSummary: options.diffSummary
     })
   });
   if (!response.ok && !response.headers.get("content-type")?.includes("text/event-stream")) {
@@ -1227,7 +1228,8 @@ async function finalizeSandbox(sandboxId, options, log) {
       port: options.port,
       projectName: options.projectName,
       workspaceId: options.workspaceId,
-      gitInfo: options.gitInfo
+      gitInfo: options.gitInfo,
+      diffSummary: options.diffSummary
     })
   });
   if (!response.ok && !response.headers.get("content-type")?.includes("text/event-stream")) {
@@ -1284,7 +1286,8 @@ async function deployProjectChunked(fileChunks, options, log) {
       port: options.port,
       projectName: options.projectName,
       workspaceId,
-      gitInfo: options.gitInfo
+      gitInfo: options.gitInfo,
+      diffSummary: options.diffSummary
     },
     log
   );
@@ -1926,6 +1929,9 @@ async function deployPrototype(args, log) {
     const dirtyMarker = gitInfo.isDirty ? " (modified)" : "";
     await log(`Git: ${gitInfo.branch}@${gitInfo.commitShort}${dirtyMarker}`);
   }
+  if (args.diffSummary) {
+    await log(`Diff summary provided: ${args.diffSummary.keyChanges.length} key changes`);
+  }
   if (!projectInfo.compatibility.canDeploy) {
     throw new Error(
       `Project cannot be deployed: ${projectInfo.compatibility.issues.join(", ")}`
@@ -1997,124 +2003,74 @@ async function deployPrototype(args, log) {
   }
   const totalSize = calculateTotalSize(files);
   await log(`Found ${fileCount} files (${(totalSize / 1024 / 1024).toFixed(1)} MB)`);
-  let existingSandbox;
-  try {
-    const sandboxes = await listSandboxes();
-    const projectName = path5.basename(projectPath);
-    existingSandbox = sandboxes.find(
-      (s) => s.projectName === projectName && s.status === "running"
-    );
-  } catch {
-  }
   let result;
-  if (existingSandbox) {
-    await log(`Found existing sandbox: ${existingSandbox.sandboxId}`);
-    await log("Syncing files...");
-    try {
-      const syncResult = await syncProject(
-        existingSandbox.sandboxId,
-        files,
-        {
-          restartServer: true,
-          startCommand: projectInfo.startCommand,
-          port: projectInfo.port,
-          gitInfo: gitInfo || void 0
-        },
-        log
-      );
-      const inflightUrl = existingSandbox.inflightUrl || "";
-      const inflightVersionId = existingSandbox.inflightVersionId || "";
-      result = {
-        url: syncResult.sandboxUrl,
-        sandboxId: existingSandbox.sandboxId,
-        projectType: projectInfo.type,
-        synced: true,
-        inflightUrl,
-        inflightVersionId,
-        startCommand: projectInfo.startCommand,
-        port: projectInfo.port,
-        packageManager: projectInfo.detectedPackageManager,
-        isMonorepo: projectInfo.isMonorepo,
-        vmTier: projectInfo.sizing.recommendedTier,
-        embedCheck
-      };
-      const elapsed = ((Date.now() - startTime) / 1e3).toFixed(1);
-      await log(`Sync complete in ${elapsed}s`);
-    } catch (syncError) {
-      const errorMsg = syncError instanceof Error ? syncError.message : String(syncError);
-      await log(`Sync failed, creating new sandbox: ${errorMsg}`, "warning");
-      existingSandbox = void 0;
-    }
-  }
-  if (!existingSandbox) {
-    const useChunkedUpload = needsChunkedUpload(files);
-    if (useChunkedUpload) {
-      const totalSize2 = calculateTotalSize(files);
-      await log(`Large project detected (${(totalSize2 / 1024 / 1024).toFixed(1)} MB), using chunked upload...`);
-      const fileChunks = chunkFiles(files);
-      await log(`Split into ${fileChunks.length} chunks`);
-      const deployResult = await deployProjectChunked(
-        fileChunks,
-        {
-          projectType: projectInfo.type,
-          installCommand: projectInfo.installCommand || void 0,
-          startCommand: projectInfo.startCommand,
-          port: projectInfo.port,
-          vmTier: projectInfo.sizing.recommendedTier,
-          projectName: path5.basename(projectPath),
-          workspaceId: args.workspaceId,
-          gitInfo: gitInfo || void 0
-        },
-        log
-      );
-      result = {
-        url: deployResult.sandboxUrl,
-        sandboxId: deployResult.sandboxId,
+  const useChunkedUpload = needsChunkedUpload(files);
+  if (useChunkedUpload) {
+    const totalSize2 = calculateTotalSize(files);
+    await log(`Large project detected (${(totalSize2 / 1024 / 1024).toFixed(1)} MB), using chunked upload...`);
+    const fileChunks = chunkFiles(files);
+    await log(`Split into ${fileChunks.length} chunks`);
+    const deployResult = await deployProjectChunked(
+      fileChunks,
+      {
         projectType: projectInfo.type,
-        synced: false,
-        inflightUrl: deployResult.inflightUrl,
-        inflightVersionId: deployResult.versionId,
+        installCommand: projectInfo.installCommand || void 0,
         startCommand: projectInfo.startCommand,
         port: projectInfo.port,
-        packageManager: projectInfo.detectedPackageManager,
-        isMonorepo: projectInfo.isMonorepo,
         vmTier: projectInfo.sizing.recommendedTier,
-        embedCheck
-      };
-    } else {
-      await log("Deploying via InFlight API...");
-      const deployResult = await deployProject(
-        files,
-        {
-          projectType: projectInfo.type,
-          installCommand: projectInfo.installCommand || void 0,
-          startCommand: projectInfo.startCommand,
-          port: projectInfo.port,
-          vmTier: projectInfo.sizing.recommendedTier,
-          projectName: path5.basename(projectPath),
-          workspaceId: args.workspaceId,
-          gitInfo: gitInfo || void 0
-        },
-        log
-      );
-      result = {
-        url: deployResult.sandboxUrl,
-        sandboxId: deployResult.sandboxId,
+        projectName: path5.basename(projectPath),
+        workspaceId: args.workspaceId,
+        gitInfo: gitInfo || void 0,
+        diffSummary: args.diffSummary || void 0
+      },
+      log
+    );
+    result = {
+      url: deployResult.sandboxUrl,
+      sandboxId: deployResult.sandboxId,
+      projectType: projectInfo.type,
+      inflightUrl: deployResult.inflightUrl,
+      inflightVersionId: deployResult.versionId,
+      startCommand: projectInfo.startCommand,
+      port: projectInfo.port,
+      packageManager: projectInfo.detectedPackageManager,
+      isMonorepo: projectInfo.isMonorepo,
+      vmTier: projectInfo.sizing.recommendedTier,
+      embedCheck
+    };
+  } else {
+    await log("Deploying via InFlight API...");
+    const deployResult = await deployProject(
+      files,
+      {
         projectType: projectInfo.type,
-        synced: false,
-        inflightUrl: deployResult.inflightUrl,
-        inflightVersionId: deployResult.versionId,
+        installCommand: projectInfo.installCommand || void 0,
         startCommand: projectInfo.startCommand,
         port: projectInfo.port,
-        packageManager: projectInfo.detectedPackageManager,
-        isMonorepo: projectInfo.isMonorepo,
         vmTier: projectInfo.sizing.recommendedTier,
-        embedCheck
-      };
-    }
-    const elapsed = ((Date.now() - startTime) / 1e3).toFixed(1);
-    await log(`Deploy complete in ${elapsed}s`);
+        projectName: path5.basename(projectPath),
+        workspaceId: args.workspaceId,
+        gitInfo: gitInfo || void 0,
+        diffSummary: args.diffSummary || void 0
+      },
+      log
+    );
+    result = {
+      url: deployResult.sandboxUrl,
+      sandboxId: deployResult.sandboxId,
+      projectType: projectInfo.type,
+      inflightUrl: deployResult.inflightUrl,
+      inflightVersionId: deployResult.versionId,
+      startCommand: projectInfo.startCommand,
+      port: projectInfo.port,
+      packageManager: projectInfo.detectedPackageManager,
+      isMonorepo: projectInfo.isMonorepo,
+      vmTier: projectInfo.sizing.recommendedTier,
+      embedCheck
+    };
   }
+  const elapsed = ((Date.now() - startTime) / 1e3).toFixed(1);
+  await log(`Deploy complete in ${elapsed}s`);
   if (hadUncommittedChanges && result.inflightVersionId) {
     const newCommit = autoCommitForShare(projectPath, result.inflightVersionId);
     if (newCommit) {
@@ -2188,6 +2144,27 @@ function createServer2() {
               workspaceId: {
                 type: "string",
                 description: "InFlight workspace ID to create the version in. If not provided, uses the last-used workspace or the user's first workspace."
+              },
+              diffSummary: {
+                type: "object",
+                description: "Claude-generated summary of git changes for review question generation. Provide this when sharing a feature branch with commits.",
+                properties: {
+                  summary: {
+                    type: "string",
+                    description: "A 2-3 sentence product-focused summary of the changes. Focus on user-visible changes, not internal code details."
+                  },
+                  keyChanges: {
+                    type: "array",
+                    items: { type: "string" },
+                    description: "List of key user-visible changes (e.g., 'Added dark mode toggle', 'Redesigned checkout flow')"
+                  },
+                  affectedAreas: {
+                    type: "array",
+                    items: { type: "string" },
+                    description: "UI/feature areas affected by the changes (e.g., 'Settings page', 'Navigation', 'Forms')"
+                  }
+                },
+                required: ["summary", "keyChanges"]
               }
             },
             required: ["path"]
@@ -2262,7 +2239,7 @@ function createServer2() {
       if (name === "share") {
         const deployArgs = args;
         const result = await deployPrototype(deployArgs, log);
-        const message = result.synced ? `Files synced. InFlight: ${result.inflightUrl}` : `Project shared successfully. InFlight: ${result.inflightUrl}`;
+        const message = `Project shared successfully. InFlight: ${result.inflightUrl}`;
         return {
           content: [
             {
@@ -2281,7 +2258,6 @@ function createServer2() {
                   },
                   // Embed check results - shows if there are iframe embedding issues
                   embedCheck: result.embedCheck,
-                  synced: result.synced ?? false,
                   inflightUrl: result.inflightUrl,
                   inflightVersionId: result.inflightVersionId,
                   message

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "mcp-inflight",
-  "version": "0.2.4",
+  "version": "0.3.1",
   "description": "MCP server for sharing prototypes via InFlight",
   "author": "InFlight <hello@inflight.co>",
   "homepage": "https://github.com/inflight/mcp-inflight",
@@ -13,11 +13,6 @@
   "bin": {
     "mcp-inflight": "dist/index.js"
   },
-  "scripts": {
-    "build": "tsup",
-    "dev": "tsup --watch",
-    "typecheck": "tsc --noEmit"
-  },
   "dependencies": {
     "@modelcontextprotocol/sdk": "^1.0.0"
   },
@@ -27,7 +22,8 @@
     "typescript": "^5.0.0"
   },
   "files": [
-    "dist"
+    "dist",
+    "commands"
   ],
   "keywords": [
     "mcp",
@@ -36,5 +32,10 @@
     "share",
     "deploy"
   ],
-  "license": "MIT"
-}
+  "license": "MIT",
+  "scripts": {
+    "build": "tsup",
+    "dev": "tsup --watch",
+    "typecheck": "tsc --noEmit"
+  }
+}