pi-roundtable 0.5.0 → 0.5.2

package/examples/migrations.test.ts

@@ -13,7 +13,7 @@ test.skipIf(!url)(
 		try {
 			for (const migration of visitCounter.migrations ?? []) {
 				await migration.up(sql);
-				await migration.up(sql); // Idempotent: the host runs it on every start.
+				await migration.up(sql); // Written to be idempotent, so a second run changes nothing.
 			}
 			const harness = await testPlugin(visitCounter, { database: sql });
 			expect(await harness.runTool("visit_count", { place: "lab" })).toBe(

package/examples/migrations.ts

@@ -2,8 +2,8 @@ import { definePlugin, defineTool } from "pi-roundtable";
 import { Type } from "typebox";
 
 /**
- * Migrations create the plugin's tables before any setup runs, on every start, so they must be
- * idempotent. Their names and the tables are shared with every other plugin: prefix them.
+ * Migrations create the plugin's tables before any setup runs. Each one runs once and is recorded
+ * in a ledger. Table names are shared with every other plugin: prefix them.
  */
 export const visitCounter = definePlugin({
 	name: "visit-counter",

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "pi-roundtable",
-	"version": "0.5.0",
+	"version": "0.5.2",
 	"description": "A plugin-driven Pi agent server for Discord",
 	"type": "module",
 	"license": "MIT",
@@ -34,6 +34,10 @@
 		"type": "git",
 		"url": "git+https://github.com/wayne930242/pi-roundtable.git"
 	},
+	"homepage": "https://pi-roundtable.wayneh.tw",
+	"bugs": {
+		"url": "https://github.com/wayne930242/pi-roundtable/issues"
+	},
 	"publishConfig": {
 		"access": "public",
 		"provenance": true
@@ -45,11 +49,11 @@
 	},
 	"dependencies": {
 		"@babel/parser": "7.29.9",
-		"@earendil-works/pi-ai": ">=0.99.2 <1",
-		"@earendil-works/pi-coding-agent": ">=0.99.2 <1",
+		"@earendil-works/pi-ai": ">=0.99.2 <2",
+		"@earendil-works/pi-coding-agent": ">=0.99.2 <2",
 		"canvas": "3.2.3",
 		"discord.js": "14.27.0",
-		"pi-mcp-adapter": "4.0.0",
+		"pi-mcp-adapter": "5.0.0",
 		"pino": "10.3.1",
 		"typebox": "1.3.34",
 		"unpdf": "1.8.1"

package/src/cli/cli.ts

@@ -79,7 +79,9 @@ export function processEnvironment(): CliEnvironment {
 			roundtable.listen();
 			await roundtable.run();
 		},
+		// pi-lens-ignore: no-console-except-error — the command line's printed output is its product
 		out: (line) => console.log(line),
+		// pi-lens-ignore: no-console-except-error — the command line's printed output is its product
 		err: (line) => console.error(line),
 	};
 }

package/src/cli/main.ts

@@ -7,6 +7,7 @@ runCli(process.argv.slice(2), processEnvironment()).then(
 		process.exitCode = code;
 	},
 	(error: unknown) => {
+		// pi-lens-ignore: no-console-except-error — the entry point reports a failed boot on stderr; no logger exists yet
 		console.error(error instanceof Error ? error.message : String(error));
 		// A boot that failed has stopped what it started; nothing may keep the process up.
 		process.exit(1);

package/src/cli/project.ts

@@ -89,6 +89,8 @@ export class Project {
 		let value: unknown = raw.value;
 		for (const key of path) {
 			if (typeof value !== "object" || value === null) return undefined;
+			// A read only: nothing is assigned by key, and every caller passes literal keys.
+			// nosemgrep: javascript.lang.security.audit.prototype-pollution.prototype-pollution-loop.prototype-pollution-loop
 			value = (value as Record<string, unknown>)[key];
 		}
 		return typeof value === "string" && value.trim() !== "" ? value : undefined;

package/src/cli/runtime.ts

@@ -26,7 +26,7 @@ export async function openModelRuntime(
 ): Promise<ModelRuntime> {
 	process.env.PI_CODING_AGENT_DIR = agentDir;
 	const authPath = join(agentDir, "auth.json");
-	return await ModelRuntime.create(
+	return ModelRuntime.create(
 		existsSync(authPath)
 			? { authPath, refreshOnCreate: false }
 			: { credentials: emptyCredentials, refreshOnCreate: false },
@@ -47,7 +47,7 @@ export async function assemble(
 	config: Parameters<typeof defineRoundtable>[0],
 	agentDir: string,
 ): Promise<DefinedRoundtable> {
-	return await defineRoundtable(config, {
+	return defineRoundtable(config, {
 		modelRuntime: await openModelRuntime(agentDir),
 	});
 }

package/src/cli/templates.ts

@@ -114,15 +114,16 @@ export function renderProject(
 	};
 	const skeleton = filesUnder(dir)
 		.map((file) => relative(dir, file))
-		.filter(
-			(path) =>
-				!path.startsWith(`${PLUGIN_DIR}/`) &&
-				!path.startsWith(`${OFFICIAL_DIR}/`),
-		)
-		.map((path) => ({
-			path: targetOf(path),
-			content: fill(readFileSync(join(dir, path), "utf8"), values),
-		}));
+		.flatMap((path) =>
+			path.startsWith(`${PLUGIN_DIR}/`) || path.startsWith(`${OFFICIAL_DIR}/`)
+				? []
+				: [
+						{
+							path: targetOf(path),
+							content: fill(readFileSync(join(dir, path), "utf8"), values),
+						},
+					],
+		);
 	return [...skeleton, ...renderPlugin(pluginNames("hello"), dir)];
 }
 

package/src/core/agents/agent-prompt.ts

@@ -39,8 +39,7 @@ export function agentSystemPrompt(input: {
 	];
 	if (group) {
 		const others = group.members
-			.filter((m) => m.name !== agent.name)
-			.map((m) => `"${m.displayName}"`)
+			.flatMap((m) => (m.name === agent.name ? [] : [`"${m.displayName}"`]))
 			.join(", ");
 		parts.push(
 			`You are now speaking in the group chat "${group.group.displayName}" with ${w.name} and ${others}. Each of your turns shows the group messages since your last turn. Speak only for yourself and add what your role brings; do not repeat what others said. To hand the next word to a member, mention them as @display name.`,

package/src/core/agents/agent-store.ts

@@ -155,8 +155,7 @@ export class PgAgentStore implements AgentDirectory {
 	#activeNames(): string {
 		return (
 			this.agents()
-				.filter((a) => a.status === "active")
-				.map((a) => a.name)
+				.flatMap((a) => (a.status === "active" ? [a.name] : []))
 				.join(", ") || "none"
 		);
 	}

package/src/core/agents/agent-team-fixture.ts

@@ -115,9 +115,11 @@ export class FakeChannels implements AgentChannels {
 		this.webhooksRemoved.push(channelId);
 	}
 	texts(channelId: string) {
-		return this.posts
-			.filter((p) => p.channelId === channelId)
-			.map((p) => `${p.post.name}: ${p.post.chunks.join("")}`);
+		return this.posts.flatMap((p) =>
+			p.channelId === channelId
+				? [`${p.post.name}: ${p.post.chunks.join("")}`]
+				: [],
+		);
 	}
 }
 

package/src/core/agents/fallback-avatar.ts

@@ -10,6 +10,7 @@ const BADGE = { x: 396, y: 396, radius: 64 };
 const hsl = (h: number, s: number, l: number): string =>
 	`hsl(${Math.round(h) % 360},${s}%,${l}%)`;
 
+// pi-lens-ignore: hardcoded-url — the SVG XML namespace, a fixed identifier rather than an address that is fetched
 const layer = (body: string): Buffer =>
 	Buffer.from(
 		`<svg xmlns="http://www.w3.org/2000/svg" width="${SIDE}" height="${SIDE}" viewBox="0 0 ${SIDE} ${SIDE}">${body}</svg>`,

package/src/core/agents/group-messages.ts

@@ -43,7 +43,7 @@ export class GroupMessages {
 		const rows: MessageRow[] = await this.#sql`
 			SELECT id, author, author_name, text, created_at FROM agent_group_messages
 			WHERE guild_id = ${this.#guild} AND group_name = ${group} ORDER BY id DESC LIMIT ${limit}`;
-		return rows.reverse().map(toMessage);
+		return rows.toReversed().map(toMessage);
 	}
 
 	/** Group messages the member has not received yet, its own excluded; the newest `limit` are carried. */

package/src/core/agents/group-round.ts

@@ -15,20 +15,21 @@ const PROMPT_CHARS_FOR_SCORING = 1_200;
 const escapeRegExp = (value: string) =>
 	value.replace(/[.*+?^${}()|[\]\\]/g, "\\$&");
 
+const mentions = (text: string, label: string) =>
+	// The label is an agent's name, escaped so every character of it matches literally.
+	// nosemgrep: javascript.lang.security.audit.detect-non-literal-regexp.detect-non-literal-regexp
+	new RegExp(`@${escapeRegExp(label)}(?![\\p{L}\\p{N}_-])`, "iu").test(text);
+
 /** Members a text names with `@display name` or `@name`, in group order. */
 export function mentionedMembers(
 	text: string,
 	members: readonly Agent[],
 ): string[] {
-	return members
-		.filter((member) =>
-			[member.displayName, member.name].some((label) =>
-				new RegExp(`@${escapeRegExp(label)}(?![\\p{L}\\p{N}_-])`, "iu").test(
-					text,
-				),
-			),
-		)
-		.map((member) => member.name);
+	return members.flatMap((member) =>
+		[member.displayName, member.name].some((label) => mentions(text, label))
+			? [member.name]
+			: [],
+	);
 }
 
 /**

package/src/core/agents/group-turns.ts

@@ -12,7 +12,12 @@ import {
 	RECENT_FOR_SCORING,
 } from "./group-round.ts";
 import { channelIdOf, groupScope, groupSessionKey } from "./team-keys.ts";
-import { BACKLOG_LIMIT, type Chain, type TurnHost } from "./team-turn-types.ts";
+import {
+	BACKLOG_LIMIT,
+	type Chain,
+	type TurnExtra,
+	type TurnHost,
+} from "./team-turn-types.ts";
 
 /** A group channel's rounds: scoring, then the chosen members answer one at a time. */
 export class GroupTurns {
@@ -99,19 +104,19 @@ export class GroupTurns {
 			// Delivered once the session has the text, so a message never comes twice.
 			if (backlog.lastId !== undefined)
 				await store.advanceCursor(group.name, name, backlog.lastId);
+			// Attachments and a confirmation belong to a member's first turn of the round only.
+			const extra: TurnExtra = { interactive: true };
+			if (first) {
+				extra.attachments = attachments;
+				if (approvals.get(name)) extra.confirmed = true;
+			}
 			const result = await this.#host.turn(
 				member,
 				groupScope(member, group),
 				channel,
 				groupTurnText(group, member, backlog),
 				chain,
-				first
-					? {
-							attachments,
-							interactive: true,
-							...(approvals.get(name) ? { confirmed: true } : {}),
-						}
-					: { interactive: true },
+				extra,
 			);
 			replies += 1;
 			last = member;

package/src/core/agents/team-layout.ts

@@ -54,17 +54,16 @@ export function layoutText(
 	layout: readonly CategoryLayout[],
 	store: TeamStore,
 ): string {
-	const lines = layout
-		.filter((c) => categoryKind(c.name))
-		.map((c) => {
-			const shown = c.channelIds.map(
-				(id) =>
-					store.agentByChannel(id)?.name ??
-					store.groupByChannel(id)?.name ??
-					`<#${id}>`,
-			);
-			return `- ${c.name}: ${shown.join(", ") || "empty"}`;
-		});
+	const lines = layout.flatMap((c) => {
+		if (!categoryKind(c.name)) return [];
+		const shown = c.channelIds.map(
+			(id) =>
+				store.agentByChannel(id)?.name ??
+				store.groupByChannel(id)?.name ??
+				`<#${id}>`,
+		);
+		return [`- ${c.name}: ${shown.join(", ") || "empty"}`];
+	});
 	return `Categories, in server order:\n${lines.join("\n") || "none"}`;
 }
 
@@ -122,9 +121,9 @@ export function planArrangement(
 			channelIds: [...channelIds, ...(existing ? stay(existing) : [])],
 		};
 	});
-	const rest = current
-		.filter((c) => !ahead.some((a) => a.id === c.id))
-		.map((c) => ({ ...c, channelIds: stay(c) }));
+	const rest = current.flatMap((c) =>
+		ahead.some((a) => a.id === c.id) ? [] : [{ ...c, channelIds: stay(c) }],
+	);
 	const layout = [...ahead, ...rest];
 	const remove = layout.flatMap((c) =>
 		c.id && c.channelIds.length === 0 && categoryKind(c.name) ? [c.id] : [],

package/src/core/agents/team-status.ts

@@ -61,12 +61,11 @@ export async function teamStatus(ctx: TeamContext): Promise<TeamStatus> {
 			schedules: home ? (await schedules.forChannel(home)).length : 0,
 		});
 	}
-	const groups: GroupStatus[] = store
-		.groups()
-		.filter((group) => group.status === "active")
-		.map((group) => {
-			const lastActive = turns.lastGroupActive(group.name);
-			return {
+	const groups: GroupStatus[] = store.groups().flatMap((group) => {
+		if (group.status !== "active") return [];
+		const lastActive = turns.lastGroupActive(group.name);
+		return [
+			{
 				name: group.name,
 				displayName: group.displayName,
 				channelId: group.channelId,
@@ -74,7 +73,8 @@ export async function teamStatus(ctx: TeamContext): Promise<TeamStatus> {
 				host: store.agent(group.host)?.displayName ?? group.host,
 				busy: queue.size(discordKey(group.channelId)),
 				...(lastActive ? { lastActive } : {}),
-			};
-		});
+			},
+		];
+	});
 	return { agents, groups };
 }

package/src/core/agents/team-turns.ts

@@ -9,7 +9,7 @@ import type { AgentTurnScope } from "../domain/ports.ts";
 import { messages } from "../i18n/index.ts";
 import { splitReply } from "../presentation/reply-splitter.ts";
 import { thinkingLine } from "../presentation/thinking-line.ts";
-import { settleTurn } from "../routing/settle-turn.ts";
+import { endOf, settleTurn } from "../routing/settle-turn.ts";
 import type { Speaker } from "../speakers.ts";
 import { toolTiers } from "../tool-tiers.ts";
 import { AgentMessages } from "./agent-messages.ts";
@@ -31,6 +31,11 @@ import {
 
 export { MAX_CHAIN_MESSAGES, type TeamTurnsOptions };
 
+/** What a failed turn posts: the stopped notice for a stop, the failure notice otherwise. */
+function noticeOf(result: { stopped?: true }): string {
+	return result.stopped ? messages().stoppedNotice : messages().failureNotice;
+}
+
 /**
  * The agent team's turns: owner messages, background turns, group rounds, and messages between
  * agents, each posted under the answering agent's name.
@@ -199,7 +204,7 @@ export class TeamTurns {
 		}
 		this.#options.events?.turnEnded({
 			...turn,
-			result: result.ok ? "ok" : result.stopped ? "stopped" : "failed",
+			result: endOf(result),
 		});
 		try {
 			if (!result.ok && !result.stopped) {
@@ -214,13 +219,7 @@ export class TeamTurns {
 					: undefined;
 			await this.#post(postTo, this.#current(agent), {
 				...(thinking ? { thinking } : {}),
-				chunks: result.ok
-					? splitReply(result.text)
-					: [
-							result.stopped
-								? messages().stoppedNotice
-								: messages().failureNotice,
-						],
+				chunks: result.ok ? splitReply(result.text) : [noticeOf(result)],
 			});
 		} catch (error) {
 			logger.error({ agent: agent.name, err: error }, "agent reply not posted");

package/src/core/define-roundtable.ts

@@ -76,7 +76,24 @@ function judgeThrough(modelRuntime: ModelRuntime, ref: ModelRef): JudgeModel {
 			throw new JudgeError(
 				`the judge's model ${formatModelRef(ref)} is not available; log in to its provider or change judge.model`,
 			);
-		return await piJudgeModel(modelRuntime, model)(system, prompt);
+		return piJudgeModel(modelRuntime, model)(system, prompt);
+	};
+}
+
+/** The listener `config.http` names: its unix socket when it has one, else its TCP port. */
+function publicListener(
+	http: ReturnType<typeof resolveConfig>["http"],
+): ListenerConfig {
+	if (http.socketPath)
+		return {
+			id: "public",
+			socketPath: http.socketPath,
+			...(http.socketMode === undefined ? {} : { mode: http.socketMode }),
+		};
+	return {
+		id: "public",
+		port: http.port,
+		...(http.hostname ? { hostname: http.hostname } : {}),
 	};
 }
 
@@ -138,24 +155,7 @@ export async function defineRoundtable(
 			},
 			database: { url: config.databaseUrl },
 			toolTiers: toolTiers(config.toolTiers),
-			listeners: [
-				config.http.socketPath
-					? {
-							id: "public",
-							socketPath: config.http.socketPath,
-							...(config.http.socketMode === undefined
-								? {}
-								: { mode: config.http.socketMode }),
-						}
-					: {
-							id: "public",
-							port: config.http.port,
-							...(config.http.hostname
-								? { hostname: config.http.hostname }
-								: {}),
-						},
-				...(overrides.listeners ?? []),
-			],
+			listeners: [publicListener(config.http), ...(overrides.listeners ?? [])],
 			judgeModel: judgeThrough(modelRuntime, config.judge.model),
 			apiKey: (provider) => registry.getApiKeyForProvider(provider),
 			...(overrides.aborted ? { aborted: overrides.aborted } : {}),

package/src/core/define.ts

@@ -120,20 +120,20 @@ export function defineTool<Schema extends TObject>(
 		}),
 	};
 	const hold = spec.hold;
-	return {
+	const contribution = {
 		name,
 		minTier: spec.minTier,
 		agent: spec.agent ?? true,
 		session,
-		...(hold
-			? {
-					hold: {
-						name: `tool:${name}`,
-						describe: (tool, input) =>
-							tool === name ? hold(input as Static<Schema>) : undefined,
-					},
-				}
-			: {}),
+	};
+	if (!hold) return contribution;
+	return {
+		...contribution,
+		hold: {
+			name: `tool:${name}`,
+			describe: (tool, input) =>
+				tool === name ? hold(input as Static<Schema>) : undefined,
+		},
 	};
 }
 

package/src/core/discord/channel-executor.ts

@@ -90,13 +90,12 @@ export function operationPermissions(
 	channel: OwnerChannel,
 ): bigint[] {
 	const needed = OPERATION_PERMISSIONS[operation];
-	return channel.isThread()
-		? needed.map((flag) =>
-				flag === PermissionFlagsBits.SendMessages
-					? PermissionFlagsBits.SendMessagesInThreads
-					: flag,
-			)
-		: [...needed];
+	if (!channel.isThread()) return [...needed];
+	return needed.map((flag) =>
+		flag === PermissionFlagsBits.SendMessages
+			? PermissionFlagsBits.SendMessagesInThreads
+			: flag,
+	);
 }
 
 interface ChannelFile {
@@ -127,6 +126,16 @@ function serializeMessage(message: Message) {
 	};
 }
 
+function uploadOf(upload: ChannelFile) {
+	return {
+		attachment: Buffer.from(upload.dataBase64, "base64"),
+		name: upload.filename,
+		...(upload.description === undefined
+			? {}
+			: { description: upload.description }),
+	};
+}
+
 function messagePayload(
 	args: Record<string, unknown>,
 ): Pick<MessageCreateOptions, "content" | "allowedMentions" | "files"> {
@@ -134,17 +143,7 @@ function messagePayload(
 	return {
 		...(args.content !== undefined ? { content: String(args.content) } : {}),
 		allowedMentions: { parse: [] },
-		...(uploads
-			? {
-					files: uploads.map((upload) => ({
-						attachment: Buffer.from(upload.dataBase64, "base64"),
-						name: upload.filename,
-						...(upload.description === undefined
-							? {}
-							: { description: upload.description }),
-					})),
-				}
-			: {}),
+		...(uploads ? { files: uploads.map(uploadOf) } : {}),
 	};
 }
 
@@ -393,6 +392,9 @@ export function discordChannelExecutor(
 							"INVALID_CHANNEL: a thread takes its parent channel's permissions",
 						);
 					return editPermissions(channel, tool, args);
+				default:
+					// A message operation, handled below.
+					break;
 			}
 			const message = await channel.messages.fetch(String(args.messageId));
 			if (tool === "discord_edit_message")

package/src/core/discord/channel-operations.ts

@@ -262,6 +262,12 @@ const DISCORD_EPOCH = 1420070400000;
 const BASE64 =
 	/^(?:[A-Za-z0-9+/]{4})*(?:[A-Za-z0-9+/]{2}==|[A-Za-z0-9+/]{3}=)?$/;
 
+/** The `=` padding a base64 string ends with, which carries no bytes. */
+function paddingOf(data: string): number {
+	if (data.endsWith("==")) return 2;
+	return data.endsWith("=") ? 1 : 0;
+}
+
 /** Checks and fills defaults into a tool's arguments; throws ChannelToolError otherwise. */
 export function parseChannelTool(
 	tool: string,
@@ -307,9 +313,7 @@ export function parseChannelTool(
 			if (!BASE64.test(upload.dataBase64))
 				throw new ChannelToolError("INVALID_CHANNEL_BASE64");
 			const data = upload.dataBase64;
-			bytes +=
-				(data.length / 4) * 3 -
-				(data.endsWith("==") ? 2 : data.endsWith("=") ? 1 : 0);
+			bytes += (data.length / 4) * 3 - paddingOf(data);
 			if (bytes > CHANNEL_UPLOAD_MAX_BYTES)
 				throw new ChannelToolError("INVALID_CHANNEL_UPLOAD_SIZE");
 		}

package/src/core/discord/inbound-message.ts

@@ -22,6 +22,16 @@ function attachmentRefs(
 	}));
 }
 
+function referenceOf(referenced: Message, mention: RegExp) {
+	return {
+		text: referenced.content.replace(mention, "").trim(),
+		attachments: attachmentRefs(referenced),
+		...(referenced.webhookId
+			? { webhookName: referenced.author.username }
+			: {}),
+	};
+}
+
 /**
  * The facts of one Discord message, as the neutral `InboundMessage`; undefined for the bot's own
  * messages and before the client is ready. A forward reports the channel it came from as a
@@ -32,6 +42,8 @@ export async function toInbound(
 	{ botId, applicationId }: BotIdentity,
 ): Promise<InboundMessage | undefined> {
 	if (!botId || message.author.id === botId) return undefined;
+	// botId is the bot's own Discord user id from the client, a snowflake, not message text.
+	// nosemgrep: javascript.lang.security.audit.detect-non-literal-regexp.detect-non-literal-regexp
 	const mention = new RegExp(`<@!?${botId}>`, "g");
 	const reference = message.reference;
 	// A forward also has a reference, to a message elsewhere; its copy is the snapshot.
@@ -84,16 +96,6 @@ export async function toInbound(
 					},
 				}
 			: {}),
-		...(referenced
-			? {
-					reference: {
-						text: referenced.content.replace(mention, "").trim(),
-						attachments: attachmentRefs(referenced),
-						...(referenced.webhookId
-							? { webhookName: referenced.author.username }
-							: {}),
-					},
-				}
-			: {}),
+		...(referenced ? { reference: referenceOf(referenced, mention) } : {}),
 	};
 }

package/src/core/discord/owner-cards.ts

@@ -362,9 +362,9 @@ export class OwnerCards implements InteractionModule {
 		}
 		if (interaction.isStringSelectMenu() && action === "pick") {
 			const values = interaction.values;
-			const choices = question.options
-				.filter((_option, i) => values.includes(String(i)))
-				.map((option) => option.label);
+			const choices = question.options.flatMap((option, i) =>
+				values.includes(String(i)) ? [option.label] : [],
+			);
 			if (values.includes(OTHER)) {
 				card.picked = choices;
 				await interaction.showModal(answerModal(id));

package/src/core/discord/owner-discord-threads.ts

@@ -64,18 +64,22 @@ export class OwnerThreads {
 					permissions.has(PermissionFlagsBits.ManageThreads))
 			);
 		};
-		return threads
-			.filter((t) => sees(owner, t) && sees(bot, t))
-			.map((t) => ({
-				id: t.id,
-				name: t.name,
-				parentId: t.parentId,
-				private: t.type === ChannelType.PrivateThread,
-				archived: t.archived,
-				locked: t.locked,
-				messageCount: t.messageCount,
-				createdAt: t.createdAt?.toISOString() ?? null,
-			}));
+		return threads.flatMap((t) =>
+			sees(owner, t) && sees(bot, t)
+				? [
+						{
+							id: t.id,
+							name: t.name,
+							parentId: t.parentId,
+							private: t.type === ChannelType.PrivateThread,
+							archived: t.archived,
+							locked: t.locked,
+							messageCount: t.messageCount,
+							createdAt: t.createdAt?.toISOString() ?? null,
+						},
+					]
+				: [],
+		);
 	}
 
 	async create(args: Record<string, unknown>) {