@pellux/goodvibes-tui 1.27.1 → 2.0.1

package/src/runtime/ui/provider-health/fallback-visualizer.ts

@@ -1,71 +0,0 @@
-/**
- * Fallback chain visualization data builder.
- *
- * Converts the raw fallback chain from ModelDomainState and current
- * provider health records into a structured FallbackChainData snapshot
- * suitable for UI rendering.
- */
-import type { ModelDomainState } from '@/runtime/index.ts';
-import type { ProviderHealthDomainState } from '@/runtime/index.ts';
-import type { FallbackChainData, FallbackChainNode } from '@/runtime/index.ts';
-
-/**
- * Build a FallbackChainData snapshot from the current model and health state.
- *
- * The primary model is always node 0. Each entry in ModelDomainState.fallbackChain
- * becomes a subsequent node. Health status is enriched from the provider health domain.
- *
- * @param modelState - Current model domain state.
- * @param healthState - Current provider health domain state.
- * @returns Immutable FallbackChainData for visualization.
- */
-export function buildFallbackChainData(
-  modelState: ModelDomainState,
-  healthState: ProviderHealthDomainState,
-): FallbackChainData {
-  const nodes: FallbackChainNode[] = [];
-
-  // Node 0 — primary (active model)
-  const primaryRecord = healthState.providers.get(modelState.activeProviderId);
-  nodes.push({
-    registryKey: `${modelState.activeProviderId}:${modelState.activeModelId}`,
-    providerId: modelState.activeProviderId,
-    modelId: modelState.activeModelId,
-    displayName: modelState.displayName,
-    position: 0,
-    isCurrent: modelState.activeFallbackIndex === -1,
-    providerStatus: primaryRecord?.status ?? 'unknown',
-    reason: undefined,
-  });
-
-  // Nodes 1..N — fallback chain entries
-  for (let i = 0; i < modelState.fallbackChain.length; i++) {
-    const entry = modelState.fallbackChain[i];
-    const record = healthState.providers.get(entry.providerId);
-    nodes.push({
-      registryKey: `${entry.providerId}:${entry.modelId}`,
-      providerId: entry.providerId,
-      modelId: entry.modelId,
-      displayName: entry.displayName,
-      position: i + 1,
-      isCurrent: modelState.activeFallbackIndex === i,
-      providerStatus: record?.status ?? 'unknown',
-      reason: entry.reason,
-    });
-  }
-
-  const hasUnhealthyNode = nodes.some(
-    (n) =>
-      n.providerStatus === 'degraded' ||
-      n.providerStatus === 'rate_limited' ||
-      n.providerStatus === 'unavailable' ||
-      n.providerStatus === 'auth_error',
-  );
-
-  return {
-    nodes,
-    activeIndex: modelState.activeFallbackIndex,
-    falloverCount: modelState.falloverCount,
-    hasUnhealthyNode,
-  };
-}

package/src/runtime/ui/provider-health/index.ts

@@ -1,46 +0,0 @@
-/**
- * Provider health UI data surface barrel.
- *
- * Re-exports all types and the ProviderHealthDataProvider class.
- * Also provides the createProviderHealthData() factory for one-shot snapshots.
- */
-export type {
-  ProviderStatus,
-  CompositeHealthStatus,
-  HealthTimelinePoint,
-  HealthTimeline,
-  ProviderHealthEntry,
-  FallbackChainNode,
-  FallbackChainData,
-  ProviderHealthData,
-} from '@/runtime/index.ts';
-
-export { ProviderHealthDataProvider } from '@/runtime/index.ts';
-export { buildFallbackChainData } from '@/runtime/index.ts';
-
-import type { ProviderHealthDomainState } from '@/runtime/index.ts';
-import type { ModelDomainState } from '@/runtime/index.ts';
-import { ProviderHealthDataProvider } from '@/runtime/index.ts';
-import type { ProviderHealthData } from '@/runtime/index.ts';
-
-/**
- * Produce a one-shot ProviderHealthData snapshot without creating a long-lived provider.
- *
- * Use this when you need a single render pass and do not require change subscriptions.
- * For reactive/subscription-based UIs, prefer ProviderHealthDataProvider.
- *
- * @param healthState - Current provider health domain state.
- * @param modelState - Current model domain state.
- * @returns Immutable ProviderHealthData snapshot.
- */
-export function createProviderHealthData(
-  healthState: ProviderHealthDomainState,
-  modelState: ModelDomainState,
-): ProviderHealthData {
-  // Delegate to the data provider for consistent derivation logic,
-  // then dispose immediately since no subscriptions are needed.
-  const dp = new ProviderHealthDataProvider(healthState, modelState);
-  const snapshot = dp.getSnapshot();
-  dp.dispose();
-  return snapshot;
-}

