@voxgig/sdkgen 0.34.8 → 0.34.10

package/bin/voxgig-sdkgen

@@ -8,7 +8,7 @@ const { Gubu, Fault, One } = require('gubu')
 
 const { SdkGen } = require('../dist/sdkgen.js')
 
-const VERSION = '0.34.8'
+const VERSION = '0.34.10'
 const KONSOLE = console
 
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@voxgig/sdkgen",
-  "version": "0.34.8",
+  "version": "0.34.10",
   "main": "dist/sdkgen.js",
   "type": "commonjs",
   "types": "dist/sdkgen.d.ts",

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

@@ -39,6 +39,7 @@ const Entity = cmp(function Entity(props: any) {
         from: ff + 'Entity.fragment.js',
         replace: {
           ...entrep,
+          entityname: entity.name,
           SdkName: model.const.Name,
           EntityName: entity.Name,
 

package/project/.sdk/src/cmp/ts/Entity_ts.ts

@@ -40,6 +40,7 @@ const Entity = cmp(function Entity(props: any) {
         from: ff + 'Entity.fragment.ts',
         replace: {
           ...entrep,
+          entityname: entity.name,
           SdkName: model.const.Name,
           EntityName: entity.Name,
 

package/project/.sdk/src/cmp/ts/TestDirect_ts.ts

@@ -121,7 +121,7 @@ function directSetup(mockres?: any) {
 
 function generateDirectLoad(model: any, entity: any) {
   const loadOp = entity.op.load
-  const loadPoint = loadOp.points[0]
+  const loadPoint = loadOp?.points?.[0]
 
   if (null == loadPoint) {
     return
@@ -218,7 +218,7 @@ ${paramAsserts}    }
 
 function generateDirectList(model: any, entity: any) {
   const listOp = entity.op.list
-  const listPoint = listOp.points[0]
+  const listPoint = listOp?.points?.[0]
 
   if (null == listPoint) {
     return

package/project/.sdk/src/cmp/ts/TestEntity_ts.ts

@@ -189,11 +189,33 @@ const generateCreate: OpGen = (
   const entvar = step.input.entvar ?? ref + '_ent'
   const datavar = step.input.datavar ?? (ref + '_data' + (step.input.suffix ?? ''))
 
+  const priorSteps = Object.values(flow.step).slice(0, Number(index))
+  const needsEnt = !priorSteps.some((s: any) =>
+    ['create', 'list', 'load', 'remove'].includes(s.op))
+
+  const hasDatvar = priorSteps.some((s: any) => {
+    if ('create' === s.op) {
+      const priorRef = s.input?.ref ?? entity.name + '_ref01'
+      const priorDatvar = s.input?.datavar ?? (priorRef + '_data' + (s.input?.suffix ?? ''))
+      return priorDatvar === datavar
+    }
+    return false
+  })
+
   Content(`
     // CREATE
-    const ${entvar} = client.${nom(entity, 'Name')}()
-    let ${datavar} = setup.data.new.${entity.name}['${ref}']
 `)
+  if (needsEnt) {
+    Content(`    const ${entvar} = client.${nom(entity, 'Name')}()
+`)
+  }
+  if (hasDatvar) {
+    Content(`    ${datavar} = setup.data.new.${entity.name}['${ref}']
+`)
+  } else {
+    Content(`    let ${datavar} = setup.data.new.${entity.name}['${ref}']
+`)
+  }
 
   each(step.match, (mi: any) => {
     Content(`    ${datavar}['${mi.key$}'] = setup.idmap['${mi.val$}']
@@ -219,8 +241,9 @@ const generateList: OpGen = (
   const matchvar = step.input.matchvar ?? (ref + '_match' + (step.input.suffix ?? ''))
   const listvar = step.input.listvar ?? (ref + '_list' + (step.input.suffix ?? ''))
 
-  const priorSteps = Object.values(flow.step).slice(0, index.key$)
-  const needsEnt = !priorSteps.some((s: any) => 'create' === s.op)
+  const priorSteps = Object.values(flow.step).slice(0, Number(index))
+  const needsEnt = !priorSteps.some((s: any) =>
+    ['create', 'list', 'load', 'remove'].includes(s.op))
 
   Content(`
     // LIST
@@ -331,13 +354,14 @@ const generateLoad: OpGen = (
   const datavar = step.input.datavar ?? (ref + '_data' + (step.input.suffix ?? ''))
   const srcdatavar = step.input.srcdatavar ?? (ref + '_data' + (step.input.suffix ?? ''))
 
-  const priorSteps = Object.values(flow.step).slice(0, index.key$)
-  const hasCreate = priorSteps.some((s: any) => 'create' === s.op)
+  const priorSteps = Object.values(flow.step).slice(0, Number(index))
+  const hasEntVar = priorSteps.some((s: any) =>
+    ['create', 'list', 'load', 'remove'].includes(s.op))
 
   Content(`
     // LOAD
 `)
-  if (!hasCreate) {
+  if (!hasEntVar) {
     Content(`    const ${entvar} = client.${nom(entity, 'Name')}()
     const ${srcdatavar} = Object.values(setup.data.existing.${entity.name})[0] as any
 `)
@@ -362,8 +386,9 @@ const generateRemove: OpGen = (
   const matchvar = step.input.matchvar ?? (ref + '_match' + (step.input.suffix ?? ''))
   const srcdatavar = step.input.srcdatavar ?? (ref + '_data')
 
-  const priorSteps = Object.values(flow.step).slice(0, index.key$)
-  const needsEnt = !priorSteps.some((s: any) => 'create' === s.op)
+  const priorSteps = Object.values(flow.step).slice(0, Number(index))
+  const needsEnt = !priorSteps.some((s: any) =>
+    ['create', 'list', 'load', 'remove'].includes(s.op))
 
   Content(`
     // REMOVE

package/project/.sdk/tm/go/utility/make_target.go

@@ -26,6 +26,11 @@ func makePointUtil(ctx *core.Context) (map[string]any, error) {
 				"\" not allowed by SDK option allow.op value: \""+allowOp+"\"")
 	}
 
+	if len(op.Points) == 0 {
+		return nil, ctx.MakeError("point_no_points",
+			"Operation \""+op.Name+"\" has no endpoint definitions.")
+	}
+
 	if len(op.Points) == 1 {
 		ctx.Point = op.Points[0]
 	} else {

package/project/.sdk/tm/ts/src/utility/MakePointUtility.ts

@@ -16,6 +16,11 @@ function makePoint(ctx: Context): Point | Error {
       '" not allowed by SDK option allow.op value: "' + options.allow.op + '"')
   }
 
+  if (0 === op.points.length) {
+    return ctx.error('point_no_points',
+      'Operation "' + op.name + '" has no endpoint definitions.')
+  }
+
   // Choose the appropriate point based on the match or data.
   if (1 === op.points.length) {
     ctx.point = op.points[0]