@voxgig/sdkgen 0.45.0 → 1.1.1

package/bin/voxgig-sdkgen

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

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@voxgig/sdkgen",
-  "version": "0.45.0",
+  "version": "1.1.1",
   "main": "dist/sdkgen.js",
   "type": "commonjs",
   "types": "dist/sdkgen.d.ts",
@@ -41,13 +41,13 @@
   ],
   "devDependencies": {
     "@types/js-yaml": "4.0.9",
-    "@types/node": "25.6.0",
+    "@types/node": "25.7.0",
     "json-schema-to-ts": "3.1.1",
     "memfs": "4.57.2",
     "typescript": "6.0.3"
   },
   "peerDependencies": {
-    "@voxgig/apidef": ">=5",
+    "@voxgig/apidef": ">=6",
     "@voxgig/struct": ">=0",
     "@voxgig/util": ">=0",
     "aontu": ">=0",

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

@@ -14,8 +14,8 @@ const Entity = cmp(function Entity(props: any) {
   const { target, entity } = props
 
   // Module name: concatenated lowercase
-  const orgPrefix = (model.origin || '').replace(/-sdk$/, '').replace(/[^a-z0-9]/gi, '')
-  const gomodule = orgPrefix + model.name.replace(/[^a-z0-9]/gi, '').toLowerCase() + 'sdk'
+  // Go module path == repo path on GitHub (org from model.origin).
+  const gomodule = `github.com/${model.origin || 'voxgig-sdk'}/${model.name}-sdk`
 
   const entrep = {
     ...stdrep,

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

@@ -32,9 +32,13 @@ const Main = cmp(async function Main(props: any) {
   const entity: ModelEntity = getModelPath(model, `main.${KIT}.entity`)
   const feature = getModelPath(model, `main.${KIT}.feature`)
 
-  // Module name: concatenated lowercase (e.g., voxgigsolardemosdk)
-  const orgPrefix = (model.origin || '').replace(/-sdk$/, '').replace(/[^a-z0-9]/gi, '')
-  const gomodule = orgPrefix + model.name.replace(/[^a-z0-9]/gi, '').toLowerCase() + 'sdk'
+  // Go module path == the repo path on GitHub (org from model.origin),
+  // e.g. github.com/voxgig-sdk/<slug>-sdk. Used in go.mod and every import.
+  const gomodule = `github.com/${model.origin || 'voxgig-sdk'}/${model.name}-sdk`
+  // The root package name must be a plain Go identifier (can't be a path),
+  // so it stays as the concatenated-lowercase form (e.g. voxgigdogsdk).
+  const gopackage = (model.origin || 'voxgig-sdk').replace(/-sdk$/, '').replace(/[^a-z0-9]/gi, '') +
+    model.name.replace(/[^a-z0-9]/gi, '').toLowerCase() + 'sdk'
 
   Package({ target })
 
@@ -121,7 +125,7 @@ var NewBaseFeatureFunc func() Feature
   const hasEntities = Object.keys(entity || {}).length > 0
   const entityImport = hasEntities ? `\n\t"${gomodule}/entity"` : ''
   File({ name: model.name + '.' + target.ext }, () => {
-    Content(`package ${gomodule}
+    Content(`package ${gopackage}
 
 import (
 	"${gomodule}/core"${entityImport}

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

@@ -19,8 +19,8 @@ const Package = cmp(async function Package(props: any) {
   const model: Model = ctx$.model
 
   // Module name: concatenated lowercase (e.g., voxgigsolardemosdk)
-  const orgPrefix = (model.origin || '').replace(/-sdk$/, '').replace(/[^a-z0-9]/gi, '')
-  const gomodule = orgPrefix + model.name.replace(/[^a-z0-9]/gi, '').toLowerCase() + 'sdk'
+  // Go module path == repo path on GitHub (org from model.origin).
+  const gomodule = `github.com/${model.origin || 'voxgig-sdk'}/${model.name}-sdk`
 
   File({ name: 'go.mod' }, () => {
     Content(`module ${gomodule}

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

@@ -5,8 +5,8 @@ import { cmp, Content } from '@voxgig/sdkgen'
 const ReadmeExplanation = cmp(function ReadmeExplanation(props: any) {
   const { target, ctx$: { model } } = props
 
-  const orgPrefix = (model.origin || '').replace(/-sdk$/, '').replace(/[^a-z0-9]/gi, '')
-  const gomodule = orgPrefix + model.name.replace(/[^a-z0-9]/gi, '').toLowerCase() + 'sdk'
+  // Go module path == repo path on GitHub (org from model.origin).
+  const gomodule = `github.com/${model.origin || 'voxgig-sdk'}/${model.name}-sdk`
 
   Content(`### Data as maps
 

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

@@ -10,8 +10,8 @@ import {
 const ReadmeHowto = cmp(function ReadmeHowto(props: any) {
   const { target, ctx$: { model } } = props
 
-  const orgPrefix = (model.origin || '').replace(/-sdk$/, '').replace(/[^a-z0-9]/gi, '')
-  const gomodule = orgPrefix + model.name.replace(/[^a-z0-9]/gi, '').toLowerCase() + 'sdk'
+  // Go module path == repo path on GitHub (org from model.origin).
+  const gomodule = `github.com/${model.origin || 'voxgig-sdk'}/${model.name}-sdk`
 
   const apikeyEnvLine = isAuthActive(model)
     ? `\n${model.NAME}_APIKEY=<your-key>`

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

@@ -6,8 +6,8 @@ const ReadmeInstall = cmp(function ReadmeInstall(props: any) {
   const { target, ctx$ } = props
   const { model } = ctx$
 
-  const orgPrefix = (model.origin || '').replace(/-sdk$/, '').replace(/[^a-z0-9]/gi, '')
-  const gomodule = orgPrefix + model.name.replace(/[^a-z0-9]/gi, '').toLowerCase() + 'sdk'
+  // Go module path == repo path on GitHub (org from model.origin).
+  const gomodule = `github.com/${model.origin || 'voxgig-sdk'}/${model.name}-sdk`
 
   Content(`\`\`\`bash
 go get ${gomodule}

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

@@ -13,8 +13,8 @@ const ReadmeModel = cmp(function ReadmeModel(props: any) {
   const entity = getModelPath(model, `main.${KIT}.entity`)
   const entityList = each(entity).filter((e: any) => e.active !== false)
 
-  const orgPrefix = (model.origin || '').replace(/-sdk$/, '').replace(/[^a-z0-9]/gi, '')
-  const gomodule = orgPrefix + model.name.replace(/[^a-z0-9]/gi, '').toLowerCase() + 'sdk'
+  // Go module path == repo path on GitHub (org from model.origin).
+  const gomodule = `github.com/${model.origin || 'voxgig-sdk'}/${model.name}-sdk`
 
   const apikeyOptionRow = isAuthActive(model)
     ? '| `"apikey"` | `string` | API key for authentication. |\n'

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

@@ -12,8 +12,8 @@ const ReadmeQuick = cmp(function ReadmeQuick(props: any) {
   const { target, ctx$: { model } } = props
 
   const entity = getModelPath(model, `main.${KIT}.entity`)
-  const orgPrefix = (model.origin || '').replace(/-sdk$/, '').replace(/[^a-z0-9]/gi, '')
-  const gomodule = orgPrefix + model.name.replace(/[^a-z0-9]/gi, '').toLowerCase() + 'sdk'
+  // Go module path == repo path on GitHub (org from model.origin).
+  const gomodule = `github.com/${model.origin || 'voxgig-sdk'}/${model.name}-sdk`
 
   // Find the first published entity for examples
   const exampleEntity = Object.values(entity).find((e: any) => e.active !== false) as any

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

@@ -12,8 +12,8 @@ const ReadmeTopQuick = cmp(function ReadmeTopQuick(props: any) {
   const { target, ctx$: { model } } = props
 
   const entity = getModelPath(model, `main.${KIT}.entity`)
-  const orgPrefix = (model.origin || '').replace(/-sdk$/, '').replace(/[^a-z0-9]/gi, '')
-  const gomodule = orgPrefix + model.name.replace(/[^a-z0-9]/gi, '').toLowerCase() + 'sdk'
+  // Go module path == repo path on GitHub (org from model.origin).
+  const gomodule = `github.com/${model.origin || 'voxgig-sdk'}/${model.name}-sdk`
 
   const exampleEntity = Object.values(entity).find((e: any) => e.active !== false) as any
 

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

@@ -12,8 +12,8 @@ const ReadmeTopTest = cmp(function ReadmeTopTest(props: any) {
   const { target, ctx$: { model } } = props
 
   const entity = getModelPath(model, `main.${KIT}.entity`)
-  const orgPrefix = (model.origin || '').replace(/-sdk$/, '').replace(/[^a-z0-9]/gi, '')
-  const gomodule = orgPrefix + model.name.replace(/[^a-z0-9]/gi, '').toLowerCase() + 'sdk'
+  // Go module path == repo path on GitHub (org from model.origin).
+  const gomodule = `github.com/${model.origin || 'voxgig-sdk'}/${model.name}-sdk`
 
   const exampleEntity = Object.values(entity).find((e: any) => e.active !== false) as any
 

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

@@ -89,13 +89,62 @@ const TestDirect = cmp(function TestDirect(props: any) {
   // Get load point info
   const loadPoint = loadOp?.points?.[0]
   const loadPath = loadPoint ? normalizePathParams(loadPoint.parts || [], loadPoint?.args?.params || [], loadPoint?.rename?.param) : ''
-  const loadParams = loadPoint?.args?.params || []
+  const allLoadParams = loadPoint?.args?.params || []
+  // Some upstream OpenAPI specs declare a parameter as `in: path` even when
+  // that path has no `{name}` placeholder for it. Only path params that
+  // actually appear in the URL template should drive direct-test path-param
+  // setup and URL-substitution asserts; otherwise the SDK silently drops
+  // them and the URL-includes assert fails.
+  const _pathPlaceholders = new Set<string>()
+  for (const part of (loadPoint?.parts || [])) {
+    if (typeof part === 'string' && part.startsWith('{') && part.endsWith('}')) {
+      _pathPlaceholders.add(part.slice(1, -1))
+    }
+  }
+  const _renameMap = (loadPoint?.rename?.param || {}) as Record<string, string>
+  const _renamedPlaceholders = new Set<string>()
+  for (const ph of _pathPlaceholders) {
+    _renamedPlaceholders.add(ph)
+    for (const [orig, renamed] of Object.entries(_renameMap)) {
+      if (renamed === ph) _renamedPlaceholders.add(orig)
+    }
+  }
+  const loadParams = allLoadParams.filter((p: any) =>
+    _renamedPlaceholders.has(p.name) || _renamedPlaceholders.has(p.orig))
 
   // Get list point info
   const listPoint = listOp?.points?.[0]
   const listPath = listPoint ? normalizePathParams(listPoint.parts || [], listPoint?.args?.params || [], listPoint?.rename?.param) : ''
   const listParams = listPoint?.args?.params || []
 
+  // Required query params with spec-provided examples — needed in live mode
+  // to satisfy API contracts (e.g. /v2018/history requires city/start/end).
+  const loadQuery = loadPoint?.args?.query || []
+  const loadLiveQueryEntries = loadQuery
+    .filter((q: any) => q.reqd && undefined !== q.example && null !== q.example)
+  const loadLiveQueryLines = loadLiveQueryEntries
+    .map((q: any) => `\t\t\tquery["${q.name}"] = ${JSON.stringify(q.example)}`)
+    .join('\n')
+
+  const listQuery = listPoint?.args?.query || []
+  const listLiveQueryEntries = listQuery
+    .filter((q: any) => q.reqd && undefined !== q.example && null !== q.example)
+  const listLiveQueryLines = listLiveQueryEntries
+    .map((q: any) => `\t\t\tquery["${q.name}"] = ${JSON.stringify(q.example)}`)
+    .join('\n')
+
+  // Path params with spec-provided examples — when ALL load params have
+  // spec examples, prefer them over list-bootstrap. Spec example values are
+  // by definition real identifiers the API accepts (e.g. casa: "blue",
+  // fecha: "2024/01/01"), avoiding the brittle list-bootstrap path-param
+  // semantic mismatch.
+  const loadAllHaveExamples =
+    loadParams.length > 0 &&
+    loadParams.every((p: any) => undefined !== p.example && null !== p.example)
+  const loadExampleLines = loadAllHaveExamples
+    ? loadParams.map((p: any) => `\t\t\tparams["${p.name}"] = ${JSON.stringify(p.example)}`).join('\n')
+    : ''
+
   // Build the ENTID env var name for this entity
   const entidEnvVar = `${PROJECTNAME}_TEST_${nom(entity, 'NAME').replace(/[^A-Z_]/g, '_')}_ENTID`
 
@@ -131,12 +180,44 @@ func Test${entity.Name}Direct(t *testing.T) {
         return { name: p.name, key }
       })
 
+      // Track idmap keys this test consumes in live mode. If any are
+      // missing (no ENTID override), skip — the request would 4xx on
+      // undefined path params.
+      const listLiveIdKeys = listParams.length > 0
+        ? listLiveParams.map((lp: any) => lp.key)
+        : []
+      const listLiveIdKeysGoLiteral = listLiveIdKeys.length > 0
+        ? `[]string{${listLiveIdKeys.map((k: string) => `"${k}"`).join(', ')}}`
+        : ''
+      const listSkipBlock = listLiveIdKeys.length > 0
+        ? `		if setup.live {
+			for _, _liveKey := range ${listLiveIdKeysGoLiteral} {
+				if v := setup.idmap[_liveKey]; v == nil {
+					t.Skipf("live test needs %s via *_ENTID env var (synthetic IDs only)", _liveKey)
+					return
+				}
+			}
+		}
+`
+        : ''
+
       Content(`	t.Run("direct-list-${entity.name}", func(t *testing.T) {
 		setup := ${entity.name}DirectSetup([]any{
 			map[string]any{"id": "direct01"},
 			map[string]any{"id": "direct02"},
 		})
-		client := setup.client
+		_mode := "unit"
+		if setup.live {
+			_mode = "live"
+		}
+		if _shouldSkip, _reason := isControlSkipped("direct", "direct-list-${entity.name}", _mode); _shouldSkip {
+			if _reason == "" {
+				_reason = "skipped via sdk-test-control.json"
+			}
+			t.Skip(_reason)
+			return
+		}
+${listSkipBlock}		client := setup.client
 
 `)
 
@@ -172,15 +253,30 @@ func Test${entity.Name}Direct(t *testing.T) {
 `)
       }
 
-      Content(`		if err != nil {
-			t.Fatalf("direct failed: %v", err)
-		}
-
-		if result["ok"] != true {
-			t.Fatalf("expected ok to be true, got %v", result["ok"])
-		}
-		if core.ToInt(result["status"]) != 200 {
-			t.Fatalf("expected status 200, got %v", result["status"])
+      Content(`		if setup.live {
+			// Live mode is lenient: synthetic IDs frequently 4xx and the
+			// list-response shape varies wildly across public APIs. Skip
+			// rather than fail when the call doesn't return a usable list.
+			if err != nil {
+				t.Skipf("list call failed (likely synthetic IDs against live API): %v", err)
+			}
+			if result["ok"] != true {
+				t.Skipf("list call not ok (likely synthetic IDs against live API): %v", result)
+			}
+			status := core.ToInt(result["status"])
+			if status < 200 || status >= 300 {
+				t.Skipf("expected 2xx status, got %v", result["status"])
+			}
+		} else {
+			if err != nil {
+				t.Fatalf("direct failed: %v", err)
+			}
+			if result["ok"] != true {
+				t.Fatalf("expected ok to be true, got %v", result["ok"])
+			}
+			if core.ToInt(result["status"]) != 200 {
+				t.Fatalf("expected status 200, got %v", result["status"])
+			}
 		}
 
 		if !setup.live {
@@ -232,22 +328,75 @@ func Test${entity.Name}Direct(t *testing.T) {
       // Identify ancestor params (not 'id') for live mode
       const ancestorParams = loadParams.filter((p: any) => p.name !== 'id')
 
+      // Determine which idmap keys this load test will consume in live mode.
+      // - allHaveExamples: spec provides example values for every load
+      //   path-param, so live mode uses them — no idmap needed.
+      // - hasList: we list-bootstrap, so we need the keys for the list call's
+      //   path params (ancestors of the list path).
+      // - synthetic-only: we'd use idmap for load path-params; without an
+      //   override they're undefined and the live request 4xx's.
+      let loadLiveIdKeys: string[] = []
+      if (loadParams.length > 0 && !loadAllHaveExamples) {
+        if (hasList) {
+          loadLiveIdKeys = listParams.map((p: any) => {
+            return p.name === 'id'
+              ? entity.name + '01'
+              : p.name.replace(/_id$/, '') + '01'
+          })
+        } else {
+          loadLiveIdKeys = loadParams.map((p: any) => p.name + '01')
+        }
+      }
+      const loadSkipBlock = loadLiveIdKeys.length > 0
+        ? `		if setup.live {
+			for _, _liveKey := range []string{${loadLiveIdKeys.map(k => `"${k}"`).join(', ')}} {
+				if v := setup.idmap[_liveKey]; v == nil {
+					t.Skipf("live test needs %s via *_ENTID env var (synthetic IDs only)", _liveKey)
+					return
+				}
+			}
+		}
+`
+        : ''
+
       Content(`	t.Run("direct-load-${entity.name}", func(t *testing.T) {
 		setup := ${entity.name}DirectSetup(map[string]any{"id": "direct01"})
-		client := setup.client
+		_mode := "unit"
+		if setup.live {
+			_mode = "live"
+		}
+		if _shouldSkip, _reason := isControlSkipped("direct", "direct-load-${entity.name}", _mode); _shouldSkip {
+			if _reason == "" {
+				_reason = "skipped via sdk-test-control.json"
+			}
+			t.Skip(_reason)
+			return
+		}
+${loadSkipBlock}		client := setup.client
 
 `)
 
-      if (loadParams.length > 0) {
+      // Always emit a query map so the test can pass it to Direct(); only
+      // set values in live mode.
+      const needsQuery = loadParams.length > 0 || loadLiveQueryLines !== ''
+      if (needsQuery) {
         Content(`		params := map[string]any{}
+		query := map[string]any{}
 `)
 
         Content(`		if setup.live {
 `)
 
-        // In live mode: first list to get a real entity, then use its ID
-        if (hasList) {
-          // Build list params from idmap
+        // Required-query setup (e.g. /v2018/history needs city/start/end).
+        if (loadLiveQueryLines) {
+          Content(loadLiveQueryLines + '\n')
+        }
+
+        if (loadAllHaveExamples) {
+          // Use spec-provided path-param examples — no list bootstrap needed.
+          Content(loadExampleLines + '\n')
+        } else if (hasList && loadParams.length > 0) {
+          // List-bootstrap: first call list, take id from response.
           Content(`			listParams := map[string]any{}
 `)
           for (const p of listParams) {
@@ -264,10 +413,10 @@ func Test${entity.Name}Direct(t *testing.T) {
 				"params": listParams,
 			})
 			if listErr != nil {
-				t.Fatalf("list for load setup failed: %v", listErr)
+				t.Skipf("list call failed (likely synthetic IDs against live API): %v", listErr)
 			}
 			if listResult["ok"] != true {
-				t.Fatalf("list for load setup not ok: %v", listResult)
+				t.Skipf("list call not ok (likely synthetic IDs against live API): %v", listResult)
 			}
 
 			// Get first entity ID from list
@@ -286,11 +435,13 @@ func Test${entity.Name}Direct(t *testing.T) {
           }
         }
 
-        Content(`		} else {
+        if (loadParams.length > 0) {
+          Content(`		} else {
 `)
-        for (let i = 0; i < loadParams.length; i++) {
-          Content(`			params["${loadParams[i].name}"] = "direct0${i + 1}"
+          for (let i = 0; i < loadParams.length; i++) {
+            Content(`			params["${loadParams[i].name}"] = "direct0${i + 1}"
 `)
+          }
         }
         Content(`		}
 `)
@@ -303,24 +454,44 @@ func Test${entity.Name}Direct(t *testing.T) {
 `)
       if (loadParams.length > 0) {
         Content(`			"params": params,
+			"query":  query,
+`)
+      } else if (loadLiveQueryLines) {
+        Content(`			"params": params,
+			"query":  query,
 `)
       } else {
         Content(`			"params": map[string]any{},
 `)
       }
       Content(`		})
-		if err != nil {
-			t.Fatalf("direct failed: %v", err)
-		}
-
-		if result["ok"] != true {
-			t.Fatalf("expected ok to be true, got %v", result["ok"])
-		}
-		if core.ToInt(result["status"]) != 200 {
-			t.Fatalf("expected status 200, got %v", result["status"])
-		}
-		if result["data"] == nil {
-			t.Fatal("expected data to be non-nil")
+		if setup.live {
+			// Live mode is lenient: synthetic IDs frequently 4xx. Skip
+			// rather than fail when the load endpoint isn't reachable with
+			// the IDs we can construct from setup.idmap.
+			if err != nil {
+				t.Skipf("load call failed (likely synthetic IDs against live API): %v", err)
+			}
+			if result["ok"] != true {
+				t.Skipf("load call not ok (likely synthetic IDs against live API): %v", result)
+			}
+			status := core.ToInt(result["status"])
+			if status < 200 || status >= 300 {
+				t.Skipf("expected 2xx status, got %v", result["status"])
+			}
+		} else {
+			if err != nil {
+				t.Fatalf("direct failed: %v", err)
+			}
+			if result["ok"] != true {
+				t.Fatalf("expected ok to be true, got %v", result["ok"])
+			}
+			if core.ToInt(result["status"]) != 200 {
+				t.Fatalf("expected status 200, got %v", result["status"])
+			}
+			if result["data"] == nil {
+				t.Fatal("expected data to be non-nil")
+			}
 		}
 
 		if !setup.live {

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

@@ -100,6 +100,7 @@ import (
 	"os"
 	"path/filepath"
 	"runtime"
+	"strings"
 	"testing"
 	"time"
 
@@ -120,6 +121,27 @@ func Test${entity.Name}Entity(t *testing.T) {
 
 	t.Run("basic", func(t *testing.T) {
 		setup := ${entity.name}BasicSetup(nil)
+		// Per-op sdk-test-control.json skip — basic test exercises a flow
+		// with multiple ops; skipping any op skips the whole flow.
+		_mode := "unit"
+		if setup.live {
+			_mode = "live"
+		}
+		for _, _op := range []string{${(Array.from(new Set((allSteps as any[]).map((s: any) => s.op).filter(Boolean)))).map(o => `"${o}"`).join(', ')}} {
+			if _shouldSkip, _reason := isControlSkipped("entityOp", "${entity.name}." + _op, _mode); _shouldSkip {
+				if _reason == "" {
+					_reason = "skipped via sdk-test-control.json"
+				}
+				t.Skip(_reason)
+				return
+			}
+		}
+		// The basic flow consumes synthetic IDs from the fixture. In live mode
+		// without an *_ENTID env override, those IDs hit the live API and 4xx.
+		if setup.syntheticOnly {
+			t.Skip("live entity test uses synthetic IDs from fixture — set ${PROJUPPER}_TEST_${entity.name.toUpperCase().replace(/[^A-Z_]/g, '_')}_ENTID JSON to run live")
+			return
+		}
 ${allSteps.length > 0 ? '\t\tclient := setup.client\n\n' : ''}`)
 
     // Check if the flow has a create step; if not, bootstrap entity data
@@ -193,6 +215,12 @@ ${allSteps.length > 0 ? '\t\tclient := setup.client\n\n' : ''}`)
     Content('\t)\n')
 
     Content(`
+	// Detect ENTID env override before envOverride consumes it. When live
+	// mode is on without a real override, the basic test runs against synthetic
+	// IDs from the fixture and 4xx's. Surface this so the test can skip.
+	entidEnvRaw := os.Getenv("${PROJUPPER}_TEST_${entity.name.toUpperCase().replace(/[^A-Z_]/g, '_')}_ENTID")
+	idmapOverridden := entidEnvRaw != "" && strings.HasPrefix(strings.TrimSpace(entidEnvRaw), "{")
+
 	env := envOverride(map[string]any{
 		"${PROJUPPER}_TEST_${entity.name.toUpperCase().replace(/[^A-Z_]/g, '_')}_ENTID": idmap,
 		"${PROJUPPER}_TEST_LIVE":      "FALSE",
@@ -224,13 +252,16 @@ ${allSteps.length > 0 ? '\t\tclient := setup.client\n\n' : ''}`)
 		client = sdk.New${model.const.Name}SDK(core.ToMapAny(mergedOpts))
 	}
 
+	live := env["${PROJUPPER}_TEST_LIVE"] == "TRUE"
 	return &entityTestSetup{
-		client:  client,
-		data:    entityData,
-		idmap:   idmapResolved,
-		env:     env,
-		explain: env["${PROJUPPER}_TEST_EXPLAIN"] == "TRUE",
-		now:     time.Now().UnixMilli(),
+		client:        client,
+		data:          entityData,
+		idmap:         idmapResolved,
+		env:           env,
+		explain:       env["${PROJUPPER}_TEST_EXPLAIN"] == "TRUE",
+		live:          live,
+		syntheticOnly: live && !idmapOverridden,
+		now:           time.Now().UnixMilli(),
 	}
 }
 `)

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

@@ -20,8 +20,8 @@ const Test = cmp(function Test(props: any) {
   const { target } = props
 
   // Module name: concatenated lowercase
-  const orgPrefix = (model.origin || '').replace(/-sdk$/, '').replace(/[^a-z0-9]/gi, '')
-  const gomodule = orgPrefix + model.name.replace(/[^a-z0-9]/gi, '').toLowerCase() + 'sdk'
+  // Go module path == repo path on GitHub (org from model.origin).
+  const gomodule = `github.com/${model.origin || 'voxgig-sdk'}/${model.name}-sdk`
 
   Folder({ name: 'test' }, () => {
 

package/project/.sdk/src/cmp/go/fragment/Main.fragment.go

@@ -1,6 +1,8 @@
 package core
 
 import (
+	"fmt"
+
 	vs "github.com/voxgig/struct"
 )
 
@@ -209,17 +211,32 @@ func (sdk *ProjectNameSDK) Direct(fetchargs map[string]any) (map[string]any, err
 
 	if fm, ok := fetched.(map[string]any); ok {
 		status := ToInt(vs.GetProp(fm, "status"))
+		headers := vs.GetProp(fm, "headers")
+
+		// No-body responses (204, 304) and explicit zero content-length
+		// must skip JSON parsing — calling json() on an empty body errors.
+		var contentLength string
+		if hm, ok := headers.(map[string]any); ok {
+			if cl, ok := hm["content-length"]; ok {
+				contentLength = fmt.Sprintf("%v", cl)
+			}
+		}
+		noBody := status == 204 || status == 304 || contentLength == "0"
+
 		var jsonData any
-		if jf := vs.GetProp(fm, "json"); jf != nil {
-			if f, ok := jf.(func() any); ok {
-				jsonData = f()
+		if !noBody {
+			if jf := vs.GetProp(fm, "json"); jf != nil {
+				if f, ok := jf.(func() any); ok {
+					// f() returns nil on parse error in our fetcher.
+					jsonData = f()
+				}
 			}
 		}
 
 		return map[string]any{
 			"ok":      status >= 200 && status < 300,
 			"status":  status,
-			"headers": vs.GetProp(fm, "headers"),
+			"headers": headers,
 			"data":    jsonData,
 		}, nil
 	}

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

@@ -18,11 +18,18 @@ const Package = cmp(async function Package(props: any) {
 
   const model: Model = ctx$.model
 
+  // Rock name is namespaced to model.origin (e.g. "voxgig-sdk"). LuaRocks has
+  // no real namespaces, so the parts are hyphen-joined. The Lua module name
+  // (`${model.name}_sdk`) used by `require` is unchanged.
+  const ns = model.origin || 'voxgig-sdk'
+  const pkgBase = ns.endsWith('-sdk') ? model.name : `${model.name}-sdk`
+  const rockName = `${ns}-${pkgBase}`
+
   File({ name: model.name + '.rockspec' }, () => {
-    Content(`package = "${model.name}-sdk"
+    Content(`package = "${rockName}"
 version = "0.0-1"
 source = {
-  url = "git://github.com/voxgig/${model.name}-sdk.git"
+  url = "git://github.com/${ns}/${model.name}-sdk.git"
 }
 description = {
   summary = "${model.const.Name} SDK for Lua",