@voxgig/sdkgen 4.32.1 → 4.33.0

package/project/.sdk/src/cmp/js/fragment/EntityBase.fragment.js

@@ -154,6 +154,8 @@ class ProjectNameEntityBase {
       // Inbound: prefer the streaming feature's incremental iterator; else
       // fall back to the materialised items so `stream` always yields.
       if (result && 'function' === typeof result.stream) {
+        // done() does not run on this path, so its record is cleaned here.
+        utility.cleanExplain(ctx)
         for await (const item of result.stream()) {
           if (signal && signal.aborted) { return }
           yield item
@@ -200,7 +202,6 @@ class ProjectNameEntityBase {
     const clean = this._utility.clean
     const struct = this._utility.struct
 
-    const delprop = struct.delprop
     const clone = struct.clone
     const merge = struct.merge
 
@@ -209,8 +210,7 @@ class ProjectNameEntityBase {
     ctrl.err = err
 
     if (ctrl.explain) {
-      ctx.ctrl.explain = clean(ctx, ctx.ctrl.explain)
-      delprop(ctx.ctrl.explain.result, 'err')
+      this._utility.cleanExplain(ctx)
 
       if (null != ctx.result && null != ctx.result.err) {
         ctrl.explain.err = clean(ctx, merge([
@@ -240,7 +240,8 @@ class ProjectNameEntityBase {
       return undefined
     }
 
-    return err
+    // An error a hook threw never passed through makeError.
+    return clean(ctx, err)
   }
 
 }

package/project/.sdk/src/cmp/js/fragment/EntityCreateOp.fragment.js

@@ -95,7 +95,13 @@ class EntityOperation {
       return (ctx.result && ctx.result.ok) ? this : out
     }
     catch (err) {
-      // #PreUnexpected-Hook
+      // What a hook throws here must not escape the cleaning below.
+      try {
+        // #PreUnexpected-Hook
+      }
+      catch (hookerr) {
+        err = hookerr
+      }
 
       err = this._unexpected(ctx, err)
 

package/project/.sdk/src/cmp/js/fragment/EntityListOp.fragment.js

@@ -88,7 +88,13 @@ class EntityOperation {
       return done(ctx)
     }
     catch (err) {
-      // #PreUnexpected-Hook
+      // What a hook throws here must not escape the cleaning below.
+      try {
+        // #PreUnexpected-Hook
+      }
+      catch (hookerr) {
+        err = hookerr
+      }
 
       err = this._unexpected(ctx, err)
 

package/project/.sdk/src/cmp/js/fragment/EntityLoadOp.fragment.js

@@ -100,7 +100,13 @@ class EntityOperation {
       return (ctx.result && ctx.result.ok) ? this : out
     }
     catch (err) {
-      // #PreUnexpected-Hook
+      // What a hook throws here must not escape the cleaning below.
+      try {
+        // #PreUnexpected-Hook
+      }
+      catch (hookerr) {
+        err = hookerr
+      }
 
       err = this._unexpected(ctx, err)
 

package/project/.sdk/src/cmp/js/fragment/EntityRemoveOp.fragment.js

@@ -107,7 +107,13 @@ class EntityOperation {
       return out
     }
     catch (err) {
-      // #PreUnexpected-Hook
+      // What a hook throws here must not escape the cleaning below.
+      try {
+        // #PreUnexpected-Hook
+      }
+      catch (hookerr) {
+        err = hookerr
+      }
 
       err = this._unexpected(ctx, err)
 

package/project/.sdk/src/cmp/js/fragment/EntityUpdateOp.fragment.js

@@ -101,7 +101,13 @@ class EntityOperation {
       return (ctx.result && ctx.result.ok) ? this : out
     }
     catch (err) {
-      // #PreUnexpected-Hook
+      // What a hook throws here must not escape the cleaning below.
+      try {
+        // #PreUnexpected-Hook
+      }
+      catch (hookerr) {
+        err = hookerr
+      }
 
       err = this._unexpected(ctx, err)
 

package/project/.sdk/src/cmp/js/fragment/Main.fragment.js

@@ -33,6 +33,15 @@ class ProjectNameSDK {
 
     this._options = this._utility.makeOptions(this._rootctx)
 
+    // Each can hold a credential; feature state a resolved or bought one.
+    // toJSON and inspect already leave them out; a spread or a structured
+    // logger walking own properties must too.
+    for (const key of ['_options', '_rootctx', '_features']) {
+      Object.defineProperty(this, key, {
+        value: this[key], enumerable: false, writable: true, configurable: true
+      })
+    }
+
     const struct = this._utility.struct
     const getpath = struct.getpath
 
@@ -195,7 +204,7 @@ class ProjectNameSDK {
         return { ok: false, err: ctx.error('direct_no_response', 'response: undefined') }
       }
       else if (fetched instanceof Error) {
-        return { ok: false, err: fetched }
+        return { ok: false, err: utility.clean(ctx, fetched) }
       }
 
       const status = fetched.status
@@ -229,7 +238,7 @@ class ProjectNameSDK {
       }
     }
     catch (err) {
-      return { ok: false, err }
+      return { ok: false, err: utility.clean(ctx, err) }
     }
   }
 

package/project/.sdk/src/cmp/js/fragment/SdkError.fragment.js

@@ -9,7 +9,9 @@ class ProjectNameError extends Error {
   constructor(code, msg, ctx) {
     super(msg)
     this.code = code
-    this.ctx = ctx
+    // Reachable for a debugger, invisible to a serialiser: the context holds
+    // the live spec and options, and an error is what gets logged.
+    Object.defineProperty(this, 'ctx', { value: ctx, enumerable: false, writable: true })
 
     // HTTP status of the response that caused this error, or -1 when the
     // request never got one (transport failure, client-side abort).
@@ -25,6 +27,20 @@ class ProjectNameError extends Error {
   // `err.notFound` rather than a magic number at every call site.
   get notFound() { return 404 === this.status }
 
+
+  // What makeError attached is already cleaned; the context is not part of
+  // the record.
+  toJSON() {
+    return {
+      sdk: this.sdk,
+      code: this.code,
+      message: this.message,
+      status: this.status,
+      result: this.result,
+      spec: this.spec,
+    }
+  }
+
 }
 
 module.exports = {

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

@@ -88,6 +88,11 @@ sourceSets["test"].resources.setSrcDirs(emptyList<String>())
 
 tasks.test {
     useJUnitPlatform()
+    // The canary sweep (test/CleanTest.kt) reports what it swept on stdout,
+    // and the generator's compile lane reads that line from gradle's output.
+    testLogging {
+        showStandardStreams = true
+    }
 }
 
 // Keep the Java and Kotlin compile tasks on the same bytecode target (the

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

@@ -179,6 +179,9 @@ function basicBlock(spec: AuthSpec): string {
       }
       val b64 = Base64.getEncoder().encodeToString(
         (apikey.toString() + ":" + (if (noSecret) "" else secret.toString())).toByteArray(Charsets.UTF_8))
+      // The joined, encoded pair is a wire form neither credential's own
+      // registration covers.
+      ctx.utility!!.cleanAdd(ctx, b64)
       if ("" == basicPrefix) {
         headers[HEADER_AUTH] = b64
       } else {