@danypops/pi-packed 0.19.6 → 0.19.9

package/extension/src/index.ts

@@ -12,8 +12,8 @@ import type { ExtensionAPI } from "@earendil-works/pi-coding-agent";
 import { formatUpdateNotice } from "./model.js";
 import { createNatives } from "./packed.js";
 import { registerProfiles } from "./profile.js";
-import { handleResourceConfigCommand } from "./resource-config.js";
 import { handleSetupCommand } from "./setup-command.js";
+import { handleResourceConfigCommand } from "./tabs/resource-config.js";
 import { registerTools } from "./tools.js";
 import { showPackedPanel } from "./tui.js";
 

package/extension/src/packed.ts

@@ -8,6 +8,13 @@
  * the rest of the Pi session -- vehicle-client's createRetryingClient detects
  * that on the failing call itself and reconnects, so the happy path reuses
  * one connected client instead of reconnecting on every single operation.
+ *
+ * Reads/idempotent lookups use call() (transparent retry once on a stale
+ * connection); install/installService/remove/update/setMutationApproval/
+ * setupApply/toggleResource use callOnce() instead -- a stale connection
+ * during any of those must surface its one real failure rather than risk a
+ * silently duplicated mutation (installing/removing/updating a package
+ * twice) if the retried call actually reached the daemon the first time.
  */
 
 import { ensurePackedClient, InstallServiceError, type PackedExtensionClient } from "@danypops/packed/client";
@@ -77,22 +84,25 @@ export async function createNatives(connect: PackageDaemonConnector = connectDef
 	const client = createRetryingClient(connect, { label: "pi-packed" });
 
 	return {
+		// Reads/idempotent lookups: transparently retried once on a stale connection.
 		search: (query, limit) => client.call((daemon) => daemon.search(query, limit)),
 		searchOffline: (query, limit) => client.call((daemon) => daemon.search(query, limit, true)),
 		info: (name) => client.call((daemon) => daemon.info(name)),
 		installed: () => client.call((daemon) => daemon.installed()),
 		updates: () => client.call((daemon) => daemon.updates()),
 		security: () => client.call((daemon) => daemon.security()),
-		setMutationApproval: (value, approved) => client.call((daemon) => daemon.setMutationApproval(value, approved)),
-		install: (source, approved) => client.call((daemon) => daemon.install(source, approved)),
-		installService: (source, approved) => client.call((daemon) => daemon.installService(source, approved)),
-		remove: (name, approved) => client.call((daemon) => daemon.remove(name, approved)),
-		update: (source, approved) => client.call((daemon) => daemon.update(source, approved)),
 		setupPlan: (manifestPath, prune) => client.call((daemon) => daemon.setupPlan(manifestPath, prune)),
-		setupApply: (manifestPath, approved, prune) => client.call((daemon) => daemon.setupApply(manifestPath, approved, prune)),
 		listResources: (projectRoot) => client.call((daemon) => daemon.listResources(projectRoot)),
-		toggleResource: (source, field, path, enabled, projectRoot, approved) =>
-			client.call((daemon) => daemon.toggleResource(source, field, path, enabled, projectRoot, approved)),
 		piStatus: () => client.call((daemon) => daemon.piStatus()),
+		// Mutations: never transparently retried -- a stale connection surfaces its one
+		// real failure instead of risking a silently duplicated install/remove/update.
+		setMutationApproval: (value, approved) => client.callOnce((daemon) => daemon.setMutationApproval(value, approved)),
+		install: (source, approved) => client.callOnce((daemon) => daemon.install(source, approved)),
+		installService: (source, approved) => client.callOnce((daemon) => daemon.installService(source, approved)),
+		remove: (name, approved) => client.callOnce((daemon) => daemon.remove(name, approved)),
+		update: (source, approved) => client.callOnce((daemon) => daemon.update(source, approved)),
+		setupApply: (manifestPath, approved, prune) => client.callOnce((daemon) => daemon.setupApply(manifestPath, approved, prune)),
+		toggleResource: (source, field, path, enabled, projectRoot, approved) =>
+			client.callOnce((daemon) => daemon.toggleResource(source, field, path, enabled, projectRoot, approved)),
 	};
 }

package/extension/src/{discover.ts → tabs/discover.ts}

@@ -13,11 +13,11 @@ import type { ExtensionCommandContext } from "@earendil-works/pi-coding-agent";
 import { rawKeyHint } from "@earendil-works/pi-coding-agent";
 import { Input, truncateToWidth, visibleWidth } from "@earendil-works/pi-tui";
 import type { Component } from "malevich-tui-components";
