@pellux/goodvibes-tui 0.19.0 → 0.19.1

package/CHANGELOG.md

@@ -4,14 +4,51 @@ All notable changes to GoodVibes TUI.
 
 ---
 
+## [0.19.1] — 2026-04-17
+
+### Changed
+- Upgraded `@pellux/goodvibes-sdk` from 0.21.1 to 0.21.6 — includes secrets-tier
+  `configuredVia` discriminator, HTTP `/api/providers` + `/api/providers/current` (GET/PATCH),
+  reactive `model.changed` SSE event, `BUILTIN_LABEL_MAP` brand-accurate provider labels,
+  and clean unconfigured-provider errors instead of silent 401 pass-through.
+
+### Added
+- `--provider <id>` and `--model <registryKey>` CLI flags for daemon startup
+  (`goodvibes-daemon`) and TUI shell startup (`goodvibes`). Both override
+  `~/.goodvibes/tui/settings.json` provider/model values for the session.
+  If `--model provider:modelId` format is used, `--provider` is inferred automatically.
+- TUI model picker now populates `configuredVia` per provider and renders it as
+  a badge (`[env]`, `[sub]`, `[anon]`) in the provider-browse view, derived from
+  env-var presence and active OAuth subscription sessions.
+
+### Fixed
+- **Venice-picking diagnosis**: the silent 401 Venice fallback was caused by the
+  pre-0.21.2 SDK passing unconfigured-provider requests through to the upstream API
+  unchanged. As of 0.21.2 (included in 0.21.6), `createCompanionProviderAdapter` now
+  checks `provider.isConfigured()` before `provider.chat()` and yields a structured
+  error immediately. The in-registry fallback to first-selectable-model (which may
+  resolve to venice on some installs) only triggers when `getCurrentModel()` cannot
+  find the stored `currentModelId` in the model registry — typically an old daemon
+  instance with stale config. Restarting the daemon after a settings.json change
+  resolves this; the `--model` CLI flag eliminates the need to edit settings.json
+  manually. ConfigManager reactivity (live disk-change → registry update) is out of
+  scope: the PATCH `/api/providers/current` route is the correct live-switch path
+  (calls `setCurrentModel()` + `configManager.set()` atomically).
+
+
 ## [0.19.0] — 2026-04-18
 
 ### Changed
 - Upgraded `@pellux/goodvibes-sdk` from 0.19.6 to 0.21.1 (soak-period release).
