pi-onedev-toolkit 0.3.0 → 0.4.0

package/CHANGELOG.md

@@ -1,5 +1,13 @@
 # Changelog
 
+## 0.4.0 — 2026-09-05
+
+### Changed
+
+- `issue`, `pull`, and `build` `list` actions with an explicit `project` now work even when the current repository has no OneDev remote: the query runs against a temporary scratch remote that is created and removed for the call.
+- Refreshed development dependencies (pi packages 0.85.1, typebox 1.3.27).
+
+
 ## 0.3.0 — 2026-09-05
 
 ### Changed

package/README.md

@@ -55,6 +55,7 @@ Attachment download is intentionally not exposed: current TOD releases may send
 
 - Update TOD regularly from its upstream install script. `get_unit_test_report` reports an actionable upgrade error when the installed TOD predates that command.
 - Plain references require the current repository to have a OneDev remote from which TOD can infer the project. The toolkit reports `no_project` rather than pretending a qualified reference can bypass TOD's resolver.
+- `issue`, `pull`, and `build` `list` calls with an explicit `project` work from any repository: the toolkit binds TOD to a throwaway Git remote for the query and removes it afterwards.
 - Tool calls use `snake_case`, for example `for_code_review`, `target_branch`, `report_name`, and `interval_seconds`.
 
 ## Skills

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "pi-onedev-toolkit",
-	"version": "0.3.0",
+	"version": "0.4.0",
 	"description": "Lightweight OneDev integration for Pi via TOD: repo-aware issue, pull request, build, and watch tools",
 	"type": "module",
 	"author": {
@@ -53,12 +53,12 @@
 		"typebox": "*"
 	},
 	"devDependencies": {
-		"@earendil-works/pi-ai": "0.85.0",
-		"@earendil-works/pi-server": "0.85.0",
-		"@earendil-works/pi-coding-agent": "0.85.0",
+		"@earendil-works/pi-ai": "0.85.1",
+		"@earendil-works/pi-coding-agent": "0.85.1",
+		"@earendil-works/pi-server": "0.85.1",
 		"@types/node": "^22.19.0",
 		"@types/proper-lockfile": "4.1.4",
-		"typebox": "1.3.25",
+		"typebox": "1.3.27",
 		"typescript": "^7.0.2",
 		"vitest": "^4.1.10"
 	},

package/src/tools/build.ts

