@voxgig/sdkgen 4.21.0 → 4.23.0

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

@@ -1,3 +1,6 @@
+import { pointFacts } from '@voxgig/sdkgen'
+import { buildIdNames } from '@voxgig/sdkgen'
+import { flowSteps } from '@voxgig/sdkgen'
 
 import * as Path from 'node:path'
 
@@ -35,7 +38,8 @@ import {
   isHttpBasicAuth,
   entityDataIdField, envName, envToken,
   jsKey,
-  jsProp
+  jsProp,
+  hasLiveScenarios,
 } from '@voxgig/sdkgen'
 
 
@@ -44,10 +48,6 @@ import {
 } from './utility_ts'
 
 
-// GenCtx is the per-language generation context passed to every OpGen.
-// Languages with extra needs (Go's `gomodule`, PHP's `accessor`) extend
-// this shape locally. The signature `(ctx, step, index)` is now uniform
-// across all seven language tracks (Phase 1 of the templates refactor).
 type GenCtx = {
   model: Model
   entity: ModelEntity
@@ -79,16 +79,6 @@ const TestEntity = cmp(function TestEntity(props: any) {
         secret: env.${PROJENVNAME}_SECRET,` : ''}`
     : ''
 
-  // A templated server URL (OpenAPI server variables) makes a LIVE client
-  // impossible to construct without values: makeOptions raises rather than
-  // request a URL with a literal `{account_id}` in it. So the live suite
-  // takes them from the environment the same way it takes the apikey.
-  //
-  // Keys are quoted and the env read is bracketed via jsKey/jsProp: a server
-  // variable name is spec-derived and need not be a JS identifier — the URL
-  // grammar admits a leading digit ({2fa}), and a declared-but-unreferenced
-  // variable ({edge-zone}) is not constrained at all. Bare `name:` and
-  // `env.PROJ_SERVER_EDGE-ZONE` are both syntax errors.
   const svars = serverVariables(model)
   const serverEnvEntry = svars
     .map((v: any) => `\n    '${serverVarEnv(PROJENVNAME, v.name)}': ${JSON.stringify(v.dflt)},`).join('')
@@ -98,7 +88,6 @@ const TestEntity = cmp(function TestEntity(props: any) {
           ${jsKey(v.name)}: ${jsProp('env', serverVarEnv(PROJENVNAME, v.name))},`).join('')}
         },`
 
-  // TODO: should be a utility function
   const ff = projectPath('src/cmp/ts/fragment/')
 
   Folder({ name: entity.name }, () => {
@@ -119,6 +108,8 @@ const TestEntity = cmp(function TestEntity(props: any) {
         const basicflow = getModelPath(model, `main.${KIT}.flow.Basic${nom(entity, 'Name')}Flow`)
 
         const dobasic = basicflow && true === basicflow.active
+        const liveFacts = Object.fromEntries(Object.values(entity.op || {}).flatMap((op: any) =>
+          (op.points || []).map((point: any) => [point.m + ' ' + point.o, pointFacts(ctx$, point)])))
 
         if (!dobasic) {
           return;
@@ -126,15 +117,7 @@ const TestEntity = cmp(function TestEntity(props: any) {
 
         const indent = 2
 
-        const idlist = flatten([
-          entity.name + '01',
-          entity.name + '02',
-          entity.name + '03',
-          flatten(items(entity.relations.ancestors, (ap: any) =>
-            items(ap[1], (a: any) =>
-              items(['01', '02', '03'], (n: any) =>
-                a[1] + n[1]))), 2)
-        ])
+        const idlist = buildIdNames(entity, basicflow)
 
         Slot({ name: 'basicSetup' }, () => {
           Content(`
@@ -222,8 +205,8 @@ function basicSetup(extra?: any) {
 
 
         Slot({ name: 'basic' }, () => {
-          const flowHasCreate = Object.values(basicflow.step).some(
-            (s: any) => s.op === 'create'
+          const flowHasCreate = Object.values(flowSteps(basicflow)).some(
+            (s: any) => s.o === 'create'
           )
 
           // The basic test exercises a flow with one or more ops (load,
@@ -232,7 +215,7 @@ function basicSetup(extra?: any) {
           // dependent (e.g. update needs prior load), skipping ANY op the
           // flow exercises skips the whole basic test.
           const flowOps = Array.from(new Set(
-            (basicflow.step as any[]).map((s: any) => s.op).filter(Boolean)
+            (flowSteps(basicflow) as any[]).map((s: any) => s.o).filter(Boolean)
           ))
           const flowOpsLiteral = '[' + flowOps.map((o: any) => `'${o}'`).join(', ') + ']'
 
@@ -242,10 +225,10 @@ function basicSetup(extra?: any) {
       if (!live && maybeSkipControl(t, 'entityOp', '${entity.name}.' + op, live)) return
     }
 
-    ${Object.values(model.main.kit.entity || {}).some((e: any) => Object.values(e.op || {}).some((o: any) => (o.points || []).some((p: any) => p.contract && JSON.parse(p.contract.json).live))) ? `if (live) { t.skip('Covered by live operation scenarios'); return }` : ''}
+    ${hasLiveScenarios(model) ? `if (live) { t.skip('Covered by live operation scenarios'); return }` : ''}
     const setup = basicSetup()
     if (setup.live) {
-      return runLiveEntity(setup, ${JSON.stringify(entity)}, ${JSON.stringify(basicflow)}, '${nom(entity, 'Name')}')
+      return runLiveEntity(setup, ${JSON.stringify(entity)}, ${JSON.stringify(basicflow)}, '${nom(entity, 'Name')}', ${JSON.stringify(liveFacts)})
     }
     const client = setup.client
     const struct = setup.struct
@@ -267,16 +250,16 @@ function basicSetup(extra?: any) {
           const genCtx: GenCtx = {
             model, entity, flow: basicflow, PROJUPPER: PROJENVNAME,
           }
-          each(basicflow.step, (step: ModelEntityFlowStep, index: number) => {
+          each(flowSteps(basicflow), (step: ModelEntityFlowStep, index: number) => {
             // Never emit a REMOVE (or its removed-item verify LIST) without a
             // preceding CREATE: a coherent CRUD flow only removes what it made,
             // 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 = GENERATE_OP[step.op]
+            const opgen = GENERATE_OP[step.o]
             if (null != opgen) {
               opgen(genCtx, step, index)
               Content('\n')
@@ -291,18 +274,18 @@ function basicSetup(extra?: any) {
 
 const generateCreate: OpGen = (ctx, step, index) => {
   const { entity, flow } = ctx
-  const ref = step.input.ref ?? entity.name + '_ref01'
-  const entvar = step.input.entvar ?? ref + '_ent'
-  const datavar = step.input.datavar ?? (ref + '_data' + (step.input.suffix ?? ''))
+  const ref = step.i.ref ?? entity.name + '_ref01'
+  const entvar = step.i.entvar ?? ref + '_ent'
+  const datavar = step.i.datavar ?? (ref + '_data' + (step.i.suffix ?? ''))
 
-  const priorSteps = flow.step.slice(0, Number(index))
+  const priorSteps = flowSteps(flow).slice(0, Number(index))
   const needsEnt = !priorSteps.some(s =>
-    ['create', 'list', 'load', 'update', 'remove'].includes(s.op))
+    ['create', 'list', 'load', 'update', 'remove'].includes(s.o))
 
   const hasDatvar = priorSteps.some(s => {
-    if ('create' === s.op) {
-      const priorRef = s.input.ref ?? entity.name + '_ref01'
-      const priorDatvar = s.input.datavar ?? (priorRef + '_data' + (s.input.suffix ?? ''))
+    if ('create' === s.o) {
+      const priorRef = s.i.ref ?? entity.name + '_ref01'
+      const priorDatvar = s.i.datavar ?? (priorRef + '_data' + (s.i.suffix ?? ''))
       return priorDatvar === datavar
     }
     return false
@@ -323,7 +306,7 @@ const generateCreate: OpGen = (ctx, step, index) => {
 `)
   }
 
-  each(step.match, (mi: any) => {
+  each(step.m, (mi: any) => {
     Content(`    ${datavar}['${mi.key$}'] = setup.idmap['${mi.val$}']
 `)
   })
@@ -347,14 +330,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 = step.input.entvar ?? ref + '_ent'
-  const matchvar = step.input.matchvar ?? (ref + '_match' + (step.input.suffix ?? ''))
-  const listvar = step.input.listvar ?? (ref + '_list' + (step.input.suffix ?? ''))
+  const ref = step.i.ref ?? entity.name + '_ref01'
+  const entvar = step.i.entvar ?? ref + '_ent'
+  const matchvar = step.i.matchvar ?? (ref + '_match' + (step.i.suffix ?? ''))
+  const listvar = step.i.listvar ?? (ref + '_list' + (step.i.suffix ?? ''))
 
-  const priorSteps = flow.step.slice(0, Number(index))
+  const priorSteps = flowSteps(flow).slice(0, Number(index))
   const needsEnt = !priorSteps.some(s =>
-    ['create', 'list', 'load', 'update', 'remove'].includes(s.op))
+    ['create', 'list', 'load', 'update', 'remove'].includes(s.o))
 
   Content(`
     // LIST
@@ -366,7 +349,7 @@ const generateList: OpGen = (ctx, step, index) => {
   Content(`    const ${matchvar}: any = {}
 `)
 
-  each(step.match, (mi: any) => {
+  each(step.m, (mi: any) => {
     Content(`    ${matchvar}['${mi.key$}'] = setup.idmap['${mi.val$}']
 `)
   })
@@ -374,19 +357,13 @@ const generateList: OpGen = (ctx, step, index) => {
   Content(`
     const ${listvar} = (await ${entvar}.list(${matchvar})).map((e: any) => e.data())
 `)
-  const allSteps = flow.step
-  for (let vI = 0; vI < step.valid.length; vI++) {
-    const validator = step.valid[vI]
+  const allSteps = flowSteps(flow)
+  for (let vI = 0; vI < step.v.length; vI++) {
+    const validator = step.v[vI]
     const validRef = validator.def?.ref
-    const hasRefData = validRef && allSteps.some(s => 'create' === s.op &&
-      ((s.input.ref ?? entity.name + '_ref01') === validRef))
-
-    // Guard on a DATA id field (entityDataIdField), NOT entity.id (the load-MATCH
-    // key): an entity can have a match id in its path while its RESPONSE record
-    // has no `id` field (e.g. Multichannel's Template — data fields meta/status/…,
-    // no id). Then `${validRef}_data.id` is undefined and select(list, { id:
-    // undefined }) spuriously matches id-less records, so ItemExists/ItemNotExists
-    // are meaningless. Emit them only when `.id` is a real data field.
+    const hasRefData = validRef && allSteps.some(s => 'create' === s.o &&
+      ((s.i.ref ?? entity.name + '_ref01') === validRef))
+
     if ('ItemExists' === validator.apply && hasRefData && hasDataId) {
       Content(`
     assert(!isempty(select(${listvar}, { id: ${validRef}_data.id })))
@@ -403,16 +380,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 = step.input.entvar ?? ref + '_ent'
-  const datavar = step.input.datavar ?? (ref + '_data' + (step.input.suffix ?? ''))
-  const resdatavar = step.input.resdatavar ?? (ref + '_resdata' + (step.input.suffix ?? ''))
-  const markdefvar = step.input.markdefvar ?? (ref + '_markdef' + (step.input.suffix ?? ''))
-  const srcdatavar = step.input.srcdatavar ?? (ref + '_data' + (step.input.suffix ?? ''))
-
-  const priorSteps = flow.step.slice(0, Number(index))
+  const ref = step.i.ref ?? entity.name + '_ref01'
+  const entvar = step.i.entvar ?? ref + '_ent'
+  const datavar = step.i.datavar ?? (ref + '_data' + (step.i.suffix ?? ''))
+  const resdatavar = step.i.resdatavar ?? (ref + '_resdata' + (step.i.suffix ?? ''))
+  const markdefvar = step.i.markdefvar ?? (ref + '_markdef' + (step.i.suffix ?? ''))
+  const srcdatavar = step.i.srcdatavar ?? (ref + '_data' + (step.i.suffix ?? ''))
+
+  const priorSteps = flowSteps(flow).slice(0, Number(index))
   const needsEnt = !priorSteps.some(s =>
-    ['create', 'list', 'load', 'update', 'remove'].includes(s.op))
+    ['create', 'list', 'load', 'update', 'remove'].includes(s.o))
 
   const hasEntIdU = null != entity.id
 
@@ -430,7 +407,7 @@ const generateUpdate: OpGen = (ctx, step, index) => {
 `)
   }
 
-  each(step.data, (mi: any) => {
+  each(step.d, (mi: any) => {
     if ('id' !== mi.key$) {
       Content(`    ${datavar} ['${mi.key$}'] = setup.idmap['${mi.key$}']
 `)
@@ -438,10 +415,10 @@ const generateUpdate: OpGen = (ctx, step, index) => {
   })
 
 
-  for (let sI = 0; sI < step.spec.length; sI++) {
-    const spec = step.spec[sI]
-    if ('TextFieldMark' === spec.apply && null != step.input.textfield) {
-      const fieldname = step.input.textfield
+  for (let sI = 0; sI < step.s.length; sI++) {
+    const spec = step.s[sI]
+    if ('TextFieldMark' === spec.apply && null != step.i.textfield) {
+      const fieldname = step.i.textfield
       const fieldvalue = spec.def.mark
       Content(`
     const ${markdefvar} = { name: '${fieldname}', value: '${fieldvalue}_' + setup.now }
@@ -462,9 +439,9 @@ const generateUpdate: OpGen = (ctx, step, index) => {
 `)
   }
 
-  for (let sI = 0; sI < step.spec.length; sI++) {
-    const spec = step.spec[sI]
-    if ('TextFieldMark' === spec.apply && null != step.input.textfield) {
+  for (let sI = 0; sI < step.s.length; sI++) {
+    const spec = step.s[sI]
+    if ('TextFieldMark' === spec.apply && null != step.i.textfield) {
       Content(`
     assert((${resdatavar} as any)[${markdefvar}.name] === ${markdefvar}.value)
 `)
@@ -476,25 +453,25 @@ 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 = step.input.entvar ?? ref + '_ent'
-  const matchvar = step.input.matchvar ?? (ref + '_match' + (step.input.suffix ?? ''))
-  const datavar = step.input.datavar ?? (ref + '_data' + (step.input.suffix ?? ''))
-  const srcdatavar = step.input.srcdatavar ?? (ref + '_data' + (step.input.suffix ?? ''))
+  const ref = step.i.ref ?? entity.name + '_ref01'
+  const entvar = step.i.entvar ?? ref + '_ent'
+  const matchvar = step.i.matchvar ?? (ref + '_match' + (step.i.suffix ?? ''))
+  const datavar = step.i.datavar ?? (ref + '_data' + (step.i.suffix ?? ''))
+  const srcdatavar = step.i.srcdatavar ?? (ref + '_data' + (step.i.suffix ?? ''))
 
-  const priorSteps = flow.step.slice(0, Number(index))
+  const priorSteps = flowSteps(flow).slice(0, Number(index))
   const hasEntVar = priorSteps.some(s =>
-    ['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 by the
   // preamble bootstrap (which runs when the flow has no create step)
-  const flowHasCreate = flow.step.some(s => s.op === 'create')
+  const flowHasCreate = flowSteps(flow).some(s => s.o === 'create')
   const preambleRef = entity.name + '_ref01'
   const hasSrcData = (!flowHasCreate && srcdatavar === preambleRef + '_data') ||
     priorSteps.some(s => {
-      if ('create' === s.op) {
-        const priorRef = s.input.ref ?? entity.name + '_ref01'
-        const priorDatvar = s.input.datavar ?? (priorRef + '_data' + (s.input.suffix ?? ''))
+      if ('create' === s.o) {
+        const priorRef = s.i.ref ?? entity.name + '_ref01'
+        const priorDatvar = s.i.datavar ?? (priorRef + '_data' + (s.i.suffix ?? ''))
         return priorDatvar === srcdatavar
       }
       return false
@@ -502,18 +479,10 @@ const generateLoad: OpGen = (ctx, step, index) => {
 
   const hasEntId = null != entity.id
 
-  // When the entity has no id model field but the load operation requires
-  // path parameters (e.g. cotizacion needs {casa}/{fecha}), calling
-  // load({}) leaves the URL with literal {param} placeholders and the live
-  // API returns 404 HTML, which the SDK then fails to parse as JSON. There
-  // is no synthetic identifier to substitute, so skip emitting the load
-  // step's call in that case — but still declare the entity-var if no
-  // prior step has, so that later flow steps (e.g. remove) referencing
-  // ${entvar} compile.
   const loadOp = entity.op?.load
   const loadPoint = loadOp?.points?.[0]
-  const loadPathParams = loadPoint?.args?.params || []
-  const loadHasRequiredParams = loadPathParams.some((p: any) => p.reqd !== false)
+  const loadPathParams = loadPoint?.g?.params || []
+  const loadHasRequiredParams = loadPathParams.some((p: any) => p.r !== false)
   if (!hasEntId && loadHasRequiredParams) {
     if (!hasEntVar) {
       Content(`
@@ -554,14 +523,14 @@ const generateLoad: OpGen = (ctx, step, index) => {
 
 const generateRemove: OpGen = (ctx, step, index) => {
   const { entity, flow } = ctx
-  const ref = step.input.ref ?? entity.name + '_ref01'
-  const entvar = step.input.entvar ?? ref + '_ent'
-  const matchvar = step.input.matchvar ?? (ref + '_match' + (step.input.suffix ?? ''))
-  const srcdatavar = step.input.srcdatavar ?? (ref + '_data')
+  const ref = step.i.ref ?? entity.name + '_ref01'
+  const entvar = step.i.entvar ?? ref + '_ent'
+  const matchvar = step.i.matchvar ?? (ref + '_match' + (step.i.suffix ?? ''))
+  const srcdatavar = step.i.srcdatavar ?? (ref + '_data')
 
-  const priorSteps = flow.step.slice(0, Number(index))
+  const priorSteps = flowSteps(flow).slice(0, Number(index))
   const needsEnt = !priorSteps.some(s =>
-    ['create', 'list', 'load', 'update', 'remove'].includes(s.op))
+    ['create', 'list', 'load', 'update', 'remove'].includes(s.o))
 
   // "Remove what you created" needs the created record's id. An entity with
   // no DATA id field (entityDataIdField null — e.g. Multichannel's Template)

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

@@ -1,4 +1,4 @@
-import { cmp, File, Content, entityCollection, envName, serverVariables, serverVarEnv } from '@voxgig/sdkgen'
+import { cmp, File, Content, entityCollection, envName, serverVariables, serverVarEnv, liveHint, pointFacts } from '@voxgig/sdkgen'
 import { nom } from '@voxgig/apidef'
 
 const TestLive = cmp(function TestLive(props: any) {
@@ -8,12 +8,18 @@ const TestLive = cmp(function TestLive(props: any) {
     if (entity.active === false) continue
     for (const [op, operation] of Object.entries(entity.op || {}) as any[]) {
       for (const point of operation.points || []) {
-        const facts = point.contract ? JSON.parse(point.contract.json) : {}
-        if (model.main.kit.info?.auth === false) facts.security = []
-        const same = operation.points.filter((p: any) => JSON.stringify(p.select || {}) === JSON.stringify(point.select || {}))
+        // Only what the runner reads, not the whole point.
+        const all: any = pointFacts(props.ctx$, point)
+        const facts: any = {
+          live: liveHint(point),
+          security: model.main.kit.info?.auth === false ? [] : all.security,
+          securitySource: all.securitySource,
+          responses: all.responses,
+        }
+        const same = operation.points.filter((p: any) => JSON.stringify(p.q || {}) === JSON.stringify(point.q || {}))
         plan.push({ entity: entity.name, accessor: nom(entity, 'Name'), op,
-          id: point.contract?.id || point.method + ' ' + point.orig, contractVersion: point.contract?.version, kind: point.kind, graphql: point.graphql, path: point.orig, method: point.method,
-          action: point.select?.$action, rename: point.rename, args: point.args, facts, reachable: same.length === 1 })
+          id: point.m + ' ' + point.o, contractVersion: 1, kind: point.k, graphql: point.gq, path: point.o, method: point.m,
+          action: point.q?.$action, rename: point.r, args: point.g, facts, reachable: same.length === 1 })
       }
     }
   }

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

@@ -7,8 +7,6 @@ import { cmp, each, Folder, entityCollection,
   TestControl } from '@voxgig/sdkgen'
 
 
-// import { Quick } from './Quick_ts'
-// import { TestMain } from './TestMain_ts'
 import { TestLive } from './TestLive_ts'
 import { TestDirect } from './TestDirect_ts'
 import { TestEntity } from './TestEntity_ts'
@@ -25,8 +23,6 @@ const Test = cmp(function Test(props: any) {
     // Write-once: a project's edited control file survives regeneration.
     TestControl({ target, dir: 'test' })
     TestLive({ target })
-    // Quick({ target })
-    // TestMain({ target })
 
     ReadmeExampleTest({ target })
     ReadmeExamplesTest({ target })

package/project/.sdk/src/cmp/ts/fragment/Config.data.fragment.ts

@@ -8,31 +8,11 @@ const FEATURE_CLASS: Record<string, typeof BaseFeature> = {
 }
 
 
-// Per-feature plugin DEFINITIONS (voxgig/plugin `Definition` values), from
-// the model's active plugin groups. A feature that takes a `plugins` option
-// (secrets over sekreto) reads its own entry; a feature with no plugins has
-// none. Named imports above make each definition statically reachable, so
-// an SDK carries exactly the plugin modules its model selects — the same
-// leanness the old side-effect registry imports bought, without a registry.
 const FEATURE_PLUGINS: Record<string, any[]> = {
   // #FeaturePlugins
 }
 
 
-// THE API MODEL, EMBEDDED AS DATA (sdkgen rung L1).
-//
-// The literal form of this file declares the whole model as nested object
-// literals on the class. For a large API that is megabytes of source that the
-// TypeScript compiler must parse and infer a type for on every build, and that
-// the JavaScript engine must build node by node on every module load.
-//
-// As a single string constant it is one token to the compiler, and V8's JSON
-// parser builds the object far faster than the equivalent literal - JSON.parse
-// is the well-worn trick for exactly this shape of data.
-//
-// Emitted only above a size threshold, or when `main.kit.config.repr` pins it:
-// for a small model the literal is smaller, loads no slower, and is far easier
-// to read when debugging.
 const CONFIG_DATA = 'CONFIGJSON'
 
 
@@ -41,7 +21,6 @@ class Config {
   makeFeature(this: any, fn: string) {
     const fc = FEATURE_CLASS[fn]
     const fi = new fc()
-    // TODO: errors etc
     return fi
   }
 
@@ -55,12 +34,6 @@ class Config {
 }
 
 
-// Parsed ONCE, at module load, exactly like the literal form was built once.
-//
-// The parsed data is assigned onto an instance rather than replacing it, so
-// `config.main` / `.feature` / `.options` / `.entity` read exactly as they did
-// when they were field initialisers, and `makeFeature` stays where it was on
-// the prototype. Callers cannot tell which representation they were given.
 const config: any = Object.assign(new Config(), JSON.parse(CONFIG_DATA))
 
 export {

package/project/.sdk/src/cmp/ts/fragment/Config.fragment.ts

@@ -8,12 +8,6 @@ const FEATURE_CLASS: Record<string, typeof BaseFeature> = {
 }
 
 
-// Per-feature plugin DEFINITIONS (voxgig/plugin `Definition` values), from
-// the model's active plugin groups. A feature that takes a `plugins` option
-// (secrets over sekreto) reads its own entry; a feature with no plugins has
-// none. Named imports above make each definition statically reachable, so
-// an SDK carries exactly the plugin modules its model selects — the same
-// leanness the old side-effect registry imports bought, without a registry.
 const FEATURE_PLUGINS: Record<string, any[]> = {
   // #FeaturePlugins
 }
@@ -24,7 +18,6 @@ class Config {
   makeFeature(this: any, fn: string) {
     const fc = FEATURE_CLASS[fn]
     const fi = new fc()
-    // TODO: errors etc
     return fi
   }
 

package/project/.sdk/src/cmp/ts/fragment/Direct.test.fragment.ts

@@ -17,10 +17,6 @@ import {
 } from '../../utility'
 
 
-// AFTER the imports on purpose: TypeScript hoists `import` above any
-// statement in the emitted CommonJS, so a loader placed above them would
-// run only after every imported module had already been evaluated - and
-// anything reading process.env at module scope would miss these values.
 loadEnvLocal(__dirname + '/../../../.env.local')
 
 
@@ -32,9 +28,6 @@ describe('EntityNameDirect', async () => {
 
   test('direct-exists', async () => {
     const sdk = new ProjectNameSDK({
-      // Concrete base: a live construction must satisfy any server
-      // variables a templated base URL declares; overriding base with a
-      // literal (as the direct flow tests do) sidesteps the requirement.
       base: 'http://localhost:8080',
       system: { fetch: async () => ({}) }
     })

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

@@ -17,7 +17,6 @@ import type {
 // #TypeImports
 
 
-// TODO: needs Entity superclass
 class EntyClass extends ProjectNameEntityBase<EntityDataType> {
 
   constructor(client: ProjectNameSDK, entopts: any) {

package/project/.sdk/src/cmp/ts/fragment/Entity.test.fragment.ts

@@ -25,10 +25,6 @@ import {
 } from '../../utility'
 
 
-// AFTER the imports on purpose: TypeScript hoists `import` above any
-// statement in the emitted CommonJS, so a loader placed above them would
-// run only after every imported module had already been evaluated - and
-// anything reading process.env at module scope would miss these values.
 loadEnvLocal(__dirname + '/../../../.env.local')
 
 

package/project/.sdk/src/cmp/ts/fragment/EntityBase.fragment.ts

@@ -14,9 +14,6 @@ import type {
 } from './types'
 
 
-// TODO: needs Entity superclass
-// `D` is the entity's typed data model (e.g. Advice); subclasses bind it via
-// `class AdviceEntity extends ProjectNameEntityBase<Advice>`.
 class ProjectNameEntityBase<D = any> {
   name = ''
   name_ = ''
@@ -32,9 +29,6 @@ class ProjectNameEntityBase<D = any> {
   _match: Partial<D>
   _entctx: Context
 
-  // Set once a successful `remove` resolves on this instance. The entity
-  // KEEPS the data it held — a caller can still read what was deleted — but
-  // it is no longer a live record.
   _deleted: boolean
 
 
@@ -114,15 +108,6 @@ class ProjectNameEntityBase<D = any> {
   }
 
 
-  // Streaming operations. Runs `action` through the full pipeline and returns
-  // an async iterator 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): iterate the yielded items/chunks (from the
-  //     streaming feature when active, else the materialised items);
-  //   - outbound (upload): pass an async-iterable `body` to stream a request
-  //     payload — it is attached to the request so the transport can send it;
-  //   - `ctrl` (pipeline control) and `signal` (AbortSignal) are honoured.
   async *stream(this: any, action: string, args?: any, callopts?: any): AsyncGenerator<any> {
     const utility = this._utility
     const {
@@ -197,11 +182,6 @@ class ProjectNameEntityBase<D = any> {
 
   toJSON() {
     const struct = this._utility.struct
-  // The marker is NAMESPACED. It used to be `entity$` — a short, generic
-  // name, and the `$`-suffix convention is not unique to sdkgen. Seneca uses
-  // `entity$` on its own entities to hold the canon, so an SDK record fed
-  // into `entize` silently overwrote it and produced entities claiming a
-  // canon that does not exist: no error, just wrong entities.
     return struct.merge([{}, struct.getdef(this._data, {}), { 'voxgig$entity': this.Name }])
   }
 

package/project/.sdk/src/cmp/ts/fragment/EntityCreateOp.fragment.ts

@@ -87,12 +87,6 @@ class EntityOperation {
 
       const out = done(ctx)
 
-      // An operation resolves to the ENTITY, not the raw data — the record
-      // has just been absorbed into this instance and is reached through
-      // data(). `done` still runs: it completes the pipeline and raises on
-      // failure, and when throwing is disabled it hands back the error
-      // payload, which passes through unchanged. See AGENTS.md "Entity
-      // operations return ENTITIES".
       return (ctx.result && ctx.result.ok) ? this : out
     }
     catch (err: any) {

package/project/.sdk/src/cmp/ts/fragment/EntityLoadOp.fragment.ts

@@ -92,12 +92,6 @@ class EntityOperation {
 
       const out = done(ctx)
 
-      // An operation resolves to the ENTITY, not the raw data — the record
-      // has just been absorbed into this instance and is reached through
-      // data(). `done` still runs: it completes the pipeline and raises on
-      // failure, and when throwing is disabled it hands back the error
-      // payload, which passes through unchanged. See AGENTS.md "Entity
-      // operations return ENTITIES".
       return (ctx.result && ctx.result.ok) ? this : out
     }
     catch (err: any) {

package/project/.sdk/src/cmp/ts/fragment/EntityRemoveOp.fragment.ts

@@ -10,14 +10,6 @@ class EntityOperation {
 
   // EJECT-START
 
-  // Resolves to THIS entity, marked as deleted — like every other operation,
-  // which resolve to the entity too (see AGENTS.md). The instance keeps the
-  // data it held, so a caller can still read what was removed; `deleted()`
-  // reports that it is no longer a live record.
-  //
-  // A DELETE that answers 204 No Content therefore still resolves to
-  // something useful, where returning the raw body resolved to `undefined`
-  // against a signature that promised a record.
   async remove(
     this: any, reqmatch?: EntityNameRemoveMatch, ctrl?: Control,
   ): Promise<EntyClass> {
@@ -103,14 +95,7 @@ class EntityOperation {
 
       const out = done(ctx)
 
-      // An operation resolves to the ENTITY, not the raw data — the record
-      // has just been absorbed into this instance and is reached through
-      // data(). `done` still runs: it completes the pipeline and raises on
-      // failure, and when throwing is disabled it hands back the error
-      // payload, which passes through unchanged. See AGENTS.md "Entity
-      // operations return ENTITIES".
       if (ctx.result && ctx.result.ok) {
-        // A removed entity keeps its data but is no longer a live record.
         this.markDeleted()
         return this
       }

package/project/.sdk/src/cmp/ts/fragment/EntityUpdateOp.fragment.ts

@@ -93,12 +93,6 @@ class EntityOperation {
 
       const out = done(ctx)
 
-      // An operation resolves to the ENTITY, not the raw data — the record
-      // has just been absorbed into this instance and is reached through
-      // data(). `done` still runs: it completes the pipeline and raises on
-      // failure, and when throwing is disabled it hands back the error
-      // payload, which passes through unchanged. See AGENTS.md "Entity
-      // operations return ENTITIES".
       return (ctx.result && ctx.result.ok) ? this : out
     }
     catch (err: any) {