@reshotdev/screenshot 0.0.1-beta.2 → 0.0.1-beta.6

package/src/index.js

@@ -1,7 +1,6 @@
 #!/usr/bin/env node
 
 // index.js - Reshot CLI entry point
-// Consolidated command structure for ease of use while maintaining power features
 
 require("dotenv").config({
   path: require("path").join(__dirname, "..", ".env"),
@@ -15,7 +14,7 @@ const program = new Command();
 
 program
   .name("reshot")
-  .description("Visual documentation that stays in sync")
+  .description("Visual capture, publishing, and governance for your UI")
   .version(pkg.version);
 
 // ============================================================================
@@ -25,7 +24,7 @@ program
 // Setup: Interactive wizard for initial configuration
 program
   .command("setup")
-  .description("Interactive setup wizard (auth, config, features)")
+  .description("Interactive setup wizard (auth, config)")
   .option("--offline", "Skip authentication (local-only mode)")
   .option("--force", "Force re-initialization even if already set up")
   .action(async (options) => {
@@ -38,13 +37,11 @@ program
     }
   });
 
-// Sync: Upload traces and documentation to platform
+// Sync: Upload Playwright traces to platform
 program
   .command("sync")
-  .description("Upload Playwright traces and documentation to Reshot")
+  .description("Upload Playwright traces to Reshot")
   .option("--trace-dir <path>", "Path to test-results directory")
-  .option("--traces", "Sync traces only")
-  .option("--docs", "Sync documentation only")
   .option("--dry-run", "Preview what would be synced")
   .option("-v, --verbose", "Show detailed output")
   .action(async (options) => {
@@ -52,8 +49,6 @@ program
       const syncCommand = require("./commands/sync");
       await syncCommand({
         traceDir: options.traceDir,
-        traces: options.traces,
-        docs: options.docs,
         dryRun: options.dryRun,
         verbose: options.verbose,
       });
@@ -63,7 +58,7 @@ program
     }
   });
 
-// Status: View project status, sync jobs, and drift summary
+// Status: View project status and drift summary
 program
   .command("status")
   .description("View project status, sync history, and drift summary")
@@ -99,27 +94,6 @@ program
     }
   });
 
-// Validate: Check configuration and documentation bindings
-program
-  .command("validate")
-  .description("Validate configuration and documentation bindings")
-  .option("--strict", "Fail on warnings (zombie links, unbound files)")
-  .option("--fix", "Attempt to auto-fix issues where possible")
-  .option("-v, --verbose", "Show detailed validation output")
-  .action(async (options) => {
-    try {
-      const validateCommand = require("./commands/validate-docs");
-      if (typeof validateCommand.validateDocSync === "function") {
-        await validateCommand.validateDocSync(options);
-      } else {
-        await validateCommand.validateDocs(options);
-      }
-    } catch (error) {
-      console.error(chalk.red("Error:"), error.message);
-      process.exit(1);
-    }
-  });
-
 // ============================================================================
 // VISUAL CAPTURE COMMANDS
 // ============================================================================
@@ -304,7 +278,7 @@ ciCommand
 ciCommand
   .command("run")
   .description("Run capture + publish in one step (CI-optimized)")
-  .option("-c, --config <path>", "Path to docsync.config.json")
+  .option("-c, --config <path>", "Path to reshot.config.json")
   .option("--tag <tag>", "Version tag for publish")
   .option("-m, --message <message>", "Commit message for publish")
   .option("--dry-run", "Preview without uploading")
@@ -323,10 +297,10 @@ ciCommand
 // DRIFT MANAGEMENT COMMANDS
 // ============================================================================
 
-// Drifts: View and manage documentation drifts
+// Drifts: View and manage visual drifts
 program
   .command("drifts [action] [id]")
-  .description("View and manage documentation drifts")
+  .description("View and manage visual drifts")
   .addHelpText(
     "after",
     `
@@ -339,7 +313,6 @@ Actions:
   sync <id>         Mark as manually synced (external_host)
   approve-all       Approve all pending drifts
   reject-all        Reject all pending drifts
-  validate          Validate journey bindings
 `,
   )
   .option("--status <status>", "Filter: PENDING, APPROVED, REJECTED, IGNORED")
