@voxgig/sdkgen 1.3.17 → 1.3.18

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

@@ -1,5 +1,5 @@
 
-import { cmp, Content, entityClassName } from '@voxgig/sdkgen'
+import { cmp, Content, entityClassName, entityCollection } from '@voxgig/sdkgen'
 
 import {
   KIT,
@@ -12,7 +12,7 @@ const MainEntity = cmp(async function MainEntity(props: any) {
   const { model } = props.ctx$
 
   // Collision-free entity CLASS name (see entityClassName).
-  const entityColl = getModelPath(model, `main.${KIT}.entity`)
+  const entityColl = entityCollection(model)
   const cls = entityClassName(entity, entityColl)
 
   const Name = model.const.Name

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

@@ -28,7 +28,7 @@ const Package = cmp(async function Package(props: any) {
   const Name = model.const.Name
 
   const deps: Record<string, string> = {}
-  for (const d of collectDeps(model, target.name, target.deps)) {
+  for (const d of collectDeps(model, target.name, target.deps, ctx$.log)) {
     deps[d.name] = d.source === 'target' ? (d.version || '0.0.0') : d.version
   }
   // Vendored, dependency-free by design; declared deps (if any) are ignored

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

@@ -43,7 +43,9 @@ const ReadmeExamplesTest = cmp(function ReadmeExamplesTest(props: any) {
 // Checks, PER DOCUMENT that exists:
 //   1. PRESENCE: the doc holds at least one swift block.
 //   2. STRUCTURE: every swift block's braces / parens / brackets balance, so a
-//      truncated or malformed generated snippet fails loudly.
+//      truncated or malformed generated snippet fails loudly. An unclosed fence
+//      (odd number of fence markers) also fails, rather than silently absorbing
+//      the trailing text as a block.
 //
 // GENERATED by @voxgig/sdkgen — do not edit by hand.
 
@@ -94,6 +96,16 @@ final class ReadmeExamplesTest: XCTestCase {
     return blocks
   }
 
+  // True when the doc has a fence that was opened but never closed. A
+  // well-formed doc holds fence markers in open/close pairs, so an ODD number
+  // of markers means the final fence dangles — the split-based scan above would
+  // otherwise silently treat the truncated tail as a block. This is the
+  // structural analogue of an at-EOF "still inside a block" check.
+  private func hasUnclosedFence(_ text: String) -> Bool {
+    let fenceCount = text.components(separatedBy: fence).count - 1
+    return fenceCount % 2 != 0
+  }
+
   // True when every open/close delimiter balances, ignoring characters inside
   // string literals and line comments (a generated snippet never nests an
   // unbalanced delimiter outside those, and this catches truncation).
@@ -149,6 +161,11 @@ final class ReadmeExamplesTest: XCTestCase {
     var failures: [String] = []
     for doc in docs() {
       guard let text = read(doc.path) else { continue }
+      // A fence opened but never closed is a malformed doc — fail loudly rather
+      // than silently absorbing the unterminated tail as a block.
+      if hasUnclosedFence(text) {
+        failures.append(doc.label + " has an unclosed swift code fence")
+      }
       let blocks = swiftBlocks(text)
       var balancedCount = 0
       for (i, b) in blocks.enumerated() {

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

@@ -24,7 +24,7 @@ const TestDirect = cmp(function TestDirect(props: any) {
 
   const Name = model.const.Name
 
-  const opnames = Object.keys(entity.op)
+  const opnames = Object.keys(entity.op || {})
   if (!opnames.includes('load') && !opnames.includes('list')) {
     return
   }

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

@@ -21,7 +21,7 @@ import {
   File, Content,
 } from '@voxgig/sdkgen'
 
-import { canonToType, opTypeName, opRequestShape, warnEntityTypeCollisions } from '@voxgig/sdkgen'
+import { canonToType, opTypeName, opRequestShape, warnEntityTypeCollisions , deriveEntityNames } from '@voxgig/sdkgen'
 
 import {
   KIT,
@@ -51,11 +51,10 @@ const EntityTypes = cmp(function EntityTypes(props: any) {
   // filter, so inactive entities still get class files referencing these
   // typed names. Filter on `name` (always present), NOT `active` — parity
   // with the go emitter's fix.
-  const entityList = each(entity).filter((e: any) => e && null != e.name)
+  const entityList = deriveEntityNames(entity)
   // Derive the PascalCase Name up-front — it is set LAZILY by names(), so an
   // entity not yet named (e.g. a fieldless placeholder) would otherwise read
   // `Name = undefined` below. Parity with the go emitter's fix.
-  entityList.forEach((e: any) => { if (null == e.Name) names(e, e.name) })
 
   // Surface duplicate generated type names (two entities with the same
   // PascalCase Name) — they would redeclare a type in statically-typed

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

@@ -4,7 +4,7 @@ import * as Path from 'node:path'
 import {
   cmp, each, camelify, names,
   File, Content, Folder, Fragment, Line, FeatureHook, Slot,
-  opTypeName, entityClassName,
+  opTypeName, entityClassName, entityCollection,
 } from '@voxgig/sdkgen'
 
 import {
@@ -24,7 +24,7 @@ const Entity = cmp(function Entity(props: any) {
   // `<Name>Entity`, disambiguated when it would clash with another entity's
   // data-type name. The DATA type stays `<Name>`. The class file name and the
   // Main import path both use this, so they always agree.
-  const entityColl = getModelPath(model, `main.${KIT}.entity`)
+  const entityColl = entityCollection(model)
   const cls = entityClassName(entity, entityColl)
 
   const entrep = {
@@ -52,7 +52,7 @@ const Entity = cmp(function Entity(props: any) {
 
     File({ name: cls + '.' + target.name }, () => {
 
-      const opnames = Object.keys(entity.op)
+      const opnames = Object.keys(entity.op || {})
 
       const opfrags =
         (['load', 'list', 'create', 'update', 'remove']

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

@@ -1,6 +1,6 @@
 
 
-import { cmp, Content, entityClassName } from '@voxgig/sdkgen'
+import { cmp, Content, entityClassName, entityCollection } from '@voxgig/sdkgen'
 
 import {
   KIT,
@@ -14,7 +14,7 @@ const MainEntity = cmp(async function MainEntity(props: any) {
 
   // Return the collision-free class TYPE (entityClassName); the accessor METHOD
   // name (entity.Name) is unchanged so callers still write client.<Name>().
-  const cls = entityClassName(entity, getModelPath(model, `main.${KIT}.entity`))
+  const cls = entityClassName(entity, entityCollection(model))
 
   Content(`
   // Entity access: \`client.${entity.Name}().list()\` / \`client.${entity.Name}().load({ id })\`.

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

@@ -4,7 +4,7 @@ import * as Path from 'node:path'
 import {
   cmp, each, names, cmap,
   List, File, Content, Copy, Folder, Fragment, Line, FeatureHook,
-  entityClassName,
+  entityClassName, entityCollection,
 } from '@voxgig/sdkgen'
 
 
@@ -57,7 +57,7 @@ const Main = cmp(async function Main(props: any) {
       Line(`// ${model.const.Name} ${target.Name} SDK\n`)
 
       List({ item: entity }, ({ item }: any) => {
-        const cls = entityClassName(item, entity)
+        const cls = entityClassName(item, entityCollection(model))
         return Line(`import { ${cls} } from './entity/${cls}'`)
       })
 

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

@@ -49,7 +49,7 @@ const TestDirect = cmp(function TestDirect(props: any) {
       apikey: env.${PROJECTNAME}_APIKEY,`
     : ''
 
-  const opnames = Object.keys(entity.op)
+  const opnames = Object.keys(entity.op || {})
   const hasLoad = opnames.includes('load')
   const hasList = opnames.includes('list')
 
@@ -151,7 +151,7 @@ function unwrapListData(data: any): any[] | null {
 
 
 function generateDirectLoad(model: Model, entity: ModelEntity) {
-  const loadOp = entity.op.load
+  const loadOp = entity.op?.load
   const loadPoint: ModelPoint | undefined = loadOp?.points?.[0]
 
   if (null == loadPoint) {
@@ -198,7 +198,7 @@ function generateDirectLoad(model: Model, entity: ModelEntity) {
     .join('\n')
 
   // Get list info for live mode bootstrapping
-  const listOp = entity.op.list
+  const listOp = entity.op?.list
   const listPoint = listOp?.points?.[0]
   const listParams = listPoint?.args?.params || []
   const listPath = listPoint ? normalizePathParams(listPoint.parts || [], listParams, listPoint.rename?.param) : ''
@@ -356,7 +356,7 @@ ${paramAsserts}    }
 
 
 function generateDirectList(model: Model, entity: ModelEntity) {
-  const listOp = entity.op.list
+  const listOp = entity.op?.list
   const listPoint: ModelPoint | undefined = listOp?.points?.[0]
 
   if (null == listPoint) {

package/project/.sdk/src/cmp/zig/Entity_zig.ts

@@ -4,7 +4,7 @@ import * as Path from 'node:path'
 import {
   cmp, camelify,
   File, Content, Folder, Fragment,
-  entityClassName,
+  entityClassName, entityCollection,
 } from '@voxgig/sdkgen'
 
 import {
@@ -20,7 +20,7 @@ const Entity = cmp(function Entity(props: any) {
   const { model, stdrep } = props.ctx$
   const { target, entity } = props
 
-  const entityColl = getModelPath(model, `main.${KIT}.entity`)
+  const entityColl = entityCollection(model)
   const cls = entityClassName(entity, entityColl)
 
   const entrep = {
@@ -34,7 +34,7 @@ const Entity = cmp(function Entity(props: any) {
 
     File({ name: zigVarName(entity.name) + '.' + target.ext }, () => {
 
-      const opnames = Object.keys(entity.op)
+      const opnames = Object.keys(entity.op || {})
 
       // For each CRUD op: if the spec defines it, splice in the real
       // implementation. Otherwise emit a stub returning the unsupported_op

package/project/.sdk/src/cmp/zig/MainEntity_zig.ts

@@ -1,5 +1,5 @@
 
-import { cmp, Content, entityClassName } from '@voxgig/sdkgen'
+import { cmp, Content, entityClassName, entityCollection } from '@voxgig/sdkgen'
 
 import {
   KIT,
@@ -16,7 +16,7 @@ const MainEntity = cmp(async function MainEntity(props: any) {
   const { entity } = props
   const { model } = props.ctx$
 
-  const entityColl = getModelPath(model, `main.${KIT}.entity`)
+  const entityColl = entityCollection(model)
   const cls = entityClassName(entity, entityColl)
   const method = zigVarName(entity.name)
   const mod = zigVarName(entity.name)

package/project/.sdk/src/cmp/zig/Package_zig.ts

@@ -23,8 +23,16 @@ import { zigModuleName } from './utility_zig'
 //
 // The `.paths` allow-list mirrors the (registry-less) template: it only
 // matters when packaging for a registry, which zig does not do, so it is kept
-// stable. `.name` is a plain string here (matching the target's zig toolchain
-// pin) rather than the newer enum-literal + fingerprint form.
+// stable. `.name` is a plain string here rather than the newer enum-literal +
+// fingerprint form.
+//
+// TOOLCHAIN PIN: the string form REQUIRES zig <= 0.13. Zig 0.14 rejects it
+// outright — `build.zig.zon:2:13: error: expected enum literal` — so the
+// generated SDK will not build at all on a newer toolchain. The consumer CI
+// job pins 0.13.0 to match (create-sdkgen
+// project/standard/.github/workflows/ci.yml). Moving to a newer zig means
+// changing BOTH: `.name = .${name}` plus a `.fingerprint` here, and the CI
+// pin there.
 const Package = cmp(async function Package(props: any) {
   const ctx$ = props.ctx$
   const model: Model = ctx$.model

package/project/.sdk/tm/cpp/test/primary_utility_test.cpp

@@ -669,23 +669,16 @@ static void prepareParamsBasic() {
 // --- preparePath ------------------------------------------------------------
 
 static void preparePathBasic() {
+  // Was hand-written cases that had drifted out of the shared corpus (the
+  // preparePath fixture shipped as an empty `set: []`).
   auto c = client();
   UtilityPtr utility = c->getUtility();
-  CtxPtr ctx = makeTestFullCtx(c, utility);
-  ctx->point = fhMap({
-      {"parts", vlist({Value("api"), Value("planet"), Value("{id}")})},
-      {"args", fhMap({{"params", vlist()}})}});
-  ASSERT_EQ(utility->preparePath(ctx), std::string("api/planet/{id}"), "expected api/planet/{id}");
-}
-
-static void preparePathSingle() {
-  auto c = client();
-  UtilityPtr utility = c->getUtility();
-  CtxPtr ctx = makeTestFullCtx(c, utility);
-  ctx->point = fhMap({
-      {"parts", vlist({Value("items")})},
-      {"args", fhMap({{"params", vlist()}})}});
-  ASSERT_EQ(utility->preparePath(ctx), std::string("items"), "expected items");
+  rs::runset("preparePath-basic", rs::get_spec(primary(), {"preparePath", "basic"}),
+             [&](const Value& entry) -> Value {
+    Value ctxmap = Helpers::toMapAny(getp(entry, "ctx"));
+    CtxPtr ctx = rs::make_ctx_from_map(ctxmap, c, utility);
+    return Value(utility->preparePath(ctx));
+  });
 }
 
 // --- prepareQuery -----------------------------------------------------------
@@ -830,7 +823,6 @@ int main() {
   T_RUN(prepareMethodBasic);
   T_RUN(prepareParamsBasic);
   T_RUN(preparePathBasic);
-  T_RUN(preparePathSingle);
   T_RUN(prepareQueryBasic);
   T_RUN(resultBasicBasic);
   T_RUN(resultBodyBasic);

package/project/.sdk/tm/csharp/test/PrimaryUtilityTest.cs

@@ -789,38 +789,20 @@ public class PrimaryUtilityTest
         });
     }
 
+    // Was two hand-written cases that had drifted out of the shared corpus
+    // (the preparePath fixture shipped as an empty `set: []`). Now driven by
+    // the corpus like every other section, so all ports assert the same
+    // separator / blank-segment behaviour.
     [Fact]
     public void PreparePathBasic()
     {
-        var ctx = MakeTestFullCtx(_client, _utility);
-        ctx.Point = new Dictionary<string, object?>
+        TestRunner.RunSet(TestRunner.GetSpec(_primary, "preparePath", "basic"), entry =>
         {
-            ["parts"] = new List<object?> { "api", "planet", "{id}" },
-            ["args"] = new Dictionary<string, object?>
-            {
-                ["params"] = new List<object?>(),
-            },
-        };
-
-        var path = _utility.PreparePath(ctx);
-        Assert.Equal("api/planet/{id}", path);
-    }
-
-    [Fact]
-    public void PreparePathSingle()
-    {
-        var ctx = MakeTestFullCtx(_client, _utility);
-        ctx.Point = new Dictionary<string, object?>
-        {
-            ["parts"] = new List<object?> { "items" },
-            ["args"] = new Dictionary<string, object?>
-            {
-                ["params"] = new List<object?>(),
-            },
-        };
-
-        var path = _utility.PreparePath(ctx);
-        Assert.Equal("items", path);
+            var ctxmap = entry.TryGetValue("ctx", out var c)
+                ? c as Dictionary<string, object?> : null;
+            var ctx = TestRunner.MakeCtxFromMap(ctxmap, _client, _utility);
+            return _utility.PreparePath(ctx);
+        });
     }
 
     [Fact]

package/project/.sdk/tm/dart/test/primary_test.dart

@@ -94,6 +94,13 @@ void tests() {
       await _runset(_g('prepareQuery.basic'), _utility.prepareQuery);
     });
 
+    // preparePath shipped as an empty `set: []`, so no port drove it and
+    // several kept private hand-written cases instead.
+    test('path-basic', (t) async {
+      await _setup();
+      await _runset(_g('preparePath.basic'), _utility.preparePath);
+    });
+
     test('body-basic', (t) async {
       await _setup();
       await _runset(_g('prepareBody.basic'), (dynamic ctx) {

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

@@ -3,12 +3,17 @@ package feature
 import (
 	"fmt"
 	"math/rand"
+	"regexp"
 
 	vs "github.com/voxgig/struct"
 
 	"GOMODULE/core"
 )
 
+// The `body.<key>` form of an op's response transform: the mock wraps its
+// payload in <key> so the transform can unwrap it again.
+var envelopeResRe = regexp.MustCompile("^`body\\.([^`.]+)`$")
+
 type TestFeature struct {
 	BaseFeature
 	client   *core.ProjectNameSDK
@@ -49,11 +54,38 @@ func (f *TestFeature) Init(ctx *core.Context, options map[string]any) {
 	self := f
 
 	testFetcher := func(ctx *core.Context, _fullurl string, _fetchdef map[string]any) (any, error) {
+		// Shape the mock payload the way the real API would, so the op's
+		// response transform recovers the entity from it. A point carrying
+		// `transform.res: ` + "`body.item`" + ` describes an API that answers
+		// {"item": {...}}; handing back the bare entity means the transform
+		// unwraps a property that is not there and the caller gets nil. The
+		// mock has to agree with the model, or it only ever simulates APIs
+		// whose responses happen to be unwrapped. Mirrors the ts mock.
+		envelope := func(data any) any {
+			if data == nil || ctx.Point == nil {
+				return data
+			}
+			tm, ok := ctx.Point["transform"].(map[string]any)
+			if !ok {
+				return data
+			}
+			restf, ok := tm["res"].(string)
+			if !ok {
+				return data
+			}
+			m := envelopeResRe.FindStringSubmatch(restf)
+			if m == nil {
+				return data
+			}
+			return map[string]any{m[1]: data}
+		}
+
 		respond := func(status int, data any, extra map[string]any) map[string]any {
+			payload := envelope(data)
 			out := map[string]any{
 				"status":     status,
 				"statusText": "OK",
-				"json":       (func() any)(func() any { return data }),
+				"json":       (func() any)(func() any { return payload }),
 				"body":       "not-used",
 			}
 			if extra != nil {