@memberjunction/cli 5.43.0 → 5.44.0

package/dist/hooks/postrun.d.ts

@@ -1,13 +1,23 @@
 /**
  * Cleanup hook that runs after any command completes.
  *
- * Commands under `mj app *` open an mssql connection pool through
- * `ensureProviderInitialized` in utils/open-app-context.ts. The pool keeps
- * sockets alive, which pins the Node event loop open and makes the CLI
- * appear to hang after the command finishes printing its output.
+ * Only `mj app *` commands open an mssql connection pool (through
+ * `ensureProviderInitialized` in utils/open-app-context.ts). The pool keeps
+ * sockets alive, which pins the Node event loop open and makes the CLI appear
+ * to hang after the command finishes printing its output — so for those
+ * commands we close it here.
  *
- * `closeConnectionPool` is idempotent — if the pool was never opened (most
- * non-`app` commands), it no-ops.
+ * The import of `open-app-context` is DEFERRED (dynamic) and gated on the
+ * command id: that module statically pulls in the entire DB-provider stack
+ * (@memberjunction/sqlserver-dataprovider, @memberjunction/metadata-sync,
+ * @memberjunction/core), which costs ~2.7s warm and far more on a cold CI
+ * runner. Loading it eagerly here charged that cost to EVERY command —
+ * including light ones like `install:claude` — which is what made the
+ * claude-pack subprocess tests exceed their spawn timeout. Non-`app` commands
+ * never open a pool, so they have nothing to close and skip the import.
+ *
+ * Dynamic import is justified per CLAUDE.md rule #8 (#3 — genuine, measured
+ * startup-cost deferral of a single heavy module).
  */
 import { Hook } from '@oclif/core';
 declare const hook: Hook<'postrun'>;

