bugfix(@neon-rs/manifest): assertIsPlatformFamily now recognizes Plat…

…formName elements
neon-bindings · Aug 25, 2024 · ab3b63d · ab3b63d
1 parent 4fb181e
commit ab3b63d
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 4 deletions.
diff --git a/src/manifest/package.json b/src/manifest/package.json
@@ -1,7 +1,7 @@
 {
   "name": "@neon-rs/manifest",
   "private": false,
-  "version": "0.1.2",
+  "version": "0.1.3",
   "description": "Library for working with Neon package configuration.",
   "exports": {
     ".": {

diff --git a/src/manifest/src/platform.cts b/src/manifest/src/platform.cts
@@ -34,6 +34,14 @@ export function isPlatformPreset(x: unknown): x is PlatformPreset {
   return (typeof x === 'string') && (x in PRESET);
 }
 
+export type PlatformName = NodePlatform | PlatformPreset;
+
+export function assertIsPlatformName(x: unknown): asserts x is PlatformName {
+  if (!isPlatformPreset(x) && !isNodePlatform(x)) {
+    throw new RangeError(`invalid platform name: ${x}`);
+  }
+}
+
 export function assertIsPlatformPreset(x: unknown): asserts x is PlatformPreset {
   if (!isPlatformPreset(x)) {
     throw new RangeError(`invalid platform family preset: ${x}`);
@@ -55,13 +63,13 @@ export function assertIsPlatformMap(json: unknown, path: string): asserts json i
 
 export function assertIsPlatformFamily(json: unknown, path: string): asserts json is PlatformFamily {
   if (typeof json === 'string') {
-    assertIsPlatformPreset(json);
+    assertIsPlatformName(json);
     return;
   }
 
   if (Array.isArray(json)) {
     for (const elt of json) {
-      assertIsPlatformPreset(elt);
+      assertIsPlatformName(elt);
     }
     return;
   }
@@ -71,7 +79,6 @@ export function assertIsPlatformFamily(json: unknown, path: string): asserts jso
 
 export type TargetPair = { node: NodePlatform, rust: RustTarget };
 export type PlatformMap = { [key in NodePlatform]?: RustTarget };
-export type PlatformName = NodePlatform | PlatformPreset;
 
 export type PlatformFamily =
     PlatformName