@voxgig/sdkgen 4.21.0 → 4.23.0

package/project/.sdk/src/cmp/py/TestDirect_py.ts

@@ -31,10 +31,10 @@ function normalizePathParams(
       // original name was renamed to another param's current name (e.g. badge
       // load: param 'group_id' has orig 'id', and another param has name 'id').
       const param = params.find((p: any) =>
-          p.name === snaked || p.name === depluralized) ||
+          p.n === snaked || p.n === depluralized) ||
         params.find((p: any) =>
-          p.orig === snaked || p.orig === depluralized)
-      if (param) return '{' + param.name + '}'
+          p.or === snaked || p.or === depluralized)
+      if (param) return '{' + param.n + '}'
 
       if (rename) {
         for (const [origCamel, renamedTo] of Object.entries(rename)) {
@@ -42,10 +42,10 @@ function normalizePathParams(
             const origSnaked = snakify(origCamel)
             const origDepluralized = depluralize(origSnaked)
             const renamedParam = params.find(
-              (p: any) => p.orig === origSnaked || p.name === origSnaked ||
-                p.orig === origDepluralized || p.name === origDepluralized
+              (p: any) => p.or === origSnaked || p.n === origSnaked ||
+                p.or === origDepluralized || p.n === origDepluralized
             )
-            if (renamedParam) return '{' + renamedParam.name + '}'
+            if (renamedParam) return '{' + renamedParam.n + '}'
           }
         }
       }
