@jskit-ai/kernel 0.1.151 → 0.1.152

package/internal/node/installedPackages.js

@@ -10,6 +10,17 @@ const ROOT_DEPENDENCY_SECTIONS = Object.freeze([
   "peerDependencies",
   "devDependencies"
 ]);
+const JSKIT_PACKAGE_CONFIG_KEYS = Object.freeze([
+  "capabilities",
+  "ci",
+  "kind",
+  "lifecycle",
+  "metadata",
+  "mutations",
+  "optionPolicies",
+  "options",
+  "runtime"
+]);
 
 async function readJsonFile(filePath, fallback = {}) {
   try {
@@ -57,6 +68,14 @@ function createPackageMetadata(packageJson = {}) {
   if (!packageId || !version) {
     return null;
   }
+  const unknownKeys = Object.keys(jskit)
+    .filter((key) => !JSKIT_PACKAGE_CONFIG_KEYS.includes(key))
+    .sort();
+  if (unknownKeys.length > 0) {
+    throw new Error(
+      `${packageId} package.json#jskit contains unknown ${unknownKeys.length === 1 ? "field" : "fields"}: ${unknownKeys.join(", ")}.`
+    );
+  }
 
   return Object.freeze({
     ...jskit,
@@ -275,6 +294,7 @@ async function discoverInstalledPackages({ appRoot } = {}) {
 }
 
 export {
+  JSKIT_PACKAGE_CONFIG_KEYS,
   ROOT_DEPENDENCY_SECTIONS,
   collectPackageDependencyIds,
   collectRootDependencySpecifiers,

package/internal/node/installedPackages.test.js

@@ -0,0 +1,51 @@
+import assert from "node:assert/strict";
+import test from "node:test";
+import {
+  createPackageMetadata,
+  JSKIT_PACKAGE_CONFIG_KEYS
+} from "./installedPackages.js";
+
+test("createPackageMetadata derives package identity from package.json", () => {
+  const metadata = createPackageMetadata({
+    name: "@example/runtime",
+    version: "0.1.0",
+    description: "Example runtime.",
+    jskit: {
+      kind: "runtime",
+      runtime: {
+        server: {
+          providers: []
+        }
+      }
+    }
+  });
+
+  assert.equal(metadata.packageId, "@example/runtime");
+  assert.equal(metadata.version, "0.1.0");
+  assert.equal(metadata.description, "Example runtime.");
+});
+
+test("createPackageMetadata accepts only the canonical package.json.jskit fields", () => {
+  assert.deepEqual(JSKIT_PACKAGE_CONFIG_KEYS, [
+    "capabilities",
+    "ci",
+    "kind",
+    "lifecycle",
+    "metadata",
+    "mutations",
+    "optionPolicies",
+    "options",
+    "runtime"
+  ]);
+  assert.throws(
+    () => createPackageMetadata({
+      name: "@example/runtime",
+      version: "0.1.0",
+      jskit: {
+        kind: "runtime",
+        packageId: "@example/other-runtime"
+      }
+    }),
+    /unknown field: packageId/
+  );
+});

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jskit-ai/kernel",
-  "version": "0.1.151",
+  "version": "0.1.152",
   "type": "module",
   "dependencies": {
     "json-rest-schema": "^1.0.17"

package/server/exportPolicy.test.js

@@ -61,7 +61,7 @@ test("kernel exported JS targets do not use star re-exports", async () => {
   assert.deepEqual(violations, []);
 });
 
-test("kernel package does not expose deprecated ./server aggregate entrypoint", async () => {
+test("kernel package exposes only its declared entrypoints", async () => {
   const packageJsonPath = path.join(PACKAGE_ROOT, "package.json");
   const packageJson = JSON.parse(await readFile(packageJsonPath, "utf8"));
   assert.equal(Object.hasOwn(packageJson.exports || {}, "./server"), false);

package/server/registries/serviceRegistrationRegistry.js

@@ -145,8 +145,9 @@ function normalizeServiceEventSpec(entry, { context = "service event" } = {}) {
 
 function normalizeServiceMetadata(value = {}) {
   const source = normalizeObject(value);
-  if (Object.hasOwn(source, "permissions")) {
-    throw new TypeError("service metadata.permissions is no longer supported. Define permissions on actions.");
+  const unsupportedFields = Object.keys(source).filter((key) => key !== "events");
+  if (unsupportedFields.length > 0) {
+    throw new TypeError(`service metadata supports only events. Received: ${unsupportedFields.sort().join(", ")}.`);
   }
   const eventsSource = normalizeObject(source.events);
   const events = {};

package/server/runtime/errors.test.js

@@ -60,7 +60,7 @@ test("isDomainError identifies DomainError subclasses and isAppError remains tru
   assert.equal(isAppError(appError), true);
 });
 
-test("createValidationError remains compatible", () => {
+test("createValidationError creates a 400 response with field errors", () => {
   const error = createValidationError({ email: "Invalid." });
 
   assert.equal(error.status, 400);

package/server/runtime/serviceRegistration.test.js

@@ -63,7 +63,7 @@ test("installServiceRegistrationApi exposes app.service and publishes declared e
   assert.equal(published[0].meta?.realtime?.event, "customers.record.changed");
 });
 
-test("app.service rejects deprecated permissions metadata", () => {
+test("app.service rejects unknown metadata fields", () => {
   const app = createContainer();
   app.singleton("domainEvents", () => ({
     async publish() {
@@ -82,14 +82,10 @@ test("app.service rejects deprecated permissions metadata", () => {
           }
         }),
         {
-          permissions: {
-            listRecords: {
-              require: "authenticated"
-            }
-          }
+          unexpected: true
         }
       ),
-    /metadata\.permissions is no longer supported/
+    /metadata supports only events.*unexpected/
   );
 });
 

package/shared/actions/actionDefinitions.test.js

@@ -138,14 +138,14 @@ test("normalizeActionDefinition stays channel-agnostic and ignores unknown extra
       schema: createSchema({})
     },
     idempotency: "none",
-    assistantTool: {
-      description: "Legacy field"
+    customExtension: {
+      description: "Ignored field"
     },
     execute: async () => ({})
   });
 
   assert.equal(typeof definition, "object");
-  assert.equal(Object.prototype.hasOwnProperty.call(definition, "assistantTool"), false);
+  assert.equal(Object.prototype.hasOwnProperty.call(definition, "customExtension"), false);
 });
 
 test("normalizeActionOutputDefinition accepts single schema definitions", () => {

package/shared/support/shellLayoutTargets.js

@@ -303,13 +303,6 @@ function normalizeShellOutletTargetRecord(
 ) {
   const record = normalizeObject(value);
   const resolvedContext = normalizeText(context) || "shell layout";
-  if (Object.hasOwn(record, "host") || Object.hasOwn(record, "position")) {
-    throw new Error(
-      `${resolvedContext} must declare ShellOutlet targets with "target" only. ` +
-      `Legacy "host" and "position" attributes are not supported.`
-    );
-  }
-
   const targetParts = resolveShellOutletTargetParts(
     {
       target: record.target

package/shared/support/shellLayoutTargets.test.js

@@ -106,15 +106,15 @@ test("discoverShellOutletTargetsFromVueSource ignores disabled default markers",
   assert.equal(discovered.defaultTargetId, "");
 });
 
-test("discoverShellOutletTargetsFromVueSource rejects split outlet attributes", () => {
+test("discoverShellOutletTargetsFromVueSource reads the target attribute", () => {
   const source = `
     <template>
       <ShellOutlet target="shell-layout:primary-menu" host="other-host" position="primary-menu" />
     </template>
   `;
 
-  assert.throws(
-    () => discoverShellOutletTargetsFromVueSource(source, { context: "ShellLayout.vue" }),
-    /must declare ShellOutlet targets with "target" only/
+  assert.deepEqual(
+    discoverShellOutletTargetsFromVueSource(source, { context: "ShellLayout.vue" }).targets,
+    [{ id: "shell-layout:primary-menu", default: false }]
   );
 });