@jackwener/opencli 1.1.0 → 1.1.1

package/dist/browser/page.d.ts

@@ -21,7 +21,10 @@ export declare class Page implements IPage {
     /** Helper: spread tabId into command params if we have one */
     private _tabOpt;
     private _workspaceOpt;
-    goto(url: string): Promise<void>;
+    goto(url: string, options?: {
+        waitUntil?: 'load' | 'none';
+        settleMs?: number;
+    }): Promise<void>;
     /** Close the automation window in the extension */
     closeWindow(): Promise<void>;
     evaluate(js: string): Promise<any>;

package/dist/browser/page.js

@@ -30,7 +30,7 @@ export class Page {
     _workspaceOpt() {
         return { workspace: this.workspace };
     }
-    async goto(url) {
+    async goto(url, options) {
         const result = await sendCommand('navigate', {
             url,
             ...this._workspaceOpt(),
@@ -40,6 +40,12 @@ export class Page {
         if (result?.tabId) {
             this._tabId = result.tabId;
         }
+        // Post-load settle: the extension already waits for tab.status === 'complete',
+        // but SPA frameworks (React/Vue) need extra time to render after DOM load.
+        if (options?.waitUntil !== 'none') {
+            const settleMs = options?.settleMs ?? 1000;
+            await new Promise(resolve => setTimeout(resolve, settleMs));
+        }
     }
     /** Close the automation window in the extension */
     async closeWindow() {

package/dist/build-manifest.js

@@ -158,18 +158,21 @@ function scanTs(filePath, site) {
     // We statically parse the source to extract metadata for the manifest stub.
     const baseName = path.basename(filePath, path.extname(filePath));
     const relativePath = `${site}/${baseName}.js`;
-    const entry = {
-        site,
-        name: baseName,
-        description: '',
-        strategy: 'cookie',
-        browser: true,
-        args: [],
-        type: 'ts',
-        modulePath: relativePath,
-    };
     try {
         const src = fs.readFileSync(filePath, 'utf-8');
+        // Helper/test modules should not appear as CLI commands in the manifest.
+        if (!/\bcli\s*\(/.test(src))
+            return null;
+        const entry = {
+            site,
+            name: baseName,
+            description: '',
+            strategy: 'cookie',
+            browser: true,
+            args: [],
+            type: 'ts',
+            modulePath: relativePath,
+        };
         // Extract description
         const descMatch = src.match(/description\s*:\s*['"`]([^'"`]*)['"`]/);
         if (descMatch)
@@ -198,11 +201,13 @@ function scanTs(filePath, site) {
         if (argsBlock) {
             entry.args = parseTsArgsBlock(argsBlock);
         }
+        return entry;
     }
-    catch {
-        // If parsing fails, fall back to empty metadata — module will self-register at runtime
+    catch (err) {
+        // If parsing fails, log a warning (matching scanYaml behaviour) and skip the entry.
+        process.stderr.write(`Warning: failed to scan ${filePath}: ${err.message}\n`);
+        return null;
     }
-    return entry;
 }
 export function buildManifest() {
     const manifest = [];
@@ -218,9 +223,11 @@ export function buildManifest() {
                     if (entry)
                         manifest.push(entry);
                 }
-                else if ((file.endsWith('.ts') && !file.endsWith('.d.ts') && file !== 'index.ts') ||
-                    (file.endsWith('.js') && !file.endsWith('.d.js') && file !== 'index.js')) {
-                    manifest.push(scanTs(filePath, site));
+                else if ((file.endsWith('.ts') && !file.endsWith('.d.ts') && !file.endsWith('.test.ts') && file !== 'index.ts') ||
+                    (file.endsWith('.js') && !file.endsWith('.d.js') && !file.endsWith('.test.js') && file !== 'index.js')) {
+                    const entry = scanTs(filePath, site);
+                    if (entry)
+                        manifest.push(entry);
                 }
             }
         }