package/src/runtime/ui/provider-health/types.ts

@@ -1,146 +0,0 @@
-/**
- * Provider health UI data types.
- *
- * Purely data-oriented — no rendering logic. These types are produced by
- * ProviderHealthDataProvider and consumed by renderers/components.
- */
-import type { ProviderStatus, CompositeHealthStatus } from '@/runtime/index.ts';
-
-// Re-exports for convenience
-export type { ProviderStatus, CompositeHealthStatus };
-
-/**
- * A single point in a provider health timeline.
- * Used to render charts of success rate, latency, or error rate over time.
- */
-export interface HealthTimelinePoint {
-  /** Epoch ms timestamp of this sample. */
-  readonly ts: number;
-  /** Observed success rate at this point (0–1). */
-  readonly successRate: number;
-  /** Observed average latency in ms at this point. */
-  readonly avgLatencyMs: number;
-  /** Observed error rate at this point (0–1). */
-  readonly errorRate: number;
-}
-
-/**
- * Timeline data for a single provider, for chart rendering.
- */
-export interface HealthTimeline {
-  /** Provider ID this timeline belongs to. */
-  readonly providerId: string;
-  /** Ordered timeline points (oldest first). */
-  readonly points: readonly HealthTimelinePoint[];
-  /** The number of points retained in this timeline. */
-  readonly length: number;
-}
-
-/**
- * Full health entry for a single provider, enriched for UI consumption.
- */
-export interface ProviderHealthEntry {
-  // ── Identity ──────────────────────────────────────────────────────────────
-  /** Provider identifier. */
-  readonly providerId: string;
-  /** Human-readable provider name. */
-  readonly displayName: string;
-
-  // ── Status ────────────────────────────────────────────────────────────────
-  /** Current health status. */
-  readonly status: ProviderStatus;
-  /** True when this provider is the active selection. */
-  readonly isActive: boolean;
-  /** True when this provider is configured with a valid API key. */
-  readonly isConfigured: boolean;
-
-  // ── Performance metrics ───────────────────────────────────────────────────
-  /** Success rate (0–1), derived from totalCalls and successCalls. */
-  readonly successRate: number;
-  /** Error rate (0–1), derived from totalCalls and errorCalls. */
-  readonly errorRate: number;
-  /** Approximate p95 latency in ms (max of recent observations). */
-  readonly p95LatencyMs: number;
-  /** Moving average latency in ms. */
-  readonly avgLatencyMs: number;
-  /** Total API calls recorded. */
-  readonly totalCalls: number;
-
-  // ── Cache metrics ─────────────────────────────────────────────────────────
-  /** Cache hit rate (0–1). Present only when cache metrics are available. */
-  readonly cacheHitRate?: number;
-  /** Cumulative cache read tokens saved. */
-  readonly cacheReadTokens?: number;
-  /** Cumulative cache write tokens. */
-  readonly cacheWriteTokens?: number;
-
-  // ── Timing ────────────────────────────────────────────────────────────────
-  /** Epoch ms of the last successful call, if any. */
-  readonly lastSuccessAt?: number;
-  /** Epoch ms of the last error, if any. */
-  readonly lastErrorAt?: number;
-  /** Most recent error message, if any. */
-  readonly lastErrorMessage?: string;
-  /** Epoch ms of the last health check. */
-  readonly lastCheckedAt?: number;
-  /** Rate limit reset time (epoch ms), if currently rate-limited. */
-  readonly rateLimitResetAt?: number;
-
-  // ── Timeline ──────────────────────────────────────────────────────────────
-  /** Timeline data for chart rendering. */
-  readonly timeline: HealthTimeline;
-}
-
-/**
- * A single node in the fallback chain visualization.
- */
-export interface FallbackChainNode {
-  /** Provider identifier. */
-  readonly providerId: string;
-  /** Model identifier on this provider. */
-  readonly modelId: string;
-  /** Human-readable display label. */
-  readonly displayName: string;
-  /** Position in the chain (0 = primary). */
-  readonly position: number;
-  /** Whether this node is the currently active entry in the chain. */
-  readonly isCurrent: boolean;
-  /** Current health status of this provider. */
-  readonly providerStatus: ProviderStatus;
-  /** Reason this fallback was configured. */
-  readonly reason?: string;
-}
-
-/**
- * Fallback chain visualization data for the current model configuration.
- */
-export interface FallbackChainData {
-  /** Ordered nodes in the fallback chain (primary first). */
-  readonly nodes: readonly FallbackChainNode[];
-  /** Index of the currently active node (-1 = primary is active). */
-  readonly activeIndex: number;
-  /** Number of fallover events since session start. */
-  readonly falloverCount: number;
-  /** True when the chain has at least one degraded or unavailable node. */
-  readonly hasUnhealthyNode: boolean;
-}
-
-/**
- * Complete data snapshot produced by ProviderHealthDataProvider.
- */
-export interface ProviderHealthData {
-  /** All provider health entries, sorted by status priority then name. */
-  readonly entries: readonly ProviderHealthEntry[];
-  /** Composite health status across all providers. */
-  readonly compositeStatus: CompositeHealthStatus;
-  /** Number of providers currently in a non-healthy state. */
-  readonly degradedCount: number;
-  /** Number of providers currently unavailable. */
-  readonly unavailableCount: number;
-  /** Fallback chain visualization data. */
-  readonly fallbackChain: FallbackChainData;
-  /** Warning messages from provider discovery. */
-  readonly warnings: readonly string[];
-  /** Epoch ms when this snapshot was produced. */
-  readonly snapshotAt: number;
-}

