@voxgig/sdkgen 1.1.1 → 1.2.1

package/bin/voxgig-sdkgen

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

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@voxgig/sdkgen",
-  "version": "1.1.1",
+  "version": "1.2.1",
   "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"
@@ -51,6 +51,7 @@
     "@voxgig/struct": ">=0",
     "@voxgig/util": ">=0",
     "aontu": ">=0",
-    "jostraca": ">=0"
+    "jostraca": ">=0",
+    "shape": ">=10.1"
   }
 }

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

@@ -165,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

@@ -153,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

@@ -199,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

@@ -128,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

@@ -117,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)