@voxgig/sdkgen 4.20.0 → 4.22.0

package/project/.sdk/src/cmp/js/PrepareAuth_js.ts

@@ -17,53 +17,17 @@ 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/js/src/utility/PrepareAuthUtility.js` that
-// hardcoded `authorization` and a header. 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.
-//
-// Ported from PrepareAuth_ts, which is the proven shape. The differences
-// are the language's, not the design's: no type import, `module.exports`
-// instead of `export {}`, and the prefix read inline off `options.auth`
-// exactly as the js template read it, rather than hoisted into a local
-// that the query and cookie bodies would leave unused.
 const PrepareAuth = cmp(async function PrepareAuth(props: any) {
   const { target } = props
   const { model } = props.ctx$
 
   const active = isAuthActive_js(model)
   const where = resolveAuthIn(model)
-  // LOWERCASED FOR A HEADER, VERBATIM OTHERWISE. The js runtime keys its
-  // header map in lower case — the old template hardcoded 'authorization',
-  // the shared corpus asserts ctx:spec:headers:authorization, and every
-  // generatedcompile probe reads the lower-case key. apidef writes
-  // name: "Authorization", so emitting it verbatim hides the credential
-  // under a key nothing reads. Query parameters and cookies ARE
-  // case-sensitive and keep the spec's spelling.
   const resolvedName = resolveAuthName(model)
   const name = 'header' === where ? resolvedName.toLowerCase() : resolvedName
   const prefix = resolveAuthPrefix(model)
   const basic = isHttpBasicAuth(model)
 
-  // No `src` folder here: Main_js already opened it, and Config, Schema,
-  // SdkError, EntityBase and EntityTypes all write straight into it. The
-  // template this replaces lived at `tm/js/src/utility/`, so relative to
-  // that open folder the file needs `utility` and nothing more. Opening a
-  // second `src` would put it at src/src/utility/ — where nothing imports
-  // it, and the stale copy at src/utility/ keeps being used.
   Folder({ name: 'utility' }, () => {
     File({ name: 'PrepareAuthUtility.' + target.ext }, () => {
       Content(render({ active, where, name, prefix, basic }))
@@ -76,8 +40,6 @@ function render(spec: {
   active: boolean, where: string, name: string, prefix: string, basic: boolean
 }): string {
 
-  // NO AUTH AT ALL. A public API's SDK gets a prepareAuth that is honest
-  // about it rather than one that deletes a header nobody set.
   if (!spec.active) {
     return `
 // This API declares no authentication, so there is no credential to
