@sublang/playbook 9.0.0 → 10.0.0

package/reference/sdlc/decide.md

@@ -36,7 +36,6 @@ Do not change code or implement the proposal.
 
 Commit the result as one new commit, following @specs/packages/git.md.
 Make the commit message explain concisely what changed and why.
-Include exactly one final-response line beginning `Commit: `, followed only by the exact commit identity; other final-response content may appear on other lines.
 Coder is <coder-llm>; format the model token in conventional human form.
 ```
 

package/reference/sdlc/decide.playbook/decide.fsm.d.ts

@@ -127,6 +127,13 @@ export declare const decideMachine: import("xstate").StateMachine<DecideContext,
     logic: import("xstate").PromiseActorLogic<PlayerOutput, PlayerInput, import("xstate").EventObject>;
     id: string | undefined;
 }, {
+    type: "playbook.acceptedOutcome";
+    params: {
+        readonly source: string;
+        readonly target: string;
+        readonly acceptedOutcome: string;
+    };
+} | {
     type: "rememberActorError";
     params: import("xstate").NonReducibleUnknown;
 } | {

package/reference/sdlc/decide.playbook/decide.fsm.js

@@ -23,7 +23,6 @@ const COMMIT_CODER_PROMPT = [
     '',
     'Commit the result as one new commit, following @specs/packages/git.md.',
     'Make the commit message explain concisely what changed and why.',
-    'Include exactly one final-response line beginning `Commit: `, followed only by the exact commit identity; other final-response content may appear on other lines.',
     'Coder is <coder-llm>; format the model token in conventional human form.',
 ].join('\n');
 const REVIEW_INPUT_TEMPLATE = [
@@ -59,13 +58,6 @@ function commitOutput(event) {
         !isNonEmptyString(output.latestCommit)) {
         return undefined;
     }
-    const commitLines = output.coderOutput
-        .split('\n')
-        .filter((line) => line.startsWith('Commit: '));
-    if (commitLines.length !== 1 ||
-        commitLines[0] !== `Commit: ${output.latestCommit}`) {
-        return undefined;
-    }
     return {
         coderOutput: output.coderOutput,
         latestCommit: output.latestCommit,
@@ -305,6 +297,7 @@ export const decideMachine = setup({
         }),
     },
     actions: {
+        'playbook.acceptedOutcome': (_args, _params) => undefined,
         copyStartTopic: assign({
             callerTopic: ({ event }) => event.type === 'START_DECIDE' ? event.callerTopic : undefined,
             coderProposal: undefined,
@@ -514,14 +507,24 @@ export const decideMachine = setup({
                                     {
                                         guard: 'needsBossReplyWithQuestion',
                                         target: 'waiting',
-                                        actions: {
-                                            type: 'setPendingBossQuestion',
-                                            params: {
-                                                ...stateMetadata.askCoderProposal,
-                                                resumeStateId: 'askCoderProposal',
-                                                question: '',
+                                        actions: [
+                                            {
+                                                type: 'playbook.acceptedOutcome',
+                                                params: {
+                                                    source: 'askCoderProposal',
+                                                    target: 'waitCoderProposalReply',
+                                                    acceptedOutcome: 'needsBossReply',
+                                                },
                                             },
-                                        },
+                                            {
+                                                type: 'setPendingBossQuestion',
+                                                params: {
+                                                    ...stateMetadata.askCoderProposal,
+                                                    resumeStateId: 'askCoderProposal',
+                                                    question: '',
+                                                },
+                                            },
+                                        ],
                                     },
                                     {
                                         guard: 'needsBossReplyWithoutQuestion',
@@ -535,6 +538,16 @@ export const decideMachine = setup({
                                         guard: 'coderProposed',
                                         target: 'complete',
                                         actions: [
+                                            {
+                                                type: 'playbook.acceptedOutcome',
+                                                params: ({ context }) => ({
+                                                    source: 'askCoderProposal',
+                                                    target: context.stagedReviewerResult === undefined
+                                                        ? 'coderProposalComplete'
+                                                        : 'commitCoderProposal',
+                                                    acceptedOutcome: 'proposed',
+                                                }),
+                                            },
                                             'stageCoderProposal',
                                             {
                                                 type: 'clearBranchBossReplyContext',
@@ -630,14 +643,24 @@ export const decideMachine = setup({
                                     {
                                         guard: 'needsBossReplyWithQuestion',
                                         target: 'waiting',
-                                        actions: {
-                                            type: 'setPendingBossQuestion',
-                                            params: {
-                                                ...stateMetadata.askReviewerProposal,
-                                                resumeStateId: 'askReviewerProposal',
-                                                question: '',
+                                        actions: [
+                                            {
+                                                type: 'playbook.acceptedOutcome',
+                                                params: {
+                                                    source: 'askReviewerProposal',
+                                                    target: 'waitReviewerProposalReply',
+                                                    acceptedOutcome: 'needsBossReply',
+                                                },
                                             },
-                                        },
+                                            {
+                                                type: 'setPendingBossQuestion',
+                                                params: {
+                                                    ...stateMetadata.askReviewerProposal,
+                                                    resumeStateId: 'askReviewerProposal',
+                                                    question: '',
+                                                },
+                                            },
+                                        ],
                                     },
                                     {
                                         guard: 'needsBossReplyWithoutQuestion',
@@ -651,6 +674,16 @@ export const decideMachine = setup({
                                         guard: 'reviewerProposed',
                                         target: 'complete',
                                         actions: [
+                                            {
+                                                type: 'playbook.acceptedOutcome',
+                                                params: ({ context }) => ({
+                                                    source: 'askReviewerProposal',
+                                                    target: context.stagedCoderResult === undefined
+                                                        ? 'reviewerProposalComplete'
+                                                        : 'commitCoderProposal',
+                                                    acceptedOutcome: 'proposed',
+                                                }),
+                                            },
                                             'stageReviewerProposal',
                                             {
                                                 type: 'clearBranchBossReplyContext',
@@ -748,14 +781,24 @@ export const decideMachine = setup({
                     {
                         guard: 'needsBossReplyWithQuestion',
                         target: 'awaitBossReply',
-                        actions: {
-                            type: 'setPendingBossQuestion',
-                            params: {
-                                ...stateMetadata.commitCoderProposal,
-                                resumeStateId: 'commitCoderProposal',
-                                question: '',
+                        actions: [
+                            {
+                                type: 'playbook.acceptedOutcome',
+                                params: {
+                                    source: 'commitCoderProposal',
+                                    target: 'awaitBossReply',
+                                    acceptedOutcome: 'needsBossReply',
+                                },
                             },
-                        },
+                            {
+                                type: 'setPendingBossQuestion',
+                                params: {
+                                    ...stateMetadata.commitCoderProposal,
+                                    resumeStateId: 'commitCoderProposal',
+                                    question: '',
+                                },
+                            },
+                        ],
                     },
                     {
                         guard: 'needsBossReplyWithoutQuestion',
@@ -769,6 +812,14 @@ export const decideMachine = setup({
                         guard: 'committed',
                         target: 'reviewCommit',
                         actions: [
+                            {
+                                type: 'playbook.acceptedOutcome',
+                                params: {
+                                    source: 'commitCoderProposal',
+                                    target: 'reviewCommit',
+                                    acceptedOutcome: 'committed',
+                                },
+                            },
                             'rememberCommit',
                             {
                                 type: 'clearBranchBossReplyContext',

package/reference/sdlc/decide.playbook/decide.fsm.ts

@@ -157,7 +157,6 @@ const COMMIT_CODER_PROMPT = [
   '',
   'Commit the result as one new commit, following @specs/packages/git.md.',
   'Make the commit message explain concisely what changed and why.',
-  'Include exactly one final-response line beginning `Commit: `, followed only by the exact commit identity; other final-response content may appear on other lines.',
   'Coder is <coder-llm>; format the model token in conventional human form.',
 ].join('\n');
 
@@ -216,15 +215,6 @@ function commitOutput(
   ) {
     return undefined;
   }
-  const commitLines = output.coderOutput
-    .split('\n')
-    .filter((line) => line.startsWith('Commit: '));
-  if (
-    commitLines.length !== 1 ||
-    commitLines[0] !== `Commit: ${output.latestCommit}`
-  ) {
-    return undefined;
-  }
   return {
     coderOutput: output.coderOutput,
     latestCommit: output.latestCommit,
@@ -533,6 +523,14 @@ export const decideMachine = setup({
     }),
   },
   actions: {
+    'playbook.acceptedOutcome': (
+      _args,
+      _params: {
+        readonly source: string;
+        readonly target: string;
+        readonly acceptedOutcome: string;
+      },
+    ) => undefined,
     copyStartTopic: assign({
       callerTopic: ({ event }) =>
         event.type === 'START_DECIDE' ? event.callerTopic : undefined,
@@ -758,14 +756,24 @@ export const decideMachine = setup({
                   {
                     guard: 'needsBossReplyWithQuestion',
                     target: 'waiting',
-                    actions: {
-                      type: 'setPendingBossQuestion',
-                      params: {
-                        ...stateMetadata.askCoderProposal,
-                        resumeStateId: 'askCoderProposal',
-                        question: '',
+                    actions: [
+                      {
+                        type: 'playbook.acceptedOutcome',
+                        params: {
+                          source: 'askCoderProposal',
+                          target: 'waitCoderProposalReply',
+                          acceptedOutcome: 'needsBossReply',
+                        },
                       },
-                    },
+                      {
+                        type: 'setPendingBossQuestion',
+                        params: {
+                          ...stateMetadata.askCoderProposal,
+                          resumeStateId: 'askCoderProposal',
+                          question: '',
+                        },
+                      },
+                    ],
                   },
                   {
                     guard: 'needsBossReplyWithoutQuestion',
@@ -779,6 +787,17 @@ export const decideMachine = setup({
                     guard: 'coderProposed',
                     target: 'complete',
                     actions: [
+                      {
+                        type: 'playbook.acceptedOutcome',
+                        params: ({ context }) => ({
+                          source: 'askCoderProposal',
+                          target:
+                            context.stagedReviewerResult === undefined
+                              ? 'coderProposalComplete'
+                              : 'commitCoderProposal',
+                          acceptedOutcome: 'proposed',
+                        }),
+                      },
                       'stageCoderProposal',
                       {
                         type: 'clearBranchBossReplyContext',
@@ -877,14 +896,24 @@ export const decideMachine = setup({
                   {
                     guard: 'needsBossReplyWithQuestion',
                     target: 'waiting',
-                    actions: {
-                      type: 'setPendingBossQuestion',
-                      params: {
-                        ...stateMetadata.askReviewerProposal,
-                        resumeStateId: 'askReviewerProposal',
-                        question: '',
+                    actions: [
+                      {
+                        type: 'playbook.acceptedOutcome',
+                        params: {
+                          source: 'askReviewerProposal',
+                          target: 'waitReviewerProposalReply',
+                          acceptedOutcome: 'needsBossReply',
+                        },
+                      },
+                      {
+                        type: 'setPendingBossQuestion',
+                        params: {
+                          ...stateMetadata.askReviewerProposal,
+                          resumeStateId: 'askReviewerProposal',
+                          question: '',
+                        },
                       },
-                    },
+                    ],
                   },
                   {
                     guard: 'needsBossReplyWithoutQuestion',
@@ -898,6 +927,17 @@ export const decideMachine = setup({
                     guard: 'reviewerProposed',
                     target: 'complete',
                     actions: [
+                      {
+                        type: 'playbook.acceptedOutcome',
+                        params: ({ context }) => ({
+                          source: 'askReviewerProposal',
+                          target:
+                            context.stagedCoderResult === undefined
+                              ? 'reviewerProposalComplete'
+                              : 'commitCoderProposal',
+                          acceptedOutcome: 'proposed',
+                        }),
+                      },
                       'stageReviewerProposal',
                       {
                         type: 'clearBranchBossReplyContext',
@@ -999,14 +1039,24 @@ export const decideMachine = setup({
           {
             guard: 'needsBossReplyWithQuestion',
             target: 'awaitBossReply',
-            actions: {
-              type: 'setPendingBossQuestion',
-              params: {
-                ...stateMetadata.commitCoderProposal,
-                resumeStateId: 'commitCoderProposal',
-                question: '',
+            actions: [
+              {
+                type: 'playbook.acceptedOutcome',
+                params: {
+                  source: 'commitCoderProposal',
+                  target: 'awaitBossReply',
+                  acceptedOutcome: 'needsBossReply',
+                },
               },
-            },
+              {
+                type: 'setPendingBossQuestion',
+                params: {
+                  ...stateMetadata.commitCoderProposal,
+                  resumeStateId: 'commitCoderProposal',
+                  question: '',
+                },
+              },
+            ],
           },
           {
             guard: 'needsBossReplyWithoutQuestion',
@@ -1020,6 +1070,14 @@ export const decideMachine = setup({
             guard: 'committed',
             target: 'reviewCommit',
             actions: [
+              {
+                type: 'playbook.acceptedOutcome',
+                params: {
+                  source: 'commitCoderProposal',
+                  target: 'reviewCommit',
+                  acceptedOutcome: 'committed',
+                },
+              },
               'rememberCommit',
               {
                 type: 'clearBranchBossReplyContext',

package/reference/sdlc/decide.playbook/decide.gears.md

@@ -62,7 +62,6 @@ When both independent proposals are complete, Captain shall prompt Coder:
 >
 > Commit the result as one new commit, following @specs/packages/git.md.
 > Make the commit message explain concisely what changed and why.
-> Include exactly one final-response line beginning `Commit: `, followed only by the exact commit identity; other final-response content may appear on other lines.
 > Coder is <coder-llm>; format the model token in conventional human form.
 
 Results:

package/reference/sdlc/decide.playbook/decide.playbook.d.ts

@@ -1,8 +1,14 @@
-import { type PlayerInput, type PlayerOutput, type DecideEvent, type DecideInput, type PendingBossQuestion } from './decide.fsm.js';
+import { type PlayerInput, type DecideEvent, type DecideInput, type PendingBossQuestion } from './decide.fsm.js';
+import type { PlaybookHostConstructionCapabilities } from '../code.playbook/playbook-captain.js';
 import type { CaptainCallOptions, CaptainResult, JsonValue, NormalizedError, PlayerCallOptions, PlayerResult, PlayerSessionStore, PlaybookCallRequest, PlaybookCallResult, PlaybookCallStart, PlaybookControlAction, PlaybookControlReceipt, PlaybookControlView, PlaybookPendingCall, PlaybookPorts, PlaybookRunResult, PlaybookRuntime, PlaybookRuntimeFactory, PlaybookRuntimeSnapshot, PlaybookSession, PlaybookState, PlaybookStateValue, PlaybookTraceEvent, PlaybookTraceType } from '@sublang/playbook/runtime';
 export type { CaptainCallOptions, CaptainResult, JsonValue, NormalizedError, PlayerCallOptions, PlayerResult, PlayerSessionStore, PlaybookCallRequest, PlaybookCallResult, PlaybookCallStart, PlaybookControlAction, PlaybookControlReceipt, PlaybookControlView, PlaybookPendingCall, PlaybookPorts, PlaybookRunResult, PlaybookRuntime, PlaybookRuntimeFactory, PlaybookRuntimeSnapshot, PlaybookSession, PlaybookState, PlaybookStateValue, PlaybookTraceEvent, PlaybookTraceType, };
 type RoleId = 'coder' | 'reviewer';
 export type PlaybookRuntimeOptions = DecideInput;
+export type DecidePlaybookHostCapabilities = PlaybookHostConstructionCapabilities;
+export interface DecidePlaybookRuntimeConstruction {
+    readonly configuredOptions: PlaybookRuntimeOptions;
+    readonly hostCapabilities: DecidePlaybookHostCapabilities;
+}
 type PromptIdentity = (roleId: RoleId) => string;
 declare function composePlayerPrompt(input: PlayerInput, promptIdentity: PromptIdentity): string;
 declare function requiredFieldsFor(description: string): string[];
@@ -14,8 +20,10 @@ declare function buildClassifierPrompt(text: string, ctx: {
 declare function parseClassification(raw: string, text: string, pendingQuestionIds?: readonly string[]): DecideEvent | {
     type: 'NO_ACTION';
 } | null;
-declare function buildAdjudicatorPrompt(input: PlayerInput, playerOutput: string): string;
-declare function parseAdjudication(raw: string, input: PlayerInput, finalText: string): PlayerOutput;
+declare function buildAdjudicatorPrompt(input: PlayerInput, playerOutput: string, correction?: {
+    readonly reply: string;
+    readonly error: string;
+}): string;
 declare function combineSignals(a: AbortSignal | undefined, b: AbortSignal | undefined): AbortSignal;
 declare function normalizeErrorCompact(err: unknown): {
     name: string;
@@ -28,7 +36,7 @@ declare function normalizeErrorFull(err: unknown): {
 } | undefined;
 declare function pendingQuestionsFromContext(context: Record<string, unknown>): PendingBossQuestion[];
 declare function pendingQuestionsForState(state: PlaybookState, context: Record<string, unknown>): PendingBossQuestion[];
-export declare const createPlaybookRuntime: PlaybookRuntimeFactory<PlaybookRuntimeOptions>;
+export declare const createPlaybookRuntime: PlaybookRuntimeFactory<DecidePlaybookRuntimeConstruction>;
 export declare const _internal: {
     composePlayerPrompt: typeof composePlayerPrompt;
     requiredFieldsFor: typeof requiredFieldsFor;
@@ -36,7 +44,6 @@ export declare const _internal: {
     buildClassifierPrompt: typeof buildClassifierPrompt;
     parseClassification: typeof parseClassification;
     buildAdjudicatorPrompt: typeof buildAdjudicatorPrompt;
-    parseAdjudication: typeof parseAdjudication;
     combineSignals: typeof combineSignals;
     pendingQuestionsFromContext: typeof pendingQuestionsFromContext;
     pendingQuestionsForState: typeof pendingQuestionsForState;
@@ -59,6 +66,7 @@ export declare const _internal: {
     ROLE_STATE_IDS: ReadonlySet<string>;
     VERBATIM_PAYLOAD_FIELDS: ReadonlySet<string>;
     BOSS_INTERRUPT_TARGETS: readonly ["independentProposals"];
+    UNFINISHED_FINAL_STATE_IDS: ReadonlySet<string>;
     CONTINUATION_PREAMBLE: string;
     TELEMETRY_TOPIC: string;
 };