@reshotdev/screenshot 0.0.1-beta.7 → 0.0.1-beta.8

package/README.md

@@ -76,7 +76,6 @@ Create `reshot.config.json` in your project root:
 | `reshot record [title]` | Interactive recording via Chrome DevTools | `--browser`, `--url`, `--port` |
 | `reshot sync` | Upload traces/docs to Reshot platform | `--trace-dir`, `--dry-run` |
 | `reshot studio` | Launch web management UI | `--port`, `--no-open` |
-| `reshot validate` | Check config and bindings | `--strict`, `--fix` |
 | `reshot status` | View project status and sync history | `--jobs`, `--drifts`, `--json` |
 | `reshot publish` | Upload assets with versioning | `--tag`, `--message`, `--dry-run` |
 | `reshot pull` | Generate asset map for builds | `--format json\|ts\|csv`, `--output`, `--status` |
@@ -225,12 +224,19 @@ The recorded scenario is appended to `reshot.config.json` automatically.
 
 ## Authentication
 
-### Storage State (from Playwright)
+### Storage State
+
+The CLI stores browser session state at `~/.reshot/session-state.json` (global).
+This is automatically captured when you run `reshot record`.
+
+To manually generate it:
 
 ```bash
-npx playwright codegen http://localhost:3000 --save-storage=.reshot/auth-state.json
+npx playwright codegen http://localhost:3000 --save-storage=$HOME/.reshot/session-state.json
 ```
 
