refactor(playbooks): drop unused readiness mapping

readinessFromPick, PlaybookReadiness, and their tests have no
production consumer: the server's agent readiness flows through the
sdk's listAgentsWithReadiness instead. Remove the dead code before
inlining the session runtime.

Change-Id: I131ba28e7fbee9ec2aaf3a44467d27bc590dfd28
Co-developed-by: OpenCode <noreply@opencode.ai>

Author: heimanba

packages/playbooks/src/index.ts

@@ -54,7 +54,6 @@ export {
 	pickPlaybookAgent,
 	PlaybookAgentIdentityMismatchError,
 	playbookIdentityMismatchMessage,
-	readinessFromPick,
 } from "./session-runtime.ts";
 
 export type {
@@ -83,7 +82,6 @@ export type {
 	PlaybookAgentAdapter,
 	PlaybookAgentPick,
 	PlaybookProviderSessionAdapter,
-	PlaybookReadiness,
 	PlaybookSessionEventsAdapter,
 	PlaybookSessionFile,
 	PlaybookSessionRuntime,

packages/playbooks/src/session-runtime.ts

@@ -76,12 +76,6 @@ export type PlaybookAgentPick<TAgent extends RemotePlaybookAgent> = {
 	identityMismatch: boolean;
 };
 
-export type PlaybookReadiness =
-	| { status: "ready"; playbookId: string; remoteAgentId: string }
-	| { status: "missing"; playbookId: string; reason: "not_provisioned" }
-	| { status: "unknown"; playbookId: string; reason: "not_checked" | "check_failed" }
-	| { status: "blocked"; playbookId: string; reason: "identity_mismatch"; message: string };
-
 export interface PlaybookAgentAdapter<TAgent extends RemotePlaybookAgent = RemotePlaybookAgent> {
 	listPlaybookAgents(input: { playbookId: string; includeArchived: boolean }): Promise<TAgent[]>;
 	ensurePlaybookAgent(input: { playbookId: string; model?: string; matched?: TAgent }): Promise<TAgent>;
@@ -159,24 +153,6 @@ export function pickPlaybookAgent<TAgent extends RemotePlaybookAgent>(
 	return { agent, duplicates, identityMismatch: false };
 }
 
-// The single source of the pick→PlaybookReadiness mapping, so identity-drift
-// handling lives in one place.
-export function readinessFromPick<TAgent extends RemotePlaybookAgent>(
-	pick: PlaybookAgentPick<TAgent>,
-	playbookId: string,
-): PlaybookReadiness {
-	if (pick.identityMismatch) {
-		return {
-			status: "blocked",
-			playbookId,
-			reason: "identity_mismatch",
-			message: playbookIdentityMismatchMessage(playbookId),
-		};
-	}
-	if (!pick.agent) return { status: "missing", playbookId, reason: "not_provisioned" };
-	return { status: "ready", playbookId, remoteAgentId: pick.agent.id };
-}
-
 export function createPlaybookSessionRuntime<
 	TDetail,
 	TEvent = unknown,

packages/playbooks/tests/session-runtime.test.ts

@@ -5,9 +5,7 @@ import {
 	PLAYBOOK_METADATA_KEY,
 	PlaybookAgentIdentityMismatchError,
 	pickPlaybookAgent,
-	playbookIdentityMismatchMessage,
 	type RemotePlaybookAgent,
-	readinessFromPick,
 } from "../src/index.ts";
 
 const APP_ID = "agents-webui";
@@ -62,43 +60,6 @@ describe("pickPlaybookAgent", () => {
 	});
 });
 
-describe("readinessFromPick", () => {
-	test("a matched agent is ready and carries its remote id", () => {
-		const pick = pickPlaybookAgent([agent({ id: "agent_ready" })], { playbookId: "designer", appId: APP_ID });
-
-		expect(readinessFromPick(pick, "designer")).toEqual({
-			status: "ready",
-			playbookId: "designer",
-			remoteAgentId: "agent_ready",
-		});
-	});
-
-	test("no candidate is missing/not_provisioned", () => {
-		const pick = pickPlaybookAgent([], { playbookId: "designer", appId: APP_ID });
-
-		expect(readinessFromPick(pick, "designer")).toEqual({
-			status: "missing",
-			playbookId: "designer",
-			reason: "not_provisioned",
-		});
-	});
-
-	test("a same-name unstamped agent is blocked with the shared mismatch message", () => {
-		const pick = pickPlaybookAgent([agent({ metadata: { [PLAYBOOK_APP_METADATA_KEY]: APP_ID } })], {
-			playbookId: "designer",
-			appId: APP_ID,
-			expectedAgentName: "Agents/设计师助手",
-		});
-
-		expect(readinessFromPick(pick, "designer")).toEqual({
-			status: "blocked",
-			playbookId: "designer",
-			reason: "identity_mismatch",
-			message: playbookIdentityMismatchMessage("designer"),
-		});
-	});
-});
-
 describe("createPlaybookSessionRuntime", () => {
 	test("start ensures the selected agent, starts the provider session, attaches events, then returns detail", async () => {
 		const calls: string[] = [];