@amodalai/amodal 0.2.1 → 0.2.3

package/src/commands/search.test.ts

@@ -1,171 +0,0 @@
-/**
- * @license
- * Copyright 2025 Amodal Labs, Inc.
- * SPDX-License-Identifier: MIT
- */
-
-import {describe, it, expect, vi, beforeEach} from 'vitest';
-
-const mockFindRepoRoot = vi.fn(() => '/test/repo');
-const mockEnsureNpmContext = vi.fn();
-const mockNpmSearch = vi.fn();
-const mockFromNpmName = vi.fn((npm: string) => {
-  const prefix = '@amodalai/';
-  if (!npm.startsWith(prefix)) throw new Error('Not amodal');
-  return npm.slice(prefix.length);
-});
-
-vi.mock('../shared/repo-discovery.js', () => ({
-  findRepoRoot: mockFindRepoRoot,
-}));
-
-vi.mock('@amodalai/core', () => ({
-  ensureNpmContext: mockEnsureNpmContext,
-  npmSearch: mockNpmSearch,
-  fromNpmName: mockFromNpmName,
-}));
-
-const mockPaths = {
-  root: '/test/repo/.amodal/packages',
-  npmDir: '/test/repo/.amodal/packages/.npm',
-  npmrc: '/test/repo/.amodal/packages/.npm/.npmrc',
-  packageJson: '/test/repo/.amodal/packages/.npm/package.json',
-  nodeModules: '/test/repo/.amodal/packages/.npm/node_modules',
-};
-
-describe('runSearch', () => {
-  let stderrOutput: string;
-  let stdoutOutput: string;
-
-  beforeEach(() => {
-    vi.clearAllMocks();
-    mockFindRepoRoot.mockReturnValue('/test/repo');
-    mockEnsureNpmContext.mockResolvedValue(mockPaths);
-    stderrOutput = '';
-    stdoutOutput = '';
-    vi.spyOn(process.stderr, 'write').mockImplementation((chunk) => {
-      stderrOutput += String(chunk);
-      return true;
-    });
-    vi.spyOn(process.stdout, 'write').mockImplementation((chunk) => {
-      stdoutOutput += String(chunk);
-      return true;
-    });
-  });
-
-  it('should search and display results', async () => {
-    mockNpmSearch.mockResolvedValue([
-      {name: '@amodalai/connection-salesforce', version: '2.1.0', description: 'Salesforce'},
-      {name: '@amodalai/connection-stripe', version: '1.0.0', description: 'Stripe'},
-    ]);
-
-    const {runSearch} = await import('./search.js');
-    const result = await runSearch({query: 'salesforce'});
-    expect(result).toBe(0);
-    expect(stdoutOutput).toContain('salesforce');
-    expect(stdoutOutput).toContain('stripe');
-  });
-
-  it('should filter by type', async () => {
-    mockNpmSearch.mockResolvedValue([
-      {name: '@amodalai/connection-salesforce', version: '2.1.0', description: 'Salesforce'},
-      {name: '@amodalai/skill-triage', version: '1.0.0', description: 'Triage'},
-    ]);
-
-    const {runSearch} = await import('./search.js');
-    const result = await runSearch({tag: 'connection'});
-    expect(result).toBe(0);
-    expect(stdoutOutput).toContain('salesforce');
-    expect(stdoutOutput).not.toContain('triage');
-  });
-
-  it('should output JSON when json flag set', async () => {
-    mockNpmSearch.mockResolvedValue([
-      {name: '@amodalai/connection-salesforce', version: '2.1.0', description: 'Salesforce'},
-    ]);
-
-    const {runSearch} = await import('./search.js');
-    const result = await runSearch({json: true});
-    expect(result).toBe(0);
-    const parsed = JSON.parse(stdoutOutput);
-    expect(parsed).toHaveLength(1);
-    expect(parsed[0].name).toBe('connection-salesforce');
-  });
-
-  it('should handle no results', async () => {
-    mockNpmSearch.mockResolvedValue([]);
-
-    const {runSearch} = await import('./search.js');
-    const result = await runSearch({query: 'nonexistent'});
-    expect(result).toBe(0);
-    expect(stderrOutput).toContain('No packages found');
-  });
-
-  it('should handle registry unreachable', async () => {
-    mockNpmSearch.mockRejectedValue(new Error('Registry unreachable'));
-
-    const {runSearch} = await import('./search.js');
-    const result = await runSearch({query: 'test'});
-    expect(result).toBe(1);
-    expect(stderrOutput).toContain('Search failed');
-  });
-
-  it('should filter out non-amodal packages', async () => {
-    mockNpmSearch.mockResolvedValue([
-      {name: '@amodalai/connection-salesforce', version: '1.0.0', description: 'Salesforce'},
-      {name: 'lodash', version: '4.0.0', description: 'Utility'},
-    ]);
-
-    const {runSearch} = await import('./search.js');
-    const result = await runSearch({});
-    expect(result).toBe(0);
-    expect(stdoutOutput).toContain('salesforce');
-    expect(stdoutOutput).not.toContain('lodash');
-  });
-
-  it('should work without repo root', async () => {
-    mockFindRepoRoot.mockImplementation(() => {
-      throw new Error('Not found');
-    });
-    mockNpmSearch.mockResolvedValue([
-      {name: '@amodalai/skill-triage', version: '1.0.0', description: 'Triage'},
-    ]);
-
-    const {runSearch} = await import('./search.js');
-    const result = await runSearch({query: 'triage'});
-    expect(result).toBe(0);
-  });
-
-  it('should list results with short names', async () => {
-    mockNpmSearch.mockResolvedValue([
-      {name: '@amodalai/connection-salesforce', version: '2.0.0', description: 'Salesforce'},
-      {name: '@amodalai/skill-triage', version: '1.0.0', description: 'Triage'},
-    ]);
-
-    const {runSearch} = await import('./search.js');
-    const result = await runSearch({});
-    expect(result).toBe(0);
-    expect(stdoutOutput).toContain('connection-salesforce');
-    expect(stdoutOutput).toContain('skill-triage');
-  });
-
-  it('should handle npm context failure', async () => {
-    mockEnsureNpmContext.mockRejectedValue(new Error('Permission denied'));
-
-    const {runSearch} = await import('./search.js');
-    const result = await runSearch({});
-    expect(result).toBe(1);
-    expect(stderrOutput).toContain('Failed to set up npm context');
-  });
-
-  it('should show package count', async () => {
-    mockNpmSearch.mockResolvedValue([
-      {name: '@amodalai/connection-salesforce', version: '1.0.0', description: 'Salesforce'},
-      {name: '@amodalai/connection-stripe', version: '1.0.0', description: 'Stripe'},
-    ]);
-
-    const {runSearch} = await import('./search.js');
-    await runSearch({});
-    expect(stderrOutput).toContain('2 packages found');
-  });
-});

