@averyyy/pi-client 0.85.1-piclient.1 → 0.85.1-piclient.3

package/CHANGELOG.md

@@ -3,6 +3,8 @@
 ### Fixed
 
 - Fixed automatic update checks to compare the installed `pi-client` version instead of the upstream `pi` version.
+- Used `pi-client` in fork help and documented `web`, `send`, server configuration, global package updates, and `/reload` without changing upstream Pi help.
+- Reported upload HTTP and connection failures with the target, response metadata, and a bounded, token-redacted body excerpt instead of JSON or network stack traces.
 
 ### Added
 

package/bin/send.js

@@ -2,6 +2,8 @@ import { lstatSync, readdirSync, readFileSync } from "node:fs";
 import { basename, join, relative, resolve, sep } from "node:path";
 import { ChunkRequest } from "@earendil-works/pi-coding-agent/pi-server-request";
 
+const MAX_ERROR_BODY_CHARS = 300;
+
 function addDirectoryEntries(root, directory, entries) {
 	const path = relative(root, directory).split(sep).join("/");
 	entries.push({ path, type: "directory" });
@@ -34,16 +36,85 @@ export async function runPiClientSend(args) {
 		console.error("Usage: pi-client send /path/to/file-or-folder");
 		return 1;
 	}
+	const serverUrl = process.env.PI_SERVER_URL ?? "http://127.0.0.1:4217";
+	const authToken = process.env.PI_SERVER_AUTH_TOKEN ?? "";
 	const request = new ChunkRequest({
-		serverUrl: process.env.PI_SERVER_URL ?? "http://127.0.0.1:4217",
-		authToken: process.env.PI_SERVER_AUTH_TOKEN ?? "",
+		serverUrl,
+		authToken,
 	});
-	const response = await request.postJson("/api/receive", createUploadBody(args[0]));
-	const body = await response.json();
+	let response;
+	try {
+		response = await request.postJson("/api/receive", createUploadBody(args[0]));
+	} catch (error) {
+		console.error(formatRequestFailure(serverUrl, error, authToken));
+		return 1;
+	}
+
+	let bodyText;
+	try {
+		bodyText = await response.text();
+	} catch (error) {
+		console.error(formatResponseFailure(serverUrl, response, error, authToken, "response body read failed"));
+		return 1;
+	}
 	if (!response.ok) {
-		console.error(`pi-client send failed (${response.status}): ${body.error ?? response.statusText}`);
+		console.error(formatResponseFailure(serverUrl, response, bodyText, authToken));
+		return 1;
+	}
+
+	let body;
+	try {
+		body = JSON.parse(bodyText);
+	} catch {
+		console.error(formatResponseFailure(serverUrl, response, bodyText, authToken, "expected JSON upload response"));
+		return 1;
+	}
+	if (!body || typeof body !== "object" || typeof body.path !== "string") {
+		console.error(
+			formatResponseFailure(
+				serverUrl,
+				response,
+				bodyText,
+				authToken,
+				"invalid upload payload; expected JSON response with a string path",
+			),
+		);
 		return 1;
 	}
 	console.log(`Saved to ${body.path}`);
 	return 0;
 }
+
+function formatResponseFailure(serverUrl, response, body, authToken, detail) {
+	const status = response.statusText ? `${response.status} ${response.statusText}` : String(response.status);
+	const contentType = response.headers.get("content-type") ?? "unknown";
+	const detailText = detail ? `; ${detail}` : "";
+	return `pi-client send failed for ${formatTarget(serverUrl)}: status ${status}; content-type: ${contentType}${detailText}; body excerpt: ${formatBodyExcerpt(body, authToken)}`;
+}
+
+function formatRequestFailure(serverUrl, error, authToken) {
+	const message = error instanceof Error ? error.message : String(error);
+	return `pi-client send failed for ${formatTarget(serverUrl)}: status unavailable; content-type: unavailable; body excerpt: ${formatBodyExcerpt(message, authToken)}`;
+}
+
+function formatTarget(serverUrl) {
+	try {
+		const target = new URL(`${serverUrl}/api/receive`);
+		target.username = "";
+		target.password = "";
+		target.search = "";
+		target.hash = "";
+		return target.toString();
+	} catch {
+		return "/api/receive";
+	}
+}
+
+function formatBodyExcerpt(body, authToken) {
+	const compacted = String(body ?? "")
+		.replace(/\s+/g, " ")
+		.trim();
+	const redacted = authToken ? compacted.replaceAll(authToken, "[redacted]") : compacted;
+	if (redacted.length <= MAX_ERROR_BODY_CHARS) return redacted || "(empty)";
+	return `${redacted.slice(0, MAX_ERROR_BODY_CHARS)}...`;
+}

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@averyyy/pi-client",
-	"version": "0.85.1-piclient.1",
+	"version": "0.85.1-piclient.3",
 	"description": "Lightweight CLI wrapper that connects to a pi-server instance",
 	"type": "module",
 	"piClient": {
@@ -25,7 +25,7 @@
 		"prepublishOnly": "npm run build"
 	},
 	"dependencies": {
-		"@earendil-works/pi-coding-agent": "npm:@averyyy/pi-coding-agent@0.85.1-piclient.1"
+		"@earendil-works/pi-coding-agent": "npm:@averyyy/pi-coding-agent@0.85.1-piclient.3"
 	},
 	"devDependencies": {
 		"shx": "0.4.0",