@tuanhv/op-cli 0.2.0 → 0.2.2

package/dist/commands/update.js

@@ -6,6 +6,9 @@ import { cliVersion } from "./version.js";
 const PACKAGE_SPEC = "@tuanhv/op-cli@latest";
 // Scoped package names must escape the slash for a registry path segment.
 const REGISTRY_URL = "https://registry.npmjs.org/@tuanhv%2Fop-cli/latest";
+// The bare name `npm ls -g` answers ownership with: exit 0 exactly when
+// npm's global tree holds the package.
+const PACKAGE_NAME = "@tuanhv/op-cli";
 // The notice is a convenience, never a delay: its lookup gets a budget far
 // below the API's own, and every failure path stays silent.
 const NOTICE_TIMEOUT_MS = 1_500;
@@ -20,9 +23,14 @@ function isHomebrewPath(path) {
         || path.includes("/usr/local/Homebrew");
 }
 // An install another manager owns would be overwritten by npm today and
-// reverted by that manager's next upgrade, so the exact command is printed
-// for the human to run instead. Detection is heuristic on purpose: npm's
-// global prefix and the resolved binary path name their manager plainly.
+// reverted by that manager's next upgrade, so the exact command is
+// printed for the human to run instead. Ownership is asked of npm
+// itself: `npm ls -g` exits 0 exactly when npm's global tree holds the
+// package, so a Homebrew-built Node with an npm -g install stays npm's.
+// A Homebrew *prefix* only names Node's installer; treating it as the
+// package's manager misroutes every npm -g install on such a Node.
+// Homebrew is claimed only when npm does not own the package and the
+// binary or prefix lands inside a Homebrew tree.
 async function detectManagingInstall(runtime) {
     const capture = runtime.captureExternal;
     if (capture === undefined) {
@@ -32,14 +40,16 @@ async function detectManagingInstall(runtime) {
     if (prefix.includes("volta")) {
         return { label: "Volta", command: `volta install ${PACKAGE_SPEC}` };
     }
-    if (isHomebrewPath(prefix)) {
-        return { label: "Homebrew", command: "brew upgrade op-cli" };
-    }
     const bin = (await capture("which", ["op-cli"]))?.trim() ?? "";
     if (bin.includes("volta")) {
         return { label: "Volta", command: `volta install ${PACKAGE_SPEC}` };
     }
-    if (isHomebrewPath(bin)) {
+    // The exit code is the answer; the listing text itself is not read.
+    const ownedByNpm = await capture("npm", ["ls", "-g", PACKAGE_NAME]);
+    if (ownedByNpm !== undefined) {
+        return undefined;
+    }
+    if (isHomebrewPath(bin) || isHomebrewPath(prefix)) {
         return { label: "Homebrew", command: "brew upgrade op-cli" };
     }
     return undefined;

package/dist/io/node-io.js

@@ -44,6 +44,12 @@ export function createNodeIo(input = process.stdin, output = process.stdout, err
         const decoder = new StringDecoder("utf8");
         let answer = "";
         let terminated = false;
+        // A preceding askLine closed its readline interface, which pauses the
+        // input; on a socket-backed stdin that stops the read handle, and the
+        // async iterator below never restarts it, so the event loop would drain
+        // and abort the awaited run(). resume() restarts the handle; the
+        // iterator's readable listener takes over before any chunk can flow.
+        input.resume();
         try {
             for await (const chunk of input) {
                 const text = typeof chunk === "string" ? chunk : decoder.write(chunk);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tuanhv/op-cli",
-  "version": "0.2.0",
+  "version": "0.2.2",
   "description": "A resolving command-line client for OpenProject",
   "license": "MIT",
   "author": "hoangvantuan",