pi-roundtable 0.7.19 → 0.9.0

package/src/core/testing/tool-set.ts

@@ -1,3 +1,5 @@
+import type { ChannelKey } from "../domain/conversation.ts";
+import type { RoundtablePlugin } from "../plugin.ts";
 import { ASK_USER_TOOL } from "../runtime/extensions/ask-user.ts";
 import { COMPACT_TOOL } from "../runtime/extensions/self-compact-guard.ts";
 import { AGENTS } from "../services.ts";
@@ -16,6 +18,36 @@ export const seat: AgentTurnScope = {
 	group: "war-room",
 };
 
+/**
+ * The conversations the recorded sessions live in, served as a host serves them: the owner's
+ * direct messages by an app's claim, such as one that answers the owner there, and the agent's
+ * channel by the agent server; both answer background turns, so their sessions schedule and
+ * delegate. Give it to the test host the tool set is captured on.
+ */
+export function servedConversations(
+	ownerChannel: ChannelKey,
+): RoundtablePlugin {
+	const served = new Set<ChannelKey>([ownerChannel, scout.home, seat.home]);
+	return {
+		name: "served-conversations",
+		setup: () => ({
+			channels: [
+				{
+					name: "served-conversations",
+					priority: 0,
+					owns: (channel) => served.has(channel),
+					admit: () => undefined,
+					background: async () => ({
+						status: "skipped",
+						reason: "the tool set's host runs no background turns",
+					}),
+					startFresh: async () => "owner",
+				},
+			],
+		}),
+	};
+}
+
 /** Tools the core's own extensions and pi-web-access register, which no fake session here loads. */
 const NAMED_ELSEWHERE = [
 	ASK_USER_TOOL,

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

@@ -37,7 +37,7 @@
 				"memory_add",
 				"memory_remove",
 				"memory_search",
-				"notify_owner",
+				"notify",
 				"schedule_cancel",
 				"schedule_create",
 				"schedule_list",
@@ -92,7 +92,7 @@
 				"memory_remove",
 				"memory_search",
 				"message_agent",
-				"notify_owner",
+				"notify",
 				"schedule_cancel",
 				"schedule_create",
 				"schedule_list",
@@ -179,7 +179,7 @@
 				"memory_add",
 				"memory_remove",
 				"memory_search",
-				"notify_owner",
+				"notify",
 				"schedule_cancel",
 				"schedule_create",
 				"schedule_list",
@@ -272,7 +272,7 @@
 		"memory_remove": "member",
 		"memory_search": "member",
 		"message_agent": "member",
-		"notify_owner": "owner",
+		"notify": "owner",
 		"read": "owner",
 		"read_attachment": "member",
 		"schedule_cancel": "admin",

package/src/core/tool-tiers.ts

@@ -1,4 +1,5 @@
 import { PluginError } from "./errors.ts";
+import type { Logger } from "./log.ts";
 import { ASK_USER_TOOL } from "./runtime/extensions/ask-user.ts";
 import { COMPACT_TOOL } from "./runtime/extensions/self-compact-guard.ts";
 import { type Tier, tierAtLeast } from "./speakers.ts";
@@ -27,6 +28,40 @@ export const CORE_TOOL_TIERS: Readonly<Record<string, Tier>> = set("member", [
 	"read_attachment",
 ]);
 
+/**
+ * Tools renamed since 0.8, by their old name: until 1.0 the old name still names the tool in a
+ * selection, a task's `exclude`, a plugin's `requiredTools`, and the operator's and plugins'
+ * `toolTiers`, where the new name's own setting wins.
+ */
+export const RENAMED_TOOLS: Readonly<Record<string, string>> = {
+	notify_owner: "notify",
+};
+
+const warnedAliases = new WeakMap<Logger, Set<string>>();
+
+/** Each logger warns once for an old name used in tiers or selections. */
+function warnAliases(tools: readonly string[], logger: Logger): void {
+	const warned = warnedAliases.get(logger) ?? new Set<string>();
+	warnedAliases.set(logger, warned);
+	for (const tool of tools) {
+		const renamed = RENAMED_TOOLS[tool];
+		if (!renamed || warned.has(tool)) continue;
+		warned.add(tool);
+		logger.warn(
+			`deprecated: ${tool} is now ${renamed}; update selections and toolTiers before 1.0`,
+		);
+	}
+}
+
+/** The tools named, each by its current name and once, in order. */
+export function currentToolNames(
+	tools: readonly string[],
+	logger?: Logger,
+): string[] {
+	if (logger) warnAliases(tools, logger);
+	return [...new Set(tools.map((tool) => RENAMED_TOOLS[tool] ?? tool))];
+}
+
 /**
  * What each tool needs: the operator's setting first, then the tier its plugin declared, then
  * the core's default, and the owner for a tool nobody named. Plugins declare theirs when the
@@ -38,7 +73,15 @@ export class ToolTierTable implements ToolTiers {
 	readonly #core = new Map(Object.entries(CORE_TOOL_TIERS));
 
 	constructor(operator: Readonly<Record<string, Tier>> = {}) {
-		this.#operator = new Map(Object.entries(operator));
+		const entries = Object.entries(operator);
+		// An old name first, so the new name's own setting overrides it.
+		this.#operator = new Map([
+			...entries.flatMap(([tool, tier]) => {
+				const renamed = RENAMED_TOOLS[tool];
+				return renamed ? [[renamed, tier] as const] : [];
+			}),
+			...entries,
+		]);
 	}
 
 	/**
@@ -46,7 +89,16 @@ export class ToolTierTable implements ToolTiers {
 	 * plugin declaring its own tool again, as when the host retries its start, replaces it.
 	 */
 	declare(plugin: string, tiers: Readonly<Record<string, Tier>>): void {
-		for (const [tool, tier] of Object.entries(tiers)) {
+		const entries = Object.entries(tiers);
+		// An old name is the new name's, declared first so the new name's own tier wins.
+		const named = [
+			...entries.flatMap(([tool, tier]) => {
+				const renamed = RENAMED_TOOLS[tool];
+				return renamed ? [[renamed, tier] as const] : [];
+			}),
+			...entries.filter(([tool]) => !RENAMED_TOOLS[tool]),
+		];
+		for (const [tool, tier] of named) {
 			const other = this.#declared.get(tool);
 			if (other && other.plugin !== plugin)
 				throw new PluginError(
@@ -73,11 +125,13 @@ export class ToolTierTable implements ToolTiers {
 /** The core's tiers with the operator's on top; plugins add theirs through `declare`. */
 export function toolTiers(
 	operator: Readonly<Record<string, Tier>> = {},
+	logger?: Logger,
 ): ToolTierTable {
+	if (logger) warnAliases(Object.keys(operator), logger);
 	return new ToolTierTable(operator);
 }
 
-/** The tools a tier may use, in order; a turn nobody spoke in has the owner's tier. */
+/** The tools a tier may use, in order; a turn's tier is its speaker's, and a turn without one is refused. */
 export function toolsForTier(
 	tools: readonly string[],
 	tier: Tier,

package/src/discord/index.ts

@@ -3,12 +3,20 @@
 
 export type { DiscordServices } from "../core/builtin/discord.ts";
 export { DISCORD } from "../core/builtin/discord.ts";
+export type { DiscordAdapterConfig } from "../core/config/config.ts";
+export type {
+	ChannelContext,
+	ChannelContextMessage,
+	ChannelContextOptions,
+} from "../core/contract/channel-context.ts";
+export { withChannelContext } from "../core/contract/channel-context.ts";
 export type { AgentPanelMessage } from "../core/discord/agent-commands.ts";
 export type {
 	AgentPanel,
 	AgentPanelOptions,
 } from "../core/discord/agent-panel.ts";
 export { agentPanel } from "../core/discord/agent-panel.ts";
+export { CHANNEL_CONTEXT_DEFAULTS } from "../core/discord/channel-context.ts";
 export type { ManagedChannel } from "../core/discord/channel-executor.ts";
 export {
 	fetchManagedChannel,
@@ -33,6 +41,8 @@ export type {
 	ChannelInfo,
 	DiscordConnection,
 } from "../core/discord/connection.ts";
+export { discord } from "../core/discord/discord-adapter.ts";
+export type { DiscordActor } from "../core/discord/discord-owners.ts";
 export type {
 	CommandGuard,
 	CommandRegistrar,

package/src/index.ts

@@ -21,6 +21,14 @@ export {
 	AGENT_TEAM_SERVICE,
 } from "./core/builtin/agent-server.ts";
 export type {
+	AccessConfig,
+	AccessOwnerConfig,
+	AccessTierConfig,
+} from "./core/config/access.ts";
+export type {
+	AdapterConfig,
+	BackgroundConfig,
+	DiscordConfig,
 	Pronouns,
 	RoundtableConfig,
 	TierConfig,
@@ -28,6 +36,7 @@ export type {
 export type {
 	Admission,
 	AttachmentRef,
+	BackgroundRunsAs,
 	BackgroundTarget,
 	BackgroundTurn,
 	ChannelClaim,
@@ -57,6 +66,12 @@ export type { ServiceKey, Services } from "./core/contract/services.ts";
 export { serviceKey } from "./core/contract/services.ts";
 export type { ChatSurface, SurfacePort } from "./core/contract/surface.ts";
 export { channelKey, parseChannelKey } from "./core/contract/surface.ts";
+export type {
+	ConversationRecord,
+	ConversationRegistration,
+	ConversationRegistry,
+	ConversationVisibility,
+} from "./core/conversations/conversation-registry.ts";
 export type { Migration, MigrationReport } from "./core/db/migrations.ts";
 export { migrateDatabase } from "./core/db/migrations.ts";
 export type { ToolContribution, ToolSpec, ToolTurn } from "./core/define.ts";
@@ -85,6 +100,7 @@ export {
 	AgentRunError,
 	ConfigError,
 	DelegationError,
+	IdentityError,
 	MemoryError,
 	ScheduleError,
 } from "./core/domain/errors.ts";
@@ -93,14 +109,8 @@ export type {
 	InterimPosts,
 	InterimTextMode,
 } from "./core/domain/interim.ts";
-export type {
-	Approval,
-	AskOption,
-	OwnerAnswer,
-	OwnerPrompts,
-	OwnerQuestion,
-} from "./core/domain/owner-prompts.ts";
-export type { TurnRequest } from "./core/domain/ports.ts";
+export type { TurnConversation, TurnRequest } from "./core/domain/ports.ts";
+export type { TurnProgress } from "./core/domain/progress.ts";
 export type { DrainOptions } from "./core/drain.ts";
 export {
 	JudgeError,
@@ -117,10 +127,46 @@ export type {
 	ListenerAddress,
 	ListenerConfig,
 } from "./core/http/listeners.ts";
+export type {
+	RouteSocket,
+	WebSocketAccept,
+	WebSocketRoute,
+	WebSocketSendResult,
+} from "./core/http/websocket.ts";
 export type { Locale } from "./core/i18n/index.ts";
+export type {
+	AccessOwner,
+	AccessRules,
+	AccessTier,
+} from "./core/identity/access-policy.ts";
+export type { ActorFacts } from "./core/identity/actor-facts.ts";
+export type { IdentityService } from "./core/identity/identity-service.ts";
+export type {
+	IdentityLink,
+	IdentityRef,
+	LinkSource,
+	Principal,
+	RoleGrant,
+	RoleSource,
+} from "./core/identity/principal-store.ts";
+export { SYSTEM_PRINCIPAL } from "./core/identity/principal-store.ts";
+export type {
+	Approval,
+	AskOption,
+	OwnerAnswer,
+	OwnerPrompts,
+	OwnerQuestion,
+	PromptScope,
+	Prompts,
+} from "./core/interactions/prompts.ts";
+export { promptScope } from "./core/interactions/prompts.ts";
 export type { JudgeModel } from "./core/judging/model-judge.ts";
 export type { LogEntry, LogFn, Logger } from "./core/log.ts";
 export type { ThinkingLevel, ThinkingSetting } from "./core/models.ts";
+export {
+	PERSONAL_TARGET,
+	type PerPrincipalLimits,
+} from "./core/modules/background/personal-target.ts";
 export type {
 	DelegationJob,
 	DelegationOutcome,
@@ -169,6 +215,7 @@ export type {
 	LinkedSessions,
 	Persona,
 	PluginContext,
+	PluginIdentity,
 	PromptSection,
 	PromptTurn,
 	RoundtablePlugin,
@@ -177,7 +224,13 @@ export type {
 	ServiceStartOutcome,
 	TurnEndEvent,
 	TurnEvent,
+	TurnProgressEvent,
 } from "./core/plugin.ts";
+export type {
+	DirectChannelProvider,
+	DirectChannels,
+	DirectReach,
+} from "./core/presence/direct-channels.ts";
 export {
 	attachReplyFile,
 	REPLY_FILE_LIMITS,
@@ -205,9 +258,12 @@ export type {
 export {
 	AGENTS,
 	BACKGROUND_TURNS,
+	CONVERSATIONS,
 	DELEGATION,
+	IDENTITY,
 	MEMORY,
 	PRECHECKS,
+	RUNTIME,
 	SCHEDULES,
 	SKILLS,
 } from "./core/services.ts";
@@ -215,6 +271,7 @@ export type {
 	AgentTurnScope,
 	ChannelKey,
 	SessionContext,
+	SessionConversation,
 	SessionPlan,
 	SessionTool,
 	SessionToolSnapshot,

package/src/kit/compaction.ts

@@ -12,3 +12,16 @@ export {
 	HARD_COMPACT_TOKENS,
 	SOFT_COMPACT_TOKENS,
 } from "../core/runtime/compaction-tiers.ts";
+export type { MemoryView } from "../core/runtime/extensions/private-memory.ts";
+export {
+	hidesPrivateExchange,
+	memoryProjection,
+	privateCompaction,
+	summaryProjection,
+} from "../core/runtime/extensions/private-memory.ts";
+export {
+	bridgeHistoryHidesMemory,
+	carriesMemory,
+	MEMORY_TURN_ENTRY,
+	recordMemoryTurn,
+} from "../core/runtime/reader-history.ts";

package/src/kit/index.ts

@@ -18,13 +18,22 @@ export type {
 	CompactionEngine,
 	CompactionHistory,
 	LatestCompaction,
+	MemoryView,
 } from "./compaction.ts";
 export {
+	bridgeHistoryHidesMemory,
 	COMPACT_HEADROOM_TOKENS,
 	CompactionTiers,
+	carriesMemory,
 	compactionEngine,
 	HARD_COMPACT_TOKENS,
+	hidesPrivateExchange,
+	MEMORY_TURN_ENTRY,
+	memoryProjection,
+	privateCompaction,
+	recordMemoryTurn,
 	SOFT_COMPACT_TOKENS,
+	summaryProjection,
 } from "./compaction.ts";
 export { scrubDiagnostic } from "./diagnostics.ts";
 export type {
@@ -122,6 +131,7 @@ export type {
 	DashboardBoard,
 	DelegationWorker,
 	GroupMessage,
+	Notifier,
 	OwnerNotifier,
 	SpeakerFacts,
 	SpeakerPolicy,

package/src/kit/support.ts

@@ -13,7 +13,7 @@ export type {
 export type { Backlog, GroupMessage } from "../core/agents/agent-rules.ts";
 export type { AgentOps } from "../core/agents/agent-tools.ts";
 export type { ThreadHost } from "../core/discord/dispatch-threads.ts";
-export type { OwnerNotifier } from "../core/domain/ports.ts";
+export type { Notifier, OwnerNotifier } from "../core/domain/ports.ts";
 export type { ThinkingPicker } from "../core/models.ts";
 export type { DelegationWorker } from "../core/modules/delegation/delegator.ts";
 export type { AgentChannelLookup } from "../core/modules/schedules/schedules.ts";

package/src/testing.ts

@@ -3,8 +3,8 @@ import { tmpdir } from "node:os";
 import { join } from "node:path";
 import type { ExtensionAPI } from "@earendil-works/pi-coding-agent";
 import type { SQL } from "bun";
-import { agentClaim, OWNER_TARGET } from "./core/agents/agent-claim.ts";
-import { DISCORD } from "./core/builtin/discord.ts";
+import { agentClaim } from "./core/agents/agent-claim.ts";
+import { DISCORD, type DiscordServices } from "./core/builtin/discord.ts";
 import type {
 	ConversationPort,
 	InboundMessage,
@@ -24,7 +24,10 @@ import type {
 	CommandRegistrar,
 	InteractionContribution,
 } from "./core/discord/interaction-module.ts";
-import { OwnerGuard, ownerRootCommand } from "./core/discord/owner-command.ts";
+import {
+	commandGuard,
+	ownerRootCommand,
+} from "./core/discord/owner-command.ts";
 import type {
 	PendingConfirmation,
 	ReplyFile,
@@ -32,10 +35,12 @@ import type {
 import { NotLinkedError, PluginError } from "./core/errors.ts";
 import { EventBus } from "./core/events.ts";
 import type { HoldCheck } from "./core/holds.ts";
+import { contactsOf } from "./core/identity/identity-view.ts";
 import type { OwnerIdentity } from "./core/identity.ts";
 import { ConfirmationJudge } from "./core/judging/confirmation-judge.ts";
 import { silentLogger } from "./core/log.ts";
 import { ConversationBackgroundTurns } from "./core/modules/background/background-turns.ts";
+import { PERSONAL_TARGET } from "./core/modules/background/personal-target.ts";
 import type {
 	Contribution,
 	HostEnv,
@@ -44,6 +49,7 @@ import type {
 	RoundtablePlugin,
 	TurnEndEvent,
 	TurnEvent,
+	TurnProgressEvent,
 } from "./core/plugin.ts";
 import {
 	collectContributions,
@@ -63,9 +69,17 @@ import {
 	conversationTurns,
 } from "./core/routing/conversation-turns.ts";
 import { surfacePort } from "./core/routing/surface-port.ts";
-import { AGENTS, type AgentServer, BACKGROUND_TURNS } from "./core/services.ts";
+import {
+	AGENTS,
+	type AgentServer,
+	BACKGROUND_TURNS,
+	CONVERSATIONS,
+	IDENTITY,
+	RUNTIME,
+} from "./core/services.ts";
 import type { ChannelKey, SessionContext } from "./core/sessions.ts";
-import { type Speaker, speakerPolicy } from "./core/speakers.ts";
+import { type Speaker, THE_SPEAKER } from "./core/speakers.ts";
+import { mapIdentity } from "./core/testing/map-identity.ts";
 import { type ToolTierTable, toolTiers } from "./core/tool-tiers.ts";
 
 // Fixtures for plugin tests.
@@ -99,6 +113,16 @@ export type {
 	RecordingLogger,
 } from "./core/testing/recording-logger.ts";
 export { recordingLogger } from "./core/testing/recording-logger.ts";
+export type {
+	SurfaceContractAnswerer,
+	SurfaceContractFailure,
+	SurfaceContractSubject,
+	SurfaceObservation,
+} from "./core/testing/surface-contract.ts";
+export {
+	checkSurfaceContract,
+	describeSurfaceContract,
+} from "./core/testing/surface-contract.ts";
 export type { TestHost, TestHostOptions } from "./core/testing/test-host.ts";
 export { testHost } from "./core/testing/test-host.ts";
 export type { FakeThreadHost } from "./core/testing/thread-host.ts";
@@ -139,20 +163,30 @@ export interface FakeDiscord {
 /**
  * A `DISCORD` service for a plugin that adds slash commands, without a Discord connection: the
  * plugin's `commands.add` calls are recorded, and `compose()` shows the tree Discord would get.
- * `rootCommand` is the root command's name, `roundtable` by default. Only `commands` and `guard` are given; reading another member throws.
+ * `rootCommand` is the root command's name, `roundtable` by default. `channelContext` is what
+ * `DISCORD.channelContext` answers, by default no context, as on a host with it off. Only
+ * `commands`, `guard`, and `channelContext` are given; reading another member throws.
  */
 export function fakeDiscord(
-	options: { ownerId?: string; rootCommand?: string } = {},
+	options: {
+		ownerId?: string;
+		rootCommand?: string;
+		channelContext?: DiscordServices["channelContext"];
+	} = {},
 ): FakeDiscord {
 	const collection = new CommandCollection();
 	const { rootCommand = "roundtable" } = options;
-	const guard = new OwnerGuard(
-		options.ownerId ?? "owner",
-		silentLogger(),
-		rootCommand,
-	);
+	const guard = commandGuard({
+		ownerId: options.ownerId ?? "owner",
+		root: rootCommand,
+		logger: silentLogger(),
+	});
 	return {
-		service: servicePair(DISCORD, { commands: collection.registrar, guard }),
+		service: servicePair(DISCORD, {
+			commands: collection.registrar,
+			guard,
+			channelContext: options.channelContext ?? (async () => undefined),
+		}),
 		commands: collection.registrar,
 		guard,
 		added: () => collection.added(),
@@ -177,13 +211,17 @@ export interface TestPluginOptions {
 	surfaces?: readonly ChatSurface[];
 	/**
 	 * What the plugin reads from `context.services`, one `servicePair(KEY, { ... })` per service.
-	 * `servicePair(AGENTS, { runtime })` is the runtime that `context.turns` runs turns on; without
-	 * it, a plugin that fills the `runtime` slot gets the runtime its provider builds. A service
+	 * `servicePair(RUNTIME, runtime)` is the runtime that `context.turns` runs turns on, given
+	 * whole rather than member by member; `servicePair(AGENTS, { runtime })` still works when no
+	 * `RUNTIME` is given. Without either, a plugin that fills the `runtime` slot gets the runtime its
+	 * provider builds, under both keys. A service
 	 * not given here reads as absent to `find`, and `get` names this option, except for what the
 	 * host supplies anyway: `BACKGROUND_TURNS`, the real background turns over the test's router,
 	 * and, once `AGENTS` is given, its `approvals` (the real confirmation judge over
 	 * `providers.judge`, when the test gives a judge). Giving `AGENTS` a `team` adds the agent
-	 * server's own claim to the router, answering as the host's would for the `owner`.
+	 * server's own claim to the router, answering as the host's would for the `owner`. The router
+	 * resolves who wrote each message through a given `IDENTITY`; without one only the `owner` is
+	 * anyone, at the owner tier, their principal their id.
 	 */
 	services?: readonly ServicePair[];
 	/**
@@ -315,6 +353,7 @@ export async function testPlugin(
 			(router ?? unlinked("conversations")).handle(message),
 		background: async (turn) =>
 			(router ?? unlinked("conversations")).background(turn),
+		runsAs: async (turn) => (router ?? unlinked("conversations")).runsAs(turn),
 		target: (name) => (router ?? unlinked("conversations")).target(name),
 		startFresh: async (channel) =>
 			(router ?? unlinked("conversations")).startFresh(channel),
@@ -323,6 +362,11 @@ export async function testPlugin(
 		stop: (channel) => (router ?? unlinked("conversations")).stop(channel),
 		postsInPlace: (channel) =>
 			(router ?? unlinked("conversations")).postsInPlace(channel),
+		owns: (channel) => (router ?? unlinked("conversations")).owns(channel),
+		takesBackground: (channel) =>
+			(router ?? unlinked("conversations")).takesBackground(channel),
+		takesSystemReports: (channel) =>
+			(router ?? unlinked("conversations")).takesSystemReports(channel),
 		...options.conversations,
 	};
 	const surfaces = surfacePort(() =>
@@ -339,6 +383,8 @@ export async function testPlugin(
 			events.push({ name: "turnEnded", turn });
 			bus.sink.turnEnded(turn);
 		},
+		// Delivered to the plugin's handlers, not recorded: a turn reports many.
+		turnProgress: (event: TurnProgressEvent) => bus.sink.turnProgress?.(event),
 		changed: () => {
 			events.push({ name: "changed" });
 			bus.sink.changed();
@@ -350,7 +396,14 @@ export async function testPlugin(
 			linked: () => {
 				if (!linked) unlinked("turns");
 			},
-			runtime: () => services.get(AGENTS).runtime,
+			runtime: () => {
+				const given = services.find(RUNTIME);
+				if (given) return given;
+				const server = services.find(AGENTS);
+				return server ? server.runtime : services.get(RUNTIME);
+			},
+			// A test that gives `CONVERSATIONS` sees each turn record its conversation.
+			registry: () => services.find(CONVERSATIONS),
 			surfaces,
 			events: sink,
 			selection: () => (linked ?? unlinked("sessions")).agentSelection(),
@@ -391,6 +444,8 @@ export async function testPlugin(
 		};
 		runtime = providers.runtime(deps);
 		// The provider's runtime serves `context.turns` unless the test gave another.
+		if (!given.has(RUNTIME.id))
+			given.set(RUNTIME.id, { key: RUNTIME, given: runtime });
 		if (!agents || !("runtime" in agents.given))
 			given.set(AGENTS.id, {
 				key: AGENTS,
@@ -411,8 +466,12 @@ export async function testPlugin(
 				}),
 			} satisfies Partial<AgentServer>,
 		});
+	// A runtime is given whole: a class's private members do not survive the partial service's proxy.
 	for (const { key, given: members } of given.values())
-		services.preset(key, partialService(key, members));
+		services.preset(
+			key,
+			key.id === RUNTIME.id ? members : partialService(key, members),
+		);
 	// The turns nobody wrote run through the same router, unless the test gave others or the plugin provides them.
 	if (
 		!given.has(BACKGROUND_TURNS.id) &&
@@ -427,7 +486,7 @@ export async function testPlugin(
 			}),
 		);
 	services.checkRequires();
-	const context: Omit<PluginContext, "services"> = {
+	const context: Omit<PluginContext, "services" | "directChannels"> = {
 		logger,
 		env,
 		sessions: () => linked ?? unlinked("sessions"),
@@ -460,13 +519,17 @@ export async function testPlugin(
 	const team = given.get(AGENTS.id)?.given;
 	const agentServer = team && "team" in team ? services.get(AGENTS) : undefined;
 	const attachmentDir = mkdtempSync(join(tmpdir(), "roundtable-test-plugin-"));
+	const identity = services.find(IDENTITY);
+	// Who wrote a message, and whom a background turn runs as: by the given IDENTITY, else the owner alone, at the owner tier.
+	const people = mapIdentity({ owners: [owner.id] });
 	router = new ChannelRouter({
+		contacts: identity ? contactsOf(identity) : people,
+		principals: identity ?? people,
 		claims: [
 			...(agentServer
 				? [
 						agentClaim({
 							owner,
-							speakers: speakerPolicy({ owners: [owner.id] }),
 							// SAFETY: the claim reads the concrete team's members (guildId, owns, answerOwner, answerGroup, answerBackground, startFresh), which a test that gives a team for it supplies.
 							team: agentServer.team as unknown as Parameters<
 								typeof agentClaim
@@ -478,6 +541,13 @@ export async function testPlugin(
 							surface: surfaces,
 							attachmentDir: () => attachmentDir,
 							logger,
+							// As on a host: the given DISCORD's channel context, when the test gives one.
+							channelContext: async (message) =>
+								(
+									given.get(DISCORD.id)?.given as
+										| Partial<DiscordServices>
+										| undefined
+								)?.channelContext?.(message),
 						}),
 					]
 				: []),
@@ -485,7 +555,9 @@ export async function testPlugin(
 		],
 		targets: (name) =>
 			registry.backgroundTargets.find((t) => t.name === name) ??
-			(agentServer && name === OWNER_TARGET.name ? OWNER_TARGET : undefined),
+			(agentServer && name === PERSONAL_TARGET.name
+				? PERSONAL_TARGET
+				: undefined),
 		queue,
 		logger,
 		...(options.forwardJoinMs === undefined
@@ -550,6 +622,9 @@ export async function testPlugin(
 				homeChannel: channel,
 				turnChannel: channel,
 				compaction: { wrap: (factory) => factory },
+				conversation: { visibility: "shared" },
+				addressee: THE_SPEAKER,
+				memory: "speaker",
 				speaker: () => runOptions?.speaker,
 				runTask: async () => {
 					throw new Error("test session cannot run tasks");

package/templates/.env.example

@@ -7,7 +7,7 @@ DISCORD_TOKEN=
 DISCORD_GUILD_ID=
 DISCORD_ENTRY_CHANNEL_ID=
 
-# You: the one owner who can change everything (right-click your name, Copy User ID).
+# You: the owner, who can change everything; OWNER_ID is your Discord user id (right-click your name, Copy User ID).
 OWNER_ID=
 OWNER_NAME=