tabctl 0.3.1 → 0.5.0

package/README.md

@@ -25,7 +25,7 @@ If you haven't run `npm link`, you can always use `node ./cli/tabctl.js` instead
 
 ### 2. Set up your browser
 
-Run the interactive setup — it syncs the extension, tells you where to load it, and prompts for the extension ID:
+Run setup — it syncs the extension, tells you where to load it, and auto-derives the extension ID:
 
 <!-- test: "setup explicit --extension-id overrides auto-derived ID" -->
 ```bash
@@ -36,11 +36,11 @@ This will:
 1. Copy the extension to a stable location (`~/.local/state/tabctl/extension/`)
 2. Print the path (and copy it to your clipboard)
 3. Ask you to load it as an unpacked extension in `chrome://extensions`
-4. Prompt you to paste the extension ID
+4. Auto-derive the extension ID from the installed path
 
 > **Edge?** Use `--browser edge` and load from `edge://extensions` instead.
 
-If you already know your extension ID, skip the interactive flow:
+If you need to override the auto-derived ID (e.g. for a custom extension path):
 
 <!-- test: "setup writes native host manifest for chrome" -->
 ```bash
@@ -59,12 +59,14 @@ tabctl list       # see your open tabs
 
 ## Commands
 
-<!-- test: "list sends list action", "analyze passes tab ids and github options", "inspect passes signal options", "close without confirm fails", "report format md returns markdown content", "undo sends undo action with txid" -->
+<!-- test: "list sends list action", "analyze passes tab ids and progress option", "inspect passes signal options", "close without confirm fails", "report format md returns markdown content", "undo sends undo action with txid" -->
 | Command | Description |
 |---------|-------------|
 | `tabctl list` | List open tabs and groups |
+| `tabctl open --url <url> --group <name>` | Open tabs into a group (reuses existing, skips duplicates) |
 | `tabctl analyze` | Find stale or duplicate tabs |
 | `tabctl inspect --tab <id>` | Extract page metadata or CSS selectors |
+| `tabctl group-gather` | Merge duplicate groups with the same name |
 | `tabctl close --tab <id>` | Close tabs with full undo support |
 | `tabctl report` | Generate reports in JSON, Markdown, or CSV |
 | `tabctl undo` | Revert the last action |
@@ -167,10 +169,10 @@ tabctl supports multiple browser profiles. Each profile connects to a different
 <!-- test: "setup writes native host manifest", "setup writes native host manifest for chrome", "setup --name creates custom-named profile", "profile-list with multiple profiles shows all", "profile-switch success updates default", "--profile flag overrides active profile" -->
 ```bash
 # Setup for Edge
-tabctl setup --browser edge --extension-id <edge-id>
+tabctl setup --browser edge
 
 # Setup for Chrome (with custom name)
-tabctl setup --browser chrome --extension-id <chrome-id> --name chrome-work
+tabctl setup --browser chrome --name chrome-work
 
 # List profiles
 tabctl profile-list
@@ -253,7 +255,7 @@ Notes:
 - `list` and `group-list` paginate by default (limit 100); use `--limit`, `--offset`, or `--no-page`.
 - Use `--group-id -1` or `--ungrouped` to target ungrouped tabs.
 - `--selector` implies `--signal selector`.
-- Unknown inspect signals are rejected (valid: `page-meta`, `github-state`, `selector`).
+- Unknown inspect signals are rejected (valid: `page-meta`, `selector`).
 - Selector `attr` supports `href-url`/`src-url` to return absolute http(s) URLs.
 - `screenshot --out` writes per-tab folders into the target directory.
 - `tabctl undo` accepts a positional txid, `--txid`, or `--latest`.

package/dist/cli/lib/client.js

@@ -7,7 +7,7 @@ exports.createRequestId = createRequestId;
 exports.sendRequest = sendRequest;
 exports.fetchSnapshot = fetchSnapshot;
 exports.sendFireAndForget = sendFireAndForget;