+Or reference a project-local path in your config:
+
 ```json
 {
   "storageStatePath": ".reshot/auth-state.json"

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@reshotdev/screenshot",
-  "version": "0.0.1-beta.7",
+  "version": "0.0.1-beta.8",
   "description": "CI/CD screenshot and video capture CLI",
   "author": "Reshot <hello@reshot.dev>",
   "license": "MIT",

package/src/commands/publish.js

@@ -809,7 +809,8 @@ async function publishWithTransactionalFlow(
   }
 
   // Build all commits for batch request
-  const MAX_BATCH_SIZE = 200;
+  // Vercel serverless functions have ~60s timeout; keep batches small enough to complete
+  const MAX_BATCH_SIZE = 25;
   const commits = [];
 
   for (const { group, scenarioConfig, assets } of groupMap.values()) {
@@ -839,14 +840,21 @@ async function publishWithTransactionalFlow(
     chalk.gray(`  Committing ${commits.length} scenario(s) to platform...`),
   );
 
+  const totalBatches = Math.ceil(commits.length / MAX_BATCH_SIZE);
   for (let i = 0; i < commits.length; i += MAX_BATCH_SIZE) {
     const chunk = commits.slice(i, i + MAX_BATCH_SIZE);
+    const batchNum = Math.floor(i / MAX_BATCH_SIZE) + 1;
+    if (totalBatches > 1) {
+      console.log(chalk.gray(`  Batch ${batchNum}/${totalBatches}...`));
+    }
 
     try {
-      const batchResult = await apiClient.publishBatch(apiKey, {
+      const rawBatchResult = await apiClient.publishBatch(apiKey, {
         commits: chunk,
         autoApprove: autoApprove || false,
       });
+      // Unwrap API envelope: response may be { data: { results, ... } } or { results, ... }
+      const batchResult = rawBatchResult.data || rawBatchResult;
 
       for (const r of batchResult.results || []) {
         if (r.status === "ok") {

package/src/commands/pull.js

@@ -142,6 +142,19 @@ async function pullCommand(options = {}) {
     process.exit(1);
   }
 
+  if (settings.projectId && projectId !== settings.projectId) {
+    console.warn(
+      chalk.yellow(
+        `  ⚠ Project ID mismatch: config uses ${projectId}, but authenticated project is ${settings.projectId}.`
+      )
+    );
+    console.warn(
+      chalk.yellow(
+        `    This will cause auth failures. Run 'reshot setup --force' to re-link.\n`
+      )
+    );
+  }
+
   try {
     // Fetch the asset map from the API
     console.log(chalk.gray(`  Fetching assets for project: ${projectId}`));
@@ -163,7 +176,18 @@ async function pullCommand(options = {}) {
       0
     );
 
-    console.log(chalk.green(`  ✓ Fetched ${assetCount} visuals\n`));
+    if (assetCount === 0) {
+      console.log(chalk.yellow(`  ⚠ Fetched 0 visuals.\n`));
+      console.log(chalk.gray(`  Possible reasons:`));
+      if (status === "approved") {
+        console.log(chalk.gray(`    • No visuals approved yet. Try: reshot pull --status all`));
+      }
+      console.log(chalk.gray(`    • No visuals published. Run: reshot publish`));
+      console.log(chalk.gray(`    • Wrong project. Config uses: ${projectId}`));
+      console.log(chalk.gray(`    • Check: ${settings.platformUrl || "https://reshot.dev"}\n`));
+    } else {
+      console.log(chalk.green(`  ✓ Fetched ${assetCount} visuals\n`));
+    }
 
     // Determine output path
     let outputPath = output;

package/src/commands/setup.js

@@ -2,6 +2,7 @@
 const chalk = require("chalk");
 const oraModule = require("ora");
 const ora = oraModule.default || oraModule;
+const { execSync } = require("child_process");
 
 const {
   writeSettings,
@@ -100,10 +101,14 @@ async function setupCommand(options = {}) {
   if (hasConfig && !force) {
     console.log(chalk.green("✔ Configuration found:"), chalk.cyan("reshot.config.json"));
   } else if (hasConfig && force) {
-    console.log(chalk.yellow("⚠ Overwriting existing reshot.config.json..."));
-    const newConfig = createDefaultConfig(existingSettings);
-    writeConfig(newConfig);
-    console.log(chalk.green("✔ Configuration updated"));
+    // Patch projectId in existing config instead of wiping scenarios
+    const fs = require("fs");
+    const path = require("path");
+    const configPath = path.join(process.cwd(), "reshot.config.json");
+    const existingConfig = JSON.parse(fs.readFileSync(configPath, "utf-8"));
+    existingConfig.projectId = existingSettings.projectId;
+    fs.writeFileSync(configPath, JSON.stringify(existingConfig, null, 2) + "\n");
+    console.log(chalk.green("✔ Configuration updated (projectId synced)"));
   } else {
     // No config - create it
     console.log(chalk.gray("Creating reshot.config.json..."));
@@ -112,6 +117,29 @@ async function setupCommand(options = {}) {
     console.log(chalk.green("✔ Configuration created:"), chalk.cyan("reshot.config.json"));
   }
 
+  // Step 3.5: Ensure @reshotdev/screenshot is in devDependencies
+  const fs = require("fs");
+  const path = require("path");
+  const pkgJsonPath = path.join(process.cwd(), "package.json");
+  if (fs.existsSync(pkgJsonPath)) {
+    const pkgJson = JSON.parse(fs.readFileSync(pkgJsonPath, "utf-8"));
+    const hasDep =
+      pkgJson.devDependencies?.["@reshotdev/screenshot"] ||
+      pkgJson.dependencies?.["@reshotdev/screenshot"];
+    if (!hasDep) {
+      console.log(chalk.gray("  Adding @reshotdev/screenshot to devDependencies..."));
+      const usePnpm = fs.existsSync(path.join(process.cwd(), "pnpm-lock.yaml"));
+      const useYarn = fs.existsSync(path.join(process.cwd(), "yarn.lock"));
+      const cmd = usePnpm ? "pnpm add -D" : useYarn ? "yarn add -D" : "npm install -D";
+      try {
+        execSync(`${cmd} @reshotdev/screenshot`, { stdio: "inherit" });
+        console.log(chalk.green("  ✔ Added @reshotdev/screenshot to devDependencies"));
+      } catch {
+        console.log(chalk.yellow("  ⚠ Could not auto-install. Run manually: " + cmd + " @reshotdev/screenshot"));
+      }
+    }
+  }
+
   // Step 4: Success summary
   console.log(chalk.green("\n━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━"));
   console.log(chalk.green.bold("✔ Project initialized successfully!"));

package/src/index.js

@@ -160,12 +160,28 @@ program
   .option("--browser", "Launch Chrome with remote debugging before recording")
   .option("-p, --port <port>", "Chrome debugging port (default: 9222)")
   .option("--url <url>", "URL to open when launching browser")
+  .option("--refresh-session", "Only refresh the auth session (no recording prompts)")
   .option("--debug", "Enable verbose debug logging")
   .action(async (title, options) => {
     if (options.debug) {
       process.env.RESHOT_DEBUG = "1";
     }
     try {
+      // If --refresh-session, just sync the session and exit
+      if (options.refreshSession) {
+        const { autoSyncSessionFromCDP, getDefaultSessionPath } = require("./lib/record-cdp");
+        const sessionPath = getDefaultSessionPath();
+        console.log(chalk.gray("  Syncing session from active browser..."));
+        const synced = await autoSyncSessionFromCDP(sessionPath);
+        if (synced) {
+          console.log(chalk.green("  ✔ Session refreshed at " + sessionPath));
+        } else {
+          console.log(chalk.yellow("  ⚠ No active CDP browser found. Launch Chrome with remote debugging first:"));
+          console.log(chalk.gray("    reshot record --browser --refresh-session"));
+        }
+        return;
+      }
+
       // If --browser flag, launch Chrome first
       if (options.browser) {
         const chromeCommand = require("./commands/chrome");

package/src/lib/capture-script-runner.js

@@ -303,7 +303,18 @@ async function preflightAuthCheck(baseUrl, options = {}) {
       };
     }
 
-    console.log(chalk.green("  ✔ Auth pre-flight check passed"));
+    // Save refreshed session back so scenarios use fresh cookies
+    if (storageStatePath && engine.context) {
+      try {
+        const refreshedState = await engine.context.storageState();
+        fs.writeFileSync(storageStatePath, JSON.stringify(refreshedState, null, 2));
+        console.log(chalk.green("  ✔ Auth pre-flight check passed (session refreshed)"));
+      } catch (_saveErr) {
+        console.log(chalk.green("  ✔ Auth pre-flight check passed"));
+      }
+    } else {
+      console.log(chalk.green("  ✔ Auth pre-flight check passed"));
+    }
     return { ok: true };
   } catch (e) {
     // If the error is an auth redirect thrown by the engine, handle gracefully
@@ -1310,6 +1321,12 @@ async function runScenarioWithStepByStepCapture(scenario, options = {}) {
                 `    Hint: If data isn't loading, run 'reshot record' to refresh your session`
               )
             );
+            failedSteps.push({
+              stepIndex: stepIndex + 1,
+              action: "waitFor",
+              target: params.target,
+              error: errMsg,
+            });
           }
         } else if (waitResult.status === "timeout") {
           if (!isOptional) {
@@ -1323,9 +1340,14 @@ async function runScenarioWithStepByStepCapture(scenario, options = {}) {
                 `    Hint: If content isn't loading, run 'reshot record' to refresh your session`
               )
             );
+            failedSteps.push({
+              stepIndex: stepIndex + 1,
+              action: "waitFor",
+              target: params.target,
+              error: `Element not found within ${waitTimeout}ms`,
+            });
           }
         }
-        // Continue with next steps - the scenario may still capture partial state
         continue;
       }