@feniix/bridgekit 0.8.0 → 0.8.1

package/CHANGELOG.md

@@ -4,7 +4,19 @@ All notable changes to `@feniix/bridgekit` are documented here. The format
 follows [Keep a Changelog](https://keepachangelog.com/en/1.1.0/) and the project
 adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
-## [0.8.0] - Unreleased
+## [0.8.1] - Unreleased
+
+### Fixed
+
+- Union-of-objects validation no longer surfaces phantom missing-required-
+  property entries for every branch. When TypeBox emits an `anyOf`/`oneOf`
+  error at a path, sibling `required`/`additionalProperties` entries at the
+  same path are now suppressed so consumers see only the union summary.
+  `const`/`enum` discriminator errors at deeper paths are kept (they're
+  signal about which branch was intended).
+  Resolves [#35](https://github.com/feniix/bridgekit/issues/35).
+
+## [0.8.0] - 2026-05-26
 
 ### Breaking
 

package/dist/src/core/execute-tool.js

@@ -1,5 +1,32 @@
 import { Check, Errors } from "typebox/value";
 const ROOT_FIELD = "(root)";
+function suppressSiblingErrorsUnderUnion(errors) {
+    // For unions of objects, TypeBox emits per-branch `required` /
+    // `additionalProperties` errors at the union's path. The user only needs
+    // to satisfy ONE branch, so those entries are phantoms; the `anyOf` /
+    // `oneOf` summary at the same path is the real signal.
+    //
+    // `const` / `enum` errors at the same path survive — they carry real
+    // discriminator info about which branch was intended.
+    //
+    // Known trade-off (#38): discriminated unions where the user picks a
+    // branch correctly but forgets that branch's required props also lose
+    // the "you picked tag=X and forgot Y" hint. Keyword-aware branch
+    // matching is the follow-up.
+    const unionPaths = new Set();
+    for (const error of errors) {
+        if (error.keyword === "anyOf" || error.keyword === "oneOf") {
+            unionPaths.add(error.instancePath);
+        }
+    }
+    if (unionPaths.size === 0)
+        return errors;
+    return errors.filter((error) => {
+        if (!unionPaths.has(error.instancePath))
+            return true;
+        return error.keyword !== "required" && error.keyword !== "additionalProperties";
+    });
+}
 function expandTypeBoxError(error) {
     // Read offending property names from TypeBox's structured `params` rather
     // than parsing the message string. Two payoffs: locale independence (the
@@ -32,11 +59,15 @@ export function validatePortableToolArgs(tool, args) {
         return { ok: true };
     }
     // TypeBox can emit multiple errors per offending field (e.g. union/anyOf
-    // mismatches fire one error per failed branch). Dedupe by (field, message)
-    // using JSON.stringify so the key can't collide regardless of what
-    // characters field or message contain.
+    // mismatches fire one error per failed branch). First, when an anyOf/oneOf
+    // error fires at a path, drop sibling `required`/`additionalProperties`
+    // errors at the same path — they're phantoms (the consumer only needs to
+    // satisfy ONE branch, not all of them). Then dedupe the survivors by
+    // (field, message) using JSON.stringify so the key can't collide regardless
+    // of what characters field or message contain.
+    const rawErrors = suppressSiblingErrorsUnderUnion([...Errors(tool.parameters, args)]);
     const seen = new Set();
-    const errors = [...Errors(tool.parameters, args)]
+    const errors = rawErrors
         .flatMap((error) => expandTypeBoxError(error))
         .filter(({ field, message }) => {
         const key = JSON.stringify([field, message]);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@feniix/bridgekit",
-  "version": "0.8.0",
+  "version": "0.8.1",
   "description": "BridgeKit defines TypeBox-backed tools once and adapts them to pi, MCP, and other hosts.",
   "keywords": [
     "pi",