@voxgig/sdkgen 4.21.0 → 4.23.0

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

@@ -1,3 +1,4 @@
+import { flowSteps } from '@voxgig/sdkgen'
 
 import {
   flatten,
@@ -36,8 +37,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
@@ -97,10 +96,10 @@ const TestEntity = cmp(function TestEntity(props: any) {
   const idnamesStr = idnames.map(n => `"${n}"`).join(', ')
 
   // Get all update data entries for alias generation
-  const allSteps = Object.values(basicflow.step) as any[]
-  const updateStep = allSteps.find((s: any) => s.op === 'update')
-  const updateData = updateStep?.data
-    ? Object.entries(updateStep.data).filter(([k]: any) => k !== 'id' && !k.endsWith('$'))
+  const allSteps = Object.values(flowSteps(basicflow)) as any[]
+  const updateStep = allSteps.find((s: any) => s.o === 'update')
+  const updateData = updateStep?.d
+    ? Object.entries(updateStep.d).filter(([k]: any) => k !== 'id' && !k.endsWith('$'))
     : []
   const aliases = updateData.map(([k, v]: any) => [k, v])
 
@@ -109,10 +108,10 @@ const TestEntity = cmp(function TestEntity(props: any) {
   // fmt is only used by the TextFieldMark Update branch — omit the import
   // when no step needs it, otherwise Go's strict unused-import check fails.
   const needsFmt = allSteps.some((s: any) =>
-    s.op === 'update' &&
-    s.input.textfield &&
-    Array.isArray(s.spec) &&
-    s.spec.some((sp: any) => sp.apply === 'TextFieldMark'))
+    s.o === 'update' &&
+    s.i.textfield &&
+    Array.isArray(s.s) &&
+    s.s.some((sp: any) => sp.apply === 'TextFieldMark'))
 
   File({ name: entity.name + '_entity_test.' + target.ext }, () => {
 
@@ -198,7 +197,7 @@ ${hasList ? `
 		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(', ')}} {
+		for _, _op := range []string{${(Array.from(new Set((allSteps as any[]).map((s: any) => s.o).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"
@@ -216,7 +215,7 @@ ${hasList ? `
 ${allSteps.length > 0 ? '\t\tclient := setup.client\n\n' : ''}`)
 
     // Check if the flow has a create step; if not, bootstrap entity data
-    const flowHasCreate = allSteps.some((s: any) => s.op === 'create')
+    const flowHasCreate = allSteps.some((s: any) => s.o === 'create')
     if (!flowHasCreate) {
       const preambleRef = entity.name + '_ref01'
       const preambleVar = goVar(preambleRef)
@@ -234,16 +233,16 @@ ${allSteps.length > 0 ? '\t\tclient := setup.client\n\n' : ''}`)
     }
 
     // Model-driven step iteration
-    each(basicflow.step, (step: any, index: any) => {
+    each(flowSteps(basicflow), (step: any, index: any) => {
       // Never emit a REMOVE (or its removed-item verify LIST) without a
       // preceding CREATE: a coherent CRUD flow only removes what it created,
       // so a create-less remove would mutate pre-existing (live) data.
       if (!flowHasCreate) {
-        if ('remove' === step.op) { return }
-        if ('list' === step.op &&
-          (step.valid || []).some((v: any) => 'ItemNotExists' === v.apply)) { return }
+        if ('remove' === step.o) { return }
+        if ('list' === step.o &&
+          (step.v || []).some((v: any) => 'ItemNotExists' === v.apply)) { return }
       }
-      const opgen: OpGen = GENERATE_OP[step.op]
+      const opgen: OpGen = GENERATE_OP[step.o]
       if (opgen) {
         opgen(genCtx, step, index)
         Content('\n')
@@ -281,12 +280,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')
@@ -367,18 +360,18 @@ ${allSteps.length > 0 ? '\t\tclient := setup.client\n\n' : ''}`)
 
 const generateCreate: OpGen = (ctx, step, index) => {
   const { entity, flow } = ctx
-  const ref = step.input.ref ?? entity.name + '_ref01'
-  const entvar = goVar(step.input.entvar ?? ref + '_ent')
-  const datavar = goVar(step.input.datavar ?? (ref + '_data' + (step.input.suffix ?? '')))
+  const ref = step.i.ref ?? entity.name + '_ref01'
+  const entvar = goVar(step.i.entvar ?? ref + '_ent')
+  const datavar = goVar(step.i.datavar ?? (ref + '_data' + (step.i.suffix ?? '')))
 
-  const priorSteps = Object.values(flow.step).slice(0, Number(index)) as any[]
+  const priorSteps = Object.values(flowSteps(flow)).slice(0, Number(index)) as any[]
   const needsEnt = !priorSteps.some((s: any) =>
-    ['create', 'list', 'load', 'update', 'remove'].includes(s.op))
+    ['create', 'list', 'load', 'update', 'remove'].includes(s.o))
 
   const hasDatvar = priorSteps.some((s: any) => {
-    if ('create' === s.op) {
-      const priorRef = s.input.ref ?? entity.name + '_ref01'
-      const priorDatvar = goVar(s.input.datavar ?? (priorRef + '_data' + (s.input.suffix ?? '')))
+    if ('create' === s.o) {
+      const priorRef = s.i.ref ?? entity.name + '_ref01'
+      const priorDatvar = goVar(s.i.datavar ?? (priorRef + '_data' + (s.i.suffix ?? '')))
       return priorDatvar === datavar
     }
     return false
@@ -431,14 +424,14 @@ const generateCreate: OpGen = (ctx, step, index) => {
 const generateList: OpGen = (ctx, step, index) => {
   const { entity, flow } = ctx
   const hasDataId = null != entityDataIdField(entity)
-  const ref = step.input.ref ?? entity.name + '_ref01'
-  const entvar = goVar(step.input.entvar ?? ref + '_ent')
-  const matchvar = goVar(step.input.matchvar ?? (ref + '_match' + (step.input.suffix ?? '')))
-  const listvar = goVar(step.input.listvar ?? (ref + '_list' + (step.input.suffix ?? '')))
+  const ref = step.i.ref ?? entity.name + '_ref01'
+  const entvar = goVar(step.i.entvar ?? ref + '_ent')
+  const matchvar = goVar(step.i.matchvar ?? (ref + '_match' + (step.i.suffix ?? '')))
+  const listvar = goVar(step.i.listvar ?? (ref + '_list' + (step.i.suffix ?? '')))
 
-  const priorSteps = Object.values(flow.step).slice(0, Number(index)) as any[]
+  const priorSteps = Object.values(flowSteps(flow)).slice(0, Number(index)) as any[]
   const needsEnt = !priorSteps.some((s: any) =>
-    ['create', 'list', 'load', 'update', 'remove'].includes(s.op))
+    ['create', 'list', 'load', 'update', 'remove'].includes(s.o))
 
   Content(`		// LIST
 `)
@@ -465,14 +458,14 @@ const generateList: OpGen = (ctx, step, index) => {
 
   // Only declare ${listvar} as a real var when a downstream validator
   // actually uses it; otherwise `_` to satisfy Go's unused-var check.
-  const allSteps = Object.values(flow.step) as any[]
+  const allSteps = Object.values(flowSteps(flow)) as any[]
   // hasDataId: the ItemExists/ItemNotExists asserts (the only listvar users) are
   // now gated on a real DATA id field, so without one the list result is unused.
-  const listvarUsed = hasDataId && !!step.valid?.some((v: any) => {
+  const listvarUsed = hasDataId && !!step.v?.some((v: any) => {
     if ('ItemExists' !== v.apply && 'ItemNotExists' !== v.apply) return false
     const validRef = v.def?.ref
-    return validRef && allSteps.some((s: any) => 'create' === s.op &&
-      ((s.input.ref ?? entity.name + '_ref01') === validRef))
+    return validRef && allSteps.some((s: any) => 'create' === s.o &&
+      ((s.i.ref ?? entity.name + '_ref01') === validRef))
   })
   const listvarBind = listvarUsed ? listvar : '_'
 
@@ -490,12 +483,12 @@ const generateList: OpGen = (ctx, step, index) => {
 		}
 `)
 
-  // Handle validators from step.valid
-  if (step.valid) {
-    for (const validator of step.valid) {
+  // Handle validators from step.v
+  if (step.v) {
+    for (const validator of step.v) {
       const validRef = validator.def?.ref
-      const hasRefData = validRef && allSteps.some((s: any) => 'create' === s.op &&
-        ((s.input.ref ?? entity.name + '_ref01') === validRef))
+      const hasRefData = validRef && allSteps.some((s: any) => 'create' === s.o &&
+        ((s.i.ref ?? entity.name + '_ref01') === validRef))
 
       if ('ItemExists' === validator.apply && hasRefData && hasDataId) {
         const refDataVar = goVar(validRef + '_data')
@@ -521,16 +514,16 @@ const generateList: OpGen = (ctx, step, index) => {
 
 const generateUpdate: OpGen = (ctx, step, index) => {
   const { entity, flow } = ctx
-  const ref = step.input.ref ?? entity.name + '_ref01'
-  const entvar = goVar(step.input.entvar ?? ref + '_ent')
-  const datavar = goVar(step.input.datavar ?? (ref + '_data' + (step.input.suffix ?? '')))
-  const resdatavar = goVar(step.input.resdatavar ?? (ref + '_resdata' + (step.input.suffix ?? '')))
-  const markdefvar = goVar(step.input.markdefvar ?? (ref + '_markdef' + (step.input.suffix ?? '')))
-  const srcdatavar = goVar(step.input.srcdatavar ?? (ref + '_data' + (step.input.suffix ?? '')))
-
-  const priorSteps = Object.values(flow.step).slice(0, Number(index)) as any[]
+  const ref = step.i.ref ?? entity.name + '_ref01'
+  const entvar = goVar(step.i.entvar ?? ref + '_ent')
+  const datavar = goVar(step.i.datavar ?? (ref + '_data' + (step.i.suffix ?? '')))
+  const resdatavar = goVar(step.i.resdatavar ?? (ref + '_resdata' + (step.i.suffix ?? '')))
+  const markdefvar = goVar(step.i.markdefvar ?? (ref + '_markdef' + (step.i.suffix ?? '')))
+  const srcdatavar = goVar(step.i.srcdatavar ?? (ref + '_data' + (step.i.suffix ?? '')))
+
+  const priorSteps = Object.values(flowSteps(flow)).slice(0, Number(index)) as any[]
   const needsEnt = !priorSteps.some((s: any) =>
-    ['create', 'list', 'load', 'update', 'remove'].includes(s.op))
+    ['create', 'list', 'load', 'update', 'remove'].includes(s.o))
 
   const hasEntIdU = null != entity.id
 
@@ -547,9 +540,9 @@ const generateUpdate: OpGen = (ctx, step, index) => {
 `)
   }
 
-  // Add data entries from step.data
-  if (step.data) {
-    const dataEntries = Object.entries(step.data).filter(([k]: any) => k !== 'id' && !k.endsWith('$'))
+  // Add data entries from step.d
+  if (step.d) {
+    const dataEntries = Object.entries(step.d).filter(([k]: any) => k !== 'id' && !k.endsWith('$'))
     for (const [key] of dataEntries) {
       Content(`			"${key}": setup.idmap["${key}"],
 `)
@@ -560,10 +553,10 @@ const generateUpdate: OpGen = (ctx, step, index) => {
 `)
 
   // Handle TextFieldMark spec
-  if (step.spec) {
-    for (const spec of step.spec) {
-      if ('TextFieldMark' === spec.apply && null != step.input.textfield) {
-        const fieldname = step.input.textfield
+  if (step.s) {
+    for (const spec of step.s) {
+      if ('TextFieldMark' === spec.apply && null != step.i.textfield) {
+        const fieldname = step.i.textfield
         const fieldvalue = spec.def?.mark ?? `Mark01-${ref}`
         Content(`
 		${markdefvar}Name := "${fieldname}"
@@ -592,9 +585,9 @@ const generateUpdate: OpGen = (ctx, step, index) => {
   }
 
   // Assert TextFieldMark
-  if (step.spec) {
-    for (const spec of step.spec) {
-      if ('TextFieldMark' === spec.apply && null != step.input.textfield) {
+  if (step.s) {
+    for (const spec of step.s) {
+      if ('TextFieldMark' === spec.apply && null != step.i.textfield) {
         Content(`		if ${resdatavar}[${markdefvar}Name] != ${markdefvar}Value {
 			t.Fatalf("expected %s to be updated, got %v", ${markdefvar}Name, ${resdatavar}[${markdefvar}Name])
 		}
@@ -607,24 +600,24 @@ const generateUpdate: OpGen = (ctx, step, index) => {
 
 const generateLoad: OpGen = (ctx, step, index) => {
   const { entity, flow } = ctx
-  const ref = step.input.ref ?? entity.name + '_ref01'
-  const entvar = goVar(step.input.entvar ?? ref + '_ent')
-  const matchvar = goVar(step.input.matchvar ?? (ref + '_match' + (step.input.suffix ?? '')))
-  const datavar = goVar(step.input.datavar ?? (ref + '_data' + (step.input.suffix ?? '')))
-  const srcdatavar = goVar(step.input.srcdatavar ?? (ref + '_data' + (step.input.suffix ?? '')))
+  const ref = step.i.ref ?? entity.name + '_ref01'
+  const entvar = goVar(step.i.entvar ?? ref + '_ent')
+  const matchvar = goVar(step.i.matchvar ?? (ref + '_match' + (step.i.suffix ?? '')))
+  const datavar = goVar(step.i.datavar ?? (ref + '_data' + (step.i.suffix ?? '')))
+  const srcdatavar = goVar(step.i.srcdatavar ?? (ref + '_data' + (step.i.suffix ?? '')))
 
-  const priorSteps = Object.values(flow.step).slice(0, Number(index)) as any[]
+  const priorSteps = Object.values(flowSteps(flow)).slice(0, Number(index)) as any[]
   const hasEntVar = priorSteps.some((s: any) =>
-    ['create', 'list', 'load', 'update', 'remove'].includes(s.op))
+    ['create', 'list', 'load', 'update', 'remove'].includes(s.o))
 
   // Check if srcdatavar was declared by a prior create step or preamble
-  const flowHasCreate = Object.values(flow.step).some((s: any) => (s as any).op === 'create')
+  const flowHasCreate = Object.values(flowSteps(flow)).some((s: any) => (s as any).o === 'create')
   const preambleRef = entity.name + '_ref01'
   const hasSrcData = (!flowHasCreate && srcdatavar === goVar(preambleRef + '_data')) ||
     priorSteps.some((s: any) => {
-      if ('create' === s.op) {
-        const priorRef = s.input.ref ?? entity.name + '_ref01'
-        const priorDatvar = goVar(s.input.datavar ?? (priorRef + '_data' + (s.input.suffix ?? '')))
+      if ('create' === s.o) {
+        const priorRef = s.i.ref ?? entity.name + '_ref01'
+        const priorDatvar = goVar(s.i.datavar ?? (priorRef + '_data' + (s.i.suffix ?? '')))
         return priorDatvar === srcdatavar
       }
       return false
@@ -682,14 +675,14 @@ const generateRemove: OpGen = (ctx, step, index) => {
   if (null == entityDataIdField(entity)) {
     return
   }
-  const ref = step.input.ref ?? entity.name + '_ref01'
-  const entvar = goVar(step.input.entvar ?? ref + '_ent')
-  const matchvar = goVar(step.input.matchvar ?? (ref + '_match' + (step.input.suffix ?? '')))
-  const srcdatavar = goVar(step.input.srcdatavar ?? (ref + '_data'))
+  const ref = step.i.ref ?? entity.name + '_ref01'
+  const entvar = goVar(step.i.entvar ?? ref + '_ent')
+  const matchvar = goVar(step.i.matchvar ?? (ref + '_match' + (step.i.suffix ?? '')))
+  const srcdatavar = goVar(step.i.srcdatavar ?? (ref + '_data'))
 
-  const priorSteps = Object.values(flow.step).slice(0, Number(index)) as any[]
+  const priorSteps = Object.values(flowSteps(flow)).slice(0, Number(index)) as any[]
   const needsEnt = !priorSteps.some((s: any) =>
-    ['create', 'list', 'load', 'update', 'remove'].includes(s.op))
+    ['create', 'list', 'load', 'update', 'remove'].includes(s.o))
 
   // Use `:=` when this is the first op step (so `err` gets declared);
   // otherwise reuse the `err` from a prior op step.

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,43 +23,21 @@ 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
+// in the op's `points[].g.params[]` exactly as the typed-model generator
 // does. Falls back to the entity field of the same name (used when the op
 // 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)
+  const found = (params as any[]).find((p: any) => p && p.n === paramName)
   if (found) {
-    return found.type
+    return found.t
   }
   const field = (entity && entity.fields ? each(entity.fields) : [])
-    .find((f: any) => f && f.name === paramName) as any
-  return field && field.type
+    .find((f: any) => f && f.n === paramName) as any
+  return field && field.t
 }
 
 
@@ -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)

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

@@ -16,12 +16,6 @@ import {
 } from '@voxgig/apidef'
 
 
-// Published boru/eng/go version. The engine module
-// github.com/boru-lang/boru/eng/go is not yet tagged under the boru path (the
-// eng/go/v0.0.1 tag predates the aql->boru rename and still declares the old
-// aql-lang/aql module path), so we pin a commit pseudo-version. Bump here —
-// to a real eng/go/v<X.Y.Z> once one is published, or a newer pseudo-version —
-// when adopting a newer engine.
 const BORU_ENG_VERSION = 'v0.0.2-0.20260804163932-0d66b55c5110'
 
 // go.sum entries for BORU_ENG_VERSION and its transitive deps. The sibling
@@ -50,54 +44,21 @@ const Main = cmp(function Main(props: any) {
   const { model } = props.ctx$
 
   const org = model.origin || 'voxgig-sdk'
-  // The two names on the next lines are DIFFERENT KINDS of name, and the
-  // adjacency is the trap. `sdkModule` is the Go SDK this CLI imports — a
-  // different target, so it keeps its own name and must NOT follow this
-  // target's alias. `cliModule` is this target itself, so it must.
   const sdkModule = goModule(model, 'go')
   const cliModule = goModule(model, target.name)
 
   const entityMap: any = getModelPath(model, `main.${KIT}.entity`)
-  // Derive each entity's PascalCase `Name` here rather than relying on another
-  // TARGET having run first. This target sets `phase.entity: { active: false }`
-  // (see model/target/go-{cli,mcp}.aon), so create-sdkgen's Root.ts never
-  // calls names() for it — the accessor emitted below used to read `Name` off
-  // objects the `go` target happened to have named as a side effect, which
-  // yields `client.undefined(nil)` whenever this target is generated without
-  // `go`, or before it in target order.
   deriveEntityNames(entityMap)
   const entityNames = Object.keys(entityMap).map(n => n.toLowerCase())
 
   const FRAGMENT = Path.normalize(__dirname + '/../../../src/cmp/go-cli/fragment')
 
-  // .gitignore — build output (dist/), any stray top-level binaries, and
-  // jostraca's bookkeeping.
-  //
-  // `.jostraca/` matters only when this target is generated OUT OF TREE
-  // (`output: path`), which is a mode every target has: jostraca drops its
-  // meta log and a full duplicate of the last generated output at the output
-  // ROOT, and the destination is a different repository that nothing here
-  // puts under version control — so without this line the first regeneration
-  // leaves that repo dirty with hundreds of untracked files. In-tree the SDK
-  // repo's own ignore file already covers it, which is why this was missed:
-  // only `seneca-provider` was ever generated out of tree, and it was the
-  // only target whose ignore file carried the line. The other 21 targets
-  // still do not — see docs/design/seneca-provider-package.md.
   File({ name: '.gitignore' }, () => Content(`/dist/
 /${model.name}-cli
 /go-cli
 .jostraca/
 `))
 
-  // ==========================================================================
-  // README.md — Diátaxis-structured, examples-first usage guide.
-  //
-  // EVERYTHING here is MODEL-DRIVEN: verbs, examples and how-to sections are
-  // gated on whether an active entity actually exposes that op
-  // (op.<name>.active !== false) — never document an operation no entity
-  // supports. The CLI implements three boru words (list / load / update; see
-  // words.fragment.go + runOp).
-  // ==========================================================================
   const bin = `${model.name}-cli`
   const entityList = entityNames.length > 0 ? entityNames.join(' ') : '(none)'
   const entityCount = entityNames.length
@@ -149,7 +110,6 @@ const Main = cmp(function Main(props: any) {
   const cliExampleExpr = exampleOp || '--help'
   const replExampleExpr = exampleOp || '/help'
 
-  // ---- Examples block (up front) -------------------------------------------
   const ex: string[] = []
   ex.push(`# 1. Build a native binary (-> dist/<os>-<arch>/${bin})`)
   ex.push('make build')
@@ -180,7 +140,6 @@ const Main = cmp(function Main(props: any) {
   ex.push(`${model.name}> /quit`)
   const exampleBlock = ex.join('\n')
 
-  // ---- How-to guides (gated) -----------------------------------------------
   const howtos: string[] = []
   if (firstHas('list')) howtos.push(`### List the records of an entity
 
@@ -406,10 +365,6 @@ replace ${sdkModule} => ../go
   // BORU_ENG_VERSION above.
   File({ name: 'go.sum' }, () => Content(BORU_ENG_GOSUM))
 
-  // Makefile — `make build` for the current machine, `make build-all` to
-  // cross-compile for the three desktop OSes (linux, darwin, windows) on
-  // amd64 + arm64. Every binary is named ${model.name}-cli (+ .exe on windows)
-  // inside its own dist/<os>-<arch>/ folder — no loose top-level binary.
   File({ name: 'Makefile' }, () => Content(`# ${model.name}-cli build. GENERATED by @voxgig/sdkgen go-cli target.
 BINARY := ${model.name}-cli
 DIST := dist
@@ -437,8 +392,6 @@ clean:
 \trm -rf $(DIST) $(BINARY) go-cli
 `))
 
-  // main.go — produced from fragment/main.fragment.go with two Slots
-  // for the prompt label and the entity help line.
   File({ name: 'main.go' }, () => {
     Fragment(
       {
@@ -459,8 +412,6 @@ clean:
     )
   })
 
-  // words.go — produced from fragment/words.fragment.go with a single
-  // Slot that emits one `case "<name>":` per entity.
   File({ name: 'words.go' }, () => {
     Fragment(
       {

package/project/.sdk/src/cmp/go-cli/fragment/words.fragment.go

@@ -9,18 +9,6 @@ import (
 	sdk "GOMODULE"
 )
 
-// registerSDKWords installs three native boru words bound to the SDK:
-// list / load / update. Each is declared with two overloads matching
-// the signature  [query?:(Node or Scalar) entity:atom]:
-//
-//   [entity:Atom]            — no query (e.g. `list book`)
-//   [query:Any entity:Atom]  — query is any Node or Scalar (e.g.
-//                              `load {id:1} book`, `load 1 book`)
-//
-// The entity slot is /q-quoted so a bareword `book` parses as the
-// Atom "book" rather than dispatching as an undefined word. Both
-// overloads are all-forward (BarrierAllForward), so args are collected
-// from the tokens following the word.
 func registerSDKWords(r *eng.Registry, client *sdk.ProjectNameSDK) {
 	for _, op := range []string{"list", "load", "update"} {
 		op := op