package/src/runtime/ui-events.ts

@@ -1 +0,0 @@
-export * from '@/runtime/index.ts';

package/src/runtime/ui-read-model-helpers.ts

@@ -1 +0,0 @@
-export * from '@/runtime/index.ts';

package/src/runtime/ui-read-models-observability-maintenance.ts

@@ -1 +0,0 @@
-export * from '@/runtime/index.ts';

package/src/runtime/ui-read-models-observability-options.ts

@@ -1 +0,0 @@
-export * from '@/runtime/index.ts';

package/src/runtime/ui-read-models-observability-remote.ts

@@ -1 +0,0 @@
-export * from '@/runtime/index.ts';

package/src/runtime/ui-read-models-observability-security.ts

@@ -1 +0,0 @@
-export * from '@/runtime/index.ts';

package/src/runtime/ui-read-models-observability-system.ts

@@ -1 +0,0 @@
-export * from '@/runtime/index.ts';

package/src/runtime/ui-read-models-observability.ts

@@ -1 +0,0 @@
-export * from '@/runtime/index.ts';

package/src/runtime/ui-service-queries.ts

@@ -1 +0,0 @@
-export * from '@/runtime/index.ts';

package/src/runtime/voice-setup-services.ts

@@ -1,98 +0,0 @@
-// ---------------------------------------------------------------------------
-// voice-setup-services.ts — managed local-voice provisioning composition (TUI wiring)
-//
-// Backs the daemon's voice.local.status / voice.local.install verbs. The
-// provisioning policy — a single-flight one-act install that provisions the
-// piper engine + a default voice (and whisper STT where a bundle is published),
-// live install-progress folded onto status(), ownership-aware preconfigure of
-// the voice.local.* keys (values THIS installer previously wrote update to the
-// new managed paths; genuinely user-set values still win; a user-cleared
-// installer value stays cleared), a no-network status() read, and critical-tier
-// admission gating — lives in the SDK's createVoiceSetupService. Earlier the
-// composer was not exported, so the TUI rebuilt it from the voice primitives;
-// SDK 1.10.1 exports it through ./platform/runtime/voice-setup, so this module
-// is now a thin adapter that maps the TUI's injected seams (configManager,
-// shellPaths, voiceProviders) onto the SDK service's flat deps.
-//
-// Extracted into its own module rather than built inline in services.ts, which
-// sits at the architecture check's 800-line cap (scripts/check-architecture.ts)
-// — new service construction gets its own module and a single wiring call there.
-// ---------------------------------------------------------------------------
-
-import {
-  createVoiceSetupService,
-  type VoiceSetupService,
-} from '@pellux/goodvibes-sdk/platform/runtime/voice-setup';
-import {
-  startWakeBootProvisioning,
-  type VoiceProviderRegistry,
-} from '@pellux/goodvibes-sdk/platform/voice';
-import type { ConfigManager } from '@pellux/goodvibes-sdk/platform/config';
-import { logger } from '@pellux/goodvibes-sdk/platform/utils';
-import type { ShellPathService } from '@/runtime/index.ts';
-
-export type { VoiceSetupService };
-
-/**
- * The recovery act this surface names when a boot provision degrades. The SDK
- * would otherwise name the control-plane verb, which is correct everywhere and
- * useless to someone sitting in a terminal.
- */
-export const WAKE_RECOVERY_COMMAND = '/voice wake setup';
-
-export interface VoiceSetupServicesDeps {
-  readonly configManager: ConfigManager;
-  readonly shellPaths: Pick<ShellPathService, 'resolveUserPath'>;
-  readonly voiceProviders: VoiceProviderRegistry;
-  /** Critical-tier admission gate — a provision run allocates archive + model buffers. */
-  readonly admitExpensiveWork: (label: string) => { allowed: boolean; reason?: string | undefined };
-  /**
-   * Opt into boot-time wake-model provisioning and the recovery sweeper.
-   *
-   * Off unless a real entrypoint asks for it, matching how this graph treats the
-   * host power seam: `daemon/cli.ts` and `runtime/bootstrap-core.ts` opt in, the
-   * one-shot CLI commands do not, and a test composing the graph gets neither a
-   * network fetch nor a repeating timer it did not ask for.
-   */
-  readonly provisionWakeModelsAtBoot?: boolean | undefined;
-  /** Injected in tests: the boot starter, so no sweeper and no fetch are real. */
-  readonly startBootProvisioning?: typeof startWakeBootProvisioning | undefined;
-}
-
-export interface VoiceSetupServices {
-  readonly voiceSetup: VoiceSetupService;
-  /**
-   * Stop the wake-word recovery sweeper and cancel a pending boot provision.
-   * A no-op when boot provisioning was not opted into. Registered on the
-   * disposal scope (disposal-wiring.ts) — an hourly timer nothing stops is a
-   * poller this surface leaked, which is exactly what that scope exists for.
-   */
-  readonly stopWakeHousekeeping: () => void;
-}
-
-/** Compose the managed-voice single-flight install + no-network status read. */
-export function wireVoiceSetup(deps: VoiceSetupServicesDeps): VoiceSetupServices {
-  const { configManager, voiceProviders } = deps;
-  const managedVoiceRoot = deps.shellPaths.resolveUserPath('voice');
-  const voiceSetup = createVoiceSetupService({
-    managedVoiceRoot,
-    getConfig: (key) => String(configManager.get(key as Parameters<typeof configManager.get>[0]) ?? ''),
-    setConfig: (key, value) => configManager.setDynamic(key as Parameters<typeof configManager.setDynamic>[0], value),
-    // A successful (re-)install clears any tripped local-engine breaker.
-    resetLocalEngineFailureState: () => voiceProviders.get('local')?.resetEngineFailureState?.(),
-    admitExpensiveWork: deps.admitExpensiveWork,
-  });
-  if (deps.provisionWakeModelsAtBoot !== true) {
-    return { voiceSetup, stopWakeHousekeeping: () => {} };
-  }
-  // Sweep the wake tree, then fetch whatever the install could not. The SDK owns
-  // the policy — never throws, one plain message, reaps before it retries — and
-  // routes the attempt through the setup service's single flight so a boot
-  // attempt and a user typing /voice wake setup are one download.
-  const boot = (deps.startBootProvisioning ?? startWakeBootProvisioning)({
-    managedRoot: managedVoiceRoot,
-    ensureProvisioned: () => voiceSetup.wakeEnsureProvisioned({ recoveryHint: WAKE_RECOVERY_COMMAND }),
-    announce: (message) => { logger.info(message); },
-  });
-  return { voiceSetup, stopWakeHousekeeping: () => boot.stop() };
-}

