@voxgig/sdkgen 1.0.1 → 1.2.0

package/bin/voxgig-sdkgen

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

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@voxgig/sdkgen",
-  "version": "1.0.1",
+  "version": "1.2.0",
   "main": "dist/sdkgen.js",
   "type": "commonjs",
   "types": "dist/sdkgen.d.ts",
@@ -41,7 +41,7 @@
   ],
   "devDependencies": {
     "@types/js-yaml": "4.0.9",
-    "@types/node": "25.7.0",
+    "@types/node": "25.8.0",
     "json-schema-to-ts": "3.1.1",
     "memfs": "4.57.2",
     "typescript": "6.0.3"

package/project/.sdk/src/cmp/go/TestEntity_go.ts

@@ -593,16 +593,14 @@ const generateRemove: OpGen = (ctx, step, index) => {
   // otherwise reuse the `err` from a prior op step.
   const errOp = needsEnt ? ':=' : '='
 
-  const hasEntIdR = null != entity.id
-
   Content(`		// REMOVE
 `)
   if (needsEnt) {
     Content(`		${entvar} := client.${entity.Name}(nil)
 `)
   }
-  if (hasEntIdR) {
-    Content(`		${matchvar} := map[string]any{
+  // Always match the prior-created entity by id to avoid mock-order flakes.
+  Content(`		${matchvar} := map[string]any{
 			"id": ${srcdatavar}["id"],
 		}
 		_, err ${errOp} ${entvar}.Remove(${matchvar}, nil)
@@ -610,15 +608,6 @@ const generateRemove: OpGen = (ctx, step, index) => {
 			t.Fatalf("remove failed: %v", err)
 		}
 `)
-  }
-  else {
-    Content(`		${matchvar} := map[string]any{}
-		_, err ${errOp} ${entvar}.Remove(${matchvar}, nil)
-		if err != nil {
-			t.Fatalf("remove failed: %v", err)
-		}
-`)
-  }
 }
 
 

package/project/.sdk/src/cmp/lua/TestEntity_lua.ts

@@ -502,28 +502,19 @@ const generateRemove: OpGen = (ctx, step, index) => {
   const needsEnt = !priorSteps.some((s: any) =>
     ['create', 'list', 'load', 'update', 'remove'].includes(s.op))
 
-  const hasEntIdR = null != entity.id
-
   Content(`    -- REMOVE
 `)
   if (needsEnt) {
     Content(`    local ${entvar} = client:${entity.Name}(nil)
 `)
   }
-  if (hasEntIdR) {
-    Content(`    local ${matchvar} = {
+  // Always match the prior-created entity by id to avoid mock-order flakes.
+  Content(`    local ${matchvar} = {
       id = ${srcdatavar}["id"],
     }
     local _, err = ${entvar}:remove(${matchvar}, nil)
     assert.is_nil(err)
 `)
-  }
-  else {
-    Content(`    local ${matchvar} = {}
-    local _, err = ${entvar}:remove(${matchvar}, nil)
-    assert.is_nil(err)
-`)
-  }
 }
 
 

package/project/.sdk/src/cmp/php/TestEntity_php.ts

@@ -507,28 +507,19 @@ const generateRemove: OpGen = (ctx, step, index) => {
   const needsEnt = !priorSteps.some((s: any) =>
     ['create', 'list', 'load', 'update', 'remove'].includes(s.op))
 
-  const hasEntIdR = null != entity.id
-
   Content(`        // REMOVE
 `)
   if (needsEnt) {
     Content(`        $${entvar} = $client->${accessor}(null);
 `)
   }
-  if (hasEntIdR) {
-    Content(`        $${matchvar} = [
+  // Always match the prior-created entity by id to avoid mock-order flakes.
+  Content(`        $${matchvar} = [
             "id" => $${srcdatavar}["id"],
         ];
         [$_, $err] = $${entvar}->remove($${matchvar}, null);
         $this->assertNull($err);
 `)
-  }
-  else {
-    Content(`        $${matchvar} = [];
-        [$_, $err] = $${entvar}->remove($${matchvar}, null);
-        $this->assertNull($err);
-`)
-  }
 }
 
 

package/project/.sdk/src/cmp/py/TestEntity_py.ts

@@ -500,28 +500,19 @@ const generateRemove: OpGen = (ctx, step, index) => {
   const needsEnt = !priorSteps.some((s: any) =>
     ['create', 'list', 'load', 'update', 'remove'].includes(s.op))
 
-  const hasEntIdR = null != entity.id
-
   Content(`        # REMOVE
 `)
   if (needsEnt) {
     Content(`        ${entvar} = client.${entity.Name}(None)
 `)
   }
-  if (hasEntIdR) {
-    Content(`        ${matchvar} = {
+  // Always match the prior-created entity by id to avoid mock-order flakes.
+  Content(`        ${matchvar} = {
             "id": ${srcdatavar}["id"],
         }
         _, err = ${entvar}.remove(${matchvar}, None)
         assert err is None
 `)
-  }
-  else {
-    Content(`        ${matchvar} = {}
-        _, err = ${entvar}.remove(${matchvar}, None)
-        assert err is None
-`)
-  }
 }
 
 

package/project/.sdk/src/cmp/rb/TestEntity_rb.ts

@@ -493,28 +493,19 @@ const generateRemove: OpGen = (ctx, step, index) => {
   const needsEnt = !priorSteps.some((s: any) =>
     ['create', 'list', 'load', 'update', 'remove'].includes(s.op))
 
-  const hasEntIdR = null != entity.id
-
   Content(`    # REMOVE
 `)
   if (needsEnt) {
     Content(`    ${entvar} = client.${entity.Name}(nil)
 `)
   }
-  if (hasEntIdR) {
-    Content(`    ${matchvar} = {
+  // Always match the prior-created entity by id to avoid mock-order flakes.
+  Content(`    ${matchvar} = {
       "id" => ${srcdatavar}["id"],
     }
     _, err = ${entvar}.remove(${matchvar}, nil)
     assert_nil err
 `)
-  }
-  else {
-    Content(`    ${matchvar} = {}
-    _, err = ${entvar}.remove(${matchvar}, nil)
-    assert_nil err
-`)
-  }
 }
 
 

package/project/.sdk/src/cmp/ts/TestEntity_ts.ts

@@ -516,8 +516,6 @@ const generateRemove: OpGen = (ctx, step, index) => {
   const needsEnt = !priorSteps.some(s =>
     ['create', 'list', 'load', 'update', 'remove'].includes(s.op))
 
-  const hasEntIdR = null != entity.id
-
   Content(`
     // REMOVE
 `)
@@ -525,13 +523,11 @@ const generateRemove: OpGen = (ctx, step, index) => {
     Content(`    const ${entvar} = client.${nom(entity, 'Name')}()
 `)
   }
-  Content(`    const ${matchvar}: any = {}
-`)
-  if (hasEntIdR) {
-    Content(`    ${matchvar}.id = ${srcdatavar}.id
-`)
-  }
-  Content(`    await ${entvar}.remove(${matchvar})
+  // Always match the prior-created entity by id. The mock test feature
+  // removes the first match in entmap, so without a specific id the
+  // result depends on hash-sort order and flakes (see cheapshark).
+  Content(`    const ${matchvar}: any = { id: ${srcdatavar}.id }
+    await ${entvar}.remove(${matchvar})
   `)
 }
 

package/project/.sdk/tm/go/feature/test_feature.go

@@ -114,8 +114,10 @@ func (f *TestFeature) Init(ctx *core.Context, options map[string]any) {
 			// Match the existing entity by id only (or its alias). Reqdata
 			// also contains the new field values, which would otherwise
 			// cause Select to filter out the entity we want to update.
-			// Falls back to first entity when no match found, mirroring
-			// the TS mock.
+			// When reqdata has no id, fall back to the id the entity
+			// client carries from a prior create/load (in ctx.Match /
+			// ctx.Data), mirroring the TS mock where param(ctx,'id')
+			// resolves from accumulated state.
 			updateMatch := map[string]any{}
 			if ctx.Reqdata != nil {
 				if v, has := ctx.Reqdata["id"]; has {
@@ -131,6 +133,9 @@ func (f *TestFeature) Init(ctx *core.Context, options map[string]any) {
 					}
 				}
 			}
+			if len(updateMatch) == 0 {
+				updateMatch = resolveMatch(map[string]any{})
+			}
 			args := self.buildArgs(ctx, op, updateMatch)
 			found := vs.Select(entmap, args)
 			ent := vs.GetElem(found, 0)
@@ -160,9 +165,8 @@ func (f *TestFeature) Init(ctx *core.Context, options map[string]any) {
 			args := self.buildArgs(ctx, op, resolveMatch(ctx.Reqmatch))
 			found := vs.Select(entmap, args)
 			ent := vs.GetElem(found, 0)
-			if ent == nil {
-				return respond(404, nil, map[string]any{"statusText": "Not found"}), nil
-			}
+			// Remove only the first matched entity. If nothing matches,
+			// succeed as a no-op rather than erroring.
 			if entm, ok := ent.(map[string]any); ok {
 				id := vs.GetProp(entm, "id")
 				vs.DelProp(entmap, id)

package/project/.sdk/tm/lua/feature/test_feature.lua

@@ -109,8 +109,10 @@ function TestFeature:init(ctx, options)
     elseif op.name == "update" then
       -- Match the existing entity by id only (or its alias). reqdata also
       -- contains the new field values, which would otherwise cause select
-      -- to filter out the entity we want to update. Falls back to first
-      -- entity when no match found, mirroring the TS mock.
+      -- to filter out the entity we want to update. When reqdata has no id,
+      -- fall back to the id the entity client carries from a prior
+      -- create/load (in fctx.match / fctx.data), mirroring the TS mock
+      -- where param(ctx,'id') resolves from accumulated state.
       local update_match = {}
       if type(fctx.reqdata) == "table" then
         if fctx.reqdata["id"] ~= nil then update_match["id"] = fctx.reqdata["id"] end
@@ -121,6 +123,9 @@ function TestFeature:init(ctx, options)
           end
         end
       end
+      if next(update_match) == nil then
+        update_match = resolve_match({})
+      end
       local args = test_self:build_args(fctx, op, update_match)
       local found = vs.select(entmap, args)
       local ent = vs.getelem(found, 0)
@@ -148,9 +153,8 @@ function TestFeature:init(ctx, options)
       local args = test_self:build_args(fctx, op, resolve_match(fctx.reqmatch))
       local found = vs.select(entmap, args)
       local ent = vs.getelem(found, 0)
-      if ent == nil then
-        return respond(404, nil, { statusText = "Not found" })
-      end
+      -- Remove only the first matched entity. If nothing matches,
+      -- succeed as a no-op rather than erroring.
       if type(ent) == "table" then
         local id = vs.getprop(ent, "id")
         vs.delprop(entmap, id)

package/project/.sdk/tm/php/feature/TestFeature.php

@@ -162,6 +162,10 @@ class ProjectNameTestFeature extends ProjectNameBaseFeature
                 // Match the existing entity by id only (or its alias). reqdata
                 // also contains the new field values, which would otherwise
                 // cause find_first to filter out the entity we want to update.
+                // When reqdata has no id, fall back to the id the entity
+                // client carries from a prior create/load (in $fctx->match /
+                // $fctx->data), mirroring the TS mock where param(ctx,'id')
+                // resolves from accumulated state.
                 $update_match = [];
                 if (is_array($fctx->reqdata)) {
                     if (array_key_exists('id', $fctx->reqdata)) {
@@ -172,6 +176,9 @@ class ProjectNameTestFeature extends ProjectNameBaseFeature
                         $update_match[$id_alias] = $fctx->reqdata[$id_alias];
                     }
                 }
+                if (empty($update_match)) {
+                    $update_match = $resolve_match([]);
+                }
                 $ent = $find_first($entmap, $update_match, $alias);
                 if ($ent === null) {
                     return $respond(404, null, ['statusText' => 'Not found']);
@@ -192,9 +199,8 @@ class ProjectNameTestFeature extends ProjectNameBaseFeature
 
             } elseif ($op->name === 'remove') {
                 $ent = $find_first($entmap, $resolve_match($fctx->reqmatch), $alias);
-                if ($ent === null) {
-                    return $respond(404, null, ['statusText' => 'Not found']);
-                }
+                // Remove only the first matched entity. If nothing matches,
+                // succeed as a no-op rather than erroring.
                 $id = is_array($ent) ? ($ent['id'] ?? null) : null;
                 if ($id !== null) {
                     unset($entmap[$id]);

package/project/.sdk/tm/py/feature/test_feature.py

@@ -90,8 +90,10 @@ class ProjectNameTestFeature(ProjectNameBaseFeature):
                 # Match the existing entity by id only (or its alias). reqdata
                 # also contains the new field values, which would otherwise
                 # cause select to filter out the entity we want to update.
-                # Falls back to first entity when no match found, mirroring
-                # the TS mock.
+                # When reqdata has no id, fall back to the id the entity
+                # client carries from a prior create/load (in fctx.match /
+                # fctx.data), mirroring the TS mock where param(ctx,'id')
+                # resolves from accumulated state.
                 update_match = {}
                 if isinstance(fctx.reqdata, dict):
                     if "id" in fctx.reqdata:
@@ -101,6 +103,8 @@ class ProjectNameTestFeature(ProjectNameBaseFeature):
                         alias_id = vs.getprop(alias_map, "id")
                         if alias_id is not None and alias_id in fctx.reqdata:
                             update_match[alias_id] = fctx.reqdata[alias_id]
+                if not update_match:
+                    update_match = _resolve_match({})
                 args = test_self.build_args(fctx, op, update_match)
                 found = vs.select(entmap, args)
                 ent = vs.getelem(found, 0)
@@ -124,8 +128,8 @@ class ProjectNameTestFeature(ProjectNameBaseFeature):
                 args = test_self.build_args(fctx, op, _resolve_match(fctx.reqmatch))
                 found = vs.select(entmap, args)
                 ent = vs.getelem(found, 0)
-                if ent is None:
-                    return respond(404, None, {"statusText": "Not found"})
+                # Remove only the first matched entity. If nothing matches,
+                # succeed as a no-op rather than erroring.
                 if isinstance(ent, dict):
                     eid = vs.getprop(ent, "id")
                     vs.delprop(entmap, eid)

package/project/.sdk/tm/rb/feature/test_feature.rb

@@ -84,10 +84,10 @@ class ProjectNameTestFeature < ProjectNameBaseFeature
       elsif op.name == "update"
         # Match the existing entity by id only (or its alias). reqdata also
         # contains the new field values, which would otherwise cause select
-        # to filter out the entity we want to update. Falls back to first
-        # entity when no id present and no match found, mirroring the TS
-        # mock's empirical behavior where param(undef) collapses to "no
-        # constraint" and select returns all.
+        # to filter out the entity we want to update. When reqdata has no id,
+        # fall back to the id the entity client carries from a prior
+        # create/load (in fctx.match / fctx.data), mirroring the TS mock
+        # where param(ctx,'id') resolves from accumulated state.
         update_match = {}
         if fctx.reqdata.is_a?(Hash)
           update_match["id"] = fctx.reqdata["id"] if fctx.reqdata.key?("id")
@@ -98,6 +98,7 @@ class ProjectNameTestFeature < ProjectNameBaseFeature
             end
           end
         end
+        update_match = resolve_match.call({}) if update_match.empty?
         args = test_self.build_args(fctx, op, update_match)
         found = VoxgigStruct.select(entmap, args)
         ent = VoxgigStruct.getelem(found, 0)
@@ -116,7 +117,8 @@ class ProjectNameTestFeature < ProjectNameBaseFeature
         args = test_self.build_args(fctx, op, resolve_match.call(fctx.reqmatch))
         found = VoxgigStruct.select(entmap, args)
         ent = VoxgigStruct.getelem(found, 0)
-        return respond.call(404, nil, { "statusText" => "Not found" }) unless ent
+        # Remove only the first matched entity. If nothing matches,
+        # succeed as a no-op rather than erroring.
         if ent.is_a?(Hash)
           id = VoxgigStruct.getprop(ent, "id")
           VoxgigStruct.delprop(entmap, id)

package/project/.sdk/tm/ts/src/feature/test/TestFeature.ts

@@ -124,13 +124,12 @@ class TestFeature extends BaseFeature {
         const args = self.buildArgs(ctx, op, ctx.reqmatch)
         const found = select(entmap, args)
         const ent = getelem(found, 0)
-        if (null == ent) {
-          return respond(404, undefined, { statusText: S_NOT_FOUND })
-        }
-        else {
+        // Remove only the first matched entity. If nothing matches,
+        // succeed as a no-op rather than erroring.
+        if (null != ent) {
           delprop(entmap, getprop(ent, 'id'))
-          return respond(200)
         }
+        return respond(200)
       }
       else if ('create' === op.name) {
         const args = self.buildArgs(ctx, op, ctx.reqdata)