pi-roundtable-web 0.7.4 → 0.7.5

package/CHANGELOG.md

@@ -5,6 +5,10 @@ The format follows [Keep a Changelog](https://keepachangelog.com/en/1.1.0/).
 
 ## [Unreleased]
 
+## [0.7.5] - 2026-10-03
+
+- Show a skill of any size, and add `skills.errorDetail` to show why one cannot be read.
+
 ## [0.7.4] - 2026-10-03
 
 - Show a `channel name unavailable` detail for a channel Discord cannot name.

package/README.md

@@ -113,7 +113,7 @@ The host wrapper must declare any additional required service keys in its plugin
 | `party` | `contains(key)` separates party channels from owner transcripts; `list()` returns `PartyView[]` with channel, profile, enabler, enabled time, container state, busy count and optional last activity. |
 | `schedules` | `count(key)` returns the workspace schedule count. |
 | `cleanup` | `startFresh(key)` archives/waits for active turns and returns its kind; `deleteConversation(key)` returns `deleted` or `busy`. The package validates key shape, ownership, existence and exclusions before calling either hook. |
-| `skills` | `catalog()` supplies public `SkillView[]`; `read(name)` reads only that catalog entry and returns `{ frontmatter, body }`. The host bounds disk reads; the API caps body at 1 MB and frontmatter at 64 KB and flattens metadata in source order. |
+| `skills` | `catalog()` supplies public `SkillView[]`; `read(name)` reads only that catalog entry and returns `{ frontmatter, body }`. The host bounds disk reads and the API shows what `read` returns, flattening metadata in source order. A skill that cannot be read answers with a fixed message; set `errorDetail: true` to append the catalog's `missing` text or the read error's message (cut at 300 characters) when the host's reasons are safe to show. |
 | `connectors` | `gateways()`, `servers()`, `usedBy(server)` provide status/tools/usage. Optional `adminUrl` accepts HTTP(S) or an absolute local path, never script URLs. |
 
 `sessionSummary(directory)` is a public helper for a trusted host to summarize separately stored party session files.

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "pi-roundtable-web",
-	"version": "0.7.4",
+	"version": "0.7.5",
 	"description": "An owner-only web console for pi-roundtable: conversations, transcripts, and memory notes, with live updates",
 	"type": "module",
 	"license": "MIT",
@@ -47,7 +47,7 @@
 		"@types/bun": "1.4.2",
 		"@types/react": "19.3.0",
 		"@types/react-dom": "19.3.0",
-		"pi-roundtable": "0.7.4",
+		"pi-roundtable": "0.7.5",
 		"react": "19.3.0",
 		"react-dom": "19.3.0",
 		"react-markdown": "10.1.0",

package/src/console-api.ts

@@ -108,7 +108,7 @@ export class ConsoleApi {
 			if (error instanceof HttpError)
 				return json(
 					{
-						error: message(this.#ports.presentation, error.message),
+						error: `${message(this.#ports.presentation, error.message)}${error.detail ?? ""}`,
 					} satisfies ApiError,
 					error.status,
 				);

package/src/features.ts

@@ -21,6 +21,12 @@ export interface ConsoleFeatures {
 		read(
 			name: string,
 		): Promise<{ frontmatter: Record<string, unknown>; body: string }>;
+		/**
+		 * Show why a skill could not be read (the catalog's `missing` text, or the read error's
+		 * message, cut at 300 characters) after the fixed message. Off by default: the reason may
+		 * name a path.
+		 */
+		errorDetail?: boolean;
 	};
 	connectors?: {
 		gateways(): Promise<ConnectorsView["gateways"]>;

package/src/http.ts

@@ -3,6 +3,8 @@ export class ConsoleHttpError extends Error {
 	constructor(
 		readonly status: number,
 		message: string,
+		/** A reason a host opted into showing; appended after the translated message. */
+		readonly detail?: string,
 	) {
 		super(message);
 	}

package/src/skills-api.ts

@@ -25,14 +25,13 @@ export async function skillDetail(
 ): Promise<SkillDetailView> {
 	const entry = skills.catalog().find((s) => s.name === name);
 	if (!entry) throw new HttpError(404, "There is no such skill.");
-	if (entry.missing) throw new HttpError(404, "The skill file is missing.");
+	if (entry.missing)
+		throw skills.errorDetail
+			? new HttpError(404, "The skill file is missing: ", entry.missing)
+			: new HttpError(404, "The skill file is missing.");
 	try {
+		// The host's `read` enforces any size limit; the console shows what it returns.
 		const parsed = await skills.read(name);
-		if (
-			Buffer.byteLength(parsed.body) > 1024 * 1024 ||
-			Buffer.byteLength(JSON.stringify(parsed.frontmatter)) > 64 * 1024
-		)
-			throw new HttpError(413, "The skill file is too large.");
 		return {
 			name,
 			body: parsed.body,
@@ -40,7 +39,16 @@ export async function skillDetail(
 		};
 	} catch (error) {
 		if (error instanceof HttpError) throw error;
-		throw new HttpError(422, "The skill frontmatter could not be read.");
+		throw skills.errorDetail
+			? new HttpError(
+					422,
+					"The skill frontmatter could not be read: ",
+					(error instanceof Error ? error.message : String(error)).slice(
+						0,
+						300,
+					),
+				)
+			: new HttpError(422, "The skill frontmatter could not be read.");
 	}
 }