package/src/commands/search.ts

@@ -1,120 +0,0 @@
-/**
- * @license
- * Copyright 2025 Amodal Labs, Inc.
- * SPDX-License-Identifier: MIT
- */
-
-import type {CommandModule} from 'yargs';
-import {
-  ensureNpmContext,
-  fromNpmName,
-  npmSearch,
-} from '@amodalai/core';
-import {findRepoRoot} from '../shared/repo-discovery.js';
-
-export interface SearchOptions {
-  cwd?: string;
-  query?: string;
-  tag?: string;
-  json?: boolean;
-}
-
-/**
- * Search the registry for packages.
- * Returns 0 on success, 1 on error.
- */
-export async function runSearch(options: SearchOptions = {}): Promise<number> {
-  let repoPath: string | undefined;
-  try {
-    repoPath = findRepoRoot(options.cwd);
-  } catch {
-    // Not in a repo — use temp context
-  }
-
-  let paths;
-  try {
-    paths = await ensureNpmContext(repoPath ?? process.cwd());
-  } catch (err) {
-    const msg = err instanceof Error ? err.message : String(err);
-    process.stderr.write(`[search] Failed to set up npm context: ${msg}\n`);
-    return 1;
-  }
-
-  // Build query
-  let query = '@amodalai/';
-  if (options.tag) {
-    query += `${options.tag}-`;
-  }
-  if (options.query) {
-    query += options.query;
-  }
-
-  process.stderr.write(`[search] Searching for "${query}"...\n`);
-
-  let results;
-  try {
-    results = await npmSearch(paths, query);
-  } catch (err) {
-    const msg = err instanceof Error ? err.message : String(err);
-    process.stderr.write(`[search] Search failed: ${msg}\n`);
-    return 1;
-  }
-
-  // Filter to @amodalai/ packages only
-  results = results.filter((r) => r.name.startsWith('@amodalai/'));
-
-  // Filter by tag if specified (match against npm name prefix)
-  if (options.tag) {
-    const prefix = `@amodalai/${options.tag}-`;
-    results = results.filter((r) => r.name.startsWith(prefix));
-  }
-
-  if (results.length === 0) {
-    process.stderr.write('[search] No packages found.\n');
-    return 0;
-  }
-
-  if (options.json) {
-    const output = results.map((r) => ({
-      npm: r.name,
-      name: fromNpmName(r.name),
-      version: r.version,
-      description: r.description,
-    }));
-    process.stdout.write(JSON.stringify(output, null, 2) + '\n');
-    return 0;
-  }
-
-  // Display results
-  const nameWidth = Math.max(4, ...results.map((r) => fromNpmName(r.name).length));
-  const versionWidth = Math.max(7, ...results.map((r) => r.version.length));
-
-  for (const r of results) {
-    const shortName = fromNpmName(r.name);
-    process.stdout.write(`  ${shortName.padEnd(nameWidth)}   ${r.version.padEnd(versionWidth)}   ${r.description}\n`);
-  }
-
-  process.stderr.write(`\n[search] ${results.length} package${results.length === 1 ? '' : 's'} found.\n`);
-  return 0;
-}
-
-export const searchCommand: CommandModule = {
-  command: 'search [query]',
-  describe: 'Search the registry for packages',
-  builder: (yargs) =>
-    yargs
-      .positional('query', {type: 'string', describe: 'Search query'})
-      .option('tag', {type: 'string', describe: 'Filter by tag prefix (e.g., connection, skill)'})
-      .option('json', {type: 'boolean', default: false, describe: 'Output as JSON'}),
-  handler: async (argv) => {
-    const code = await runSearch({
-      // eslint-disable-next-line @typescript-eslint/no-unsafe-type-assertion
-      query: argv['query'] as string | undefined,
-      // eslint-disable-next-line @typescript-eslint/no-unsafe-type-assertion
-      tag: argv['tag'] as string | undefined,
-      // eslint-disable-next-line @typescript-eslint/no-unsafe-type-assertion
-      json: argv['json'] as boolean,
-    });
-    process.exit(code);
-  },
-};

