dialai 1.2.0 → 1.3.0

package/.claude/specs/proposal-metadata-update.md

@@ -0,0 +1,211 @@
+# Strategy Result Operational Metrics
+
+## Executive Summary
+
+Extend `ProposerStrategyResult` with optional operational fields (`costUSD`, `latencyMsec`, `numInputTokens`, `numOutputTokens`) so that when `submitProposal` invokes a strategy internally, the resulting proposal carries the same cost and latency data it would carry if the caller had submitted explicitly.
+
+## Objective
+
+Close the data gap between the two `submitProposal` paths so both produce proposals with full operational metrics.
+
+## In Scope
+
+- Extend `ProposerStrategyResult` in `types.ts` with four optional fields
+- Update `executeProposerLlm` in `llm.ts` to capture timing and token usage from `callLlm`
+- Update `invokeProposerStrategy` in `api.ts` to use `ProposerStrategyResult` as its return type
+- Update `submitProposal` in `api.ts` to merge strategy-returned metrics into the proposal
+- Unit tests for the new behavior
+- Verify all existing tests still pass
+
+## Out of Scope
+
+- Changing `callLlm`'s return type or signature
+- Changing `ArbiterStrategyResult`
+- Adding `costUSD` calculation to `executeProposerLlm`
+- Webhook proposer timing (`executeProposerWebhook`)
+- Changes to `executeContextWebhookProposer` (it delegates to `executeProposerLlm`; metrics cascade automatically)
+
+## Assumptions and Constraints
+
+- `callLlm` returns `{ content: string; usage?: { prompt_tokens?: number; completion_tokens?: number } }` — confirmed at `llm.ts:110-115`
+- `executeProposerLlm` currently discards `usage` and does not measure wall-clock time
+- `submitProposal` opts values (`costUSD`, `latencyMsec`, etc.) take precedence over strategy-returned values when both present
+- `ProposerStrategyResult` is the return type for `strategyFn`, `executeProposerLlm`, `executeProposerWebhook`, and `executeContextWebhookProposer`
+- Custom `strategyFn` implementations are not required to return the new fields (they are optional)
+- `invokeProposerStrategy` is a private function in `api.ts` with explicit return type that must be updated
+
+## Files to Modify
+
+| File | Action |
+|------|--------|
+| `src/dialai/types.ts` | Add 4 optional fields to `ProposerStrategyResult` |
+| `src/dialai/llm.ts` | Update `executeProposerLlm` to capture timing + usage |
+| `src/dialai/api.ts` | Update `invokeProposerStrategy` return type; update `submitProposal` to merge metrics |
+| `tests/unit/submit-proposal.test.ts` | Add 3 new tests |
+| `src/dialai/llm.test.ts` | Add 1 new test |
+
+## Files to Read (do not modify)
+
+| File | Why |
+|------|-----|
+| `src/dialai/llm-audit.test.ts` | Mock patterns for `callLlm` |
+| `src/dialai/store.ts` | Understand `Proposal` storage |
+
+## Implementation Plan
+
+### Phase 1: Extend the type
+
+In `src/dialai/types.ts`, add four optional fields to `ProposerStrategyResult` (after `reasoning`):
+
+```typescript
+export interface ProposerStrategyResult {
+  transitionName: string;
+  toState: string;
+  reasoning: string;
+  /** Cost in USD to produce this result, if known */
+  costUSD?: number;
+  /** Wall-clock time in milliseconds, if measured */
+  latencyMsec?: number;
+  /** Input tokens consumed, if known */
+  numInputTokens?: number;
+  /** Output tokens consumed, if known */
+  numOutputTokens?: number;
+}
+```
+
+**Validate:** Run `npm run typecheck`. This is backward-compatible; expect no errors.
+
+### Phase 2: Update `executeProposerLlm`
+
+In `src/dialai/llm.ts`, the function currently (lines 238-259) calls `callLlm`, parses the JSON, and returns the parsed result directly. Update it to:
+
+1. Record `Date.now()` before calling `callLlm`
+2. Compute `latencyMsec` after `callLlm` returns
+3. Extract `usage.prompt_tokens` and `usage.completion_tokens` from the result
+4. Return an explicit object with the three existing fields plus the three new measurement fields
+
+The try/catch structure must be preserved. The `as ProposerStrategyResult` cast on the parsed JSON should be removed in favor of explicit field extraction (to avoid casting an object that lacks the metric fields into the type that now declares them).
+
+**Validate:** Run `npm test`. All existing tests must pass.
+
+### Phase 3: Update `invokeProposerStrategy` and `submitProposal`
+
+**Step 3a:** In `src/dialai/api.ts`, change the return type of `invokeProposerStrategy` (line 569) from `Promise<{ transitionName: string; toState: string; reasoning: string }>` to `Promise<ProposerStrategyResult>`. Import `ProposerStrategyResult` if not already imported. No other changes to this function are needed — it already delegates to functions that return `ProposerStrategyResult`.
+
+**Step 3b:** In `submitProposal`, update the strategy-invocation branch. Before the `if (!finalTransitionName)` block, declare:
+
+```typescript
+let finalCostUSD = costUSD;
+let finalLatencyMsec = latencyMsec;
+let finalNumInputTokens = numInputTokens;
+let finalNumOutputTokens = numOutputTokens;
+```
+
+Inside the block, after setting `finalReasoning`, add:
+
+```typescript
+finalCostUSD = finalCostUSD ?? result.costUSD;
+finalLatencyMsec = finalLatencyMsec ?? result.latencyMsec;
+finalNumInputTokens = finalNumInputTokens ?? result.numInputTokens;
+finalNumOutputTokens = finalNumOutputTokens ?? result.numOutputTokens;
+```
+
+Then use the `final*` variables in the proposal construction object instead of the raw `costUSD`, `latencyMsec`, `numInputTokens`, `numOutputTokens`.
+
+**Validate:** Run `npm run typecheck && npm test`. All must pass.
+
+### Phase 4: Tests
+
+Add these tests:
+
+**In `src/dialai/llm.test.ts`:**
+
+1. **`executeProposerLlm` returns metrics.** Mock `callLlm` to return `{ content: '{"transitionName":"close","toState":"closed","reasoning":"test"}', usage: { prompt_tokens: 100, completion_tokens: 50 } }`. Assert the result includes `latencyMsec` (a number > 0), `numInputTokens` (100), `numOutputTokens` (50).
+
+**In `tests/unit/submit-proposal.test.ts`:**
+
+2. **Strategy metrics flow to proposal.** Register a proposer with a `strategyFn` that returns `{ transitionName: "t", toState: "s", reasoning: "r", latencyMsec: 500, numInputTokens: 200, numOutputTokens: 80 }`. Call `submitProposal({ sessionId, specialistId })` without `transitionName`. Assert the stored proposal has `latencyMsec: 500`, `numInputTokens: 200`, `numOutputTokens: 80`.
+
+3. **Opts override strategy metrics.** Same setup as test 2, but call `submitProposal({ sessionId, specialistId, latencyMsec: 999 })`. Assert proposal has `latencyMsec: 999` (opts wins), `numInputTokens: 200` (strategy fills gap).
+
+4. **Backward compat — no metrics from strategy.** Register a proposer with a `strategyFn` that returns `{ transitionName: "t", toState: "s", reasoning: "r" }` (no metric fields). Call `submitProposal` without `transitionName`. Assert the proposal is created with `latencyMsec: undefined`, `numInputTokens: undefined`. No crash.
+
+**Validate:** Run `npm run ci`. All must pass (typecheck + lint + test + build).
+
+## Acceptance Criteria
+
+### Functional
+
+- `ProposerStrategyResult` has optional `costUSD`, `latencyMsec`, `numInputTokens`, `numOutputTokens` fields
+- `executeProposerLlm` returns `latencyMsec` and token counts from the underlying `callLlm` response
+- `invokeProposerStrategy` return type is `ProposerStrategyResult`
+- `submitProposal` populates proposal metrics from strategy result when caller does not provide them via opts
+- Caller-provided opts values always take precedence over strategy-returned values
+
+### Quality
+
+- Existing `strategyFn` implementations that do not return the new fields continue to work
+- All existing tests pass without modification
+- New tests cover: metrics flow, opts precedence, backward compat
+
+### Operational
+
+- `npm run ci` passes (typecheck + lint + test + build)
+
+## Failure and Recovery Rules
+
+1. Run `npm run typecheck` after Phase 1. Run `npm test` after Phases 2 and 3. Run `npm run ci` after Phase 4.
+2. If adding fields to `ProposerStrategyResult` causes type errors, verify they are marked optional. If a destructure or spread assumes the exact shape, update it.
+3. If `executeProposerLlm` tests fail because `callLlm` is hard to mock, follow the pattern in `src/dialai/llm-audit.test.ts` (mock `globalThis.fetch`, set `OPENROUTER_API_TOKEN`).
+4. If changing `invokeProposerStrategy` return type causes downstream type errors, check that `submitProposal` correctly accesses the new optional fields with nullish coalescing.
+5. Do not declare completion while any acceptance criterion is unmet.
+
+## Completion Signal
+
+Output exactly `COMPLETE` only when:
+- All acceptance criteria are met
+- `npm run ci` passes
+- No blocking errors remain
+
+## Ralph Prompt Draft
+
+```
+Implement operational metrics on ProposerStrategyResult.
+
+Spec location: .claude/specs/proposal-metadata-update.md
+
+Constraints:
+- Do not change callLlm's return type or signature
+- Do not change ArbiterStrategyResult
+- Do not modify executeContextWebhookProposer (metrics cascade from executeProposerLlm)
+- All new fields on ProposerStrategyResult must be optional
+
+Files to modify:
+- src/dialai/types.ts (ProposerStrategyResult)
+- src/dialai/llm.ts (executeProposerLlm)
+- src/dialai/api.ts (invokeProposerStrategy return type, submitProposal merge logic)
+- tests/unit/submit-proposal.test.ts (3 new tests)
+- src/dialai/llm.test.ts (1 new test)
+
+Required deliverables:
+- ProposerStrategyResult has optional costUSD, latencyMsec, numInputTokens, numOutputTokens
+- executeProposerLlm returns latencyMsec and token counts from callLlm
+- invokeProposerStrategy return type updated to ProposerStrategyResult
+- submitProposal merges strategy metrics into proposal (opts take precedence)
+- 4 new tests covering metrics flow, opts precedence, and backward compat
+
+Acceptance criteria:
+- npm run ci passes (typecheck + lint + test + build)
+- All existing tests pass without modification
+- New tests verify: (1) executeProposerLlm returns metrics, (2) strategy metrics flow to proposal, (3) opts override strategy metrics, (4) missing metrics don't crash
+
+Execution rules:
+1. Work in phase order: types → llm.ts → api.ts → tests
+2. Run npm run typecheck after Phase 1
+3. Run npm test after Phases 2 and 3
+4. Run npm run ci after Phase 4
+5. If tests fail, inspect and fix the root cause before continuing
+6. If blocked after 3 attempts on the same issue, report the blocker
+
+Output exactly COMPLETE when all criteria are met.
+```