-  TUI adaptation required: regenerated `docs/foundation-artifacts/operator-contract.json`
-  to match the updated `buildOperatorContract()` output in the new SDK version.
-  `peer-contract.json`, `knowledge-graphql.graphql`, and `knowledge-store.sql` were
-  unchanged by this SDK bump.
+  TUI adaptations required:
+  1. `docs/foundation-artifacts/operator-contract.json` — regenerated to match
+     updated `buildOperatorContract()` output (`peer-contract.json`, knowledge
+     artifacts unchanged).
+  2. `scripts/perf-check.ts` — `platform/runtime/perf/index` barrel removed;
+     `createPerfMonitor()` factory removed; migrated to `new PerfMonitor()` with
+     imports split to `perf/monitor` and `perf/reporter` sub-paths.
+  3. `scripts/eval-gate.ts` — `platform/runtime/eval/index` barrel removed;
+     imports split to `eval/baseline`, `eval/format`, and `eval/scorecard`.
 
 ### Added
 - Wave B panel migration: migrated 5 panels (knowledge, marketplace, memory,

package/README.md

@@ -2,7 +2,7 @@
 
 [![CI](https://github.com/mgd34msu/goodvibes-tui/actions/workflows/ci.yml/badge.svg)](https://github.com/mgd34msu/goodvibes-tui/actions/workflows/ci.yml)
 [![License: MIT](https://img.shields.io/badge/License-MIT-yellow.svg)](https://opensource.org/licenses/MIT)
-[![Version](https://img.shields.io/badge/version-0.19.0-blue.svg)](https://github.com/mgd34msu/goodvibes-tui)
+[![Version](https://img.shields.io/badge/version-0.19.1-blue.svg)](https://github.com/mgd34msu/goodvibes-tui)
 
 A terminal-native AI coding, operations, automation, knowledge, and integration console with a typed runtime, omnichannel surfaces, structured memory/knowledge, and a raw ANSI renderer.
 

package/docs/foundation-artifacts/operator-contract.json

@@ -3,7 +3,7 @@
   "product": {
     "id": "goodvibes",
     "surface": "operator",
-    "version": "0.21.1"
+    "version": "0.21.6"
   },
   "auth": {
     "modes": [

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pellux/goodvibes-tui",
-  "version": "0.19.0",
+  "version": "0.19.1",
   "description": "Terminal-native GoodVibes product for coding, operations, automation, knowledge, channels, and daemon-backed control-plane workflows.",
   "type": "module",
   "main": "src/main.ts",
@@ -89,7 +89,7 @@
     "@anthropic-ai/vertex-sdk": "^0.16.0",
     "@ast-grep/napi": "^0.42.0",
     "@aws/bedrock-token-generator": "^1.1.0",
-    "@pellux/goodvibes-sdk": "0.21.1",
+    "@pellux/goodvibes-sdk": "0.21.6",
     "bash-language-server": "^5.6.0",
     "fuse.js": "^7.1.0",
     "graphql": "^16.13.2",

package/src/daemon/cli.ts

@@ -22,6 +22,49 @@ type DaemonCliOwnership = {
   readonly homeDirectory: string;
 };
 
+// ---------------------------------------------------------------------------
+// CLI flag parsing
+// ---------------------------------------------------------------------------
+
+type DaemonCliFlags = {
+  readonly provider: string | undefined;
+  readonly model: string | undefined;
+};
+
+function parseDaemonCliFlags(argv: readonly string[]): DaemonCliFlags {
+  let provider: string | undefined;
+  let model: string | undefined;
+
+  for (let i = 0; i < argv.length; i++) {
+    const arg = argv[i];
+    if (arg === '--help' || arg === '-h') {
+      // eslint-disable-next-line no-console
+      console.log([
+        'Usage: goodvibes-daemon [options]',
+        '',
+        'Options:',
+        '  --provider <id>          Override the provider from settings.json at startup',
+        '  --model <registryKey>    Override the model from settings.json at startup',
+        '                           Format: provider:modelId (e.g. inception:mercury-2)',
+        '                           If provider:modelId format is used, --provider is inferred',
+        '  --help, -h               Show this help message',
+      ].join('\n'));
+      process.exit(0);
+    }
+    if (arg === '--provider' && argv[i + 1] !== undefined) {
+      provider = argv[++i];
+    } else if (arg === '--model' && argv[i + 1] !== undefined) {
+      model = argv[++i];
+      // Infer provider from registryKey format (provider:modelId) if --provider not given
+      if (typeof model === 'string' && model.includes(':') && provider === undefined) {
+        provider = model.split(':')[0];
+      }
+    }
+  }
+
+  return { provider, model };
+}
+
 type DaemonCliTokens = {
   readonly daemonToken: string | undefined;
   readonly httpToken: string | undefined;
@@ -72,6 +115,17 @@ async function main(): Promise<void> {
   const { workingDirectory: workingDir, homeDirectory } = resolveDaemonCliOwnership();
   const config = new ConfigManager({ workingDir, homeDir: homeDirectory, surfaceRoot: 'tui' });
   new GlobalNetworkTransportInstaller().install(config);
+
+  // Apply CLI flags — override settings.json before the provider registry is constructed
+  const cliFlags = parseDaemonCliFlags(process.argv.slice(2));
+  if (cliFlags.provider !== undefined) {
+    config.set('provider.provider', cliFlags.provider);
+    logger.info('daemon: --provider flag applied', { provider: cliFlags.provider });
+  }
+  if (cliFlags.model !== undefined) {
+    config.set('provider.model', cliFlags.model);
+    logger.info('daemon: --model flag applied', { model: cliFlags.model });
+  }
   const runtimeBus = new RuntimeEventBus();
   const runtimeStore = createRuntimeStore();
   const runtimeServices = createRuntimeServices({

package/src/input/model-picker.ts

@@ -99,6 +99,8 @@ export interface PickerItem {
   isFree?: boolean;
   /** True when this provider item has a configured API key. */
   isConfigured?: boolean;
+  /** How the provider is configured — shown as a badge in provider mode. */
+  configuredVia?: 'env' | 'secrets' | 'subscription' | 'anonymous';
 }
 
 /** Provider IDs treated as "Popular" in the provider picker. */
@@ -166,6 +168,8 @@ export class ModelPickerModal {
   public availableOnly = true;
   /** Set of provider names that have a configured key (used for availableOnly filter). */
   public configuredProviders: Set<string> = new Set();
+  /** How each provider is configured — drives badge display in provider mode. */
+  public configuredViaMap: Map<string, 'env' | 'secrets' | 'subscription' | 'anonymous'> = new Map();
   /** IDs of pinned/favorite models — shown at top of list. */
   public pinnedIds: Set<string> = new Set();
   /** IDs of recently used models — shown after pinned, before the rest. */
@@ -612,13 +616,13 @@ export class ModelPickerModal {
       if (filteredPopular.length > 0) {
         providerItems.push({ id: '__header__popular', label: 'Popular', isGroupHeader: true });
         for (const p of filteredPopular) {
-          providerItems.push({ id: p, label: p, isConfigured: this.configuredProviders.has(p) });
+          providerItems.push({ id: p, label: p, isConfigured: this.configuredProviders.has(p), configuredVia: this.configuredViaMap.get(p) });
         }
       }
       if (filteredAll.length > 0) {
         providerItems.push({ id: '__header__all', label: 'All Providers', isGroupHeader: true });
         for (const p of filteredAll) {
-          providerItems.push({ id: p, label: p, isConfigured: this.configuredProviders.has(p) });
+          providerItems.push({ id: p, label: p, isConfigured: this.configuredProviders.has(p), configuredVia: this.configuredViaMap.get(p) });
         }
       }
 

package/src/main.ts

@@ -80,6 +80,49 @@ function resolveShellEntrypointOwnership(): ShellEntrypointOwnership {
   };
 }
 
+// ---------------------------------------------------------------------------
+// CLI flag parsing (TUI shell entry point)
+// ---------------------------------------------------------------------------
+
+type TuiCliFlags = {
+  readonly provider: string | undefined;
+  readonly model: string | undefined;
+};
+
+function parseTuiCliFlags(argv: readonly string[]): TuiCliFlags {
+  let provider: string | undefined;
+  let model: string | undefined;
+
+  for (let i = 0; i < argv.length; i++) {
+    const arg = argv[i];
+    if (arg === '--help' || arg === '-h') {
+      // eslint-disable-next-line no-console
+      console.log([
+        'Usage: goodvibes [options]',
+        '',
+        'Options:',
+        '  --provider <id>          Override the provider from settings.json at startup',
+        '  --model <registryKey>    Override the model from settings.json at startup',
+        '                           Format: provider:modelId (e.g. inception:mercury-2)',
+        '                           If provider:modelId format is used, --provider is inferred',
+        '  --help, -h               Show this help message',
+      ].join('\n'));
+      process.exit(0);
+    }
+    if (arg === '--provider' && argv[i + 1] !== undefined) {
+      provider = argv[++i];
+    } else if (arg === '--model' && argv[i + 1] !== undefined) {
+      model = argv[++i];
+      // Infer provider from registryKey format (provider:modelId) if --provider not given
+      if (typeof model === 'string' && model.includes(':') && provider === undefined) {
+        provider = model.split(':')[0];
+      }
+    }
+  }
+
+  return { provider, model };
+}
+
 async function main() {
   const stdout = process.stdout;
   const stdin = process.stdin;
@@ -95,6 +138,15 @@ async function main() {
   });
   new GlobalNetworkTransportInstaller().install(configManager);
 
+  // Apply CLI flags — override settings.json before the provider registry is constructed
+  const cliFlags = parseTuiCliFlags(process.argv.slice(2));
+  if (cliFlags.provider !== undefined) {
+    configManager.set('provider.provider', cliFlags.provider);
+  }
+  if (cliFlags.model !== undefined) {
+    configManager.set('provider.model', cliFlags.model);
+  }
+
   // ── Bootstrap all runtime subsystems ─────────────────────────────────────
   // bootstrapRuntime initializes all subsystems in dependency order and returns
   // a fully-wired BootstrapContext. main.ts owns terminal setup, the render loop,

package/src/renderer/model-picker-overlay.ts

@@ -294,12 +294,19 @@ export function renderModelPickerOverlay(
         const isSelected = selectableIdx === picker.selectedIndex;
         const indicator = isSelected ? `${OVERLAY_GLYPHS.selected} ` : '  ';
         const checkmark = item.isConfigured ? '✓ ' : '  ';
-        const labelW = contentW - 2 - 2; // indicator(2) + checkmark(2)
+        // configuredVia badge: right-aligned short label (env/sub/anon)
+        const viaBadge = item.configuredVia === 'env' ? ' [env]'
+          : item.configuredVia === 'secrets' ? ' [key]'
+          : item.configuredVia === 'subscription' ? ' [sub]'
+          : item.configuredVia === 'anonymous' ? ' [anon]'
+          : '';
+        const badgeW = viaBadge.length;
+        const labelW = contentW - 2 - 2 - badgeW; // indicator(2) + checkmark(2) + badge
         const labelStr = item.label.length > labelW
           ? item.label.slice(0, labelW - 3) + '...'
           : item.label.padEnd(labelW);
         const row = createOverlayContentLine(width, layout, borderFg, isSelected ? selectedBg : DEFAULT_OVERLAY_PALETTE.bodyBg);
-        const rowText = indicator + checkmark + labelStr;
+        const rowText = indicator + checkmark + labelStr + viaBadge;
         putRowText(row, layout.margin + 2, contentW, fitDisplay(truncateDisplay(rowText, contentW), contentW), isSelected ? titleFg : bodyFg, isSelected ? selectedBg : DEFAULT_OVERLAY_PALETTE.bodyBg, isSelected);
         lines.push(row);
       }

package/src/shell/ui-openers.ts

@@ -27,6 +27,42 @@ type WireShellUiOpenersOptions = {
   render: () => void;
 };
 
+/**
+ * Derive the configuredVia tier for a provider.
+ * Checks env-vars first (env tier), then falls back to subscription (from configuredIds
+ * that aren't env-keyed). Returns undefined when not configured.
+ */
+function deriveConfiguredVia(
+  providerId: string,
+  configuredIds: Set<string>,
+  subscriptionManager: SubscriptionManager,
+): 'env' | 'secrets' | 'subscription' | 'anonymous' | undefined {
+  if (!configuredIds.has(providerId)) return undefined;
+
+  // Check if a subscription session is active for this provider
+  const subs = subscriptionManager.list();
+  if (subs.some((s) => s.provider === providerId)) return 'subscription';
+
+  // Assume env-var backed (anonymous providers don't appear in configuredIds)
+  return 'env';
+}
+
+/**
+ * Build a configuredViaMap for the given provider list.
+ */
+function buildConfiguredViaMap(
+  providers: string[],
+  configuredIds: Set<string>,
+  subscriptionManager: SubscriptionManager,
+): Map<string, 'env' | 'secrets' | 'subscription' | 'anonymous'> {
+  const map = new Map<string, 'env' | 'secrets' | 'subscription' | 'anonymous'>();
+  for (const p of providers) {
+    const via = deriveConfiguredVia(p, configuredIds, subscriptionManager);
+    if (via !== undefined) map.set(p, via);
+  }
+  return map;
+}
+
 export function wireShellUiOpeners(options: WireShellUiOpenersOptions): void {
   const {
     commandContext,
@@ -47,7 +83,10 @@ export function wireShellUiOpeners(options: WireShellUiOpenersOptions): void {
 
   commandContext.openModelPicker = () => {
     const models = providerRegistry.getSelectableModels();
-    input.modelPicker.configuredProviders = new Set(getConfiguredProviderIds());
+    const configuredIds = new Set(getConfiguredProviderIds());
+    input.modelPicker.configuredProviders = configuredIds;
+    const providerIds = [...new Set(models.map((m) => m.provider))];
+    input.modelPicker.configuredViaMap = buildConfiguredViaMap(providerIds, configuredIds, subscriptionManager);
     void getPinned().then((pinned) => {
       input.modelPicker.pinnedIds = new Set(pinned);
     });
@@ -59,7 +98,9 @@ export function wireShellUiOpeners(options: WireShellUiOpenersOptions): void {
 
   commandContext.openProviderPicker = () => {
     const providers = [...new Set(providerRegistry.listModels().map((model) => model.provider))];
-    input.modelPicker.configuredProviders = new Set(getConfiguredProviderIds());
+    const configuredIds = new Set(getConfiguredProviderIds());
+    input.modelPicker.configuredProviders = configuredIds;
+    input.modelPicker.configuredViaMap = buildConfiguredViaMap(providers, configuredIds, subscriptionManager);
     input.modalOpened('modelPicker');
     input.modelPicker.openProviders(providers, runtime.provider);
     render();

package/src/version.ts

@@ -6,7 +6,7 @@ import { join } from 'node:path';
 // The prebuild script updates the fallback value before compilation.
 // Uses import.meta.dir (Bun) to locate package.json relative to this file,
 // which is correct regardless of the process working directory.
-let _version = '0.19.0';
+let _version = '0.19.1';
 try {
   const pkg = JSON.parse(readFileSync(join(import.meta.dir, '..', 'package.json'), 'utf-8'));
   _version = pkg.version ?? _version;