@aarwitz/tapp 0.17.0-rc.6 → 0.17.0-rc.7

package/bin/tapp.js

@@ -1460,9 +1460,15 @@ switch (command) {
     const runs = roots
       .flatMap((root) => fs.readdirSync(root, { withFileTypes: true }).filter((e) => e.isDirectory()).map((e) => path.join(root, e.name)))
       .sort((a, b) => fs.statSync(b).mtimeMs - fs.statSync(a).mtimeMs);
-    const wanted = rest[0] && rest[0] !== "latest" ? runs.find((r) => path.basename(r) === rest[0]) : runs[0];
+    const hasMarkers = (dir) => fs.existsSync(path.join(dir, "ocqa-markers.txt"));
+    // `latest` (default) resolves to the newest *exploration* capture — the captures directory is
+    // also full of flow-*/scenario-* evidence dirs with no ocqa-markers.txt, and picking the newest
+    // of those made `tapp report` fail even though valid exploration captures existed. An explicitly
+    // named capture is honored as-is so a non-exploration capture still gets a clear "no markers".
+    const explicit = rest[0] && rest[0] !== "latest";
+    const wanted = explicit ? runs.find((r) => path.basename(r) === rest[0]) : runs.find(hasMarkers);
     if (!wanted) {
-      bad("No captures found", rest[0] ? `no capture named "${rest[0]}"` : "run a QA exploration first");
+      bad("No captures found", explicit ? `no capture named "${rest[0]}"` : "run an exploration first (no capture with exploration markers was found)");
       process.exit(1);
     }
     const { writeHtmlReport } = await import(path.join(packageRoot, "mcp-server", "src", "html-report.js"));
@@ -1472,7 +1478,9 @@ switch (command) {
       process.exit(1);
     }
     ok("Evidence report", out);
-    spawnSync("open", [out], { stdio: "ignore" });
+    // Only launch a browser from an interactive terminal — agents, scripts, and tests that invoke
+    // `tapp report` non-interactively get the path without a surprise GUI window.
+    if (process.stdout.isTTY) spawnSync("open", [out], { stdio: "ignore" });
     break;
   }
 

package/mcp-server/src/html-report.js

@@ -14,6 +14,22 @@ const SEV_COLOR = { critical: "#cf222e", high: "#bc4c00", medium: "#9a6700", low
 
 const esc = (s) => String(s ?? "").replace(/[&<>"]/g, (c) => ({ "&": "&amp;", "<": "&lt;", ">": "&gt;", '"': "&quot;" }[c]));
 
+const VALID_PLATFORMS = new Set(["web", "android", "ios"]);
+
+// Recover a capture's platform when no in-memory report is available. Prefer the run's persisted
+// metadata (ui-map.json → app.platforms) — it is authoritative and survives a renamed folder — and
+// fall back to the capture-id prefix (web-*/android-*, ios unprefixed) only for legacy captures that
+// predate the map. Defaults to ios if nothing is resolvable, matching buildQaReport's own default.
+export function capturePlatform(captureDir) {
+  try {
+    const map = JSON.parse(fs.readFileSync(path.join(captureDir, "ui-map.json"), "utf8"));
+    const fromMap = (map?.app?.platforms || []).find((p) => VALID_PLATFORMS.has(p));
+    if (fromMap) return fromMap;
+  } catch { /* no map, unreadable, or no valid platform — fall back to the id prefix */ }
+  const base = path.basename(captureDir);
+  return base.startsWith("web-") ? "web" : base.startsWith("android-") ? "android" : "ios";
+}
+
 // Two capture layouts exist: web runs write state_*.png at the capture root; iOS runs
 // export XCUITest attachments into screenshots/ as UUID files with a manifest carrying
 // the human-readable state_N_<Screen> names.
@@ -50,7 +66,11 @@ function collectShots(captureDir) {
 }
 
 export function writeHtmlReport(captureDir, { report, label = "", recordingWarning = "" } = {}) {
-  const r = report || buildQaReport(path.join(captureDir, "ocqa-markers.txt"));
+  // Rebuild path (e.g. `tapp report`): no report object is passed, so buildQaReport would default to
+  // native — rendering a web/android page with the wrong "Checked / Not checked" scope, overclaiming
+  // native checks it never ran. Recover the run's real platform (metadata first, id prefix as a
+  // legacy fallback). When a report object IS passed (explore/init), it is already correct.
+  const r = report || buildQaReport(path.join(captureDir, "ocqa-markers.txt"), { platform: capturePlatform(captureDir) });
   if (!r) return null;
 
   const shots = collectShots(captureDir);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@aarwitz/tapp",
-  "version": "0.17.0-rc.6",
+  "version": "0.17.0-rc.7",
   "mcpName": "io.github.aarwitz/tapp",
   "description": "Release contracts, autonomous QA, and evidence-backed CI gates for iOS, Android, and web.",
   "license": "MIT",