webhelm 1.0.23 → 1.0.25

package/README.md

@@ -39,7 +39,7 @@ The daemon-managed Chrome Extension is installed from the npm Runtime and does n
 
 The compact popup shows daemon connectivity and per-workspace capacity and Session counts, with confirmation before stopping a Session or clearing Sessions in one workspace. The popup does not include a global stop-all action. These actions are executed by the same Node daemon as the CLI, over the existing CDP-backed Extension RPC. No shell access or HTTP listener is granted to the Extension. The persistent ChatGPT queue is not deleted, and queued work may create new Sessions afterward.
 
-New installations use the default Chrome tab group `WebHelm` and the idle page `helm.html`. Idle URLs use only `workspace` and `role=idle`; Lease URLs use Chrome's actual `tabId` for initial CDP target discovery. There is no separate slot identity. The default `WebHelm` tab group is placed at the first position after pinned tabs when created or restored, while Session allocation does not override later user rearrangement. Chrome-saved tab group shortcuts in the bookmarks bar are separate from active groups and cannot safely be deduplicated through the Extension API. Legacy `Harness` default groups migrate only when their tabs are idle. Custom workspace names are unchanged; the old `workspace.html` path and lease titles remain readable until their Sessions finish. Closing an allocated Session returns its managed tabs to the idle page, retaining physical Chrome targets, while unleased popup children are closed. Do not manually remove active managed tabs.
+New installations use the default Chrome tab group `WebHelm` and the idle page `helm.html`. Idle URLs use only `workspace` and `role=idle`; Lease URLs use Chrome's actual `tabId` for initial CDP target discovery. There is no separate slot identity. Every newly created WebHelm-managed tab group (default and custom) is placed at the far left of its Chrome window, immediately after pinned tabs. Existing groups retain their user-chosen position during restoration, legacy migration and Session allocation. Chrome-saved tab group shortcuts in the bookmarks bar are separate from active groups and cannot safely be deduplicated through the Extension API. Legacy `Harness` default groups migrate only when their tabs are idle. Custom workspace names are unchanged; the old `workspace.html` path and lease titles remain readable until their Sessions finish. Closing an allocated Session returns its managed tabs to the idle page, retaining physical Chrome targets, while unleased popup children are closed. Do not manually remove active managed tabs.
 
 For a real browser integration check on macOS, run `node test/e2e/extension-control.e2e.mjs`; it creates a temporary isolated Chrome profile and daemon and does not touch the current production Sessions.
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "webhelm",
-  "version": "1.0.23",
+  "version": "1.0.25",
   "description": "Agent browser automation in a visible managed Chrome with persistent JavaScript sessions",
   "type": "module",
   "bin": {

package/runtime/extension/manifest.json

@@ -1,7 +1,7 @@
 {
   "manifest_version": 3,
   "name": "WebHelm",
-  "version": "0.4.8",
+  "version": "0.4.10",
   "description": "Own named Chrome tab-group workspaces and lazy reusable tab pools for browser agents.",
   "permissions": [
     "storage",

package/runtime/extension/workspace-manager.mjs

@@ -242,20 +242,14 @@ export class WorkspaceManager {
     return await this.chrome.tabs.query({ groupId });
   }
 
-  // Only position the default group when it is created, migrated or restored.
-  // Never move it during routine Session allocation or periodic health checks,
-  // since the user may intentionally reorder the tab strip.
-  async positionDefaultGroupFirst(groupId, name) {
-    if (name !== "WebHelm") return false;
+  // New groups belong immediately after pinned tabs, regardless of Chrome's
+  // insertion policy. Existing groups keep their order on restore and lease.
+  async positionNewGroupLeftmost(groupId) {
     const groupTabs = await this.groupTabs(groupId);
-    if (!groupTabs.length) return false;
-    const windowId = groupTabs[0].windowId;
-    const windowTabs = await this.chrome.tabs.query({windowId});
-    const firstUnpinnedIndex = windowTabs.filter(tab => tab.pinned).length;
-    const firstGroupIndex = Math.min(...groupTabs.map(tab => tab.index));
-    if (firstGroupIndex === firstUnpinnedIndex) return false;
-    await this.chrome.tabGroups.move(groupId, {index: firstUnpinnedIndex});
-    return true;
+    if (!groupTabs.length) return;
+    const windowTabs = await this.chrome.tabs.query({ windowId: groupTabs[0].windowId });
+    const firstUnpinned = windowTabs.filter(tab => tab.pinned).length;
+    await this.chrome.tabGroups.move(groupId, { index: firstUnpinned });
   }
 
   tabUrl(tab) {
@@ -507,7 +501,7 @@ export class WorkspaceManager {
     for (const tab of created) this.markOwned(tab.id);
     const groupId = await this.chrome.tabs.group({ tabIds: created.map((tab) => tab.id) });
     await this.chrome.tabGroups.update(groupId, { title: name, collapsed: true });
-    await this.positionDefaultGroupFirst(groupId, name);
+    await this.positionNewGroupLeftmost(groupId);
     for (const tab of created) await this.waitForIdleTab(tab.id, name);
     return await this.status(name);
   }
@@ -586,8 +580,6 @@ export class WorkspaceManager {
     for (const name of Object.keys(config.workspaces || {}).sort()) {
       restored.push(await this.ensureWorkspace(name));
     }
-    const defaultGroup = restored.find(item => item.name === "WebHelm" && item.initialized);
-    if (defaultGroup) await this.positionDefaultGroupFirst(defaultGroup.groupId, "WebHelm");
     return { workspaces: restored };
   }
 
@@ -625,9 +617,6 @@ export class WorkspaceManager {
         await this.chrome.tabs.update(tab.id, {url: this.idleUrl(newName), active: false});
         updated.push(tab);
       }
-      // Position before committing the new config so move failures still
-      // unwind the legacy migration coherently in the catch block.
-      if (group) await this.positionDefaultGroupFirst(group.id, newName);
       config.workspaces[newName] = config.workspaces[oldName];
       delete config.workspaces[oldName];
       await this.saveConfig(config);