@robota-sdk/agent-transport 3.0.0-beta.66 → 3.0.0-beta.68

package/src/tui/__tests__/model-change-side-effect.test.ts

@@ -1,91 +0,0 @@
-import { describe, expect, it } from 'vitest';
-import type { IHistoryEntry, TSessionEndReason } from '@robota-sdk/agent-core';
-import {
-  addModelChangeCancelledMessage,
-  applyConfirmedModelChange,
-  formatModelChangeConfirmationMessage,
-} from '../hooks/model-change-side-effect.js';
-
-interface IShutdownCall {
-  reason: TSessionEndReason;
-  message: string;
-}
-
-interface IModelChangeCall {
-  cwd: string;
-  modelId: string;
-  options?: { providerOverride?: string };
-}
-
-function entryContent(entry: IHistoryEntry): string {
-  return (entry.data as { content: string }).content;
-}
-
-describe('model change side effect', () => {
-  it('persists the selected model for the active provider override before exiting', () => {
-    const entries: IHistoryEntry[] = [];
-    const shutdowns: IShutdownCall[] = [];
-    const calls: IModelChangeCall[] = [];
-    const applyModelChange = (
-      cwd: string,
-      modelId: string,
-      options?: { providerOverride?: string },
-    ): { applied: boolean } => {
-      calls.push({ cwd, modelId, ...(options !== undefined && { options }) });
-      return { applied: true };
-    };
-
-    applyConfirmedModelChange({
-      cwd: '/tmp/project',
-      modelId: 'gpt-4.1',
-      providerOverride: 'openai',
-      addEntry: (entry) => entries.push(entry),
-      requestShutdown: (reason, message) => shutdowns.push({ reason, message }),
-      applyModelChange,
-    });
-
-    expect(calls).toEqual([
-      {
-        cwd: '/tmp/project',
-        modelId: 'gpt-4.1',
-        options: { providerOverride: 'openai' },
-      },
-    ]);
-    expect(entryContent(entries[0]!)).toBe(
-      'Model changed to gpt-4.1. Exiting so the next session uses it.',
-    );
-    expect(shutdowns).toEqual([{ reason: 'other', message: 'Model change applied' }]);
-  });
-
-  it('does not exit when persistence fails', () => {
-    const entries: IHistoryEntry[] = [];
-    const shutdowns: IShutdownCall[] = [];
-
-    applyConfirmedModelChange({
-      cwd: '/tmp/project',
-      modelId: 'gpt-4.1',
-      addEntry: (entry) => entries.push(entry),
-      requestShutdown: (reason, message) => shutdowns.push({ reason, message }),
-      applyModelChange: () => {
-        throw new Error('settings write failed');
-      },
-    });
-
-    expect(entryContent(entries[0]!)).toBe('Failed: settings write failed');
-    expect(shutdowns).toEqual([]);
-  });
-
-  it('reports cancellation without applying settings or exiting', () => {
-    const entries: IHistoryEntry[] = [];
-
-    addModelChangeCancelledMessage((entry) => entries.push(entry));
-
-    expect(entryContent(entries[0]!)).toBe('Model change cancelled.');
-  });
-
-  it('describes model changes as an exit for the next session', () => {
-    expect(formatModelChangeConfirmationMessage('gpt-4.1')).toBe(
-      'Change model to gpt-4.1? This will exit the current session so the next session uses it.',
-    );
-  });
-});

package/src/tui/hooks/model-change-side-effect.ts

@@ -1,63 +0,0 @@
-import {
-  createSystemMessage,
-  getModelName,
-  messageToHistoryEntry,
-  type IHistoryEntry,
-  type TSessionEndReason,
-} from '@robota-sdk/agent-core';
-
-type TApplyModelChange = (
-  cwd: string,
-  modelId: string,
-  options?: { providerOverride?: string },
-) => { applied: boolean };
-
-export interface IApplyConfirmedModelChangeDeps {
-  cwd: string;
-  modelId: string;
-  providerOverride?: string | undefined;
-  addEntry: (entry: IHistoryEntry) => void;
-  requestShutdown: (reason: TSessionEndReason, message: string) => void;
-  applyModelChange?: TApplyModelChange;
-}
-
-export function formatModelChangeConfirmationMessage(modelId: string): string {
-  return `Change model to ${getModelName(modelId)}? This will exit the current session so the next session uses it.`;
-}
-
-export function formatModelChangeExitMessage(modelId: string): string {
-  return `Model changed to ${getModelName(modelId)}. Exiting so the next session uses it.`;
-}
-
-export function applyConfirmedModelChange(deps: IApplyConfirmedModelChangeDeps): void {
-  const applyModelChange = deps.applyModelChange;
-  if (!applyModelChange) {
-    deps.addEntry(
-      messageToHistoryEntry(createSystemMessage('Model change unavailable: no adapter provided.')),
-    );
-    return;
-  }
-
-  const options =
-    deps.providerOverride !== undefined ? { providerOverride: deps.providerOverride } : undefined;
-
-  try {
-    // allow-fallback: user-facing error display for model change failure
-    applyModelChange(deps.cwd, deps.modelId, options);
-    deps.addEntry(
-      messageToHistoryEntry(createSystemMessage(formatModelChangeExitMessage(deps.modelId))),
-    );
-    deps.requestShutdown('other', 'Model change applied');
-  } catch (error) {
-    // allow-fallback: user-facing error display for model change failure
-    deps.addEntry(
-      messageToHistoryEntry(
-        createSystemMessage(`Failed: ${error instanceof Error ? error.message : String(error)}`),
-      ),
-    );
-  }
-}
-
-export function addModelChangeCancelledMessage(addEntry: (entry: IHistoryEntry) => void): void {
-  addEntry(messageToHistoryEntry(createSystemMessage('Model change cancelled.')));
-}