@covibes/zeroshot 5.4.0 → 5.4.1

package/lib/completion.js

@@ -1,174 +0,0 @@
-const omelette = require('omelette');
-const fs = require('fs');
-const path = require('path');
-
-function setupCompletion() {
-  const complete = omelette('zeroshot');
-
-  complete.on('start', ({ reply }) => {
-    reply(['--issue', '--text', '--config']);
-  });
-
-  complete.on('list', ({ reply }) => {
-    reply([]);
-  });
-
-  complete.on('status', ({ reply }) => {
-    // Complete with cluster IDs
-    try {
-      const clustersDir = path.join(process.env.HOME, '.zeroshot', 'clusters');
-      if (fs.existsSync(clustersDir)) {
-        const clusterIds = fs
-          .readdirSync(clustersDir)
-          .filter((f) => f.endsWith('.json'))
-          .map((f) => f.replace('.json', ''));
-        reply(clusterIds);
-      } else {
-        reply([]);
-      }
-    } catch {
-      reply([]);
-    }
-  });
-
-  complete.on('logs', ({ reply }) => {
-    // Complete with cluster IDs or flags
-    try {
-      const clustersDir = path.join(process.env.HOME, '.zeroshot', 'clusters');
-      const completions = ['--follow', '-f', '--limit', '-n'];
-
-      if (fs.existsSync(clustersDir)) {
-        const clusterIds = fs
-          .readdirSync(clustersDir)
-          .filter((f) => f.endsWith('.json'))
-          .map((f) => f.replace('.json', ''));
-        reply([...clusterIds, ...completions]);
-      } else {
-        reply(completions);
-      }
-    } catch {
-      reply(['--follow', '-f', '--limit', '-n']);
-    }
-  });
-
-  complete.on('stop', ({ reply }) => {
-    // Complete with active cluster IDs
-    try {
-      const clustersDir = path.join(process.env.HOME, '.zeroshot', 'clusters');
-      if (fs.existsSync(clustersDir)) {
-        const clusterIds = fs
-          .readdirSync(clustersDir)
-          .filter((f) => f.endsWith('.json'))
-          .map((f) => f.replace('.json', ''));
-        reply(clusterIds);
-      } else {
-        reply([]);
-      }
-    } catch {
-      reply([]);
-    }
-  });
-
-  complete.on('kill', ({ reply }) => {
-    // Complete with active cluster IDs
-    try {
-      const clustersDir = path.join(process.env.HOME, '.zeroshot', 'clusters');
-      if (fs.existsSync(clustersDir)) {
-        const clusterIds = fs
-          .readdirSync(clustersDir)
-          .filter((f) => f.endsWith('.json'))
-          .map((f) => f.replace('.json', ''));
-        reply(clusterIds);
-      } else {
-        reply([]);
-      }
-    } catch {
-      reply([]);
-    }
-  });
-
-  complete.on('export', ({ reply }) => {
-    // Complete with cluster IDs
-    try {
-      const clustersDir = path.join(process.env.HOME, '.zeroshot', 'clusters');
-      if (fs.existsSync(clustersDir)) {
-        const clusterIds = fs
-          .readdirSync(clustersDir)
-          .filter((f) => f.endsWith('.json'))
-          .map((f) => f.replace('.json', ''));
-        reply(clusterIds);
-      } else {
-        reply([]);
-      }
-    } catch {
-      reply([]);
-    }
-  });
-
-  complete.on('finish', ({ reply }) => {
-    // Complete with cluster IDs and flags
-    try {
-      const clustersDir = path.join(process.env.HOME, '.zeroshot', 'clusters');
-      const completions = ['--merge', '--pr', '--push'];
-
-      if (fs.existsSync(clustersDir)) {
-        const clusterIds = fs
-          .readdirSync(clustersDir)
-          .filter((f) => f.endsWith('.json'))
-          .map((f) => f.replace('.json', ''));
-        reply([...clusterIds, ...completions]);
-      } else {
-        reply(completions);
-      }
-    } catch {
-      reply(['--merge', '--pr', '--push']);
-    }
-  });
-
-  complete.on('resume', ({ reply }) => {
-    // Complete with cluster IDs
-    try {
-      const clustersDir = path.join(process.env.HOME, '.zeroshot', 'clusters');
-      if (fs.existsSync(clustersDir)) {
-        const clusterIds = fs
-          .readdirSync(clustersDir)
-          .filter((f) => f.endsWith('.json'))
-          .map((f) => f.replace('.json', ''));
-        reply(clusterIds);
-      } else {
-        reply([]);
-      }
-    } catch {
-      reply([]);
-    }
-  });
-
-  complete.on('ui', ({ reply }) => {
-    reply(['--port']);
-  });
-
-  complete.on('watch', ({ reply }) => {
-    reply(['--filter', '--refresh-rate', 'running', 'stopped', 'all']);
-  });
-
-  // Default completion - show commands
-  complete.on('', ({ reply }) => {
-    reply([
-      'start',
-      'list',
-      'status',
-      'logs',
-      'stop',
-      'kill',
-      'finish',
-      'resume',
-      'export',
-      'ui',
-      'watch',
-    ]);
-  });
-
-  complete.init();
-}
-
-module.exports = { setupCompletion };