package/dist/hooks/postrun.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"postrun.d.ts","sourceRoot":"","sources":["../../src/hooks/postrun.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;GAUG;AACH,OAAO,EAAE,IAAI,EAAE,MAAM,aAAa,CAAC;AAGnC,QAAA,MAAM,IAAI,EAAE,IAAI,CAAC,SAAS,CAMzB,CAAC;AAEF,eAAe,IAAI,CAAC"}
+{"version":3,"file":"postrun.d.ts","sourceRoot":"","sources":["../../src/hooks/postrun.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;;GAoBG;AACH,OAAO,EAAE,IAAI,EAAE,MAAM,aAAa,CAAC;AAEnC,QAAA,MAAM,IAAI,EAAE,IAAI,CAAC,SAAS,CAYzB,CAAC;AAEF,eAAe,IAAI,CAAC"}

package/dist/hooks/postrun.js

@@ -1,6 +1,11 @@
-import { closeConnectionPool } from '../utils/open-app-context.js';
-const hook = async function () {
+const hook = async function (options) {
+    const commandId = options.Command?.id ?? '';
+    // Pools are opened exclusively by `app:*` commands; nothing else can leave one open.
+    if (!commandId.startsWith('app:')) {
+        return;
+    }
     try {
+        const { closeConnectionPool } = await import('../utils/open-app-context.js');
         await closeConnectionPool();
     }
     catch {

package/dist/hooks/postrun.js.map

@@ -1 +1 @@
-{"version":3,"file":"postrun.js","sourceRoot":"","sources":["../../src/hooks/postrun.ts"],"names":[],"mappings":"AAYA,OAAO,EAAE,mBAAmB,EAAE,MAAM,8BAA8B,CAAC;AAEnE,MAAM,IAAI,GAAoB,KAAK;IACjC,IAAI,CAAC;QACH,MAAM,mBAAmB,EAAE,CAAC;IAC9B,CAAC;IAAC,MAAM,CAAC;QACP,8DAA8D;IAChE,CAAC;AACH,CAAC,CAAC;AAEF,eAAe,IAAI,CAAC"}
+{"version":3,"file":"postrun.js","sourceRoot":"","sources":["../../src/hooks/postrun.ts"],"names":[],"mappings":"AAuBA,MAAM,IAAI,GAAoB,KAAK,WAAW,OAAO;IACnD,MAAM,SAAS,GAAG,OAAO,CAAC,OAAO,EAAE,EAAE,IAAI,EAAE,CAAC;IAC5C,qFAAqF;IACrF,IAAI,CAAC,SAAS,CAAC,UAAU,CAAC,MAAM,CAAC,EAAE,CAAC;QAClC,OAAO;IACT,CAAC;IACD,IAAI,CAAC;QACH,MAAM,EAAE,mBAAmB,EAAE,GAAG,MAAM,MAAM,CAAC,8BAA8B,CAAC,CAAC;QAC7E,MAAM,mBAAmB,EAAE,CAAC;IAC9B,CAAC;IAAC,MAAM,CAAC;QACP,8DAA8D;IAChE,CAAC;AACH,CAAC,CAAC;AAEF,eAAe,IAAI,CAAC"}

package/oclif.manifest.json

@@ -377,67 +377,6 @@
         "upgrade.js"
       ]
     },
-    "artifacts:reclassify": {
-      "aliases": [],
-      "args": {},
-      "description": "Upgrade-recovery utility for v5.35 artifact unification. Re-evaluates rows whose TypeID is the deleted JSON fallback but whose actual content does not parse as JSON. Normally a no-op: the v5.35 backfill migration handles this automatically. This command is only relevant when the migration ran BEFORE the artifact-type metadata (Generic Text / Generic Binary) was pushed via `mj sync push`, in which case the migration prints a notice telling you to run this. Reports \"no candidates\" once everything is reconciled. Dry-run by default; --apply writes through Record Changes.",
-      "examples": [
-        "<%= config.bin %> <%= command.id %>",
-        "<%= config.bin %> <%= command.id %> --apply --limit 50"
-      ],
-      "flags": {
-        "conversation-id": {
-          "description": "Scope to artifacts attached to the given conversation.",
-          "name": "conversation-id",
-          "hasDynamicHelp": false,
-          "multiple": false,
-          "type": "option"
-        },
-        "since": {
-          "description": "Scope to artifacts created on/after this ISO date.",
-          "name": "since",
-          "hasDynamicHelp": false,
-          "multiple": false,
-          "type": "option"
-        },
-        "limit": {
-          "description": "Maximum number of rows to inspect.",
-          "name": "limit",
-          "default": 100,
-          "hasDynamicHelp": false,
-          "multiple": false,
-          "type": "option"
-        },
-        "apply": {
-          "description": "Write changes. Without this flag, this command only reports what it would do.",
-          "name": "apply",
-          "allowNo": false,
-          "type": "boolean"
-        },
-        "verbose": {
-          "char": "v",
-          "description": "Print every candidate row, not just the summary.",
-          "name": "verbose",
-          "allowNo": false,
-          "type": "boolean"
-        }
-      },
-      "hasDynamicHelp": false,
-      "hiddenAliases": [],
-      "id": "artifacts:reclassify",
-      "pluginAlias": "@memberjunction/cli",
-      "pluginName": "@memberjunction/cli",
-      "pluginType": "core",
-      "strict": true,
-      "enableJsonFlag": false,
-      "isESM": true,
-      "relativePath": [
-        "dist",
-        "commands",
-        "artifacts",
-        "reclassify.js"
-      ]
-    },
     "baseline:build": {
       "aliases": [],
       "args": {},
@@ -966,6 +905,67 @@
         "index.js"
       ]
     },
+    "artifacts:reclassify": {
+      "aliases": [],
+      "args": {},
+      "description": "Upgrade-recovery utility for v5.35 artifact unification. Re-evaluates rows whose TypeID is the deleted JSON fallback but whose actual content does not parse as JSON. Normally a no-op: the v5.35 backfill migration handles this automatically. This command is only relevant when the migration ran BEFORE the artifact-type metadata (Generic Text / Generic Binary) was pushed via `mj sync push`, in which case the migration prints a notice telling you to run this. Reports \"no candidates\" once everything is reconciled. Dry-run by default; --apply writes through Record Changes.",
+      "examples": [
+        "<%= config.bin %> <%= command.id %>",
+        "<%= config.bin %> <%= command.id %> --apply --limit 50"
+      ],
+      "flags": {
+        "conversation-id": {
+          "description": "Scope to artifacts attached to the given conversation.",
+          "name": "conversation-id",
+          "hasDynamicHelp": false,
+          "multiple": false,
+          "type": "option"
+        },
+        "since": {
+          "description": "Scope to artifacts created on/after this ISO date.",
+          "name": "since",
+          "hasDynamicHelp": false,
+          "multiple": false,
+          "type": "option"
+        },
+        "limit": {
+          "description": "Maximum number of rows to inspect.",
+          "name": "limit",
+          "default": 100,
+          "hasDynamicHelp": false,
+          "multiple": false,
+          "type": "option"
+        },
+        "apply": {
+          "description": "Write changes. Without this flag, this command only reports what it would do.",
+          "name": "apply",
+          "allowNo": false,
+          "type": "boolean"
+        },
+        "verbose": {
+          "char": "v",
+          "description": "Print every candidate row, not just the summary.",
+          "name": "verbose",
+          "allowNo": false,
+          "type": "boolean"
+        }
+      },
+      "hasDynamicHelp": false,
+      "hiddenAliases": [],
+      "id": "artifacts:reclassify",
+      "pluginAlias": "@memberjunction/cli",
+      "pluginName": "@memberjunction/cli",
+      "pluginType": "core",
+      "strict": true,
+      "enableJsonFlag": false,
+      "isESM": true,
+      "relativePath": [
+        "dist",
+        "commands",
+        "artifacts",
+        "reclassify.js"
+      ]
+    },
     "codegen:5-0-fix-entity-names": {
       "aliases": [],
       "args": {},
@@ -5453,5 +5453,5 @@
       ]
     }
   },
