@notis_ai/cli 0.2.0-beta.179.1 → 0.2.0-beta.181.1

package/dist/agent-hooks/notis-agent-hook.mjs

@@ -11820,7 +11820,7 @@ var SAFE_VIEWPORT_HEIGHT_SCRIPT = `(() => {
 })()`;
 
 // src/runtime/agent-browser.js
-var PREPARE_CAPTURE_SCRIPT = "(() => { const s = document.getElementById('harness-status'); if (s) s.style.display = 'none'; const r = document.getElementById('root'); if (r) { r.style.paddingTop = '0'; r.style.minHeight = '0'; } document.body.style.minHeight = '0'; document.documentElement.style.minHeight = '0'; return true; })()";
+var PREPARE_CAPTURE_SCRIPT = "(() => { const r = document.getElementById('root'); if (r) r.style.minHeight = '0'; document.body.style.minHeight = '0'; document.documentElement.style.minHeight = '0'; return true; })()";
 var FONTS_STATUS_SCRIPT = "(() => (document.fonts ? document.fonts.status : 'loaded'))()";
 var CONTENT_HEIGHT_SCRIPT = "(() => { const r = document.getElementById('root'); if (!r) return Math.ceil(document.documentElement.scrollHeight); return Math.ceil(Math.max(r.scrollHeight, r.getBoundingClientRect().height)); })()";
 var MIN_CSS_VIEWPORT_HEIGHT = 720;
