@voxgig/sdkgen 4.2.3 → 4.2.4

package/bin/voxgig-sdkgen

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

package/package.json

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

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

@@ -6,7 +6,7 @@ import {
   getModelPath,
 } from '@voxgig/apidef'
 
-import { rustVarName } from './utility_rust'
+import { rustVarName, rustMethodName } from './utility_rust'
 
 
 // Type names come from the shared canonToType 'rust' column (single source of truth).
@@ -56,7 +56,7 @@ const ReadmeEntity = cmp(function ReadmeEntity(props: any) {
     const fields = entity.fields || []
     const idF = entityIdField(entity)
     const eVar = rustVarName(entity.name)
-    const method = rustVarName(entity.name)
+    const method = rustMethodName(entity.name)
 
     Content(`
 ### ${entity.Name}

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

@@ -7,7 +7,7 @@ import {
   nom,
 } from '@voxgig/apidef'
 
-import { crateIdent, rustVarName } from './utility_rust'
+import { crateIdent, rustVarName, rustMethodName } from './utility_rust'
 
 
 // A type-correct rust expression constructing a voxgig struct Value.
@@ -31,7 +31,7 @@ const ReadmeHowto = cmp(function ReadmeHowto(props: any) {
   const { entity: exampleEntity, primaryOp } = pickExampleEntity(entity)
   const eName = exampleEntity ? nom(exampleEntity, 'Name') : 'Entity'
   const eVar = exampleEntity ? rustVarName(exampleEntity.name) : 'entity'
-  const method = exampleEntity ? rustVarName(exampleEntity.name) : 'entity'
+  const method = exampleEntity ? rustMethodName(exampleEntity.name) : 'entity'
   const idF = exampleEntity ? entityIdField(exampleEntity) : null
   const isMatchOp = 'load' === primaryOp || 'remove' === primaryOp
   let testArg = 'Value::Noval'

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

@@ -6,7 +6,7 @@ import {
   getModelPath,
 } from '@voxgig/apidef'
 
-import { rustVarName } from './utility_rust'
+import { rustVarName, rustMethodName } from './utility_rust'
 
 
 const ReadmeIntro = cmp(function ReadmeIntro(props: any) {
@@ -30,7 +30,7 @@ The ${target.title} SDK for the ${model.Name} API — an entity-oriented client
   const exampleEntity = Object.values(entity || {})
     .find((e: any) => e.active !== false) as any
   const example = exampleEntity
-    ? `for example \`client.${rustVarName(exampleEntity.name)}(Value::Noval)\` — each`
+    ? `for example \`client.${rustMethodName(exampleEntity.name)}(Value::Noval)\` — each`
     : 'each'
 
   // Model-driven op list — only the operations the active entities actually

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

@@ -6,7 +6,7 @@ import {
   getModelPath,
 } from '@voxgig/apidef'
 
-import { crateIdent, rustVarName } from './utility_rust'
+import { crateIdent, rustVarName, rustMethodName } from './utility_rust'
 
 
 const ReadmeModel = cmp(function ReadmeModel(props: any) {
@@ -77,7 +77,7 @@ Creates a test-mode client with mock transport. Both arguments may be
 
   each(entityList, (ent: any) => {
     const article = /^[aeiou]/i.test(ent.Name) ? 'an' : 'a'
-    Content(`| \`${rustVarName(ent.name)}\` | \`(entopts: Value) -> Rc<${ent.Name}Entity>\` | Create ${article} ${ent.Name} entity instance. |
+    Content(`| \`${rustMethodName(ent.name)}\` | \`(entopts: Value) -> Rc<${ent.Name}Entity>\` | Create ${article} ${ent.Name} entity instance. |
 `)
   })
 

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

@@ -7,7 +7,7 @@ import {
   nom,
 } from '@voxgig/apidef'
 
-import { crateIdent, rustVarName } from './utility_rust'
+import { crateIdent, rustVarName, rustMethodName } from './utility_rust'
 
 
 const ReadmeQuick = cmp(function ReadmeQuick(props: any) {
@@ -58,7 +58,7 @@ let client = ${ctor};
     const eName = nom(exampleEntity, 'Name')
     const article = /^[aeiou]/i.test(eName) ? 'an' : 'a'
     const eVar = rustVarName(exampleEntity.name)
-    const method = rustVarName(exampleEntity.name)
+    const method = rustMethodName(exampleEntity.name)
     const opnames = entityOps(exampleEntity)
     // Model-driven id key: `idF` is the entity's id-like MATCH field name, or
     // null. `dataIdF` is the id on the RETURNED record's data type.
@@ -91,7 +91,7 @@ match client.${method}(Value::Noval).list(Value::Noval, Value::Noval) {
       const neName = nom(nestedEntity, 'Name')
       const neArticle = /^[aeiou]/i.test(neName) ? 'an' : 'a'
       const neVar = rustVarName(nestedEntity.name)
-      const neMethod = rustVarName(nestedEntity.name)
+      const neMethod = rustMethodName(nestedEntity.name)
 
       const neIdF = entityIdField(nestedEntity)
       const neRequired = opRequestShape(nestedEntity, 'load').items

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

@@ -6,7 +6,7 @@ import {
   getModelPath,
 } from '@voxgig/apidef'
 
-import { crateIdent, rustVarName } from './utility_rust'
+import { crateIdent, rustVarName, rustMethodName } from './utility_rust'
 
 
 // Type names come from the shared canonToType 'rust' column (single source of truth).
@@ -120,7 +120,7 @@ let client = test_sdk(Value::Noval, Value::Noval);
 
     // Entity factory methods
     publishedEntities.map((ent: any) => {
-      Content(`#### \`${rustVarName(ent.name)}(entopts: Value) -> Rc<${ent.Name}Entity>\`
+      Content(`#### \`${rustMethodName(ent.name)}(entopts: Value) -> Rc<${ent.Name}Entity>\`
 
 Create a new \`${ent.Name}Entity\` instance. Pass \`Value::Noval\` for no
 initial options.
@@ -168,7 +168,7 @@ Prepare a fetch definition without sending. Returns the fetchdef on \`Ok\`.
       const fields = ent.fields || []
       const idF = entityIdField(ent)
       const eVar = rustVarName(ent.name)
-      const method = rustVarName(ent.name)
+      const method = rustMethodName(ent.name)
 
       Content(`
 ---

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

@@ -7,7 +7,7 @@ import {
   nom,
 } from '@voxgig/apidef'
 
-import { crateIdent, rustVarName } from './utility_rust'
+import { crateIdent, rustVarName, rustMethodName } from './utility_rust'
 
 
 // A type-correct rust expression constructing a voxgig struct Value for a
@@ -46,7 +46,7 @@ let client = ${ctor};
   if (exampleEntity) {
     const eName = nom(exampleEntity, 'Name')
     const eVar = rustVarName(exampleEntity.name)
-    const method = rustVarName(exampleEntity.name)
+    const method = rustMethodName(exampleEntity.name)
     const opnames = Object.keys(exampleEntity.op || {})
     // Model-driven id key: null when the entity has no id-like field, in which
     // case the load example takes no match argument.

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

@@ -7,7 +7,7 @@ import {
   nom,
 } from '@voxgig/apidef'
 
-import { crateIdent, rustVarName } from './utility_rust'
+import { crateIdent, rustVarName, rustMethodName } from './utility_rust'
 
 
 // A type-correct rust expression constructing a voxgig struct Value for a
@@ -40,7 +40,7 @@ let client = test_sdk(Value::Noval, Value::Noval);
 
   if (exampleEntity && primaryOp) {
     const eName = nom(exampleEntity, 'Name')
-    const method = rustVarName(exampleEntity.name)
+    const method = rustMethodName(exampleEntity.name)
     const idF = entityIdField(exampleEntity)
     const isMatchOp = 'load' === primaryOp || 'remove' === primaryOp
     let arg = 'Value::Noval'

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

@@ -21,7 +21,7 @@ import {
 } from '@voxgig/sdkgen'
 
 
-import { rustVarName } from './utility_rust'
+import { rustVarName, rustMethodName } from './utility_rust'
 
 
 // Rust's GenCtx mirrors the shared shape (see TestEntity_ts.ts) plus the
@@ -100,7 +100,7 @@ const TestEntity = cmp(function TestEntity(props: any) {
     }
 `
 
-  const method = rustVarName(entity.name)
+  const method = rustMethodName(entity.name)
   const evar = rustVarName(entity.name)
 
   File({ name: entity.name + '_entity_test.' + target.ext }, () => {
@@ -330,7 +330,7 @@ const generateCreate: OpGen = (ctx, step, index) => {
   Content(`    // CREATE
 `)
   if (needsEnt) {
-    Content(`    let ${entvar} = client.${rustVarName(entity.name)}(Value::Noval);
+    Content(`    let ${entvar} = client.${rustMethodName(entity.name)}(Value::Noval);
 `)
   }
 
@@ -381,7 +381,7 @@ const generateList: OpGen = (ctx, step, index) => {
   Content(`    // LIST
 `)
   if (needsEnt) {
-    Content(`    let ${entvar} = client.${rustVarName(entity.name)}(Value::Noval);
+    Content(`    let ${entvar} = client.${rustMethodName(entity.name)}(Value::Noval);
 `)
   }
 
@@ -464,7 +464,7 @@ const generateUpdate: OpGen = (ctx, step, index) => {
   Content(`    // UPDATE
 `)
   if (needsEnt) {
-    Content(`    let ${entvar} = client.${rustVarName(entity.name)}(Value::Noval);
+    Content(`    let ${entvar} = client.${rustMethodName(entity.name)}(Value::Noval);
 `)
   }
   Content(`    let ${datavar}_up = Value::empty_map();
@@ -568,7 +568,7 @@ const generateLoad: OpGen = (ctx, step, index) => {
   Content(`    // LOAD
 `)
   if (!hasEntVar) {
-    Content(`    let ${entvar} = client.${rustVarName(entity.name)}(Value::Noval);
+    Content(`    let ${entvar} = client.${rustMethodName(entity.name)}(Value::Noval);
 `)
   }
   if (!hasSrcData && hasEntId) {
@@ -626,7 +626,7 @@ const generateRemove: OpGen = (ctx, step, index) => {
   Content(`    // REMOVE
 `)
   if (needsEnt) {
-    Content(`    let ${entvar} = client.${rustVarName(entity.name)}(Value::Noval);
+    Content(`    let ${entvar} = client.${rustMethodName(entity.name)}(Value::Noval);
 `)
   }
   // Always match the prior-created entity by id to avoid mock-order flakes.

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

@@ -26,17 +26,74 @@ defmodule Voxgig.Struct do
   # ---------------------------------------------------------------------------
 
   @heap :vox_struct_heap
-
+  @heap_owner :vox_struct_heap_owner
+
+  # A named ETS table is owned by the process that created it and dies with
+  # that process. The values here are HANDLES into it - {:vmap, id} - and they
+  # are passed across process boundaries freely: ExUnit runs every test in its
+  # own process, and a generated SDK caches its config in :persistent_term,
+  # which outlives all of them. So the table has to outlive its creator, or a
+  # handle that is still very much alive resolves into a fresh empty table.
+  #
+  # Adding ensure_heap to hget stopped that raising ArgumentError, but a silent
+  # nil turned out to be worse than the raise: the generated config's staleness
+  # check (`usable?`) was written to read the raise as "rebuild me", so once the
+  # raise stopped, a dead handle was cached as good and clone/1 crashed further
+  # down with "Enumerable not implemented for nil of type Atom" - nothing in the
+  # message pointing at the heap.
+  #
+  # The table is therefore owned by a dedicated process that never exits, and
+  # every id stays valid for the life of the VM.
   defp ensure_heap do
     case :ets.whereis(@heap) do
-      :undefined ->
+      :undefined -> start_heap()
+      _ -> :ok
+    end
+  end
+
+  defp start_heap do
+    caller = self()
+
+    spawn(fn ->
+      owner =
         try do
-          :ets.new(@heap, [:set, :public, :named_table])
+          Process.register(self(), @heap_owner)
+          true
         rescue
-          ArgumentError -> :ok
+          ArgumentError -> false
         end
 
-        :ok
+      if owner do
+        case :ets.whereis(@heap) do
+          :undefined -> :ets.new(@heap, [:set, :public, :named_table])
+          _ -> :ok
+        end
+      end
+
+      send(caller, :vox_heap_ready)
+
+      # Hold the table for the life of the VM. This process does nothing else.
+      if owner, do: Process.sleep(:infinity)
+    end)
+
+    receive do
+      :vox_heap_ready -> :ok
+    after
+      5_000 -> :ok
+    end
+
+    await_heap(1_000)
+  end
+
+  # The loser of a registration race reports ready before the winner has
+  # necessarily reached :ets.new, so wait on the table, not on the message.
+  defp await_heap(0), do: :ok
+
+  defp await_heap(n) do
+    case :ets.whereis(@heap) do
+      :undefined ->
+        Process.sleep(1)
+        await_heap(n - 1)
 
       _ ->
         :ok
@@ -50,13 +107,10 @@ 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.
+  # ensure_heap here too, not only in alloc, so a read can never hit a missing
+  # table. With a permanent owner (above) that should not arise; before there
+  # was one, it raised ArgumentError from whichever unrelated test happened to
+  # run second.
   defp hget(id) do
     ensure_heap()
 

package/project/.sdk/tm/zig/Makefile

@@ -8,10 +8,30 @@ build:
 # tests pass, due to *MapRef/*ListRef cross-references in arena teardown. We
 # filter the output: if "N/N tests passed" with N==total, treat as success.
 # `--summary all` is required so the count is printed.
+#
+# A TEST THAT DID NOT COMPILE IS NOT IN THE TALLY. The tolerance above is for a
+# crash AFTER the tests ran, and it must never cover a build that failed: a
+# compile error means those test binaries never ran, so they never reach
+# "total", and "60/66 passed; 6 skipped" reads as a clean 66/66 while five of
+# thirteen build steps are red. That is exactly how a `.undef` tag that does
+# not exist in JsonValue shipped in 4.2.2 and stayed green in every sweep
+# afterwards. So: if any build step failed, this target fails, whatever the
+# test counts say.
 test:
 	@out=$$(zig build test --summary all 2>&1); code=$$?; \
 	echo "$$out"; \
 	if [ $$code -eq 0 ]; then exit 0; fi; \
+	steps=$$(echo "$$out" | grep -oE '[0-9]+/[0-9]+ steps succeeded' | tail -1); \
+	sdone=$$(echo "$$steps" | sed -E 's#([0-9]+)/([0-9]+).*#\1#'); \
+	stotal=$$(echo "$$steps" | sed -E 's#([0-9]+)/([0-9]+).*#\2#'); \
+	if [ -n "$$steps" ] && [ "$$sdone" != "$$stotal" ]; then \
+		echo "zig: $$steps — a build step failed, so the test tally is incomplete"; \
+		exit 1; \
+	fi; \
+	if echo "$$out" | grep -q 'error: the following command failed with .* compilation errors'; then \
+		echo "zig: compilation errors — not a teardown crash"; \
+		exit 1; \
+	fi; \
 	summary=$$(echo "$$out" | grep -oE '[0-9]+/[0-9]+ tests passed(; [0-9]+ skipped)?' | tail -1); \
 	passed=$$(echo "$$summary" | sed -E 's#([0-9]+)/([0-9]+).*#\1#'); \
 	total=$$(echo "$$summary" | sed -E 's#([0-9]+)/([0-9]+).*#\2#'); \

package/project/.sdk/tm/zig/feature/test.zig

@@ -105,7 +105,7 @@ fn fixIds(_: Allocator, key: ?[]const u8, val: Value, _: Value, path: []const []
 // while the fixture plainly held two. Mirrors the go/ts/lua/php mocks, which
 // already wrap; rust, c and zig were the three that did not.
 fn envelope(ctx: *Context, data: Value) Value {
-    if (data == .undef or data == .null) return data;
+    if (data == .null) return data;
     const restf = h.getpath(&.{ "transform", "res" }, ctx.point);
     if (restf != .string) return data;
     const spec = restf.string;

package/project/sdkgen-package.json

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