@@ -99,8 +99,8 @@ const TestDirect = cmp(function TestDirect(props: any) {
   const listOp = entity.op?.list
 
   const loadPoint = loadOp?.points?.[0]
-  const loadPath = loadPoint ? normalizePathParams(pointParts(loadPoint), loadPoint?.args?.params || [], loadPoint?.rename?.param) : ''
-  const allLoadParams = loadPoint?.args?.params || []
+  const loadPath = loadPoint ? normalizePathParams(pointParts(loadPoint), loadPoint?.g?.params || [], loadPoint?.r?.param) : ''
+  const allLoadParams = loadPoint?.g?.params || []
   // Some upstream OpenAPI specs declare a parameter as `in: path` even when
   // that path has no `{name}` placeholder for it. Only path params that
   // actually appear in the URL template should drive direct-test path-param
@@ -112,7 +112,7 @@ const TestDirect = cmp(function TestDirect(props: any) {
       _pathPlaceholders.add(part.slice(1, -1))
     }
   }
-  const _renameMap = (loadPoint?.rename?.param || {}) as Record<string, string>
+  const _renameMap = (loadPoint?.r?.param || {}) as Record<string, string>
   const _renamedPlaceholders = new Set<string>()
   for (const ph of _pathPlaceholders) {
     _renamedPlaceholders.add(ph)
@@ -121,28 +121,26 @@ const TestDirect = cmp(function TestDirect(props: any) {
     }
   }
   const loadParams = allLoadParams.filter((p: any) =>
-    _renamedPlaceholders.has(p.name) || _renamedPlaceholders.has(p.orig))
+    _renamedPlaceholders.has(p.n) || _renamedPlaceholders.has(p.or))
 
   const listPoint = listOp?.points?.[0]
-  const listPath = listPoint ? normalizePathParams(pointParts(listPoint), listPoint?.args?.params || [], listPoint?.rename?.param) : ''
-  const listParams = listPoint?.args?.params || []
+  const listPath = listPoint ? normalizePathParams(pointParts(listPoint), listPoint?.g?.params || [], listPoint?.r?.param) : ''
+  const listParams = listPoint?.g?.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 loadQuery = loadPoint?.g?.query || []
   const loadLiveQueryEntries = loadQuery
-    .filter((q: any) => q.reqd && undefined !== q.example && null !== q.example)
+    .filter((q: any) => q.r && undefined !== q.ex && null !== q.ex)
   const loadLiveQueryLines = loadLiveQueryEntries
-    .map((q: any) => `            query["${q.name}"] = ${JSON.stringify(q.example)}`)
+    .map((q: any) => `            query["${q.n}"] = ${JSON.stringify(q.ex)}`)
     .join('\n')
 
   // Path params with spec-provided examples — when ALL load params have
   // spec examples, prefer them over list-bootstrap.
   const loadAllHaveExamples =
     loadParams.length > 0 &&
-    loadParams.every((p: any) => undefined !== p.example && null !== p.example)
+    loadParams.every((p: any) => undefined !== p.ex && null !== p.ex)
   const loadExampleLines = loadAllHaveExamples
-    ? loadParams.map((p: any) => `            params["${p.name}"] = ${JSON.stringify(p.example)}`).join('\n')
+    ? loadParams.map((p: any) => `            params["${p.n}"] = ${JSON.stringify(p.ex)}`).join('\n')
     : ''
 
   const entidEnvVar = `${PROJECTNAME}_TEST_${envToken(entity.name)}_ENTID`
@@ -167,9 +165,9 @@ class Test${entity.Name}Direct:
     if (hasList && listPoint) {
       // Track idmap keys this list test consumes in live mode.
       const listLiveIdKeys: string[] = listParams.map((lp: any) => {
-        return lp.name === 'id'
+        return lp.n === 'id'
           ? entity.name + '01'
-          : lp.name.replace(/_id$/, '') + '01'
+          : lp.n.replace(/_id$/, '') + '01'
       })
       const listSkipBlock = listLiveIdKeys.length > 0
         ? `        if setup["live"]:
@@ -199,13 +197,13 @@ ${listSkipBlock}        client = setup["client"]
         Content(`        params = {}
 `)
         for (const lp of listParams) {
-          const key = lp.name === 'id'
+          const key = lp.n === 'id'
             ? entity.name + '01'
-            : lp.name.replace(/_id$/, '') + '01'
+            : lp.n.replace(/_id$/, '') + '01'
           Content(`        if setup["live"]:
-            params["${lp.name}"] = setup["idmap"]["${key}"]
+            params["${lp.n}"] = setup["idmap"]["${key}"]
         else:
-            params["${lp.name}"] = "direct01"
+            params["${lp.n}"] = "direct01"
 `)
         }
         Content(`
@@ -283,7 +281,7 @@ ${loadLiveQueryLines ? loadLiveQueryLines + '\n' : ''}${loadExampleLines}
         else:
 `)
           for (let i = 0; i < loadParams.length; i++) {
-            Content(`            params["${loadParams[i].name}"] = "direct0${i + 1}"
+            Content(`            params["${loadParams[i].n}"] = "direct0${i + 1}"
 `)
           }
         } else if (loadParams.length > 0) {
@@ -295,7 +293,7 @@ ${loadLiveQueryLines}
           Content(`        if not setup["live"]:
 `)
           for (let i = 0; i < loadParams.length; i++) {
-            Content(`            params["${loadParams[i].name}"] = "direct0${i + 1}"
+            Content(`            params["${loadParams[i].n}"] = "direct0${i + 1}"
 `)
           }
         } else if (loadLiveQueryLines) {

package/project/.sdk/src/cmp/py/TestEntity_py.ts

@@ -1,3 +1,4 @@
+import { flowSteps } from '@voxgig/sdkgen'
 
 import {
   flatten,
@@ -85,10 +86,10 @@ const TestEntity = cmp(function TestEntity(props: any) {
   const idnames = buildIdNames(entity, basicflow)
   const idnamesStr = idnames.map(n => `"${n}"`).join(', ')
 
-  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])
 
@@ -165,7 +166,7 @@ ${hasList ? `
         # multiple ops; skipping any one skips the whole flow (steps depend
         # on each other).
         _live = setup.get("live", False)
-        for _op in [${(Array.from(new Set((basicflow.step as any[]).map((s: any) => s.op).filter(Boolean)))).map(o => `"${o}"`).join(', ')}]:
+        for _op in [${(Array.from(new Set((flowSteps(basicflow) as any[]).map((s: any) => s.o).filter(Boolean)))).map(o => `"${o}"`).join(', ')}]:
             _skip, _reason = runner.is_control_skipped("entityOp", "${entity.name}." + _op, "live" if _live else "unit")
             if _skip:
                 pytest.skip(_reason or "skipped via sdk-test-control.json")
@@ -180,7 +181,7 @@ ${hasList ? `
 `)
 
     // Check if the flow has a create step
-    const flowHasCreate = allSteps.some((s: any) => s.op === 'create')
+    const flowHasCreate = allSteps.some((s: any) => s.o === 'create')
     if (!flowHasCreate) {
       Content(`        # Bootstrap entity data from existing test data.
         ${entity.name}_ref01_data_raw = vs.items(helpers.to_map(
@@ -193,16 +194,16 @@ ${hasList ? `
     }
 
     // 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')
@@ -298,18 +299,18 @@ def _${entity.name}_basic_setup(extra):
 
 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 = 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 = 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
@@ -354,14 +355,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 = 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
 `)
@@ -391,12 +392,12 @@ const generateList: OpGen = (ctx, step, index) => {
 `)
 
   // Handle validators
-  const allSteps = Object.values(flow.step) as any[]
-  if (step.valid) {
-    for (const validator of step.valid) {
+  const allSteps = Object.values(flowSteps(flow)) as any[]
+  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 = validRef + '_data'
@@ -422,16 +423,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 = Object.values(flow.step).slice(0, Number(index)) as any[]
+  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 = 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
 
@@ -448,8 +449,8 @@ const generateUpdate: OpGen = (ctx, step, index) => {
 `)
   }
 
-  if (step.data) {
-    const dataEntries = Object.entries(step.data).filter(([k]: any) => k !== 'id' && !k.endsWith('$'))
+  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}"],
 `)
@@ -459,10 +460,10 @@ const generateUpdate: OpGen = (ctx, step, index) => {
   Content(`        }
 `)
 
-  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}"
@@ -482,9 +483,9 @@ const generateUpdate: OpGen = (ctx, step, index) => {
 `)
   }
 
-  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(`        assert ${resdatavar}[${markdefvar}_name] == ${markdefvar}_value
 `)
       }
@@ -495,23 +496,23 @@ 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 = 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))
 
-  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 === (preambleRef + '_data')) ||
     priorSteps.some((s: any) => {
-      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
@@ -557,14 +558,14 @@ const generateRemove: OpGen = (ctx, step, index) => {
   if (null == entityDataIdField(entity)) {
     return
   }
-  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 = 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(`        # REMOVE
 `)

package/project/.sdk/src/cmp/py/utility_py.ts

@@ -66,28 +66,6 @@ function formatPyValue(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

package/project/.sdk/src/cmp/py-data/Main_py-data.ts

@@ -14,26 +14,8 @@ import {
 } from '@voxgig/apidef'
 
 
-// The `py-data` target: a Python package for DATA ANALYSTS working in
-// notebooks (Colab, Jupyter), layered on the sibling `py` SDK generated into
-// the same repo. It is a consumer target in the go-cli / go-mcp mould — every
-// standard generation phase is switched off in model/target/py-data.aon and
-// this component emits the whole package.
-//
-// The deliverable an analyst wants is not "a client", it is TABLES: one line
-// from API to a typed pandas DataFrame, discovered by tab-completion rather
-// than by reading reference docs.
-//
-// Layered, NOT forked from `py`: the dependency arrow points one way
-// (py-data -> py) and `py` must never learn this target exists. A fork would
-// duplicate the entity/feature machinery and drift, which is the known
-// failure mode.
-
-
-// Accessors are generated only for entities exposing these ops. `list` gives
-// a DataFrame; `load` gives a Series. Everything else (create/update/remove)
-// is a write path and deliberately out of scope — see the README's
-// "What this package does not do".
+
+
 const FRAME_OP = 'list'
 const SERIES_OP = 'load'
 
@@ -42,9 +24,6 @@ const Main = cmp(function Main(props: any) {
   const { target, ctx$ } = props
   const { model, log } = ctx$
 
-  // HARD REQUIREMENT: this package imports the sibling py SDK. Generating it
-  // without `py` produces a package whose every import fails at runtime, so
-  // fail at GENERATE time with an actionable message instead.
   const targets = model.main[KIT].target || {}
   if (null == targets.py) {
     throw new SdkGenError(
@@ -54,12 +33,12 @@ const Main = cmp(function Main(props: any) {
       'then regenerate.')
   }
 
-  const name = model.const.Name              // PascalCase, e.g. Univec
-  const lower = String(name).toLowerCase()   // univec
-  const ENV = envName(model)                 // UNIVEC
-  const sdkModule = `${lower}_sdk`           // univec_sdk
-  const sdkClass = `${name}SDK`              // UnivecSDK
-  const dataModule = `${lower}_data`         // univec_data
+  const name = model.const.Name
+  const lower = String(name).toLowerCase()
+  const ENV = envName(model)
+  const sdkModule = `${lower}_sdk`
+  const sdkClass = `${name}SDK`
+  const dataModule = `${lower}_data`
 
   // Multi-tenant APIs (a server URL carrying an unresolved {variable}, e.g.
   // https://{instance}.example.com/api) have no single host, so base_url
@@ -80,25 +59,12 @@ const Main = cmp(function Main(props: any) {
     note: `frames:${frameEnts.length} series:${seriesEnts.length} of ${ents.length}`,
   })
 
-  // Root-level statics: Makefile, LICENSE, the template-side tests.
   Copy({
     from: 'tm/' + target.name,
     exclude: [/src\//, /pkg\//],
     replace: { ...ctx$.stdrep },
   })
 
-  // The runtime modules live INSIDE the package, not at the top level.
-  //
-  // They were originally emitted flat as auth.py / fetch.py / frames.py, which
-  // is actively dangerous here: `auth`, `fetch`, `frames`, `core`, `entity`
-  // and `utility` are all real PyPI distributions AND exactly the names an
-  // analyst leaves lying around next to a notebook. In Colab the working
-  // directory is on sys.path, so one stray auth.py shadowed ours and `data()`
-  // died with `module 'auth' has no attribute 'resolve'`. A package keeps
-  // every internal module behind the distinctive `<name>_data.` prefix.
-  //
-  // The folder name is model-derived, so it comes from the component; the file
-  // contents are the same for every API, so they stay in tm/py-data/pkg.
   Folder({ name: dataModule }, () => {
     Copy({
       from: 'tm/' + target.name + '/pkg',
@@ -351,11 +317,11 @@ class EntityFrames:
 // and the doc table the docstring shows.
 function entFields(ent: any): { name: string, dtype: string, req: boolean }[] {
   return each(ent.fields)
-    .filter((f: any) => f && null != f.name && false !== f.active)
+    .filter((f: any) => f && null != f.n && false !== f.a)
     .map((f: any) => ({
-      name: String(f.name),
-      dtype: canonToDtype(f.type),
-      req: true === f.req,
+      name: String(f.n),
+      dtype: canonToDtype(f.t),
+      req: true === f.r,
     }))
 }
 
@@ -436,7 +402,7 @@ function fieldDoc(fields: any[], indent: string): string {
 
 function frameAccessor(ent: any): string {
   const acc = pluralAccessor(ent)
-  const entMethod = ent.Name                 // PascalCase SDK accessor
+  const entMethod = ent.Name
   const fields = entFields(ent)
   const dtypes = pyDict(fields.map((f) => [f.name, f.dtype] as [string, string]), '            ')
   const order = pyList(fields.map((f) => f.name), '            ')

package/project/.sdk/src/cmp/rb/Config_rb.ts

@@ -36,30 +36,7 @@ import {
 } from './utility_rb'
 
 
-// PLUGIN DEFINITION REQUIRES AND THE FEATURE_PLUGINS MAP (the ruby peer of
-// cmp/py/Config_py.ts's pluginImports/pluginDefs).
-//
-// Upstream sekreto replaced its self-registration registry with
-// voxgig/plugin definitions: a provider kind the caller did not pass in via
-// `plugins: [...]` is unknown to that Sekreto. So config requires each
-// active plugin's module and names its exported definition CONSTANT (the
-// model's `def.rb` map), handing the list to the feature through
-// FEATURE_PLUGINS.
-//
-// The `def` map is declared in the model rather than derived from filenames
-// because one module may export several definitions (sekreto's aws.rb
-// exports AWSSECRETS and AWSPARAMS) - hence the de-duplication by path, so
-// a two-definition module yields ONE require. A def value is the module's
-// path under tm/rb, which is this target's root, so the require_relative is
-// that path without its extension.
-//
-// GATED, unlike go and py: the whole block is emitted only when an active
-// feature DECLARES a plugin catalogue for this target. An SDK that does not
-// carry the secrets feature must be byte-identical to what it was before
-// the feature existed, and an unread constant is not a thing a simple SDK
-// should have to explain.
 function rbPlugins(model: any, feature: any) {
-  // path -> [constant, ...], so one require serves a two-definition module.
   const bypath: Record<string, string[]> = {}
   const defs: Record<string, string[]> = {}
   let declared = false
@@ -111,24 +88,12 @@ const Config = cmp(async function Config(props: any) {
   const model: Model = ctx$.model
 
   const entity = getModelPath(model, `main.${KIT}.entity`)
-  // Gated by the applicability tags, so this target never imports or
-  // registers a feature it has no source for. One rule, one place:
-  // helpers/applicability.
   const feature = targetFeatures(model, target)
 
   const headers = getModelPath(model, `main.${KIT}.config.headers`) || {}
 
   const authActive = isAuthActive(model)
-  // config.auth.prefix override -> spec-derived info.security.prefix -> 'Bearer'
   const authPrefix = resolveAuthPrefix(model)
-  // `in` and `name` travel with the prefix now. They were resolved by
-  // apidef all along and dropped here, so an apiKey-in-query API got an
-  // `authorization` header it does not read. `basic` joins them because
-  // PrepareAuth_rb only emits its HTTP Basic branch for a basic scheme, and
-  // that branch reads this option at runtime.
-  //
-  // Each is emitted ONLY when it differs from the default, so an SDK with a
-  // plain header/Authorization bearer scheme regenerates byte-identical.
   const authBasic = isHttpBasicAuth(model)
   const authIn = resolveAuthIn(model)
   const authName = resolveAuthName(model)
@@ -155,13 +120,6 @@ const Config = cmp(async function Config(props: any) {
         },\n`
     : ''
 
-  // The same config as an OBJECT, built by the shared helper so this target's
-  // literal and the data that replaces it above the threshold are the same
-  // config by construction. The JSON is what the threshold is measured on -
-  // emitted source size varies by language, the model does not.
-  // Passing the target name opts in to the main slug/version/target identity
-  // fields (station descriptor inputs) - the literal path below emits them
-  // too, keeping data and literal reps in step.
   const { def: configDef, json: configJson } = configDefinition(model, target.name)
   const asData = isConfigData(configJson, configReprSetting(model))
 
@@ -170,9 +128,6 @@ const Config = cmp(async function Config(props: any) {
   const pluginRequireBlock = 0 === requires.length ? '' :
     '\n' + requires.join('\n') + '\n'
 
-  // Emitted whenever a catalogue is declared, even with every group off:
-  // the feature module reads the map unconditionally, and an SDK whose
-  // chain is all built-ins still has to answer with an empty list.
   const featurePluginsBlock = !declared ? '' :
     `  # The sekreto plugin DEFINITIONS the model selected per feature,
   # required above from the modules the catalogue's active \`plugin.def\`
@@ -205,17 +160,6 @@ ${featurePluginsBlock}  # Return the process-wide config, built once on first us
 
 `)
 
-    // ABOVE THE THRESHOLD: emit the model as DATA.
-    //
-    // A hash literal makes the Ruby parser build a node per entry and the VM
-    // execute an instruction per entry on every load. A string constant is one
-    // token, and `JSON.parse` (a C extension) builds the hash far faster.
-    //
-    // `JSON.parse` yields exactly what the literal did - String keys, Integer
-    // for whole numbers, true/false/nil - so make_config's result is unchanged.
-    //
-    // A SINGLE-quoted literal, so the JSON survives verbatim: a double-quoted
-    // Ruby string would interpolate any `#{` the model happens to contain.
     if (asData) {
       Content(`  # THE API MODEL, EMBEDDED AS DATA (sdkgen rung L1).
   #