@@ -193,16 +155,10 @@ function clear(where: string): string {
 
 function place(where: string): string {
   if ('query' === where) {
-    // 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.
     return `    setprop(query, CRED_name, apikey)`
   }
 
   if ('cookie' === where) {
-    // APPEND, never clobber: the request may already carry a session
-    // cookie set from options.headers, and a cookie header holds a
-    // '; '-joined list of pairs.
     return `    const existing = getprop(headers, 'cookie', '')
     const pair = CRED_name + '=' + apikey
     setprop(headers, 'cookie', existing ? existing + '; ' + pair : pair)`
@@ -224,19 +180,6 @@ export {
 }
 
 
-// NOT `isAuthActive`, AND THE DIFFERENCE IS LOAD-BEARING. That helper is
-// also false whenever the SPEC declares no security scheme
-// (`main.kit.info.auth: false`) — a statement about the DEFINITION, not a
-// ban on ever sending a credential. `optspec` still declares `apikey` and
-// makeOptions fills `options.auth` from its defaults, so the runtime
-// `options.auth == null` guard never fired and those SDKs have always sent
-// the credential. Gating the body on `isAuthActive` does not trim dead
-// code, it removes working authentication — which
-// `js: auth null suppresses the credential` catches, and which takes the
-// secrets feature down with it.
-//
-// `main.kit.config.auth.active: false` is the project saying "no credential,
-// ever", and it is the only signal that can be honoured before runtime.
 function isAuthActive_js(model: any): boolean {
   const auth = getModelPath(model, `main.${KIT}.config.auth`,
     { only_active: false, required: false })

package/project/.sdk/src/cmp/js/ReadmeEntity_js.ts

@@ -9,14 +9,6 @@ import {
 import { exampleValue } from './utility_js'
 
 
-// Operation method spelling differs between Go and other languages — Go
-// uses PascalCase methods with explicit ctrl arg, others use lowercase
-// methods with optional ctrl. The op descriptions are language-agnostic.
-// 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('ts', ent, 'list', idF, idLiteral(ent, 'list', idF))
@@ -56,7 +48,6 @@ const ReadmeEntity = cmp(function ReadmeEntity(props: any) {
     const fields = entity.fields || []
     // Model-driven id key: null when this entity has no id-like field.
     const idF = entityIdField(entity)
-    // Variable-safe lowercase name (a `Delete` entity must not bind `delete`).
     const eVar = exampleVarName(entity.name, 'js')
 
     Content(`

package/project/.sdk/src/cmp/js/ReadmeHowto_js.ts

@@ -32,8 +32,6 @@ const ReadmeHowto = cmp(function ReadmeHowto(props: any) {
     if (!exampleEntity || !primaryOp) return ''
     if ('list' === primaryOp) return ''
     if (isMatchOp) {
-      // Every REQUIRED match key (id first), not just idF — a composite-match
-      // entity (database_id + id) needs them all. Mirrors ReadmeTopTest.
       const items = opRequestShape(exampleEntity, primaryOp).items
         .filter((it: any) => !it.optional || it.name === idF)
         .sort((a: any, b: any) => (a.name === idF ? 0 : 1) - (b.name === idF ? 0 : 1))

package/project/.sdk/src/cmp/js/ReadmeQuick_js.ts

@@ -10,11 +10,6 @@ import {
 import { exampleValue } from './utility_js'
 
 
-// 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('ts', ent, 'list', idF, idLiteral(ent, 'list', idF))
@@ -26,7 +21,6 @@ const ReadmeQuick = cmp(function ReadmeQuick(props: any) {
 
   const entity = getModelPath(model, `main.${KIT}.entity`)
 
-  // Find the first published entity for examples
   const exampleEntity = Object.values(entity).find((e: any) => e.active !== false) as any
 
   const ctor = isAuthActive(model)
@@ -47,8 +41,6 @@ const client = ${ctor}
   if (exampleEntity) {
     const eName = nom(exampleEntity, 'Name')
     const article = /^[aeiou]/i.test(eName) ? 'an' : 'a'
-    // ACTIVE ops only — an inactive op generates no method, so an example
-    // calling it would be wrong.
     const opnames = entityOps(exampleEntity)
 
     // Model-driven example fields, in parity with the ts target: derive the
@@ -57,20 +49,13 @@ const client = ${ctor}
     // `idF` is the entity's id-like field name, or null when it has none —
     // then load/remove match on no argument and update omits the id.
     const idF = entityIdField(exampleEntity)
-    // Variable-safe lowercase name (a `Delete` entity must not bind `delete`).
     const eVar = exampleVarName(exampleEntity.name, 'js')
     const exampleFields = (opname: string): string[] => {
-      // ids are rendered separately as the match key for update/remove; a
-      // REQUIRED id stays for create (dropping it makes the payload
-      // incomplete).
       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 for parity with the typed ts target;
-      // 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))

package/project/.sdk/src/cmp/js/ReadmeRef_js.ts

@@ -1,5 +1,5 @@
 
-import { cmp, each, Content, canonToType, File, isAuthActive, entityIdField, opRequestShape, safeVarName, exampleVarName, jsKey } from '@voxgig/sdkgen'
+import { cmp, each, Content, canonToType, File, isAuthActive, entityIdField, opRequestShape, safeVarName, exampleVarName, jsKey , targetFeatures } from '@voxgig/sdkgen'
 import { ReadmeRefFeatures } from '@voxgig/sdkgen'
 
 import {
@@ -44,7 +44,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)
 
@@ -185,7 +185,6 @@ Alias for \`${model.Name}SDK.test()\`.
       // Model-driven id key: null when this entity has no id-like field, in
       // which case load/remove match on no argument and update omits the id.
       const idF = entityIdField(ent)
-      // Variable-safe lowercase name (a `Delete` entity must not bind `delete`).
       const eVar = exampleVarName(ent.name, 'js')
 
       Content(`
@@ -208,7 +207,6 @@ const ${eVar} = client.${ent.Name}()
 `)
 
 
-      // Field schema
       if (fields.length > 0) {
         Content(`### Fields
 
@@ -406,9 +404,6 @@ const client = new ${model.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/js/ReadmeTopQuick_js.ts

@@ -10,11 +10,6 @@ import {
 import { exampleValue } from './utility_js'
 
 
-// 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('ts', ent, 'list', idF, idLiteral(ent, 'list', idF))
@@ -41,8 +36,6 @@ const client = ${ctor}
   if (exampleEntity) {
     const eName = nom(exampleEntity, 'Name')
     const eVar = exampleVarName(eName.toLowerCase(), 'js')
-    // ACTIVE ops only — an inactive op generates no method, so an example
-    // calling it would be wrong.
     const opnames = entityOps(exampleEntity)
 
     if (opnames.includes('list')) {

package/project/.sdk/src/cmp/js/Schema_js.ts

@@ -12,27 +12,6 @@ import {
 } from '@voxgig/apidef'
 
 
-// THE GENERATED SCHEMA MODULE: the model's schemas, as data the SDK can run.
-//
-// Two exports, both struct.validate specs:
-//
-//   OPTSPEC     what makeOptions validates the caller's options against —
-//               the standard options from `main.kit.optspec` plus one entry
-//               per feature this target carries, built by helpers/optspec.
-//
-//   ENTITYSPEC  per entity, `{ data, op: { <opname> } }` — the record shape
-//               and each operation's request shape, mapped from the field
-//               type sentinels the model already carries (helpers/canonSpec).
-//               Emitted as an EMPTY MAP unless the `validate` feature is
-//               active, because nothing else reads it and every byte here is
-//               a byte in the consumer's package.
-//
-// A JSON OBJECT LITERAL, not a parsed string: JSON is a subset of the
-// language's own literal syntax, so the values land as data with no parse
-// step, and the backticks the sentinels carry (`\`$STRING\``) survive because
-// JSON.stringify quotes and escapes them. The config emitter has to choose
-// between a literal and a parsed blob because the config can be megabytes;
-// the spec is bounded by the model's field count, so it does not.
 const Schema = cmp(async function Schema(props: any) {
   const ctx$ = props.ctx$
   const target = props.target

package/project/.sdk/src/cmp/js/TestDirect_js.ts

@@ -22,6 +22,7 @@ import {
   jsProp, envName, envToken,
   jsKey,
   pointParts,
+  hasLiveScenarios,
 } from '@voxgig/sdkgen'
 
 
@@ -52,16 +53,6 @@ const TestDirect = cmp(function TestDirect(props: any) {
       apikey: env.${PROJECTNAME}_APIKEY,`
     : ''
 
-  // A templated server URL (OpenAPI server variables) makes a LIVE client
-  // impossible to construct without values: makeOptions raises rather than
-  // request a URL with a literal `{account_id}` in it. Taken from the
-  // environment, the same way the apikey is.
-  //
-  // Keys are quoted and the env read is bracketed via jsKey/jsProp: a server
-  // variable name is spec-derived and need not be a JS identifier — the URL
-  // grammar admits a leading digit ({2fa}), and a declared-but-unreferenced
-  // variable ({edge-zone}) is not constrained at all. Bare `name:` and
-  // `env.PROJ_SERVER_EDGE-ZONE` are both syntax errors.
   const svars = serverVariables(model)
   const serverEnvEntry = svars
     .map((v: any) => `\n    '${serverVarEnv(PROJECTNAME, v.name)}': ${JSON.stringify(v.dflt)},`).join('')
@@ -96,7 +87,7 @@ const TestDirect = cmp(function TestDirect(props: any) {
 
         Slot({ name: 'directSetup' }, () => {
           Content(`
-function liveScenariosActive() { return ${Object.values(model.main.kit.entity || {}).some((e: any) => Object.values(e.op || {}).some((o: any) => (o.points || []).some((p: any) => p.contract && JSON.parse(p.contract.json).live)))} && process.env.${PROJECTNAME}_TEST_LIVE === 'TRUE' }
+function liveScenariosActive() { return ${hasLiveScenarios(model)} && process.env.${PROJECTNAME}_TEST_LIVE === 'TRUE' }
 function directSetup(mockres) {
   const calls = []
 
@@ -389,11 +380,6 @@ ${varAsserts}    }
 }
 
 
-// Replace raw OpenAPI parameter names in path parts with model parameter names.
-// Path parts may have e.g. {subBreed} while model params use sub_breed.
-// When a rename mapping exists (e.g. closureId -> id), path parts contain the
-// renamed form {id} but params still use the original name closure_id.
-// The rename mapping is used to reverse-lookup the original param name.
 function normalizePathParams(
   parts: string[],
   params: any[],

package/project/.sdk/src/cmp/js/TestEntity_js.ts

@@ -34,7 +34,8 @@ import {
   serverVariables,
   entityDataIdField, envName, envToken,
   jsKey,
-  jsProp
+  jsProp,
+  hasLiveScenarios,
 } from '@voxgig/sdkgen'
 
 
@@ -74,16 +75,6 @@ const TestEntity = cmp(function TestEntity(props: any) {
         apikey: env.${PROJENVNAME}_APIKEY,`
     : ''
 
-  // A templated server URL (OpenAPI server variables) makes a LIVE client
-  // impossible to construct without values: makeOptions raises rather than
-  // request a URL with a literal `{account_id}` in it. So the live suite
-  // takes them from the environment the same way it takes the apikey.
-  //
-  // Keys are quoted and the env read is bracketed via jsKey/jsProp: a server
-  // variable name is spec-derived and need not be a JS identifier — the URL
-  // grammar admits a leading digit ({2fa}), and a declared-but-unreferenced
-  // variable ({edge-zone}) is not constrained at all. Bare `name:` and
-  // `env.PROJ_SERVER_EDGE-ZONE` are both syntax errors.
   const svars = serverVariables(model)
   const serverEnvEntry = svars
     .map((v: any) => `\n    '${serverVarEnv(PROJENVNAME, v.name)}': ${JSON.stringify(v.dflt)},`).join('')
@@ -93,7 +84,6 @@ const TestEntity = cmp(function TestEntity(props: any) {
           ${jsKey(v.name)}: ${jsProp('env', serverVarEnv(PROJENVNAME, v.name))},`).join('')}
         },`
 
-  // TODO: should be a utility function
   const ff = projectPath('src/cmp/js/fragment/')
 
   Folder({ name: entity.name }, () => {
@@ -219,7 +209,7 @@ function basicSetup(extra) {
           )
 
           Content(`
-    ${Object.values(model.main.kit.entity || {}).some((e: any) => Object.values(e.op || {}).some((o: any) => (o.points || []).some((p: any) => p.contract && JSON.parse(p.contract.json).live))) ? `if (process.env.${PROJENVNAME}_TEST_LIVE === 'TRUE') { t.skip('Covered by live operation scenarios'); return }` : ''}
+    ${hasLiveScenarios(model) ? `if (process.env.${PROJENVNAME}_TEST_LIVE === 'TRUE') { t.skip('Covered by live operation scenarios'); return }` : ''}
     const setup = basicSetup()
     if (setup.live) {
       return runLiveEntity(setup, ${JSON.stringify(entity)}, ${JSON.stringify(basicflow)}, '${nom(entity, 'Name')}')

package/project/.sdk/src/cmp/js/TestLive_js.ts

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

package/project/.sdk/src/cmp/js/utility_js.ts

@@ -15,35 +15,12 @@ import {
 } from '@voxgig/struct'
 
 
-// --- Model-driven example literals -----------------------------------------
-// Doc snippets must use example values whose TYPE matches the generated
-// types, or the snippet misrepresents the API. The id/match params of
-// load/remove/update are the common trap: their generated
-// `<Name><Op>Match` / `<Name><Op>Data` type is built from the op's params
-// (see EntityTypes_js.ts), so a numeric id shown as a quoted string is
-// wrong. These helpers derive the example literal from the SAME model
-// source, so the docs and the generated types can never disagree.
 
 // The declared canon-type sentinel of a named parameter of an op — looked up
 // in the op's `points[].args.params[]` exactly as the typed-model generator
 // does. Falls back to the entity field of the same name (used when the op
 // has no params). Returns undefined when neither is present.
 function paramCanonType(entity: any, op: any, paramName: string): unknown {
-  // opParams, NOT a raw walk of op.points.
-  //
-  // opParams drops points flagged with select['$action'] and merges what is
-  // left; the typed-model generator reaches the op's params through it, so
-  // anything else is a DIFFERENT set of params wearing the same name.
-  //
-  // A raw walk returned the first match on ANY point, action points included.
-  // github's `action` entity has a field `owner` (`$OBJECT`, "A GitHub user")
-  // and seven create points whose path carries `{owner}` (a string) — so the
-  // type said Record<string, any> and the doc example said 'example_owner',
-  // and ts/README.md stopped compiling. Same for workflow_id: `number` in the
-  // type, quoted string in the example.
-  //
-  // Deriving from opParams is what actually makes good on the promise below —
-  // that the docs and the generated types cannot disagree.
   const params = op ? each(opParams(op)) : []
   const found = (params as any[]).find((p: any) => p && p.name === paramName)
   if (found) {
@@ -55,11 +32,6 @@ function paramCanonType(entity: any, op: any, paramName: string): unknown {
 }
 
 
-// A type-correct example literal for a named match/data parameter of an op,
-// derived entirely from the model. INTEGER/NUMBER render as the bare number
-// `1`, BOOLEAN as `true`, ARRAY as the empty `[]` and OBJECT as the empty
-// `{}`, everything else (STRING, unknown, missing) as the quoted
-// `placeholder`.
 function exampleValue(entity: any, op: any, paramName: string, placeholder: string): string {
   // canonScalarKey, not canonKey: a nullable field's sentinel is the union
   // ['`$ONE`', ['`$NUMBER`','`$NULL`']], which canonKey stringifies into
@@ -105,17 +77,14 @@ function formatJson(obj: any, flags?: { line?: boolean, margin?: number }): stri
   let json: string
 
   if (flags?.line) {
-    // One line with spaces for clarity
     json = JSON.stringify(obj)
       .replace(/([{:\[,])/g, '$1 ')
       .replace(/([}\]])/g, ' $1')
   }
   else {
-    // Pretty printed with 2-space indentation
     json = JSON.stringify(obj, null, 2)
   }
 
-  // Add margin to the left of every line
   if (marginSize > 0) {
     json = json.split('\n').map(line => marginStr + line).join('\n')
   }
@@ -124,41 +93,14 @@ function formatJson(obj: any, flags?: { line?: boolean, margin?: number }): stri
 }
 
 
-// Emission-time normalisation of a model subtree (L0).
-//
-// Always drops jostraca's iteration metadata (`$`-suffixed keys: index$,
-// key$, val$). With `dropDefaults`, also drops keys whose value IS the
-// default the runtime already assumes when the key is absent, which is pure
-// payload — see CONFIG_DEFAULT.
-//
-// Rebuilds the tree rather than mutating during a walk. The previous
-// implementation walked a clone calling `delete p[k]`, but walk() assigns its
-// callback's result back over the child (`setprop(out, ckey, walk(...))`), so
-// the delete was undone on the way out and the helper silently did nothing.
-// Returning `undefined` from the callback does not fix it either: setprop
-// stores undefined rather than removing the key, which then emits as a null.
-//
-// `dropDefaults` is opt-in and must be passed ONLY for the entity subtree.
-// `active` means something different in feature config, where absent reads as
-// INACTIVE (see feature_init) — dropping `active: true` there would silently
-// disable the feature.
-// jostraca's iteration metadata, injected by each()/names() while it walks the
-// model. Listed explicitly rather than matched by trailing-dollar suffix: a
-// trailing dollar is not exclusive to jostraca -- Seneca uses entity$ as real
-// data -- so a blanket suffix match can silently drop a legitimate API field.
 const MODEL_META = ['index$', 'key$', 'val$']
 
-// Keys whose value IS the default the runtime already assumes when the key is
-// absent, so emitting them is pure payload.
 const CONFIG_DEFAULT: Record<string, any> = {
   active: true,
   req: false,
   reqd: false,
 }
 
-// Subtrees carrying user payload rather than schema. An active:true inside an
-// OpenAPI example is DATA, not a default, so default-pruning stops at these
-// keys and everything below them is passed through untouched.
 const PAYLOAD_KEYS = ['default', 'example', 'examples']
 
 function clean(o: any, dropDefaults?: boolean): any {

package/project/.sdk/src/cmp/kotlin/Config_kotlin.ts

@@ -40,9 +40,6 @@ const Config = cmp(async function Config(props: any) {
   const kotlinpackage = kotlinPackage(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`) || {}
@@ -50,24 +47,12 @@ const Config = cmp(async function Config(props: any) {
   const authActive = isAuthActive(model)
   const authPrefix = resolveAuthPrefix(model)
   const authBasic = isHttpBasicAuth(model)
-  // `in` and `name` travel with the prefix now. apidef resolved both all
-  // along, into main.kit.info.security, and this emitter dropped them - so
-  // an apiKey-in-query API got an Authorization header it does not read.
-  // The generated utility/PrepareAuth.kt is built from the same two values.
-  //
-  // Emitted only when they DIFFER from the defaults, so a
-  // header/Authorization SDK's Config.kt is byte-identical to what it
-  // generated before.
   const authIn = resolveAuthIn(model)
   const authName = resolveAuthName(model)
 
   let baseUrl = ''
   try { baseUrl = getModelPath(model, `main.${KIT}.info.servers.0.url`) } catch (_e) { }
 
-  // Identity comes from configDefinition's def, not re-derived here, so
-  // this target cannot disagree with the shared emitter on main.slug /
-  // main.version / main.target (the three station descriptor fields,
-  // station design §4) — passing target.name is what opts this target in.
   const { def: configDef } = configDefinition(model, target.name)
 
   // The feature block comes from configDefinition's def, not from
@@ -101,48 +86,17 @@ const Config = cmp(async function Config(props: any) {
   options.headers = headers
   options.entity = optionsEntity
 
-  // configDefinition's `def.entity` verbatim, NOT rebuilt here. This reduce
-  // was one of fourteen copies of that function's entityDefs loop, and when
-  // configDefinition started reconstructing a point's `parts` from apidef's
-  // segment vector (its ADR-003), only the copies that read `configDef` got
-  // it — this target's literal config emitted paths with no parts at all
-  // while its data config had them. One rule, one place.
   const entityConfig = configDef.entity
 
-  // THE PLUGIN DEFINITIONS the model selected, per feature.
-  //
-  // sekreto's contract since the registry was retired: a kind not passed
-  // in `plugins` is unknown to that Sekreto, so the model's choice of
-  // plugin groups IS the SDK's provider vocabulary. This walks the
-  // catalogue's active `plugin.def.kotlin` entries and hands the list to
-  // the feature through `Config.featurePlugins`.
-  //
-  // Kotlin's Definition symbols are PLAIN TOP-LEVEL VALS
-  // (`val hashicorp: Definition = providerplugin("hashicorp") {...}`), so
-  // the def key is the val name and the import is that name qualified by
-  // the file's package - DERIVED from the def path (`feature/secrets/
-  // sekreto/plugins/Hashicorp.kt` -> `feature.secrets.sekreto.plugins`)
-  // rather than hardcoding `secrets`, exactly as the go emitter does.
-  //
-  // Emitted in core, which already names KOTLINPACKAGE.feature classes in
-  // makeFeature - Kotlin has no package-cycle rule, so nothing here can
-  // introduce one. The feature reads it back as List<Any?> and filters by
-  // type, so a tree with the feature present but no group selected still
-  // compiles with no import at all.
   const pluginImports = new Set<string>()
   const featurePlugins: Record<string, string[]> = {}
 
   each(feature, (f: any) => {
     const syms: string[] = []
     each(f.plugin, (plugin: any) => {
-      // Filter on `active` HERE rather than trusting the feature object to
-      // arrive filtered: getting this wrong emits an import for a file the
-      // plugin trim just deleted, and the SDK does not compile.
       if (false === plugin.active || null == plugin.active) return
       for (const [sym, one] of Object.entries(plugin.def?.kotlin || {})) {
         const dir = String(one).replace(/\/[^/]+$/, '').replace(/\//g, '.')
-        // A dotted symbol (another language's class-qualified spelling)
-        // imports its HEAD; kotlin's own keys are bare val names.
         pluginImports.add(
           kotlinpackage + '.' + dir + '.' + String(sym).split('.')[0])
         syms.push(sym)

package/project/.sdk/src/cmp/kotlin/EntityOperation_kotlin.ts

@@ -29,8 +29,6 @@ const EntityOperation = cmp(function Operation(props: any) {
       EntityName: entity.Name,
       entityname: entity.name,
 
-      // Class tokens are decoupled from the EntityName data-type token so the
-      // class can be renamed independently (collision handling).
       EntyClass: cls,
 
       '#Feature-Hook': ({ name, indent }: any) =>