@@ -12300,7 +12300,7 @@ var DESIGN_ASSERTIONS_SCRIPT = `(() => {
   };
   const all = Array.from(root.querySelectorAll('*'));
   for (const el of all) {
-    if (el.id === 'harness-status' || !visible(el) || isControl(el)) continue;
+    if (!visible(el) || isControl(el)) continue;
     if (isBox(el)) {
       let p = el.parentElement; let depth = 0;
       while (p && p !== root && depth < 6) { if (!isControl(p) && isBox(p)) { push('nested_border_box', el); break; } p = p.parentElement; depth += 1; }
@@ -12317,7 +12317,7 @@ var DESIGN_ASSERTIONS_SCRIPT = `(() => {
     const text = (node.textContent || '').trim();
     if (!text) continue;
     const el = node.parentElement;
-    if (!el || el.closest('#harness-status') || !visible(el)) continue;
+    if (!el || !visible(el)) continue;
     const size = parseFloat(getComputedStyle(el).fontSize);
     if (size > 0 && size < 12) push('text_below_12px', el, { font_size: size });
     if (/^Loading(\\.{3}|\\u2026)?(\\s|$)/.test(text)) push('loading_placeholder_after_mount', el);
@@ -14201,15 +14201,20 @@ async function appsDeployHandler(ctx) {
     } catch {
       warnings.push("The app was updated, but the temporary release directory needs local cleanup.");
     }
+    const skillReviews = (Array.isArray(result.payload.skill_reviews) ? result.payload.skill_reviews : []).filter((review) => review && typeof review.review_url === "string");
     return ctx.output.emitSuccess({
       command: ctx.spec.command_path.join(" "),
       data: {
         app_id: appId,
         version: deployedVersion,
-        idempotency_key: idempotencyKey
+        idempotency_key: idempotencyKey,
+        ...skillReviews.length ? { skill_reviews: skillReviews } : {}
       },
       warnings,
-      humanSummary: `Deployed to app ${appId} (version ${deployedVersion})`,
+      humanSummary: [
+        `Deployed to app ${appId} (version ${deployedVersion})`,
+        ...skillReviews.map((review) => `Review ${review.skill} changes: ${review.review_url}`)
+      ].join("\n"),
       meta: { mutating: true, idempotency_key: idempotencyKey }
     });
   } finally {

package/dist/sdk/src/runtime.ts

@@ -368,6 +368,8 @@ export interface RuntimeResource {
   kind: 'app' | 'report';
   /** Host-provided adoption eligibility, never report-authored authority. */
   analytics_eligible?: boolean;
+  /** Host-derived report family for adoption analytics, e.g. skill review reports. */
+  report_kind?: 'skill_review' | 'custom';
   id: string;
   revision: number;
   name?: string;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@notis_ai/cli",
-  "version": "0.2.0-beta.179.1",
+  "version": "0.2.0-beta.181.1",
   "description": "Agent-first Notis CLI for apps and generic tool execution",
   "type": "module",
   "bin": {

package/src/command-specs/apps.js

@@ -1257,15 +1257,22 @@ async function appsDeployHandler(ctx) {
     try { release.close(); }
     catch { warnings.push('The app was updated, but the temporary release directory needs local cleanup.'); }
 
+    // One review report per skill this release changed; share the link when the user should review it.
+    const skillReviews = (Array.isArray(result.payload.skill_reviews) ? result.payload.skill_reviews : [])
+      .filter((review) => review && typeof review.review_url === 'string');
     return ctx.output.emitSuccess({
       command: ctx.spec.command_path.join(' '),
       data: {
         app_id: appId,
         version: deployedVersion,
         idempotency_key: idempotencyKey,
+        ...(skillReviews.length ? { skill_reviews: skillReviews } : {}),
       },
       warnings,
-      humanSummary: `Deployed to app ${appId} (version ${deployedVersion})`,
+      humanSummary: [
+        `Deployed to app ${appId} (version ${deployedVersion})`,
+        ...skillReviews.map((review) => `Review ${review.skill} changes: ${review.review_url}`),
+      ].join('\n'),
       meta: { mutating: true, idempotency_key: idempotencyKey },
     });
   } finally {

package/src/runtime/agent-browser.js

@@ -3,18 +3,13 @@ import { mkdirSync, readFileSync, writeFileSync } from 'node:fs';
 import { dirname } from 'node:path';
 import { contentCapturePlan, MEASURE_SCREENSHOT_CONTENT_SCRIPT, SAFE_VIEWPORT_HEIGHT_SCRIPT } from './screenshot-framing.js';
 
-export const HIDE_HARNESS_STATUS_SCRIPT =
-  "(() => { const el = document.getElementById('harness-status'); if (el) el.style.opacity = '0'; return true; })()";
-
 /**
- * Prepare the mounted harness page for a listing capture: remove the debug
- * banner and the layout the harness adds around the app (top padding for the
- * banner, 100vh min-height) so content height can be measured and framed
- * without artificial whitespace.
+ * Prepare the mounted harness page for a listing capture: remove the 100vh
+ * min-height the harness adds around the app so content height can be
+ * measured and framed without artificial whitespace.
  */
 const PREPARE_CAPTURE_SCRIPT =
-  "(() => { const s = document.getElementById('harness-status'); if (s) s.style.display = 'none';" +
-  " const r = document.getElementById('root'); if (r) { r.style.paddingTop = '0'; r.style.minHeight = '0'; }" +
+  "(() => { const r = document.getElementById('root'); if (r) r.style.minHeight = '0';" +
   " document.body.style.minHeight = '0'; document.documentElement.style.minHeight = '0'; return true; })()";
 
 const FONTS_STATUS_SCRIPT =
@@ -583,7 +578,7 @@ export const DESIGN_ASSERTIONS_SCRIPT = `(() => {
   };
   const all = Array.from(root.querySelectorAll('*'));
   for (const el of all) {
-    if (el.id === 'harness-status' || !visible(el) || isControl(el)) continue;
+    if (!visible(el) || isControl(el)) continue;
     if (isBox(el)) {
       let p = el.parentElement; let depth = 0;
       while (p && p !== root && depth < 6) { if (!isControl(p) && isBox(p)) { push('nested_border_box', el); break; } p = p.parentElement; depth += 1; }
@@ -600,7 +595,7 @@ export const DESIGN_ASSERTIONS_SCRIPT = `(() => {
     const text = (node.textContent || '').trim();
     if (!text) continue;
     const el = node.parentElement;
-    if (!el || el.closest('#harness-status') || !visible(el)) continue;
+    if (!el || !visible(el)) continue;
     const size = parseFloat(getComputedStyle(el).fontSize);
     if (size > 0 && size < 12) push('text_below_12px', el, { font_size: size });
     if (/^Loading(\\.{3}|\\u2026)?(\\s|$)/.test(text)) push('loading_placeholder_after_mount', el);

package/template/.harness/index.html.tmpl

@@ -93,12 +93,10 @@
       color: hsl(var(--foreground));
       -webkit-font-smoothing: antialiased;
     }
-    #harness-status { position: fixed; top: 0; left: 0; right: 0; padding: 4px 8px; font-size: 11px; background: #111; color: #0f0; font-family: ui-monospace, SFMono-Regular, Menlo, monospace; z-index: 9999; }
-    #root { padding-top: 24px; min-height: 100vh; }
+    #root { min-height: 100vh; }
   </style>
 </head>
 <body>
-  <div id="harness-status">harness: booting</div>
   <div id="root" data-notis-app-root></div>
   <script type="module">
     const routeExport = {{ROUTE_EXPORT}};
@@ -134,10 +132,9 @@
       return Object.prototype.hasOwnProperty.call(tools, name) ? tools[name] : undefined;
     }
 
-    const status = document.getElementById('harness-status');
-    const setStatus = (msg, color = '#0f0') => {
-      status.textContent = `harness: ${msg}`;
-      status.style.color = color;
+    // Boot progress goes to the console so the rendered page stays identical to the app.
+    const setStatus = (msg, failed = false) => {
+      (failed ? console.error : console.info)(`[notis harness] ${msg}`);
     };
 
     window.__harness = {
@@ -416,13 +413,13 @@
           window.__harness.mounted = true;
           const errors = window.__harness.errors.length;
           const calls = window.__harness.runtimeCalls.length;
-          setStatus(`mounted (errors=${errors}, runtimeCalls=${calls})`, errors ? '#f55' : '#0f0');
+          setStatus(`mounted (errors=${errors}, runtimeCalls=${calls})`, errors > 0);
         } catch (error) {
           window.__harness.errors.push({
             type: 'scenario',
             message: error && error.message ? error.message : String(error),
           });
-          setStatus(`scenario failed: ${error && error.message ? error.message : String(error)}`, '#f55');
+          setStatus(`scenario failed: ${error && error.message ? error.message : String(error)}`, true);
         }
       }));
     } catch (error) {
@@ -431,7 +428,7 @@
         message: error && error.message ? error.message : String(error),
         stack: error && error.stack,
       });
-      setStatus(`boot failed: ${error && error.message ? error.message : String(error)}`, '#f55');
+      setStatus(`boot failed: ${error && error.message ? error.message : String(error)}`, true);
     }
   </script>
 </body>