@danypops/pi-lector 0.9.3 → 0.9.4

package/extension/src/apply-patch-operations.ts

@@ -21,7 +21,7 @@ export function createLectorApplyPatchOperations(): ApplyPatchOperations {
 				async ({ workspaceId, root }) => {
 					const client = await lectorClient();
 					const relativePath = toWorkspaceRelativePath(root, path);
-					return client.call("workspace.applyPatch", { workspaceId, path: relativePath, expectedHash, patchText });
+					return client.callOnce("workspace.applyPatch", { workspaceId, path: relativePath, expectedHash, patchText });
 				},
 			);
 		},

package/extension/src/code-intelligence-operations.ts

@@ -140,7 +140,7 @@ export function createLectorCodeIntelligenceOperations(): CodeIntelligenceOperat
 				() => workspaceForPathOrDirectory(path),
 				async ({ workspaceId }) => {
 					const client = await lectorClient();
-					const { job } = await client.call("job.submit", {
+					const { job } = await client.callOnce("job.submit", {
 						operation: "workspace.populateSymbolGraph",
 						input: { workspaceId, maxFiles, maxSymbolsPerFile },
 						waitMs,

package/extension/src/edit-operations.ts

@@ -50,7 +50,7 @@ export function createLectorEditOperations(): EditOperations {
 					const client = await lectorClient();
 					const relativePath = toWorkspaceRelativePath(root, absolutePath);
 					try {
-						await client.call("workspace.exactEdit", { workspaceId, path: relativePath, expectedHash, content });
+						await client.callOnce("workspace.exactEdit", { workspaceId, path: relativePath, expectedHash, content });
 					} catch (error) {
 						if (remoteErrorIs(error, "StaleExpectedHash")) {
 							throw new Error(`"${relativePath}" changed on disk since it was last read; re-read the file and retry the edit.`);

package/extension/src/lector-client.ts

@@ -53,7 +53,14 @@ export function setNewWorkspaceObserver(observer: ((root: string) => void) | und
 }
 
 export interface RetryingLectorClient {
+	/** Transparently retries once on a stale connection -- right for a read-only or genuinely idempotent operation. */
 	call<Name extends OperationName>(operation: Name, input: OperationInputs[Name]): Promise<OperationOutputs[Name]>;
+	/**
+	 * Like call(), but never retries the operation itself after a failure -- only the underlying
+	 * connection resets, so the *next* call()/callOnce() reconnects. Use for a mutating/non-idempotent
+	 * operation, where transparently re-running it after a transport failure could double the side effect.
+	 */
+	callOnce<Name extends OperationName>(operation: Name, input: OperationInputs[Name]): Promise<OperationOutputs[Name]>;
 }
 
 // Kept async even though its own body has no await: every call site across this package does
@@ -62,6 +69,7 @@ export interface RetryingLectorClient {
 export async function lectorClient(): Promise<RetryingLectorClient> {
 	return {
 		call: (operation, input) => retryingClient.call((client) => client.call(operation, input)),
+		callOnce: (operation, input) => retryingClient.callOnce((client) => client.call(operation, input)),
 	};
 }
 
@@ -75,7 +83,7 @@ async function workspaceForRoot(root: string): Promise<ResolvedWorkspace> {
 	const cached = workspaceIdByRoot.get(root);
 	if (cached) return { workspaceId: cached, root };
 	const client = await lectorClient();
-	const { workspaceId } = await client.call("workspace.registerPath", { path: root });
+	const { workspaceId } = await client.callOnce("workspace.registerPath", { path: root });
 	workspaceIdByRoot.set(root, workspaceId);
 	onNewWorkspace?.(root);
 	return { workspaceId, root };

package/extension/src/line-edit-operations.ts

@@ -23,7 +23,7 @@ export function createLectorLineEditOperations(): LineEditOperations {
 				async ({ workspaceId, root }) => {
 					const client = await lectorClient();
 					const relativePath = toWorkspaceRelativePath(root, path);
-					return client.call("workspace.lineEdit", { workspaceId, path: relativePath, edits });
+					return client.callOnce("workspace.lineEdit", { workspaceId, path: relativePath, edits });
 				},
 			);
 		},

package/extension/src/mutation-history-operations.ts

@@ -26,7 +26,7 @@ export function createMutationHistoryOperations(): MutationHistoryOperations {
 				() => workspaceForPath(absolutePath),
 				async ({ workspaceId }) => {
 					const client = await lectorClient();
-					return client.call("workspace.revertMutation", { workspaceId, entryId });
+					return client.callOnce("workspace.revertMutation", { workspaceId, entryId });
 				},
 			);
 		},

package/extension/src/package-source-operations.ts

@@ -17,7 +17,7 @@ export function createLectorPackageSourceOperations(): PackageSourceOperations {
 	return {
 		async resolve(directory, name, requestedVersion, registry) {
 			const client = await lectorClient();
-			return client.call("package.resolveSource", {
+			return client.callOnce("package.resolveSource", {
 				request: {
 					projectRoot: directory,
 					coordinate: { ecosystem: "npm", registry, name, requestedVersion },
@@ -31,11 +31,11 @@ export function createLectorPackageSourceOperations(): PackageSourceOperations {
 		},
 		async remove(ecosystem, registry, name, resolvedVersion) {
 			const client = await lectorClient();
-			return client.call("package.removeSource", { ecosystem, registry, name, resolvedVersion });
+			return client.callOnce("package.removeSource", { ecosystem, registry, name, resolvedVersion });
 		},
 		async clean(ecosystem) {
 			const client = await lectorClient();
-			return client.call("package.cleanSources", { ecosystem });
+			return client.callOnce("package.cleanSources", { ecosystem });
 		},
 	};
 }

package/extension/src/reference-based-rename-operations.ts

@@ -18,7 +18,7 @@ export function createReferenceBasedRenameOperations(): ReferenceBasedRenameOper
 				() => workspaceForCodeIntelligencePath(fromPath),
 				async ({ workspaceId }) => {
 					const client = await lectorClient();
-					return client.call("workspace.referenceBasedRename", { workspaceId, fromPath, toPath, maxFiles, maxSymbolsPerFile });
+					return client.callOnce("workspace.referenceBasedRename", { workspaceId, fromPath, toPath, maxFiles, maxSymbolsPerFile });
 				},
 			);
 		},

package/extension/src/rename-operations.ts

@@ -28,7 +28,7 @@ export function createRenameOperations(): RenameOperations {
 				() => workspaceForCodeIntelligencePath(path),
 				async ({ workspaceId }) => {
 					const client = await lectorClient();
-					return client.call("workspace.rename", { workspaceId, path, line, character, newName });
+					return client.callOnce("workspace.rename", { workspaceId, path, line, character, newName });
 				},
 			);
 		},

package/extension/src/repo-cache-evict-operations.ts

@@ -13,7 +13,7 @@ export function createRepoCacheEvictOperations(): RepoCacheEvictOperations {
 	return {
 		async evict(host, owner, repo, ref) {
 			const client = await lectorClient();
-			return client.call("repo.evictCache", { host, owner, repo, ref });
+			return client.callOnce("repo.evictCache", { host, owner, repo, ref });
 		},
 	};
 }

package/extension/src/repo-fetch-operations.ts

@@ -14,7 +14,7 @@ export function createLectorRepoFetchOperations(): RepoFetchOperations {
 	return {
 		async fetch(host, owner, repo, ref, forceRefresh) {
 			const client = await lectorClient();
-			return client.call("repo.fetch", { host, owner, repo, ref, forceRefresh });
+			return client.callOnce("repo.fetch", { host, owner, repo, ref, forceRefresh });
 		},
 	};
 }

package/extension/src/symbol-annotation-operations.ts

@@ -49,7 +49,7 @@ export function createLectorSymbolAnnotationOperations(): SymbolAnnotationOperat
 				() => workspaceForCodeIntelligencePath(path),
 				async ({ workspaceId }) => {
 					const client = await lectorClient();
-					return client.call("workspace.createAnnotation", { workspaceId, subtype, title, body, anchors });
+					return client.callOnce("workspace.createAnnotation", { workspaceId, subtype, title, body, anchors });
 				},
 			);
 		},
@@ -82,7 +82,7 @@ export function createLectorSymbolAnnotationOperations(): SymbolAnnotationOperat
 				() => workspaceForCodeIntelligencePath(path),
 				async ({ workspaceId }) => {
 					const client = await lectorClient();
-					return client.call("workspace.refreshAnnotation", { workspaceId, id, subtype, title, body, anchors });
+					return client.callOnce("workspace.refreshAnnotation", { workspaceId, id, subtype, title, body, anchors });
 				},
 			);
 		},
@@ -91,7 +91,7 @@ export function createLectorSymbolAnnotationOperations(): SymbolAnnotationOperat
 				() => workspaceForCodeIntelligencePath(path),
 				async ({ workspaceId }) => {
 					const client = await lectorClient();
-					return client.call("workspace.scrubAnnotation", { workspaceId, id });
+					return client.callOnce("workspace.scrubAnnotation", { workspaceId, id });
 				},
 			);
 		},
@@ -100,7 +100,7 @@ export function createLectorSymbolAnnotationOperations(): SymbolAnnotationOperat
 				() => workspaceForCodeIntelligencePath(path),
 				async ({ workspaceId }) => {
 					const client = await lectorClient();
-					return client.call("workspace.restoreAnnotation", { workspaceId, id });
+					return client.callOnce("workspace.restoreAnnotation", { workspaceId, id });
 				},
 			);
 		},
@@ -109,7 +109,7 @@ export function createLectorSymbolAnnotationOperations(): SymbolAnnotationOperat
 				() => workspaceForCodeIntelligencePath(path),
 				async ({ workspaceId }) => {
 					const client = await lectorClient();
-					return client.call("workspace.containAnnotation", { workspaceId, parentId, childId });
+					return client.callOnce("workspace.containAnnotation", { workspaceId, parentId, childId });
 				},
 			);
 		},
@@ -118,7 +118,7 @@ export function createLectorSymbolAnnotationOperations(): SymbolAnnotationOperat
 				() => workspaceForCodeIntelligencePath(path),
 				async ({ workspaceId }) => {
 					const client = await lectorClient();
-					return client.call("workspace.uncontainAnnotation", { workspaceId, parentId, childId });
+					return client.callOnce("workspace.uncontainAnnotation", { workspaceId, parentId, childId });
 				},
 			);
 		},

