cdp-skill 1.0.18 → 1.0.19

package/EXAMPLES.md

@@ -68,7 +68,7 @@ Close when done:
   },
   "screenshot": "/tmp/cdp-skill/t1.after.png",
   "fullSnapshot": "/tmp/cdp-skill/t1.after.yaml",
-  "viewportSnapshot": "- heading \"Title\" [level=1]\n- button \"Submit\" [ref=f0s1e1]\n...",
+  "viewportSnapshot": "/tmp/cdp-skill/t1.viewport.yaml",
   "steps": [{"action": "goto", "status": "ok"}]
 }
 ```
@@ -130,7 +130,7 @@ Close when done:
   },
   "screenshot": "/tmp/cdp-skill/t1.after.png",
   "fullSnapshot": "/tmp/cdp-skill/t1.after.yaml",
-  "viewportSnapshot": "- heading \"New Page\" [level=1]\n- button \"Submit\" [ref=f0s1e1]\n...",
+  "viewportSnapshot": "/tmp/cdp-skill/t1.viewport.yaml",
   "steps": [{"action": "click", "status": "ok"}]
 }
 ```

package/SKILL.md

@@ -33,12 +33,12 @@ Tab IDs (t1, t2, ...) persist across CLI invocations. Chrome auto-launches if no
 **Output fields:**
 - `status`: "ok" or "error"
 - `tab`: short tab ID (e.g. "t1")
-- `siteProfile`: full markdown content of existing profile (after goto/newTab to known site)
+- `siteProfile`: path to site profile file (after goto/newTab to known site)
 - `actionRequired`: `{action, domain, message}` — **MUST be handled immediately** before continuing (see Site Profiles)
 - `context`: `{url, title, scroll: {y, percent}, viewport: {width, height}, activeElement?, modal?}`
 - `screenshot`: path to after-screenshot (auto-captured on every visual action)
 - `fullSnapshot`: path to full-page accessibility snapshot file
-- `viewportSnapshot`: inline viewport-only snapshot YAML
+- `viewportSnapshot`: path to viewport-only accessibility snapshot file
 - `changes`: `{summary, added[], removed[], changed[]}` — viewport diff on same-page interactions
 - `navigated`: true when URL pathname changed
 - `console`: `{errors, warnings, messages[]}` — captured errors/warnings

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "cdp-skill",
-  "version": "1.0.18",
+  "version": "1.0.19",
   "description": "Browser automation skill using Chrome DevTools Protocol for Claude Code and AI agents",
   "type": "module",
   "main": "scripts/index.js",

package/scripts/cdp-skill.js

@@ -785,15 +785,15 @@ async function main() {
       truncated: result.truncated
     };
 
-    // Remove null/undefined fields for compactness
+    // Remove null/undefined/false-y fields for compactness
     if (!output.siteProfile) delete output.siteProfile;
     if (!output.actionRequired) delete output.actionRequired;
-    if (output.navigated === undefined) delete output.navigated;
+    if (!output.navigated) delete output.navigated;
     if (!output.fullSnapshot) delete output.fullSnapshot;
     if (!output.context) delete output.context;
     if (!output.changes) delete output.changes;
     if (!output.viewportSnapshot) delete output.viewportSnapshot;
-    if (output.truncated === undefined) delete output.truncated;
+    if (!output.truncated) delete output.truncated;
     if (!output.screenshot) delete output.screenshot;
     if (!output.console) delete output.console;
     if (output.errors.length === 0) delete output.errors;

package/scripts/runner/execute-dynamic.js

@@ -408,13 +408,14 @@ function sanitizeDomain(domain) {
  * Load a site profile for the given domain.
  *
  * @param {string} domain - hostname (e.g. "github.com")
- * @returns {Promise<string|null>} profile markdown or null
+ * @returns {Promise<string|null>} profile file path or null
  */
 export async function loadSiteProfile(domain) {
   const clean = sanitizeDomain(domain);
   const profilePath = path.join(SITES_DIR, `${clean}.md`);
   try {
-    return await fs.readFile(profilePath, 'utf8');
+    await fs.access(profilePath);
+    return profilePath;
   } catch {
     return null;
   }
@@ -452,9 +453,9 @@ export async function executeReadSiteProfile(params) {
     throw new Error('readSiteProfile requires a domain string');
   }
 
-  const content = await loadSiteProfile(domain);
-  if (content) {
-    return { found: true, domain, content };
+  const profilePath = await loadSiteProfile(domain);
+  if (profilePath) {
+    return { found: true, domain, path: profilePath };
   }
   return { found: false, domain };
 }

package/scripts/runner/step-executors.js

@@ -668,7 +668,11 @@ export async function runSteps(deps, steps, options = {}) {
       result.navigated = navigated;
       result.fullSnapshot = fullSnapshotPath;
       result.context = afterContext;
-      result.viewportSnapshot = afterViewport.yaml;
+
+      // Write viewport snapshot to file to keep response concise
+      const viewportPath = await resolveTempPath(`${options.tabAlias || 'command'}.viewport.yaml`, '.yaml');
+      await fs.writeFile(viewportPath, afterViewport.yaml || '', 'utf8');
+      result.viewportSnapshot = viewportPath;
       result.truncated = afterViewport.truncated || false;
 
       if (!navigated && beforeViewport?.yaml) {

package/scripts/tests/ExecuteDynamic.test.js

@@ -680,7 +680,7 @@ describe('step-executors goto profile integration', () => {
     assert.strictEqual(result.action, 'goto');
     assert.strictEqual(result.status, 'ok');
     assert.ok(result.siteProfile);
-    assert.ok(result.siteProfile.includes('Pre-existing profile'));
+    assert.ok(result.siteProfile.endsWith('.md'), 'siteProfile should be a file path');
   });
 
   it('should return profileAvailable false on goto for unknown domain', async () => {