@voxgig/sdkgen 4.21.0 → 4.22.0

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

@@ -6,36 +6,6 @@ import {
 } from '@voxgig/apidef'
 
 
-// Emits go/test/readme_examples_test.go — a COMPLETENESS GATE over every
-// ```go block in ALL THREE docs that ship go examples:
-//   - the root README.md (top-level, multi-language quick start)
-//   - the per-language go/README.md
-//   - the per-language go/REFERENCE.md
-//
-// For every ```go block, tagged by (source doc, index), it exercises the
-// block against the real generated SDK:
-//
-//   - Fragments (statement snippets) are wrapped in a function with a SEEDED
-//     test-mode `client` in scope and `go build`-checked. Any stray `import`
-//     line inside the fragment is stripped and constructor calls are rewritten
-//     to the seeded test client (so undefined-variable placeholders like
-//     sdk.TestSDK(testopts, sdkopts) still type-check). A call to a method
-//     that does not exist, a wrong argument count/type, or `.data`/`.ok` field
-//     access on a bare entity-op result fails to compile — and so fails.
-//   - Complete programs (a block with `func main`) are built as-is (the
-//     documented, possibly live, form must compile) AND a test-mode variant —
-//     its constructor rewritten to a seeded `sdk.TestSDK(...)` — is RUN with
-//     `go run`. A genuine runtime panic (nil pointer, nil map, index,
-//     interface conversion) FAILS the test; a tolerated not-found domain
-//     error does not.
-//   - Illustrative blocks are the ONLY blocks skipped, and the class is
-//     NARROW: a bare signature (a `func` line with no body) or a comment-only
-//     / `/* ... */` placeholder. Everything else must compile or run.
-//
-// Completeness: per doc, total == compiled + illustration is asserted. A block
-// that is neither compiled nor a recognized illustration — e.g. real code
-// hidden behind a `/* ... */` comment that the old logic silently skipped —
-// FAILS the gate, so no compilable example can escape the check.
 const ReadmeExamplesTest = cmp(function ReadmeExamplesTest(props: any) {
   const { ctx$: { model } } = props
 

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

@@ -10,7 +10,6 @@ import {
 import { goVarName } from './utility_go'
 
 
-// A type-correct Go literal for a field's canonical type.
 function goLit(type: any): string {
   const k = canonScalarKey(type)
   if ('INTEGER' === k || 'NUMBER' === k) return '1'
@@ -28,7 +27,6 @@ function cap(s: string): string {
 const ReadmeHowto = cmp(function ReadmeHowto(props: any) {
   const { target, ctx$: { model } } = props
 
-  // Go module path == repo path on GitHub (org from model.origin).
   const gomodule = goModule(model, target.name)
 
   const entity = getModelPath(model, `main.${KIT}.entity`)
@@ -40,7 +38,6 @@ const ReadmeHowto = cmp(function ReadmeHowto(props: any) {
   // camelCase Go identifier (never snake_case or flattened lowercase,
   // never a Go keyword).
   const eLower = exampleEntity ? goVarName(exampleEntity.name) : 'entity'
-  // Model-driven id key: null when the entity has no id-like field.
   const idF = exampleEntity ? entityIdField(exampleEntity) : null
   const isMatchOp = 'load' === primaryOp || 'remove' === primaryOp
   let testArg = 'nil'

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

@@ -37,13 +37,6 @@ const ReadmeIntro = cmp(function ReadmeIntro(props: any) {
   const opList = (opNames.length ? opNames : ['list', 'load'])
     .map((o) => '`' + cap(o) + '`').join(', ')
 
-  // Sibling surfaces, MODEL-DRIVEN. This line used to hardcode "the CLI, and
-  // MCP server" — those are the `go-cli` and `go-mcp` targets, and this project
-  // has neither. So both per-language READMEs advertised surfaces nothing here
-  // generates, while the top-level Surfaces table (which does gate on active
-  // targets) listed only ts/ and go/. Name what is actually active, and drop
-  // the sentence entirely when this is the only surface, so it cannot outrun
-  // the model again.
   const targets = getModelPath(model, `main.${KIT}.target`) || {}
   const siblings = Object.entries(targets)
     .filter(([name, t]: any) => name !== target.name && false !== t.active)

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

@@ -33,9 +33,6 @@ const ReadmeModel = cmp(function ReadmeModel(props: any) {
   const opRows = ['load', 'list', 'create', 'update', 'remove']
     .filter((o) => opUnion.has(o)).map((o) => opRowDefs[o]).join('\n')
 
-  // Model-driven Result-shape rows: only describe the operations that
-  // actually exist. Record-returning ops (Load/Create/Update/Remove) share
-  // one row; List has its own — never name a missing op.
   const recordOps = ['load', 'create', 'update', 'remove'].filter((o) => opUnion.has(o))
     .map((o) => '`' + o.charAt(0).toUpperCase() + o.slice(1) + '`')
   const resultRows: string[] = []
@@ -43,7 +40,6 @@ const ReadmeModel = cmp(function ReadmeModel(props: any) {
   if (opUnion.has('list')) resultRows.push('| `List` | a `[]any` of entity records |')
   const resultShapeRows = resultRows.join('\n')
 
-  // Go module path == repo path on GitHub (org from model.origin).
   const gomodule = goModule(model, target.name)
 
   const apikeyOptionRow = isAuthActive(model)
@@ -57,11 +53,7 @@ const ReadmeModel = cmp(function ReadmeModel(props: any) {
   const firstWithOp = entityList.find((e: any) => entityPrimaryOp(e) != null)
   const firstPrimaryOp = firstWithOp ? entityPrimaryOp(firstWithOp) : null
   const firstEntityName = firstWithOp ? ((firstWithOp as any).Name || 'Entity') : 'Entity'
-  // camelCase Go identifier (never snake_case or flattened lowercase,
-  // never a Go keyword).
   const firstEntityVar = goVarName((firstWithOp as any)?.name || 'entity')
-  // Model-driven id key: null when the example entity has no id-like field, so
-  // the Result-shape illustration passes a nil match.
   const firstIdF = firstWithOp ? entityIdField(firstWithOp) : null
   const firstPrimaryMethod = firstPrimaryOp
     ? firstPrimaryOp.charAt(0).toUpperCase() + firstPrimaryOp.slice(1)
@@ -71,8 +63,6 @@ const ReadmeModel = cmp(function ReadmeModel(props: any) {
     ? (firstIdF ? `map[string]any{"${firstIdF}": "example_id"}` : 'nil')
     : 'map[string]any{/* fields */}'
 
-  // The Result-shape call illustration, shown only when some entity exposes an
-  // op. A direct()-only SDK (no entity ops) omits it — there is no op to call.
   const resultCallExample = firstPrimaryOp
     ? `Check \`err\` first, then use the value directly (or the typed
 \`...Typed\` variants, which return the entity's model struct and a typed
@@ -153,7 +143,6 @@ ${resultCallExample}Only \`Direct()\` returns a response envelope — a \`map[st
 
 `)
 
-  // Entities summary
   Content(`### Entities
 
 `)

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

@@ -10,16 +10,10 @@ import {
 import { exampleValue, goVarName } from './utility_go'
 
 
-// Emits the go/README.md Quickstart as ONE complete, compilable program.
-// Every entity operation returns `(value, error)` where `value` is the
-// data itself — NOT a `{ok, data, ...}` envelope (only Direct returns
-// that). So examples check `err` and use the value directly. The whole
-// program is compiled by the README snippet test (readme_examples_test.go).
 const ReadmeQuick = cmp(function ReadmeQuick(props: any) {
   const { target, ctx$: { model } } = props
 
   const entity = getModelPath(model, `main.${KIT}.entity`)
-  // Go module path == repo path on GitHub (org from model.origin).
   const gomodule = goModule(model, target.name)
 
   const exampleEntity = Object.values(entity).find((e: any) => e.active !== false) as any
@@ -42,20 +36,12 @@ const ReadmeQuick = cmp(function ReadmeQuick(props: any) {
       ? `sdk.New()`
       : `sdk.New${model.const.Name}SDK(map[string]any{${goServerField}\n    })`)
 
-  // Build the body of main() from the operations the example entity
-  // supports. Each op names a fresh value var, so `:=` always declares a
-  // new variable (reusing `err`), and every var is used (printed).
   const body: string[] = []
   let usesFmt = false
 
   if (exampleEntity) {
     const eName = nom(exampleEntity, 'Name')
-    // camelCase variable name — a `status_embed_config` entity must not bind
-    // a snake_case Go variable, and a `Type`/`Range` entity must not bind a
-    // Go keyword (`type, err := ...` fails `go build`).
     const eLower = goVarName(exampleEntity.name)
-    // ACTIVE ops only — an inactive op generates no method, so an example
-    // calling it would not compile.
     const opnames = entityOps(exampleEntity)
 
     // Model-driven id key: null when the entity has no id-like field (a
@@ -63,12 +49,6 @@ const ReadmeQuick = cmp(function ReadmeQuick(props: any) {
     // update omits the id member.
     const idF = entityIdField(exampleEntity)
 
-    // Model-driven example members for an op body, from the SAME op shape the
-    // request types are built from (opRequestShape), so create/update
-    // reference REAL writable fields, not a hardcoded "name", and every value
-    // is a type-correct Go literal (exampleValue). ids are rendered
-    // separately as the match key for update/remove; a REQUIRED id stays for
-    // create (dropping it makes the payload incomplete).
     const exampleFields = (opname: string): string[] => {
       const items = opRequestShape(exampleEntity, opname).items
         .filter((it: any) => (it.name !== idF && it.name !== 'id') ||
@@ -162,7 +142,6 @@ const ReadmeQuick = cmp(function ReadmeQuick(props: any) {
     }
   }
 
-  // Drop trailing blank lines from the body.
   while (body.length > 0 && body[body.length - 1] === '') {
     body.pop()
   }

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

@@ -1,5 +1,5 @@
 
-import { cmp, each, Content, canonToType, File, isAuthActive, entityIdField, entityOps, opRequestShape } from '@voxgig/sdkgen'
+import { cmp, each, Content, canonToType, File, isAuthActive, entityIdField, entityOps, opRequestShape , targetFeatures } from '@voxgig/sdkgen'
 import { ReadmeRefFeatures } from '@voxgig/sdkgen'
 
 import {
@@ -44,7 +44,7 @@ const ReadmeRef = cmp(function ReadmeRef(props: any) {
   const { model } = props.ctx$
 
   const entity = getModelPath(model, `main.${KIT}.entity`)
-  const feature = getModelPath(model, `main.${KIT}.feature`)
+  const feature = targetFeatures(model, target)
 
   const publishedEntities = each(entity).filter((e: any) => e.active !== false)
 
@@ -113,7 +113,6 @@ client := sdk.TestSDK(testopts, sdkopts)
 `)
 
 
-    // Entity factory methods
     publishedEntities.map((ent: any) => {
       Content(`#### \`${ent.Name}(data map[string]any) ${model.const.Name}Entity\`
 
@@ -159,7 +158,6 @@ same parameters as \`Direct()\`.
 `)
 
 
-    // Entity reference sections
     publishedEntities.map((ent: any) => {
       // ACTIVE ops only — an inactive op generates no method, so an example
       // calling it would not compile.
@@ -168,8 +166,6 @@ same parameters as \`Direct()\`.
       // Model-driven id key: null when this entity has no id-like field, in
       // which case load/remove pass a nil match and update omits the id.
       const idF = entityIdField(ent)
-      // camelCase Go identifier (a `status_embed_config` entity must not bind
-      // a snake_case Go variable).
       const eVar = goVarName(ent.name)
 
       Content(`
@@ -193,7 +189,6 @@ fmt.Println(${eVar}.GetName()) // "${ent.name}"
 `)
 
 
-      // Field schema
       if (fields.length > 0) {
         Content(`### Fields
 
@@ -210,7 +205,6 @@ fmt.Println(${eVar}.GetName()) // "${ent.name}"
         Content(`
 `)
 
-        // Field operations breakdown
         const hasFieldOps = fields.some((f: any) => f.op && Object.keys(f.op).length > 0)
         if (hasFieldOps) {
           // Only emit columns for operations this entity actually exposes —
@@ -241,7 +235,6 @@ fmt.Println(${eVar}.GetName()) // "${ent.name}"
       }
 
 
-      // Operation details
       if (opnames.length > 0) {
         Content(`### Operations
 
@@ -346,7 +339,6 @@ fmt.Println(result)
       }
 
 
-      // Common methods
       Content(`### Common Methods
 
 #### \`Data(args ...any) any\`
@@ -372,7 +364,6 @@ Return the entity name.
     })
 
 
-    // Features section
     const activeFeatures = each(feature).filter((f: any) => f.active)
     if (activeFeatures.length > 0) {
       Content(`
@@ -408,9 +399,6 @@ client := sdk.New${model.const.Name}SDK(map[string]any{
 \`\`\`
 
 `)
-      // The shared feature reference: options, defaults, usage and the
-      // considerations. Model facts, identical in every target, so they are
-      // written once in cmp/ReadmeRefFeatures.ts rather than here.
       ReadmeRefFeatures({ target })
     }
 

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

@@ -14,7 +14,6 @@ const ReadmeTopQuick = cmp(function ReadmeTopQuick(props: any) {
   const { target, ctx$: { model } } = props
 
   const entity = getModelPath(model, `main.${KIT}.entity`)
-  // Go module path == repo path on GitHub (org from model.origin).
   const gomodule = goModule(model, target.name)
 
   const exampleEntity = Object.values(entity).find((e: any) => e.active !== false) as any
@@ -46,10 +45,7 @@ client := ${ctor}
 
   if (exampleEntity) {
     const eName = nom(exampleEntity, 'Name')
-    // camelCase Go identifier (never snake_case, never a Go keyword).
     const eVar = goVarName(exampleEntity.name)
-    // ACTIVE ops only — an inactive op generates no method, so an example
-    // calling it would not compile.
     const opnames = entityOps(exampleEntity)
 
     let hasCall = false
@@ -65,9 +61,6 @@ fmt.Println(${eVar}s)
       hasCall = true
     }
 
-    // Find a nested entity for a more interesting example: one with a parent
-    // chain (relations.ancestors), an active load op, and a required non-id
-    // load param to demonstrate (the parent key, e.g. page_id).
     const nestedEntity = Object.values(entity).find((e: any) =>
       e.active !== false &&
       e.relations && e.relations.ancestors && 0 < e.relations.ancestors.length &&
@@ -106,9 +99,7 @@ fmt.Println(${neVar})
       hasCall = true
     }
 
-    // Fallback: APIs with only `load` (no list, no nested) — still show one call.
     if (!hasCall && opnames.includes('load')) {
-      // Every REQUIRED load-match key (id first) — nil when there are none.
       const idF = entityIdField(exampleEntity)
       const loadItems = opRequestShape(exampleEntity, 'load').items
         .filter((it: any) => !it.optional || it.name === idF)

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

@@ -17,27 +17,6 @@ import {
 } from './utility_go'
 
 
-// THE GENERATED SCHEMA MODULE: the model's schemas, as data the SDK can run.
-//
-// The go peer of src/cmp/ts/Schema_ts.ts. Same two exports, same source:
-// OPTSPEC from `main.kit.optspec` plus each feature's own `config.options`,
-// ENTITYSPEC from the entity field sentinels — both built by the shared
-// helpers, so what go validates against and what ts validates against cannot
-// drift.
-//
-// EMBEDDED AS JSON, PARSED AT LOAD, where ts emits an object literal. JSON is
-// a subset of TypeScript's own literal syntax and is not a subset of go's, so
-// the literal ts writes directly is a `map[string]any{...}` tree here — which
-// this could render (utility_go has the formatter config.go's literal branch
-// uses), but the JSON string is the smaller, simpler artefact and every
-// target this is being ported to already carries one for its config.
-//
-// The round-trip is exact because the spec holds only strings and booleans:
-// pinned by "strings and booleans only, so the JSON round-trip is lossless"
-// in ts/test/optspec.test.ts. That is not incidental. json.Unmarshal decodes
-// EVERY JSON number as float64, and struct reads a spec by example, so one
-// number in here would mean something different in go than in ts — the same
-// disagreement configNormalise exists to repair in config.go.
 const Schema = cmp(async function Schema(props: any) {
   const ctx$ = props.ctx$
   const target = props.target

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

@@ -107,7 +107,6 @@ const TestDirect = cmp(function TestDirect(props: any) {
   const loadOp = entity.op?.load
   const listOp = entity.op?.list
 
-  // Get load point info
   const loadPoint = loadOp?.points?.[0]
   const loadIsGraphql = 'graphql' === (loadPoint as any)?.kind
   const loadPath = loadPoint ? normalizePathParams(pointParts(loadPoint), loadPoint?.args?.params || [], loadPoint?.rename?.param) : ''
@@ -134,14 +133,11 @@ const TestDirect = cmp(function TestDirect(props: any) {
   const loadParams = allLoadParams.filter((p: any) =>
     _renamedPlaceholders.has(p.name) || _renamedPlaceholders.has(p.orig))
 
-  // Get list point info
   const listPoint = listOp?.points?.[0]
   const listIsGraphql = 'graphql' === (listPoint as any)?.kind
   const listPath = listPoint ? normalizePathParams(pointParts(listPoint), 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)
@@ -168,7 +164,6 @@ const TestDirect = cmp(function TestDirect(props: any) {
     ? 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_${envToken(entity.name)}_ENTID`
 
   File({ name: entity.name + '_direct_test.' + target.ext }, () => {
@@ -188,7 +183,6 @@ import (
 func Test${entity.Name}Direct(t *testing.T) {
 `)
 
-    // Generate list test first (load needs list results in live mode)
     if (hasList && listPoint && listIsGraphql) {
       generateDirectGraphqlGo('list', entity, listPoint, liveFail)
     }
@@ -199,7 +193,6 @@ func Test${entity.Name}Direct(t *testing.T) {
           `"${p.name}": "direct0${i + 1}"`).join(', ')
         : ''
 
-      // Build live params for list
       const listLiveParams = listParams.map((p: any) => {
         const key = p.name === 'id'
           ? entity.name + '01'
@@ -252,9 +245,6 @@ ${listSkipBlock}		client := setup.client
         Content(`		params := map[string]any{}
 `)
         listLiveParams.forEach((lp: any, i: number) => {
-          // Each path param gets its own placeholder ("direct01", "direct02", ...)
-          // in non-live mode so URL-shape assertions can verify the params
-          // landed in distinct positions in the URL.
           const placeholder = 'direct0' + (i + 1)
           Content(`		if setup.live {
 			params["${lp.name}"] = setup.idmap["${lp.key}"]
@@ -357,16 +347,8 @@ ${listSkipBlock}		client := setup.client
           `"${p.name}": "direct0${i + 1}"`).join(', ')
         : ''
 
-      // 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) {
@@ -419,16 +401,13 @@ ${loadSkipBlock}		client := setup.client
         Content(`		if setup.live {
 `)
 
-        // 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) {
@@ -459,7 +438,6 @@ ${loadSkipBlock}		client := setup.client
 			firstEnt := core.ToMapAny(listData[0])
 			params["id"] = firstEnt["id"]
 `)
-          // Set ancestor params from idmap
           for (const p of ancestorParams) {
             const key = p.name.replace(/_id$/, '') + '01'
             Content(`			params["${p.name}"] = setup.idmap["${key}"]
@@ -632,7 +610,6 @@ func ${entity.name}DirectSetup(mockres any) *${entity.name}DirectSetupResult {
 }
 `)
 
-    // Suppress unused import warnings
     Content(`
 var _ = os.Getenv
 var _ = json.Unmarshal

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

@@ -36,8 +36,6 @@ function goVar(name: string): string {
 }
 
 
-// Go's GenCtx mirrors the shared shape (see TestEntity_ts.ts) plus a
-// `gomodule` slot used to build qualified package paths in emitted code.
 type GenCtx = {
   model: Model
   entity: ModelEntity
@@ -281,12 +279,6 @@ ${allSteps.length > 0 ? '\t\tclient := setup.client\n\n' : ''}`)
 
 `)
 
-    // Generate idmap via vs.Transform.
-    //
-    // The error return Transform gained in struct go 0.1.3 is discarded
-    // deliberately: this transforms the entity's own id names, which the
-    // generator produced, so an error would be a generator bug rather than
-    // anything the generated test can act on.
     Content('\t// Generate idmap via transform, matching TS pattern.\n')
     Content('\tidmap, _ := vs.Transform(\n')
     Content('\t\t[]any{' + idnamesStr + '},\n')

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

@@ -50,10 +50,6 @@ func NewEntyClass(client *core.ProjectNameSDK, entopts map[string]any) *EntyClas
 
 func (e *EntyClass) GetName() string { return e.name }
 
-// Deleted marks this instance as removed. `Remove` resolves to the entity
-// like every other operation, and the instance KEEPS the data it held — a
-// caller can still read what was deleted — but it is no longer a live
-// record. See AGENTS.md "Entity operations return ENTITIES".
 func (e *EntyClass) MarkDeleted() {
 	e.deleted = true
 }
@@ -122,15 +118,6 @@ func (e *EntyClass) MatchTyped(match ...EntityName) EntityName {
 	return typedFrom[EntityName](e.Match())
 }
 
-// Stream (feature #4). Runs `action` through the full pipeline and returns a
-// channel over result items, so the `streaming` feature's incremental output
-// is reachable from a generated entity (a normal op call materialises the
-// whole result). `callopts` parameterises the call:
-//   - inbound (download): the channel yields items/chunks (from the streaming
-//     feature when active, else the materialised items);
-//   - outbound (upload): a `body` in callopts is attached to the request so the
-//     transport can stream the payload;
-//   - `ctrl` (pipeline control) and `signal` (a done channel) are honoured.
 func (e *EntyClass) Stream(action string, args map[string]any, callopts map[string]any) <-chan any {
 	out := make(chan any)
 
@@ -317,14 +304,6 @@ func (e *EntyClass) runOp(ctx *core.Context, postDone func()) (any, error) {
 		return out, doneErr
 	}
 
-	// An operation resolves to the ENTITY, not the raw data. Entities are
-	// stateful: post_done has just absorbed resdata/resmatch into this
-	// instance, and the caller reaches the record through data(). Two
-	// structural exceptions: `list` resolves to the ARRAY of entity
-	// instances make_result built, and a failed op with throwing disabled
-	// hands back the error payload unchanged. `remove` additionally marks
-	// the entity deleted; it KEEPS its data, so a caller can still read
-	// what was removed. See AGENTS.md "Entity operations return ENTITIES".
 	opname := ""
 	if ctx.Op != nil {
 		opname = ctx.Op.Name

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

@@ -264,7 +264,6 @@ func (sdk *ProjectNameSDK) rawRequest(fetchargs map[string]any) (map[string]any,
 		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()
 				}
 			}
@@ -281,17 +280,6 @@ func (sdk *ProjectNameSDK) rawRequest(fetchargs map[string]any) (map[string]any,
 	return map[string]any{"ok": false, "err": ctx.MakeError("direct_invalid", "invalid response type")}, nil
 }
 
-// Raw GraphQL access: the pressure valve that makes the generated surface's
-// deliberate omissions (per-call selection sets, typed filter builders,
-// batching, subscriptions) livable — the whole schema stays reachable.
-//
-// Thin wrapper over the same prepare/fetch path Direct uses, with the one
-// thing raw Direct cannot do for GraphQL: a GraphQL failure rides HTTP 200
-// as a top-level `errors` array, so status alone would report a failed query
-// as ok.
-//
-// NOTE: like Direct, this bypasses the feature pipeline — no retry,
-// ratelimit or paging features apply.
 func (sdk *ProjectNameSDK) Graphql(
 	query string, variables map[string]any, ctrl map[string]any,
 ) (map[string]any, error) {

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

@@ -23,13 +23,6 @@ function projectPath(suffix?: string): string {
 }
 
 
-// --- Model-driven example literals -----------------------------------------
-// Doc snippets must use example values whose TYPE matches the model's
-// declared param/field type — a number id shown as a quoted string documents
-// a call the API would reject. These helpers derive the example literal from
-// the SAME model source the request shapes are built from (opRequestShape),
-// so the docs and the model can never disagree. Go twin of the ts
-// exampleValue (cmp/ts/utility_ts.ts), rendering Go literals.
 
 // The declared canon-type sentinel of a named parameter of an op — looked up
 // in the op's `points[].args.params[]` exactly as the typed-model generator
@@ -37,21 +30,6 @@ function projectPath(suffix?: string): string {
 // has no params and the request shape mirrors the entity fields). Returns
 // undefined when neither is present.
 function paramCanonType(entity: any, op: any, paramName: string): unknown {
-  // opParams, NOT a raw walk of op.points.
-  //
-  // opParams drops points flagged with select['$action'] and merges what is
-  // left; the typed-model generator reaches the op's params through it, so
-  // anything else is a DIFFERENT set of params wearing the same name.
-  //
-  // A raw walk returned the first match on ANY point, action points included.
-  // github's `action` entity has a field `owner` (`$OBJECT`, "A GitHub user")
-  // and seven create points whose path carries `{owner}` (a string) — so the
-  // type said Record<string, any> and the doc example said 'example_owner',
-  // and ts/README.md stopped compiling. Same for workflow_id: `number` in the
-  // type, quoted string in the example.
-  //
-  // Deriving from opParams is what actually makes good on the promise below —
-  // that the docs and the generated types cannot disagree.
   const params = op ? each(opParams(op)) : []
   const found = (params as any[]).find((p: any) => p && p.name === paramName)
   if (found) {
@@ -94,13 +72,8 @@ function exampleValue(entity: any, op: any, paramName: string, placeholder: stri
 }
 
 
-// A camelCase Go identifier for a snake_case model name
-// (`status_embed_config` -> `statusEmbedConfig`) — Go variables are
-// camelCase, never snake_case — with the reserved-word guard applied (a
-// `type`/`range` entity must not bind a Go keyword).
 function goVarName(name: string): string {
   const pascal = camelify(name)
-  // exampleVarName also guards `client`, the doc examples' SDK-instance var.
   return exampleVarName(pascal.charAt(0).toLowerCase() + pascal.slice(1), 'go')
 }
 
@@ -136,8 +109,6 @@ function formatGoMap(obj: any, indent: number = 0): string {
 }
 
 
-// Go rejects a literal BOM anywhere after the start of a source file,
-// including inside strings. Preserve its value through an escaped literal.
 function formatGoString(value: string): string {
   return JSON.stringify(value).replace(/\uFEFF/g, '\\ufeff')
 }
@@ -169,41 +140,14 @@ function formatGoValue(val: any, indent: number = 0): string {
 }
 
 
-// Emission-time normalisation of a model subtree (L0).
-//
-// Always drops jostraca's iteration metadata (`$`-suffixed keys: index$,
-// key$, val$). With `dropDefaults`, also drops keys whose value IS the
-// default the runtime already assumes when the key is absent, which is pure
-// payload — see CONFIG_DEFAULT.
-//
-// Rebuilds the tree rather than mutating during a walk. The previous
-// implementation walked a clone calling `delete p[k]`, but walk() assigns its
-// callback's result back over the child (`setprop(out, ckey, walk(...))`), so
-// the delete was undone on the way out and the helper silently did nothing.
-// Returning `undefined` from the callback does not fix it either: setprop
-// stores undefined rather than removing the key, which then emits as a null.
-//
-// `dropDefaults` is opt-in and must be passed ONLY for the entity subtree.
-// `active` means something different in feature config, where absent reads as
-// INACTIVE (see feature_init) — dropping `active: true` there would silently
-// disable the feature.
-// jostraca's iteration metadata, injected by each()/names() while it walks the
-// model. Listed explicitly rather than matched by trailing-dollar suffix: a
-// trailing dollar is not exclusive to jostraca -- Seneca uses entity$ as real
-// data -- so a blanket suffix match can silently drop a legitimate API field.
 const MODEL_META = ['index$', 'key$', 'val$']
 
-// Keys whose value IS the default the runtime already assumes when the key is
-// absent, so emitting them is pure payload.
 const CONFIG_DEFAULT: Record<string, any> = {
   active: true,
   req: false,
   reqd: false,
 }
 
-// Subtrees carrying user payload rather than schema. An active:true inside an
-// OpenAPI example is DATA, not a default, so default-pruning stops at these
-// keys and everything below them is passed through untouched.
 const PAYLOAD_KEYS = ['default', 'example', 'examples']
 
 function clean(o: any, dropDefaults?: boolean): any {
@@ -231,20 +175,6 @@ function clean(o: any, dropDefaults?: boolean): any {
 
 
 
-// The Go identifier fragment for a feature's generated constructor
-// (New<Fname>Feature / New<Fname>FeatureFunc).
-//
-// SHARED because the identifier is DECLARED in Main_go.ts (registry.go and the
-// root init()) and REFERENCED in Config_go.ts (makeFeature). Two copies of the
-// derivation is a latent undefined-identifier bug in the generated SDK: they
-// were both `name.charAt(0).toUpperCase() + name.slice(1)`, consistently wrong
-// for a name needing real normalisation but at least agreeing, until one side
-// was fixed alone and `rate_limit` became NewRateLimitFeatureFunc in the
-// registry and NewRate_limitFeatureFunc in config.
-//
-// Uses the jostraca-derived PascalCase `Name` (deriving it if absent), which is
-// what Main_ts.ts uses, so a hyphenated or underscored feature name yields a
-// legal Go identifier instead of `NewRate-limitFeatureFunc`.
 function goFeatureName(feat: any): string {
   if (null == feat.Name) {
     names(feat, feat.name)