package/lib/docker-config.js

@@ -1,220 +0,0 @@
-/**
- * Docker mount configuration for isolation mode
- * Fully generic - no hardcoded paths, all configurable
- */
-
-/**
- * Built-in mount presets
- * Uses $HOME placeholder - resolved at runtime based on dockerContainerHome setting
- * Host paths use ~ (expanded to host user's home)
- * Container paths use $HOME (expanded to configured container home)
- */
-const MOUNT_PRESETS = {
-  gh: { host: '~/.config/gh', container: '$HOME/.config/gh', readonly: false },
-  git: { host: '~/.gitconfig', container: '$HOME/.gitconfig', readonly: true },
-  ssh: { host: '~/.ssh', container: '$HOME/.ssh', readonly: true },
-  aws: { host: '~/.aws', container: '$HOME/.aws', readonly: true },
-  azure: { host: '~/.azure', container: '$HOME/.azure', readonly: true },
-  kube: { host: '~/.kube', container: '$HOME/.kube', readonly: true },
-  terraform: { host: '~/.terraform.d', container: '$HOME/.terraform.d', readonly: false },
-  gcloud: { host: '~/.config/gcloud', container: '$HOME/.config/gcloud', readonly: true },
-  claude: { host: '~/.claude', container: '$HOME/.claude', readonly: true },
-  codex: { host: '~/.config/codex', container: '$HOME/.config/codex', readonly: true },
-  gemini: { host: '~/.config/gemini', container: '$HOME/.config/gemini', readonly: true },
-  opencode: {
-    host: '~/.local/share/opencode',
-    container: '$HOME/.local/share/opencode',
-    readonly: true,
-  },
-};
-
-/**
- * Environment variables to auto-pass for each preset
- * Supports:
- * - Simple: 'VAR_NAME' (pass if set in host env)
- * - Pattern: 'VAR_*' (pass all matching vars)
- * - Forced: 'VAR=value' (always set to value)
- * - Empty: 'VAR=' (always set to empty string)
- */
-const { CLAUDE_AUTH_ENV_VARS } = require('./settings/claude-auth');
-
-const ENV_PRESETS = {
-  aws: ['AWS_REGION', 'AWS_DEFAULT_REGION', 'AWS_PROFILE', 'AWS_PAGER='],
-  azure: ['AZURE_SUBSCRIPTION_ID', 'AZURE_TENANT_ID', 'AZURE_CLIENT_ID'],
-  gcloud: ['CLOUDSDK_CORE_PROJECT', 'GOOGLE_CLOUD_PROJECT'],
-  kube: ['KUBECONFIG'],
-  terraform: ['TF_VAR_*'],
-  claude: CLAUDE_AUTH_ENV_VARS,
-};
-
-/**
- * Resolve mount config to actual mount specs
- * @param {Array<string|object>} config - Preset names or {host, container, readonly?} objects
- * @param {object} options - Resolution options
- * @param {string} [options.containerHome='/root'] - Container home directory for $HOME expansion
- * @returns {Array<{host: string, container: string, readonly: boolean}>}
- */
-function resolveMounts(config, options = {}) {
-  if (!Array.isArray(config)) {
-    throw new Error('dockerMounts must be an array');
-  }
-
-  const containerHome = options.containerHome || '/root';
-
-  return config.flatMap((item) => {
-    if (typeof item === 'string') {
-      const preset = MOUNT_PRESETS[item];
-      if (!preset) {
-        throw new Error(
-          `Unknown mount preset: "${item}". Valid presets: ${Object.keys(MOUNT_PRESETS).join(', ')}`
-        );
-      }
-      return {
-        host: preset.host,
-        container: preset.container.replace(/\$HOME/g, containerHome),
-        readonly: preset.readonly,
-      };
-    }
-
-    if (typeof item === 'object' && item !== null) {
-      if (!item.host || !item.container) {
-        throw new Error('Custom mount must have "host" and "container" properties');
-      }
-      return {
-        host: item.host,
-        container: item.container.replace(/\$HOME/g, containerHome),
-        readonly: item.readonly !== false,
-      };
-    }
-
-    throw new Error(
-      `Invalid mount config: ${JSON.stringify(item)}. Use preset name or {host, container, readonly?}`
-    );
-  });
-}
-
-/**
- * Resolve env vars to pass based on enabled presets + explicit additions
- * @param {Array<string|object>} mountConfig - The mount config (to detect presets)
- * @param {Array<string>} extraEnvs - Additional env vars to pass
- * @returns {Array<string>} - List of env var specs
- */
-function resolveEnvs(mountConfig, extraEnvs = []) {
-  const envs = new Set(extraEnvs);
-
-  for (const item of mountConfig) {
-    if (typeof item === 'string' && ENV_PRESETS[item]) {
-      for (const envVar of ENV_PRESETS[item]) {
-        envs.add(envVar);
-      }
-    }
-  }
-
-  return [...envs];
-}
-
-/**
- * Expand env var patterns and resolve values
- * Supports:
- * - Simple: 'VAR_NAME' (use value from env if set)
- * - Pattern: 'VAR_*' (expand to all matching vars)
- * - Forced: 'VAR=value' (always set to value)
- * - Empty: 'VAR=' (always set to empty string)
- *
- * @param {Array<string>} envVars - List of env var specs
- * @param {object} env - Environment object (defaults to process.env)
- * @returns {Array<{name: string, value: string|null, forced: boolean}>}
- */
-function expandEnvPatterns(envVars, env = process.env) {
-  const result = [];
-
-  for (const envVar of envVars) {
-    // Forced value: VAR=value or VAR=
-    if (envVar.includes('=')) {
-      const [name, ...valueParts] = envVar.split('=');
-      const value = valueParts.join('=');
-      result.push({ name, value, forced: true });
-    }
-    // Pattern: VAR_*
-    else if (envVar.endsWith('*')) {
-      const prefix = envVar.slice(0, -1);
-      for (const key of Object.keys(env)) {
-        if (key.startsWith(prefix)) {
-          result.push({ name: key, value: null, forced: false });
-        }
-      }
-    }
-    // Simple: VAR_NAME
-    else {
-      result.push({ name: envVar, value: null, forced: false });
-    }
-  }
-
-  return result;
-}
-
-/**
- * Validate mount config
- * @param {unknown} value - Value to validate
- * @returns {string|null} - Error message if invalid, null if valid
- */
-function validateMountConfig(value) {
-  if (!Array.isArray(value)) {
-    return 'dockerMounts must be an array';
-  }
-
-  for (const item of value) {
-    if (typeof item === 'string') {
-      if (!MOUNT_PRESETS[item]) {
-        return `Unknown mount preset: "${item}". Valid: ${Object.keys(MOUNT_PRESETS).join(', ')}`;
-      }
-    } else if (typeof item === 'object' && item !== null) {
-      if (!item.host) {
-        return 'Custom mount missing "host" property';
-      }
-      if (!item.container) {
-        return 'Custom mount missing "container" property';
-      }
-      if (item.readonly !== undefined && typeof item.readonly !== 'boolean') {
-        return '"readonly" must be a boolean';
-      }
-    } else {
-      return `Invalid mount: ${JSON.stringify(item)}. Use preset name or {host, container, readonly?}`;
-    }
-  }
-
-  return null;
-}
-
-/**
- * Validate env passthrough config
- * @param {unknown} value - Value to validate
- * @returns {string|null} - Error message if invalid, null if valid
- */
-function validateEnvPassthrough(value) {
-  if (!Array.isArray(value)) {
-    return 'dockerEnvPassthrough must be an array';
-  }
-
-  for (const item of value) {
-    if (typeof item !== 'string') {
-      return `Invalid env var: ${JSON.stringify(item)}. Must be a string`;
-    }
-    // Allow: VAR, VAR_*, VAR=value, VAR=
-    if (!/^[A-Z_][A-Z0-9_]*(\*|=.*)?$/i.test(item)) {
-      return `Invalid env var spec: "${item}". Use VAR, VAR_*, or VAR=value`;
-    }
-  }
-
-  return null;
-}
-
-module.exports = {
-  MOUNT_PRESETS,
-  ENV_PRESETS,
-  resolveMounts,
-  resolveEnvs,
-  expandEnvPatterns,
-  validateMountConfig,
-  validateEnvPassthrough,
-};

