@voxgig/sdkgen 4.21.0 → 4.22.0

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

@@ -16,28 +16,6 @@ import {
 } from '@voxgig/apidef'
 
 
-// WHERE THE CREDENTIAL GOES IS A FACT ABOUT THE API, so it is generated
-// rather than templated.
-//
-// This was a static file at `tm/rb/utility/prepare_auth.rb` that hardcoded
-// `HEADER_AUTH = "authorization"`. apidef has always resolved the scheme's
-// `in` and `name` into `main.kit.info.security` — joplin's says
-// `in: "query", name: "token"` — and generation dropped both. The result
-// was an SDK that sent a header the API does not read and never sent the
-// query parameter it does, so it could not authenticate at all. Four repos
-// in the cedar fleet shipped that way: joplin (`token`), pipedrive
-// (`api_token`), trello (`key`), lm-umbrella (`apiKey`).
-//
-// A template cannot fix this, because the three placements need three
-// different bodies and a template has to pick one. A component emits the
-// branch this API actually uses and nothing else — no dead query code in a
-// bearer-token SDK, and no runtime `if` on a value that is fixed at
-// generation time.
-//
-// The ts port is the reference (cmp/ts/PrepareAuth_ts.ts). Two things are
-// deliberately NOT the same here, both recorded at their site below: the
-// header name is lower-cased, and the cookie clear removes our own pair
-// instead of a header that was never set.
 const PrepareAuth = cmp(async function PrepareAuth(props: any) {
   const { target } = props
   const { model } = props.ctx$
@@ -47,17 +25,6 @@ const PrepareAuth = cmp(async function PrepareAuth(props: any) {
   const name = resolveAuthName(model)
   const basic = isHttpBasicAuth(model)
 
-  // RB'S LAYOUT IS FLAT, WHICH IS WHY THIS FOLDER IS OPENED HERE AND THE TS
-  // PORT OPENS NONE. In ts, Main wraps its whole body in
-  // `Folder({name:'src'})` and the utilities live at `src/utility/`, so a
-  // second `src` there would write `src/src/utility/`. Main_rb opens no such
-  // folder: `Package`, `Gitignore`, the SDK file and `Copy({from:'tm/rb'})`
-  // all land at the target root, and `Config` sits inside a `Folder({name:
-  // '.'})` that is the root too. The template this replaces was
-  // `tm/rb/utility/prepare_auth.rb`, and that Copy's root IS the target root,
-  // so the file it produced was `<root>/utility/prepare_auth.rb`. One folder,
-  // `utility`, reproduces exactly that path — and register.rb's
-  // `require_relative 'prepare_auth'` (from `utility/`) only resolves there.
   Folder({ name: 'utility' }, () => {
     File({ name: 'prepare_auth.' + target.ext }, () => {
       Content(render({ Name: model.const.Name, active, where, name, basic }))
@@ -101,22 +68,8 @@ end
   // mean something.
   const basicHere = spec.basic && 'header' === spec.where
 
-  // The constant keeps the template's name per placement, so a
-  // header-based SDK regenerates the file it already had.
   const CRED = query ? 'QUERY_AUTH' : cookie ? 'COOKIE_AUTH' : 'HEADER_AUTH'
 
-  // HEADER NAMES ARE LOWER-CASED; QUERY AND COOKIE NAMES ARE NOT.
-  //
-  // `resolveAuthName` answers 'Authorization' by default, but the whole rb
-  // runtime spells header names in lower case — the template's
-  // `HEADER_AUTH = "authorization"`, secrets_feature.rb's
-  // `headers["authorization"]`, debug_feature.rb's mask list. A Ruby Hash key
-  // is case-sensitive, so emitting "Authorization" here would leave the
-  // secrets feature writing a SECOND, separate header and the copied
-  // pipeline_test.rb assertions on `headers["authorization"]` failing. HTTP
-  // header names are case-insensitive on the wire, so lower-casing costs
-  // nothing. Query parameter and cookie names ARE case-sensitive (`?Token=`
-  // is not `?token=`), so those go in verbatim.
   const credName = query || cookie ? spec.name : spec.name.toLowerCase()
 
   const consts =
@@ -140,10 +93,6 @@ end
     : `    headers = spec.headers
 `
 
-  // Removing OUR pair, not the whole cookie header. The ts port deletes
-  // `headers[CRED_name]` for the cookie case, which is a header nothing ever
-  // set — so a stale cookie survives a withdrawn credential there. This
-  // splits the header, drops the one pair we own, and puts the rest back.
   const dropCookie = !cookie ? '' : `
     # Our own pair, and only ours: another cookie the caller set survives.
     drop_cookie = ->(hs) {
@@ -209,9 +158,6 @@ end
 // The one thing that actually differs between placements: where the
 // credential is written, and whether the prefix travels with it.
 function place(where: string, clear: string): string {
-  // NO PREFIX IN A QUERY STRING. `?token=Bearer%20abc` is not a thing any
-  // API reads; the prefix is a header convention and is dropped here
-  // deliberately rather than silently concatenated.
   if ('query' === where) {
     return `
     if apikey.nil? || (apikey.is_a?(String) && (apikey == NOT_FOUND || apikey == ""))
@@ -244,7 +190,6 @@ function place(where: string, clear: string): string {
 `
   }
 
-  // HEADER: the template's body, unchanged.
   return `
     if apikey.nil? || (apikey.is_a?(String) && (apikey == NOT_FOUND || apikey == ""))
       ${clear}
@@ -259,30 +204,6 @@ function place(where: string, clear: string): string {
 }
 
 
-// NOT `isAuthActive`, AND THE DIFFERENCE IS LOAD-BEARING (the py port found
-// this first; same reasoning, same fix).
-//
-// `isAuthActive` is false whenever the SPEC declares no security scheme
-// (`main.kit.info.auth: false`). That is a statement about the DEFINITION,
-// not a ban on ever sending a credential: apidef writes it for every spec
-// with no securitySchemes block, and those SDKs are still expected to honour
-// an `apikey` the caller passes. The template placed the credential
-// unconditionally, so they did.
-//
-// Gating the body on `isAuthActive` therefore does not trim dead code, it
-// deletes working authentication. MEASURED, not assumed: with the wider gate
-// this component emitted the no-op for generatedcompile's own fixture
-// (`main: kit: info: { ... auth: false }`) and two running rb lanes went red
-// —
-//
-//   rb: auth null beats an explicit apikey
-//     FAIL: baseline broken: an ordinary apikey was not sent
-//   rb: the secrets feature runs with the feature active
-//     expected the authorization header to carry OPTKEY01, got: nil
-//
-// So the no-op is emitted only when the PROJECT says so: `config.auth.active:
-// false`, an explicit per-SDK switch nobody sets by accident. A spec that is
-// merely silent keeps the credential path it has always had.
 function authSwitchedOn(model: any): boolean {
   const auth = getModelPath(model, `main.${KIT}.config.auth`,
     { only_active: false, required: false })
@@ -290,8 +211,6 @@ function authSwitchedOn(model: any): boolean {
 }
 
 
-// A Ruby double-quoted literal. `#` is escaped only where it would OPEN an
-// interpolation, matching utility_rb.formatRubyValue.
 function rbstr(s: string): string {
   return '"' + String(s)
     .replace(/\\/g, '\\\\')

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

@@ -7,9 +7,6 @@ import {
 } from '@voxgig/apidef'
 
 
-// A type-correct Ruby literal for a field's canonical type — the create body
-// is EXECUTED by the doc test, so it must carry a real value per field.
-// Strings render the quoted placeholder.
 function rbLit(type: any, placeholder: string = 'example'): string {
   const k = canonScalarKey(type)
   if ('INTEGER' === k || 'NUMBER' === k) return '1'
@@ -54,7 +51,6 @@ const ReadmeEntity = cmp(function ReadmeEntity(props: any) {
   publishedEntities.map((entity: any) => {
     const opnames = Object.keys(entity.op || {})
     const fields = entity.fields || []
-    // Model-driven id key: null when this entity has no id-like field.
     const idF = entityIdField(entity)
     // Sanitise the local variable name — an entity whose lowercased name is a
     // Ruby keyword (e.g. `self`) would otherwise emit uncompilable code.
@@ -111,9 +107,6 @@ const ReadmeEntity = cmp(function ReadmeEntity(props: any) {
     }
 
     if (opnames.includes('load')) {
-      // The id key plus every REQUIRED match key (parent path params like
-      // page_id) — the same shape the runtime resolves path params from, so
-      // the example always works.
       const loadItems = opRequestShape(entity, 'load').items
         .filter((it: any) => !it.optional || it.name === idF)
         .sort((a: any, b: any) =>

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

@@ -7,38 +7,6 @@ import {
 } from '@voxgig/apidef'
 
 
-// Emits test/readme_examples_test.rb — a Minitest COMPLETENESS GATE that
-// guarantees every fenced ruby code example in the package docs is unit-tested.
-// It reads ALL THREE docs — the root (multi-language) ../../README.md, the
-// Ruby-specific ../README.md, and ../REFERENCE.md — extracts every fenced ruby
-// block (tagged by source doc + index) and enforces four properties:
-//
-//   1. SYNTAX — `ruby -c` on every block. Every documented example must parse.
-//   2. RUN — every RUNNABLE block (one that constructs the SDK, drives
-//      client., or performs an entity op) is EXECUTED offline in seeded test
-//      mode against the real SDK. The captured stdout+stderr is scanned for
-//      FATAL programming-error markers REGARDLESS of exit code, so a bug that a
-//      documented begin/rescue swallows and prints is still caught. Only a
-//      not-found / domain error is tolerated.
-//   3. COMPLETENESS — every block is partitioned into exactly one of
-//      {executed, syntaxchecked-nonrunnable, illustration} and the counts must
-//      sum to the total. "illustration" is a NARROW explicit class (a
-//      signature / method-table block that names the SDK class or a documented
-//      method but never uses a live client) — never a catch-all. A
-//      runnable-looking block that was NOT executed lands in neither bucket and
-//      FAILS the gate: no runnable example can be silently skipped.
-//   4. A per-doc summary (total / executed / syntaxchecked / illustration) is
-//      printed.
-//
-// A runnable block is rewritten so its client is a test-mode client
-// (<Sdk>SDK.test) seeded with an in-memory fixture for every entity it
-// references; any real .new/.test constructor is rewritten and the doc's own
-// require of the SDK file is stripped (we require it by absolute path). A block
-// that only *uses* `client` (constructed in an earlier fenced block) gets a
-// test client prepended.
-//
-// The emitted Ruby builds the ``` fence via 96.chr and shells out via Open3,
-// so this generator string contains no backticks of its own.
 const ReadmeExamplesTest = cmp(function ReadmeExamplesTest(props: any) {
   const { target } = props
   const { model } = props.ctx$

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

@@ -8,7 +8,6 @@ import {
 } from '@voxgig/apidef'
 
 
-// A type-correct Ruby literal for a field's canonical type.
 function rbLit(type: any): string {
   const k = canonScalarKey(type)
   if ('INTEGER' === k || 'NUMBER' === k) return '1'
@@ -23,9 +22,6 @@ const ReadmeHowto = cmp(function ReadmeHowto(props: any) {
   const { target, ctx$: { model } } = props
 
   const entity = getModelPath(model, `main.${KIT}.entity`)
-  // Pick an entity with a real op (prefer a read op) — never fabricate a
-  // `load` on an op-less entity like Cloudsmith's `Abort`. primaryOp is null
-  // only when NO entity exposes any op (a direct()-only SDK).
   const { entity: exampleEntity, primaryOp } = pickExampleEntity(entity)
   const eName = exampleEntity ? nom(exampleEntity, 'Name') : 'Entity'
   // Sanitise the local variable name — an entity whose lowercased name is a
@@ -33,9 +29,6 @@ const ReadmeHowto = cmp(function ReadmeHowto(props: any) {
   // fixture KEY (`"${eName.toLowerCase()}"`) stays raw — it must match the
   // entity's registered name for the mock lookup to resolve.
   const eVar = exampleVarName(eName.toLowerCase(), 'rb')
-  // Model-driven id key: null when the entity has no id-like field (a
-  // response-wrapped spec). When null the fixture seeds no id and a match op
-  // takes no argument.
   const idF = exampleEntity ? entityIdField(exampleEntity) : null
   const isMatchOp = 'load' === primaryOp || 'remove' === primaryOp
   const seedSentence = idF
@@ -44,8 +37,6 @@ const ReadmeHowto = cmp(function ReadmeHowto(props: any) {
   const testCtor = idF
     ? `${model.const.Name}SDK.test({\n  "entity" => { "${eName.toLowerCase()}" => { "test01" => { "${idF}" => "test01" } } },\n})`
     : `${model.const.Name}SDK.test`
-  // A type-correct argument for the primary-op call: a match hash for load/
-  // remove, a required-field body for create/update, nothing for list.
   let testCallArg = ''
   if (exampleEntity && isMatchOp) {
     const items = opRequestShape(exampleEntity, primaryOp).items

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

@@ -47,12 +47,6 @@ client = ${ctor}
     // Ruby keyword (e.g. `self`) would otherwise emit uncompilable code.
     const eVar = exampleVarName(eName.toLowerCase(), 'rb')
     const opnames = entityOps(exampleEntity)
-    // Model-driven id keys: `idF` is the load-MATCH key (null when the entity
-    // has no id-like field — a response-wrapped spec); when null, load/remove
-    // take no argument. `dataIdF` is the id on the RETURNED record's data type —
-    // an entity can key its match on an id it does not carry as data, so both a
-    // listed record's id column and a `created["id"]` read must be guarded on
-    // this, not the match key.
     const idF = entityIdField(exampleEntity)
     const dataIdF = entityDataIdField(exampleEntity)
 
@@ -136,9 +130,6 @@ end
 `)
     }
     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) =>
@@ -164,30 +155,18 @@ end
 `)
     }
 
-    // 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 Ruby-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))
       return chosen.map((it: any) => `"${it.name}" => ${rbLit(it.type, 'example_' + it.name)}`)
     }
 
-    // The id VALUE for an update/remove match: off the returned `created`
-    // record only when its data type carries the id AND a create ran, else a
-    // type-correct literal (so an update-without-create never references an
-    // undefined `created`).
     const idParamType = (opname: string): any => {
       const it = opRequestShape(exampleEntity, opname).items.find((x: any) => x.name === idF)
       return it && it.type
@@ -216,8 +195,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/rb/ReadmeRef_rb.ts

@@ -1,5 +1,5 @@
 
-import { cmp, each, Content, canonToType, canonKey, canonScalarKey, File, isAuthActive, entityIdField, opRequestShape, safeVarName, exampleVarName } from '@voxgig/sdkgen'
+import { cmp, each, Content, canonToType, canonKey, canonScalarKey, File, isAuthActive, entityIdField, opRequestShape, safeVarName, exampleVarName , targetFeatures } from '@voxgig/sdkgen'
 import { ReadmeRefFeatures } from '@voxgig/sdkgen'
 
 import {
@@ -55,7 +55,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)
 
@@ -197,7 +197,6 @@ ${eVar} = client.${ent.Name}
 `)
 
 
-      // Field schema
       if (fields.length > 0) {
         Content(`### Fields
 
@@ -244,7 +243,6 @@ ${eVar} = client.${ent.Name}
       }
 
 
-      // Operation details
       if (opnames.length > 0) {
         Content(`### Operations
 
@@ -260,7 +258,6 @@ ${info.desc}
 
 `)
 
-          // Show example
           if ('load' === opname || 'remove' === opname) {
             // The id key plus every REQUIRED match key (parent path params
             // like page_id) — the same shape the runtime resolves path
@@ -329,7 +326,6 @@ ${updateLines}  # Fields to update
       }
 
 
-      // Common methods
       Content(`### Common Methods
 
 #### \`data_get -> Hash\`
@@ -361,7 +357,6 @@ Return the entity name.
     })
 
 
-    // Features section
     const activeFeatures = each(feature).filter((f: any) => f.active)
     if (activeFeatures.length > 0) {
       Content(`
@@ -397,9 +392,6 @@ client = ${model.const.Name}SDK.new({
 \`\`\`
 
 `)
-      // 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/rb/ReadmeTopQuick_rb.ts

@@ -47,8 +47,6 @@ client = ${ctor}
     // Ruby keyword (e.g. `self`) would otherwise emit uncompilable code.
     const eVar = exampleVarName(eName.toLowerCase(), 'rb')
     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
@@ -62,9 +60,6 @@ puts ${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/rb/ReadmeTopTest_rb.ts

@@ -8,7 +8,6 @@ import {
 } from '@voxgig/apidef'
 
 
-// A type-correct Ruby literal for a field's canonical type.
 function rbLit(type: any): string {
   const k = canonScalarKey(type)
   if ('INTEGER' === k || 'NUMBER' === k) return '1'
@@ -24,8 +23,6 @@ const ReadmeTopTest = cmp(function ReadmeTopTest(props: any) {
 
   const entity = getModelPath(model, `main.${KIT}.entity`)
 
-  // Pick an entity with a real op (prefer a read op) — never fabricate a
-  // `load` on an op-less entity like Cloudsmith's `Abort`.
   const { entity: exampleEntity, primaryOp } = pickExampleEntity(entity)
 
   if (exampleEntity && primaryOp) {

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

@@ -13,21 +13,6 @@ import {
 } from '@voxgig/apidef'
 
 
-// THE GENERATED SCHEMA MODULE: the model's schemas, as data the SDK can run.
-//
-// The ruby 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 — built by the shared helpers,
-// so what ruby validates against and what ts validates against cannot drift.
-//
-// EMBEDDED AS JSON, PARSED AT LOAD, the mechanism config.rb already uses for
-// the model above the size threshold, and SINGLE-quoted for the reason
-// recorded there: a double-quoted ruby string would interpolate any `#{` the
-// model happens to contain.
-//
-// 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.
 const Schema = cmp(async function Schema(props: any) {
   const ctx$ = props.ctx$
   const target = props.target

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

@@ -250,8 +250,6 @@ ${listSkipBlock}    client = setup[:client]
     }
 
     if (hasLoad && loadPoint) {
-      // Skip live direct-load only when we can't fill path params:
-      // no spec examples and no list-bootstrap. Spec examples win first.
       const loadSkipBlock = (loadParams.length > 0 && !loadAllHaveExamples)
         ? `    if setup[:live]
       skip "live direct-load needs real ID — set *_ENTID env var with real IDs to run"

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

@@ -68,28 +68,6 @@ function formatRubyValue(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/rust/Config_rust.ts

@@ -52,34 +52,8 @@ const Config = cmp(async function Config(props: any) {
   let baseUrl = ''
   try { baseUrl = getModelPath(model, `main.${KIT}.info.servers.0.url`) } catch (_e) { }
 
-  // The canonical config OBJECT and its JSON, from the shared helper. Both
-  // representations render from the same `def`, so they cannot describe
-  // different configs - and this target picks up `options.server` (the
-  // OpenAPI server-variable defaults), which the hand-rolled build here
-  // omitted entirely. Passing target.name opts into the main
-  // slug/version/target identity fields (station descriptor input, mirrors
-  // Config_ts) - both reps below render from this same def, so the data
-  // and literal branches pick the fields up together.
   const { def: config, json: baseJson } = configDefinition(model, target.name)
 
-  // `in` and `name` TRAVEL WITH THE PREFIX NOW.
-  //
-  // apidef resolved both from the spec's securityScheme all along and this
-  // generator dropped them, so an apiKey-in-query API got an `authorization`
-  // header it does not read (see PrepareAuth_rust). The generated
-  // prepare_auth is built from the same two values, and the config carries
-  // them so the placement is a visible, overridable runtime option beside
-  // auth.prefix - which is also what keeps the SDK from rejecting its own
-  // config: the auth optspec in tm/rust/utility/make_options.rs is a CLOSED
-  // map, and an undeclared key there fails validate with "Unexpected keys at
-  // field auth".
-  //
-  // Emitted ONLY when they differ from header/Authorization, so a
-  // header-based SDK's config.rs is byte-identical to what it generated
-  // before. Both representations are patched through the one object, so the
-  // literal (formatRustValue) and the data rep (rustRawString) cannot
-  // disagree - the json is re-derived from the def, exactly as
-  // configDefinition derives it.
   const authIn = resolveAuthIn(model)
   const authName = resolveAuthName(model)
   let configJson = baseJson
@@ -103,15 +77,6 @@ const Config = cmp(async function Config(props: any) {
 
   File({ name: 'config.' + target.ext }, () => {
 
-    // ABOVE THE THRESHOLD: emit the model as DATA.
-    //
-    // The literal is one deeply nested expression. rustc type-checks and
-    // monomorphises it as a single item, so compile time and memory grow with
-    // the whole model at once; a string constant is one token, and json_parse
-    // builds the same Value tree at runtime.
-    //
-    // No number-type question here, unlike Go: `Value::Num` is f64 in both
-    // representations, so there is nothing for the two paths to disagree about.
     if (asData) {
       Content(`// Generated API configuration (mirrors go core/config.go).
 

package/project/.sdk/src/cmp/rust/EntityTypes_rust.ts

@@ -1,28 +1,5 @@
 
 
-// Typed-model generator (Rust target). Port of EntityTypes_go.ts.
-//
-// Reads main.<KIT>.entity.<e>.fields[] and per-op params
-// (op.<name>.points[].args.params[]) and emits one file, entity/types.rs, with
-// a rust `struct <Name> { ... }` per entity plus a request/match struct per
-// active op. Field/param sentinels ($STRING, $INTEGER, ...) map to rust types
-// via the SHARED canonToType 'rust' column (the single source of truth per
-// language — do not keep a local table here). Note OBJECT renders as
-// std::collections::HashMap<String, Value> (fully qualified, no import
-// needed).
-//
-// DESIGN NOTE (rust specifics vs the go reference):
-//   * The rust runtime is fully DYNAMIC: every op takes and returns the
-//     vendored `Value` enum (Result<Value, ...Error>), and the op fragments
-//     emit no typed wrappers. Go additionally emits `LoadTyped`/`ListTyped`
-//     wrappers plus json round-trip helpers; rust has no serde/json-derive
-//     dependency (only `ureq`), so these typed models are DOCUMENTARY: they
-//     mirror the entity/op shapes for reference and IDE support, but are not
-//     wired into the runtime. They compile as part of the crate (declared in
-//     entity/mod.rs) so they stay in sync with the model.
-//   * Optional (req:false) member -> `Option<T>`.
-//   * The module carries broad `#![allow(...)]` since nothing consumes the
-//     types yet (parity intent, not dead-code churn).
 
 import {
   cmp, each,
@@ -52,8 +29,6 @@ function fieldLine(name: string, sentinel: any, optional: boolean): string {
 }
 
 
-// Emit a struct named `typeName` from {name, type, optional} items, dropping
-// duplicate rust identifiers (two model names can collapse to one ident).
 function emitStruct(comment: string, typeName: string, items: any[]): void {
   Content(`${comment}
 #[derive(Debug, Clone)]
@@ -82,14 +57,8 @@ const EntityTypes = cmp(function EntityTypes(props: any) {
   // 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 an entity file (filter on `name`, always
-  // present; derive `Name` here so the struct set is deterministic — parity
-  // with the go emitter's fix).
   const entityList = deriveEntityNames(entity)
 
-  // 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)
 
   Folder({ name: 'entity' }, () => {
@@ -116,15 +85,12 @@ use crate::utility::voxgigstruct::Value;
         const fields = (ent.fields ? each(ent.fields) : [])
           .filter((f: any) => f.active !== false)
 
-        // Entity data model: one field per model field. req:false -> Option.
         emitStruct(
           `/// ${Name} is the typed data model for the ${ent.name} entity.`,
           Name,
           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).
         const ops = ent.op || {}
         ;['load', 'list', 'create', 'update', 'remove'].forEach((opname: string) => {
           if (null == ops[opname]) {

package/project/.sdk/src/cmp/rust/Entity_rust.ts

@@ -32,19 +32,12 @@ const Entity = cmp(function Entity(props: any) {
 
   const ff = Path.normalize(__dirname + '/../../../src/cmp/rust/fragment/')
 
-  // Entity files go to entity/ as one module per entity.
   Folder({ name: 'entity' }, () => {
 
     File({ name: rustVarName(entity.name) + '.' + target.ext }, () => {
 
       const opnames = Object.keys(entity.op || {})
 
-      // For each CRUD op: if the spec defines it, splice in the real
-      // implementation. Otherwise emit a stub that satisfies the static
-      // ProjectNameEntity trait (so the crate compiles) but errors at
-      // runtime if the caller invokes an unsupported op.
-      // Tag-form keys (`#LoadOp`) match the marker line `// #LoadOp` and
-      // capture its indent, so the spliced method nests inside the impl.
       const opfrags =
         (['load', 'list', 'create', 'update', 'remove']
           .reduce((a: any, opname: string) =>
@@ -74,8 +67,6 @@ const Entity = cmp(function Entity(props: any) {
           EntityName: entity.Name,
           entityname: entity.name,
 
-          // Class/constructor tokens are decoupled from the EntityName
-          // data-type token so the class can be renamed independently.
           EntyClass: cls,
 
           // Matches every `// #<Stage>-Hook` marker in the fragment (all of