package/src/runtime/workstream-attempts-validation.ts

@@ -1,77 +0,0 @@
-// ---------------------------------------------------------------------------
-// workstream-attempts-validation.ts — plan-format validation for best-of-N.
-//
-// The plan format re-enables the per-item `attempts` field (SDK WorkItem/
-// WorkItemSpec.attempts — best-of-N). A best-of-N item MAY be non-leaf: it may
-// declare its own `dependsOn` (each sibling inherits it) and other items may
-// depend on it — a dependent is held by the dependency gate until the group's
-// winner is picked and merged, then resolves to that winner (SDK
-// WorkItemSpec.attempts doc). The TUI enforces the engine's genuine constraints
-// at plan validation rather than letting a launch silently drop the field:
-//   • WORKTREE isolation. Best-of-N needs isolated worktrees to run and compare
-//     the N sibling attempts; under `shared` isolation the value is ignored.
-//   • STABLE id when depended upon. A dependency edge references an id, so a
-//     best-of-N item that others depend on must carry a stable `id`; an
-//     anonymous best-of-N item stays a leaf by construction (nothing can name it).
-// Values above the engine's MAX_ATTEMPTS (5) are clamped by the engine; the TUI
-// surfaces that as a note so the preview never overstates the fan-out.
-// ---------------------------------------------------------------------------
-
-import type { CreateWorkstreamInput } from '@pellux/goodvibes-sdk/platform/orchestration';
-
-/** The engine's hard cap on sibling attempts per work item (SDK MAX_ATTEMPTS). */
-export const MAX_ATTEMPTS = 5;
-
-export interface AttemptsValidation {
-  /** True when any item requests best-of-N (attempts >= 2). */
-  readonly hasAttempts: boolean;
-  /** Hard rule breaks that must block a launch (worktree isolation + stable id when depended upon). */
-  readonly violations: readonly string[];
-  /** Non-blocking notes (e.g. an over-cap value the engine will clamp). */
-  readonly notes: readonly string[];
-}
-
-function itemKey(item: CreateWorkstreamInput['items'][number]): string {
-  return item.id ?? item.title;
-}
-
-/**
- * Validate a draft spec's best-of-N usage against the worktree-isolation and
- * stable-id (when depended upon) constraints. Non-leaf best-of-N is allowed.
- * Pure — no I/O. `violations` are hard (block launch); `notes` are advisory.
- */
-export function validateAttempts(spec: CreateWorkstreamInput): AttemptsValidation {
-  const violations: string[] = [];
-  const notes: string[] = [];
-  const worktree = spec.isolation === 'worktree';
-
-  // Which items are depended upon (so a best-of-N item that is a dependency
-  // target can be checked for the stable-id requirement).
-  const dependedUpon = new Set<string>();
-  for (const item of spec.items) {
-    for (const dep of item.dependsOn ?? []) dependedUpon.add(dep);
-  }
-
-  let hasAttempts = false;
-  for (const item of spec.items) {
-    const attempts = item.attempts ?? 1;
-    if (attempts < 2) continue;
-    hasAttempts = true;
-    const label = item.title;
-
-    if (!worktree) {
-      violations.push(`"${label}" requests ${attempts} attempts, but the workstream is not worktree-isolated — best-of-N needs isolated trees. Set --isolation worktree.`);
-    }
-    // Non-leaf best-of-N is allowed. The only structural requirement: if other
-    // items depend on this one, it must carry a stable id for the dependency
-    // edge to name — an anonymous best-of-N item cannot be a dependency target.
-    if (dependedUpon.has(itemKey(item)) && !item.id) {
-      violations.push(`another item depends on best-of-N item "${label}", but it has no stable id — give the item an id so the dependency can resolve to the winner (an anonymous best-of-N item cannot be a dependency target).`);
-    }
-    if (attempts > MAX_ATTEMPTS) {
-      notes.push(`"${label}" requests ${attempts} attempts — the engine caps best-of-N at ${MAX_ATTEMPTS}, so ${MAX_ATTEMPTS} siblings will run.`);
-    }
-  }
-
-  return { hasAttempts, violations, notes };
-}

