savant-code 0.0.19 → 0.0.21

package/cli/src/components/validation-error-popover.tsx

@@ -1,191 +0,0 @@
-import { pluralize } from '@savant-code/common/util/string'
-import React, { useState } from 'react'
-
-import { Button } from './button'
-import { TerminalLink } from './terminal-link'
-import { useTheme } from '../hooks/use-theme'
-import { getLoadedAgentsData } from '../utils/local-agent-registry'
-import { openFileAtPath } from '../utils/open-file'
-import { formatValidationError } from '../utils/validation-error-formatting'
-import { NETWORK_ERROR_ID } from '../utils/validation-error-helpers'
-
-import type { LocalAgentInfo } from '../utils/local-agent-registry'
-import type { FeedbackCategory } from '@savant-code/common/constants/feedback'
-
-interface ValidationErrorPopoverProps {
-  errors: Array<{ id: string; message: string }>
-  onOpenFeedback?: (options: {
-    category: FeedbackCategory
-    footerMessage: string
-    errors: Array<{ id: string; message: string }>
-  }) => void
-  onClose?: () => void
-}
-
-export const ValidationErrorPopover: React.FC<ValidationErrorPopoverProps> = ({
-  errors,
-  onOpenFeedback,
-  onClose,
-}) => {
-  const theme = useTheme()
-  const [isReportHovered, setIsReportHovered] = useState(false)
-  const [isCloseHovered, setIsCloseHovered] = useState(false)
-  const loadedAgentsData = getLoadedAgentsData()
-
-  const errorCount = errors.length
-
-  return (
-    <box
-      style={{
-        flexDirection: 'column',
-        padding: 1,
-        border: true,
-        borderStyle: 'single',
-        borderColor: theme.warning,
-        backgroundColor: theme.surface,
-        width: '100%',
-      }}
-    >
-      <box style={{ flexDirection: 'column', gap: 0 }}>
-        <box
-          style={{
-            flexDirection: 'row',
-            justifyContent: 'space-between',
-            alignItems: 'center',
-          }}
-        >
-          <text style={{ fg: theme.warning, wrapMode: 'word' }}>
-            {pluralize(errorCount, 'Error')}
-          </text>
-          {onClose && (
-            <Button
-              style={{ paddingRight: 1 }}
-              onClick={onClose}
-              onMouseOver={() => setIsCloseHovered(true)}
-              onMouseOut={() => setIsCloseHovered(false)}
-            >
-              <text
-                style={{
-                  fg: isCloseHovered ? theme.foreground : theme.secondary,
-                  wrapMode: 'none',
-                }}
-              >
-                [x]
-              </text>
-            </Button>
-          )}
-        </box>
-
-        <box style={{ flexDirection: 'column', paddingTop: 1, gap: 0 }}>
-          {errors.slice(0, 3).map((error, index) => {
-            const errorId = error.id ?? ''
-            const agentId = errorId.replace(/_\d+$/, '')
-            const isNetworkError = errorId === NETWORK_ERROR_ID
-            const agentInfo = loadedAgentsData?.agents.find(
-              (a) => a.id === agentId,
-            ) as LocalAgentInfo | undefined
-
-            const { fieldName, message } = formatValidationError(error.message)
-            const errorMsg = fieldName ? `${fieldName}: ${message}` : message
-
-            // Special handling for network errors - show message only without ID
-            if (isNetworkError) {
-              return (
-                <box
-                  key={errorId || `error-${index}`}
-                  style={{ flexDirection: 'column', paddingTop: 0.5 }}
-                >
-                  <text style={{ fg: theme.muted, wrapMode: 'word' }}>
-                    {`• ${errorMsg}`}
-                  </text>
-                </box>
-              )
-            }
-
-            if (agentInfo?.filePath) {
-              return (
-                <box
-                  key={errorId || `error-${index}`}
-                  style={{ flexDirection: 'column', paddingTop: 0.5 }}
-                >
-                  <text style={{ fg: theme.muted, wrapMode: 'word' }}>
-                    {'• '}
-                    <TerminalLink
-                      text={agentId}
-                      color={theme.info}
-                      inline={true}
-                      onActivate={() => openFileAtPath(agentInfo.filePath)}
-                    />
-                  </text>
-                  <text
-                    style={{
-                      fg: theme.muted,
-                      wrapMode: 'word',
-                      paddingLeft: 2,
-                    }}
-                  >
-                    {errorMsg}
-                  </text>
-                </box>
-              )
-            }
-
-            return (
-              <box
-                key={errorId || `error-${index}`}
-                style={{ flexDirection: 'column', paddingTop: 0.5 }}
-              >
-                <text style={{ fg: theme.muted, wrapMode: 'word' }}>
-                  {`• ${agentId || 'Unknown'}`}
-                </text>
-                <text
-                  style={{
-                    fg: theme.muted,
-                    wrapMode: 'word',
-                    paddingLeft: 2,
-                  }}
-                >
-                  {errorMsg}
-                </text>
-              </box>
-            )
-          })}
-
-          {errorCount > 3 && (
-            <text
-              style={{ fg: theme.muted, wrapMode: 'word', paddingTop: 0.5 }}
-            >
-              {`+ ${errorCount - 3} more`}
-            </text>
-          )}
-        </box>
-
-        {onOpenFeedback && (
-          <box style={{ paddingTop: 1, justifyContent: 'flex-end' }}>
-            <Button
-              onClick={() =>
-                onOpenFeedback({
-                  category: 'app_bug',
-                  footerMessage: 'Validation errors are auto-attached',
-                  errors,
-                })
-              }
-              onMouseOver={() => setIsReportHovered(true)}
-              onMouseOut={() => setIsReportHovered(false)}
-            >
-              <text style={{ wrapMode: 'none' }}>
-                {isReportHovered ? (
-                  <u>
-                    <span fg={theme.info}>Report issue</span>
-                  </u>
-                ) : (
-                  <span fg={theme.info}>Report issue</span>
-                )}
-              </text>
-            </Button>
-          </box>
-        )}
-      </box>
-    </box>
-  )
-}

