circuitjson-toolkit 1.1.1 → 1.1.2

package/README.md

@@ -43,6 +43,12 @@ caller-buffer mutation; explicit transfers detach exact buffers immediately
 after admission. See the
 [1.1.1 release notes](docs/release-notes-v1.1.1.md).
 
+Version 1.1.2 makes schematic SVG paint deterministic. Open primitives always
+emit `fill="none"`, filled primitives retain authored paint or use the shared
+theme fallback, and generic component and symbol bodies explicitly use the
+ECAD Forge schematic palette. See the
+[1.1.2 release notes](docs/release-notes-v1.1.2.md).
+
 Before 1.1.0:
 
 ```js
@@ -72,6 +78,8 @@ const model = document.model
 - Copy-on-write normalization of supported legacy CircuitJSON aliases through
   `CircuitJsonDocument.normalizeModel()`
 - Deterministic PCB, schematic, and BOM renderers
+- Explicit schematic SVG stroke/fill paint that does not depend on browser
+  defaults
 - Asset-backed `schematic_image` rows and hierarchical
   `schematic_sheet_symbol` rows with shared bounds and SVG behavior
 - Reusable exact PCB interaction and spatial indexes
@@ -301,6 +309,7 @@ copy while keeping sync, direct async, and worker results mutation-isolated.
 - [Testing and downstream conformance](docs/testing.md)
 - [1.1.0 release notes](docs/release-notes-v1.1.0.md)
 - [1.1.1 release notes](docs/release-notes-v1.1.1.md)
+- [1.1.2 release notes](docs/release-notes-v1.1.2.md)
 - [Library scope](spec/library-scope.md)
 
 ## Package scope

package/docs/api.md

@@ -330,7 +330,12 @@ the same option names across toolkits. Canonical `schematic_image` rows resolve
 their exact `asset_id` from `document.assets`; full asset mode renders the
 payload, while metadata-only or unresolved assets are omitted without a
 placeholder. `schematic_sheet_symbol` renders a hierarchical child box and is
-never treated as a selectable `schematic_sheet` page.
+never treated as a selectable `schematic_sheet` page. Since 1.1.2, every
+primitive emits explicit SVG fill paint: open arcs, polylines, and shapes use
+`fill="none"`; filled primitives keep valid authored paint or fall back to
+`var(--schematic-fill-color, #f1d8bd)`. Generic component and symbol bodies use
+the same fill variable plus
+`var(--schematic-default-ink-color, #008aa3)` for their stroke.
 
 ### `BomTableRenderer.render(document, options?)`
 

package/docs/migration.md

@@ -98,6 +98,13 @@ mutations cannot alter a queued parser input, project entry, or asset. With
 `transferInput: true`, exact transferable buffers detach at admission;
 partial, resizable, and shared ranges continue to use isolated copies.
 
+Since 1.1.2, schematic SVG renderers no longer inherit the browser's default
+black fill. Open primitives always render with `fill="none"`. A primitive with
+`is_filled: true` uses its valid authored `fill_color`/`fillColor`, or the
+shared `--schematic-fill-color` theme variable when no paint is authored.
+Generic component and symbol bodies explicitly use the shared schematic fill
+and ink variables. Public names, parameters, and return shapes are unchanged.
+
 ## Package subpaths
 
 - `circuitjson-toolkit/parser`

package/docs/release-notes-v1.1.2.md

@@ -0,0 +1,20 @@
+# circuitjson-toolkit 1.1.2
+
+## Deterministic schematic paint
+
+This patch removes browser-dependent SVG fill behavior from the canonical
+schematic renderer.
+
+- Open arcs, polylines, and unfilled shapes now emit `fill="none"` explicitly,
+  so they cannot become black-filled in a browser or host stylesheet.
+- Filled schematic primitives retain a valid authored `fill_color` or
+  `fillColor`. When `is_filled: true` has no authored paint, rendering uses
+  `var(--schematic-fill-color, #f1d8bd)`.
+- Generic `schematic_component` and `schematic_symbol` bodies now emit the
+  shared schematic fill and default-ink theme variables explicitly.
+- Filled paths render as polygons; open paths remain polylines. This preserves
+  geometry while making fill intent unambiguous.
+- Unsafe authored paints remain rejected by the existing SVG paint sanitizer.
+
+Public exports, method names, parameters, CircuitJSON document envelopes, and
+renderer return shapes are unchanged from 1.1.1.

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "circuitjson-toolkit",
-    "version": "1.1.1",
+    "version": "1.1.2",
     "description": "Canonical CircuitJSON parsing, project, rendering, query, manufacturing, simulation, and scene contracts",
     "keywords": [
         "circuitjson",
@@ -52,6 +52,7 @@
         "docs/provenance.md",
         "docs/release-notes-v1.1.0.md",
         "docs/release-notes-v1.1.1.md",
+        "docs/release-notes-v1.1.2.md",
         "docs/testing.md",
         "spec",
         "LICENSE",

package/src/ui/CircuitJsonSchematicSvgPrimitiveAttributes.mjs

@@ -1,5 +1,7 @@
 import { SafeXmlText } from './SafeXmlText.mjs'
 
+const DEFAULT_FILLED_PAINT = 'var(--schematic-fill-color, #f1d8bd)'
+
 /**
  * Builds SVG drawing attributes for schematic primitive rows.
  */
@@ -46,11 +48,16 @@ export class CircuitJsonSchematicSvgPrimitiveAttributes {
                     '"'
             )
         }
-        if (options.fill !== false && safeFill) {
+        const resolvedFill = CircuitJsonSchematicSvgPrimitiveAttributes.#fill(
+            element,
+            options,
+            safeFill
+        )
+        if (resolvedFill) {
             attributes.push(
                 'fill="' +
                     CircuitJsonSchematicSvgPrimitiveAttributes.#escapeHtml(
-                        safeFill
+                        resolvedFill
                     ) +
                     '"'
             )
@@ -71,6 +78,21 @@ export class CircuitJsonSchematicSvgPrimitiveAttributes {
         return attributes.length ? ' ' + attributes.join(' ') : ''
     }
 
+    /**
+     * Resolves deterministic fill semantics for one SVG primitive.
+     * @param {object} element CircuitJSON element row.
+     * @param {{ fill?: boolean }} options Attribute options.
+     * @param {string} safeFill Valid authored fill paint.
+     * @returns {string} Explicit SVG fill paint.
+     */
+    static #fill(element, options, safeFill) {
+        if (options.fill === false || element?.is_filled === false) {
+            return 'none'
+        }
+        if (safeFill) return safeFill
+        return element?.is_filled === true ? DEFAULT_FILLED_PAINT : 'none'
+    }
+
     /**
      * Resolves a finite number.
      * @param {unknown} value Number candidate.

package/src/ui/CircuitJsonSchematicSvgRenderer.mjs

@@ -151,7 +151,7 @@ export class CircuitJsonSchematicSvgRenderer {
                     component.schematic_component_id || ''
                 ) +
                 '">' +
-                '<rect class="schematic-component__body" x="' +
+                '<rect class="schematic-component__body" fill="var(--schematic-fill-color, #f1d8bd)" stroke="var(--schematic-default-ink-color, #008aa3)" x="' +
                 CircuitJsonSchematicSvgRenderer.#formatNumber(x) +
                 '" y="' +
                 CircuitJsonSchematicSvgRenderer.#formatNumber(y) +
@@ -218,7 +218,7 @@ export class CircuitJsonSchematicSvgRenderer {
             CircuitJsonSchematicSvgRenderer.#escapeHtml(
                 symbol.schematic_symbol_id || ''
             ) +
-            '"><rect class="schematic-symbol__body" x="' +
+            '"><rect class="schematic-symbol__body" fill="var(--schematic-fill-color, #f1d8bd)" stroke="var(--schematic-default-ink-color, #008aa3)" x="' +
             CircuitJsonSchematicSvgRenderer.#formatNumber(x) +
             '" y="' +
             CircuitJsonSchematicSvgRenderer.#formatNumber(y) +
@@ -517,7 +517,9 @@ export class CircuitJsonSchematicSvgRenderer {
                 CircuitJsonSchematicSvgRenderer.#pointsAttribute(points)
             ) +
             '"' +
-            CircuitJsonSchematicSvgPrimitiveAttributes.attributes(element) +
+            CircuitJsonSchematicSvgPrimitiveAttributes.attributes(element, {
+                fill: tag === 'polygon'
+            }) +
             '></' +
             tag +
             '>'