pi-roundtable 0.1.0 → 0.2.1

package/src/core/modules/schedules/recurrence.ts

@@ -2,7 +2,10 @@ import { ScheduleError } from "../../domain/errors.ts";
 import { messages } from "../../i18n/index.ts";
 import { timeZone, zonedDate, zonedInstant, zonedStamp } from "../../time.ts";
 
-export const WEEKDAYS = [
+export type Weekday = "sun" | "mon" | "tue" | "wed" | "thu" | "fri" | "sat";
+
+/** In the order of `Date#getUTCDay`. */
+const WEEKDAYS: readonly Weekday[] = [
 	"sun",
 	"mon",
 	"tue",
@@ -10,8 +13,7 @@ export const WEEKDAYS = [
 	"thu",
 	"fri",
 	"sat",
-] as const;
-export type Weekday = (typeof WEEKDAYS)[number];
+];
 
 /** When a schedule runs, in the configured wall-clock time. */
 export type Recurrence =

package/src/core/modules/schedules/schedule-store.ts

@@ -1,16 +1,15 @@
 import type { SQL } from "bun";
 import type { Migration } from "../../db/migrations.ts";
 import type { ChannelKey } from "../../domain/conversation.ts";
+import type { ScheduleStore } from "../../services.ts";
 import type { Tier } from "../../speakers.ts";
 import type { Recurrence } from "./recurrence.ts";
 
-/** Owner schedules run the owner agent; party schedules run the channel's party role. */
-export type ScheduleMode = "owner" | "party";
-
 export interface Schedule {
 	id: number;
 	channel: ChannelKey;
-	mode: ScheduleMode;
+	/** The name of the background target whose conversation runs it; stored in the `mode` column. */
+	target: string;
 	title: string;
 	prompt: string;
 	recurrence: Recurrence;
@@ -26,7 +25,7 @@ export interface Schedule {
 
 export interface NewSchedule {
 	channel: ChannelKey;
-	mode: ScheduleMode;
+	target: string;
 	title: string;
 	prompt: string;
 	recurrence: Recurrence;
@@ -46,7 +45,8 @@ export interface ScheduleChange {
 interface Row {
 	id: string | number;
 	channel_key: ChannelKey;
-	mode: ScheduleMode;
+	/** The target's name; the column keeps the name it had before targets existed. */
+	mode: string;
 	title: string;
 	prompt: string;
 	recurrence: string;
@@ -63,7 +63,7 @@ function toSchedule(row: Row): Schedule {
 	return {
 		id: Number(row.id),
 		channel: row.channel_key,
-		mode: row.mode,
+		target: row.mode,
 		title: row.title,
 		prompt: row.prompt,
 		// pi-lens-ignore: unchecked-throwing-call — this store wrote the JSON; a corrupt row should fail loudly
@@ -79,7 +79,7 @@ function toSchedule(row: Row): Schedule {
 }
 
 /** Scheduled turns, in the database. A one-time schedule's row is deleted once it fires. */
-export class ScheduleStore {
+export class PgScheduleStore implements ScheduleStore {
 	readonly #sql: SQL;
 
 	private constructor(sql: SQL) {
@@ -113,15 +113,15 @@ export class ScheduleStore {
 	};
 
 	/** The store over the host's migrated pool. */
-	static async attach(sql: SQL): Promise<ScheduleStore> {
-		return new ScheduleStore(sql);
+	static async attach(sql: SQL): Promise<PgScheduleStore> {
+		return new PgScheduleStore(sql);
 	}
 
 	async create(schedule: NewSchedule): Promise<Schedule> {
 		const rows: Row[] = await this.#sql`
 			INSERT INTO schedules (channel_key, mode, title, prompt, recurrence, next_run,
 				created_by_id, created_by_name, created_tier)
-			VALUES (${schedule.channel}, ${schedule.mode}, ${schedule.title}, ${schedule.prompt},
+			VALUES (${schedule.channel}, ${schedule.target}, ${schedule.title}, ${schedule.prompt},
 				${JSON.stringify(schedule.recurrence)}, ${schedule.nextRun},
 				${schedule.createdById}, ${schedule.createdByName}, ${schedule.createdTier})
 			RETURNING *`;

package/src/core/modules/schedules/schedule-tools.ts

@@ -1,6 +1,8 @@
+import type { BackgroundTarget } from "../../contract/channels.ts";
 import type { ChannelKey } from "../../domain/conversation.ts";
 import { ScheduleError } from "../../domain/errors.ts";
 import { messages } from "../../i18n/index.ts";
+import type { ScheduleStore } from "../../services.ts";
 import type { ScheduleToolName } from "../../shared/schedule-tools.ts";
 import { type Tier, tierAtLeast } from "../../speakers.ts";
 import { timeZone, zonedStamp } from "../../time.ts";
@@ -11,22 +13,8 @@ import {
 	type Recurrence,
 	type RecurrenceInput,
 } from "./recurrence.ts";
-import type {
-	Schedule,
-	ScheduleMode,
-	ScheduleStore,
-} from "./schedule-store.ts";
+import type { Schedule } from "./schedule-store.ts";
 
-/** Party channels are open to anyone, so their roles get tighter limits than the owner's agent. */
-export const SCHEDULE_LIMITS: Readonly<
-	Record<
-		ScheduleMode,
-		{ perChannel: number; promptChars: number; aheadDays: number }
-	>
-> = {
-	owner: { perChannel: 20, promptChars: 8_000, aheadDays: 366 },
-	party: { perChannel: 5, promptChars: 2_000, aheadDays: 90 },
-};
 const TITLE_CHARS = 80;
 const LIST_PROMPT_PREVIEW = 200;
 const DAY_MS = 86_400_000;
@@ -37,12 +25,25 @@ export interface ScheduleToolContext {
 		"create" | "get" | "forChannel" | "update" | "remove"
 	>;
 	channel: ChannelKey;
-	mode: ScheduleMode;
+	/** Whose schedules these are; its `schedules` limits apply, and without them nothing is scheduled. */
+	target: BackgroundTarget;
 	/** Who asked, recorded on created schedules; a scheduled run speaks for its creator. */
 	author: { id: string; name: string; tier?: Tier };
 	now: Date;
 }
 
+/** The limits of the context's target; a target without them may not schedule. */
+function limitsOf(
+	ctx: ScheduleToolContext,
+): NonNullable<BackgroundTarget["schedules"]> {
+	const { schedules } = ctx.target;
+	if (!schedules)
+		throw new ScheduleError(
+			`schedules are not available for ${ctx.target.name} conversations`,
+		);
+	return schedules;
+}
+
 type Input = Record<string, unknown> & RecurrenceInput;
 
 function text(input: Input, name: string, max: number): string {
@@ -82,7 +83,7 @@ function timing(ctx: ScheduleToolContext, input: Input): [Recurrence, Date] {
 		throw new ScheduleError(
 			`that time has already passed; it is ${zonedStamp(ctx.now)} in ${messages().zoneName(timeZone())} now`,
 		);
-	const { aheadDays } = SCHEDULE_LIMITS[ctx.mode];
+	const { aheadDays } = limitsOf(ctx);
 	if (next.getTime() - ctx.now.getTime() > aheadDays * DAY_MS)
 		throw new ScheduleError(`the first run must be within ${aheadDays} days`);
 	return [recurrence, next];
@@ -121,9 +122,10 @@ function changeable(ctx: ScheduleToolContext, schedule: Schedule): Schedule {
 export async function callScheduleTool(
 	ctx: ScheduleToolContext,
 	name: ScheduleToolName,
-	input: Input,
+	rawInput: Record<string, unknown>,
 ): Promise<string> {
-	const limits = SCHEDULE_LIMITS[ctx.mode];
+	const input = rawInput as Input;
+	const limits = limitsOf(ctx);
 	switch (name) {
 		case "schedule_create": {
 			const title = text(input, "title", TITLE_CHARS);
@@ -136,7 +138,7 @@ export async function callScheduleTool(
 				);
 			const created = await ctx.store.create({
 				channel: ctx.channel,
-				mode: ctx.mode,
+				target: ctx.target.name,
 				title,
 				prompt,
 				recurrence,

package/src/core/modules/schedules/scheduler.ts

@@ -1,7 +1,8 @@
 import type { ScheduledOutcome } from "../../contract/channels.ts";
 import type { Logger } from "../../log.ts";
+import type { ScheduleStore } from "../../services.ts";
 import { nextRun } from "./recurrence.ts";
-import type { Schedule, ScheduleStore } from "./schedule-store.ts";
+import type { Schedule } from "./schedule-store.ts";
 
 export type { ScheduledOutcome };
 
@@ -104,7 +105,11 @@ export class Scheduler {
 	async #fire(schedule: Schedule, firedAt: Date): Promise<void> {
 		const { store, runner, logger } = this.#options;
 		logger.info(
-			{ schedule: schedule.id, channel: schedule.channel, mode: schedule.mode },
+			{
+				schedule: schedule.id,
+				channel: schedule.channel,
+				target: schedule.target,
+			},
 			"schedule firing",
 		);
 		const outcome = await runner.runScheduled(schedule, firedAt);

package/src/core/modules/schedules/schedules.ts

@@ -1,18 +1,21 @@
 import type { ExtensionFactory } from "@earendil-works/pi-coding-agent";
 import { Type } from "typebox";
+import { OWNER_TARGET } from "../../agents/agent-claim.ts";
 import type { ChannelKey } from "../../domain/conversation.ts";
 import { ScheduleError } from "../../domain/errors.ts";
+import { activeLocale } from "../../i18n/index.ts";
 import type { OwnerIdentity } from "../../identity.ts";
 import {
 	type ToolInput,
 	textToolsExtension,
 } from "../../runtime/text-tools.ts";
+import type { ScheduleStore } from "../../services.ts";
 import {
 	type ScheduleToolSpec,
 	scheduleToolSpecs,
 } from "../../shared/schedule-tools.ts";
 import type { Speaker } from "../../speakers.ts";
-import type { ScheduleStore } from "./schedule-store.ts";
+import { timeZone } from "../../time.ts";
 import { callScheduleTool } from "./schedule-tools.ts";
 
 export interface OwnerSchedules {
@@ -56,7 +59,10 @@ export function schedulesExtension(
 	speaker: () => Speaker | undefined = () => undefined,
 ): ExtensionFactory {
 	const { store, owner, channelFor } = schedules;
-	const defs = scheduleToolSpecs().map((base) => {
+	const defs = scheduleToolSpecs({
+		locale: activeLocale(),
+		timeZone: timeZone(),
+	}).map((base) => {
 		const spec = agents ? withAgentOption(base) : base;
 		return {
 			...spec,
@@ -70,7 +76,7 @@ export function schedulesExtension(
 					{
 						store,
 						channel: target,
-						mode: "owner",
+						target: OWNER_TARGET,
 						author: speaker() ?? owner,
 						now: new Date(),
 					},

package/src/core/modules/skills/skill-registry.ts

@@ -1,6 +1,7 @@
 import { mkdirSync, readdirSync, rmSync, writeFileSync } from "node:fs";
 import { join } from "node:path";
 import { AgentError } from "../../domain/errors.ts";
+import type { SkillRegistry } from "../../services.ts";
 import { planLinkedSkills } from "./skill-link.ts";
 import { skillCatalog, skillListText } from "./skill-listing.ts";
 import {
@@ -28,8 +29,8 @@ export {
 	type SkillSource,
 } from "./skill-rules.ts";
 
-/** The skill registry and what each agent carries (repos-and-skills spec behaviors 13–22). */
-export class SkillRegistry {
+/** The skill registry and what each agent carries. */
+export class StoredSkillRegistry implements SkillRegistry {
 	readonly #options: SkillRegistryOptions;
 
 	constructor(options: SkillRegistryOptions) {

package/src/core/modules/skills/skill-store.ts

@@ -4,7 +4,7 @@ import type { Migration } from "../../db/migrations.ts";
 import { AgentError } from "../../domain/errors.ts";
 
 /**
- * A registered skill (repos-and-skills spec behavior 13). A linked skill's name and description
+ * A registered skill. A linked skill's name and description
  * live in its repository's file; an agent-written one keeps them here.
  */
 export type SkillEntry =
@@ -17,7 +17,7 @@ export interface SkillGroup {
 	skills: string[];
 }
 
-/** Who lost a removed skill (behavior 17). */
+/** Who lost a removed skill. */
 export interface Detached {
 	agents: string[];
 	groups: string[];
@@ -59,12 +59,6 @@ const toGroup = (row: GroupRow): SkillGroup => ({
 	skills: row.skills.split(",").filter(Boolean),
 });
 
-/**
- * The kind an agent-written skill was stored under before it became 'written': the first
- * consumer's own name, spelled from parts so the public tree names no consumer.
- */
-export const LEGACY_WRITTEN_KIND = ["mer", "lin"].join("");
-
 /**
  * Skills, skill groups, and the skills each agent carries, in the database. All of it is
  * also held in memory, because every agent turn resolves its skills and only this process
@@ -128,13 +122,6 @@ export class SkillStore {
 						{ table: "agent_skills", key: ["agent", "skill"] },
 					]),
 			},
-			{
-				// Skills an agent wrote were stored under the legacy kind; the kind is now 'written'.
-				name: "skills-written-kind",
-				up: async (sql) => {
-					await sql`UPDATE skills SET kind = 'written' WHERE kind = ${LEGACY_WRITTEN_KIND}`;
-				},
-			},
 		];
 	}
 

package/src/core/modules/skills/skill-tools.ts

@@ -7,7 +7,11 @@ import {
 	stringList,
 	textToolsExtension,
 } from "../../runtime/text-tools.ts";
-import { MAX_SKILL_BODY_CHARS, type SkillRegistry } from "./skill-registry.ts";
+import type { SkillRegistry } from "../../services.ts";
+import {
+	MAX_SKILL_BODY_CHARS,
+	type StoredSkillRegistry,
+} from "./skill-registry.ts";
 
 export const SKILL_LIST_TOOL = "skill_list";
 
@@ -43,7 +47,9 @@ const BODY = Type.String({
 });
 
 /** skill_list, for the DM and for agents. */
-export function skillListExtension(registry: SkillRegistry): ExtensionFactory {
+export function skillListExtension(
+	registry: Pick<SkillRegistry, "list">,
+): ExtensionFactory {
 	return textToolsExtension(
 		[
 			{
@@ -75,9 +81,9 @@ export function skillListExtension(registry: SkillRegistry): ExtensionFactory {
 	);
 }
 
-/** The skill tools of an agent session (repos-and-skills spec behaviors 15–19). */
+/** The skill tools of an agent session. */
 export function skillToolsExtension(
-	registry: SkillRegistry,
+	registry: StoredSkillRegistry,
 	/** Throws AgentError unless the name is an active agent. */
 	checkAgent: (name: string) => void,
 ): ExtensionFactory {

package/src/core/ops/error-reporter.ts

@@ -16,7 +16,7 @@ const FIELD_LIMIT = 200;
 const STACK_FRAMES = 8;
 /** pino's own keys, and the ones the report shows apart from the context. */
 const OWN_KEYS = new Set(["level", "time", "pid", "hostname", "msg", "err"]);
-const HEAD_KEYS = ["app", "module"];
+const HEAD_KEYS = ["app", "plugin", "module"];
 const APP_FRAME = /((?:src|shared|worker)\/[^\s():]+\.ts):(\d+)/;
 
 /** Where the reports go, known once the Discord surface and the agent team are up. */