@voxgig/sdkgen 4.20.0 → 4.22.0

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

@@ -1,5 +1,5 @@
 
-import { cmp, each, Content, canonToType, canonKey, canonScalarKey, File, isAuthActive, entityIdField, opRequestShape } from '@voxgig/sdkgen'
+import { cmp, each, Content, canonToType, canonKey, canonScalarKey, File, isAuthActive, entityIdField, opRequestShape , targetFeatures } from '@voxgig/sdkgen'
 import { ReadmeRefFeatures } from '@voxgig/sdkgen'
 
 import {
@@ -29,7 +29,7 @@ const ReadmeRef = cmp(function ReadmeRef(props: any) {
 
   const rustcrate = crateIdent(model)
   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)
 
@@ -374,9 +374,6 @@ let client = ${model.const.Name}SDK::new(jo(vec![
 \`\`\`
 
 `)
-      // 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/rust/Schema_rust.ts

@@ -0,0 +1,89 @@
+import {
+  Content,
+  File,
+  cmp,
+  entitySpecMap,
+  optionSpec,
+} from '@voxgig/sdkgen'
+
+
+import {
+  Model,
+} from '@voxgig/apidef'
+
+
+import {
+  rustRawString,
+} from './utility_rust'
+
+
+const Schema = cmp(async function Schema(props: any) {
+  const ctx$ = props.ctx$
+  const target = props.target
+
+  const model: Model = ctx$.model
+
+  const optspec = optionSpec(model, target.name)
+  const entityspec = entitySpecMap(model, target.name) || {}
+
+  // Beside config.rs, in core, and for the same reason: core owns the
+  // generated data, so nothing in utility or feature has to reach up for it.
+  File({ name: 'schema.' + target.ext }, () => {
+
+    Content(`// ${model.const.Name} SDK: generated schemas. Do not edit.
+//
+// Built from the model: \`main.kit.optspec\` and each feature's
+// \`config.options\` for the option spec; entity \`fields[].type\` for the
+// entity specs.
+
+use crate::utility::jsonparse::json_parse;
+use crate::utility::voxgigstruct::Value;
+
+const OPTSPEC_DATA: &str = ${rustRawString(JSON.stringify(optspec))};
+
+const ENTITYSPEC_DATA: &str = ${rustRawString(JSON.stringify(entityspec))};
+
+/// The option spec make_options validates client options against.
+///
+/// Unreachable on error: the constant is generated by a JSON serialiser and
+/// never edited by hand. Panicking beats returning an empty spec, which would
+/// validate everything and fail far from the cause.
+pub fn make_optspec() -> Value {
+    json_parse(OPTSPEC_DATA).expect("${model.const.Name}: embedded option spec is not valid JSON")
+}
+
+/// Per-entity data and request specs, keyed by entity name.
+pub fn make_entityspec() -> Value {
+    json_parse(ENTITYSPEC_DATA).expect("${model.const.Name}: embedded entity spec is not valid JSON")
+}
+
+// SHARED SPECS, the shape core::config::shared_config uses and for the same
+// reasons: the spec is read on every client construction and never mutated,
+// so a per-call parse would be pure waste — and THREAD-LOCAL rather than
+// global, because Value is Rc/RefCell-backed and so is neither Send nor Sync.
+// The clone is an Rc bump, not a deep copy.
+//
+// The returned Value SHARES its nodes: treat it as read-only. make_options
+// validates AGAINST the spec and writes into the options, never into the spec.
+thread_local! {
+    static SHARED_OPTSPEC: Value = make_optspec();
+    static SHARED_ENTITYSPEC: Value = make_entityspec();
+}
+
+/// The per-thread option spec, built once on first use.
+pub fn optspec() -> Value {
+    SHARED_OPTSPEC.with(|s| s.clone())
+}
+
+/// The per-thread entity specs, built once on first use.
+pub fn entityspec() -> Value {
+    SHARED_ENTITYSPEC.with(|s| s.clone())
+}
+`)
+  })
+})
+
+
+export {
+  Schema
+}

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

@@ -76,13 +76,6 @@ const TestDirect = cmp(function TestDirect(props: any) {
     ? `("apikey", getp(&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. So the live suite
-  // takes them from the environment the same way it takes the apikey.
-  //
-  // Emitted as a jo() tuple entry, so it carries its own leading comma only
-  // when an apikey entry precedes it - otherwise `jo(vec![, ...])`.
   const svars = serverVariables(model)
   const serverEnvEntry = svars
     .map((v: any) => `\n        ("${serverVarEnv(PROJECTNAME, v.name)}", Value::str(${JSON.stringify(v.dflt)})),`).join('')

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

@@ -26,8 +26,6 @@ import {
 import { rustVarName, rustMethodName } from './utility_rust'
 
 
-// Rust's GenCtx mirrors the shared shape (see TestEntity_ts.ts) plus the
-// crate ident used to build qualified paths in emitted code.
 type GenCtx = {
   model: Model
   entity: ModelEntity
@@ -64,13 +62,6 @@ const TestEntity = cmp(function TestEntity(props: any) {
     ? `("apikey", getp(&env, "${PROJUPPER}_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.
-  //
-  // Emitted as a jo() tuple entry, so it carries its own leading comma only
-  // when an apikey entry precedes it - otherwise `jo(vec![, ...])`.
   const svars = serverVariables(model)
   const serverEnvEntry = svars
     .map((v: any) => `\n        ("${serverVarEnv(PROJUPPER, v.name)}", Value::str(${JSON.stringify(v.dflt)})),`).join('')

package/project/.sdk/src/cmp/rust/fragment/Entity.fragment.rs

@@ -132,17 +132,6 @@ impl EntyClass {
         utility.done(ctx)
     }
 
-    /// Streaming operation. Runs `action` through the full pipeline and
-    /// returns an iterator over the result items, so the `streaming`
-    /// feature's incremental output is reachable from a generated entity (a
-    /// normal op call materialises the whole result). This runtime is
-    /// synchronous, so the returned iterator is a lazy cursor over items the
-    /// pipeline produced. `callopts` parameterises the call:
-    ///   - inbound (download): iterate items/chunks from the streaming
-    ///     feature when active, else the materialised items;
-    ///   - outbound (upload): a `body` in `callopts` is attached to the
-    ///     request (reqdata `body$`) so the transport can stream a payload;
-    ///   - `ctrl` (pipeline control) threads pipeline options.
     pub fn stream(
         &self,
         action: &str,
@@ -239,9 +228,6 @@ impl Entity for EntyClass {
         self.name.clone()
     }
 
-    // `remove` resolves to the entity, marked. The instance KEEPS the data
-    // it held — a caller can still read what was deleted — but it is no
-    // longer a live record.
     fn mark_deleted(&self) {
         *self.deleted.borrow_mut() = true;
     }

package/project/.sdk/src/cmp/rust/fragment/EntityRemoveOp.fragment.rs

@@ -31,10 +31,6 @@ fn remove(self: &Rc<Self>, reqmatch: Value, ctrl: Value) -> Result<Rc<Self>, Pro
         }
     })?;
 
-    // The operation resolves to THIS entity: `run_op` has just absorbed the
-    // result into it, and the caller reaches the record through `.data(None)`.
-    // See AGENTS.md "Entity operations return ENTITIES". A removed entity
-    // keeps its data but is no longer a live record.
     self.mark_deleted();
 
     Ok(self.clone())

package/project/.sdk/src/cmp/rust/fragment/Main.fragment.rs

@@ -57,12 +57,6 @@ impl ProjectNameSDK {
         *rootctx.options.borrow_mut() = opts.clone();
         *sdk.rootctx.borrow_mut() = Some(rootctx.clone());
 
-        // Add features in the resolved order (make_options puts an explicit
-        // List order first, else defaults to test-first). Ordering matters:
-        // the `test` feature installs the base mock transport and the
-        // transport features (retry/cache/netsim/proxy/ratelimit) wrap
-        // whatever is current, so `test` must be added before them to sit at
-        // the base of the transport wrapper chain.
         let feature_opts = to_map(&getp(&opts, "feature"));
         if let Value::List(order) = getpath(&["__derived__", "featureorder"], &opts) {
             let names: Vec<String> = order
@@ -302,18 +296,6 @@ impl ProjectNameSDK {
         ]))
     }
 
-    // Raw GraphQL access: the pressure valve that makes the generated
-    // surface's deliberate omissions (per-call selection sets, typed filter
-    // builders, batching, subscriptions) livable — the whole schema stays
-    // reachable.
-    //
-    // Thin wrapper over the same prepare/fetch path direct uses, with the one
-    // thing raw direct cannot do for GraphQL: a GraphQL failure rides HTTP
-    // 200 as a top-level `errors` array, so status alone would report a
-    // failed query as ok.
-    //
-    // NOTE: like direct, this bypasses the feature pipeline — no retry,
-    // ratelimit or paging features apply.
     pub fn graphql(
         &self, query: &str, variables: Value, ctrl: Value,
     ) -> Result<Value, ProjectNameError> {
@@ -342,7 +324,6 @@ impl ProjectNameSDK {
         // { errors: [...] } body, and the raw path represents a non-2xx as
         // ok:false with no err — so returning early on status would discard
         // the server's own diagnostics, which are the only useful part of
-        // that response.
         let errors = getpath(&["data", "errors"], &res);
 
         if let Value::List(items) = &errors {
@@ -377,7 +358,6 @@ pub fn test_sdk(testopts: Value, sdkopts: Value) -> Rc<ProjectNameSDK> {
     setp(&testopts, "active", Value::Bool(true));
 
     // set_path mutates `sdkopts` in place and returns the inner parent node;
-    // discard the return so we pass the full options root (mirrors go's
     // vs.SetPath(sdkopts, ...) which does not rebind).
     vs::set_path(
         &sdkopts,

package/project/.sdk/src/cmp/rust/fragment/SdkError.fragment.rs

@@ -1,5 +1,3 @@
-// ProjectNameError: the SDK error type (mirrors go core/error.go). The
-// pipeline error discipline is Result<T, ProjectNameError> throughout.
 
 use crate::utility::voxgigstruct::Value;
 
@@ -11,10 +9,6 @@ pub struct ProjectNameError {
     // Cleaned snapshots attached by makeError (Noval until then).
     pub result: Value,
     pub spec: Value,
-    // HTTP status of the response that caused this error, or -1 when the
-    // request never got one. PROMOTED to the top level: it used to be
-    // reachable only inside `result`, so every consumer coupled itself to
-    // the internal shape of that snapshot.
     pub status: i64,
 }
 

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

@@ -18,17 +18,6 @@ const RUST_RESERVED = new Set<string>([
 ])
 
 
-// Method names on a generated struct that must never be an entity accessor.
-//
-// An entity called `clone` generated `pub fn clone(&self, Value)` on the SDK
-// struct, which SHADOWS Clone::clone — so every `sdk.clone()` the generated
-// code makes elsewhere resolved to the entity accessor and the crate failed
-// with "this method takes 1 argument but 0 arguments were supplied", 16 times
-// over, pointing at the call sites and never at the entity that caused it.
-// `new` would collide with the struct's own constructor the same way.
-//
-// These are not Rust keywords, so RUST_RESERVED does not catch them; they are
-// inherent or std-trait methods the generated code actually calls.
 const RUST_METHOD_RESERVED = new Set<string>([
   'clone', 'new', 'default', 'drop', 'from', 'into', 'to_string',
   'as_ref', 'as_mut', 'borrow', 'borrow_mut', 'deref', 'deref_mut',
@@ -44,13 +33,8 @@ function rustMethodName(name: string): string {
 }
 
 
-// A collision-free snake_case rust identifier for a model name.
 function rustVarName(name: string): string {
   let snake = name.replace(/[^a-zA-Z0-9_]/g, '_').toLowerCase()
-  // No Rust identifier may begin with a digit. A `3d_id` field became
-  // `pub 3_d_id: String`, which does not compile ("expected identifier, found
-  // `3_d_id`") and took the whole crate with it. Prefixed with an underscore,
-  // the convention cmp/c/utility_c.ts already applies.
   if (/^[0-9]/.test(snake)) {
     snake = '_' + snake
   }
@@ -58,8 +42,6 @@ function rustVarName(name: string): string {
 }
 
 
-// The Cargo package name, e.g. voxgig-solar-sdk (mirrors the go module
-// naming: org prefix from model.origin).
 function crateName(model: any): string {
   const org = (model.origin || 'voxgig-sdk').replace(/-sdk$/, '')
   return `${org}-${model.name}-sdk`.toLowerCase().replace(/[^a-z0-9-]/g, '-')
@@ -117,29 +99,6 @@ function formatRustValue(val: any, indent: number = 0): string {
 }
 
 
-// Deep-remove meta keys (`foo$`) from a model subtree (twin of go's clean).
-// 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
@@ -180,12 +139,6 @@ function clean(o: any, dropDefaults?: boolean): any {
 
 
 
-// The JSON as a Rust RAW string literal.
-//
-// A raw string processes no escapes, so the JSON's own `\n` and `\uXXXX`
-// survive byte for byte and reach the JSON parser as written - which a normal
-// Rust string would not, having consumed them itself. The hash level is raised
-// until its terminator does not occur in the text.
 function rustRawString(s: string): string {
   let level = 0
   while (s.includes('"' + '#'.repeat(level))) {

package/project/.sdk/src/cmp/scala/Config_scala.ts

@@ -41,29 +41,10 @@ const Config = cmp(async function Config(props: any) {
 
   const entity = getModelPath(model, `main.${KIT}.entity`)
   const { def: configDef } = configDefinition(model, target.name)
-  // 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`) || {}
 
-  // PLUGIN DEFINITIONS, per feature (the scala peer of Config_go's
-  // featurePlugins map and Config_ts's pluginDefs).
-  //
-  // Since sekreto retired its import-time registry, provider kinds are
-  // voxgig/plugin definitions: a kind the caller did not pass in via
-  // `plugins` is unknown to that Sekreto. So the config names each active
-  // plugin's exported Definition and hands the list to the feature.
-  //
-  // The scala symbols are fully-qualified top-level `val`s
-  // (`com.voxgig.sekreto.plugins.hashicorp`), so unlike go there is no
-  // import to emit - and an inactive group therefore leaves no reference at
-  // all behind for the generate-time plugin trim to dangle.
-  //
-  // Typed List[Any], never List[Definition]: core must not name a vendored
-  // type, or a tree carrying the feature source without the feature selected
-  // would fail to compile.
   const featurePlugins: Record<string, string[]> = {}
   each(feature, (f: any) => {
     const syms: string[] = []
@@ -113,12 +94,6 @@ const Config = cmp(async function Config(props: any) {
   }
   if (authActive) {
     const auth: Record<string, any> = { prefix: authPrefix }
-    // ONLY WHEN THEY DIFFER FROM THE DEFAULT. `in: 'header'` and
-    // `name: 'Authorization'` are what resolveAuthIn/resolveAuthName answer
-    // for a spec that says nothing, and what the generated
-    // utility/PrepareAuth.scala assumes, so emitting them would be pure
-    // payload AND would move every existing header SDK's Config.scala. An
-    // apiKey-in-query API (joplin's `?token=`) is the case that needs them.
     if (authBasic) auth.basic = true
     if ('header' !== authIn) auth.in = authIn
     if ('Authorization' !== authName) auth.name = authName
@@ -127,12 +102,6 @@ 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
 
   const config = {

package/project/.sdk/src/cmp/scala/EntityTypes_scala.ts

@@ -1,37 +1,5 @@
 
 
-// Typed-model generator (Scala target). Port of EntityTypes_java.ts /
-// EntityTypes_py.ts.
-//
-// Reads main.<KIT>.entity.<e>.fields[] and per-op params
-// (op.<name>.points[].args.params[]) and emits ONE file, core/<Name>Types.scala,
-// holding an `object <Name>Types` with a nested `final case class` per active
-// entity plus a request/match case class per active op.
-//
-// WHY A CONTAINER OBJECT: keeps the whole reference model in one file whatever
-// the entity count and matches the java target's single-container layout
-// (<Name>Types.java). Scala permits multiple top-level types per file, but a
-// container object mirrors the JVM sibling and keeps the generated core/ dir
-// tidy.
-//
-// TYPE CHOICE: reference `case class` types with BOXED (nullable) component
-// types, NOT the wired runtime type. The generated ops take/return the loose
-// object model (java.util.Map[String, Object] / Object), so these case classes
-// are NOT wired into the op signatures — they are documentation/DX reference
-// shapes a caller may use to describe a payload before converting it to a map.
-// This mirrors the java target's reference records and the JS target's JSDoc
-// typedefs (annotation only, no runtime effect). Emitting unused case classes
-// is harmless — they compile and have no runtime effect.
-//
-// OPTIONAL FIELDS: every component is a boxed reference type, hence inherently
-// nullable, so a `req:false` field/param needs no distinct rendering in Scala.
-//
-// Sentinels map to Scala types via the SHARED canonToType 'scala' column (the
-// single source of truth per language — do not keep a local table here).
-//
-// Keep the SAME type-name scheme as every other language: <Name>,
-// <Name>LoadMatch, <Name>ListMatch, <Name>CreateData, <Name>UpdateData,
-// <Name>RemoveMatch (via the shared opTypeName helper).
 
 import {
   cmp, each, names,
@@ -62,20 +30,11 @@ const SCALA_KEYWORDS = new Set<string>([
 ])
 
 
-// A field/param name that has a safe Scala identifier rendering. Names that are
-// not valid identifiers (hyphens, leading digits) have no clean parameter form
-// and are skipped — they remain reachable via the runtime map.
 function scalaIdent(name: string): boolean {
   return /^[A-Za-z_][A-Za-z0-9_]*$/.test(name) && !SCALA_KEYWORDS.has(name)
 }
 
 
-// Emit a nested `final case class <typeName>(...)` from {name, type} items. An
-// item whose name is not a legal identifier is skipped (WITH a warning — the
-// key stays reachable via the runtime map, but its absence from the typed
-// model should be visible, not silent); a duplicate component name (after the
-// identifier filter) is dropped. An empty case class is a valid,
-// zero-parameter shape.
 function emitCaseClass(typeName: string, items: any[], log?: any): void {
   const seen = new Set<string>()
   const usable = items.filter((it: any) => {
@@ -123,10 +82,6 @@ const EntityTypes = cmp(function EntityTypes(props: any) {
 
   const scalapackage = props.scalapackage || scalaPackage(model)
 
-  // 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
@@ -138,9 +93,6 @@ const EntityTypes = cmp(function EntityTypes(props: any) {
   // entity not yet named (e.g. a fieldless placeholder) would otherwise read
   // `Name = undefined` below. Parity with the go/py/java/csharp 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.' + ext }, () => {
@@ -170,13 +122,10 @@ object ${model.const.Name}Types {
       const fields = (ent.fields ? each(ent.fields) : [])
         .filter((f: any) => f.active !== false)
 
-      // Entity data model: one component per field.
       emitCaseClass(Name, fields.map((f: any) => ({
         name: f.name, type: f.type,
       })), log)
 
-      // Per active op: a request/match case class. Members 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/scala/Main_scala.ts

@@ -23,6 +23,7 @@ import {
 
 import { Package } from './Package_scala'
 import { Config } from './Config_scala'
+import { Schema } from './Schema_scala'
 import { Gitignore } from './Gitignore_scala'
 import { MainEntity } from './MainEntity_scala'
 import { EntityBase } from './EntityBase_scala'
@@ -45,31 +46,6 @@ const Main = cmp(async function Main(props: any) {
   // The Scala package root for every runtime piece (like GOMODULE for go).
   const scalapackage = scalaPackage(model)
 
-  // THE PLUGIN TRIM, over the WHOLE model rather than its active half.
-  //
-  // pluginExcludes(model) walks only the model's ACTIVE features
-  // (helpers/featureSource), which is right for a target whose feature trim
-  // is on: an inactive feature's whole tree is already gone at `target add`,
-  // so it has no plugin files left to exclude. Scala's feature trim is OFF
-  // (see the Copy below), so the opposite holds - an inactive feature's tree
-  // is all still here, and its plugins are the part of it that costs the
-  // most. Before this, a scala SDK whose model never mentioned `secrets`
-  // shipped and compiled all nine vendored provider clients: the one trim
-  // the target has walked straight past the feature that owns them.
-  //
-  // EVERY declared group of an inactive feature goes, not just the groups
-  // marked inactive. A feature that is off has no active plugins whatever
-  // its `plugin` map says - `main: kit: feature: secrets: { active: false
-  // plugin: vault: active: true }` selects nothing, because targetFeatures
-  // drops the feature before Config ever reads a group - so shipping vault's
-  // files for it would leave a provider client in the tree that nothing in
-  // the SDK can reach. That is why this reads the declared paths directly
-  // rather than reusing pluginExcludesFor, whose rule (exclude the groups
-  // marked `active: false`) is the right one only for a LIVE feature.
-  //
-  // Rooting matches pluginExcludes': scala's declared paths are
-  // target-root-relative, which is this Copy's root, and the other targets'
-  // paths in the same list cannot match anything in a scala tree.
   const esc = (s: string) => s.replace(/[.*+?^${}()|[\]\\]/g, '\\$&')
   const allfeature = getModelPath(model, `main.${KIT}.feature`,
     { required: false, only_active: false }) || {}
@@ -87,27 +63,6 @@ const Main = cmp(async function Main(props: any) {
     }
   }
 
-  // THE FEATURE THE MODEL NEVER MENTIONS AT ALL.
-  //
-  // inactivePluginExcludes above walks the model's DECLARED features and skips
-  // the active ones, which covers `secrets: { active: false }`. It cannot cover
-  // the case that actually shipped: a model that does not mention secrets
-  // anywhere. Then `main.kit.feature` has no `secrets` key, the loop never sees
-  // it, nothing is excluded, and all nine provider clients are copied and
-  // compiled — which is what `vocabulary.off.notrimmed` reports, and it failed
-  // on 12 of the 68 cedar SDKs.
-  //
-  // PRECISE, NOT THE WHOLE TREE. The first cut of this excluded
-  // `feature/<inactive>/` wholesale and broke the build: SecretsFeature.scala
-  // ships to every scala SDK regardless (this target's feature trim is off) and
-  // imports `com.voxgig.sekreto` and `voxgig.plugin`, so removing the vendored
-  // cores left it uncompilable — "value voxgig is not a member of com".
-  //
-  // What may go is exactly the nine provider clients under sekreto/plugins.
-  // Httpjson.scala and Sigv4.scala live there too and must STAY: both belong to
-  // no group and are shared, which model/feature/secrets.aon documents at
-  // length after a cloud-only trim once deleted Sigv4 and scalac failed with
-  // four "Not found: uriescape".
   const SHARED_SEKRETO_PLUGINS = ['Httpjson.scala', 'Sigv4.scala']
   const pluginDirExcludes: RegExp[] = []
   if (null == (feature as any).secrets) {
@@ -125,25 +80,6 @@ const Main = cmp(async function Main(props: any) {
   // ProjectName carries the SDK name into vendored template strings.
   Copy({
     from: 'tm/' + target.name,
-    // The generate-time plugin trim, in both halves: pluginExcludes for an
-    // ACTIVE feature's inactive groups, inactivePluginExcludes (above) for
-    // the groups of a feature that is itself off. An inactive group's
-    // declared files stay out of the tree - the model's `path` entries are
-    // target-root-relative, which is this Copy's root.
-    //
-    // Scala's FEATURE-level trim is off (model/target/scala.aon `feature: {
-    // trim: false }`, because the cross-feature tests are fused into one
-    // sdktest/SdkTestMain.scala, which constructs fifteen feature classes by
-    // name), so this is the ONLY trim the target has, and what it does NOT
-    // reach is worth stating rather than leaving to be discovered: every
-    // feature's own source ships to every scala SDK whatever the model says,
-    // secrets included, so a secrets-off SDK still carries SecretsFeature,
-    // the vendored sekreto and voxgig/plugin cores, and the two UNGROUPED
-    // plugin files (plugins/Sigv4.scala's HMAC-SHA256 signer and
-    // plugins/Httpjson.scala's fetch + ProcessBuilder helpers, which belong
-    // to no group because four and eight kinds respectively compile against
-    // them). What it DOES reach is the nine provider clients, and they are
-    // the bulk: measured on a feature-off SDK, 26 files rather than 35.
     exclude: [
       /src\//,
       TEST_CONTROL_EXCLUDE,
@@ -161,20 +97,8 @@ const Main = cmp(async function Main(props: any) {
   // Shared entity runtime (entity/EntityBase.scala).
   EntityBase({ target })
 
-  // utility/PrepareAuth.scala. WHERE the credential goes (header, query or
-  // cookie) and under what name is a fact about THIS API, and
-  // tm/scala/utility/Prepare.scala can only hold one answer - which is why
-  // its `object PrepareAuth` was removed and this generates the object
-  // instead. See PrepareAuth_scala.
-  //
-  // CALLED AT THE TOP LEVEL, NOT INSIDE `Folder({ name: 'core' })` below:
-  // the component opens `utility` itself, reproducing the path the blanket
-  // Copy above gives every other tm/scala/utility file. Nested in core/ it
-  // would write core/utility/PrepareAuth.scala, contradicting its own
-  // package declaration while Register.scala bound nothing.
   PrepareAuth({ target })
 
-  // Generate the client class and config in core/.
   Folder({ name: 'core' }, () => {
 
     File({ name: model.const.Name + 'SDK.' + target.ext }, () => {
@@ -189,7 +113,6 @@ const Main = cmp(async function Main(props: any) {
           }
         },
 
-        // Entities - injected at SLOT
         () => {
           each(entity, (entity: ModelEntity) => {
             const entitySDK = getModelPath(model, `main.${KIT}.entity.${entity.name}`)
@@ -201,6 +124,8 @@ const Main = cmp(async function Main(props: any) {
 
     Config({ target })
 
+    Schema({ target })
+
     // Generate the typed reference-model file (<Name>Types.scala) beside the
     // other generated core files. Documentation/DX shapes only — not wired
     // into the loose-object-model op signatures.