@vicaura/agent 0.1.0 → 0.1.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@vicaura/agent",
-  "version": "0.1.0",
+  "version": "0.1.1",
   "description": "Install Vicaura product intelligence for Codex and Claude Code",
   "type": "module",
   "bin": {

package/src/cli.mjs

@@ -107,6 +107,7 @@ async function installSkill(client, options) {
 
 async function configureMcp(client, options) {
   const current = mcpConfig(client, options.serverName);
+  let loginHandledByAdd = false;
   if (current.exists && current.url === options.mcpUrl) {
     ok(`${label(client)} MCP connection is current`);
   } else {
@@ -117,7 +118,7 @@ async function configureMcp(client, options) {
       change(`${current.exists ? "Replace" : "Add"} ${label(client)} MCP connection: ${options.mcpUrl}`);
     } else {
       if (current.exists) removeMcp(client, options.serverName);
-      addMcp(client, options.mcpUrl, options.serverName);
+      loginHandledByAdd = addMcp(client, options.mcpUrl, options.serverName, options.login);
       ok(`${label(client)} MCP connection registered`);
     }
   }
@@ -125,7 +126,7 @@ async function configureMcp(client, options) {
   // `update` must remain non-interactive when the server registration is
   // already healthy. `install` performs the initial OAuth hand-off; users can
   // explicitly reconnect later with the command printed by `doctor`.
-  if (client === "codex" && options.login && options.command === "install" && !options.dryRun) {
+  if (client === "codex" && options.login && options.command === "install" && !options.dryRun && !loginHandledByAdd) {
     const login = runClient("codex", ["mcp", "login", options.serverName, "--oauth-client-registration", "DCR"], "inherit");
     if (login.status !== 0) throw new Error(`Codex OAuth did not complete. Run \`codex mcp login ${options.serverName} --oauth-client-registration DCR\` to retry.`);
     ok("Codex OAuth completed");
@@ -186,11 +187,18 @@ function mcpConfig(client, serverName) {
   return { exists: true, url: match?.[0]?.replace(/\/$/, "") ?? null };
 }
 
-function addMcp(client, url, serverName) {
+function addMcp(client, url, serverName, login) {
+  const immediateCodexLogin = client === "codex" && login && codexAddSupportsOAuthRegistration();
   const result = client === "codex"
-    ? runClient("codex", ["mcp", "add", serverName, "--url", url], "inherit")
+    ? runClient("codex", ["mcp", "add", serverName, ...(immediateCodexLogin ? ["--oauth-client-registration", "DCR"] : []), "--url", url], "inherit")
     : runClient("claude", ["mcp", "add", "--transport", "http", "--scope", "user", serverName, url], "inherit");
   if (result.status !== 0) throw new Error(`${label(client)} could not register the Vicaura MCP connection`);
+  return immediateCodexLogin;
+}
+
+function codexAddSupportsOAuthRegistration() {
+  const help = runClient("codex", ["mcp", "add", "--help"], "pipe");
+  return help.status === 0 && help.stdout.includes("--oauth-client-registration");
 }
 
 function removeMcp(client, serverName) {
@@ -270,7 +278,7 @@ Options:
   --client <auto|all|codex|claude|codex,claude>
   --mcp-url <url>   Vicaura MCP endpoint (default: ${DEFAULT_MCP_URL})
   --server-name <name>  MCP entry name (default: ${DEFAULT_SERVER_NAME})
-  --no-login        Configure without starting Codex OAuth
+  --no-login        Skip the explicit OAuth hand-off when the client permits it
   --force           Back up and replace a conflicting Vicaura entry
   --dry-run         Print intended changes only
   --help             Show this help`);