package/lib/git-remote-utils.js

@@ -1,165 +0,0 @@
-/**
- * Git remote URL parsing and provider detection.
- * Automatically detects issue provider from git remote URL.
- */
-
-const { execSync } = require('../src/lib/safe-exec');
-
-/**
- * Parse git remote URL into structured provider context.
- * Supports GitHub, GitLab, and Azure DevOps (cloud + self-hosted).
- * Handles both HTTPS and SSH URL formats.
- *
- * @param {string} remoteUrl - Git remote URL
- * @returns {Object|null} Provider context or null if unparseable
- *
- * @example
- * parseGitRemoteUrl('https://github.com/org/repo.git')
- * // → { provider: 'github', host: 'github.com', org: 'org', repo: 'repo', fullRepo: 'org/repo' }
- *
- * @example
- * parseGitRemoteUrl('git@gitlab.com:org/repo.git')
- * // → { provider: 'gitlab', host: 'gitlab.com', org: 'org', repo: 'repo', fullRepo: 'org/repo' }
- *
- * @example
- * parseGitRemoteUrl('https://dev.azure.com/myorg/myproject/_git/myrepo')
- * // → { provider: 'azure-devops', host: 'dev.azure.com', azureOrg: 'https://dev.azure.com/myorg', azureProject: 'myproject', repo: 'myrepo' }
- */
-function parseGitRemoteUrl(remoteUrl) {
-  if (!remoteUrl || typeof remoteUrl !== 'string') {
-    return null;
-  }
-
-  const url = remoteUrl.trim();
-
-  // Normalize SSH URLs to HTTPS format for easier parsing
-  // git@host:path → https://host/path
-  let normalizedUrl = url;
-  const sshMatch = url.match(/^git@([^:]+):(.+)$/);
-  if (sshMatch) {
-    const [, host, path] = sshMatch;
-    normalizedUrl = `https://${host}/${path}`;
-  }
-
-  // Remove .git suffix if present
-  normalizedUrl = normalizedUrl.replace(/\.git$/, '');
-
-  // Azure DevOps: https://dev.azure.com/org/project/_git/repo
-  // Azure Legacy: https://org.visualstudio.com/project/_git/repo
-  // Azure SSH: git@ssh.dev.azure.com:v3/org/project/repo
-  const azureMatch =
-    normalizedUrl.match(/https:\/\/dev\.azure\.com\/([^/]+)\/([^/]+)\/_git\/([^/]+)/) ||
-    normalizedUrl.match(/https:\/\/([^.]+)\.visualstudio\.com\/([^/]+)\/_git\/([^/]+)/) ||
-    // After normalization, `git@ssh.dev.azure.com:v3/org/project/repo` becomes
-    // `https://ssh.dev.azure.com/v3/org/project/repo`
-    normalizedUrl.match(/https:\/\/ssh\.dev\.azure\.com\/v3\/([^/]+)\/([^/]+)\/([^/]+)/);
-
-  if (azureMatch) {
-    const [, orgPart, project, repo] = azureMatch;
-    // For dev.azure.com, org is the first path segment
-    // For visualstudio.com, org is the subdomain
-    const isLegacy = normalizedUrl.includes('visualstudio.com');
-    const azureOrg = isLegacy
-      ? `https://${orgPart}.visualstudio.com`
-      : `https://dev.azure.com/${orgPart}`;
-
-    return {
-      provider: 'azure-devops',
-      host: isLegacy ? `${orgPart}.visualstudio.com` : 'dev.azure.com',
-      azureOrg,
-      azureProject: project,
-      repo,
-    };
-  }
-
-  // GitHub: https://github.com/org/repo
-  // GitLab: https://gitlab.com/org/repo (or self-hosted)
-  // Generic: https://host/org/repo
-  const httpsMatch = normalizedUrl.match(/https?:\/\/([^/]+)\/([^/]+)\/([^/]+)/);
-  if (httpsMatch) {
-    const [, host, org, repo] = httpsMatch;
-
-    let provider = null;
-    if (host === 'github.com') {
-      provider = 'github';
-    } else if (host.includes('gitlab')) {
-      // Matches gitlab.com or any gitlab.* subdomain or *gitlab* in hostname
-      provider = 'gitlab';
-    } else {
-      // Unknown provider - could be self-hosted GitLab or other
-      // Return null to fall back to settings
-      return null;
-    }
-
-    return {
-      provider,
-      host,
-      org,
-      repo,
-      fullRepo: `${org}/${repo}`,
-    };
-  }
-
-  return null;
-}
-
-/**
- * Detect git repository context from current working directory.
- * Returns provider context extracted from git remote URL.
- *
- * @param {string} [cwd=process.cwd()] - Directory to check
- * @returns {Object|null} Git context or null
- *
- * Gracefully returns null for:
- * - Not in git repository
- * - No remote configured
- * - Remote URL unparseable
- * - Git command fails
- *
- * @example
- * // In a GitHub repo with remote
- * detectGitContext()
- * // → { provider: 'github', host: 'github.com', org: 'myorg', repo: 'myrepo', fullRepo: 'myorg/myrepo' }
- *
- * @example
- * // Not in git repo or no remote
- * detectGitContext()
- * // → null
- */
-function detectGitContext(cwd = process.cwd()) {
-  try {
-    // Check if we're in a git repository
-    execSync('git rev-parse --git-dir', {
-      cwd,
-      stdio: 'pipe',
-      encoding: 'utf8',
-    });
-  } catch {
-    // Not a git repository
-    return null;
-  }
-
-  try {
-    // Try to get remote URL (origin by default)
-    const remoteUrl = execSync('git remote get-url origin', {
-      cwd,
-      stdio: 'pipe',
-      encoding: 'utf8',
-    }).trim();
-
-    if (!remoteUrl) {
-      return null;
-    }
-
-    // Parse the remote URL to extract provider context
-    return parseGitRemoteUrl(remoteUrl);
-  } catch {
-    // No remote configured or command failed
-    return null;
-  }
-}
-
-module.exports = {
-  parseGitRemoteUrl,
-  detectGitContext,
-};