+import type { Natives, PackageSummary } from "../packed.js";
+import { InstallServiceError } from "../packed.js";
+import type { TabHost } from "../tab-host.js";
+import { approvePackageOperation } from "../tools.js";
 import { shouldSearch } from "./discover-model.js";
-import type { Natives, PackageSummary } from "./packed.js";
-import { InstallServiceError } from "./packed.js";
-import type { TabHost } from "./tab-host.js";
-import { approvePackageOperation } from "./tools.js";
 
 const SEARCH_LIMIT = 20;
 

package/extension/src/{resource-config.ts → tabs/resource-config.ts}

@@ -17,10 +17,10 @@ import type { ExtensionCommandContext } from "@earendil-works/pi-coding-agent";
 import { rawKeyHint } from "@earendil-works/pi-coding-agent";
 import { Input, truncateToWidth, visibleWidth } from "@earendil-works/pi-tui";
 import type { Component } from "malevich-tui-components";
-import type { Natives, PackageResources, ResourceField } from "./packed.js";
-import { packagePermissionDecision } from "./permission.js";
-import { confirmReload } from "./reload.js";
-import type { TabHost } from "./tab-host.js";
+import { packagePermissionDecision } from "../approval/permission.js";
+import { confirmReload } from "../approval/reload.js";
+import type { Natives, PackageResources, ResourceField } from "../packed.js";
+import type { TabHost } from "../tab-host.js";
 
 type Scope = "global" | "project";
 const RESOURCE_FIELDS = ["extensions", "skills", "prompts", "themes"] as const satisfies readonly ResourceField[];

package/extension/src/{security-tui.ts → tabs/security-tui.ts}

@@ -15,8 +15,8 @@ import type { Component } from "malevich-tui-components";
 
 type MutationApproval = SecuritySettings["mutationApproval"];
 
-import type { Natives } from "./packed.ts";
-import type { TabHost } from "./tab-host.ts";
+import type { Natives } from "../packed.ts";
+import type { TabHost } from "../tab-host.ts";
 
 const OPTIONS: Array<{ value: MutationApproval; label: string }> = [
 	{ value: "always", label: "Always require mutation approval (recommended)" },

package/extension/src/tools.ts

@@ -1,9 +1,9 @@
 /** Agent-facing package tools over the authenticated daemon. */
 import type { ExtensionAPI } from "@earendil-works/pi-coding-agent";
 import { Type } from "typebox";
+import { type PackageOperation, packagePermissionDecision } from "./approval/permission.js";
+import { reloadWarning } from "./approval/reload.js";
 import { InstallServiceError, type Natives, PI_COMMAND_NAME } from "./packed.js";
-import { type PackageOperation, packagePermissionDecision } from "./permission.js";
-import { reloadWarning } from "./reload.js";
 import {
 	createInfoDetails,
 	createModelContent,

package/extension/src/tui.ts

@@ -58,15 +58,15 @@ import {
 	Table,
 	type TextMeasure,
 } from "malevich-tui-components";
-import { createFindTab } from "./discover.js";
+import { confirmReload } from "./approval/reload.js";
 import { dialogTheme, menuTheme, tabBarTheme } from "./menu-theme.js";
 import type { Row, ViewMode } from "./model.js";
 import { filterRows, mergeRows, nextMode, visibleRows } from "./model.js";
 import type { Natives, PackageResources } from "./packed.js";
-import { confirmReload } from "./reload.js";
-import { applyResourceToggle, ConfigTab } from "./resource-config.js";
-import { SettingsTab } from "./security-tui.js";
 import type { TabHost } from "./tab-host.js";
+import { createFindTab } from "./tabs/discover.js";
+import { applyResourceToggle, ConfigTab } from "./tabs/resource-config.js";
+import { SettingsTab } from "./tabs/security-tui.js";
 import { approvePackageOperation } from "./tools.js";
 
 export type PackedTabKey = "packages" | "find" | "config" | "settings";

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@danypops/pi-packed",
-  "version": "0.19.6",
+  "version": "0.19.9",
   "description": "Pi package tools, commands, profiles, and TUI for the Packed daemon",
   "type": "module",
   "keywords": ["pi-package"],
@@ -13,7 +13,7 @@
   },
   "dependencies": {
     "@danypops/packed": "^0.5.0",
-    "@danypops/vehicle-client": "^0.1.1",
+    "@danypops/vehicle-client": "^0.2.0",
     "@danypops/vehicle-client-pi": "^0.1.5",
     "malevich-tui-components": "^0.16.1"
   },