@maka/meteor-sdk 0.2.2 → 0.2.3

package/README.md

@@ -42,6 +42,15 @@ the target app's own `.meteor/release` (e.g. `"METEOR@3.5.1"`) -- it pins
 which Meteor release's package versions and build behavior this process
 uses.
 
+Passing a specific release requires that release to be present in the
+local package catalog (the Meteor warehouse) -- on a machine that has
+never downloaded it, `init` rejects with a `MeteorSdkError` naming the
+missing release. **Embedders that only need `connectDdp()` (or other
+non-build APIs) should call `init({})` with no release**: that uses
+checkout semantics (the tool copy this package itself ships), needs no
+warehouse or catalog access, and works on a machine whose only Meteor
+footprint is this npm package.
+
 ### `loadProject(appDir)`
 
 Resolves an app directory into a `Project`, ready to bundle or run.

package/isopackets/combined/npm/node_modules/meteor/babel-compiler/node_modules/.package-lock.json

@@ -1,5 +1,5 @@
 {
-  "name": "package-new-10g0ajfucol",
+  "name": "package-new-6xobneyfbqr",
   "lockfileVersion": 3,
   "requires": true,
   "packages": {

package/isopackets/combined/npm/node_modules/meteor/boilerplate-generator/node_modules/.package-lock.json

@@ -1,5 +1,5 @@
 {
-  "name": "package-new-1slwkyce3sz",
+  "name": "package-new-r3e8f88flkl",
   "lockfileVersion": 3,
   "requires": true,
   "packages": {

package/isopackets/combined/npm/node_modules/meteor/constraint-solver/node_modules/.package-lock.json

@@ -1,5 +1,5 @@
 {
-  "name": "package-new-fmtb13edl7",
+  "name": "package-new-4pbying366l",
   "lockfileVersion": 3,
   "requires": true,
   "packages": {

package/isopackets/combined/npm/node_modules/meteor/ddp-client/node_modules/.package-lock.json

@@ -1,5 +1,5 @@
 {
-  "name": "package-new-17ww1zgjylj",
+  "name": "package-new-w88s7v8a2tb",
   "lockfileVersion": 3,
   "requires": true,
   "packages": {

package/isopackets/combined/npm/node_modules/meteor/ecmascript/node_modules/.package-lock.json

@@ -1,5 +1,5 @@
 {
-  "name": "package-new-g6t4wfio7ve",
+  "name": "package-new-f69i13ow6p7",
   "lockfileVersion": 3,
   "requires": true,
   "packages": {

package/isopackets/combined/npm/node_modules/meteor/ecmascript-runtime/node_modules/.package-lock.json

@@ -1,5 +1,5 @@
 {
-  "name": "package-new-aqph2241bof",
+  "name": "package-new-p1w5mkk0978",
   "lockfileVersion": 3,
   "requires": true,
   "packages": {

package/isopackets/combined/npm/node_modules/meteor/ecmascript-runtime-client/node_modules/.package-lock.json

@@ -1,5 +1,5 @@
 {
-  "name": "package-new-xcpgqd2fln9",
+  "name": "package-new-1ib48dwuck5",
   "lockfileVersion": 3,
   "requires": true,
   "packages": {

package/isopackets/combined/npm/node_modules/meteor/ecmascript-runtime-server/node_modules/.package-lock.json

@@ -1,5 +1,5 @@
 {
-  "name": "package-new-1cqofk7h0vn",
+  "name": "package-new-1157kjfj2gw",
   "lockfileVersion": 3,
   "requires": true,
   "packages": {

package/isopackets/combined/npm/node_modules/meteor/fetch/node_modules/.package-lock.json

@@ -1,5 +1,5 @@
 {
-  "name": "package-new-1g6gs3yfiwp",
+  "name": "package-new-1wg8ssdz0fmf",
   "lockfileVersion": 3,
   "requires": true,
   "packages": {

package/isopackets/combined/npm/node_modules/meteor/inter-process-messaging/node_modules/.package-lock.json

@@ -1,5 +1,5 @@
 {
-  "name": "package-new-nt8samf9uwk",
+  "name": "package-new-1vnwfx4b68v",
   "lockfileVersion": 3,
   "requires": true,
   "packages": {

package/isopackets/combined/npm/node_modules/meteor/logging/node_modules/.package-lock.json

@@ -1,5 +1,5 @@
 {
-  "name": "package-new-ksjvqwow5eq",
+  "name": "package-new-kid7k7quu7",
   "lockfileVersion": 3,
   "requires": true,
   "packages": {

package/isopackets/combined/npm/node_modules/meteor/logic-solver/node_modules/.package-lock.json

@@ -1,5 +1,5 @@
 {
-  "name": "package-new-usn32d5f7m",
+  "name": "package-new-1a6dsn3qfc2",
   "lockfileVersion": 3,
   "requires": true,
   "packages": {

package/isopackets/combined/npm/node_modules/meteor/meteor/node_modules/.package-lock.json

@@ -1,5 +1,5 @@
 {
-  "name": "package-new-1wbkujoz6iei",
+  "name": "package-new-v6v3rjtfql",
   "lockfileVersion": 3,
   "requires": true,
   "packages": {

package/isopackets/combined/npm/node_modules/meteor/modules/node_modules/.package-lock.json

@@ -1,5 +1,5 @@
 {
-  "name": "package-new-s9fhxhunuf",
+  "name": "package-new-1lii7blq24r",
   "lockfileVersion": 3,
   "requires": true,
   "packages": {

package/isopackets/combined/npm/node_modules/meteor/modules-runtime/node_modules/.package-lock.json

@@ -1,5 +1,5 @@
 {
-  "name": "package-new-1v67hnpaktth",
+  "name": "package-new-1wg5bygb6ij",
   "lockfileVersion": 3,
   "requires": true,
   "packages": {

package/isopackets/combined/npm/node_modules/meteor/npm-mongo/node_modules/.package-lock.json

@@ -1,5 +1,5 @@
 {
-  "name": "package-new-3xnmqbs5th5",
+  "name": "package-new-uir4qgmp1sp",
   "lockfileVersion": 3,
   "requires": true,
   "packages": {

package/isopackets/combined/npm/node_modules/meteor/package-version-parser/node_modules/.package-lock.json

@@ -1,5 +1,5 @@
 {
-  "name": "package-new-xjm9993m8hk",
+  "name": "package-new-461z05pva2w",
   "lockfileVersion": 3,
   "requires": true,
   "packages": {

package/isopackets/combined/npm/node_modules/meteor/promise/node_modules/.package-lock.json

@@ -1,5 +1,5 @@
 {
-  "name": "package-new-d5x03oy37qo",
+  "name": "package-new-23rno2x3x0o",
   "lockfileVersion": 3,
   "requires": true,
   "packages": {

package/isopackets/combined/npm/node_modules/meteor/react-fast-refresh/node_modules/.package-lock.json

@@ -1,5 +1,5 @@
 {
-  "name": "package-new-15icl6ez9mo",
+  "name": "package-new-1w81jifsunlg",
   "lockfileVersion": 3,
   "requires": true,
   "packages": {

package/isopackets/combined/npm/node_modules/meteor/socket-stream-client/node_modules/.package-lock.json

@@ -1,5 +1,5 @@
 {
-  "name": "package-new-1wtb47fkaa7",
+  "name": "package-new-1b9umwwq6uc",
   "lockfileVersion": 3,
   "requires": true,
   "packages": {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@maka/meteor-sdk",
-  "version": "0.2.2",
+  "version": "0.2.3",
   "description": "Programmatic Node API over the Meteor build tool (bundle/run/create/etc.) -- for driving Meteor's build system from plain Node code instead of shelling out to a meteor CLI (there isn't one anymore; see README.md). \"dependencies\" is generated by scripts/write-tool-package-json.js from scripts/dev-bundle-tool-package.js at the monorepo root -- do not hand-edit that field, edit that file and re-run the script instead.",
   "main": "src/index.js",
   "exports": {
@@ -45,9 +45,7 @@
     "@types/semver": "7.5.8",
     "semver": "7.6.3",
     "request": "2.88.2",
-    "uuid": "8.3.2",
     "graceful-fs": "4.2.11",
-    "tar": "6.1.11",
     "tar-fs": "2.1.1",
     "source-map": "0.7.4",
     "chalk": "4.1.2",
@@ -56,8 +54,7 @@
     "is-reachable": "3.1.0",
     "wordwrap": "1.0.0",
     "moment": "2.30.1",
-    "rimraf": "2.6.2",
-    "glob": "11.0.1",
+    "glob": "11.1.0",
     "ignore": "7.0.5",
     "commonmark": "0.15.0",
     "escope": "4.0.0",

package/src/release.js

@@ -36,7 +36,43 @@ async function init(options) {
   // release cached (or be reachable); see release.load's own doc comment
   // in meteor-tool for the exact fetch/cache behavior.
   const releaseName = options.release == null ? null : options.release;
-  const releaseObject = await release.load(releaseName);
+  let releaseObject;
+  try {
+    releaseObject = await release.load(releaseName);
+  } catch (e) {
+    // release.NoSuchReleaseError is an EMPTY constructor -- not even an
+    // Error, no .message -- so embedders logging err.message printed
+    // "undefined" and the real cause (release not in the local package
+    // catalog, e.g. a machine with no populated Meteor warehouse) was
+    // invisible. Cost a real debugging evening in the field; wrap it
+    // (and any other message-less throw from release.load) in a proper
+    // MeteorSdkError that names the release and where the catalog lives.
+    let warehouse = "~/.meteor";
+    try {
+      warehouse = require("../tools/packaging/tropohouse.js").default.root;
+    } catch (_) {
+      // Best effort -- the default hint above is close enough.
+    }
+    if (e instanceof release.NoSuchReleaseError) {
+      throw new MeteorSdkError(
+        "init(): release " + JSON.stringify(releaseName) + " is not in " +
+          "the local package catalog (warehouse: " + warehouse + "). " +
+          "On a machine that has never downloaded that Meteor release " +
+          "this is expected -- either install/download the release " +
+          "first, or, for embedders that only need connectDdp() and " +
+          "other non-build APIs, call init({}) with no release (checkout " +
+          "semantics, no catalog needed)."
+      );
+    }
+    if (e instanceof Error) {
+      throw e;
+    }
+    throw new MeteorSdkError(
+      "init(): release.load(" + JSON.stringify(releaseName) + ") failed " +
+        "with a message-less throw (" + Object.prototype.toString.call(e) +
+        "); warehouse: " + warehouse
+    );
+  }
   // forced is always true: init({release}) is the embedder explicitly
   // choosing the release for this process -- the moral equivalent of the
   // CLI's --release flag, which is exactly what release.forced means.

package/tools/fs/dev-bundle-links.js

@@ -22,7 +22,7 @@ exports.makeLink = function (target, linkPath) {
     fs.renameSync(tempPath, linkPath);
   } catch (e) {
     // If renaming fails, try unlinking first.
-    require("rimraf").sync(linkPath);
+    require("fs").rmSync(linkPath, { recursive: true, force: true });
     fs.renameSync(tempPath, linkPath);
   }
 };

package/tools/fs/files.ts

@@ -30,7 +30,6 @@ import { realpathSync } from './fsFixPath';
 
 const _ = require('underscore');
 
-const rimraf = require('rimraf');
 const sourcemap = require('source-map');
 const sourceMapRetrieverStack = require('../tool-env/source-map-retriever-stack.js');
 
@@ -322,7 +321,16 @@ export function getCurrentNodeBinDir() {
 // Return the top-level directory for this meteor install or checkout
 export function getCurrentToolsDir() {
   if (!process.env.SANDBOX && process.env.METEOR_WAREHOUSE_DIR) {
-    return pathDirname(realpathSync(pathJoin(process.env.METEOR_WAREHOUSE_DIR, 'meteor')));
+    // Only follow the warehouse's `meteor` link when it actually exists:
+    // an empty/fresh METEOR_WAREHOUSE_DIR (observed: SDK embedders
+    // pointing the env var at a scratch dir) made realpathSync throw
+    // ENOENT uncaught through getToolBundleRoot() and everything above
+    // it. Falling through to the __dirname-based answer keeps the
+    // running tool copy resolvable regardless of warehouse state.
+    const warehouseMeteor = pathJoin(process.env.METEOR_WAREHOUSE_DIR, 'meteor');
+    if (exists(warehouseMeteor)) {
+      return pathDirname(realpathSync(warehouseMeteor));
+    }
   }
 
   return pathDirname(pathDirname(convertToStandardPath(__dirname)));
@@ -457,16 +465,21 @@ export function realpathOrNull(path: string) {
 
 export function rm_recursive_async(path: string) {
   return new Promise<void>((resolve, reject) => {
-    rimraf(convertToOSPath(path), (err: Error) => err
-      ? reject(err)
-      : resolve());
+    // fs.rm with force+maxRetries replaces the old rimraf dependency
+    // (Node's built-in grew the same EPERM/EBUSY retry behavior it
+    // existed for).
+    fs.rm(
+      convertToOSPath(path),
+      { recursive: true, force: true, maxRetries: 3 },
+      (err) => err ? reject(err) : resolve()
+    );
   });
 }
 
 // Like rm -r.
 export const rm_recursive = Profile("files.rm_recursive", async (path: string) => {
   try {
-    rimraf.sync(convertToOSPath(path));
+    fs.rmSync(convertToOSPath(path), { recursive: true, force: true });
   } catch (e: any) {
     if ((e.code === "ENOTEMPTY" ||
          e.code === "EPERM")) {
@@ -1734,7 +1747,7 @@ export const rename = isWindowsLikeFilesystem() ? function (from: string, to: st
         // Despite previous failures, the top-level destination directory
         // may have been successfully created, so we must remove it to
         // avoid moving the source file *into* the destination directory.
-        rimraf.sync(osTo);
+        fs.rmSync(osTo, { recursive: true, force: true });
         wrappedRename(from, to);
         resolve();
       } catch (err: any) {