fix: use exact regex match for auth status check in postinstall

greynewell · claude · greynewell · commit 6ed203bc77e3 · 2026-03-09T16:54:02.000-04:00
out.includes("authenticated") matches "not authenticated", causing the
postinstall to always take the already-authenticated branch even after
logout. Use /^Status: authenticated$/m to match the exact output line.

Co-Authored-By: Grey Newell &lt;greyshipscode@gmail.com&gt;
Co-Authored-By: Claude Sonnet 4.6 &lt;noreply@anthropic.com&gt;
diff --git a/npm/install.js b/npm/install.js
@@ -191,7 +191,7 @@ async function main() {
   let isAuthenticated = false;
   try {
     const out = execFileSync(destPath, ["auth", "status"], { stdio: "pipe" }).toString();
-    isAuthenticated = out.includes("authenticated");
+    isAuthenticated = /^Status: authenticated$/m.test(out);
   } catch (err) {
     // binary failed or not runnable — fall through to install
   }

Original file line number	Diff line number	Diff line change
`@@ -191,7 +191,7 @@ async function main() {`
`191`	`191`	`let isAuthenticated = false;`
`192`	`192`	`try {`
`193`	`193`	`const out = execFileSync(destPath, ["auth", "status"], { stdio: "pipe" }).toString();`
`194`		`- isAuthenticated = out.includes("authenticated");`
	`194`	`+ isAuthenticated = /^Status: authenticated$/m.test(out);`
`195`	`195`	`} catch (err) {`
`196`	`196`	`// binary failed or not runnable — fall through to install`
`197`	`197`	`}`