package/src/runtime/workstream-draft-edits.ts

@@ -1,95 +0,0 @@
-// ---------------------------------------------------------------------------
-// workstream-draft-edits.ts — pure item edits for a not-yet-launched proposal
-//
-// The plan-review gate (see /workstream, input/commands/workstream-runtime.ts)
-// lets a human reshape a decomposed proposal BEFORE anything is spent: rewrite
-// one item's brief, drop an item, or reorder items. These are pure functions
-// over the launchable CreateWorkstreamInput (@pellux/goodvibes-sdk/platform/
-// orchestration) — no engine calls, no I/O — so the same spec that is edited
-// here is byte-for-byte what launch hands createWorkstream. Extracted from
-// workstream-services.ts to keep that construction module lean (its own
-// header explains the 800-line-cap discipline) and to give the edits a clean
-// unit-test target.
-//
-// HONEST SCOPE (stated so the review surface never overstates what an edit
-// does):
-//   - editItemBrief rewrites an item's `task` (the instructions its agent
-//     runs), never its `title` (the short label). The engineer/review phase
-//     template is unchanged — a brief is WHAT the item does, not HOW the
-//     pipeline runs it.
-//   - removeItem also strips the removed item's id from every other item's
-//     `dependsOn`, so the spec never carries a dangling dependency into
-//     fromPlanProposal's assembly (which would otherwise reject it). It
-//     refuses to remove the last item — a workstream needs at least one.
-//   - moveItem reorders the AUTHORING order (the ordinals shown in the review
-//     and the tie-break when the engine has a free capacity slot and several
-//     claimable items). It never rewrites dependencies, so it cannot create a
-//     cycle; the engine still schedules strictly by dependency + capacity, not
-//     by array position. Reordering is a presentation/authoring act, not a
-//     scheduling override — and is documented as such where it is offered.
-// ---------------------------------------------------------------------------
-
-import type { CreateWorkstreamInput, WorkItemSpec } from '@pellux/goodvibes-sdk/platform/orchestration';
-
-/** Success carries the rewritten spec; failure carries an honest, user-facing reason. */
-export type DraftEditResult = { readonly spec: CreateWorkstreamInput } | { readonly error: string };
-
-/**
- * Resolve a user-typed item reference to a 0-based index. A reference is
- * either the 1-based ordinal shown in the proposal (the primary handle) or an
- * item id / unique id-prefix (the stable handle that survives reordering). An
- * ambiguous or unmatched reference resolves to undefined so the caller can
- * refuse honestly rather than editing the wrong item.
- */
-export function resolveItemIndex(items: readonly WorkItemSpec[], ref: string): number | undefined {
-  const trimmed = ref.trim();
-  const ordinal = Number(trimmed);
-  if (Number.isInteger(ordinal) && ordinal >= 1 && ordinal <= items.length) return ordinal - 1;
-  const matches = items
-    .map((item, index) => ({ item, index }))
-    .filter(({ item }) => item.id !== undefined && item.id.startsWith(trimmed));
-  return matches.length === 1 ? matches[0]!.index : undefined;
-}
-
-function unresolvedError(items: readonly WorkItemSpec[], ref: string): string {
-  return `No draft item matches "${ref}". Use a number from 1 to ${items.length}, or an item id.`;
-}
-
-/** Rewrite one item's brief (its `task`). Empty/whitespace briefs are refused. */
-export function editItemBrief(spec: CreateWorkstreamInput, ref: string, brief: string): DraftEditResult {
-  const index = resolveItemIndex(spec.items, ref);
-  if (index === undefined) return { error: unresolvedError(spec.items, ref) };
-  const trimmed = brief.trim();
-  if (!trimmed) return { error: 'A new brief cannot be empty.' };
-  const items = spec.items.map((item, i) => (i === index ? { ...item, task: trimmed } : item));
-  return { spec: { ...spec, items } };
-}
-
-/** Drop one item and strip its id from every sibling's dependsOn. Refuses to empty the plan. */
-export function removeItemFromSpec(spec: CreateWorkstreamInput, ref: string): DraftEditResult {
-  const index = resolveItemIndex(spec.items, ref);
-  if (index === undefined) return { error: unresolvedError(spec.items, ref) };
-  if (spec.items.length <= 1) return { error: 'Cannot remove the last item — a workstream needs at least one.' };
-  const removedId = spec.items[index]!.id;
-  const items = spec.items
-    .filter((_, i) => i !== index)
-    .map((item) =>
-      removedId && item.dependsOn?.includes(removedId)
-        ? { ...item, dependsOn: item.dependsOn.filter((dep) => dep !== removedId) }
-        : item,
-    );
-  return { spec: { ...spec, items } };
-}
-
-/** Reorder one item to a new 1-based authoring position. Never touches dependencies. */
-export function moveItemInSpec(spec: CreateWorkstreamInput, ref: string, toPosition: number): DraftEditResult {
-  const index = resolveItemIndex(spec.items, ref);
-  if (index === undefined) return { error: unresolvedError(spec.items, ref) };
-  if (!Number.isInteger(toPosition) || toPosition < 1 || toPosition > spec.items.length) {
-    return { error: `Position must be a whole number from 1 to ${spec.items.length}.` };
-  }
-  const items = [...spec.items];
-  const [moved] = items.splice(index, 1);
-  items.splice(toPosition - 1, 0, moved!);
-  return { spec: { ...spec, items } };
-}