package/.claude/specs/submit-proposal-cleanup.md

@@ -0,0 +1,319 @@
+# submitProposal: Remove `final*` Prefix Pattern
+
+## Executive Summary
+
+Refactor `submitProposal` in `api.ts` to drop the `final*` variable prefix introduced in commit `0f273a9`. Instead of destructuring all opts fields as `const` and then creating parallel `let final*` variables, destructure the immutable fields as `const` and declare the mutable ones as `let` directly. Also bring `metaJson` into the same pattern so strategy-returned `metaJson` flows to the proposal.
+
+## Objective
+
+Simplify `submitProposal` so the "opts wins, strategy fills gaps" merging logic reads naturally without the `final*` indirection, and add `metaJson` to the merge pattern so it works identically to the other fields.
+
+## In Scope
+
+- Refactor variable declarations in `submitProposal`
+- Bring `metaJson` into the same `let` / `??=` pattern as `costUSD`, `latencyMsec`, etc.
+- Update existing tests if assertions reference the old behavior
+- Add `metaJson` to `ProposerStrategyResult` (if not already there from the enriched-transitions spec)
+
+## Out of Scope
+
+- Changes to `submitArbitration` (it doesn't have the `final*` pattern)
+- Changes to any other function
+
+## Assumptions and Constraints
+
+- `ProposerStrategyResult` must have an optional `metaJson` field. If the enriched-transitions spec runs first, it will already exist. If this spec runs first, add it.
+- The merge semantics are: caller-provided opts value wins; strategy-returned value fills the gap.
+- `transitionName`, `toState`, and `reasoning` also follow the same pattern — they just don't use the `final*` prefix inconsistently (they already do use it). Clean those up too.
+
+## Files to Modify
+
+| File | Action |
+|------|--------|
+| `src/dialai/api.ts` | Refactor `submitProposal` variable declarations |
+| `src/dialai/types.ts` | Add `metaJson` to `ProposerStrategyResult` if not already present |
+| `tests/unit/submit-proposal.test.ts` | Add tests for metaJson merging |
+| `website/docs/api/types.md` | Add `metaJson` field to `ProposerStrategyResult` docs |
+| `.claude/skills/dial-machine/references/api-reference.md` | Add `metaJson` field to `ProposerStrategyResult` docs |
+
+## Implementation Plan
+
+### Phase 1: Refactor `submitProposal`
+
+**Before (current code, lines 639-717):**
+
+```typescript
+export async function submitProposal(
+  opts: SubmitProposalOptions
+): Promise<Proposal> {
+  const {
+    sessionId,
+    specialistId,
+    roundId,
+    transitionName,
+    reasoning,
+    metaJson,
+    costUSD,
+    latencyMsec,
+    numInputTokens,
+    numOutputTokens,
+  } = opts;
+  // ...
+  let finalTransitionName = transitionName;
+  let finalToState: string | undefined;
+  let finalReasoning = reasoning;
+  let finalCostUSD = costUSD;
+  let finalLatencyMsec = latencyMsec;
+  let finalNumInputTokens = numInputTokens;
+  let finalNumOutputTokens = numOutputTokens;
+
+  if (!finalTransitionName) {
+    // ...
+    finalTransitionName = result.transitionName;
+    finalToState = result.toState;
+    finalReasoning = finalReasoning ?? result.reasoning;
+    finalCostUSD = finalCostUSD ?? result.costUSD;
+    // ...
+  }
+
+  const proposal: Proposal = {
+    // ...
+    transitionName: finalTransitionName,
+    toState: finalToState,
+    reasoning: finalReasoning ?? "",
+    metaJson,                          // <-- BUG: ignores strategy metaJson
+    costUSD: finalCostUSD,
+    // ...
+  };
+}
+```
+
+**After:**
+
+```typescript
+export async function submitProposal(
+  opts: SubmitProposalOptions
+): Promise<Proposal> {
+  const { sessionId, specialistId, roundId } = opts;
+
+  let transitionName = opts.transitionName;
+  let toState: string | undefined;
+  let reasoning = opts.reasoning;
+  let metaJson = opts.metaJson;
+  let costUSD = opts.costUSD;
+  let latencyMsec = opts.latencyMsec;
+  let numInputTokens = opts.numInputTokens;
+  let numOutputTokens = opts.numOutputTokens;
+
+  const session = await getSession(sessionId);
+  const specialist = await getStore().getSpecialist(specialistId);
+
+  if (!specialist) {
+    throw new Error(`Specialist not found: ${specialistId}`);
+  }
+
+  if (specialist.role !== "proposer") {
+    throw new Error(`Specialist ${specialistId} is not a proposer`);
+  }
+
+  const proposer = specialist;
+  const effectiveRoundId = roundId ?? session.currentRoundId;
+  const isHuman = proposer.isHuman ?? false;
+
+  if (!transitionName) {
+    const ctx = buildProposerContext(session);
+    const result = await invokeProposerStrategy(proposer, ctx);
+    transitionName = result.transitionName;
+    toState = result.toState;
+    reasoning = reasoning ?? result.reasoning;
+    metaJson = metaJson ?? result.metaJson;
+    costUSD = costUSD ?? result.costUSD;
+    latencyMsec = latencyMsec ?? result.latencyMsec;
+    numInputTokens = numInputTokens ?? result.numInputTokens;
+    numOutputTokens = numOutputTokens ?? result.numOutputTokens;
+  } else {
+    const currentStateDef = session.machine.states[session.currentState];
+    if (!currentStateDef?.transitions?.[transitionName]) {
+      throw new Error(
+        `Invalid transition "${transitionName}" from state "${session.currentState}"`
+      );
+    }
+    toState = currentStateDef.transitions[transitionName];
+  }
+
+  const proposal: Proposal = {
+    proposalId: generateUUID(),
+    sessionId,
+    roundId: effectiveRoundId,
+    specialistId,
+    isHuman,
+    transitionName,
+    toState,
+    reasoning: reasoning ?? "",
+    metaJson,
+    costUSD,
+    latencyMsec,
+    numInputTokens,
+    numOutputTokens,
+    createdAt: new Date(),
+  };
+
+  await getStore().setProposal(proposal);
+  return proposal;
+}
+```
+
+Key changes:
+1. `const` destructuring only for truly immutable fields: `sessionId`, `specialistId`, `roundId`
+2. All mutable fields declared as `let` from `opts.*`
+3. No `final*` prefix anywhere
+4. `metaJson` follows the same `??=` pattern as the metrics fields
+5. `toState` replaces `finalToState` (was already the only one using a different name)
+
+### Phase 2: Add `metaJson` to `ProposerStrategyResult`
+
+In `types.ts`, add to `ProposerStrategyResult` if not already present:
+
+```typescript
+/** Structured metadata from the strategy (e.g., tool arguments) */
+metaJson?: Record<string, unknown>;
+```
+
+**Validate:** `npm run typecheck`
+
+### Phase 3: Tests
+
+**In `tests/unit/submit-proposal.test.ts`:**
+
+**Test 1: strategy-returned metaJson flows to proposal**
+
+```
+Setup: strategyFn returns { transitionName: "approve", toState: "approved", reasoning: "r", metaJson: { key: "from-strategy" } }
+Call:   submitProposal({ sessionId, specialistId }) — no metaJson in opts
+Assert: proposal.metaJson deep-equals { key: "from-strategy" }
+```
+
+**Test 2: caller-provided metaJson takes precedence over strategy metaJson**
+
+```
+Setup: same strategyFn returning metaJson: { key: "from-strategy" }
+Call:   submitProposal({ sessionId, specialistId, metaJson: { key: "from-caller" } })
+Assert: proposal.metaJson deep-equals { key: "from-caller" }
+```
+
+**Test 3: no metaJson from either source**
+
+```
+Setup: strategyFn returns { transitionName: "approve", toState: "approved", reasoning: "r" } (no metaJson)
+Call:   submitProposal({ sessionId, specialistId })
+Assert: proposal.metaJson is undefined
+Assert: no crash
+```
+
+Existing tests (`strategy metrics flow to proposal`, `opts override strategy metrics`, `backward compat — no metrics from strategy`) must continue to pass without modification.
+
+**Validate:** `npm run ci`
+
+### Phase 4: Update Documentation
+
+**`website/docs/api/types.md` — `ProposerStrategyResult` section:**
+
+Add `metaJson` to the interface listing. Currently shows:
+
+```typescript
+interface ProposerStrategyResult {
+  transitionName: string;
+  toState: string;
+  reasoning: string;
+  costUSD?: number;
+  latencyMsec?: number;
+  numInputTokens?: number;
+  numOutputTokens?: number;
+}
+```
+
+Add after `reasoning`:
+
+```typescript
+  metaJson?: Record<string, unknown>; // Structured metadata (e.g., tool arguments)
+```
+
+Update the prose above the code block to mention metaJson: "The optional metric and metadata fields are merged into the resulting `Proposal` when the strategy is invoked via `submitProposal`. Values passed via `SubmitProposalOptions` take precedence over strategy-returned values."
+
+**`.claude/skills/dial-machine/references/api-reference.md` — same change:**
+
+Add `metaJson` to `ProposerStrategyResult` in the same position.
+
+## Acceptance Criteria
+
+### Functional
+
+- `submitProposal` has no variables with the `final` prefix
+- Immutable fields (`sessionId`, `specialistId`, `roundId`) are `const` destructured
+- Mutable fields (`transitionName`, `reasoning`, `metaJson`, `costUSD`, `latencyMsec`, `numInputTokens`, `numOutputTokens`, `toState`) are `let` declarations
+- `metaJson` merging follows the same `opts ?? strategy` pattern as the metrics fields
+- Strategy-returned `metaJson` flows to the proposal when caller doesn't provide it
+- Caller-provided `metaJson` takes precedence over strategy-returned `metaJson`
+
+### Quality
+
+- All existing `submit-proposal.test.ts` tests pass unchanged
+- All existing tests across the project pass
+- `npm run typecheck` passes
+- `npm run lint` passes
+
+### Operational
+
+- `npm run ci` passes
+
+## Failure and Recovery Rules
+
+1. If `npm run lint` complains about variable shadowing (the `let` declarations shadow the destructured names), that means the `const` destructuring still includes those fields. Remove them from the destructuring.
+2. If existing tests break, check that the proposal construction object uses the same field names (no `final*` prefix).
+3. The `toState` line in the `else` branch (`toState = currentStateDef.transitions[transitionName]`) will need updating if the enriched-transitions spec has already run (it would return a `TransitionDefinition` instead of a string). If so, use `typeof` check.
+
+## Completion Signal
+
+Output exactly `COMPLETE` only when:
+- All acceptance criteria are met
+- `npm run ci` passes
+- No `final*` variables remain in `submitProposal`
+
+## Ralph Prompt Draft
+
+```
+Refactor submitProposal in src/dialai/api.ts to remove the final* variable prefix.
+
+Spec location: .claude/specs/submit-proposal-cleanup.md
+
+The current code destructures all opts fields as const, then creates parallel
+let final* variables for the mutable ones. This is noisy and metaJson was
+missed (line 708 uses raw opts.metaJson, so strategy-returned metaJson never
+reaches the proposal).
+
+Required changes:
+- const destructure only: sessionId, specialistId, roundId
+- let declarations for: transitionName, toState, reasoning, metaJson, costUSD,
+  latencyMsec, numInputTokens, numOutputTokens (initialized from opts.*)
+- Strategy merge block: field = field ?? result.field (no final prefix)
+- metaJson follows the same pattern as the metrics fields
+- Proposal construction uses the let variables directly (no final prefix)
+- Add metaJson to ProposerStrategyResult in types.ts if not already present
+
+Tests to add in tests/unit/submit-proposal.test.ts:
+1. Strategy-returned metaJson flows to proposal when caller omits it
+2. Caller-provided metaJson wins over strategy-returned metaJson
+3. No metaJson from either source — proposal.metaJson is undefined
+
+Docs to update:
+- website/docs/api/types.md — add metaJson to ProposerStrategyResult
+- .claude/skills/dial-machine/references/api-reference.md — same
+
+Acceptance criteria:
+- No final* variables in submitProposal
+- metaJson merges with same opts-wins semantics as costUSD etc.
+- All existing tests pass unchanged
+- npm run ci passes
+
+Output exactly COMPLETE when all criteria are met.
+```

package/dist/dialai/api.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"api.d.ts","sourceRoot":"","sources":["../../src/dialai/api.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAaH,OAAO,KAAK,EACV,iBAAiB,EACjB,OAAO,EACP,QAAQ,EACR,QAAQ,EACR,OAAO,EACP,UAAU,EACV,uBAAuB,EACvB,sBAAsB,EACtB,eAAe,EACf,sBAAsB,EACtB,cAAc,EACd,eAAe,EACf,iBAAiB,EACjB,eAAe,EAGf,qBAAqB,EACrB,wBAAwB,EACzB,MAAM,YAAY,CAAC;AAMpB;;;;;GAKG;AACH,wBAAsB,aAAa,CACjC,OAAO,EAAE,iBAAiB,EAC1B,QAAQ,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,GACjC,OAAO,CAAC,OAAO,CAAC,CAgDlB;AAED;;;;;;GAMG;AACH,wBAAsB,UAAU,CAAC,SAAS,EAAE,MAAM,GAAG,OAAO,CAAC,OAAO,CAAC,CAMpE;AAED;;;;GAIG;AACH,wBAAsB,WAAW,IAAI,OAAO,CAAC,OAAO,EAAE,CAAC,CAEtD;AAyID;;;;;GAKG;AACH,wBAAsB,gBAAgB,CACpC,IAAI,EAAE,uBAAuB,GAC5B,OAAO,CAAC,QAAQ,CAAC,CA4BnB;AAED;;;;;GAKG;AACH,wBAAsB,eAAe,CACnC,IAAI,EAAE,sBAAsB,GAC3B,OAAO,CAAC,OAAO,CAAC,CAwBlB;AAED;;GAEG;AACH,wBAAsB,aAAa,CAAC,YAAY,EAAE,MAAM,GAAG,OAAO,CAAC,CAAC,UAAU,GAAG,OAAO,CAAC,GAAG,SAAS,CAAC,CAErG;AAED;;GAEG;AACH,wBAAsB,YAAY,CAAC,WAAW,EAAE,MAAM,GAAG,OAAO,CAAC,QAAQ,EAAE,CAAC,CAG3E;AAED;;GAEG;AACH,wBAAsB,UAAU,CAAC,WAAW,EAAE,MAAM,GAAG,OAAO,CAAC,OAAO,GAAG,SAAS,CAAC,CAGlF;AAMD;;GAEG;AACH,wBAAsB,gBAAgB,CAAC,YAAY,EAAE,MAAM,GAAG,OAAO,CAAC,IAAI,CAAC,CAO1E;AAED;;GAEG;AACH,wBAAsB,iBAAiB,CAAC,YAAY,EAAE,MAAM,GAAG,OAAO,CAAC,IAAI,CAAC,CAO3E;AAED;;GAEG;AACH,wBAAsB,mBAAmB,CAAC,WAAW,EAAE,MAAM,GAAG,OAAO,CAAC,QAAQ,EAAE,CAAC,CAGlF;AAED;;GAEG;AACH,wBAAsB,iBAAiB,CAAC,WAAW,EAAE,MAAM,GAAG,OAAO,CAAC,OAAO,GAAG,SAAS,CAAC,CAIzF;AAMD;;;;GAIG;AACH,wBAAsB,oBAAoB,CAAC,OAAO,EAAE,OAAO,GAAG,OAAO,CAAC,QAAQ,EAAE,CAAC,CAchF;AAED;;;GAGG;AACH,wBAAsB,2BAA2B,CAAC,OAAO,EAAE,OAAO,GAAG,OAAO,CAAC,QAAQ,EAAE,CAAC,CAgBvF;AAED;;;GAGG;AACH,wBAAsB,kBAAkB,CAAC,OAAO,EAAE,OAAO,GAAG,OAAO,CAAC,OAAO,GAAG,SAAS,CAAC,CAevF;AAED;;;GAGG;AACH,wBAAsB,qBAAqB,CAAC,OAAO,EAAE,OAAO,GAAG,OAAO,CAAC,MAAM,GAAG,SAAS,CAAC,CAazF;AA2CD;;GAEG;AACH,wBAAgB,uBAAuB,CACrC,QAAQ,EAAE,QAAQ,GACjB,CAAC,CAAC,GAAG,EAAE,eAAe,KAAK,OAAO,CAAC,sBAAsB,CAAC,CAAC,GAAG,IAAI,CAUpE;AAED;;GAEG;AACH,wBAAgB,sBAAsB,CACpC,OAAO,EAAE,OAAO,GACf,CAAC,CAAC,GAAG,EAAE,cAAc,KAAK,OAAO,CAAC,eAAe,CAAC,CAAC,GAAG,IAAI,CAU5D;AAyED;;;GAGG;AACH,wBAAsB,cAAc,CAClC,IAAI,EAAE,qBAAqB,GAC1B,OAAO,CAAC,QAAQ,CAAC,CA6EnB;AAED;;GAEG;AACH,wBAAsB,oBAAoB,CACxC,SAAS,EAAE,MAAM,EACjB,OAAO,EAAE,MAAM,GACd,OAAO,CAAC,QAAQ,EAAE,CAAC,CAErB;AAED;;;GAGG;AACH,wBAAsB,iBAAiB,CACrC,SAAS,EAAE,MAAM,GAChB,OAAO,CAAC,eAAe,CAAC,CA6B1B;AAED;;;GAGG;AACH,wBAAgB,mBAAmB,CACjC,cAAc,EAAE,MAAM,EACtB,gBAAgB,EAAE,MAAM,EACxB,OAAO,EAAE,OAAO,EAChB,cAAc,EAAE,MAAM,GAAG,SAAS,EAClC,uBAAuB,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,GAAG,SAAS,EAC3D,aAAa,EAAE,MAAM,EACrB,YAAY,EAAE,MAAM,GACnB,eAAe,CA2BjB;AA+CD;;GAEG;AACH,wBAAsB,iBAAiB,CACrC,IAAI,EAAE,wBAAwB,GAC7B,OAAO,CAAC,iBAAiB,CAAC,CAkM5B;AAED;;GAEG;AACH,wBAAsB,iBAAiB,CACrC,SAAS,EAAE,MAAM,EACjB,cAAc,EAAE,MAAM,EACtB,OAAO,EAAE,MAAM,EACf,SAAS,CAAC,EAAE,MAAM,GACjB,OAAO,CAAC,OAAO,CAAC,CAwClB"}
+{"version":3,"file":"api.d.ts","sourceRoot":"","sources":["../../src/dialai/api.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAaH,OAAO,KAAK,EACV,iBAAiB,EACjB,OAAO,EACP,QAAQ,EACR,QAAQ,EACR,OAAO,EACP,UAAU,EACV,uBAAuB,EACvB,sBAAsB,EACtB,eAAe,EACf,sBAAsB,EACtB,cAAc,EACd,eAAe,EACf,iBAAiB,EACjB,eAAe,EAGf,qBAAqB,EACrB,wBAAwB,EACzB,MAAM,YAAY,CAAC;AAMpB;;;;;GAKG;AACH,wBAAsB,aAAa,CACjC,OAAO,EAAE,iBAAiB,EAC1B,QAAQ,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,GACjC,OAAO,CAAC,OAAO,CAAC,CAgDlB;AAED;;;;;;GAMG;AACH,wBAAsB,UAAU,CAAC,SAAS,EAAE,MAAM,GAAG,OAAO,CAAC,OAAO,CAAC,CAMpE;AAED;;;;GAIG;AACH,wBAAsB,WAAW,IAAI,OAAO,CAAC,OAAO,EAAE,CAAC,CAEtD;AAyID;;;;;GAKG;AACH,wBAAsB,gBAAgB,CACpC,IAAI,EAAE,uBAAuB,GAC5B,OAAO,CAAC,QAAQ,CAAC,CA4BnB;AAED;;;;;GAKG;AACH,wBAAsB,eAAe,CACnC,IAAI,EAAE,sBAAsB,GAC3B,OAAO,CAAC,OAAO,CAAC,CAwBlB;AAED;;GAEG;AACH,wBAAsB,aAAa,CAAC,YAAY,EAAE,MAAM,GAAG,OAAO,CAAC,CAAC,UAAU,GAAG,OAAO,CAAC,GAAG,SAAS,CAAC,CAErG;AAED;;GAEG;AACH,wBAAsB,YAAY,CAAC,WAAW,EAAE,MAAM,GAAG,OAAO,CAAC,QAAQ,EAAE,CAAC,CAG3E;AAED;;GAEG;AACH,wBAAsB,UAAU,CAAC,WAAW,EAAE,MAAM,GAAG,OAAO,CAAC,OAAO,GAAG,SAAS,CAAC,CAGlF;AAMD;;GAEG;AACH,wBAAsB,gBAAgB,CAAC,YAAY,EAAE,MAAM,GAAG,OAAO,CAAC,IAAI,CAAC,CAO1E;AAED;;GAEG;AACH,wBAAsB,iBAAiB,CAAC,YAAY,EAAE,MAAM,GAAG,OAAO,CAAC,IAAI,CAAC,CAO3E;AAED;;GAEG;AACH,wBAAsB,mBAAmB,CAAC,WAAW,EAAE,MAAM,GAAG,OAAO,CAAC,QAAQ,EAAE,CAAC,CAGlF;AAED;;GAEG;AACH,wBAAsB,iBAAiB,CAAC,WAAW,EAAE,MAAM,GAAG,OAAO,CAAC,OAAO,GAAG,SAAS,CAAC,CAIzF;AAMD;;;;GAIG;AACH,wBAAsB,oBAAoB,CAAC,OAAO,EAAE,OAAO,GAAG,OAAO,CAAC,QAAQ,EAAE,CAAC,CAchF;AAED;;;GAGG;AACH,wBAAsB,2BAA2B,CAAC,OAAO,EAAE,OAAO,GAAG,OAAO,CAAC,QAAQ,EAAE,CAAC,CAgBvF;AAED;;;GAGG;AACH,wBAAsB,kBAAkB,CAAC,OAAO,EAAE,OAAO,GAAG,OAAO,CAAC,OAAO,GAAG,SAAS,CAAC,CAevF;AAED;;;GAGG;AACH,wBAAsB,qBAAqB,CAAC,OAAO,EAAE,OAAO,GAAG,OAAO,CAAC,MAAM,GAAG,SAAS,CAAC,CAazF;AA2CD;;GAEG;AACH,wBAAgB,uBAAuB,CACrC,QAAQ,EAAE,QAAQ,GACjB,CAAC,CAAC,GAAG,EAAE,eAAe,KAAK,OAAO,CAAC,sBAAsB,CAAC,CAAC,GAAG,IAAI,CAUpE;AAED;;GAEG;AACH,wBAAgB,sBAAsB,CACpC,OAAO,EAAE,OAAO,GACf,CAAC,CAAC,GAAG,EAAE,cAAc,KAAK,OAAO,CAAC,eAAe,CAAC,CAAC,GAAG,IAAI,CAU5D;AAyED;;;GAGG;AACH,wBAAsB,cAAc,CAClC,IAAI,EAAE,qBAAqB,GAC1B,OAAO,CAAC,QAAQ,CAAC,CAqEnB;AAED;;GAEG;AACH,wBAAsB,oBAAoB,CACxC,SAAS,EAAE,MAAM,EACjB,OAAO,EAAE,MAAM,GACd,OAAO,CAAC,QAAQ,EAAE,CAAC,CAErB;AAED;;;GAGG;AACH,wBAAsB,iBAAiB,CACrC,SAAS,EAAE,MAAM,GAChB,OAAO,CAAC,eAAe,CAAC,CA6B1B;AAED;;;GAGG;AACH,wBAAgB,mBAAmB,CACjC,cAAc,EAAE,MAAM,EACtB,gBAAgB,EAAE,MAAM,EACxB,OAAO,EAAE,OAAO,EAChB,cAAc,EAAE,MAAM,GAAG,SAAS,EAClC,uBAAuB,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,GAAG,SAAS,EAC3D,aAAa,EAAE,MAAM,EACrB,YAAY,EAAE,MAAM,GACnB,eAAe,CA2BjB;AA+CD;;GAEG;AACH,wBAAsB,iBAAiB,CACrC,IAAI,EAAE,wBAAwB,GAC7B,OAAO,CAAC,iBAAiB,CAAC,CAkM5B;AAED;;GAEG;AACH,wBAAsB,iBAAiB,CACrC,SAAS,EAAE,MAAM,EACjB,cAAc,EAAE,MAAM,EACtB,OAAO,EAAE,MAAM,EACf,SAAS,CAAC,EAAE,MAAM,GACjB,OAAO,CAAC,OAAO,CAAC,CAwClB"}

package/dist/dialai/api.js

@@ -480,7 +480,15 @@ async function invokeArbiterStrategy(arbiter, ctx) {
  * If transitionName is omitted, invokes the specialist's registered strategy.
  */
 export async function submitProposal(opts) {
-    const { sessionId, specialistId, roundId, transitionName, reasoning, metaJson, costUSD, latencyMsec, numInputTokens, numOutputTokens, } = opts;
+    const { sessionId, specialistId, roundId } = opts;
+    let transitionName = opts.transitionName;
+    let toState;
+    let reasoning = opts.reasoning;
+    let metaJson = opts.metaJson;
+    let costUSD = opts.costUSD;
+    let latencyMsec = opts.latencyMsec;
+    let numInputTokens = opts.numInputTokens;
+    let numOutputTokens = opts.numOutputTokens;
     const session = await getSession(sessionId);
     const specialist = await getStore().getSpecialist(specialistId);
     if (!specialist) {
@@ -492,32 +500,26 @@ export async function submitProposal(opts) {
     const proposer = specialist;
     const effectiveRoundId = roundId ?? session.currentRoundId;
     const isHuman = proposer.isHuman ?? false;
-    let finalTransitionName = transitionName;
-    let finalToState;
-    let finalReasoning = reasoning;
-    let finalCostUSD = costUSD;
-    let finalLatencyMsec = latencyMsec;
-    let finalNumInputTokens = numInputTokens;
-    let finalNumOutputTokens = numOutputTokens;
     // If transitionName not provided, invoke strategy
-    if (!finalTransitionName) {
+    if (!transitionName) {
         const ctx = buildProposerContext(session);
         const result = await invokeProposerStrategy(proposer, ctx);
-        finalTransitionName = result.transitionName;
-        finalToState = result.toState;
-        finalReasoning = finalReasoning ?? result.reasoning;
-        finalCostUSD = finalCostUSD ?? result.costUSD;
-        finalLatencyMsec = finalLatencyMsec ?? result.latencyMsec;
-        finalNumInputTokens = finalNumInputTokens ?? result.numInputTokens;
-        finalNumOutputTokens = finalNumOutputTokens ?? result.numOutputTokens;
+        transitionName = result.transitionName;
+        toState = result.toState;
+        reasoning = reasoning ?? result.reasoning;
+        metaJson = metaJson ?? result.metaJson;
+        costUSD = costUSD ?? result.costUSD;
+        latencyMsec = latencyMsec ?? result.latencyMsec;
+        numInputTokens = numInputTokens ?? result.numInputTokens;
+        numOutputTokens = numOutputTokens ?? result.numOutputTokens;
     }
     else {
         // Validate the transition
         const currentStateDef = session.machine.states[session.currentState];
-        if (!currentStateDef?.transitions?.[finalTransitionName]) {
-            throw new Error(`Invalid transition "${finalTransitionName}" from state "${session.currentState}"`);
+        if (!currentStateDef?.transitions?.[transitionName]) {
+            throw new Error(`Invalid transition "${transitionName}" from state "${session.currentState}"`);
         }
-        finalToState = currentStateDef.transitions[finalTransitionName];
+        toState = currentStateDef.transitions[transitionName];
     }
     const proposal = {
         proposalId: generateUUID(),
@@ -525,14 +527,14 @@ export async function submitProposal(opts) {
         roundId: effectiveRoundId,
         specialistId,
         isHuman,
-        transitionName: finalTransitionName,
-        toState: finalToState,
-        reasoning: finalReasoning ?? "",
+        transitionName,
+        toState,
+        reasoning: reasoning ?? "",
         metaJson,
-        costUSD: finalCostUSD,
-        latencyMsec: finalLatencyMsec,
-        numInputTokens: finalNumInputTokens,
-        numOutputTokens: finalNumOutputTokens,
+        costUSD,
+        latencyMsec,
+        numInputTokens,
+        numOutputTokens,
         createdAt: new Date(),
     };
     await getStore().setProposal(proposal);