pi-roundtable 0.4.0 → 0.5.0

package/CHANGELOG.md

@@ -5,6 +5,18 @@ The format follows [Keep a Changelog](https://keepachangelog.com/en/1.1.0/).
 
 ## [Unreleased]
 
+## [0.5.0] - 2026-10-01
+
+### Changed
+
+- `@earendil-works/pi-ai` and `@earendil-works/pi-coding-agent` are no longer pinned to one version: they take `>=0.99.2 <1`, so a host picks up a newer model table with `bun update @earendil-works/pi-ai @earendil-works/pi-coding-agent`, or its own direct dependency, without waiting for a pi-roundtable release. Pinned to 0.87.1 before, the host's model list stopped at the models that version knew. 0.99.2 is the lowest version the tests run against, and the lockfile keeps CI on it, so a host that updates further runs on versions this package was not tested with.
+- `pi-mcp-adapter` is 4.0.0 (was 2.37.0). Version 2.37.0 declared support for pi-ai up to 0.87 only; 4.0.0 declares 0.99. It stays pinned. Checked by connecting a session built the way the core builds one to a local MCP server that requires a bearer token: the tool registered and a call returned its result.
+- A judge or agent model that the newer table lists but the host has no login for fails with the provider's "Provider is not configured" error, where an unlisted model fails with "is not available".
+
+### Fixed
+
+- Agents had no `skill_list` tool, although the `agent_create` description, the built-in `writing-skills` skill and the skill errors all tell them to call it. The `skills` addon now mounts it for agent sessions, at the member tier like its entry in the tool tiers. A host that mounted `skillListExtension` itself for agents (the kit still exports it) now offers the tool twice and should drop its own copy.
+
 ## [0.4.0] - 2026-10-01
 
 ### Added

package/README.zh-TW.md

@@ -1,6 +1,6 @@
 # pi-roundtable
 
-[English](README.md) | 繁體中文
+[English](./README.md) | 繁體中文
 
 一個建立在 [Pi](https://github.com/earendil-works/pi) 上的 Discord 智慧體（agent）伺服器。
 你會在一個 Discord 伺服器裡得到一組 AI 智慧體：每個智慧體擁有一個頻道和一段對話，彼此共用工具與記憶，你用 TypeScript 寫外掛（plugin）來擴充這個 bot。

package/docs/plugins.md

@@ -289,7 +289,7 @@ They are on by default, so a bot that says nothing about them has all three.
 | Addon | Plugin | Switch in `roundtable.config.ts` | What it adds | When it is off |
 |---|---|---|---|---|
 | Memory | `memory` (provides `MEMORY`) | `memory: false` | The memory table, the `memory_add`, `memory_search` and `memory_remove` tools, and the memory block of every system prompt | No memory tools and no block; the table is left as it is |
-| Skills | `skills` (provides `SKILLS`) | `skills: false` | The skill tables, the skill tools of agent sessions (`skill_link`, `skill_create`, `agent_skills`, and the rest), and the skills every agent carries, `writing-skills` included | No skill tools and no skills in any session; `agent_get` has no skills line; `agent_create` leaves out its `skills` parameter and refuses a call that passes some with `Skills are off on this host`; the tables are left as they are |
+| Skills | `skills` (provides `SKILLS`) | `skills: false` | The skill tables, the skill tools of agent sessions (`skill_list`, `skill_link`, `skill_create`, `agent_skills`, and the rest), and the skills every agent carries, `writing-skills` included | No skill tools and no skills in any session; `agent_get` has no skills line; `agent_create` leaves out its `skills` parameter and refuses a call that passes some with `Skills are off on this host`; the tables are left as they are |
 | Discord administration | `discord-admin` | `discord: { admin: false }` | The `discord_*` tools that read and manage the server, for the owner | No `discord_*` tools; the channel executor that remote MCP uses is the connection's, so it stays |
 
 A switched-off addon's tables and rows are never touched: turning it on again finds them as they were.

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "pi-roundtable",
-	"version": "0.4.0",
+	"version": "0.5.0",
 	"description": "A plugin-driven Pi agent server for Discord",
 	"type": "module",
 	"license": "MIT",
@@ -45,11 +45,11 @@
 	},
 	"dependencies": {
 		"@babel/parser": "7.29.9",
-		"@earendil-works/pi-ai": "0.87.1",
-		"@earendil-works/pi-coding-agent": "0.87.1",
+		"@earendil-works/pi-ai": ">=0.99.2 <1",
+		"@earendil-works/pi-coding-agent": ">=0.99.2 <1",
 		"canvas": "3.2.3",
 		"discord.js": "14.27.0",
-		"pi-mcp-adapter": "2.37.0",
+		"pi-mcp-adapter": "4.0.0",
 		"pino": "10.3.1",
 		"typebox": "1.3.34",
 		"unpdf": "1.8.1"

package/src/core/builtin/skills.ts

@@ -4,6 +4,7 @@ import { SkillStore } from "../modules/skills/skill-store.ts";
 import {
 	SKILL_LIST_TOOL,
 	SKILL_TOOLS,
+	skillListExtension,
 	skillToolsExtension,
 } from "../modules/skills/skill-tools.ts";
 import type { RoundtablePlugin } from "../plugin.ts";
@@ -57,6 +58,7 @@ export function skillsPlugin(
 			services.provide(SKILLS, built);
 			return {
 				sessionTools: [
+					agentOnly("skill-list", () => skillListExtension(built)),
 					agentOnly("skill-tools", () =>
 						skillToolsExtension(built, (name) => {
 							// The agent server is set up after this plugin and read when a session is built.
@@ -64,7 +66,10 @@ export function skillsPlugin(
 						}),
 					),
 				],
-				agentSelection: () => ({ tools: SKILL_TOOLS, groups: [] }),
+				agentSelection: () => ({
+					tools: [SKILL_LIST_TOOL, ...SKILL_TOOLS],
+					groups: [],
+				}),
 				toolTiers: SKILL_TIERS,
 			};
 		},

package/src/core/config/config.ts

@@ -27,7 +27,10 @@ import {
 /** How prompts refer back to the owner. */
 export type Pronouns = "he" | "she" | "they";
 
-/** Who holds a tier besides the owner: user ids, role ids, and for members everyone. */
+/**
+ * Who holds a tier besides the owner: user ids, role ids, or everyone. `everyone` works under any
+ * tier it is written in, so under `admins` it makes every author an admin.
+ */
 export interface TierConfig {
 	users?: readonly string[];
 	roles?: readonly string[];

package/src/core/speakers.ts

@@ -75,7 +75,10 @@ export interface SpeakerPolicy {
 	resolve(author: SpeakerFacts): Speaker | undefined;
 }
 
-/** Who holds one tier: user ids, role ids, and (for members) everyone. */
+/**
+ * Who holds one tier: user ids, role ids, or everyone. `everyone` works under any tier it is
+ * written in, so under `admins` it makes every author an admin.
+ */
 export interface TierMembers {
 	users?: readonly string[];
 	roles?: readonly string[];

package/src/core/tool-set.snapshot.json

@@ -102,6 +102,7 @@
 				"skill_group_delete",
 				"skill_group_set",
 				"skill_link",
+				"skill_list",
 				"skill_unlink",
 				"skill_update"
 			],
@@ -125,6 +126,7 @@
 				"skill_group_delete",
 				"skill_group_set",
 				"skill_link",
+				"skill_list",
 				"skill_unlink",
 				"skill_update",
 				"write"
@@ -187,6 +189,7 @@
 				"skill_group_delete",
 				"skill_group_set",
 				"skill_link",
+				"skill_list",
 				"skill_unlink",
 				"skill_update"
 			],
@@ -209,6 +212,7 @@
 				"skill_group_delete",
 				"skill_group_set",
 				"skill_link",
+				"skill_list",
 				"skill_unlink",
 				"skill_update",
 				"write"
@@ -280,6 +284,7 @@
 		"skill_group_delete": "admin",
 		"skill_group_set": "admin",
 		"skill_link": "admin",
+		"skill_list": "member",
 		"skill_unlink": "admin",
 		"skill_update": "admin",
 		"web_search": "member",

package/templates/official/codex-images/plugin.ts

@@ -35,7 +35,7 @@ export interface CodexImagesOptions {
 
 /** The ChatGPT account the login token belongs to, from the token's claims. */
 function chatGptAccountId(token: string): string {
-	const payload = token.split(".")[1];
+	const [, payload] = token.split(".");
 	if (!payload) throw new Error("the Codex token is not a JWT");
 	let claims: Record<string, unknown>;
 	try {
@@ -58,43 +58,64 @@ function object(value: unknown): Record<string, unknown> {
 		: {};
 }
 
-/** Reads the server-sent events until the response completes, and returns its one image. */
+/** The one image an `image_generation_call` item carries; anything else is not an image. */
+function imageOf(value: unknown): Uint8Array | undefined {
+	const item = object(value);
+	if (item.type !== "image_generation_call") return undefined;
+	if (
+		item.status !== "completed" ||
+		typeof item.result !== "string" ||
+		!item.result
+	) {
+		const { result: _result, ...detail } = item;
+		throw new ImageNotGeneratedError(
+			`Codex image generation did not complete (status ${String(item.status)}): ${JSON.stringify(detail).slice(0, 300)}`,
+		);
+	}
+	if (item.result.length > MAX_IMAGE_BASE64)
+		throw new Error("the image exceeds 32 MiB");
+	return new Uint8Array(Buffer.from(item.result, "base64"));
+}
+
+/** The frames of a server-sent event stream: the text between blank lines. */
+async function* frames(
+	body: ReadableStream<Uint8Array>,
+): AsyncGenerator<string> {
+	const decoder = new TextDecoder();
+	let buffer = "";
+	for await (const chunk of body) {
+		buffer += decoder.decode(chunk, { stream: true });
+		let match = /\r?\n\r?\n/.exec(buffer);
+		while (match) {
+			yield buffer.slice(0, match.index);
+			buffer = buffer.slice(match.index + match[0].length);
+			match = /\r?\n\r?\n/.exec(buffer);
+		}
+	}
+	if (buffer.trim()) yield buffer;
+}
+
+/** The event a frame carries, or undefined for a frame without data. */
+function eventOf(frame: string): Record<string, unknown> | undefined {
+	const data = frame
+		.split(/\r?\n/)
+		.flatMap((line) => (line.startsWith("data:") ? [line.slice(5).trim()] : []))
+		.join("\n");
+	if (!data || data === "[DONE]") return undefined;
+	try {
+		return object(JSON.parse(data));
+	} catch {
+		throw new Error(`Codex sent a malformed event: ${data.slice(0, 200)}`);
+	}
+}
+
+/** Reads the events until the response completes, and returns its one image. */
 async function parseImageStream(response: Response): Promise<Uint8Array> {
 	if (!response.body) throw new Error("Codex returned no body");
 	let image: Uint8Array | undefined;
-	let completed = false;
-	const take = (value: unknown) => {
-		const item = object(value);
-		if (item.type !== "image_generation_call" || image) return;
-		if (
-			item.status !== "completed" ||
-			typeof item.result !== "string" ||
-			!item.result
-		) {
-			const { result: _result, ...detail } = item;
-			throw new ImageNotGeneratedError(
-				`Codex image generation did not complete (status ${String(item.status)}): ${JSON.stringify(detail).slice(0, 300)}`,
-			);
-		}
-		if (item.result.length > MAX_IMAGE_BASE64)
-			throw new Error("the image exceeds 32 MiB");
-		image = new Uint8Array(Buffer.from(item.result, "base64"));
-	};
-	const handle = (frame: string) => {
-		const data = frame
-			.split(/\r?\n/)
-			.filter((line) => line.startsWith("data:"))
-			.map((line) => line.slice(5).trim())
-			.join("\n");
-		if (!data || data === "[DONE]") return;
-		let parsed: unknown;
-		try {
-			parsed = JSON.parse(data);
-		} catch {
-			throw new Error(`Codex sent a malformed event: ${data.slice(0, 200)}`);
-		}
-		const event = object(parsed);
-		switch (event.type) {
+	for await (const frame of frames(response.body)) {
+		const event = eventOf(frame);
+		switch (event?.type) {
 			case "error":
 			case "response.failed": {
 				const error = object(object(event.response).error ?? event.error);
@@ -105,34 +126,21 @@ async function parseImageStream(response: Response): Promise<Uint8Array> {
 			case "response.incomplete":
 				throw new Error("the Codex response was incomplete");
 			case "response.output_item.done":
-				take(event.item);
+				image ??= imageOf(event.item);
 				break;
 			case "response.completed": {
 				const output = object(event.response).output;
-				if (!image && Array.isArray(output))
-					for (const item of output) take(item);
-				completed = true;
-				break;
+				if (Array.isArray(output))
+					for (const item of output) image ??= imageOf(item);
+				if (!image)
+					throw new ImageNotGeneratedError("Codex answered without an image");
+				return image;
 			}
+			default:
+				break;
 		}
-	};
-	const decoder = new TextDecoder();
-	let buffer = "";
-	for await (const chunk of response.body) {
-		buffer += decoder.decode(chunk, { stream: true });
-		let match = /\r?\n\r?\n/.exec(buffer);
-		while (match && !completed) {
-			handle(buffer.slice(0, match.index));
-			buffer = buffer.slice(match.index + match[0].length);
-			match = /\r?\n\r?\n/.exec(buffer);
-		}
-		if (completed) break;
 	}
-	if (!completed && buffer.trim()) handle(buffer);
-	if (!completed) throw new Error("the Codex stream ended before completion");
-	if (!image)
-		throw new ImageNotGeneratedError("Codex answered without an image");
-	return image;
+	throw new Error("the Codex stream ended before completion");
 }
 
 async function generateImage(

package/templates/official/dice/plugin.ts

@@ -41,6 +41,8 @@ function removed(
 			return ascending.slice(0, n);
 		case "dh":
 			return ascending.slice(rolls.length - n);
+		default:
+			return rule satisfies never;
 	}
 }
 
@@ -51,49 +53,53 @@ interface Rolled {
 	dice: number;
 }
 
-function rollTerm(term: string, random: Random): Rolled {
-	if (/^\d+$/.test(term)) {
-		if (term.length > 7 || Number(term) > MAX_MODIFIER)
-			return refuse(`The number ${term} is over ${MAX_MODIFIER}.`);
-		return { shown: String(Number(term)), value: Number(term), dice: 0 };
-	}
-	const match = TERM.exec(term);
-	if (!match) return refuse(`"${term}" is not a dice term.`);
-	const [, countText, fate, sidesText, rule, nText] = match;
-	const count = countText ? Number(countText) : fate ? FATE_DEFAULT : 1;
-	if (count < 1) return refuse(`"${term}" rolls no dice.`);
-	if (count > MAX_DICE)
-		return refuse(`"${term}" rolls more than ${MAX_DICE} dice.`);
-	if (fate) {
-		const faces = Array.from(
-			{ length: count },
-			() => Math.floor(random() * 3) - 1,
-		);
-		const symbols = faces.map((face) =>
-			face > 0 ? "+" : face < 0 ? "-" : "0",
-		);
-		return {
-			shown: `[${symbols.join(", ")}]`,
-			value: faces.reduce((sum, face) => sum + face, 0),
-			dice: count,
-		};
-	}
-	const sides = Number(sidesText);
+/** A plain number term. */
+function rollNumber(term: string): Rolled {
+	if (term.length > 7 || Number(term) > MAX_MODIFIER)
+		return refuse(`The number ${term} is over ${MAX_MODIFIER}.`);
+	return { shown: String(Number(term)), value: Number(term), dice: 0 };
+}
+
+/** How a fate die shows: + for 1, - for -1, and 0. */
+function fateSymbol(face: number): string {
+	if (face > 0) return "+";
+	if (face < 0) return "-";
+	return "0";
+}
+
+/** `count` fate dice, each one of -1, 0, or +1. */
+function rollFate(count: number, random: Random): Rolled {
+	const faces = Array.from(
+		{ length: count },
+		() => Math.floor(random() * 3) - 1,
+	);
+	const symbols = faces.map(fateSymbol);
+	return {
+		shown: `[${symbols.join(", ")}]`,
+		value: faces.reduce((sum, face) => sum + face, 0),
+		dice: count,
+	};
+}
+
+/** `count` dice of `sides` sides, with an optional keep or drop rule that removes some of them. */
+function rollPool(
+	term: string,
+	count: number,
+	sides: number,
+	rule: { kind: KeepDrop; n: number } | undefined,
+	random: Random,
+): Rolled {
 	if (sides < 2 || sides > MAX_SIDES)
 		return refuse(`"${term}" needs between 2 and ${MAX_SIDES} sides.`);
 	const rolls = Array.from(
 		{ length: count },
 		() => Math.floor(random() * sides) + 1,
 	);
-	const dropped = new Set<number>();
-	if (rule) {
-		const n = Number(nText);
-		if (n < 1 || n >= count)
-			return refuse(
-				`"${term}" must keep or drop at least 1 and fewer than ${count} dice.`,
-			);
-		for (const index of removed(rolls, rule as KeepDrop, n)) dropped.add(index);
-	}
+	if (rule && (rule.n < 1 || rule.n >= count))
+		return refuse(
+			`"${term}" must keep or drop at least 1 and fewer than ${count} dice.`,
+		);
+	const dropped = new Set(rule ? removed(rolls, rule.kind, rule.n) : []);
 	const shown = rolls.map((value, index) =>
 		dropped.has(index) ? `(${value})` : String(value),
 	);
@@ -107,6 +113,31 @@ function rollTerm(term: string, random: Random): Rolled {
 	};
 }
 
+/** How many dice a term rolls: the count it writes, else 4 for fate dice and 1 for the rest. */
+function diceCount(countText: string | undefined, fate: string | undefined) {
+	if (countText) return Number(countText);
+	return fate ? FATE_DEFAULT : 1;
+}
+
+function rollTerm(term: string, random: Random): Rolled {
+	if (/^\d+$/.test(term)) return rollNumber(term);
+	const match = TERM.exec(term);
+	if (!match) return refuse(`"${term}" is not a dice term.`);
+	const [, countText, fate, sidesText, rule, nText] = match;
+	const count = diceCount(countText, fate);
+	if (count < 1) return refuse(`"${term}" rolls no dice.`);
+	if (count > MAX_DICE)
+		return refuse(`"${term}" rolls more than ${MAX_DICE} dice.`);
+	if (fate) return rollFate(count, random);
+	return rollPool(
+		term,
+		count,
+		Number(sidesText),
+		rule ? { kind: rule as KeepDrop, n: Number(nText) } : undefined,
+		random,
+	);
+}
+
 /**
  * Rolls an expression of dice terms and numbers joined by `+` and `-`, and writes the result as
  * text such as `2d6+3: [3, 5] + 3 = 11`. A term is `NdS`, an optional keep or drop rule
@@ -133,10 +164,8 @@ export function roll(expression: string, random: Random = Math.random): string {
 		if (dice > MAX_DICE)
 			return refuse(`The expression rolls more than ${MAX_DICE} dice in all.`);
 		total += sign === "-" ? -rolled.value : rolled.value;
-		line +=
-			index === 0
-				? `${sign === "-" ? "-" : ""}${rolled.shown}`
-				: ` ${sign} ${rolled.shown}`;
+		const lead = sign === "-" ? "-" : "";
+		line += index === 0 ? `${lead}${rolled.shown}` : ` ${sign} ${rolled.shown}`;
 	}
 	return `${compact}: ${line} = ${total}`;
 }