-  "version": "5.43.0"
+  "version": "5.44.0"
 }

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@memberjunction/cli",
   "type": "module",
-  "version": "5.43.0",
+  "version": "5.44.0",
   "description": "MemberJunction command line tools",
   "keywords": [
     "oclif"
@@ -55,25 +55,25 @@
   },
   "dependencies": {
     "@inquirer/prompts": "^8.2.0",
-    "@memberjunction/ai-cli": "5.43.0",
-    "@memberjunction/cli-core": "5.43.0",
-    "@memberjunction/codegen-lib": "5.43.0",
-    "@memberjunction/config": "5.43.0",
-    "@memberjunction/core": "5.43.0",
-    "@memberjunction/generic-database-provider": "5.43.0",
-    "@memberjunction/installer": "5.43.0",
-    "@memberjunction/db-auto-doc": "5.43.0",
-    "@memberjunction/metadata-sync": "5.43.0",
-    "@memberjunction/open-app-engine": "5.43.0",
-    "@memberjunction/query-gen": "5.43.0",
-    "@memberjunction/server-bootstrap-lite": "5.43.0",
+    "@memberjunction/ai-cli": "5.44.0",
+    "@memberjunction/cli-core": "5.44.0",
+    "@memberjunction/codegen-lib": "5.44.0",
+    "@memberjunction/config": "5.44.0",
+    "@memberjunction/core": "5.44.0",
+    "@memberjunction/generic-database-provider": "5.44.0",
+    "@memberjunction/installer": "5.44.0",
+    "@memberjunction/db-auto-doc": "5.44.0",
+    "@memberjunction/metadata-sync": "5.44.0",
+    "@memberjunction/open-app-engine": "5.44.0",
+    "@memberjunction/query-gen": "5.44.0",
+    "@memberjunction/server-bootstrap-lite": "5.44.0",
     "@memberjunction/skyway-core": "^0.6.2",
     "@memberjunction/skyway-postgres": "^0.6.2",
     "@memberjunction/skyway-sqlserver": "^0.6.2",
-    "@memberjunction/sql-converter": "5.43.0",
-    "@memberjunction/sqlglot-ts": "5.43.0",
-    "@memberjunction/sqlserver-dataprovider": "5.43.0",
-    "@memberjunction/testing-cli": "5.43.0",
+    "@memberjunction/sql-converter": "5.44.0",
+    "@memberjunction/sqlglot-ts": "5.44.0",
+    "@memberjunction/sqlserver-dataprovider": "5.44.0",
+    "@memberjunction/testing-cli": "5.44.0",
     "@oclif/core": "^3.27.0",
     "@oclif/plugin-help": "^6.2.37",
     "@oclif/plugin-version": "^2.2.36",