-const net_1 = __importDefault(require("net"));
+const node_net_1 = __importDefault(require("node:net"));
 const constants_1 = require("./constants");
 function createRequestId() {
     return `req-${Date.now()}-${Math.random().toString(16).slice(2, 10)}`;
@@ -15,7 +15,7 @@ function createRequestId() {
 function sendRequest(payload, onProgress) {
     return new Promise((resolve, reject) => {
         const { socketPath } = (0, constants_1.resolveConfig)();
-        const client = net_1.default.createConnection(socketPath);
+        const client = node_net_1.default.createConnection(socketPath);
         let buffer = "";
         client.on("connect", () => {
             client.write(`${JSON.stringify(payload)}\n`);
@@ -65,7 +65,7 @@ async function fetchSnapshot() {
 function sendFireAndForget(payload) {
     try {
         const { socketPath } = (0, constants_1.resolveConfig)();
-        const client = net_1.default.createConnection(socketPath);
+        const client = node_net_1.default.createConnection(socketPath);
         client.on("connect", () => {
             client.write(`${JSON.stringify(payload)}\n`);
             // Unref after write so Node can exit without waiting for response

package/dist/cli/lib/commands/doctor.js

@@ -0,0 +1,134 @@
+"use strict";
+/**
+ * Doctor command handler: diagnose and repair profile health.
+ *
+ * Checks each profile's wrapper for valid Node/host paths, verifies
+ * extension sync status, and optionally auto-repairs broken wrappers.
+ */
+var __importDefault = (this && this.__importDefault) || function (mod) {
+    return (mod && mod.__esModule) ? mod : { "default": mod };
+};
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.runDoctor = runDoctor;
+const node_fs_1 = __importDefault(require("node:fs"));
+const node_path_1 = __importDefault(require("node:path"));
+const config_1 = require("../../../shared/config");
+const profiles_1 = require("../../../shared/profiles");
+const wrapper_health_1 = require("../../../shared/wrapper-health");
+const extension_sync_1 = require("../../../shared/extension-sync");
+const setup_1 = require("./setup");
+const output_1 = require("../output");
+function checkProfile(name, entry, fix) {
+    const wrapperPath = (0, wrapper_health_1.resolveWrapperPath)(entry.dataDir);
+    const check = (0, wrapper_health_1.checkWrapper)(wrapperPath);
+    const issues = [...check.issues];
+    let fixed = false;
+    if (fix && !check.ok && check.info) {
+        const needsNodeFix = !node_fs_1.default.existsSync(check.info.nodePath);
+        const needsHostFix = !node_fs_1.default.existsSync(check.info.hostPath);
+        if (needsNodeFix || needsHostFix) {
+            const newNodePath = needsNodeFix ? process.execPath : check.info.nodePath;
+            let newHostPath = check.info.hostPath;
+            if (needsHostFix) {
+                // Use the stable bundled host path
+                try {
+                    const config = (0, config_1.resolveConfig)();
+                    newHostPath = (0, extension_sync_1.resolveInstalledHostPath)(config.baseDataDir);
+                    if (!node_fs_1.default.existsSync(newHostPath)) {
+                        issues.push(`Bundled host not found at ${newHostPath} — run: tabctl setup --browser ${entry.browser}`);
+                    }
+                }
+                catch {
+                    issues.push("Could not resolve bundled host path");
+                }
+            }
+            try {
+                (0, setup_1.writeWrapper)(newNodePath, newHostPath, check.info.profileName, node_path_1.default.dirname(wrapperPath));
+                fixed = true;
+                // Update issue messages to show they were fixed
+                const fixedIssues = [];
+                if (needsNodeFix) {
+                    fixedIssues.push(`Fixed Node path: ${check.info.nodePath} → ${newNodePath}`);
+                }
+                if (needsHostFix) {
+                    fixedIssues.push(`Fixed host path: ${check.info.hostPath} → ${newHostPath}`);
+                }
+                // Replace original issues with fixed messages
+                issues.length = 0;
+                issues.push(...fixedIssues);
+            }
+            catch (err) {
+                issues.push(`Failed to fix wrapper: ${err instanceof Error ? err.message : String(err)}`);
+            }
+        }
+    }
+    return {
+        ok: check.ok || fixed,
+        browser: entry.browser,
+        dataDir: entry.dataDir,
+        wrapperPath,
+        issues,
+        fixed,
+    };
+}
+function runDoctor(options, prettyOutput) {
+    const fix = options.fix === true;
+    const config = (0, config_1.resolveConfig)();
+    const registry = (0, profiles_1.loadProfiles)(config.configDir);
+    const profileNames = Object.keys(registry.profiles);
+    if (profileNames.length === 0) {
+        (0, output_1.errorOut)("No profiles configured. Run: tabctl setup --browser <edge|chrome>");
+    }
+    // Check each profile
+    const profiles = {};
+    for (const name of profileNames) {
+        profiles[name] = checkProfile(name, registry.profiles[name], fix);
+    }
+    // Check extension sync status
+    let extensionCheck;
+    try {
+        const sync = (0, extension_sync_1.checkExtensionSync)(config.baseDataDir);
+        extensionCheck = {
+            ok: !sync.needsSync,
+            synced: !sync.needsSync,
+            bundledVersion: sync.bundledVersion,
+            installedVersion: sync.installedVersion,
+        };
+    }
+    catch {
+        extensionCheck = {
+            ok: false,
+            synced: false,
+            bundledVersion: null,
+            installedVersion: null,
+        };
+    }
+    // Summary
+    const total = profileNames.length;
+    const healthy = Object.values(profiles).filter(p => p.ok).length;
+    const broken = total - healthy;
+    const fixed = Object.values(profiles).filter(p => p.fixed).length;
+    const allOk = broken === 0 && extensionCheck.ok;
+    (0, output_1.printJson)({
+        ok: allOk,
+        action: "doctor",
+        data: {
+            profiles,
+            extension: extensionCheck,
+            summary: { total, healthy, broken, fixed },
+        },
+    }, prettyOutput);
+    // Helpful stderr hints
+    if (!allOk && !fix) {
+        const brokenNames = Object.entries(profiles)
+            .filter(([, p]) => !p.ok)
+            .map(([n]) => n);
+        if (brokenNames.length > 0) {
+            process.stderr.write(`\nBroken profiles: ${brokenNames.join(", ")}\n`);
+            process.stderr.write("Run: tabctl doctor --fix\n\n");
+        }
+    }
+    if (fixed > 0) {
+        process.stderr.write(`\nFixed ${fixed} profile(s). Verify: tabctl ping\n\n`);
+    }
+}

package/dist/cli/lib/commands/index.js

@@ -4,10 +4,13 @@
  * Re-exports all command handlers and parameter builders.
  */
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.buildUndoParams = exports.buildHistoryParams = exports.buildScreenshotParams = exports.buildReportParams = exports.buildCloseParams = exports.buildArchiveParams = exports.buildMergeWindowParams = exports.buildMoveGroupParams = exports.buildMoveTabParams = exports.buildGroupAssignParams = exports.buildGroupUngroupParams = exports.buildGroupUpdateParams = exports.buildOpenParams = exports.buildRefreshParams = exports.buildFocusParams = exports.buildInspectParams = exports.buildAnalyzeParams = exports.runProfileRemove = exports.runProfileSwitch = exports.runProfileShow = exports.runProfileList = exports.runGroupList = exports.runList = exports.runPing = exports.runUndo = exports.runHistory = exports.runPolicy = exports.runVersion = exports.runSkillInstall = exports.runSetup = void 0;
+exports.buildUndoParams = exports.buildHistoryParams = exports.buildScreenshotParams = exports.buildReportParams = exports.buildCloseParams = exports.buildArchiveParams = exports.buildMergeWindowParams = exports.buildMoveGroupParams = exports.buildMoveTabParams = exports.buildGroupGatherParams = exports.buildGroupAssignParams = exports.buildGroupUngroupParams = exports.buildGroupUpdateParams = exports.buildOpenParams = exports.buildRefreshParams = exports.buildFocusParams = exports.buildInspectParams = exports.buildAnalyzeParams = exports.runProfileRemove = exports.runProfileSwitch = exports.runProfileShow = exports.runProfileList = exports.runGroupList = exports.runList = exports.runPing = exports.runUndo = exports.runHistory = exports.runPolicy = exports.runVersion = exports.runSkillInstall = exports.runDoctor = exports.runSetup = void 0;
 // Setup command
 var setup_1 = require("./setup");
 Object.defineProperty(exports, "runSetup", { enumerable: true, get: function () { return setup_1.runSetup; } });
+// Doctor command
+var doctor_1 = require("./doctor");
+Object.defineProperty(exports, "runDoctor", { enumerable: true, get: function () { return doctor_1.runDoctor; } });
 // Meta commands (version, ping, skill, policy, history, undo)
 var meta_1 = require("./meta");
 Object.defineProperty(exports, "runSkillInstall", { enumerable: true, get: function () { return meta_1.runSkillInstall; } });
@@ -36,6 +39,7 @@ Object.defineProperty(exports, "buildOpenParams", { enumerable: true, get: funct
 Object.defineProperty(exports, "buildGroupUpdateParams", { enumerable: true, get: function () { return params_1.buildGroupUpdateParams; } });
 Object.defineProperty(exports, "buildGroupUngroupParams", { enumerable: true, get: function () { return params_1.buildGroupUngroupParams; } });
 Object.defineProperty(exports, "buildGroupAssignParams", { enumerable: true, get: function () { return params_1.buildGroupAssignParams; } });
+Object.defineProperty(exports, "buildGroupGatherParams", { enumerable: true, get: function () { return params_1.buildGroupGatherParams; } });
 Object.defineProperty(exports, "buildMoveTabParams", { enumerable: true, get: function () { return params_1.buildMoveTabParams; } });
 Object.defineProperty(exports, "buildMoveGroupParams", { enumerable: true, get: function () { return params_1.buildMoveGroupParams; } });
 Object.defineProperty(exports, "buildMergeWindowParams", { enumerable: true, get: function () { return params_1.buildMergeWindowParams; } });

package/dist/cli/lib/commands/meta.js

@@ -15,9 +15,9 @@ exports.runPolicy = runPolicy;
 exports.runHistory = runHistory;
 exports.runUndo = runUndo;
 exports.runPing = runPing;
-const fs_1 = __importDefault(require("fs"));
-const os_1 = __importDefault(require("os"));
-const path_1 = __importDefault(require("path"));
+const node_fs_1 = __importDefault(require("node:fs"));
+const node_os_1 = __importDefault(require("node:os"));
+const node_path_1 = __importDefault(require("node:path"));
 const node_child_process_1 = require("node:child_process");
 const constants_1 = require("../constants");
 const output_1 = require("../output");
@@ -31,20 +31,20 @@ Object.defineProperty(exports, "runSetup", { enumerable: true, get: function ()
 // ============================================================================
 function resolveProjectRoot() {
     try {
-        return fs_1.default.realpathSync(process.cwd());
+        return node_fs_1.default.realpathSync(process.cwd());
     }
     catch {
-        return path_1.default.resolve(process.cwd());
+        return node_path_1.default.resolve(process.cwd());
     }
 }
 function resolveConfigHome() {
-    return process.env.XDG_CONFIG_HOME || path_1.default.join(os_1.default.homedir(), ".config");
+    return process.env.XDG_CONFIG_HOME || node_path_1.default.join(node_os_1.default.homedir(), ".config");
 }
 function resolveSkillTargetDir(globalInstall) {
     if (globalInstall) {
-        return path_1.default.join(resolveConfigHome(), "opencode", "skills", constants_1.SKILL_NAME);
+        return node_path_1.default.join(resolveConfigHome(), "opencode", "skills", constants_1.SKILL_NAME);
     }
-    return path_1.default.join(resolveProjectRoot(), ".opencode", "skills", constants_1.SKILL_NAME);
+    return node_path_1.default.join(resolveProjectRoot(), ".opencode", "skills", constants_1.SKILL_NAME);
 }
 function runSkillsCli(args) {
     const result = (0, node_child_process_1.spawnSync)("npx", ["skills", ...args], { stdio: "pipe", shell: process.platform === "win32" });
@@ -116,7 +116,7 @@ function runVersion(prettyOutput) {
 function runPolicy(options, policyContext, prettyOutput) {
     const policyPath = (0, policy_1.defaultPolicyPath)();
     if (options.init) {
-        if (fs_1.default.existsSync(policyPath)) {
+        if (node_fs_1.default.existsSync(policyPath)) {
             (0, output_1.printJson)({
                 ok: true,
                 data: {
@@ -126,9 +126,9 @@ function runPolicy(options, policyContext, prettyOutput) {
             }, prettyOutput);
             return;
         }
-        const dir = path_1.default.dirname(policyPath);
-        fs_1.default.mkdirSync(dir, { recursive: true });
-        fs_1.default.writeFileSync(policyPath, JSON.stringify((0, policy_1.defaultPolicyTemplate)(), null, 2), "utf8");
+        const dir = node_path_1.default.dirname(policyPath);
+        node_fs_1.default.mkdirSync(dir, { recursive: true });
+        node_fs_1.default.writeFileSync(policyPath, JSON.stringify((0, policy_1.defaultPolicyTemplate)(), null, 2), "utf8");
         (0, output_1.printJson)({
             ok: true,
             data: {

package/dist/cli/lib/commands/params-groups.js

@@ -6,6 +6,7 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.buildGroupUpdateParams = buildGroupUpdateParams;
 exports.buildGroupUngroupParams = buildGroupUngroupParams;
 exports.buildGroupAssignParams = buildGroupAssignParams;
+exports.buildGroupGatherParams = buildGroupGatherParams;
 const params_1 = require("./params");
 function buildGroupUpdateParams(options) {
     const windowValue = (0, params_1.parseWindowScope)(options.window, { allowNew: false });
@@ -38,3 +39,10 @@ function buildGroupAssignParams(options) {
         collapsed: options.collapsed === true ? true : options.expanded === true ? false : undefined,
     };
 }
+function buildGroupGatherParams(options) {
+    const windowValue = (0, params_1.parseWindowScope)(options.window, { allowNew: false });
+    return {
+        windowId: windowValue,
+        groupTitle: options.group,
+    };
+}

package/dist/cli/lib/commands/params.js

@@ -7,7 +7,7 @@ var __importDefault = (this && this.__importDefault) || function (mod) {
     return (mod && mod.__esModule) ? mod : { "default": mod };
 };
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.buildMergeWindowParams = exports.buildMoveGroupParams = exports.buildMoveTabParams = exports.buildGroupUngroupParams = exports.buildGroupAssignParams = exports.buildGroupUpdateParams = void 0;
+exports.buildMergeWindowParams = exports.buildMoveGroupParams = exports.buildMoveTabParams = exports.buildGroupGatherParams = exports.buildGroupUngroupParams = exports.buildGroupAssignParams = exports.buildGroupUpdateParams = void 0;
 exports.buildAnalyzeParams = buildAnalyzeParams;
 exports.buildInspectParams = buildInspectParams;
 exports.buildFocusParams = buildFocusParams;
@@ -20,7 +20,7 @@ exports.buildScreenshotParams = buildScreenshotParams;
 exports.parseWindowScope = parseWindowScope;
 exports.buildHistoryParams = buildHistoryParams;
 exports.buildUndoParams = buildUndoParams;
-const fs_1 = __importDefault(require("fs"));
+const node_fs_1 = __importDefault(require("node:fs"));
 const output_1 = require("../output");
 const args_1 = require("../args");
 // ============================================================================
@@ -46,14 +46,11 @@ function buildAnalyzeParams(options) {
     const { tabIds, groupTitle, groupId, windowId, all } = buildScopeFromOptions(options);
     return {
         staleDays: options["stale-days"] ? Number(options["stale-days"]) : undefined,
-        checkGitHub: Boolean(options.github),
         tabIds,
         groupTitle,
         groupId,
         windowId,
         all,
-        githubConcurrency: options["github-concurrency"] ? Number(options["github-concurrency"]) : undefined,
-        githubTimeoutMs: options["github-timeout-ms"] ? Number(options["github-timeout-ms"]) : undefined,
         progress: Boolean(options.progress),
     };
 }
@@ -99,7 +96,7 @@ function buildInspectParams(options) {
     let signalConfig;
     if (options["signal-config"]) {
         try {
-            const configRaw = fs_1.default.readFileSync(String(options["signal-config"]), "utf8");
+            const configRaw = node_fs_1.default.readFileSync(String(options["signal-config"]), "utf8");
             signalConfig = JSON.parse(configRaw);
         }
         catch {
@@ -164,6 +161,8 @@ function buildOpenParams(options) {
         windowGroupTitle: options["window-group"],
         windowTabId: options["window-tab"] ? Number(options["window-tab"]) : undefined,
         windowUrl: options["window-url"],
+        newGroup: options["new-group"] === true,
+        allowDuplicates: options["allow-duplicates"] === true,
     };
 }
 // ============================================================================
@@ -173,6 +172,7 @@ var params_groups_1 = require("./params-groups");
 Object.defineProperty(exports, "buildGroupUpdateParams", { enumerable: true, get: function () { return params_groups_1.buildGroupUpdateParams; } });
 Object.defineProperty(exports, "buildGroupAssignParams", { enumerable: true, get: function () { return params_groups_1.buildGroupAssignParams; } });
 Object.defineProperty(exports, "buildGroupUngroupParams", { enumerable: true, get: function () { return params_groups_1.buildGroupUngroupParams; } });
+Object.defineProperty(exports, "buildGroupGatherParams", { enumerable: true, get: function () { return params_groups_1.buildGroupGatherParams; } });
 // ============================================================================
 // Move Command Parameters (re-exported from params-move.ts)
 // ============================================================================