@amodalai/amodal 0.3.89 → 0.3.91

package/src/commands/deploy.ts

@@ -6,10 +6,7 @@
 
 import type {CommandModule} from 'yargs';
 import {findRepoRoot} from '../shared/repo-discovery.js';
-import {PlatformClient} from '../shared/platform-client.js';
 import {runValidate} from './validate.js';
-import {createRepoTarball} from '../shared/tarball.js';
-import {readProjectLink} from './link.js';
 
 export interface DeployOptions {
   cwd?: string;
@@ -19,7 +16,7 @@ export interface DeployOptions {
 }
 
 /**
- * Deploy to the platform: validate → tarball → trigger build → poll.
+ * Validate a repo before cloud deployment.
  *
  * Returns 0 on success, 1 on error.
  */
@@ -48,69 +45,10 @@ export async function runDeploy(options: DeployOptions = {}): Promise<number> {
     return 0;
   }
 
-  // Create platform client
-  let client: PlatformClient;
-  try {
-    client = await PlatformClient.create();
-  } catch (err) {
-    const msg = err instanceof Error ? err.message : String(err);
-    process.stderr.write(`[deploy] ${msg}\n`);
-    return 1;
-  }
-
-  const projectLink = await readProjectLink();
-  const appId = projectLink?.appId;
-
-  if (!appId) {
-    process.stderr.write('[deploy] No app linked. Run `amodal deploy link` first.\n');
-    return 1;
-  }
-
-  // Create tarball
-  process.stderr.write('[deploy] Creating tarball...\n');
-  const tarballPath = await createRepoTarball(repoPath);
-
-  try {
-    // Trigger remote build
-    process.stderr.write('[deploy] Triggering build...\n');
-    const buildResult = await client.triggerRemoteBuild(appId, environment, tarballPath, options.message);
-    const buildId = buildResult.buildId;
-
-    process.stderr.write(`[deploy] Build ${buildId} accepted. Waiting for completion...\n`);
-
-    // Poll for completion
-    const maxWaitMs = 5 * 60 * 1000; // 5 minutes
-    const pollIntervalMs = 3000;
-    const startTime = Date.now();
-
-    while (Date.now() - startTime < maxWaitMs) {
-      await new Promise((r) => setTimeout(r, pollIntervalMs));
-
-      const status = await client.getBuildStatus(buildId);
-
-      if (status.status === 'complete') {
-        process.stderr.write(`[deploy] Deployed ${status.deployId} to ${status.environment ?? environment}\n`);
-        return 0;
-      }
-
-      if (status.status === 'error') {
-        process.stderr.write(`[deploy] Build failed: ${status.error ?? 'unknown error'}\n`);
-        return 1;
-      }
-
-      // Still building — continue polling
-    }
-
-    process.stderr.write(`[deploy] Build timed out after 5 minutes. Build ${buildId} may still be running.\n`);
-    return 1;
-  } catch (err) {
-    const msg = err instanceof Error ? err.message : String(err);
-    process.stderr.write(`[deploy] Deploy failed: ${msg}\n`);
-    return 1;
-  } finally {
-    const {unlinkSync} = await import('node:fs');
-    try { unlinkSync(tarballPath); } catch { /* best-effort */ }
-  }
+  process.stderr.write(
+    '[deploy] Cloud deployment is not included in the OSS CLI. Use `amodal deploy build` to create a local snapshot, then deploy it with your hosted control-plane tooling.\n',
+  );
+  return 1;
 }
 
 export const deployCommand: CommandModule = {

package/src/commands/deployments.ts

@@ -5,7 +5,6 @@
  */
 
 import type {CommandModule} from 'yargs';
-import {PlatformClient} from '../shared/platform-client.js';
 
 export interface DeploymentsOptions {
   env?: string;
@@ -17,44 +16,9 @@ export interface DeploymentsOptions {
  * List deployment history.
  */
 export async function runDeployments(options: DeploymentsOptions = {}): Promise<number> {
-  let client: PlatformClient;
-  try {
-    client = new PlatformClient();
-  } catch (err) {
-    const msg = err instanceof Error ? err.message : String(err);
-    process.stderr.write(`[deployments] ${msg}\n`);
-    return 1;
-  }
-
-  try {
-    const deployments = await client.listDeployments({
-      environment: options.env,
-      limit: options.limit ?? 10,
-    });
-
-    if (options.json) {
-      process.stdout.write(JSON.stringify(deployments, null, 2) + '\n');
-      return 0;
-    }
-
-    if (deployments.length === 0) {
-      process.stderr.write('[deployments] No deployments found.\n');
-      return 0;
-    }
-
-    for (const d of deployments) {
-      const active = d.isActive ? ' [ACTIVE]' : '';
-      const msg = d.message ? ` — ${d.message}` : '';
-      const sha = d.commitSha ? ` (${d.commitSha.slice(0, 7)})` : '';
-      process.stdout.write(`${d.id}  ${d.environment}${active}  ${d.createdBy ?? d.source}${sha}  ${d.createdAt}${msg}\n`);
-    }
-
-    return 0;
-  } catch (err) {
-    const msg = err instanceof Error ? err.message : String(err);
-    process.stderr.write(`[deployments] Failed: ${msg}\n`);
-    return 1;
-  }
+  void options;
+  process.stderr.write('[deployments] Hosted deployment history is not included in the OSS CLI.\n');
+  return 1;
 }
 
 export const deploymentsCommand: CommandModule = {

package/src/commands/dev.ts

@@ -203,13 +203,11 @@ export function formatLineForDev(line: string): string | null {
       return null;
     }
     case 'tool_log': {
-      // Tools call ctx.log(...) for noteworthy progress (e.g.
-      // install_template emits "Cloned whodatdev/template-X into
-      // agent repo (N connection packages installed)"). When fired
+      // Tools call ctx.log(...) for noteworthy progress. When fired
       // during an intent run the callId starts with `intent_`, so
-      // these lines pass the quiet filter — but as raw JSON they
-      // bury the useful message inside callId/session noise. Strip
-      // to a clean nested bullet.
+      // these lines pass the quiet filter, but as raw JSON they bury
+      // the useful message inside callId/session noise. Strip to a
+      // clean nested bullet.
       const msg = str('message');
       if (!msg) return null;
       return `    · ${msg}`;
@@ -753,9 +751,8 @@ Or add it to your agent's .env file:
     // Two flows watched by the same poller:
     //
     //   1. AUTO-BOOT — runtime didn't start at CLI launch (no manifest
-    //      yet). Once amodal.json lands (commit_setup, the user-button
-    //      commit-setup endpoint, or init-repo's skip-onboarding
-    //      write), boot the runtime in place. Studio's runtime URL
+    //      yet). Once amodal.json lands, boot the runtime in place.
+    //      Studio's runtime URL
     //      probe picks it up on the next tick.
     //
     //   2. AUTO-RESTART — runtime is already up but amodal.json has
@@ -766,7 +763,7 @@ Or add it to your agent's .env file:
     //      bundle in memory and the new packages/config never load.
     //
     // 500ms debounce after detecting a change — lets the writer
-    // (commit_setup's atomic rename, init-repo's full write) settle
+    // (commit_setup's atomic rename or a direct file write) settle
     // before loadRepo tries to read.
     // -------------------------------------------------------------------
 

package/src/commands/experiment.ts

@@ -5,128 +5,22 @@
  */
 
 import type {CommandModule} from 'yargs';
-import {resolvePlatformConfig} from '../shared/platform-client.js';
 
 export interface ExperimentOptions {
   action: 'create' | 'deploy' | 'watch' | 'list';
   name?: string;
   id?: string;
-  platformUrl?: string;
-  platformApiKey?: string;
   controlConfig?: string;
   variantConfig?: string;
 }
 
 /**
- * Manage model experiments via the platform API.
+ * Hosted experiment management is not available from the OSS CLI.
  */
 export async function runExperimentCommand(options: ExperimentOptions): Promise<void> {
-  let platformUrl: string;
-  let apiKey: string;
-  try {
-    const config = await resolvePlatformConfig({
-      url: options.platformUrl,
-      apiKey: options.platformApiKey,
-    });
-    platformUrl = config.url;
-    apiKey = config.apiKey;
-  } catch (err) {
-    const msg = err instanceof Error ? err.message : String(err);
-    process.stderr.write(`[experiment] ${msg}\n`);
-    process.exit(1);
-  }
-
-  const headers = {
-    'Content-Type': 'application/json',
-    'Authorization': `Bearer ${apiKey}`,
-  };
-
-  switch (options.action) {
-    case 'list': {
-      const res = await fetch(`${platformUrl}/api/experiments`, {headers});
-      if (!res.ok) {
-        process.stderr.write(`[experiment] HTTP ${res.status}\n`);
-        return;
-      }
-      // eslint-disable-next-line @typescript-eslint/no-unsafe-type-assertion -- platform response
-      const data = await res.json() as {experiments: Array<Record<string, unknown>>};
-      if (data.experiments.length === 0) {
-        process.stdout.write('No experiments found.\n');
-        return;
-      }
-      for (const exp of data.experiments) {
-        process.stdout.write(`${exp['id']}  ${exp['name']}  [${exp['status']}]\n`);
-      }
-      break;
-    }
-
-    case 'create': {
-      if (!options.name) {
-        process.stderr.write('[experiment] --name is required for create\n');
-        return;
-      }
-      const controlConfig: Record<string, unknown> = options.controlConfig
-        // eslint-disable-next-line @typescript-eslint/no-unsafe-type-assertion -- CLI JSON input
-        ? JSON.parse(options.controlConfig) as Record<string, unknown>
-        : {};
-      const variantConfig: Record<string, unknown> = options.variantConfig
-        // eslint-disable-next-line @typescript-eslint/no-unsafe-type-assertion -- CLI JSON input
-        ? JSON.parse(options.variantConfig) as Record<string, unknown>
-        : {};
-
-      const res = await fetch(`${platformUrl}/api/experiments`, {
-        method: 'POST',
-        headers,
-        body: JSON.stringify({name: options.name, controlConfig, variantConfig}),
-      });
-      if (!res.ok) {
-        process.stderr.write(`[experiment] HTTP ${res.status}\n`);
-        return;
-      }
-      // eslint-disable-next-line @typescript-eslint/no-unsafe-type-assertion -- platform response
-      const {id} = await res.json() as {id: string};
-      process.stdout.write(`Created experiment: ${id}\n`);
-      break;
-    }
-
-    case 'deploy': {
-      if (!options.id) {
-        process.stderr.write('[experiment] --id is required for deploy\n');
-        return;
-      }
-      const res = await fetch(`${platformUrl}/api/experiments/${options.id}`, {
-        method: 'PUT',
-        headers,
-        body: JSON.stringify({status: 'deployed'}),
-      });
-      if (!res.ok) {
-        process.stderr.write(`[experiment] HTTP ${res.status}\n`);
-        return;
-      }
-      process.stdout.write(`Deployed experiment: ${options.id}\n`);
-      break;
-    }
-
-    case 'watch': {
-      if (!options.id) {
-        process.stderr.write('[experiment] --id is required for watch\n');
-        return;
-      }
-      const res = await fetch(`${platformUrl}/api/experiments/${options.id}`, {headers});
-      if (!res.ok) {
-        process.stderr.write(`[experiment] HTTP ${res.status}\n`);
-        return;
-      }
-      // eslint-disable-next-line @typescript-eslint/no-unsafe-type-assertion -- platform response
-      const exp = await res.json() as Record<string, unknown>;
-      process.stdout.write(JSON.stringify(exp, null, 2) + '\n');
-      break;
-    }
-
-    default:
-      process.stderr.write(`[experiment] Unknown action: ${options.action}\n`);
-      break;
-  }
+  void options;
+  process.stderr.write('[experiment] Hosted experiment management is not included in the OSS CLI.\n');
+  process.exit(1);
 }
 
 export const experimentCommand: CommandModule = {

package/src/commands/promote.ts

@@ -5,7 +5,6 @@
  */
 
 import type {CommandModule} from 'yargs';
-import {PlatformClient} from '../shared/platform-client.js';
 
 export interface PromoteOptions {
   fromEnv: string;
@@ -16,26 +15,9 @@ export interface PromoteOptions {
  * Promote a deployment from one environment to another.
  */
 export async function runPromote(options: PromoteOptions): Promise<number> {
-  let client: PlatformClient;
-  try {
-    client = new PlatformClient();
-  } catch (err) {
-    const msg = err instanceof Error ? err.message : String(err);
-    process.stderr.write(`[promote] ${msg}\n`);
-    return 1;
-  }
-
-  const toEnv = options.toEnv ?? 'production';
-
-  try {
-    const result = await client.promote(options.fromEnv, toEnv);
-    process.stderr.write(`[promote] Promoted from ${options.fromEnv} to ${toEnv}: ${result.id}\n`);
-    return 0;
-  } catch (err) {
-    const msg = err instanceof Error ? err.message : String(err);
-    process.stderr.write(`[promote] Failed: ${msg}\n`);
-    return 1;
-  }
+  void options;
+  process.stderr.write('[promote] Hosted deployment promotion is not included in the OSS CLI.\n');
+  return 1;
 }
 
 export const promoteCommand: CommandModule = {

package/src/commands/rollback.ts

@@ -5,7 +5,6 @@
  */
 
 import type {CommandModule} from 'yargs';
-import {PlatformClient} from '../shared/platform-client.js';
 
 export interface RollbackOptions {
   deployId?: string;
@@ -17,29 +16,9 @@ export interface RollbackOptions {
  * Rollback to a previous deployment.
  */
 export async function runRollback(options: RollbackOptions = {}): Promise<number> {
-  let client: PlatformClient;
-  try {
-    client = new PlatformClient();
-  } catch (err) {
-    const msg = err instanceof Error ? err.message : String(err);
-    process.stderr.write(`[rollback] ${msg}\n`);
-    return 1;
-  }
-
-  const environment = options.env ?? 'production';
-
-  try {
-    const result = await client.rollback({
-      deployId: options.deployId,
-      environment,
-    });
-    process.stderr.write(`[rollback] Rolled back to ${result.id} in ${result.environment}\n`);
-    return 0;
-  } catch (err) {
-    const msg = err instanceof Error ? err.message : String(err);
-    process.stderr.write(`[rollback] Failed: ${msg}\n`);
-    return 1;
-  }
+  void options;
+  process.stderr.write('[rollback] Hosted deployment rollback is not included in the OSS CLI.\n');
+  return 1;
 }
 
 export const rollbackCommand: CommandModule = {

package/src/commands/secrets.test.ts

@@ -4,22 +4,15 @@
  * SPDX-License-Identifier: MIT
  */
 
-import {describe, it, expect, vi, beforeEach, afterEach} from 'vitest';
+import {afterEach, beforeEach, describe, expect, it, vi} from 'vitest';
 
 describe('runSecrets', () => {
   let stderrOutput: string;
   let stdoutOutput: string;
-  // eslint-disable-next-line @typescript-eslint/no-explicit-any
-  let fetchSpy: any;
-  const originalEnv = process.env;
+  let fetchSpy: ReturnType<typeof vi.spyOn>;
 
   beforeEach(() => {
     vi.clearAllMocks();
-    process.env = {
-      ...originalEnv,
-      PLATFORM_API_URL: 'https://api.example.com',
-      PLATFORM_API_KEY: 'test-key',
-    };
     stderrOutput = '';
     stdoutOutput = '';
     vi.spyOn(process.stderr, 'write').mockImplementation((chunk) => {
@@ -34,135 +27,20 @@ describe('runSecrets', () => {
   });
 
   afterEach(() => {
-    process.env = originalEnv;
-    fetchSpy.mockRestore();
+    vi.restoreAllMocks();
   });
 
-  it('should set a secret', async () => {
-    fetchSpy.mockResolvedValue(new Response('{}', {status: 200}));
-
-    const {runSecrets} = await import('./secrets.js');
-    const result = await runSecrets({subcommand: 'set', key: 'DB_URL', value: 'postgres://...'});
-    expect(result).toBe(0);
-    expect(stderrOutput).toContain('set successfully');
-  });
-
-  it('should return 1 when key missing for set', async () => {
-    const {runSecrets} = await import('./secrets.js');
-    const result = await runSecrets({subcommand: 'set'});
-    expect(result).toBe(1);
-    expect(stderrOutput).toContain('Missing key');
-  });
-
-  it('should return 1 when value missing for set', async () => {
-    const {runSecrets} = await import('./secrets.js');
-    const result = await runSecrets({subcommand: 'set', key: 'DB_URL'});
-    expect(result).toBe(1);
-    expect(stderrOutput).toContain('Missing value');
-  });
-
-  it('should list secrets', async () => {
-    fetchSpy.mockResolvedValue(new Response(JSON.stringify([
-      {key: 'DB_URL'},
-      {key: 'API_KEY'},
-    ]), {status: 200}));
-
-    const {runSecrets} = await import('./secrets.js');
-    const result = await runSecrets({subcommand: 'list'});
-    expect(result).toBe(0);
-    expect(stdoutOutput).toContain('DB_URL');
-    expect(stdoutOutput).toContain('API_KEY');
-  });
-
-  it('should list secrets as JSON', async () => {
-    fetchSpy.mockResolvedValue(new Response(JSON.stringify([{key: 'DB_URL'}]), {status: 200}));
-
-    const {runSecrets} = await import('./secrets.js');
-    const result = await runSecrets({subcommand: 'list', json: true});
-    expect(result).toBe(0);
-    const parsed = JSON.parse(stdoutOutput);
-    expect(parsed).toHaveLength(1);
-  });
-
-  it('should handle empty secrets list', async () => {
-    fetchSpy.mockResolvedValue(new Response('[]', {status: 200}));
-
+  it.each([
+    {subcommand: 'set' as const, key: 'DB_URL', value: 'postgres://example'},
+    {subcommand: 'list' as const},
+    {subcommand: 'delete' as const, key: 'DB_URL'},
+  ])('fails closed for hosted secret management in OSS: $subcommand', async (options) => {
     const {runSecrets} = await import('./secrets.js');
-    const result = await runSecrets({subcommand: 'list'});
-    expect(result).toBe(0);
-    expect(stderrOutput).toContain('No secrets configured');
-  });
-
-  it('should delete a secret', async () => {
-    fetchSpy.mockResolvedValue(new Response('{}', {status: 200}));
+    const result = await runSecrets(options);
 
-    const {runSecrets} = await import('./secrets.js');
-    const result = await runSecrets({subcommand: 'delete', key: 'DB_URL'});
-    expect(result).toBe(0);
-    expect(stderrOutput).toContain('deleted');
-  });
-
-  it('should return 1 when key missing for delete', async () => {
-    const {runSecrets} = await import('./secrets.js');
-    const result = await runSecrets({subcommand: 'delete'});
     expect(result).toBe(1);
-    expect(stderrOutput).toContain('Missing key');
-  });
-
-  it('should return 1 when platform not configured', async () => {
-    delete process.env['PLATFORM_API_URL'];
-    delete process.env['PLATFORM_API_KEY'];
-
-    const {runSecrets} = await import('./secrets.js');
-    const result = await runSecrets({subcommand: 'list'});
-    expect(result).toBe(1);
-  });
-
-  it('should handle auth failure', async () => {
-    fetchSpy.mockResolvedValue(new Response('Unauthorized', {status: 401}));
-
-    const {runSecrets} = await import('./secrets.js');
-    const result = await runSecrets({subcommand: 'list'});
-    expect(result).toBe(1);
-    expect(stderrOutput).toContain('401');
-  });
-
-  it('should handle network error', async () => {
-    fetchSpy.mockRejectedValue(new Error('ECONNREFUSED'));
-
-    const {runSecrets} = await import('./secrets.js');
-    const result = await runSecrets({subcommand: 'set', key: 'K', value: 'V'});
-    expect(result).toBe(1);
-    expect(stderrOutput).toContain('ECONNREFUSED');
-  });
-
-  it('should handle set server error', async () => {
-    fetchSpy.mockResolvedValue(new Response('Internal Error', {status: 500}));
-
-    const {runSecrets} = await import('./secrets.js');
-    const result = await runSecrets({subcommand: 'set', key: 'K', value: 'V'});
-    expect(result).toBe(1);
-    expect(stderrOutput).toContain('500');
-  });
-
-  it('should show count for listed secrets', async () => {
-    fetchSpy.mockResolvedValue(new Response(JSON.stringify([
-      {key: 'A'},
-      {key: 'B'},
-      {key: 'C'},
-    ]), {status: 200}));
-
-    const {runSecrets} = await import('./secrets.js');
-    await runSecrets({subcommand: 'list'});
-    expect(stderrOutput).toContain('3 secrets');
-  });
-
-  it('should use singular for single secret', async () => {
-    fetchSpy.mockResolvedValue(new Response(JSON.stringify([{key: 'ONLY'}]), {status: 200}));
-
-    const {runSecrets} = await import('./secrets.js');
-    await runSecrets({subcommand: 'list'});
-    expect(stderrOutput).toContain('1 secret configured');
-    expect(stderrOutput).not.toContain('1 secrets');
+    expect(stderrOutput).toContain('Hosted secret management is not included in the OSS CLI');
+    expect(stdoutOutput).toBe('');
+    expect(fetchSpy).not.toHaveBeenCalled();
   });
 });