@@ -348,7 +321,6 @@ Actions:
   .action(async (action, id, options) => {
     try {
       const driftsCommand = require("./commands/drifts");
-      // Map action/id to the expected subcommand/args format
       const subcommand = action || "list";
       const args = id ? [id] : [];
       await driftsCommand(subcommand, args, options);
@@ -364,7 +336,7 @@ Actions:
 
 // Auth: Standalone authentication (for re-auth scenarios)
 program
-  .command("auth", { hidden: true })
+  .command("auth")
   .description("Authenticate with Reshot Cloud")
   .action(async () => {
     try {
@@ -406,57 +378,4 @@ program
     }
   });
 
-// Ingest: Alias for sync (backward compatibility)
-program
-  .command("ingest", { hidden: true })
-  .description("Upload traces and docs (alias for sync)")
-  .option("--trace-dir <path>", "Path to test-results directory")
-  .option("--dry-run", "Preview what would be synced")
-  .action(async (options) => {
-    console.log(
-      chalk.yellow("Note: 'ingest' is now 'sync'. Running sync...\n"),
-    );
-    try {
-      const syncCommand = require("./commands/sync");
-      await syncCommand({
-        traceDir: options.traceDir,
-        dryRun: options.dryRun,
-      });
-    } catch (error) {
-      console.error(chalk.red("Error:"), error.message);
-      process.exit(1);
-    }
-  });
-
-// UI: Alias for studio (backward compatibility)
-program
-  .command("ui", { hidden: true })
-  .description("Launch Reshot Studio (alias for studio)")
-  .option("--port <port>", "Port for web UI", "4300")
-  .option("--host <host>", "Host for web UI", "127.0.0.1")
-  .option("--no-open", "Do not automatically open browser")
-  .action(async (options) => {
-    try {
-      const uiCommand = require("./commands/ui");
-      await uiCommand(options);
-    } catch (error) {
-      console.error(chalk.red("Error:"), error.message);
-      process.exit(1);
-    }
-  });
-
-// Init-docs: Alias hidden (merged into setup wizard)
-program
-  .command("init-docs", { hidden: true })
-  .description("Initialize documentation sync (use setup instead)")
-  .action(async () => {
-    try {
-      const initDocsCommand = require("./commands/init-docs");
-      await initDocsCommand();
-    } catch (error) {
-      console.error(chalk.red("Error:"), error.message);
-      process.exit(1);
-    }
-  });
-
 program.parse(process.argv);

package/src/lib/api-client.js

@@ -5,7 +5,7 @@ const fs = require("fs");
 
 const baseUrl =
   process.env.RESHOT_API_BASE_URL ||
-  process.env.DOCSYNC_API_BASE_URL ||
+  process.env.RESHOT_API_BASE_URL ||
   "http://localhost:3000/api";
 
 function getApiBaseUrl() {
@@ -648,7 +648,7 @@ async function getBaselines(projectId, apiKey) {
  */
 async function exportVisuals(projectId, options = {}) {
   const { format = "json", status = "approved" } = options;
-  const settings = require("./config").loadSettings();
+  const settings = require("./config").readSettings();
   const apiKey = settings?.apiKey;
 
   if (!apiKey) {
@@ -688,7 +688,7 @@ async function post(endpoint, data, options = {}) {
 }
 
 /**
- * DocSync: Initialize ingestion job with manifest
+ * Initialize ingestion job with manifest
  */
 async function initIngest(apiKey, projectId, manifest) {
   return withRetry(async () => {
@@ -705,7 +705,7 @@ async function initIngest(apiKey, projectId, manifest) {
 }
 
 /**
- * DocSync: Commit ingestion job after uploads complete
+ * Commit ingestion job after uploads complete
  */
 async function commitIngest(apiKey, projectId, uploadResults, git, cli) {
   return withRetry(async () => {
@@ -722,7 +722,7 @@ async function commitIngest(apiKey, projectId, uploadResults, git, cli) {
 }
 
 /**
- * DocSync: Get drift records for a project
+ * Get drift records for a project
  */
 async function getDrifts(apiKey, projectId, options = {}) {
   return withRetry(async () => {
@@ -742,7 +742,7 @@ async function getDrifts(apiKey, projectId, options = {}) {
 }
 
 /**
- * DocSync: Get sync jobs for a project
+ * Get sync jobs for a project
  */
 async function getSyncJobs(apiKey, projectId, options = {}) {
   return withRetry(async () => {
@@ -762,7 +762,7 @@ async function getSyncJobs(apiKey, projectId, options = {}) {
 }
 
 /**
- * DocSync: Perform action on a drift record
+ * Perform action on a drift record
  */
 async function driftAction(apiKey, projectId, driftId, action, options = {}) {
   return withRetry(async () => {
@@ -805,7 +805,7 @@ module.exports = {
   getBaselines,
   // Export support
   exportVisuals,
-  // DocSync
+  // Reshot
   post,
   initIngest,
   commitIngest,

package/src/lib/capture-engine.js

@@ -412,11 +412,11 @@ class CaptureEngine {
       projectId = settings.urlVariables?.PROJECT_ID;
       // 2. Check settings projectId
       if (!projectId) projectId = settings.projectId;
-      // 3. Check docsync.config.json urlVariables
+      // 3. Check reshot.config.json urlVariables
       if (!projectId) {
         try {
-          const docsyncConfig = config.readConfig() || {};
-          projectId = docsyncConfig.urlVariables?.PROJECT_ID;
+          const reshotConfig = config.readConfig() || {};
+          projectId = reshotConfig.urlVariables?.PROJECT_ID;
         } catch (_e) {
           // Config may not exist
         }

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

@@ -1931,12 +1931,17 @@ async function runScenarioWithVideoCapture(scenario, options = {}) {
 
       await fs.writeFile(sentinelPath, buffer);
       sentinelPaths.push({ index: sentinelIndex, label, path: sentinelPath });
+      if (firstSentinelTimestamp === null) {
+        firstSentinelTimestamp = (Date.now() - startTime) / 1000;
+        debug(`First sentinel captured at ${firstSentinelTimestamp.toFixed(2)}s`);
+      }
       sentinelIndex++;
       return sentinelPath;
     }
 
     // Capture initial state BEFORE first navigation (placeholder - actual capture after goto)
     let hasNavigated = false;
+    let firstSentinelTimestamp = null;
 
     // Execute all steps and capture timeline
     for (let stepIndex = 0; stepIndex < script.length; stepIndex++) {
@@ -2292,20 +2297,32 @@ async function runScenarioWithVideoCapture(scenario, options = {}) {
       )
     );
 
-    // Convert to MP4 with ffmpeg, trimming to actual content duration
-    // Add a small buffer (0.5s) after the final action
-    const trimDuration = finalTimestamp + 0.5;
-    console.log(
-      chalk.cyan(
-        `  📹 Converting to MP4 (trimmed to ${trimDuration.toFixed(1)}s)...`
-      )
-    );
-    debug(`Running ffmpeg conversion with trim to ${trimDuration}s...`);
+    // Convert to MP4 with ffmpeg, trimming blank loading frames from start
+    // and excess frames from end
+    const startOffset = Math.max(0, (firstSentinelTimestamp || 0) - 0.3);
+    const endTimestamp = finalTimestamp + 0.5;
+    const contentDuration = endTimestamp - startOffset;
+    if (startOffset > 0) {
+      console.log(
+        chalk.cyan(
+          `  📹 Converting to MP4 (${startOffset.toFixed(1)}s–${endTimestamp.toFixed(1)}s, ${contentDuration.toFixed(1)}s content)...`
+        )
+      );
+    } else {
+      console.log(
+        chalk.cyan(
+          `  📹 Converting to MP4 (trimmed to ${contentDuration.toFixed(1)}s)...`
+        )
+      );
+    }
+    debug(`Running ffmpeg: start=${startOffset}s, duration=${contentDuration}s`);
     await runFFmpegConvert([
+      "-ss",
+      startOffset.toFixed(2),
       "-i",
       recordedVideoPath,
       "-t",
-      trimDuration.toFixed(2),
+      contentDuration.toFixed(2),
       "-c:v",
       "libx264",
       "-preset",
@@ -2330,33 +2347,6 @@ async function runScenarioWithVideoCapture(scenario, options = {}) {
       )
     );
 
-    // Extract poster frame from the video (first frame at 0.5s for non-blank content)
-    const posterPath = finalVideoPath.replace(/\.mp4$/, "-poster.png");
-    try {
-      await runFFmpegConvert([
-        "-i",
-        finalVideoPath,
-        "-ss",
-        "0.5",
-        "-frames:v",
-        "1",
-        "-q:v",
-        "2",
-        "-y",
-        posterPath,
-      ]);
-      if (fs.existsSync(posterPath)) {
-        const posterSize = fs.statSync(posterPath).size;
-        console.log(
-          chalk.green(
-            `  ✔ Poster frame: ${posterPath} (${(posterSize / 1024).toFixed(1)} KB)`
-          )
-        );
-      }
-    } catch (e) {
-      debug(`Poster frame extraction failed: ${e.message}`);
-    }
-
     // Save timeline for reference
     const timelinePath = path.join(
       outputDir || path.join(".reshot/output", scenario.key, "default"),

package/src/lib/config.js

@@ -64,7 +64,7 @@ function createAuthErrorResponse(message) {
     code: "AUTH_REQUIRED",
   };
 }
-const CONFIG_PATH = path.join(process.cwd(), "docsync.config.json");
+const CONFIG_PATH = path.join(process.cwd(), "reshot.config.json");
 const WORKSPACE_PATH = path.join(process.cwd(), SETTINGS_DIR, "workspace.json");
 
 /**
@@ -470,84 +470,22 @@ function readConfig() {
     }
   }
 
-  // Validate optional docs block
-  if (config.docs !== undefined) {
-    if (
-      typeof config.docs !== "object" ||
-      config.docs === null ||
-      Array.isArray(config.docs)
-    ) {
-      throw new Error("docs must be an object");
-    }
-    if (
-      config.docs.root !== undefined &&
-      typeof config.docs.root !== "string"
-    ) {
-      throw new Error("docs.root must be a string");
-    }
-    if (
-      config.docs.include !== undefined &&
-      !Array.isArray(config.docs.include)
-    ) {
-      throw new Error("docs.include must be an array of strings");
-    }
-    if (
-      config.docs.exclude !== undefined &&
-      !Array.isArray(config.docs.exclude)
-    ) {
-      throw new Error("docs.exclude must be an array of strings");
-    }
-  }
-
   return config;
 }
 
 /**
- * Read docsync.config.json with DocSync-specific configuration
- * Returns the full config including the documentation block for ingestion
- * @returns {Object} DocSync configuration
+ * Read reshot.config.json without requiring scenarios array
+ * Used by sync and other commands that don't need scenario validation
+ * @returns {Object} Configuration
  */
-function readDocSyncConfig() {
+function readConfigLenient() {
   if (!fs.existsSync(CONFIG_PATH)) {
     throw new Error(
       `Config file not found at ${CONFIG_PATH}. Run \`reshot init\` to create one.`
     );
   }
 
-  const config = fs.readJSONSync(CONFIG_PATH);
-  
-  // Validate documentation block if present
-  if (config.documentation) {
-    const doc = config.documentation;
-    
-    // Validate required fields
-    if (!doc.strategy) {
-      throw new Error('documentation.strategy is required (git_pr or external_host)');
-    }
-    
-    if (!['git_pr', 'external_host'].includes(doc.strategy)) {
-      throw new Error('documentation.strategy must be "git_pr" or "external_host"');
-    }
-    
-    // Validate optional fields
-    if (doc.assetFormat && !['cdn_link', 'markdown'].includes(doc.assetFormat)) {
-      throw new Error('documentation.assetFormat must be "cdn_link" or "markdown"');
-    }
-    
-    if (doc.include && !Array.isArray(doc.include)) {
-      throw new Error('documentation.include must be an array of glob patterns');
-    }
-    
-    if (doc.exclude && !Array.isArray(doc.exclude)) {
-      throw new Error('documentation.exclude must be an array of glob patterns');
-    }
-    
-    if (doc.mappings && typeof doc.mappings !== 'object') {
-      throw new Error('documentation.mappings must be an object');
-    }
-  }
-  
-  return config;
+  return fs.readJSONSync(CONFIG_PATH);
 }
 
 /**
@@ -851,8 +789,6 @@ async function initializeProject(projectId, apiKey, options = {}) {
         contextCount: 1,
         features: {
           visuals: true,
-          docs: false,
-          changelog: true,
         },
       },
     };
@@ -1230,8 +1166,8 @@ module.exports = {
   // Mode & validation
   getModeInfo,
   validateConfig,
-  // DocSync configuration
-  readDocSyncConfig,
+  // Lenient config read (no scenario validation)
+  readConfigLenient,
   // Paths
   SETTINGS_PATH,
   SETTINGS_DIR,

package/src/lib/record-config.js

@@ -600,7 +600,7 @@ async function finalizeScenarioAndWriteConfig(
 
   console.log(
     chalk.green(
-      "\n✔ docsync.config.json has been updated. Please review and commit the changes to your repository.\n"
+      "\n✔ reshot.config.json has been updated. Please review and commit the changes to your repository.\n"
     )
   );
   console.log(chalk.gray(`Scenario: ${result.scenarioName}`));

package/src/lib/standalone-mode.js

@@ -103,7 +103,7 @@ const DEFAULT_STANDALONE_SETTINGS = {
 };
 
 const SETTINGS_DIR = ".reshot";
-const CONFIG_PATH = "docsync.config.json";
+const CONFIG_PATH = "reshot.config.json";
 const SETTINGS_PATH = path.join(SETTINGS_DIR, "settings.json");
 
 /**

package/src/lib/storage-providers.js

@@ -114,7 +114,7 @@ ${chalk.cyan('AWS S3 Setup:')}
    ${chalk.gray('export AWS_SECRET_ACCESS_KEY="your-secret-access-key"')}
    ${chalk.gray('export AWS_REGION="us-east-1"  # optional, defaults to us-east-1')}
 
-3. ${chalk.yellow('Update docsync.config.json:')}
+3. ${chalk.yellow('Update reshot.config.json:')}
    ${chalk.gray(JSON.stringify({
      storage: {
        type: 's3',
@@ -143,7 +143,7 @@ ${chalk.cyan('Cloudflare R2 Setup:')}
    ${chalk.gray('export R2_ACCESS_KEY_ID="your-r2-access-key"')}
    ${chalk.gray('export R2_SECRET_ACCESS_KEY="your-r2-secret-key"')}
 
-3. ${chalk.yellow('Update docsync.config.json:')}
+3. ${chalk.yellow('Update reshot.config.json:')}
    ${chalk.gray(JSON.stringify({
      storage: {
        type: 'r2',
@@ -165,7 +165,7 @@ ${chalk.cyan('Local Storage Setup:')}
 
 For local testing or self-hosted scenarios:
 
-1. ${chalk.yellow('Update docsync.config.json:')}
+1. ${chalk.yellow('Update reshot.config.json:')}
    ${chalk.gray(JSON.stringify({
      storage: {
        type: 'local',
@@ -536,7 +536,7 @@ function createStorageProvider(config) {
 
 /**
  * Determine storage mode from config
- * @param {object} docSyncConfig - The docsync.config.json content
+ * @param {object} docSyncConfig - The reshot.config.json content
  * @returns {'platform'|'byos'}
  */
 function getStorageMode(docSyncConfig) {

package/src/lib/ui-api.js

@@ -48,7 +48,7 @@ function getPlatformUrl(settings) {
     return settings.platformUrl;
   }
   const envUrl =
-    process.env.RESHOT_API_BASE_URL || process.env.DOCSYNC_API_BASE_URL;
+    process.env.RESHOT_API_BASE_URL || process.env.RESHOT_API_BASE_URL;
   if (envUrl) {
     // Remove /api suffix if present to get platform URL
     return envUrl.replace(/\/api\/?$/, "");
@@ -431,7 +431,7 @@ function attachApiRoutes(app, context) {
 
   /**
    * PUT /api/privacy
-   * Update privacy configuration in docsync.config.json
+   * Update privacy configuration in reshot.config.json
    */
   app.put("/api/privacy", async (req, res, next) => {
     try {
@@ -472,7 +472,7 @@ function attachApiRoutes(app, context) {
 
   /**
    * PUT /api/style
-   * Update style configuration in docsync.config.json
+   * Update style configuration in reshot.config.json
    */
   app.put("/api/style", async (req, res, next) => {
     try {