package/extension/src/workspace-cache-operations.ts

@@ -23,7 +23,7 @@ export function createWorkspaceCacheOperations(): WorkspaceCacheOperations {
 				() => workspaceForDirectory(directory),
 				async ({ workspaceId }) => {
 					const client = await lectorClient();
-					const { job } = await client.call("job.submit", {
+					const { job } = await client.callOnce("job.submit", {
 						operation: "workspace.populateSymbolGraph",
 						input: { workspaceId, maxFiles, maxSymbolsPerFile },
 						waitMs: 0,

package/extension/src/write-operations.ts

@@ -41,7 +41,7 @@ export function createLectorWriteOperations(): WriteOperations {
 
 					for (let attempt = 0; attempt < MAX_STALE_HASH_RETRIES; attempt++) {
 						try {
-							await client.call("workspace.exactEdit", { workspaceId, path: relativePath, expectedHash, content });
+							await client.callOnce("workspace.exactEdit", { workspaceId, path: relativePath, expectedHash, content });
 							return;
 						} catch (error) {
 							if (!remoteErrorIs(error, "StaleExpectedHash") || attempt === MAX_STALE_HASH_RETRIES - 1) throw error;

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@danypops/pi-lector",
-	"version": "0.9.3",
+	"version": "0.9.4",
 	"description": "Pi host adapter for Lector: overrides read/write/edit with a daemon-backed, hash-guarded filesystem",
 	"license": "MIT",
 	"type": "module",
@@ -18,7 +18,7 @@
 		"typebox": "*"
 	},
 	"dependencies": {
-		"@danypops/vehicle-client": "^0.1.1",
+		"@danypops/vehicle-client": "^0.2.0",
 		"@danypops/lector": "^0.12.0",
 		"malevich-tui-components": "^0.19.0"
 	},