package/lib/id-detector.js

@@ -1,56 +0,0 @@
-/**
- * ID Detector - Determines if an ID is a task or cluster
- *
- * Strategy:
- * 1. Check if ID exists in cluster storage
- * 2. If not, check if ID exists in task storage (SQLite)
- * 3. Return type: 'cluster', 'task', or null
- */
-
-const path = require('path');
-const fs = require('fs');
-const os = require('os');
-const Database = require('better-sqlite3');
-
-// Storage paths
-const CLUSTER_DIR = path.join(os.homedir(), '.zeroshot');
-const TASK_DIR = path.join(os.homedir(), '.claude-zeroshot');
-const TASK_DB_FILE = path.join(TASK_DIR, 'store.db');
-
-/**
- * Detect if ID is a cluster or task
- * @param {string} id - The ID to check
- * @returns {'cluster'|'task'|null} - Type of ID or null if not found
- */
-function detectIdType(id) {
-  // Check clusters
-  const clusterFile = path.join(CLUSTER_DIR, 'clusters.json');
-  if (fs.existsSync(clusterFile)) {
-    try {
-      const clusters = JSON.parse(fs.readFileSync(clusterFile, 'utf8'));
-      if (clusters[id]) {
-        return 'cluster';
-      }
-    } catch {
-      // Ignore parse errors
-    }
-  }
-
-  // Check tasks in SQLite
-  if (fs.existsSync(TASK_DB_FILE)) {
-    try {
-      const db = new Database(TASK_DB_FILE, { readonly: true, timeout: 5000 });
-      const row = db.prepare('SELECT id FROM tasks WHERE id = ?').get(id);
-      db.close();
-      if (row) {
-        return 'task';
-      }
-    } catch {
-      // Ignore DB errors
-    }
-  }
-
-  return null;
-}
-
-module.exports = { detectIdType };

