@voxgig/sdkgen 1.3.17 → 1.3.18

package/project/.sdk/src/cmp/go/TestEntity_go.ts

@@ -59,7 +59,7 @@ const TestEntity = cmp(function TestEntity(props: any) {
   // it only applies to entities that declare a list op. Others (e.g. Batch =
   // create/load) have no list endpoint — make_point errors and the stream
   // yields nothing — so skip the stream test for them.
-  const hasList = !!(entity.op && (entity.op as any).list)
+  const hasList = !!(entity.op && (entity.op as any)?.list)
 
   const basicflow: ModelEntityFlow | undefined =
     getModelPath(model, `main.${KIT}.flow.Basic${nom(entity, 'Name')}Flow`)

package/project/.sdk/src/cmp/go/utility_go.ts

@@ -7,6 +7,7 @@ import {
   canonKey,
   each,
   exampleVarName,
+  names,
 } from '@voxgig/sdkgen'
 
 import {
@@ -149,7 +150,31 @@ function clean(o: any) {
 }
 
 
+
+// The Go identifier fragment for a feature's generated constructor
+// (New<Fname>Feature / New<Fname>FeatureFunc).
+//
+// SHARED because the identifier is DECLARED in Main_go.ts (registry.go and the
+// root init()) and REFERENCED in Config_go.ts (makeFeature). Two copies of the
+// derivation is a latent undefined-identifier bug in the generated SDK: they
+// were both `name.charAt(0).toUpperCase() + name.slice(1)`, consistently wrong
+// for a name needing real normalisation but at least agreeing, until one side
+// was fixed alone and `rate_limit` became NewRateLimitFeatureFunc in the
+// registry and NewRate_limitFeatureFunc in config.
+//
+// Uses the jostraca-derived PascalCase `Name` (deriving it if absent), which is
+// what Main_ts.ts uses, so a hyphenated or underscored feature name yields a
+// legal Go identifier instead of `NewRate-limitFeatureFunc`.
+function goFeatureName(feat: any): string {
+  if (null == feat.Name) {
+    names(feat, feat.name)
+  }
+  return feat.Name
+}
+
+
 export {
+  goFeatureName,
   clean,
   exampleValue,
   formatGoMap,

package/project/.sdk/src/cmp/go-cli/Main_go-cli.ts

@@ -2,7 +2,7 @@
 import * as Path from 'node:path'
 
 import {
-  cmp, each,
+  cmp, each, deriveEntityNames,
   File, Content, Fragment, Slot,
 } from '@voxgig/sdkgen'
 
@@ -40,6 +40,14 @@ const Main = cmp(function Main(props: any) {
   const cliModule = `github.com/${org}/${model.name}-sdk/go-cli`
 
   const entityMap: any = getModelPath(model, `main.${KIT}.entity`)
+  // Derive each entity's PascalCase `Name` here rather than relying on another
+  // TARGET having run first. This target sets `phase.entity: { active: false }`
+  // (see model/target/go-{cli,mcp}.aontu), so create-sdkgen's Root.ts never
+  // calls names() for it — the accessor emitted below used to read `Name` off
+  // objects the `go` target happened to have named as a side effect, which
+  // yields `client.undefined(nil)` whenever this target is generated without
+  // `go`, or before it in target order.
+  deriveEntityNames(entityMap)
   const entityNames = Object.keys(entityMap).map(n => n.toLowerCase())
 
   const FRAGMENT = Path.normalize(__dirname + '/../../../src/cmp/go-cli/fragment')

package/project/.sdk/src/cmp/go-mcp/Main_go-mcp.ts

@@ -2,7 +2,7 @@
 import * as Path from 'node:path'
 
 import {
-  cmp, each,
+  cmp, each, deriveEntityNames,
   File, Content, Fragment, Slot,
 } from '@voxgig/sdkgen'
 
@@ -54,6 +54,14 @@ const Main = cmp(function Main(props: any) {
   const mcpModule = `github.com/${org}/${model.name}-sdk/go-mcp`
 
   const entityMap: any = getModelPath(model, `main.${KIT}.entity`)
+  // Derive each entity's PascalCase `Name` here rather than relying on another
+  // TARGET having run first. This target sets `phase.entity: { active: false }`
+  // (see model/target/go-{cli,mcp}.aontu), so create-sdkgen's Root.ts never
+  // calls names() for it — the accessor emitted below used to read `Name` off
+  // objects the `go` target happened to have named as a side effect, which
+  // yields `client.undefined(nil)` whenever this target is generated without
+  // `go`, or before it in target order.
+  deriveEntityNames(entityMap)
   const entityNames = Object.keys(entityMap).map(n => n.toLowerCase())
   const entityHelp = entityNames.length > 0 ? entityNames.join(' | ') : '(none)'
 

package/project/.sdk/src/cmp/haskell/ReadmeExamplesTest_haskell.ts

@@ -67,35 +67,44 @@ readDoc path = do
   pure (either (const "") id r)
 
 
--- Extract the body of every fenced Haskell block from markdown. The fence
--- marker (three backticks) is built from its char code so this source stays
--- backtick-free.
-haskellBlocks :: String -> [String]
+-- Extract the body of every fenced Haskell block from markdown. Returns
+-- (blocks, allClosed) where allClosed is False when a fence was opened but
+-- never closed before EOF — a malformed doc that must fail rather than pass
+-- silently on the unterminated block. The fence marker (three backticks) is
+-- built from its char code so this source stays backtick-free.
+haskellBlocks :: String -> ([String], Bool)
 haskellBlocks src = go (lines src)
   where
     fence3 = replicate 3 (toEnum 96 :: Char)
     fenceHs = fence3 ++ "haskell"
-    go [] = []
+    go [] = ([], True)
     go (l : ls)
       | trimWs l == fenceHs =
-          let (blk, rest) = break (\\x -> trimWs x == fence3) ls
-          in unlines blk : go (drop 1 rest)
+          case break (\\x -> trimWs x == fence3) ls of
+            (blk, [])       -> ([unlines blk], False)
+            (blk, _ : rest) -> let (bs, ok) = go rest in (unlines blk : bs, ok)
       | otherwise = go ls
 
 
--- Balanced (), [], {} ignoring the contents of double-quoted string literals.
--- The generated snippets never embed an escaped quote inside a string.
+-- Balanced (), [], {} outside of double-quoted string literals, with each
+-- closer matching the most-recent opener via a stack (so "([)]" is rejected,
+-- not merely counted). The generated snippets never embed an escaped quote
+-- inside a string.
 balanced :: String -> Bool
-balanced = go False (0 :: Int)
+balanced = go False []
   where
-    go inStr d [] = not inStr && d == 0
-    go inStr d (c : cs)
-      | c == '"'                          = go (not inStr) d cs
-      | inStr                             = go inStr d cs
-      | c == '(' || c == '[' || c == '{'  = go inStr (d + 1) cs
+    go inStr stack [] = not inStr && null stack
+    go inStr stack (c : cs)
+      | c == '"'                          = go (not inStr) stack cs
+      | inStr                             = go inStr stack cs
+      | c == '('                          = go inStr (')' : stack) cs
+      | c == '['                          = go inStr (']' : stack) cs
+      | c == '{'                          = go inStr ('}' : stack) cs
       | c == ')' || c == ']' || c == '}'  =
-          if d <= 0 then False else go inStr (d - 1) cs
-      | otherwise                         = go inStr d cs
+          case stack of
+            (top : rest) | top == c -> go inStr rest cs
+            _                        -> False
+      | otherwise                         = go inStr stack cs
 
 
 -- The three documents held to the gate, tagged by human label.
@@ -111,7 +120,8 @@ tests :: Counters -> IO ()
 tests c =
   forM_ docs (\\(label, path) -> do
     src <- readDoc path
-    let blocks = haskellBlocks src
+    let (blocks, closed) = haskellBlocks src
+    runTest c (label ++ " haskell fences closed") (pure closed)
     forM_ (zip [1 :: Int ..] blocks) (\\(i, blk) ->
       runTest c (label ++ " haskell block " ++ show i ++ " well-formed")
         (pure (not (null (trimWs blk)) && balanced blk))))

package/project/.sdk/src/cmp/java/EntityTypes_java.ts

@@ -37,7 +37,7 @@ import {
   File, Content,
 } from '@voxgig/sdkgen'
 
-import { canonToType, opTypeName, opRequestShape, warnEntityTypeCollisions } from '@voxgig/sdkgen'
+import { canonToType, opTypeName, opRequestShape, warnEntityTypeCollisions , deriveEntityNames } from '@voxgig/sdkgen'
 
 import {
   KIT,
@@ -134,11 +134,10 @@ const EntityTypes = cmp(function EntityTypes(props: any) {
   // 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 = each(entity).filter((e: any) => e && null != e.name)
+  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/py/csharp emitter's fix.
-  entityList.forEach((e: any) => { if (null == e.Name) names(e, e.name) })
 
   // Surface duplicate generated type names (two entities with the same
   // PascalCase Name) — they would redeclare a type in statically-typed

package/project/.sdk/src/cmp/java/Entity_java.ts

@@ -4,7 +4,7 @@ import * as Path from 'node:path'
 import {
   cmp, camelify, names,
   File, Content, Folder, Fragment,
-  entityClassName,
+  entityClassName, entityCollection,
 } from '@voxgig/sdkgen'
 
 import {
@@ -25,7 +25,7 @@ const Entity = cmp(function Entity(props: any) {
   // Collision-free entity CLASS name (see entityClassName): normally
   // `<Name>Entity`, but disambiguated (e.g. `<Name>EntityClient`) when it
   // would clash with another entity's data-type name.
-  const entityColl = getModelPath(model, `main.${KIT}.entity`)
+  const entityColl = entityCollection(model)
   const cls = entityClassName(entity, entityColl)
 
   const entrep = {
@@ -42,7 +42,7 @@ const Entity = cmp(function Entity(props: any) {
 
     File({ name: cls + '.' + target.ext }, () => {
 
-      const opnames = Object.keys(entity.op)
+      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 SdkEntity

package/project/.sdk/src/cmp/java/MainEntity_java.ts

@@ -1,6 +1,6 @@
 
 
-import { cmp, Content, entityClassName } from '@voxgig/sdkgen'
+import { cmp, Content, entityClassName, entityCollection } from '@voxgig/sdkgen'
 
 import {
   KIT,
@@ -18,7 +18,7 @@ const MainEntity = cmp(async function MainEntity(props: any) {
   const { model } = props.ctx$
 
   const javapackage = javaPackage(model)
-  const entityColl = getModelPath(model, `main.${KIT}.entity`)
+  const entityColl = entityCollection(model)
   const cls = entityClassName(entity, entityColl)
   const accessor = javaVarName(entity.name)
 

package/project/.sdk/src/cmp/java/Package_java.ts

@@ -31,7 +31,7 @@ const Package = cmp(async function Package(props: any) {
 
   // Dep name convention: "groupId:artifactId" with an explicit version.
   const deps: { groupId: string, artifactId: string, version: string, kind: string }[] = []
-  for (const d of collectDeps(model, target.name, target.deps)) {
+  for (const d of collectDeps(model, target.name, target.deps, ctx$.log)) {
     const [dgroup, dartifact] = String(d.name).split(':')
     if (dgroup && dartifact && d.version) {
       deps.push({

package/project/.sdk/src/cmp/java/ReadmeExamplesTest_java.ts

@@ -95,6 +95,23 @@ public class ReadmeExamplesTest {
     return blocks;
   }
 
+  // A well-formed doc has an even number of triple-backtick fence markers:
+  // every opening fence has a matching close. An odd count means a fence was
+  // opened but never closed — a truncated or malformed generated doc. The
+  // split-based javaBlocks() would silently treat the unterminated tail as a
+  // complete block, so this must be checked explicitly.
+  private static boolean fencesClosed(String text) {
+    String fence = String.valueOf((char) 96).repeat(3);
+    int count = 0;
+    int from = 0;
+    int idx;
+    while ((idx = text.indexOf(fence, from)) >= 0) {
+      count++;
+      from = idx + fence.length();
+    }
+    return count % 2 == 0;
+  }
+
   private static String readDoc(Path path, String label) {
     assertTrue(Files.exists(path), label + " not found: " + path);
     try {
@@ -158,7 +175,12 @@ public class ReadmeExamplesTest {
       if (!Files.exists(path)) {
         continue;
       }
-      List<String> blocks = javaBlocks(readDoc(path, doc[0]));
+      String src = readDoc(path, doc[0]);
+      if (!fencesClosed(src)) {
+        failures.add(doc[0] +
+            " has an unclosed code fence (opened but never closed)");
+      }
+      List<String> blocks = javaBlocks(src);
       int balancedCount = 0;
       for (int i = 0; i < blocks.size(); i++) {
         if (balanced(blocks.get(i))) {

package/project/.sdk/src/cmp/java/TestDirect_java.ts

@@ -75,7 +75,7 @@ const TestDirect = cmp(function TestDirect(props: any) {
 
   const authActive = isAuthActive(model)
 
-  const opnames = Object.keys(entity.op)
+  const opnames = Object.keys(entity.op || {})
   const hasLoad = opnames.includes('load')
   const hasList = opnames.includes('list')
 
@@ -83,8 +83,8 @@ const TestDirect = cmp(function TestDirect(props: any) {
     return
   }
 
-  const loadOp = entity.op.load
-  const listOp = entity.op.list
+  const loadOp = entity.op?.load
+  const listOp = entity.op?.list
 
   // Get load point info
   const loadPoint = loadOp?.points?.[0]

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

@@ -26,7 +26,7 @@ import {
   File, Content,
 } from '@voxgig/sdkgen'
 
-import { canonToType, opTypeName, opRequestShape, warnEntityTypeCollisions } from '@voxgig/sdkgen'
+import { canonToType, opTypeName, opRequestShape, warnEntityTypeCollisions , deriveEntityNames } from '@voxgig/sdkgen'
 
 import {
   KIT,
@@ -57,11 +57,10 @@ const EntityTypes = cmp(function EntityTypes(props: any) {
   // 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 = each(entity).filter((e: any) => e && null != e.name)
+  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.
-  entityList.forEach((e: any) => { if (null == e.Name) names(e, e.name) })
 
   // Surface duplicate generated type names (two entities with the same
   // PascalCase Name) — they would redeclare a type in statically-typed

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

@@ -4,7 +4,7 @@ import * as Path from 'node:path'
 import {
   cmp, each, camelify, names,
   File, Content, Folder, Fragment, Line, FeatureHook, Slot,
-  entityClassName,
+  entityClassName, entityCollection,
 } from '@voxgig/sdkgen'
 
 import {
@@ -22,7 +22,7 @@ const Entity = cmp(function Entity(props: any) {
   // Collision-free entity CLASS name (see entityClassName): normally
   // `<Name>Entity`, disambiguated when it would clash with another entity's
   // data-type name. The class file name and the Main require path both use this.
-  const entityColl = getModelPath(model, `main.${KIT}.entity`)
+  const entityColl = entityCollection(model)
   const cls = entityClassName(entity, entityColl)
 
   const entrep = {
@@ -37,7 +37,7 @@ const Entity = cmp(function Entity(props: any) {
 
     File({ name: cls + '.' + target.name }, () => {
 
-      const opnames = Object.keys(entity.op)
+      const opnames = Object.keys(entity.op || {})
 
       const opfrags =
         (['load', 'list', 'create', 'update', 'remove']

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

@@ -1,6 +1,6 @@
 
 
-import { cmp, Content, entityClassName } from '@voxgig/sdkgen'
+import { cmp, Content, entityClassName, entityCollection } from '@voxgig/sdkgen'
 
 import {
   KIT,
@@ -14,7 +14,7 @@ const MainEntity = cmp(async function MainEntity(props: any) {
 
   // Return the collision-free class (entityClassName); the accessor METHOD name
   // (entity.Name) is unchanged so callers still write client.<Name>().
-  const cls = entityClassName(entity, getModelPath(model, `main.${KIT}.entity`))
+  const cls = entityClassName(entity, entityCollection(model))
 
   Content(`
   // Entity access: \`client.${entity.Name}().list()\` / \`client.${entity.Name}().load({ id })\`.

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

@@ -4,7 +4,7 @@ import * as Path from 'node:path'
 import {
   cmp, each, names, cmap,
   List, File, Content, Copy, Folder, Fragment, Line, FeatureHook,
-  entityClassName,
+  entityClassName, entityCollection,
 } from '@voxgig/sdkgen'
 
 
@@ -57,7 +57,7 @@ const Main = cmp(async function Main(props: any) {
       Line(`// ${model.const.Name} ${target.Name} SDK\n`)
 
       List({ item: entity }, ({ item }: any) => {
-        const cls = entityClassName(item, entity)
+        const cls = entityClassName(item, entityCollection(model))
         return Line(`const { ${cls} } = require('./entity/${cls}')`)
       })
 

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

@@ -48,7 +48,7 @@ const TestDirect = cmp(function TestDirect(props: any) {
       apikey: env.${PROJECTNAME}_APIKEY,`
     : ''
 
-  const opnames = Object.keys(entity.op)
+  const opnames = Object.keys(entity.op || {})
   const hasLoad = opnames.includes('load')
   const hasList = opnames.includes('list')
 
@@ -134,7 +134,7 @@ function directSetup(mockres) {
 
 
 function generateDirectLoad(model: Model, entity: ModelEntity) {
-  const loadOp = entity.op.load
+  const loadOp = entity.op?.load
   const loadPoint: ModelPoint | undefined = loadOp?.points?.[0]
 
   if (null == loadPoint) {
@@ -145,7 +145,7 @@ function generateDirectLoad(model: Model, entity: ModelEntity) {
   const loadPath = normalizePathParams(loadPoint.parts || [], loadParams, loadPoint.rename?.param)
 
   // Get list info for live mode bootstrapping
-  const listOp = entity.op.list
+  const listOp = entity.op?.list
   const listPoint = listOp?.points?.[0]
   const listParams = listPoint?.args?.params || []
   const listPath = listPoint ? normalizePathParams(listPoint.parts || [], listParams, listPoint.rename?.param) : ''
@@ -231,7 +231,7 @@ ${paramAsserts}    }
 
 
 function generateDirectList(model: Model, entity: ModelEntity) {
-  const listOp = entity.op.list
+  const listOp = entity.op?.list
   const listPoint: ModelPoint | undefined = listOp?.points?.[0]
 
   if (null == listPoint) {

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

@@ -39,7 +39,7 @@ import {
   File, Content,
 } from '@voxgig/sdkgen'
 
-import { canonToType, opTypeName, opRequestShape, warnEntityTypeCollisions } from '@voxgig/sdkgen'
+import { canonToType, opTypeName, opRequestShape, warnEntityTypeCollisions , deriveEntityNames } from '@voxgig/sdkgen'
 
 import {
   KIT,
@@ -132,11 +132,10 @@ const EntityTypes = cmp(function EntityTypes(props: any) {
   // 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 = each(entity).filter((e: any) => e && null != e.name)
+  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/py/csharp/java emitter's fix.
-  entityList.forEach((e: any) => { if (null == e.Name) names(e, e.name) })
 
   // Surface duplicate generated type names (two entities with the same
   // PascalCase Name) — they would redeclare a type in statically-typed

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

@@ -4,7 +4,7 @@ import * as Path from 'node:path'
 import {
   cmp, camelify, names,
   File, Content, Folder, Fragment,
-  entityClassName,
+  entityClassName, entityCollection,
 } from '@voxgig/sdkgen'
 
 import {
@@ -25,7 +25,7 @@ const Entity = cmp(function Entity(props: any) {
   // Collision-free entity CLASS name (see entityClassName): normally
   // `<Name>Entity`, but disambiguated (e.g. `<Name>EntityClient`) when it
   // would clash with another entity's data-type name.
-  const entityColl = getModelPath(model, `main.${KIT}.entity`)
+  const entityColl = entityCollection(model)
   const cls = entityClassName(entity, entityColl)
 
   const entrep = {
@@ -41,7 +41,7 @@ const Entity = cmp(function Entity(props: any) {
 
     File({ name: cls + '.' + target.ext }, () => {
 
-      const opnames = Object.keys(entity.op)
+      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 SdkEntity

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

@@ -1,5 +1,5 @@
 
-import { cmp, Content, entityClassName } from '@voxgig/sdkgen'
+import { cmp, Content, entityClassName, entityCollection } from '@voxgig/sdkgen'
 
 import {
   KIT,
@@ -17,7 +17,7 @@ const MainEntity = cmp(async function MainEntity(props: any) {
   const { model } = props.ctx$
 
   const kotlinpackage = kotlinPackage(model)
-  const entityColl = getModelPath(model, `main.${KIT}.entity`)
+  const entityColl = entityCollection(model)
   const cls = entityClassName(entity, entityColl)
   const accessor = kotlinVarName(entity.name)
 

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

@@ -30,7 +30,7 @@ const Package = cmp(async function Package(props: any) {
 
   // Dep name convention: "group:artifact" with an explicit version.
   const deps: { group: string, artifact: string, version: string, kind: string }[] = []
-  for (const d of collectDeps(model, target.name, target.deps)) {
+  for (const d of collectDeps(model, target.name, target.deps, ctx$.log)) {
     const [dgroup, dartifact] = String(d.name).split(':')
     if (dgroup && dartifact && d.version) {
       deps.push({

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

@@ -46,7 +46,9 @@ const ReadmeExamplesTest = cmp(function ReadmeExamplesTest(props: any) {
 // Checks, PER DOCUMENT:
 //   1. PRESENCE: the doc exists and holds at least one kotlin block.
 //   2. STRUCTURE: every kotlin block's braces / parens / brackets balance, so a
-//      truncated or malformed generated snippet fails loudly.
+//      truncated or malformed generated snippet fails loudly. Every code fence
+//      is also terminated (even fence count), so an unclosed fence at EOF is
+//      caught rather than accepted as a complete block.
 //
 // GENERATED by @voxgig/sdkgen — do not edit by hand.
 
@@ -97,6 +99,22 @@ class ReadmeExamplesTest {
     return Files.readString(path)
   }
 
+  // Count triple-backtick fence markers. A well-formed document opens and
+  // closes every fenced block, so the total must be EVEN. An odd count means a
+  // fence was opened but never closed (a truncated / malformed doc) — the
+  // split-based extraction in kotlinBlocks() would otherwise accept the
+  // unterminated trailing segment as a complete block and pass silently.
+  private fun fenceCount(text: String): Int {
+    val fence = 96.toChar().toString().repeat(3)
+    var count = 0
+    var idx = text.indexOf(fence)
+    while (idx >= 0) {
+      count++
+      idx = text.indexOf(fence, idx + fence.length)
+    }
+    return count
+  }
+
   // True when every open/close delimiter balances (ignores the subtlety of
   // delimiters inside string literals — a generated snippet never nests an
   // unbalanced delimiter in a string, and this catches truncation).
@@ -145,7 +163,14 @@ class ReadmeExamplesTest {
       if (!Files.exists(path)) {
         continue
       }
-      val blocks = kotlinBlocks(readDoc(path, doc[0]))
+      val src = readDoc(path, doc[0])
+      // A fence opened but never closed is a malformed doc — fail loudly
+      // rather than silently accepting the unterminated trailing segment as a
+      // complete block.
+      if (fenceCount(src) % 2 != 0) {
+        failures.add(doc[0] + " has an unterminated code fence")
+      }
+      val blocks = kotlinBlocks(src)
       var balancedCount = 0
       for (i in blocks.indices) {
         if (balanced(blocks[i])) {

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

@@ -72,7 +72,7 @@ const TestDirect = cmp(function TestDirect(props: any) {
 
   const authActive = isAuthActive(model)
 
-  const opnames = Object.keys(entity.op)
+  const opnames = Object.keys(entity.op || {})
   const hasLoad = opnames.includes('load')
   const hasList = opnames.includes('list')
 
@@ -80,8 +80,8 @@ const TestDirect = cmp(function TestDirect(props: any) {
     return
   }
 
-  const loadOp = entity.op.load
-  const listOp = entity.op.list
+  const loadOp = entity.op?.load
+  const listOp = entity.op?.list
 
   const loadPoint = loadOp?.points?.[0]
   const loadPath = loadPoint ? normalizePathParams(loadPoint.parts || [], loadPoint?.args?.params || [], loadPoint?.rename?.param) : ''