@neon-rs/cli 0.0.167 → 0.0.168

package/index.js

@@ -12052,12 +12052,12 @@ function assertIsNodeTarget(x) {
         throw new RangeError(`invalid Node target: ${x}`);
     }
 }
-function isTargetFamilyKey(x) {
+function isTargetPreset(x) {
     return (typeof x === 'string') && (x in family_namespaceObject);
 }
-function assertIsTargetFamilyKey(x) {
-    if (!isTargetFamilyKey(x)) {
-        throw new RangeError(`invalid target family name: ${x}`);
+function assertIsTargetPreset(x) {
+    if (!isTargetPreset(x)) {
+        throw new RangeError(`invalid target family preset: ${x}`);
     }
 }
 function lookupTargetFamily(key) {
@@ -12071,7 +12071,7 @@ function merge(maps) {
     return merged;
 }
 function expandTargetFamily(family) {
-    return isTargetFamilyKey(family)
+    return isTargetPreset(family)
         ? expandTargetFamily(lookupTargetFamily(family))
         : Array.isArray(family)
             ? merge(family.map(expandTargetFamily))
@@ -12175,6 +12175,19 @@ function assertIsTargetMap(json, path) {
         }
     }
 }
+function assertIsTargetFamily(json, path) {
+    if (typeof json === 'string') {
+        assertIsTargetPreset(json);
+        return;
+    }
+    if (Array.isArray(json)) {
+        for (const elt of json) {
+            assertIsTargetPreset(elt);
+        }
+        return;
+    }
+    assertIsTargetMap(json, path);
+}
 function assertIsBinaryV1(json) {
     assertHasProps(['binary'], json, "neon");
     const binary = json.binary;
@@ -12218,7 +12231,7 @@ function assertIsSourceCfg(json) {
     if (typeof json.org !== 'string') {
         throw new TypeError(`expected "neon.org" to be a string, found ${json.org}`);
     }
-    assertIsTargetMap(json.targets, "neon.targets");
+    assertIsTargetFamily(json.targets, "neon.targets");
 }
 function assertIsPreamble(json) {
     if (!json || typeof json !== 'object') {
@@ -12355,11 +12368,13 @@ function normalizeSourceCfg(json) {
 }
 class SourceManifest extends AbstractManifest {
     _sourceJSON;
+    _expandedTargets;
     constructor(json) {
         super(json);
         this._upgraded = normalizeSourceCfg(this._json);
         assertHasSourceCfg(this._json);
         this._sourceJSON = this._json;
+        this._expandedTargets = expandTargetFamily(this._sourceJSON.neon.targets);
     }
     static async load(dir) {
         return new SourceManifest(await readManifest(dir));
@@ -12369,18 +12384,21 @@ class SourceManifest extends AbstractManifest {
     }
     packageNames() {
         const cfg = this.cfg();
-        return Object.keys(cfg.targets).map(key => `${cfg.org}/${key}`);
+        return Object.keys(this._expandedTargets).map(key => `${cfg.org}/${key}`);
     }
     packageFor(target) {
         const cfg = this.cfg();
-        for (const key in cfg.targets) {
-            const value = cfg.targets[key];
+        for (const key in this._expandedTargets) {
+            const value = this._expandedTargets[key];
             if (value === target) {
                 return `${cfg.org}/${key}`;
             }
         }
         return undefined;
     }
+    rustTargetFor(node) {
+        return this._expandedTargets[node];
+    }
     manifestFor(target) {
         const targetInfo = getTargetDescriptor(target);
         const name = this.packageFor(target);
@@ -12416,11 +12434,10 @@ class SourceManifest extends AbstractManifest {
     }
     async addTargetPair(pair) {
         const { node, rust } = pair;
-        const targets = this.cfg().targets;
-        if (targets[node] === rust) {
+        if (this._expandedTargets[node] === rust) {
             return null;
         }
-        targets[node] = rust;
+        this._expandedTargets[node] = rust;
         await this.save();
         return pair;
     }
@@ -12434,22 +12451,43 @@ class SourceManifest extends AbstractManifest {
     async addRustTarget(target) {
         return await this.addTargetPair({ node: rust2Node(target), rust: target });
     }
-    async addTargets(family) {
-        const targets = this.cfg().targets;
-        let modified = [];
+    filterNewTargets(family) {
+        let newTargets = [];
         for (const [key, value] of Object.entries(family)) {
             const node = key;
             const rust = value;
-            if (targets[node] === rust) {
+            if (this._expandedTargets[node] === rust) {
                 continue;
             }
-            targets[node] = rust;
-            modified.push({ node, rust });
+            newTargets.push({ node, rust });
         }
-        if (modified.length) {
-            await this.save();
+        return newTargets;
+    }
+    async addTargets(family, opts = {}) {
+        let newTargets = this.filterNewTargets(family);
+        if (!newTargets.length) {
+            return [];
+        }
+        for (const { node, rust } of newTargets) {
+            if (opts.targetsSrc) {
+                opts.targetsSrc[node] = rust;
+            }
+            this._expandedTargets[node] = rust;
+        }
+        await this.save();
+        return newTargets;
+    }
+    async addTargetPreset(preset) {
+        const targetsSrc = this.cfg().targets;
+        if (typeof targetsSrc === 'string') {
+            this.cfg().targets = [targetsSrc, preset];
+            return this.addTargets(expandTargetFamily(preset));
+        }
+        if (Array.isArray(targetsSrc)) {
+            targetsSrc.push(preset);
+            return this.addTargets(expandTargetFamily(preset));
         }
-        return modified;
+        return this.addTargets(expandTargetFamily(preset), { targetsSrc });
     }
     async updateTargets(log, bundle) {
         const packages = this.packageNames();
@@ -12752,8 +12790,8 @@ class AddTarget {
             this.log(`adding Node target ${this._target}`);
             return optionArray(await sourceManifest.addNodeTarget(this._target));
         }
-        else if (isTargetFamilyKey(this._target)) {
-            return sourceManifest.addTargets(expandTargetFamily(this._target));
+        else if (isTargetPreset(this._target)) {
+            return sourceManifest.addTargetPreset(this._target);
         }
         else {
             throw new Error(`unrecognized target ${this._target}`);
@@ -12909,8 +12947,7 @@ class RustTarget {
         this.log(`reading package.json`);
         const sourceManifest = await SourceManifest.load();
         this.log(`manifest: ${sourceManifest.stringify()}`);
-        const targets = sourceManifest.cfg().targets;
-        const rust = targets[this._target];
+        const rust = sourceManifest.rustTargetFor(this._target);
         if (!rust) {
             throw new Error(`no Rust target found for ${this._target}`);
         }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@neon-rs/cli",
-  "version": "0.0.167",
+  "version": "0.0.168",
   "description": "Command-line build tool for Neon modules.",
   "type": "module",
   "exports": "./index.js",
@@ -27,12 +27,12 @@
   },
   "homepage": "https://github.com/dherman/neon-rs#readme",
   "optionalDependencies": {
-    "@cargo-messages/android-arm-eabi": "0.0.167",
-    "@cargo-messages/darwin-arm64": "0.0.167",
-    "@cargo-messages/darwin-x64": "0.0.167",
-    "@cargo-messages/linux-arm-gnueabihf": "0.0.167",
-    "@cargo-messages/linux-x64-gnu": "0.0.167",
-    "@cargo-messages/win32-arm64-msvc": "0.0.167",
-    "@cargo-messages/win32-x64-msvc": "0.0.167"
+    "@cargo-messages/android-arm-eabi": "0.0.168",
+    "@cargo-messages/darwin-arm64": "0.0.168",
+    "@cargo-messages/darwin-x64": "0.0.168",
+    "@cargo-messages/linux-arm-gnueabihf": "0.0.168",
+    "@cargo-messages/linux-x64-gnu": "0.0.168",
+    "@cargo-messages/win32-arm64-msvc": "0.0.168",
+    "@cargo-messages/win32-x64-msvc": "0.0.168"
   }
 }