package/lib/provider-defaults.js

@@ -1,62 +0,0 @@
-/**
- * Provider defaults module
- *
- * Separated from settings.js to break circular dependency:
- * - settings.js requires provider defaults
- * - providers require settings.js (for loadSettings, getClaudeCommand, etc.)
- *
- * CRITICAL: This module should NOT import from settings.js
- */
-
-// Cache provider defaults to avoid repeated instantiation
-let _providerDefaultsCache = null;
-
-/**
- * Build provider default settings by instantiating each provider
- * and calling getDefaultSettings()
- * @returns {Object} Map of provider name to default settings
- */
-function buildProviderDefaults() {
-  const { listProviders, getProvider } = require('../src/providers');
-
-  const defaults = {};
-  for (const providerName of listProviders()) {
-    try {
-      const provider = getProvider(providerName);
-      defaults[providerName] = provider.getDefaultSettings();
-    } catch (error) {
-      // If provider fails to instantiate, use basic defaults
-      console.warn(`Warning: Could not get defaults for ${providerName}: ${error.message}`);
-      defaults[providerName] = {
-        maxLevel: 'level3',
-        minLevel: 'level1',
-        defaultLevel: 'level2',
-        levelOverrides: {},
-      };
-    }
-  }
-  return defaults;
-}
-
-/**
- * Get or build cached provider defaults
- * @returns {Object} Provider defaults
- */
-function getProviderDefaults() {
-  if (!_providerDefaultsCache) {
-    _providerDefaultsCache = buildProviderDefaults();
-  }
-  return _providerDefaultsCache;
-}
-
-/**
- * Clear the provider defaults cache (primarily for testing)
- */
-function clearProviderDefaultsCache() {
-  _providerDefaultsCache = null;
-}
-
-module.exports = {
-  getProviderDefaults,
-  clearProviderDefaultsCache,
-};