package/cli/src/data/slash-commands.ts

@@ -1,306 +0,0 @@
-import { CHATGPT_OAUTH_ENABLED } from '@savant-code/common/constants/chatgpt-oauth'
-
-import { AGENT_MODES, IS_SAVANT_FREE } from '../utils/constants'
-
-import type { SkillsMap } from '@savant-code/common/types/skill'
-
-export interface SlashCommand {
-  id: string
-  label: string
-  description: string
-  aliases?: string[]
-  /**
-   * If true, this command can be invoked without a leading slash when the
-   * input matches the command id exactly (no arguments).
-   */
-  implicitCommand?: boolean
-  /**
-   * If set, selecting this command inserts this text into the input field
-   * instead of executing a command. Useful for agent shortcuts.
-   */
-  insertText?: string
-}
-
-// Generate mode commands from the AGENT_MODES constant (excluded in SavantFree)
-const MODE_COMMANDS: SlashCommand[] = IS_SAVANT_FREE
-  ? []
-  : AGENT_MODES.map((mode) => ({
-      id: `mode:${mode.toLowerCase()}`,
-      label: `mode:${mode.toLowerCase()}`,
-      description: `Switch to ${mode} mode`,
-      aliases: [`model:${mode.toLowerCase()}`],
-    }))
-
-// FID-007 D1: `init` was listed here (menu-only removal) while the command
-// registry still registered it in free builds — free users could run an
-// undiscoverable command. Aligned to the registry: `/init` is available in
-// free builds, so it stays in the menu.
-// FID-007 V4: exported for the gating-parity test.
-export const SAVANT_FREE_REMOVED_COMMAND_IDS = new Set([
-  'ads:enable',
-  'ads:disable',
-  'usage',
-  'subscribe',
-  'image',
-  'publish',
-])
-
-export const SAVANT_FREE_ONLY_COMMAND_IDS = new Set([
-  'connect',
-  'plan',
-  'end-session',
-])
-
-const ALL_SLASH_COMMANDS: SlashCommand[] = [
-  {
-    id: 'help',
-    label: 'help',
-    description: 'Display keyboard shortcuts and tips',
-    aliases: ['h', '?'],
-    implicitCommand: true,
-  },
-  {
-    id: 'diagnostics',
-    label: 'diagnostics',
-    description: 'Show local CLI resource usage and terminal tool process IDs',
-    aliases: ['diag', 'processes'],
-  },
-  {
-    id: 'health',
-    label: 'health',
-    description: 'Show CLI and backend health status',
-    aliases: ['status', 'check'],
-  },
-  ...(CHATGPT_OAUTH_ENABLED
-    ? [
-        {
-          id: 'connect',
-          label: 'connect',
-          description: 'Connect your ChatGPT account',
-          aliases: ['connect:chatgpt', 'chatgpt'],
-        },
-      ]
-    : []),
-
-  {
-    id: 'ads:enable',
-    label: 'ads:enable',
-    description: 'Enable contextual ads',
-  },
-  {
-    id: 'ads:disable',
-    label: 'ads:disable',
-    description: 'Disable contextual ads',
-  },
-  {
-    id: 'telemetry',
-    label: 'telemetry',
-    description: 'Show or change remote analytics consent',
-    aliases: ['analytics'],
-  },
-  {
-    id: 'init',
-    label: 'init',
-    description: 'Create a starter knowledge.md file',
-    implicitCommand: true,
-  },
-
-  {
-    id: 'usage',
-    label: 'usage',
-    description: 'View credits and subscription quota',
-    aliases: ['credits'],
-  },
-  {
-    id: 'subscribe',
-    label: 'subscribe',
-    description: 'Subscribe to get more usage',
-    aliases: ['strong', 'sub', 'buy-credits'],
-  },
-  {
-    id: 'interview',
-    label: 'interview',
-    description:
-      'AI asks a series of questions to flesh out request into a spec',
-  },
-  {
-    id: 'plan',
-    label: 'plan',
-    description: 'Create a plan for how to implement a request',
-  },
-  {
-    id: 'review',
-    label: 'review',
-    description: 'Review code changes',
-  },
-  {
-    id: 'new',
-    label: 'new',
-    description: 'Clear the conversation history and start a new chat',
-    aliases: ['n', 'clear', 'c', 'reset'],
-    implicitCommand: true,
-  },
-  {
-    id: 'history',
-    label: 'history',
-    description: 'Browse and resume past conversations',
-    aliases: ['chats'],
-  },
-  {
-    id: 'copy',
-    label: 'copy',
-    description:
-      'Copy the full conversation (messages + tool results) to the clipboard',
-    aliases: ['copy-chat', 'export'],
-  },
-
-  {
-    id: 'feedback',
-    label: 'feedback',
-    description: IS_SAVANT_FREE
-      ? 'Share general feedback about SavantFree'
-      : 'Share general feedback about SavantCode',
-  },
-  {
-    id: 'bash',
-    label: 'bash',
-    description: 'Enter bash mode ("!" at beginning enters bash mode)',
-    aliases: ['!'],
-  },
-  {
-    id: 'image',
-    label: 'image',
-    description: 'Attach an image file (or Ctrl+V to paste from clipboard)',
-    aliases: ['img', 'attach'],
-  },
-  {
-    id: 'publish',
-    label: 'publish',
-    description: 'Publish an agent to the registry',
-  },
-  ...MODE_COMMANDS,
-
-  {
-    id: 'theme:toggle',
-    label: 'theme:toggle',
-    description: 'Toggle between light and dark mode',
-  },
-  {
-    id: 'end-session',
-    label: 'end-session',
-    description: 'End your free session (lets you switch model)',
-    aliases: ['model'],
-  },
-  ...(IS_SAVANT_FREE
-    ? []
-    : [
-        {
-          id: 'model',
-          label: 'model',
-          description:
-            'Switch the active model (e.g. /model anthropic/claude-opus-4.6)',
-          aliases: ['switch-model'],
-        },
-        {
-          id: 'provider',
-          label: 'provider',
-          description:
-            'Configure a provider API key (stored locally and masked)',
-        },
-      ]),
-  {
-    id: 'login',
-    label: 'login',
-    description: 'Open the login screen',
-    aliases: ['signin'],
-    implicitCommand: true,
-  },
-  {
-    id: 'logout',
-    label: 'logout',
-    description: 'Sign out of your session',
-    aliases: ['signout'],
-    implicitCommand: true,
-  },
-  {
-    id: 'exit',
-    label: 'exit',
-    description: 'Quit the CLI',
-    aliases: ['quit', 'q'],
-    implicitCommand: true,
-  },
-  {
-    id: 'goal',
-    label: 'goal',
-    description:
-      'Run agent until a condition is met (e.g. /goal all tests pass)',
-    aliases: ['g'],
-  },
-  {
-    id: 'loop',
-    label: 'loop',
-    description: 'Run a prompt on a cadence (e.g. /loop 1h "check staging")',
-    aliases: ['repeat'],
-  },
-  {
-    id: 'verify',
-    label: 'verify',
-    description:
-      'Run typechecks across all workspaces (or one: sdk, common, agent-runtime, cli)',
-    aliases: ['typecheck', 'check'],
-  },
-  {
-    id: 'permissions',
-    label: 'permissions',
-    description:
-      'Show or set the sandbox permission mode: /permissions [safe|prompt|unsafe]',
-    aliases: ['sandbox', 'safety'],
-  },
-  {
-    id: 'rewind',
-    label: 'rewind',
-    description:
-      'Restore files and/or conversation to before a previous turn (/rewind, /rewind 2, /rewind 2 fork)',
-    aliases: ['undo', 'checkpoint'],
-  },
-]
-
-export const SLASH_COMMANDS = IS_SAVANT_FREE
-  ? ALL_SLASH_COMMANDS.filter(
-      (cmd) => !SAVANT_FREE_REMOVED_COMMAND_IDS.has(cmd.id),
-    )
-  : ALL_SLASH_COMMANDS.filter(
-      (cmd) => !SAVANT_FREE_ONLY_COMMAND_IDS.has(cmd.id),
-    )
-
-export const SLASHLESS_COMMAND_IDS = new Set(
-  SLASH_COMMANDS.filter((cmd) => cmd.implicitCommand).map((cmd) =>
-    cmd.id.toLowerCase(),
-  ),
-)
-
-/** Maximum description length for skill commands in the slash menu */
-const SKILL_MENU_DESCRIPTION_MAX_LENGTH = 50
-
-function truncateDescription(description: string): string {
-  if (description.length <= SKILL_MENU_DESCRIPTION_MAX_LENGTH) {
-    return description
-  }
-  return description.slice(0, SKILL_MENU_DESCRIPTION_MAX_LENGTH - 1) + '…'
-}
-
-/**
- * Returns SLASH_COMMANDS merged with skill commands.
- * Skills become slash commands that users can invoke directly.
- */
-export function getSlashCommandsWithSkills(skills: SkillsMap): SlashCommand[] {
-  const skillCommands: SlashCommand[] = Object.values(skills).map((skill) => ({
-    id: `skill:${skill.name}`,
-    label: `skill:${skill.name}`,
-    description: truncateDescription(skill.description),
-  }))
-
-  const commands = [...SLASH_COMMANDS, ...skillCommands]
-
-  return commands
-}

