@henryqw/pi-herdr-done 0.5.1 → 0.5.3

package/README.md

@@ -34,17 +34,3 @@ The parent pull runs only when this session ran in a linked worktree with a non-
 Tabs in the current Herdr workspace close automatically. When a tab from another workspace still uses the current checkout, `/done` refuses and lists it by name; use `/done --force` to remove the checkout regardless. Command requires Pi running inside Herdr with `HERDR_ENV=1`, `HERDR_WORKSPACE_ID`, and `HERDR_TAB_ID` set.
 
 Dirty worktrees make `/done` fail. Commit or discard changes, or use `/done --force` to explicitly delete them.
-
-## Remove
-
-```bash
-pi remove npm:@henryqw/pi-herdr-done
-```
-
-## Development
-
-```bash
-npm test --workspace @henryqw/pi-herdr-done
-npm run typecheck --workspace @henryqw/pi-herdr-done
-npm run pack:check --workspace @henryqw/pi-herdr-done
-```

package/extensions/done.ts

@@ -64,15 +64,13 @@ export default function herdrDoneExtension(pi: ExtensionAPI): void {
 				const listing = await herdr.json(["tab", "list"], { cwd: ctx.cwd });
 				const tabs = (listing.result as { tabs?: TabEntry[] } | undefined)?.tabs;
 				if (!Array.isArray(tabs)) throw new Error("herdr tab list returned no tabs.");
-				const tabsById = new Map(tabs
-					.filter((tab): tab is TabEntry & { tab_id: string } => typeof tab.tab_id === "string")
-					.map((tab) => [tab.tab_id, tab]));
+				const validTabs = tabs.filter((tab): tab is TabEntry & { tab_id: string } => typeof tab.tab_id === "string");
+				const tabsById = new Map(validTabs.map((tab) => [tab.tab_id, tab]));
 				if (tabsById.get(tabId)?.workspace_id !== workspaceId) {
 					throw new Error(`Herdr tab ${tabId} does not belong to workspace ${workspaceId}.`);
 				}
-				siblingTabIds = tabs
-					.filter((tab): tab is TabEntry & { tab_id: string } =>
-						typeof tab.tab_id === "string" && tab.tab_id !== tabId && tab.workspace_id === workspaceId)
+				siblingTabIds = validTabs
+					.filter((tab) => tab.tab_id !== tabId && tab.workspace_id === workspaceId)
 					.map((tab) => tab.tab_id);
 				const blockers = dependentIds.filter((id) => tabsById.get(id)?.workspace_id !== workspaceId);
 				if (blockers.length > 0) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@henryqw/pi-herdr-done",
-  "version": "0.5.1",
+  "version": "0.5.3",
   "description": "Close and remove the current Herdr worktree from Pi.",
   "keywords": [
     "pi-package",