@voxgig/sdkgen 4.2.1 → 4.2.2

package/bin/voxgig-sdkgen

@@ -8,7 +8,7 @@ const { Shape, One, Skip } = require('shape')
 
 const { SdkGen } = require('../dist/sdkgen.js')
 
-const VERSION = '4.2.1'
+const VERSION = '4.2.2'
 const KONSOLE = console
 
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@voxgig/sdkgen",
-  "version": "4.2.1",
+  "version": "4.2.2",
   "main": "dist/sdkgen.js",
   "type": "commonjs",
   "engines": {

package/project/.sdk/src/cmp/dart/TestEntity_dart.ts

@@ -382,18 +382,21 @@ const generateList: OpGen = (ctx, step, index) => {
     const hasRefData = validRef && allSteps.some(s => 'create' === s.op &&
       ((s.input.ref ?? entity.name + '_ref01') === validRef))
 
-    // Entities in the list are entity instances; select over their data maps.
+    // listvar is ALREADY a list of data maps — the list call above maps
+    // .data() over the entity instances. Mapping .data() a second time here
+    // called it on a plain Map and every entity test died at runtime with
+    // "Class '_Map<String, dynamic>' has no instance method 'data'".
     if ('ItemExists' === validator.apply && hasRefData) {
       Content(`
       ok(!isempty(select(
-          (${listvar} as List).map((e) => e.data()).toList(),
+          ${listvar},
           {'id': ${validRef}_data['id']})));
 `)
     }
     else if ('ItemNotExists' === validator.apply && hasRefData) {
       Content(`
       ok(isempty(select(
-          (${listvar} as List).map((e) => e.data()).toList(),
+          ${listvar},
           {'id': ${validRef}_data['id']})));
 `)
     }

package/project/.sdk/src/cmp/java/TestDirect_java.ts

@@ -55,9 +55,21 @@ function normalizePathParams(
 
 
 // Render a JSON value as a Java literal for query examples.
+// A Java source literal for an example value from the model.
+//
+// An integer beyond Java's int range needs the L suffix: javac rejects a bare
+// 120615523104 outright ("integer number too large"), so one large id in a
+// spec's example makes the whole generated test package fail to compile.
+// A non-integral double is fine unsuffixed.
 function javaLiteral(v: any): string {
   if (null == v) return 'null'
-  if ('number' === typeof v || 'boolean' === typeof v) return String(v)
+  if ('boolean' === typeof v) return String(v)
+  if ('number' === typeof v) {
+    if (Number.isInteger(v) && (v > 2147483647 || v < -2147483648)) {
+      return String(v) + 'L'
+    }
+    return String(v)
+  }
   return JSON.stringify(String(v))
 }
 

package/project/.sdk/src/cmp/kotlin/TestDirect_kotlin.ts

@@ -158,7 +158,7 @@ class ${entity.Name}DirectTest {
         : []
       const listSkipBlock = listLiveIdKeys.length > 0
         ? `    if (setup.live) {
-      for (liveKey in arrayOf(${listLiveIdKeys.map((k: string) => `"${k}"`).join(', ')})) {
+      for (liveKey in arrayOf<String>(${listLiveIdKeys.map((k: string) => `"${k}"`).join(', ')})) {
         Assumptions.assumeTrue(setup.idmap[liveKey] != null,
             "live test needs " + liveKey + " via *_ENTID env var (synthetic IDs only)")
       }
@@ -265,7 +265,7 @@ ${listSkipBlock}    val client = setup.client
       }
       const loadSkipBlock = loadLiveIdKeys.length > 0
         ? `    if (setup.live) {
-      for (liveKey in arrayOf(${loadLiveIdKeys.map(k => `"${k}"`).join(', ')})) {
+      for (liveKey in arrayOf<String>(${loadLiveIdKeys.map(k => `"${k}"`).join(', ')})) {
         Assumptions.assumeTrue(setup.idmap[liveKey] != null,
             "live test needs " + liveKey + " via *_ENTID env var (synthetic IDs only)")
       }

package/project/.sdk/src/cmp/kotlin/TestEntity_kotlin.ts

@@ -114,7 +114,7 @@ class ${entity.Name}EntityTest {
     val setup = ${accessor}BasicSetup(null)
     // Per-op sdk-test-control.json skip.
     val mode = if (setup.live) "live" else "unit"
-    for (op in arrayOf(${stepOps.map(o => `"${o}"`).join(', ')})) {
+    for (op in arrayOf<String>(${stepOps.map(o => `"${o}"`).join(', ')})) {
       val reason = RunnerSupport.skipReason("entityOp", "${entity.name}.\$op", mode)
       Assumptions.assumeTrue(
         reason == null,

package/project/.sdk/src/cmp/perl/TestEntity_perl.ts

@@ -70,6 +70,11 @@ const TestEntity = cmp(function TestEntity(props: any) {
     : []
   const aliases = updateData.map(([k, v]: any) => [k, v])
 
+  // An entity whose basic flow has no steps yields an EMPTY list, and Perl's
+  // `for my $x () {` is a syntax error (unlike `for my $x (()) {`, which is a
+  // valid empty loop) — the whole test file then fails to compile. The loop
+  // below therefore wraps this in a second pair of parens, which is a no-op
+  // for a non-empty list because Perl flattens it.
   const opsList = Array.from(new Set(
     (allSteps as any[]).map((s: any) => s.op).filter(Boolean)))
     .map(o => `'${o}'`).join(', ')
@@ -101,7 +106,7 @@ BASIC_FLOW: {
   my $setup = ${entity.name}_basic_setup(undef);
   my $_live = $setup->{live} ? 1 : 0;
   # Per-op sdk-test-control.json skip.
-  for my $_op (${opsList}) {
+  for my $_op ((${opsList})) {
     my ($_should_skip, $_reason) = ${N}TestRunner::is_control_skipped(
       'entityOp', "${entity.name}." . $_op, $_live ? 'live' : 'unit');
     if ($_should_skip) {

package/project/.sdk/src/cmp/rust/utility_rust.ts

@@ -20,7 +20,14 @@ const RUST_RESERVED = new Set<string>([
 
 // A collision-free snake_case rust identifier for a model name.
 function rustVarName(name: string): string {
-  const snake = name.replace(/[^a-zA-Z0-9_]/g, '_').toLowerCase()
+  let snake = name.replace(/[^a-zA-Z0-9_]/g, '_').toLowerCase()
+  // No Rust identifier may begin with a digit. A `3d_id` field became
+  // `pub 3_d_id: String`, which does not compile ("expected identifier, found
+  // `3_d_id`") and took the whole crate with it. Prefixed with an underscore,
+  // the convention cmp/c/utility_c.ts already applies.
+  if (/^[0-9]/.test(snake)) {
+    snake = '_' + snake
+  }
   return RUST_RESERVED.has(snake) ? snake + '_' : snake
 }
 

package/project/.sdk/src/cmp/swift/utility_swift.ts

@@ -22,13 +22,27 @@ function projectPath(suffix?: string): string {
 function swiftVarName(name: string): string {
   const pascal = camelify(name)
   const lower = pascal.charAt(0).toLowerCase() + pascal.slice(1)
-  return SWIFT_RESERVED.has(lower) ? lower + '_' : lower
+  if (SWIFT_RESERVED.has(lower)) {
+    return lower + '_'
+  }
+  return leadingDigitSafe(lower)
+}
+
+
+// No Swift identifier may begin with a digit, but a model field may: a
+// `3d_id` field camelifies to `3DId`, and `public var 3DId: String` does not
+// merely fail to bind — the compiler reads it as a malformed number and the
+// whole module dies on "'D' is not a valid digit in integer literal", naming
+// nothing that leads back to the field. Prefixed with an underscore, the same
+// convention cmp/c/utility_c.ts already applies.
+function leadingDigitSafe(ident: string): string {
+  return /^[0-9]/.test(ident) ? '_' + ident : ident
 }
 
 
 // A PascalCase Swift identifier for a snake_case model name.
 function swiftPascalName(name: string): string {
-  return camelify(name)
+  return leadingDigitSafe(camelify(name))
 }
 
 

package/project/.sdk/tm/elixir/lib/utility/struct/voxgig_struct.ex

@@ -50,14 +50,26 @@ defmodule Voxgig.Struct do
     id
   end
 
+  # ensure_heap here too, not only in alloc. A named ETS table belongs to the
+  # process that created it and dies with it — and ExUnit runs each test in its
+  # own process, so a table created by an alloc in one test is gone by the
+  # next. hget then raised ArgumentError ("the table identifier does not refer
+  # to an existing ETS table") from whichever unrelated test happened to run
+  # second, which is why this surfaced as a pipeline/netsim failure rather than
+  # as anything to do with the heap.
   defp hget(id) do
+    ensure_heap()
+
     case :ets.lookup(@heap, id) do
       [{_, c}] -> c
       _ -> nil
     end
   end
 
-  defp hset(id, contents), do: :ets.insert(@heap, {id, contents})
+  defp hset(id, contents) do
+    ensure_heap()
+    :ets.insert(@heap, {id, contents})
+  end
 
   defp vmap_new(pairs), do: {:vmap, alloc(pairs)}
   defp vlist_new(items), do: {:vlist, alloc(items)}

package/project/sdkgen-package.json

@@ -3,7 +3,7 @@
     "package": 1
   },
   "name": "@voxgig/sdkgen",
-  "version": "4.2.1",
+  "version": "4.2.2",
   "provides": {
     "target": [
       "c",