@niroai/niro 0.9.0 → 0.10.0

package/src/commands/login.js

@@ -55,7 +55,7 @@ async function loginWithApiKey({ apiUrl }) {
     return;
   }
 
-  const sp = spinner.start("Authenticating...");
+  const sp = spinner.start("        Authenticating...");
   try {
     const data = await api.post(
       "/api/public/auth/api-key-login",
@@ -157,11 +157,16 @@ async function loginWithDeviceFlow({ apiUrl }) {
     throw new Error("Server did not return a complete device-code payload.");
   }
 
-  console.log(`\n  Your one-time code: ${color.bold(color.cyan(userCode))}`);
-  console.log(`  Open: ${color.cyan(verificationUri)}\n`);
+  // The verification URI is the web app's own address, chosen by whichever server we are
+  // talking to. Recording it is the only way the CLI can later link someone to their project
+  // without hardcoding a host that is wrong for UAT and wrong for every self-hosted install.
+  rememberAppUrl(verificationUri);
+
+  console.log(`\n    Your one-time code: ${color.bold(color.cyan(userCode))}`);
+  console.log(`    Open: ${color.cyan(verificationUri)}\n`);
   browser.open(verificationUri);
 
-  const sp = spinner.start("Waiting for browser approval...");
+  const sp = spinner.start("        Waiting for browser approval...");
   const deadline = Date.now() + expiresIn * 1000;
   try {
     while (Date.now() < deadline) {
@@ -199,6 +204,21 @@ async function loginWithDeviceFlow({ apiUrl }) {
   }
 }
 
+/**
+ * Store the web app's origin, taken from the device-flow verification URI.
+ *
+ * Best-effort by design: a malformed or missing URI must never fail a login that is otherwise
+ * fine. The only cost of not recording it is that `niro start` ends without a browser link.
+ */
+function rememberAppUrl(verificationUri) {
+  try {
+    const origin = new URL(verificationUri).origin;
+    require("../lib/config").save({ appUrl: origin });
+  } catch {
+    // Not a URL we can parse. Leave the previous value alone rather than storing a guess.
+  }
+}
+
 function isPending(err) {
   const body = err.body;
   if (!body) return false;
@@ -211,4 +231,4 @@ function sleep(ms) {
   return new Promise((resolve) => setTimeout(resolve, ms));
 }
 
-module.exports = { login };
+module.exports = { login, rememberAppUrl };

package/src/commands/projects.js

@@ -69,7 +69,7 @@ async function projects(opts = {}) {
     const repos = `${r.repo_count} repo${r.repo_count === 1 ? "" : "s"}`;
     const created = fmtDate(r.created_at);
     const meta = [repos, created].filter(Boolean).join(", ");
-    console.log(`  ${name}  ${color.dim(r.project_id)}  ${color.dim("— " + meta)}`);
+    console.log(`    ${name}  ${color.dim(r.project_id)}  ${color.dim("— " + meta)}`);
   }
 }
 

package/src/commands/release.js

@@ -141,11 +141,11 @@ async function release(pathArg, opts = {}) {
     } catch { /* can't inspect — don't block on it */ }
   }
   if (stillDiverged.length) {
-    console.log(warn(`\n  ⚠ This temporary project still covers in-flight work:`));
+    console.log(warn(`\n    ⚠ This temporary project still covers in-flight work:`));
     for (const d of stillDiverged) {
       console.log(warn(`    ${d.folder} is on \`${d.branch}\` (source: \`${d.sourceBranch}\`)`));
     }
-    console.log(warn("  Discarding deletes the WHOLE temporary project — those repos lose their coverage" +
+    console.log(warn("    Discarding deletes the WHOLE temporary project — those repos lose their coverage" +
         " until you run `niro new-temp-project` there again (local code is untouched)."));
     if (!opts.yes) {
       if (!process.stdout.isTTY || prompt.isNonInteractive()) {
@@ -233,25 +233,25 @@ async function release(pathArg, opts = {}) {
   }
 
   if (alreadyGone) {
-    console.log(`\n  ${ok("✓")} Temporary project ${color.bold(tempProjectId)} was already gone from the server` +
+    console.log(`\n    ${ok("✓")} Temporary project ${color.bold(tempProjectId)} was already gone from the server` +
         color.dim(" (deleted from the UI, or reaped after sitting idle)."));
     console.log(repinned.length
       ? `  ${ok("✓")} Cleaned up ${repinned.length} local registration(s) that still pointed at it. ` +
         "Niro routes back to the original project automatically."
       : `  ${ok("✓")} Nothing left to clean up locally.`);
   } else {
-    console.log(`\n  ${ok("✓")} Temporary project discarded. Niro routes back to the original project automatically.`);
+    console.log(`\n    ${ok("✓")} Temporary project discarded. Niro routes back to the original project automatically.`);
   }
   if (healed.length) {
-    console.log(color.dim("  Healed old-style pins:"));
+    console.log(color.dim("    Healed old-style pins:"));
     for (const h of healed) console.log(color.dim(`    ${h.folder} → ${h.to}`));
   }
   if (failed.length) {
-    console.log(warn(`\n  ⚠ ${failed.length} folder(s) could not be restored locally (the temporary project` +
+    console.log(warn(`\n    ⚠ ${failed.length} folder(s) could not be restored locally (the temporary project` +
         " is already gone server-side):"));
     for (const f of failed) console.log(color.dim(`    ${f.folder} — ${f.error}`));
   }
-  console.log(color.dim("  The original project picks up your merged commits via its normal remote polling."));
+  console.log(color.dim("    The original project picks up your merged commits via its normal remote polling."));
 }
 
 module.exports = { release };

package/src/commands/remove.js

@@ -71,9 +71,9 @@ async function remove(opts = {}) {
     const expected = alias ? `"${alias}" or ${projectId}` : projectId;
     console.error(`${color.red("✗")} Refusing to delete: --confirm-name must equal the project's name (${expected}).`);
     if (confirm == null) {
-      console.error(color.dim("   Pass --confirm-name <name> to confirm this destructive delete."));
+      console.error(color.dim("    Pass --confirm-name <name> to confirm this destructive delete."));
     } else {
-      console.error(color.dim(`   You passed --confirm-name "${confirm}", which does not match.`));
+      console.error(color.dim(`    You passed --confirm-name "${confirm}", which does not match.`));
     }
     process.exitCode = 1;
     return;
@@ -86,9 +86,9 @@ async function remove(opts = {}) {
 
 function printUsage() {
   console.error("niro remove — remove a project or a repo mapping. Exactly one mode required:");
-  console.error("  --project                  delete the whole project (DESTRUCTIVE)");
-  console.error("                             requires --confirm-name <project alias or id>");
-  console.error("  --unmap-repo <gitId>       remove one repo from the project (non-destructive)");
+  console.error("    --project                  delete the whole project (DESTRUCTIVE)");
+  console.error("                                requires --confirm-name <project alias or id>");
+  console.error("    --unmap-repo <gitId>       remove one repo from the project (non-destructive)");
   console.error("Target project: --target <alias|id>, or a .niro-project pin in this folder.");
 }