@reshotdev/screenshot 0.0.1-beta.19 → 0.0.1-beta.20

package/package.json

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

package/src/commands/publish.js

@@ -56,20 +56,44 @@ function loadDiffManifests(outputBaseDir) {
       const latestVersion = versions.sort().reverse()[0];
       if (!latestVersion) continue;
 
-      const manifestPath = path.join(
-        scenarioDir,
-        latestVersion,
-        "diff-manifest.json",
-      );
+      const versionDir = path.join(scenarioDir, latestVersion);
+
+      // `run` writes diff-manifest.json into the ACTUAL output dir, which
+      // includes the variant subdir (e.g. <version>/default/diff-manifest.json),
+      // not the version dir itself. Read the version-level manifest if present
+      // AND every variant subdir's manifest, merging their `assets` so the diff
+      // the CLI computed actually reaches publish (otherwise the platform stores
+      // diffPercentage: null and Activity shows "0 changed" — audit run-13 H1).
+      const candidatePaths = [path.join(versionDir, "diff-manifest.json")];
+      try {
+        for (const entry of fs.readdirSync(versionDir)) {
+          const entryPath = path.join(versionDir, entry);
+          if (fs.statSync(entryPath).isDirectory()) {
+            candidatePaths.push(path.join(entryPath, "diff-manifest.json"));
+          }
+        }
+      } catch (e) {
+        // version dir unreadable — fall back to the version-level path only
+      }
 
-      if (fs.existsSync(manifestPath)) {
+      const merged = { assets: {} };
+      let foundAny = false;
+      for (const manifestPath of candidatePaths) {
+        if (!fs.existsSync(manifestPath)) continue;
         try {
           const manifest = fs.readJSONSync(manifestPath);
-          manifests.set(`${scenarioKey}/${latestVersion}`, manifest);
+          foundAny = true;
+          Object.assign(merged, manifest, {
+            assets: { ...merged.assets, ...(manifest.assets || {}) },
+          });
         } catch (e) {
           // Skip malformed manifests
         }
       }
+
+      if (foundAny) {
+        manifests.set(`${scenarioKey}/${latestVersion}`, merged);
+      }
     }
   } catch (e) {
     // Return empty if directory structure is unexpected