@@ -275,6 +275,7 @@ export function registerBuildTool(pi: ExtensionAPI, deps: ToolDeps): void {
 			if (params.action === "list") {
 				output = await runListInContext(deps, args, {
 					signal,
+					project: params.project,
 					full: params.full,
 					maxOutputBytes: params.max_bytes,
 					digest: digestBuilds,

package/src/tools/common.ts

@@ -1,3 +1,6 @@
+import { mkdtemp, rm } from "node:fs/promises";
+import { tmpdir } from "node:os";
+import { join } from "node:path";
 import type { ExtensionContext } from "@earendil-works/pi-coding-agent";
 import { Type } from "typebox";
 import type {
@@ -285,40 +288,57 @@ export function digestProjects(text: string): string | undefined {
 }
 
 /**
- * Fetch a whole bounded list payload, then reduce it to a digest unless the
- * caller opted into raw output. The internal fetch budget exceeds the
- * model-visible cap on purpose: the digest, not the payload, reaches the
- * model.
+ * Run tod inside a throwaway Git repository whose only remote points at the
+ * requested OneDev project. TOD refuses `--project` queries when the working
+ * directory has no OneDev remote; the scratch repository supplies one
+ * without touching the user's checkout.
  */
-export async function runListInContext(
-	deps: ToolDeps,
+async function runTodWithScratchRemote(
+	exec: CommandExecutor,
 	args: readonly string[],
+	serverUrl: string,
+	project: string,
+	options: { signal?: AbortSignal; maxOutputBytes: number },
+): Promise<TodOutput> {
+	const directory = await mkdtemp(join(tmpdir(), "pi-onedev-scratch-"));
+	try {
+		for (const gitArgs of [
+			["init", "--quiet", directory],
+			[
+				"remote",
+				"add",
+				"origin",
+				`${serverUrl.replace(/\/$/, "")}/${project}.git`,
+			],
+		]) {
+			const done = await exec("git", gitArgs, { cwd: directory });
+			if (done.code !== 0 || done.killed) {
+				throw new TodError(
+					`could not prepare a temporary OneDev project context: ${(done.stderr || done.stdout).trim() || `git ${gitArgs[0] ?? ""} failed`}`,
+				);
+			}
+		}
+		return await runTod(exec, [...args, "--working-dir", directory], {
+			signal: options.signal,
+			maxOutputBytes: options.maxOutputBytes,
+		});
+	} finally {
+		await rm(directory, { recursive: true, force: true });
+	}
+}
+
+async function finishListOutput(
+	fetch: (maxOutputBytes: number) => Promise<TodOutput>,
 	options: {
-		signal?: AbortSignal;
 		full?: boolean;
 		maxOutputBytes?: number;
 		digest: (text: string) => string | undefined;
 	},
 ): Promise<TodOutput> {
-	const context = deps.context();
-	if (context.status !== "ready" || !context.project) {
-		throw new TodError(
-			context.problem ??
-				"OneDev tools require a repository whose project can be inferred from a OneDev remote",
-		);
-	}
 	if (options.full === true) {
-		return runTod(deps.exec, args, {
-			cwd: context.cwd,
-			signal: options.signal,
-			maxOutputBytes: modelOutputLimit(options.maxOutputBytes),
-		});
+		return fetch(modelOutputLimit(options.maxOutputBytes));
 	}
-	const output = await runTod(deps.exec, args, {
-		cwd: context.cwd,
-		signal: options.signal,
-		maxOutputBytes: DIGEST_FETCH_BYTES,
-	});
+	const output = await fetch(DIGEST_FETCH_BYTES);
 	const limit = modelOutputLimit(options.maxOutputBytes);
 	const digested = options.digest(output.text);
 	if (
@@ -341,6 +361,61 @@ export async function runListInContext(
 	};
 }
 
+/**
+ * Fetch a whole bounded list payload, then reduce it to a digest unless the
+ * caller opted into raw output. The internal fetch budget exceeds the
+ * model-visible cap on purpose: the digest, not the payload, reaches the
+ * model. Without an inferred project, an explicit `project` list query runs
+ * against a temporary scratch remote instead of failing.
+ */
+export async function runListInContext(
+	deps: ToolDeps,
+	args: readonly string[],
+	options: {
+		signal?: AbortSignal;
+		full?: boolean;
+		maxOutputBytes?: number;
+		project?: string;
+		digest: (text: string) => string | undefined;
+	},
+): Promise<TodOutput> {
+	const context = deps.context();
+	if (context.status !== "ready" || !context.project) {
+		const scratchServer = context.serverUrl;
+		const scratchProject = options.project;
+		if (
+			context.status !== "no_project" ||
+			scratchServer === undefined ||
+			scratchProject === undefined
+		) {
+			throw new TodError(
+				context.problem ??
+					"OneDev tools require a repository whose project can be inferred from a OneDev remote",
+			);
+		}
+		return finishListOutput(
+			(maxOutputBytes) =>
+				runTodWithScratchRemote(
+					deps.exec,
+					args,
+					scratchServer,
+					scratchProject,
+					{ signal: options.signal, maxOutputBytes },
+				),
+			options,
+		);
+	}
+	return finishListOutput(
+		(maxOutputBytes) =>
+			runTod(deps.exec, args, {
+				cwd: context.cwd,
+				signal: options.signal,
+				maxOutputBytes,
+			}),
+		options,
+	);
+}
+
 export function confirmMutation(
 	deps: ToolDeps,
 	ctx: ExtensionContext,

package/src/tools/issue.ts

@@ -233,6 +233,7 @@ export function registerIssueTool(pi: ExtensionAPI, deps: ToolDeps): void {
 				params.action === "list"
 					? await runListInContext(deps, args, {
 							signal,
+							project: params.project,
 							full: params.full,
 							maxOutputBytes: params.max_bytes,
 							digest: digestIssues,

package/src/tools/pull.ts

@@ -308,6 +308,7 @@ export function registerPullTool(pi: ExtensionAPI, deps: ToolDeps): void {
 				params.action === "list"
 					? await runListInContext(deps, args, {
 							signal,
+							project: params.project,
 							full: params.full,
 							maxOutputBytes: params.max_bytes,
 							digest: digestPulls,