@pro-vi/designer 0.3.3 → 0.3.5

package/dist/browser.js

@@ -1,7 +1,7 @@
 import { spawn } from 'node:child_process';
 const BIN = process.env.DESIGNER_AGENT_BROWSER_BIN || 'agent-browser';
 const DEFAULT_SESSION = process.env.DESIGNER_SESSION_NAME || 'designer';
-const CDP = process.env.DESIGNER_CDP || '';
+const CDP = process.env.DESIGNER_CDP ?? '9222';
 export function createBrowser({ session = DEFAULT_SESSION, headed = true, timeoutMs = 30_000, cdp = CDP } = {}) {
     const baseEnv = {
         ...process.env,
@@ -53,6 +53,17 @@ export function createBrowser({ session = DEFAULT_SESSION, headed = true, timeou
         close: () => run(['close']).catch(() => null),
         url: () => run(['get', 'url']),
         title: () => run(['get', 'title']),
+        tabs: async () => {
+            const out = await run(['tab', 'list', '--json']);
+            const env = JSON.parse(out);
+            if (env.success === false) {
+                throw new Error(`agent-browser tab list failed: ${JSON.stringify(env.error)}`);
+            }
+            return env.data?.tabs ?? [];
+        },
+        activateTab: async (index) => {
+            await run(['tab', String(index)]);
+        },
         snapshot: ({ interactive = true, scope } = {}) => {
             const args = ['snapshot', '--json'];
             if (interactive)

package/dist/cli.js

@@ -66,7 +66,8 @@ async function main() {
             const res = await c.iterate(prompt, {
                 file: flags.file,
                 timeoutMs: flags.timeoutMs ? Number(flags.timeoutMs) : undefined,
-                stabilityMs: flags.stabilityMs ? Number(flags.stabilityMs) : undefined
+                stabilityMs: flags.stabilityMs ? Number(flags.stabilityMs) : undefined,
+                decisive: Boolean(flags.decisive)
             });
             if (res.url)
                 console.log(`\nTaste here: ${res.url}\n`);
@@ -343,6 +344,7 @@ Flags:
   --file <f.html>         switch to this file before prompting
   --timeoutMs <n>         default 20 minutes
   --stabilityMs <n>       default 4 seconds
+  --decisive              tell Claude not to stop on clarifying questions; pick defaults and proceed
 
 Output: prints 'Taste here: <url>' then JSON metadata (done, newFiles, htmlPath, screenshotPath,
 htmlHash, chatReply). HTML is written to disk (read htmlPath if needed); it's not inline.
@@ -350,6 +352,10 @@ htmlHash, chatReply). HTML is written to disk (read htmlPath if needed); it's no
 Auto-appended to every prompt: 'Keep all generated files at the project root; no subfolders.'
 Override by explicitly contradicting it in your prompt text.
 
+With --decisive, also append: 'If you would otherwise stop to ask clarifying questions, do not.
+Choose the most defensible answer for each axis yourself and proceed.' Use when you want Claude
+to commit to a direction instead of blocking on the clarifying-questions affordance.
+
 Examples:
   designer prompt "add a Remember-me checkbox" --key feat-x
   designer prompt --prompt-file ./brief.md --key feat-x
@@ -517,9 +523,6 @@ async function checkAgentBrowser() {
 }
 async function checkCdp() {
     const port = process.env.DESIGNER_CDP || '9222';
-    if (!process.env.DESIGNER_CDP) {
-        return { name: `CDP at port ${port}`, status: 'warn', detail: 'DESIGNER_CDP not set; defaulting to 9222. export DESIGNER_CDP=9222 to silence.' };
-    }
     try {
         const res = await fetch(`http://127.0.0.1:${port}/json/version`);
         if (!res.ok)

package/dist/designer-controller.js

@@ -10,6 +10,7 @@ import { REPO_ROOT } from "./repo-root.js";
 import { ensureCdpUp } from "./cdp-ensure.js";
 const DESIGN_HOME = 'https://claude.ai/design';
 const FLAT_LAYOUT_SUFFIX = '\n\nFile layout: keep all generated files at the project root. No subfolders.';
+const DECISIVE_SUFFIX = '\n\nIf you would otherwise stop to ask clarifying questions, do not. Choose the most defensible answer for each axis yourself and proceed. Note your assumption in a one-line `<!-- assumed: ... -->` comment at the top of the relevant file so I can override on the next turn.';
 function loadSelectors() {
     const base = JSON.parse(fs.readFileSync(path.join(REPO_ROOT, 'selectors.json'), 'utf8'));
     const overridePath = path.join(os.homedir(), '.designer', 'selectors.override.json');
@@ -59,15 +60,26 @@ export class DesignerController {
         const url = await this.currentUrl();
         const inSession = /\/design\/p\/[a-f0-9-]+/i.test(url);
         const availableFiles = inSession ? await this.listFiles().catch(() => []) : [];
+        const awaitingClarification = inSession ? await this.detectAwaitingClarification() : false;
         return {
             key: this.key,
             stored,
             currentUrl: url,
             inSession,
             onHome: /\/design\/?$/.test(url) || url.endsWith('/design'),
-            availableFiles
+            availableFiles,
+            awaitingClarification
         };
     }
+    async detectAwaitingClarification() {
+        const turns = await this.getChatTurns().catch(() => []);
+        if (turns.length === 0)
+            return false;
+        const last = turns[turns.length - 1];
+        if (!last || last.role !== 'assistant')
+            return false;
+        return /Claude has some questions/i.test(last.text);
+    }
     async session({ action = 'status', name, fidelity = 'wireframe' } = {}) {
         if (action === 'status')
             return this.getStatus();
@@ -90,19 +102,55 @@ export class DesignerController {
         }
         throw new Error(`Unknown action: ${action}`);
     }
+    async selectMatchingTab() {
+        const tabs = await this.browser.tabs().catch(() => []);
+        if (tabs.length === 0)
+            return { matched: false, candidates: 0 };
+        const stored = getSession(this.key);
+        const targetRoot = stored?.designUrl?.split('?')[0];
+        const candidates = tabs.filter((t) => {
+            if (t.type !== 'page' || !t.url)
+                return false;
+            if (targetRoot)
+                return t.url.startsWith(targetRoot);
+            return /^https:\/\/claude\.ai\/design(\/|$|\?)/.test(t.url);
+        });
+        if (candidates.length === 0)
+            return { matched: false, candidates: 0 };
+        candidates.sort((a, b) => (Number(b.active) - Number(a.active)) || (a.index - b.index));
+        for (const cand of candidates) {
+            await this.browser.activateTab(cand.index).catch(() => null);
+            const composerOk = await this.browser.isVisible(this.selectors.composer.promptTextarea).catch(() => false);
+            const homeOk = this.selectors.login.signedInIndicator
+                ? await this.browser.isVisible(this.selectors.login.signedInIndicator).catch(() => false)
+                : false;
+            if (composerOk || homeOk) {
+                upsertSession(this.key, { lastUrl: await this.currentUrl() });
+                return { matched: true, candidates: candidates.length };
+            }
+        }
+        return { matched: false, candidates: candidates.length };
+    }
     async ensureReady() {
         await ensureCdpUp();
-        const u = await this.currentUrl();
-        if (!/claude\.ai\/design/.test(u)) {
-            await this.browser.open(DESIGN_HOME);
-            await this.browser.waitLoad('networkidle').catch(() => null);
+        const picked = await this.selectMatchingTab();
+        if (picked.matched) {
+            return { ok: true, url: await this.currentUrl(), inSession: await this.isInSession() };
+        }
+        if (picked.candidates === 0) {
+            const u = await this.currentUrl();
+            if (!/claude\.ai\/design/.test(u)) {
+                await this.browser.open(DESIGN_HOME);
+                await this.browser.waitLoad('networkidle').catch(() => null);
+            }
         }
         const homeOk = this.selectors.login.signedInIndicator
             ? await this.browser.isVisible(this.selectors.login.signedInIndicator).catch(() => false)
             : false;
         const sessionOk = await this.browser.isVisible(this.selectors.composer.promptTextarea).catch(() => false);
         if (!homeOk && !sessionOk) {
-            throw new Error('Not signed in to claude.ai/design, or on an unrecognized page. Sign in in the CDP-attached Chrome.');
+            const suffix = picked.candidates > 0 ? ` (checked ${picked.candidates} tab(s))` : '';
+            throw new Error(`Not signed in to claude.ai/design, or on an unrecognized page${suffix}. Sign in in the CDP-attached Chrome.`);
         }
         upsertSession(this.key, { lastUrl: await this.currentUrl() });
         return { ok: true, url: await this.currentUrl(), inSession: await this.isInSession() };
@@ -168,12 +216,13 @@ export class DesignerController {
         return true;
       })()`);
     }
-    async sendPrompt(prompt) {
+    async sendPrompt(prompt, { decisive = false } = {}) {
         const before = await this.fetchServedHtml();
         this._preSendHtml = before.html;
-        const effective = prompt + FLAT_LAYOUT_SUFFIX;
+        const effective = prompt + FLAT_LAYOUT_SUFFIX + (decisive ? DECISIVE_SUFFIX : '');
         await this._submitPrompt(effective);
-        appendHistory(this.key, { kind: 'prompt', prompt, suffixApplied: 'flat_layout' });
+        const suffixApplied = decisive ? 'flat_layout+decisive' : 'flat_layout';
+        appendHistory(this.key, { kind: 'prompt', prompt, suffixApplied });
         return { ok: true };
     }
     async waitForGenerationDone({ timeoutMs = 20 * 60_000, stabilityMs = 4000, pollMs = 1500 } = {}) {
@@ -238,13 +287,13 @@ export class DesignerController {
             throw new Error(`No active session for key=${this.key}. Call createSession first.`);
         await this.resumeSession();
     }
-    async iterate(prompt, { file, timeoutMs, stabilityMs } = {}) {
+    async iterate(prompt, { file, timeoutMs, stabilityMs, decisive } = {}) {
         await this._ensureInSession();
         if (file)
             await this.openFile(file);
         const preFiles = await this.listFiles().catch(() => []);
         const preChatCount = (await this.getChatTurns()).length;
-        await this.sendPrompt(prompt);
+        await this.sendPrompt(prompt, { decisive });
         const done = await this.waitForGenerationDone({ timeoutMs, stabilityMs });
         const postFiles = await this.listFiles().catch(() => []);
         const postTurns = await this.getChatTurns();
@@ -334,10 +383,12 @@ export class DesignerController {
         // scraping misses them; text-node walking is resilient.
         const seen = new Set();
         const files = [];
+        let designFilesLabelVisible = false;
         const walker = document.createTreeWalker(document.body, NodeFilter.SHOW_TEXT);
         let node;
         while ((node = walker.nextNode())) {
           const t = (node.textContent || '').trim();
+          if (t === 'Design Files') designFilesLabelVisible = true;
           if (!/^[A-Za-z0-9 _.()\\-]+\\.(html|js|css|jsx|tsx|ts|md|json|svg)$/i.test(t)) continue;
           if (t.length > 80 || seen.has(t)) continue;
           seen.add(t);
@@ -355,12 +406,15 @@ export class DesignerController {
             folderSet.add(lines[0]);
           }
         }
-        return { files, folders: Array.from(folderSet) };
-      })()`).catch(() => ({ files: [], folders: [] }));
+        return { files, folders: Array.from(folderSet), designFilesLabelVisible };
+      })()`).catch(() => ({ files: [], folders: [], designFilesLabelVisible: false }));
+        const files = Array.isArray(result.files) ? result.files : [];
+        const folders = Array.isArray(result.folders) ? result.folders : [];
+        const emptyButLabelVisible = files.length === 0 && result.designFilesLabelVisible === true;
         return {
-            files: Array.isArray(result.files) ? result.files : [],
-            folders: Array.isArray(result.folders) ? result.folders : [],
-            authoritative: (result.folders?.length ?? 0) === 0
+            files,
+            folders,
+            authoritative: !emptyButLabelVisible && folders.length === 0
         };
     }
     async openFile(filename) {

package/dist/mcp-server.js

@@ -37,9 +37,13 @@ server.registerTool('designer_prompt', {
         prompt: z.string(),
         file: z.string().optional().describe('Switch to this file before sending (targets the prompt at it).'),
         timeoutMs: z.number().optional().describe('Default 20m. Hi-fi generations can take 15+ min; bump this for complex multi-variant prompts.'),
-        stabilityMs: z.number().optional().describe('Default 4s.')
+        stabilityMs: z.number().optional().describe('Default 4s.'),
+        decisive: z
+            .boolean()
+            .optional()
+            .describe("Append a 'do not stop to ask clarifying questions' instruction. Use when you want Claude to commit to a direction (pick defensible defaults itself, document the assumption inline) instead of blocking on the ephemeral clarifying-questions affordance — which disappears on refresh and has no stable DOM contract to scrape.")
     }
-}, async ({ key, prompt, file, timeoutMs, stabilityMs }) => textResult(await getController(key).iterate(prompt, { file, timeoutMs, stabilityMs })));
+}, async ({ key, prompt, file, timeoutMs, stabilityMs, decisive }) => textResult(await getController(key).iterate(prompt, { file, timeoutMs, stabilityMs, decisive })));
 server.registerTool('designer_ask', {
     description: "Q&A with the design assistant — text-only, doesn't change any file. Use for 'why did you choose X?', 'compare A vs B', 'suggest 3 alternatives before I commit'. Returns the assistant's reply.",
     inputSchema: {

package/dist/scripts/probe.js

@@ -3,9 +3,6 @@ import { createBrowser } from "../browser.js";
 const cmd = process.argv[2];
 const arg = process.argv[3];
 const browser = createBrowser({ headed: true });
-if (!process.env.DESIGNER_CDP) {
-    console.error('[probe] DESIGNER_CDP not set — using agent-browser-managed session (may be blocked by Cloudflare/SSO). Prefer: export DESIGNER_CDP=9222 and relaunch Chrome with --remote-debugging-port=9222.');
-}
 async function main() {
     switch (cmd) {
         case 'login':
@@ -43,12 +40,26 @@ async function main() {
         case 'close':
             await browser.close();
             break;
+        case 'tabs': {
+            const tabs = await browser.tabs();
+            const composer = '[data-testid="chat-composer-input"]';
+            const signedIn = '[data-testid="create-project-button"]';
+            for (const t of tabs) {
+                await browser.activateTab(t.index).catch(() => null);
+                const composerOk = await browser.isVisible(composer).catch(() => false);
+                const signedInOk = await browser.isVisible(signedIn).catch(() => false);
+                const flag = composerOk ? 'composer' : signedInOk ? 'home' : 'unrecognized';
+                console.log(`[${t.index}] active=${t.active ? 'Y' : 'N'} ${flag.padEnd(12)} ${t.url}`);
+            }
+            break;
+        }
         default:
             console.log(`Usage:
   probe.ts login                  open headed window for manual login
   probe.ts open <url>             navigate
   probe.ts url                    print current url
   probe.ts title                  print current title
+  probe.ts tabs                   list CDP tabs with readiness verdict
   probe.ts snapshot [scope]       interactive a11y tree (text)
   probe.ts snapshot-json [scope]  interactive a11y tree (JSON)
   probe.ts screenshot [path]      full-page screenshot

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pro-vi/designer",
-  "version": "0.3.3",
+  "version": "0.3.5",
   "type": "module",
   "description": "MCP + CLI for autonomous iteration of claude.ai/design — drives the design surface via agent-browser, downloads handoff bundles, and exposes a tasting harness for full-viewport variant comparison.",
   "license": "MIT",