codeant-cli 0.5.3 → 0.5.4

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "codeant-cli",
-  "version": "0.5.3",
+  "version": "0.5.4",
   "description": "Code review CLI tool",
   "type": "module",
   "bin": {

package/src/commands/getDashboardUrl.js

@@ -0,0 +1,49 @@
+import React, { useState, useEffect } from 'react';
+import { Text, Box, useApp } from 'ink';
+import { getConfigValue } from '../utils/config.js';
+import { getBaseUrl, getDashboardUrl as resolveDashboardUrl } from '../utils/baseUrl.js';
+
+export default function GetDashboardUrl() {
+  const { exit } = useApp();
+  const [state, setState] = useState({ status: 'loading' });
+
+  const envValue = process.env.CODEANT_DASHBOARD_URL;
+  const configValue = getConfigValue('dashboardUrl');
+  const source = envValue ? 'env' : configValue ? 'config' : 'upstream (queried from API base URL)';
+
+  useEffect(() => {
+    (async () => {
+      try {
+        const url = await resolveDashboardUrl();
+        setState({ status: 'resolved', url });
+      } catch (err) {
+        setState({ status: 'error', message: err.message });
+      }
+      exit();
+    })();
+  }, []);
+
+  if (state.status === 'loading') {
+    return React.createElement(
+      Box,
+      { flexDirection: 'column', padding: 1 },
+      React.createElement(Text, null, 'Querying dashboard URL...')
+    );
+  }
+
+  if (state.status === 'error') {
+    return React.createElement(
+      Box,
+      { flexDirection: 'column', padding: 1 },
+      React.createElement(Text, { color: 'red' }, '✗ ', state.message),
+      React.createElement(Text, { color: 'gray' }, 'Base URL: ', getBaseUrl())
+    );
+  }
+
+  return React.createElement(
+    Box,
+    { flexDirection: 'column', padding: 1 },
+    React.createElement(Text, { bold: true }, 'Dashboard URL: ', state.url),
+    React.createElement(Text, { color: 'gray' }, 'Source: ', source)
+  );
+}

package/src/commands/login.js

@@ -30,7 +30,15 @@ export default function Login() {
     let timeoutId;
 
     (async () => {
-      const dashboardUrl = await getDashboardUrl();
+      let dashboardUrl;
+      try {
+        dashboardUrl = await getDashboardUrl();
+      } catch (err) {
+        setError(err.message);
+        setStatus('error');
+        setTimeout(() => exit(err), 100);
+        return;
+      }
       const loginUrl = `${dashboardUrl}?ideLoginToken=${token}`;
       setLoginUrl(loginUrl);
 

package/src/commands/removeDashboardUrl.js

@@ -0,0 +1,31 @@
+import React, { useEffect } from 'react';
+import { Text, Box, useApp } from 'ink';
+import { getConfigValue, setConfigValue } from '../utils/config.js';
+
+export default function RemoveDashboardUrl() {
+  const { exit } = useApp();
+
+  const hadValue = !!getConfigValue('dashboardUrl');
+
+  useEffect(() => {
+    if (hadValue) {
+      setConfigValue('dashboardUrl', null);
+    }
+    exit();
+  }, []);
+
+  if (!hadValue) {
+    return React.createElement(
+      Box,
+      { flexDirection: 'column', padding: 1 },
+      React.createElement(Text, { color: 'yellow' }, 'No dashboard URL override was set.')
+    );
+  }
+
+  return React.createElement(
+    Box,
+    { flexDirection: 'column', padding: 1 },
+    React.createElement(Text, { color: 'green' }, '✓ Dashboard URL override removed.'),
+    React.createElement(Text, { color: 'gray' }, 'It will be auto-detected from the API base URL on next login.')
+  );
+}

package/src/commands/setDashboardUrl.js

@@ -0,0 +1,37 @@
+import React, { useEffect } from 'react';
+import { Text, Box, useApp } from 'ink';
+import { setConfigValue, CONFIG_FILE } from '../utils/config.js';
+
+export default function SetDashboardUrl({ url }) {
+  const { exit } = useApp();
+
+  useEffect(() => {
+    if (!url) {
+      exit(new Error('URL is required'));
+      return;
+    }
+
+    try {
+      setConfigValue('dashboardUrl', url);
+      exit();
+    } catch (err) {
+      exit(err);
+    }
+  }, []);
+
+  if (!url) {
+    return React.createElement(
+      Box,
+      { flexDirection: 'column', padding: 1 },
+      React.createElement(Text, { color: 'red' }, '✗ Error: URL is required'),
+      React.createElement(Text, { color: 'gray' }, 'Usage: codeant set-dashboard-url <url>')
+    );
+  }
+
+  return React.createElement(
+    Box,
+    { flexDirection: 'column', padding: 1 },
+    React.createElement(Text, { color: 'green' }, '✓ Dashboard URL set to: ', url),
+    React.createElement(Text, { color: 'gray' }, 'Saved to: ', CONFIG_FILE)
+  );
+}

package/src/index.js

@@ -7,6 +7,9 @@ import { createRequire } from 'module';
 import Secrets from './commands/secrets.js';
 import SetBaseUrl from './commands/setBaseUrl.js';
 import GetBaseUrl from './commands/getBaseUrl.js';
+import SetDashboardUrl from './commands/setDashboardUrl.js';
+import GetDashboardUrl from './commands/getDashboardUrl.js';
+import RemoveDashboardUrl from './commands/removeDashboardUrl.js';
 import SetApiKey from './commands/setApiKey.js';
 import GetApiKey from './commands/getApiKey.js';
 import Login from './commands/login.js';
@@ -188,6 +191,27 @@ program
       render(React.createElement(GetBaseUrl));
     });
 
+  program
+    .command('set-dashboard-url <url>')
+    .description('Set the web app dashboard URL used for login')
+    .action((url) => {
+      render(React.createElement(SetDashboardUrl, { url }));
+    });
+
+  program
+    .command('get-dashboard-url')
+    .description('Show the current dashboard URL override')
+    .action(() => {
+      render(React.createElement(GetDashboardUrl));
+    });
+
+  program
+    .command('remove-dashboard-url')
+    .description('Remove the dashboard URL override')
+    .action(() => {
+      render(React.createElement(RemoveDashboardUrl));
+    });
+
   program
     .command('set-codeant-api-key <key>')
     .description('Set the CodeAnt API key')

package/src/utils/baseUrl.js

@@ -2,6 +2,11 @@ import { getConfigValue } from './config.js';
 
 //
 const NEW_BASE_URL = 'https://service.codeant.ai';
+const DEFAULT_BASE_URLS = new Set([
+  'https://service.codeant.ai',
+  'https://api.codeant.ai',
+  'https://dev-api.codeant.ai',
+]);
 
 const getBaseUrl = () => {
   const url = process.env.CODEANT_API_URL || getConfigValue('baseUrl') || NEW_BASE_URL;
@@ -9,15 +14,34 @@ const getBaseUrl = () => {
   return url;
 };
 
+const isDefaultBaseUrl = () => {
+  const configuredUrl = process.env.CODEANT_API_URL || getConfigValue('baseUrl');
+  return !configuredUrl || DEFAULT_BASE_URLS.has(configuredUrl);
+};
+
 const getDashboardUrl = async () => {
+  const override = process.env.CODEANT_DASHBOARD_URL || getConfigValue('dashboardUrl');
+  if (override) return override;
+
+  const usingDefaultBaseUrl = isDefaultBaseUrl();
+
   try {
     const response = await fetch(`${getBaseUrl()}/extension/get/dashboard`);
-    // console.log('Fetching dashboard URL from:', `${getBaseUrl()}/extension/get/dashboard`);
     const data = await response.json();
-    return data.dashboard_url || 'https://app.codeant.ai';
+    // On a custom base URL, never silently trust an auto-detected app.codeant.ai —
+    // that's the SaaS dashboard, and a self-hosted instance's OAuth apps won't
+    // recognize it as a valid redirect target. Require an explicit override instead.
+    if (data.dashboard_url && (usingDefaultBaseUrl || data.dashboard_url !== 'https://app.codeant.ai')) {
+      return data.dashboard_url;
+    }
   } catch {
-    return 'https://app.codeant.ai';
+    // fall through to the error below
   }
+
+  throw new Error(
+    `Could not determine the dashboard URL for base URL "${getBaseUrl()}". ` +
+    `Set it explicitly with: codeant set-dashboard-url <your web app URL>`
+  );
 };
 
 export { getBaseUrl, getDashboardUrl };