@clipit-ai/cli 0.2.1 → 0.2.3

package/bin/clipit.mjs

@@ -9,7 +9,7 @@ import path from 'node:path';
 import process from 'node:process';
 import { fileURLToPath } from 'node:url';
 
-const VERSION = '0.2.1';
+const VERSION = '0.2.3';
 const DEFAULT_BASE_URL = 'https://clipit.dev';
 const DEFAULT_SCOPES = [
   'clippy_agent',
@@ -724,14 +724,33 @@ async function enforceRunMaxCredits(config, options, functionName) {
 async function login(config, options) {
   const { verifier, challenge } = pkcePair();
   const baseUrl = getBaseUrl(config, options);
-  const started = await apiFetch(config, options, 'POST', '/api/v1/cli-auth/start', {
-    codeChallenge: challenge,
-    codeChallengeMethod: 'S256',
-    requestedScopes: DEFAULT_SCOPES,
-    deviceName: os.hostname(),
-    cliVersion: VERSION,
-    platform: `${process.platform}/${process.arch}`,
-  }, { noAuth: true });
+  let started;
+  try {
+    started = await apiFetch(config, options, 'POST', '/api/v1/cli-auth/start', {
+      codeChallenge: challenge,
+      codeChallengeMethod: 'S256',
+      requestedScopes: DEFAULT_SCOPES,
+      deviceName: os.hostname(),
+      cliVersion: VERSION,
+      platform: `${process.platform}/${process.arch}`,
+    }, { noAuth: true });
+  } catch (error) {
+    // A login start should never need credentials — a 401/404 here means the
+    // server predates the CLI auth API entirely.
+    if (error.status === 401 || error.status === 404) {
+      throw Object.assign(
+        new Error([
+          `${baseUrl} does not support browser login yet (its API layer predates this CLI).`,
+          'Use an API key instead:',
+          `  1. Create one at ${baseUrl} -> Settings -> API Keys -> Connect an Agent`,
+          "  2. Store it: printf '%s' \"$CLIPPER_API_KEY\" | clipit auth set-key --stdin",
+          '  3. Verify: clipit videos list',
+        ].join('\n')),
+        { exitCode: EXIT.AUTH, status: error.status, requestId: error.requestId },
+      );
+    }
+    throw error;
+  }
 
   if (!options['no-browser']) {
     try {
@@ -1806,25 +1825,61 @@ async function jobs(config, options, action, args) {
   }
 }
 
-function appUrl(config, options, kind, id) {
+async function appUrl(config, options, kind, id) {
   const baseUrl = getBaseUrl(config, options);
   if (!kind || kind === 'home') return baseUrl;
   if (kind === 'settings') return `${baseUrl}/settings?tab=${encodeURIComponent(options.tab || 'api-keys')}`;
   if (kind === 'dashboard') return `${baseUrl}/dashboard`;
   if (kind === 'library') return `${baseUrl}/clips`;
-  if (kind === 'editor') return `${baseUrl}/editor`;
+  if (kind === 'editor') {
+    // The editor reads ?project=, ?video=, ?clip= (VideoEditorPage). Thread any
+    // provided context so `open editor --video <id>` lands on real content rather
+    // than the empty "select a video" state.
+    const params = new URLSearchParams();
+    const project = id || options.project || null;
+    if (project) params.set('project', project);
+    if (options.video || options.videoId) params.set('video', options.video || options.videoId);
+    if (options.clip || options.clipId) params.set('clip', options.clip || options.clipId);
+    const qs = params.toString();
+    return `${baseUrl}/editor${qs ? `?${qs}` : ''}`;
+  }
   if (kind === 'pricing') return `${baseUrl}/pricing`;
   if (kind === 'credits') return `${baseUrl}/settings/credits`;
-  if (kind === 'clip') return `${baseUrl}/clips/review${id ? `?clipId=${encodeURIComponent(id)}` : ''}`;
-  if (kind === 'video') return `${baseUrl}/clips${id ? `?videoId=${encodeURIComponent(id)}` : ''}`;
-  if (kind === 'project') return `${baseUrl}/editor/projects${id ? `?projectId=${encodeURIComponent(id)}` : ''}`;
-  if (kind === 'sequence') return `${baseUrl}/editor/projects${id ? `?sequenceId=${encodeURIComponent(id)}` : ''}`;
+  if (kind === 'clip') {
+    // A clip lives inside a video. The Clips Review page keys off ?video=<videoId>
+    // — without it the page renders "No video selected", so a bare clip link
+    // dead-ends. Resolve the clip's parent video so the link lands on the clip,
+    // selected. ?clip=<clipId> focuses it (the page reads `clip`, not `clipId`).
+    if (!id) return `${baseUrl}/clips/review`;
+    let videoId = options.video || options.videoId || null;
+    if (!videoId) {
+      try {
+        const clip = await apiFetch(config, options, 'GET', `/api/v1/clips/${encodeURIComponent(id)}`);
+        videoId = clip?.videoId || clip?.video?.id || null;
+      } catch {
+        // Network/auth/not-found — fall through to a clip-only link below.
+      }
+    }
+    const params = new URLSearchParams();
+    if (videoId) params.set('video', videoId);
+    params.set('clip', id);
+    return `${baseUrl}/clips/review?${params.toString()}`;
+  }
+  // A video link should open that video's clips in the Review page (which keys
+  // off ?video=), not the generic /clips library list which has no selection.
+  if (kind === 'video') return `${baseUrl}/clips/review${id ? `?video=${encodeURIComponent(id)}` : ''}`;
+  // The editor lives at /editor and reads ?project= (NOT /editor/projects, which
+  // redirects to /clips and drops the query — the same dead-end class as the
+  // clip/video link bug). There is no surface that consumes a raw sequenceId, so
+  // a sequence link just opens the editor.
+  if (kind === 'project') return `${baseUrl}/editor${id ? `?project=${encodeURIComponent(id)}` : ''}`;
+  if (kind === 'sequence') return `${baseUrl}/editor`;
   if (kind === 'route') return `${baseUrl}/${String(id || '').replace(/^\/+/, '')}`;
   return baseUrl;
 }
 
 async function openCommand(config, options, kind, id) {
-  const url = appUrl(config, options, kind, id);
+  const url = await appUrl(config, options, kind, id);
   if (options.print) {
     output(url, options);
     return;
@@ -1837,7 +1892,7 @@ async function links(config, options, kind, id) {
   const result = {
     kind: kind || 'home',
     id: id || null,
-    appUrl: appUrl(config, options, kind, id),
+    appUrl: await appUrl(config, options, kind, id),
   };
   if (kind === 'clip' && id && boolOption(options.download)) {
     try {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@clipit-ai/cli",
-  "version": "0.2.1",
+  "version": "0.2.3",
   "description": "ClipIt CLI for connecting shell-capable agents to ClipIt.",
   "type": "module",
   "bin": {