package/src/commands/update.test.ts

@@ -1,256 +0,0 @@
-/**
- * @license
- * Copyright 2025 Amodal Labs, Inc.
- * SPDX-License-Identifier: MIT
- */
-
-import {describe, it, expect, vi, beforeEach} from 'vitest';
-
-const mockFindRepoRoot = vi.fn(() => '/test/repo');
-const mockEnsureNpmContext = vi.fn();
-const mockReadLockFile = vi.fn();
-const mockNpmViewVersions = vi.fn();
-const mockNpmInstall = vi.fn();
-const mockFromNpmName = vi.fn((npm: string) => npm.replace('@amodalai/', ''));
-const mockDiscoverInstalledPackages = vi.fn();
-const mockBuildLockFile = vi.fn();
-
-vi.mock('../shared/repo-discovery.js', () => ({
-  findRepoRoot: mockFindRepoRoot,
-}));
-
-vi.mock('@amodalai/core', () => ({
-  ensureNpmContext: mockEnsureNpmContext,
-  readLockFile: mockReadLockFile,
-  npmViewVersions: mockNpmViewVersions,
-  npmInstall: mockNpmInstall,
-  fromNpmName: mockFromNpmName,
-  discoverInstalledPackages: mockDiscoverInstalledPackages,
-  buildLockFile: mockBuildLockFile,
-}));
-
-vi.mock('semver', () => ({
-  maxSatisfying: vi.fn((versions: string[], range: string) => {
-    // Simple mock: return the last version for '*', first matching for '^'
-    if (range === '*') return versions[versions.length - 1];
-    // For ^x.y.z, return last version with same major
-    const major = range.replace('^', '').split('.')[0];
-    const matching = versions.filter((v) => v.startsWith(major + '.'));
-    return matching.length > 0 ? matching[matching.length - 1] : null;
-  }),
-}));
-
-const mockPaths = {
-  root: '/test/repo/.amodal/packages',
-  npmDir: '/test/repo/.amodal/packages/.npm',
-  npmrc: '/test/repo/.amodal/packages/.npm/.npmrc',
-  packageJson: '/test/repo/.amodal/packages/.npm/package.json',
-  nodeModules: '/test/repo/.amodal/packages/.npm/node_modules',
-};
-
-describe('runUpdate', () => {
-  let stderrOutput: string;
-
-  beforeEach(() => {
-    vi.clearAllMocks();
-    mockFindRepoRoot.mockReturnValue('/test/repo');
-    mockEnsureNpmContext.mockResolvedValue(mockPaths);
-    mockNpmInstall.mockResolvedValue({version: '2.0.0', integrity: 'sha512-new'});
-    mockDiscoverInstalledPackages.mockResolvedValue([]);
-    mockBuildLockFile.mockResolvedValue(undefined);
-    stderrOutput = '';
-    vi.spyOn(process.stderr, 'write').mockImplementation((chunk) => {
-      stderrOutput += String(chunk);
-      return true;
-    });
-  });
-
-  it('should update all packages', async () => {
-    mockReadLockFile.mockResolvedValue({
-      lockVersion: 2,
-      packages: {
-        '@amodalai/connection-salesforce': {version: '1.0.0', integrity: 'sha512-old'},
-      },
-    });
-    mockNpmViewVersions.mockResolvedValue(['1.0.0', '1.1.0', '1.2.0']);
-
-    const {runUpdate} = await import('./update.js');
-    const result = await runUpdate();
-    expect(result).toBe(0);
-    expect(mockNpmInstall).toHaveBeenCalled();
-    expect(stderrOutput).toContain('Updated');
-  });
-
-  it('should update single package by name', async () => {
-    mockReadLockFile.mockResolvedValue({
-      lockVersion: 2,
-      packages: {
-        '@amodalai/connection-salesforce': {version: '1.0.0', integrity: 'sha512-old'},
-        '@amodalai/skill-triage': {version: '1.0.0', integrity: 'sha512-old'},
-      },
-    });
-    mockNpmViewVersions.mockResolvedValue(['1.0.0', '1.1.0']);
-
-    const {runUpdate} = await import('./update.js');
-    const result = await runUpdate({name: 'connection-salesforce'});
-    expect(result).toBe(0);
-    expect(mockNpmViewVersions).toHaveBeenCalledTimes(1);
-  });
-
-  it('should report already up to date', async () => {
-    mockReadLockFile.mockResolvedValue({
-      lockVersion: 2,
-      packages: {
-        '@amodalai/connection-salesforce': {version: '1.2.0', integrity: 'sha512-old'},
-      },
-    });
-    mockNpmViewVersions.mockResolvedValue(['1.0.0', '1.1.0', '1.2.0']);
-
-    const {runUpdate} = await import('./update.js');
-    const result = await runUpdate();
-    expect(result).toBe(0);
-    expect(stderrOutput).toContain('up to date');
-    expect(mockNpmInstall).not.toHaveBeenCalled();
-  });
-
-  it('should show dry run output', async () => {
-    mockReadLockFile.mockResolvedValue({
-      lockVersion: 2,
-      packages: {
-        '@amodalai/connection-salesforce': {version: '1.0.0', integrity: 'sha512-old'},
-      },
-    });
-    mockNpmViewVersions.mockResolvedValue(['1.0.0', '1.1.0']);
-
-    const {runUpdate} = await import('./update.js');
-    const result = await runUpdate({dryRun: true});
-    expect(result).toBe(0);
-    expect(stderrOutput).toContain('Dry run');
-    expect(stderrOutput).toContain('1.1.0');
-    expect(mockNpmInstall).not.toHaveBeenCalled();
-  });
-
-  it('should use latest flag for cross-major updates', async () => {
-    mockReadLockFile.mockResolvedValue({
-      lockVersion: 2,
-      packages: {
-        '@amodalai/connection-salesforce': {version: '1.0.0', integrity: 'sha512-old'},
-      },
-    });
-    mockNpmViewVersions.mockResolvedValue(['1.0.0', '1.1.0', '2.0.0', '3.0.0']);
-
-    const {runUpdate} = await import('./update.js');
-    const result = await runUpdate({latest: true});
-    expect(result).toBe(0);
-    expect(stderrOutput).toContain('3.0.0');
-  });
-
-  it('should return 1 when no lock file', async () => {
-    mockReadLockFile.mockResolvedValue(null);
-
-    const {runUpdate} = await import('./update.js');
-    const result = await runUpdate();
-    expect(result).toBe(1);
-    expect(stderrOutput).toContain('No lock file');
-  });
-
-  it('should return 1 when repo not found', async () => {
-    mockFindRepoRoot.mockImplementation(() => {
-      throw new Error('Not found');
-    });
-
-    const {runUpdate} = await import('./update.js');
-    const result = await runUpdate();
-    expect(result).toBe(1);
-    expect(stderrOutput).toContain('Not found');
-  });
-
-  it('should handle registry unreachable', async () => {
-    mockReadLockFile.mockResolvedValue({
-      lockVersion: 2,
-      packages: {
-        '@amodalai/connection-salesforce': {version: '1.0.0', integrity: 'sha512-old'},
-      },
-    });
-    mockNpmViewVersions.mockRejectedValue(new Error('Registry unreachable'));
-
-    const {runUpdate} = await import('./update.js');
-    const result = await runUpdate();
-    expect(result).toBe(1);
-    expect(stderrOutput).toContain('Failed to check');
-  });
-
-  it('should continue on partial failure', async () => {
-    mockReadLockFile.mockResolvedValue({
-      lockVersion: 2,
-      packages: {
-        '@amodalai/connection-salesforce': {version: '1.0.0', integrity: 'sha512-old'},
-        '@amodalai/skill-triage': {version: '1.0.0', integrity: 'sha512-old'},
-      },
-    });
-    mockNpmViewVersions
-      .mockResolvedValueOnce(['1.0.0', '1.1.0'])
-      .mockResolvedValueOnce(['1.0.0', '1.1.0']);
-    mockNpmInstall
-      .mockRejectedValueOnce(new Error('Install failed'))
-      .mockResolvedValueOnce({version: '1.1.0', integrity: 'sha512-new'});
-
-    const {runUpdate} = await import('./update.js');
-    const result = await runUpdate();
-    expect(result).toBe(1);
-    expect(stderrOutput).toContain('1 of 2');
-  });
-
-  it('should handle empty lock file', async () => {
-    mockReadLockFile.mockResolvedValue({lockVersion: 2, packages: {}});
-
-    const {runUpdate} = await import('./update.js');
-    const result = await runUpdate();
-    expect(result).toBe(0);
-    expect(stderrOutput).toContain('No packages installed');
-  });
-
-  it('should report no matching packages', async () => {
-    mockReadLockFile.mockResolvedValue({
-      lockVersion: 2,
-      packages: {
-        '@amodalai/connection-salesforce': {version: '1.0.0', integrity: 'sha512-old'},
-      },
-    });
-
-    const {runUpdate} = await import('./update.js');
-    const result = await runUpdate({name: 'nonexistent'});
-    expect(result).toBe(0);
-    expect(stderrOutput).toContain('No matching packages');
-  });
-
-  it('should rebuild lock file after update', async () => {
-    mockReadLockFile.mockResolvedValue({
-      lockVersion: 2,
-      packages: {
-        '@amodalai/connection-salesforce': {version: '1.0.0', integrity: 'sha512-old'},
-      },
-    });
-    mockNpmViewVersions.mockResolvedValue(['1.0.0', '1.1.0']);
-
-    const {runUpdate} = await import('./update.js');
-    await runUpdate();
-    expect(mockDiscoverInstalledPackages).toHaveBeenCalledWith(mockPaths);
-    expect(mockBuildLockFile).toHaveBeenCalled();
-  });
-
-  it('should use singular form for 1 package', async () => {
-    mockReadLockFile.mockResolvedValue({
-      lockVersion: 2,
-      packages: {
-        '@amodalai/connection-salesforce': {version: '1.0.0', integrity: 'sha512-old'},
-      },
-    });
-    mockNpmViewVersions.mockResolvedValue(['1.0.0', '1.1.0']);
-
-    const {runUpdate} = await import('./update.js');
-    await runUpdate();
-    expect(stderrOutput).toContain('1 package updated');
-    expect(stderrOutput).not.toContain('1 packages');
-  });
-});