@voxgig/sdkgen 4.21.0 → 4.22.0

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

@@ -8,11 +8,6 @@ import {
 } from '@voxgig/apidef'
 
 
-// A `list()` on a NESTED entity needs its parent path params. The
-// quickstart used to emit `client.Moon().list()` for an entity at
-// `/planet/{planet_id}/moon`, which 404s against a live server from a
-// half-built URL — indistinguishable from "no such record". The model
-// already marks those params `reqd: true`; matchArg renders exactly them.
 function listMatchArg(ent: any): string {
   const idF = entityIdField(ent)
   return matchArg('py', ent, 'list', idF, idLiteral(ent, 'list', idF))
@@ -26,9 +21,6 @@ const ReadmeQuick = cmp(function ReadmeQuick(props: any) {
 
   const exampleEntity = Object.values(entity).find((e: any) => e.active !== false) as any
 
-  // Find a nested entity if available: one with a parent chain
-  // (relations.ancestors), an active load op, and a required non-id load
-  // param to demonstrate (the parent key, e.g. page_id).
   const nestedEntity = Object.values(entity).find((e: any) =>
     e.active !== false &&
     e.relations && e.relations.ancestors && 0 < e.relations.ancestors.length &&
@@ -39,10 +31,6 @@ const ReadmeQuick = cmp(function ReadmeQuick(props: any) {
 
   const authActive = isAuthActive(model)
 
-  // Server variables (a templated server URL) are REQUIRED at construction
-  // - the SDK refuses rather than request a URL with a literal
-  // {account_id} in it - so a quickstart that omits them is a quickstart
-  // that fails on its first line.
   const svars = serverVariables(model)
   const pyServerField = 0 === svars.length ? '' :
     `\n    "server": {` +
@@ -117,9 +105,6 @@ except Exception as err:
       const neArticle = /^[aeiou]/i.test(neName) ? 'an' : 'a'
       const neVar = exampleVarName(neName.toLowerCase(), 'py')
 
-      // Model-driven match: every REQUIRED load-match key — the same shape
-      // the runtime resolves path params from, so the example always works.
-      // Parent keys (e.g. page_id) first, the entity's own id last.
       const neIdF = entityIdField(nestedEntity)
       const neRequired = opRequestShape(nestedEntity, 'load').items
         .filter((it: any) => !it.optional)
@@ -148,9 +133,6 @@ except Exception as err:
 `)
     }
     else if (opnames.includes('load')) {
-      // Every REQUIRED load-match key (id first, then parent path params like
-      // page_id) — the same shape the runtime resolves path params from, so
-      // the example always works.
       const loadRequired = opRequestShape(exampleEntity, 'load').items
         .filter((it: any) => !it.optional || it.name === idF)
         .sort((a: any, b: any) =>
@@ -176,20 +158,12 @@ except Exception as err:
 `)
     }
 
-    // Model-driven example fields: derive the create/update body from the op
-    // shape (opRequestShape) so the docs reference REAL writable fields, not a
-    // hardcoded "name" the entity may not have. Literals are Python-typed by
-    // the field's canonical type. ids are rendered separately as the match key
-    // for update/remove; a REQUIRED create id stays (the call is invalid
-    // without it).
     const examplePairs = (opname: string): string[] => {
       const items = opRequestShape(exampleEntity, opname).items
         .filter((it: any) => (it.name !== idF && it.name !== 'id') ||
           ('create' === opname && !it.optional))
       const required = items.filter((it: any) => !it.optional)
       const optional = items.filter((it: any) => it.optional)
-      // create needs ALL required fields; update is a patch, so the required
-      // members plus a sample optional field or two suffice.
       const chosen = 'create' === opname
         ? (required.length ? required : items.slice(0, 2))
         : required.concat(optional).slice(0, Math.max(2, required.length))
@@ -227,8 +201,6 @@ client.${eName}().update({${updatePairs.join(', ')}})
 `)
       }
       if (opnames.includes('remove')) {
-        // Every REQUIRED remove-match key: the id (off the created record
-        // when possible) plus parent keys like page_id.
         const removePairs = opRequestShape(exampleEntity, 'remove').items
           .filter((it: any) => !it.optional || it.name === idF)
           .sort((a: any, b: any) =>

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

@@ -1,5 +1,5 @@
 
-import { cmp, each, Content, canonToType, canonKey, canonScalarKey, File, isAuthActive, entityIdField, opRequestShape, safeVarName, exampleVarName, matchArg, idLiteral } from '@voxgig/sdkgen'
+import { cmp, each, Content, canonToType, canonKey, canonScalarKey, File, isAuthActive, entityIdField, opRequestShape, safeVarName, exampleVarName, matchArg, idLiteral, targetFeatures } from '@voxgig/sdkgen'
 import { ReadmeRefFeatures } from '@voxgig/sdkgen'
 
 import {
@@ -21,11 +21,6 @@ function pyLit(type: any, placeholder: string = 'example'): string {
 }
 
 
-// A `list()` on a NESTED entity needs its parent path params. The
-// quickstart used to emit `client.Moon().list()` for an entity at
-// `/planet/{planet_id}/moon`, which 404s against a live server from a
-// half-built URL — indistinguishable from "no such record". The model
-// already marks those params `reqd: true`; matchArg renders exactly them.
 function listMatchArg(ent: any): string {
   const idF = entityIdField(ent)
   return matchArg('py', ent, 'list', idF, idLiteral(ent, 'list', idF))
@@ -66,7 +61,7 @@ const ReadmeRef = cmp(function ReadmeRef(props: any) {
   const { model } = props.ctx$
 
   const entity = getModelPath(model, `main.${KIT}.entity`)
-  const feature = getModelPath(model, `main.${KIT}.feature`)
+  const feature = targetFeatures(model, target)
 
   const publishedEntities = each(entity).filter((e: any) => e.active !== false)
 
@@ -129,7 +124,6 @@ client = ${model.const.Name}SDK.test()
 `)
 
 
-    // Entity factory methods
     publishedEntities.map((ent: any) => {
       Content(`#### \`${ent.Name}(data=None)\`
 
@@ -171,7 +165,6 @@ Prepare a fetch definition without sending. Returns the \`fetchdef\` and raises
 `)
 
 
-    // Entity reference sections
     publishedEntities.map((ent: any) => {
       const opnames = Object.keys(ent.op || {})
       const fields = ent.fields || []
@@ -202,7 +195,6 @@ ${eVar} = client.${ent.Name}()
 `)
 
 
-      // Field schema
       if (fields.length > 0) {
         Content(`### Fields
 
@@ -249,7 +241,6 @@ ${eVar} = client.${ent.Name}()
       }
 
 
-      // Operation details
       if (opnames.length > 0) {
         Content(`### Operations
 
@@ -337,7 +328,6 @@ ${updateLines}    # Fields to update
       }
 
 
-      // Common methods
       Content(`### Common Methods
 
 #### \`data_get() -> dict\`
@@ -368,7 +358,6 @@ Return the entity name.
     })
 
 
-    // Features section
     const activeFeatures = each(feature).filter((f: any) => f.active)
     if (activeFeatures.length > 0) {
       Content(`
@@ -404,9 +393,6 @@ client = ${model.const.Name}SDK({
 \`\`\`
 
 `)
-      // The shared feature reference: options, defaults, usage and the
-      // considerations. Model facts, identical in every target, so they are
-      // written once in cmp/ReadmeRefFeatures.ts rather than here.
       ReadmeRefFeatures({ target })
     }
 

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

@@ -22,11 +22,6 @@ function pyLit(type: any, placeholder: string = 'example'): string {
 }
 
 
-// A `list()` on a NESTED entity needs its parent path params. The
-// quickstart used to emit `client.Moon().list()` for an entity at
-// `/planet/{planet_id}/moon`, which 404s against a live server from a
-// half-built URL — indistinguishable from "no such record". The model
-// already marks those params `reqd: true`; matchArg renders exactly them.
 function listMatchArg(ent: any): string {
   const idF = entityIdField(ent)
   return matchArg('py', ent, 'list', idF, idLiteral(ent, 'list', idF))
@@ -42,10 +37,6 @@ const ReadmeTopQuick = cmp(function ReadmeTopQuick(props: any) {
 
   const authActive = isAuthActive(model)
 
-  // Server variables (a templated server URL) are REQUIRED at construction
-  // - the SDK refuses rather than request a URL with a literal
-  // {account_id} in it - so a quickstart that omits them is a quickstart
-  // that fails on its first line.
   const svars = serverVariables(model)
   const pyServerField = 0 === svars.length ? '' :
     `\n    "server": {` +
@@ -72,8 +63,6 @@ client = ${ctor}
     // Python keyword (e.g. `class`) would otherwise emit uncompilable code.
     const eVar = exampleVarName(eName.toLowerCase(), 'py')
     const opnames = Object.keys(exampleEntity.op || {})
-    // Model-driven id key: null when the entity has no id-like field, in which
-    // case the load example takes no match argument.
     const idF = entityIdField(exampleEntity)
 
     let hasCall = false
@@ -88,9 +77,6 @@ for ${eVar} in ${eVar}s:
     }
 
     if (opnames.includes('load')) {
-      // Every REQUIRED load-match key (id first, then parent path params like
-      // page_id) — the same shape the runtime resolves path params from, so
-      // the example always works.
       const loadItems = opRequestShape(exampleEntity, 'load').items
         .filter((it: any) => !it.optional || it.name === idF)
         .sort((a: any, b: any) =>

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

@@ -12,25 +12,6 @@ import {
 } from '@voxgig/apidef'
 
 
-// THE GENERATED SCHEMA MODULE: the model's schemas, as data the SDK can run.
-//
-// The python peer of src/cmp/ts/Schema_ts.ts. Same two exports, same source:
-// OPTSPEC from `main.kit.optspec` plus each feature's own `config.options`,
-// ENTITYSPEC from the entity field sentinels — both built by the shared
-// helpers, so what python validates against and what ts validates against
-// cannot drift.
-//
-// EMBEDDED AS JSON, PARSED AT IMPORT, the same mechanism config.py already
-// uses for the model above the size threshold. A python dict literal would
-// also work here, but `json.loads` (the C accelerator) builds it from one
-// string constant rather than making the byte-compiler walk several hundred
-// literal nodes, and it is the one mechanism every ported target shares.
-//
-// The round-trip is exact because the spec holds only strings and booleans —
-// pinned by "strings and booleans only, so the JSON round-trip is lossless"
-// in ts/test/optspec.test.ts. JSON's single number type is the hazard that
-// test exists for: `true`/`false` and `null` differ between JSON and python
-// source, which is why this is a parsed string and not a literal paste.
 const Schema = cmp(async function Schema(props: any) {
   const ctx$ = props.ctx$
   const target = props.target

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

@@ -127,8 +127,6 @@ const TestDirect = cmp(function TestDirect(props: any) {
   const listPath = listPoint ? normalizePathParams(pointParts(listPoint), listPoint?.args?.params || [], listPoint?.rename?.param) : ''
   const listParams = listPoint?.args?.params || []
 
-  // Required query params with spec-provided examples — needed in live mode
-  // to satisfy API contracts (e.g. /v2018/history requires city/start/end).
   const loadQuery = loadPoint?.args?.query || []
   const loadLiveQueryEntries = loadQuery
     .filter((q: any) => q.reqd && undefined !== q.example && null !== q.example)

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',
@@ -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).
   #

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

@@ -1,29 +1,5 @@
 
 
-// Typed-model generator (Ruby target).
-//
-// Ported from the TypeScript reference EntityTypes_ts.ts. Reads
-// main.<KIT>.entity.<e>.fields[] and per-op params (op.<name>.points[].args.params[])
-// and emits one file, <Sdk>_types.rb, containing a keyword-init Struct per active
-// entity plus a request/match Struct per active op. Ruby has no enforced static
-// types, so each member carries a YARD `@!attribute`/`@return` annotation whose
-// element type comes from the canonical sentinels via the shared sdkgen helper
-// `canonToType` (source of truth: @voxgig/apidef VALID_CANON):
-//   $STRING -> String, $INTEGER -> Integer, $NUMBER -> Float, $BOOLEAN -> Boolean,
-//   $OBJECT -> Hash, $ARRAY -> Array, unknown -> Object.
-//
-// Type-name scheme (as TS, with one Ruby-only guard): data type = <Name>,
-// except where Ruby core already owns that constant — `File`, `Time`, `Data`,
-// … — which would be silently REPLACED by the assignment; those become
-// <Name>Type (see rbSafeTypeName). Per-op request
-// <Name>LoadMatch / <Name>ListMatch / <Name>RemoveMatch (query/id ops),
-// <Name>CreateData / <Name>UpdateData (body ops). An op WITH params -> a Struct of
-// those params (reqd:false -> annotated `[Type, nil]`). An op WITHOUT params -> a
-// Struct over the entity fields (Struct members are always optional/nil, so this is
-// the Ruby stand-in for TS `Partial<Name>`; callers may equally pass a plain Hash).
-//
-// Struct.new(keyword_init: true) gives value objects usable as `Advice.new(id: ...)`.
-// The file is required by the main SDK module so these constants are always loaded.
 
 import {
   cmp, each, names,
@@ -47,17 +23,6 @@ function symName(name: string): string {
 }
 
 
-// Emit one typed value class: a YARD @!attribute block plus a keyword-init
-// Struct. Ruby's `Struct.new` rejects zero members, so an empty shape falls
-// back to a plain empty class (keeps the type name available).
-//
-// A member with an EMPTY name is dropped first. `Struct.new(:"")` raises
-// `NameError: cannot make operator ID : attrset` when the file loads, which
-// takes the whole SDK down rather than just that type -- and quoting does
-// not help, since the empty symbol is the one Ruby will not accept (`:"a-b"`
-// and `:"1x"` are both fine). A model should not carry a nameless field, so
-// upstream is where it gets fixed; this keeps one bad field from costing the
-// target everything.
 function emitStruct(
   typeName: string,
   doc: string,
@@ -92,24 +57,12 @@ function emitStruct(
 const EntityTypes = cmp(function EntityTypes(props: any) {
   const { model, log } = props.ctx$
 
-  // only_active:false — getModelPath DROPS active:false entries by default,
-  // but the consumer scaffold (create-sdkgen Root.ts) iterates the RAW entity
-  // collection, so inactive entities still get generated entity code that
-  // references these typed names. The typed model must cover them too.
   const entity = getModelPath(model, `main.${KIT}.entity`, { only_active: false, required: false })
-  // Emit for EVERY entity that gets generated entity code: the consumer
-  // scaffold (create-sdkgen Root.ts) iterates entities WITHOUT an active
-  // filter, so inactive entities still get class files referencing these
-  // typed names. Filter on `name` (always present), NOT `active` — parity
-  // with the go emitter's fix.
   const entityList = deriveEntityNames(entity)
   // Derive the PascalCase Name up-front — it is set LAZILY by names(), so an
   // entity not yet named (e.g. a fieldless placeholder) would otherwise read
   // `Name = undefined` below. Parity with the go emitter's fix.
 
-  // Surface duplicate generated type names (two entities with the same
-  // PascalCase Name) — they would redeclare a type in statically-typed
-  // targets. Detection only; renaming is a model-level decision.
   warnEntityTypeCollisions(entity, log, LANG)
 
   File({ name: model.const.Name + '_types.' + LANG }, () => {
@@ -128,27 +81,16 @@ const EntityTypes = cmp(function EntityTypes(props: any) {
 
     entityList.forEach((ent: any) => {
       const Name = ent.Name
-      // Ruby constants share ONE namespace with the core classes, and
-      // assigning one silently replaces it — an entity named `File` emitting
-      // `File = Struct.new(...)` clobbers ::File and every later `File.join`
-      // raises NoMethodError. Only the bare data-type constant needs the
-      // guard; the per-op names below always carry a suffix that no core
-      // constant matches. The `client.File(...)` accessor is a METHOD, a
-      // separate namespace, so the public surface is unchanged.
       const TypeName = rbSafeTypeName(Name)
       const fields = (ent.fields ? each(ent.fields) : [])
         .filter((f: any) => f.active !== false)
 
-      // Entity data model: one member per field (`req:false` -> nilable).
       emitStruct(
         TypeName,
         `${Name} entity data model.`,
         fields.map((f: any) => ({ name: f.name, type: f.type, optional: false === f.req })),
       )
 
-      // Per active op: a request/match Struct. Members and their optionality
-      // come from the shared partiality policy (opRequestShape); this file only
-      // renders them as a keyword-init Struct.
       const ops = ent.op || {}
       ;['load', 'list', 'create', 'update', 'remove'].forEach((opname: string) => {
         if (null == ops[opname]) {

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

@@ -33,7 +33,6 @@ const Entity = cmp(function Entity(props: any) {
 
   const ff = Path.normalize(__dirname + '/../../../src/cmp/rb/fragment/')
 
-  // Entity files go to entity/ folder
   Folder({ name: 'entity' }, () => {
 
     File({ name: entity.name + '_entity.' + target.ext }, () => {
@@ -56,8 +55,6 @@ const Entity = cmp(function Entity(props: any) {
           EntityName: entity.Name,
           entityname: entity.name,
 
-          // Class token decoupled from the EntityName data-type token in
-          // Entity.fragment.rb so the class can be renamed independently.
           EntyClass: cls,
 
           // Feature-hook markers. jostraca's built-in `#Name-Tag` pattern is

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

@@ -59,7 +59,6 @@ const Main = cmp(async function Main(props: any) {
     }
   })
 
-  // Generate main SDK file
   File({ name: model.const.Name + '_sdk.' + target.ext }, () => {
 
     Fragment(
@@ -83,7 +82,6 @@ utility.feature_hook.call(@_rootctx, "${name}")
         }
       },
 
-      // Entities - injected at SLOT
       () => {
         each(entity, (entity: ModelEntity) => {
           const entitySDK = getModelPath(model, `main.${KIT}.entity.${entity.name}`)
@@ -93,19 +91,11 @@ utility.feature_hook.call(@_rootctx, "${name}")
       })
   })
 
-  // Generate config module
   Folder({ name: '.' }, () => {
     Config({ target })
     Schema({ target })
   })
 
-  // GENERATED, NOT COPIED. Where the credential goes is a fact about the
-  // API, and tm/ can only hold one answer. See PrepareAuth_rb.
-  //
-  // At the TARGET ROOT, not inside the `.` folder above and not inside any
-  // `src`: rb's tree is flat (utility/, core/, feature/ sit at the root),
-  // the component opens the one `utility` folder it needs, and that
-  // reproduces tm/rb/utility/prepare_auth.rb's old path exactly.
   PrepareAuth({ target })
 
   // Generate typed models (<Sdk>_types.rb) — required by the main SDK file.

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

@@ -28,9 +28,6 @@ const Package = cmp(async function Package(props: any) {
   // constant here did.
   const author = authorInfo(model, target.name)
 
-  // Gem name is namespaced to model.origin (e.g. "voxgig-sdk"). RubyGems
-  // names can't contain "/", so the parts are hyphen-joined. The require
-  // path (`${model.name}_sdk`) is unchanged.
   const ns = model.origin || 'voxgig-sdk'
   const pkgBase = ns.endsWith('-sdk') ? model.name : `${model.name}-sdk`
   const gemName = packageName(model, target.name)
@@ -39,7 +36,6 @@ const Package = cmp(async function Package(props: any) {
   const versionOf = (d: { version: string; source: 'feature' | 'target' }) =>
     d.source === 'target' ? (d.version || '0.0') : d.version
 
-  // Generate Gemfile
   File({ name: 'Gemfile' }, () => {
     Content(`source "https://rubygems.org"
 
@@ -53,7 +49,6 @@ gemspec
     }
   })
 
-  // Generate gemspec
   File({ name: model.const.Name + '_sdk.gemspec' }, () => {
     // RubyGems rejects a gemspec that declares the same runtime dependency
     // twice (Gem::InvalidSpecificationException at `gem build`), so the