@appsemble/lang-sdk 0.40.7 → 0.40.8

package/BasicValidator.js

@@ -1,6 +1,7 @@
 import { Validator, } from 'jsonschema';
 import { isValidIconReference } from './icons.js';
 import { schemas as allSchemas } from './index.js';
+import { skipBaseResolutionForSchemasWithoutId } from './jsonschemaBaseResolution.js';
 import { escapeJsonPointer } from './jsonPointer.js';
 import { has } from './miscellaneous.js';
 export class BaseValidatorFactory {
@@ -9,6 +10,7 @@ export class BaseValidatorFactory {
         this.schemas = schemas;
     }
     build() {
+        skipBaseResolutionForSchemasWithoutId();
         const validator = new Validator();
         for (const [key, format] of Object.entries({
             ...BaseValidatorFactory.defaultCustomFormats,

package/blockUtils.js

@@ -49,6 +49,9 @@ export function getAppBlocks(definition) {
     const result = [];
     iterApp(definition, {
         onBlock({ type, version }) {
+            if (typeof type !== 'string' || typeof version !== 'string') {
+                return;
+            }
             const name = normalizeBlockName(type);
             const asString = `${name}@${version}`;
             if (!visited.has(asString)) {

package/jsonschema.d.ts

@@ -23,6 +23,7 @@ declare module 'jsonschema' {
     }
     interface SchemaContext {
         path: (number | string)[];
+        resolve: (target: string) => string;
     }
 }
 /**

package/jsonschemaBaseResolution.d.ts

@@ -0,0 +1,13 @@
+/**
+ * Resolve a child schema's base URI only when that schema declares an id.
+ *
+ * `jsonschema` resolves a base URI for every schema node it visits, and that URL resolution is the
+ * single most expensive part of validating a large app definition. A schema without an `$id` or
+ * `id` always resolves to its parent's base, so the resolution only carries information when an id
+ * is present. Appsemble's schemas declare no ids, which makes the resolution pure overhead for
+ * every node in the document.
+ *
+ * Calling this more than once is harmless: it installs the same implementation each time rather
+ * than wrapping whatever is already there.
+ */
+export declare function skipBaseResolutionForSchemasWithoutId(): void;

package/jsonschemaBaseResolution.js

@@ -0,0 +1,30 @@
+// `jsonschema` ships no types for its internal helpers module, which is where the schema context
+// it builds while validating lives.
+// @ts-expect-error 7016 -- untyped module
+import { SchemaContext as UntypedSchemaContext } from 'jsonschema/lib/helpers.js';
+const SchemaContextConstructor = UntypedSchemaContext;
+/**
+ * Resolve a child schema's base URI only when that schema declares an id.
+ *
+ * `jsonschema` resolves a base URI for every schema node it visits, and that URL resolution is the
+ * single most expensive part of validating a large app definition. A schema without an `$id` or
+ * `id` always resolves to its parent's base, so the resolution only carries information when an id
+ * is present. Appsemble's schemas declare no ids, which makes the resolution pure overhead for
+ * every node in the document.
+ *
+ * Calling this more than once is harmless: it installs the same implementation each time rather
+ * than wrapping whatever is already there.
+ */
+export function skipBaseResolutionForSchemasWithoutId() {
+    SchemaContextConstructor.prototype.makeChild = function makeChild(schema, propertyName) {
+        const path = propertyName === undefined ? this.path : [...this.path, propertyName];
+        const id = schema.$id ?? schema.id;
+        const base = id ? this.resolve(id) : this.base;
+        const ctx = new SchemaContextConstructor(schema, this.options, path, base, Object.create(this.schemas));
+        if (id && !ctx.schemas[base]) {
+            ctx.schemas[base] = schema;
+        }
+        return ctx;
+    };
+}
+//# sourceMappingURL=jsonschemaBaseResolution.js.map

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@appsemble/lang-sdk",
-  "version": "0.40.7",
+  "version": "0.40.8",
   "description": "Language SDK for Appsemble",
   "keywords": [
     "app",