@danypops/pi-lector 0.9.5 → 0.10.0

package/extension/src/{package-source-rendering.ts → package-source/rendering.ts}

@@ -1,7 +1,7 @@
 import type { PackageSourceListEntry, PackageSourceOperationResult } from "@danypops/lector";
 import { keyHint } from "@earendil-works/pi-coding-agent";
 import { renderTruncatedList, type TableColumn } from "malevich-tui-components";
-import type { LectorTheme } from "./lector-tui-theme.ts";
+import type { LectorTheme } from "../lector-tui-theme.ts";
 
 const DEFAULT_VISIBLE_CANDIDATES = 5;
 
@@ -36,33 +36,40 @@ export function formatPackageSourceCall(
 export function formatPackageSourceResult(result: PackageSourceOperationResult | undefined, expanded: boolean, theme: LectorTheme): string {
 	if (!result) return theme.fg("dim", "No package-source result.");
 	const { outcome } = result;
-	if (outcome.status === "verified") {
-		return [
-			`${theme.fg("accent", result.workspaceId ?? "unregistered")} ${theme.fg("success", `${outcome.coordinate.name}@${outcome.coordinate.resolvedVersion}`)}`,
-			`${outcome.workspace.cachePath}`,
-			`${outcome.repository.url ?? "local source"}@${outcome.repository.resolvedRef ?? "local"} ${outcome.repository.commit ?? outcome.verification.integrity}`,
-		].join("\n");
+	switch (outcome.status) {
+		case "verified":
+			return [
+				`${theme.fg("accent", result.workspaceId ?? "unregistered")} ${theme.fg("success", `${outcome.coordinate.name}@${outcome.coordinate.resolvedVersion}`)}`,
+				`${outcome.workspace.cachePath}`,
+				`${outcome.repository.url ?? "local source"}@${outcome.repository.resolvedRef ?? "local"} ${outcome.repository.commit ?? outcome.verification.integrity}`,
+			].join("\n");
+		case "ambiguous": {
+			const lines = [
+				theme.fg("warning", `Ambiguous package source (${outcome.code})`),
+				...renderTruncatedList({
+					items: outcome.candidates,
+					expanded,
+					visibleCount: DEFAULT_VISIBLE_CANDIDATES,
+					formatItem: (candidate) => `${candidate.version} -- ${candidate.source}`,
+					moreLine: (hidden) => theme.fg("dim", `… ${hidden} more`),
+					truncationWarning: outcome.truncated ? theme.fg("dim", "More candidates were truncated by the daemon.") : undefined,
+				}),
+			];
+			return lines.join("\n");
+		}
+		case "unauthenticated":
+			return theme.fg("warning", `Authentication required (${outcome.code}): configure ${outcome.requiredCredentialNames.join(", ")}`);
+		case "oversized":
+			return theme.fg("warning", `Source resolution exceeded ${outcome.resource} limit ${outcome.limit}.`);
+		case "mismatched":
+			return theme.fg("error", `Source mismatch (${outcome.code}): expected ${outcome.expected}, got ${outcome.actual}.`);
+		case "unavailable":
+			return theme.fg("warning", `Source unavailable (${outcome.code}).`);
+		default: {
+			const exhaustive: never = outcome;
+			throw new Error(`unhandled package source outcome status: ${JSON.stringify(exhaustive)}`);
+		}
 	}
-	if (outcome.status === "ambiguous") {
-		const lines = [
-			theme.fg("warning", `Ambiguous package source (${outcome.code})`),
-			...renderTruncatedList({
-				items: outcome.candidates,
-				expanded,
-				visibleCount: DEFAULT_VISIBLE_CANDIDATES,
-				formatItem: (candidate) => `${candidate.version} -- ${candidate.source}`,
-				moreLine: (hidden) => theme.fg("dim", `… ${hidden} more`),
-				truncationWarning: outcome.truncated ? theme.fg("dim", "More candidates were truncated by the daemon.") : undefined,
-			}),
-		];
-		return lines.join("\n");
-	}
-	if (outcome.status === "unauthenticated") {
-		return theme.fg("warning", `Authentication required (${outcome.code}): configure ${outcome.requiredCredentialNames.join(", ")}`);
-	}
-	if (outcome.status === "oversized") return theme.fg("warning", `Source resolution exceeded ${outcome.resource} limit ${outcome.limit}.`);
-	if (outcome.status === "mismatched") return theme.fg("error", `Source mismatch (${outcome.code}): expected ${outcome.expected}, got ${outcome.actual}.`);
-	return theme.fg("warning", `Source unavailable (${outcome.code}).`);
 }
 
 /** Empty-state fallback only -- a non-empty page renders as a real Table (see buildPackageSourceListTableRows) so the human channel actually shows what's resolved, not just a bare count. Mirrors formatRepoCacheListResult. */
@@ -74,7 +81,7 @@ export function formatPackageSourceListResult(
 	return count === 0 ? theme.fg("dim", "no resolved package sources") : theme.fg("success", `${count} resolved package source${count === 1 ? "" : "s"}`);
 }
 
-/** Powers of 1024, one decimal past the first. Mirrors repo-cache-rendering.ts's own formatCacheSize -- kept as a small local duplicate rather than a shared export, matching that file's own precedent. */
+/** Powers of 1024, one decimal past the first. Mirrors repo-cache/rendering.ts's own formatCacheSize -- kept as a small local duplicate rather than a shared export, matching that file's own precedent. */
 function formatCacheSize(bytes: number): string {
 	const units = ["B", "KB", "MB", "GB", "TB"] as const;
 	let value = bytes;

package/extension/src/{read-operations.ts → read/operations.ts}

@@ -1,8 +1,8 @@
 import { constants } from "node:fs";
 import { access as fsAccess, readFile as fsReadFile } from "node:fs/promises";
 import type { ReadOperations } from "@earendil-works/pi-coding-agent";
-import { lectorClient, withWorkspace, workspaceForPath } from "./lector-client.ts";
-import { toWorkspaceRelativePath } from "./workspace-relative-path.ts";
+import { lectorClient, withWorkspace, workspaceForPath } from "../lector-client.ts";
+import { toWorkspaceRelativePath } from "../workspace-relative-path.ts";
 
 /**
  * Lector's core domain (RawRead/ExpectedHashEdit) is deliberately text-only

package/extension/src/{reference-based-rename-operations.ts → reference-based-rename/operations.ts}

@@ -1,5 +1,5 @@
 import type { OperationOutputs } from "@danypops/lector";
-import { lectorClient, withWorkspace, workspaceForCodeIntelligencePath } from "./lector-client.ts";
+import { lectorClient, withWorkspace, workspaceForCodeIntelligencePath } from "../lector-client.ts";
 
 /**
  * Thin wrapper over Lector's non-LSP reference-based rename: moves a file and rewrites every

package/extension/src/{rename-operations.ts → rename/operations.ts}

@@ -1,5 +1,5 @@
 import type { OperationOutputs } from "@danypops/lector";
-import { lectorClient, withWorkspace, workspaceForCodeIntelligencePath } from "./lector-client.ts";
+import { lectorClient, withWorkspace, workspaceForCodeIntelligencePath } from "../lector-client.ts";
 
 /**
  * Thin wrappers over Lector's LSP-driven prepareRename/rename -- position-based (path + 1-indexed

package/extension/src/{repo-cache-evict-operations.ts → repo-cache/evict-operations.ts}

@@ -1,8 +1,8 @@
-import { lectorClient } from "./lector-client.ts";
+import { lectorClient } from "../lector-client.ts";
 
 /**
  * Thin wrapper over repo.evictCache -- no `directory`/workspaceForDirectory resolution (matching
- * repo-fetch-operations.ts and repo-cache-list-operations.ts: this targets the daemon-wide fetch
+ * repo-fetch/operations.ts and repo-cache/list-operations.ts: this targets the daemon-wide fetch
  * cache, not a workspace-scoped concept).
  */
 export interface RepoCacheEvictOperations {

package/extension/src/{repo-cache-list-operations.ts → repo-cache/list-operations.ts}

@@ -1,9 +1,9 @@
 import type { CachedRepositoryPage } from "@danypops/lector";
-import { lectorClient } from "./lector-client.ts";
+import { lectorClient } from "../lector-client.ts";
 
 /**
  * Thin wrapper over repo.listCache -- no network, no cache mutation, no `directory`/
- * workspaceForDirectory resolution (matching repo-fetch-operations.ts: this queries the
+ * workspaceForDirectory resolution (matching repo-fetch/operations.ts: this queries the
  * daemon-wide fetch cache, not a workspace-scoped concept).
  */
 export interface RepoCacheListOperations {

package/extension/src/{repo-cache-rendering.ts → repo-cache/rendering.ts}

@@ -1,7 +1,7 @@
 import type { CachedRepositoryEntry, CachedRepositoryPage, RepoFetchResult } from "@danypops/lector";
 import { keyHint } from "@earendil-works/pi-coding-agent";
 import type { TableColumn } from "malevich-tui-components";
-import type { LectorTheme } from "./lector-tui-theme.ts";
+import type { LectorTheme } from "../lector-tui-theme.ts";
 
 /** Table has no row-count bound of its own; a cache can grow arbitrarily large even though repo_cache's own `maxResults` bounds any one page, so the display itself still needs a cap independent of that. */
 export const REPO_CACHE_VISIBLE_ROWS = 20;

package/extension/src/{repo-fetch-operations.ts → repo-fetch/operations.ts}

@@ -1,5 +1,5 @@
 import type { RepoFetchResult } from "@danypops/lector";
-import { lectorClient } from "./lector-client.ts";
+import { lectorClient } from "../lector-client.ts";
 
 /**
  * Thin wrapper over repo.fetch. No `directory`/workspaceForDirectory resolution here -- unlike

package/extension/src/{search-operations.ts → search/operations.ts}

@@ -1,5 +1,5 @@
 import type { TextSearchResult } from "@danypops/lector";
-import { lectorClient, withWorkspace, workspaceForDirectory } from "./lector-client.ts";
+import { lectorClient, withWorkspace, workspaceForDirectory } from "../lector-client.ts";
 
 /**
  * Thin wrapper over workspace.searchText. `directory` is required, same convention as

package/extension/src/{search-rendering.ts → search/rendering.ts}

@@ -1,7 +1,7 @@
 import type { TextSearchResult } from "@danypops/lector";
 import { keyHint } from "@earendil-works/pi-coding-agent";
 import { renderTruncatedList } from "malevich-tui-components";
-import type { LectorTheme } from "./lector-tui-theme.ts";
+import type { LectorTheme } from "../lector-tui-theme.ts";
 
 const DEFAULT_VISIBLE_MATCHES = 20;
 

package/extension/src/{symbol-annotation-operations.ts → symbol-annotation/operations.ts}

@@ -1,5 +1,5 @@
 import type { OperationInputs, OperationOutputs } from "@danypops/lector";
-import { lectorClient, withWorkspace, workspaceForCodeIntelligencePath } from "./lector-client.ts";
+import { lectorClient, withWorkspace, workspaceForCodeIntelligencePath } from "../lector-client.ts";
 
 /** A bare symbol position an anchor is given as -- symbolNodeId and the anchor's baseline file hash are derived server-side, never supplied by the caller. */
 export interface AnnotationAnchorInput {

package/extension/src/{symbol-annotation-rendering.ts → symbol-annotation/rendering.ts}

@@ -1,5 +1,5 @@
 import type { SymbolAnnotation } from "@danypops/lector";
-import type { LectorTheme } from "./lector-tui-theme.ts";
+import type { LectorTheme } from "../lector-tui-theme.ts";
 
 const STATUS_COLOR: Record<SymbolAnnotation["status"], "success" | "warning" | "dim"> = {
 	fresh: "success",

package/extension/src/{workspace-cache-operations.ts → workspace-cache/operations.ts}

@@ -1,5 +1,5 @@
 import type { JobSnapshot, PopulateSymbolGraphResult, WorkspaceCacheStatus } from "@danypops/lector";
-import { lectorClient, withWorkspace, workspaceForDirectory } from "./lector-client.ts";
+import { lectorClient, withWorkspace, workspaceForDirectory } from "../lector-client.ts";
 
 export interface WorkspaceCacheOperations {
 	status(directory: string, maxFiles: number, maxSymbolsPerFile: number): Promise<WorkspaceCacheStatus>;

package/extension/src/{workspace-cache-rendering.ts → workspace-cache/rendering.ts}

@@ -1,5 +1,5 @@
 import type { JobSnapshot, PopulateSymbolGraphResult, WorkspaceCacheStatus } from "@danypops/lector";
-import type { LectorTheme } from "./lector-tui-theme.ts";
+import type { LectorTheme } from "../lector-tui-theme.ts";
 
 type WorkspaceCacheAction = "status" | "populate" | "job_status";
 

package/extension/src/{write-operations.ts → write/operations.ts}

@@ -1,7 +1,7 @@
 import { type ContentHash, remoteErrorIs } from "@danypops/lector";
 import type { WriteOperations } from "@earendil-works/pi-coding-agent";
-import { lectorClient, withWorkspace, workspaceForPath } from "./lector-client.ts";
-import { toWorkspaceRelativePath } from "./workspace-relative-path.ts";
+import { lectorClient, withWorkspace, workspaceForPath } from "../lector-client.ts";
+import { toWorkspaceRelativePath } from "../workspace-relative-path.ts";
 
 const MAX_STALE_HASH_RETRIES = 3;
 

package/package.json

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

package/extension/src/cross-workspace-search-operations.ts

@@ -1,43 +0,0 @@
-import type { SymbolSearchResult, TextSearchResult, WorkspaceQueryOutcome } from "@danypops/lector";
-import { lectorClient, workspaceForDirectory } from "./lector-client.ts";
-
-/**
- * Fans out across explicitly-named directories only -- never the daemon's own "every registered
- * workspace" default. Lector's daemon is a shared, system-wide service: leaving workspaceIds
- * unset would search every project any other concurrent Pi session has ever registered against
- * it, not just this session's own (confirmed live, not assumed -- a real fetched jittor workspace
- * from an unrelated session showed up in an early test of this exact feature). `directories` is
- * required, same "no implicit fallback" convention as find_symbols/search_code.
- */
-export interface CrossWorkspaceSearchOperations {
-	findSymbols(query: string, directories: readonly string[], timeoutMs?: number): Promise<readonly WorkspaceQueryOutcome<SymbolSearchResult>[]>;
-	searchText(
-		query: string,
-		directories: readonly string[],
-		maxMatches: number,
-		maxBytes: number,
-		timeoutMs?: number,
-	): Promise<readonly WorkspaceQueryOutcome<TextSearchResult>[]>;
-}
-
-async function resolveWorkspaceIds(directories: readonly string[]): Promise<readonly string[]> {
-	const resolved = await Promise.all(directories.map((directory) => workspaceForDirectory(directory)));
-	return resolved.map((r) => r.workspaceId);
-}
-
-export function createLectorCrossWorkspaceSearchOperations(): CrossWorkspaceSearchOperations {
-	return {
-		async findSymbols(query, directories, timeoutMs) {
-			const workspaceIds = await resolveWorkspaceIds(directories);
-			const client = await lectorClient();
-			const { results } = await client.call("search.symbols", { query, workspaceIds, timeoutMs });
-			return results;
-		},
-		async searchText(query, directories, maxMatches, maxBytes, timeoutMs) {
-			const workspaceIds = await resolveWorkspaceIds(directories);
-			const client = await lectorClient();
-			const { results } = await client.call("search.text", { query, maxMatches, maxBytes, workspaceIds, timeoutMs });
-			return results;
-		},
-	};
-}