package/cli/src/hooks/__tests__/holds-live-savant-free-slot.test.ts

@@ -1,49 +0,0 @@
-import { describe, test, expect } from 'bun:test'
-
-import { holdsLiveSavantFreeSlot } from '../use-savant-free-session'
-
-import type { SavantFreeSession } from '../../types/savant-free-session'
-
-// Gate predicate for both the message queue (may queued work still be sent?)
-// and slot release (is there a server row to DELETE?). The load-bearing cases
-// are the two 'ended' variants: WITH an instance id we're inside the server's
-// grace window and requests are still admissible; WITHOUT one the session is
-// fully over and any request would be rejected by the chat-completions gate.
-
-const activeSession: SavantFreeSession = {
-  status: 'active',
-  accessTier: 'full',
-  instanceId: 'inst-1',
-  model: 'deepseek/deepseek-v4',
-  admittedAt: new Date().toISOString(),
-  expiresAt: new Date(Date.now() + 60_000).toISOString(),
-  remainingMs: 60_000,
-}
-
-describe('holdsLiveSavantFreeSlot', () => {
-  test('active session holds a slot', () => {
-    expect(holdsLiveSavantFreeSlot(activeSession)).toBe(true)
-  })
-
-  test('ended WITH instance id (grace window) still holds a slot', () => {
-    expect(
-      holdsLiveSavantFreeSlot({ status: 'ended', instanceId: 'inst-1' }),
-    ).toBe(true)
-  })
-
-  test('ended WITHOUT instance id (post-grace) does not hold a slot', () => {
-    expect(holdsLiveSavantFreeSlot({ status: 'ended' })).toBe(false)
-  })
-
-  test('null session does not hold a slot', () => {
-    expect(holdsLiveSavantFreeSlot(null)).toBe(false)
-  })
-
-  test('pre-join and terminal states do not hold a slot', () => {
-    expect(holdsLiveSavantFreeSlot({ status: 'none' })).toBe(false)
-    expect(holdsLiveSavantFreeSlot({ status: 'superseded' })).toBe(false)
-    expect(
-      holdsLiveSavantFreeSlot({ status: 'takeover_prompt', model: 'm' }),
-    ).toBe(false)
-  })
-})