fix(renderer): stop emitting require_live_session, rejected by bread 0.3.0 (#202)

bread 0.3.0 removed the `hardware.require_live_session` field and rejects it at
load time as an unknown key. The renderer still emitted it, the catalog still
declared it as a bread topology field, both project templates still seeded it,
and the TS contract still typed it — so every bread config workbench generated
was dead on arrival: the provider exits immediately, the runtime's hello
handshake fails on the dead stdout pipe, and the runtime aborts.

The field's intent now rides on bus_path: a real path opens a live CRUMBS
session and startup throws if the bus cannot be opened, while a mock:// path
seeds an inventory instead. Both templates already set bus_path, so behaviour is
preserved.

The checked-in golden fixture encoded the broken output as expected, so it
asserted the bug was correct; updated to match.

Verified by rendering both templates and running the released bread 0.3.0
binary's --check-config over the output: bioreactor-manual (/dev/i2c-1) and
mixed-bus-mock (mock://) both validate.

Same root cause as the failure found on real hardware during the
anolishq/anolis#138 run (anolishq/anolis-projects#31).

Author: CameronBrooks11

anolis_workbench/catalog/providers.json

@@ -48,7 +48,6 @@
       "check_config_flag": "--check-config",
       "topology_fields": [
         { "key": "provider_name",       "type": "string", "required": false, "default": "bread0" },
-        { "key": "require_live_session","type": "bool",   "default": false },
         { "key": "query_delay_us",      "type": "int",    "default": 10000 },
         { "key": "timeout_ms",          "type": "int",    "default": 100 },
         { "key": "retry_count",         "type": "int",    "default": 2 }

anolis_workbench/core/renderer.py

@@ -221,9 +221,11 @@ def _render_bread(pdata: dict, path_data: dict) -> dict:
     if "provider_name" in pdata:
         doc["provider"] = {"name": pdata["provider_name"]}
 
+    # No require_live_session: bread 0.3.0 removed the field and rejects it as an
+    # unknown key. The intent now rides on bus_path — a real path opens a live
+    # CRUMBS session and startup throws if the bus cannot be opened, while a
+    # mock:// path seeds an inventory instead.
     hardware: dict = {"bus_path": path_data.get("bus_path", "")}
-    if "require_live_session" in pdata:
-        hardware["require_live_session"] = pdata["require_live_session"]
     if "query_delay_us" in pdata:
         hardware["query_delay_us"] = pdata["query_delay_us"]
     if "timeout_ms" in pdata:

anolis_workbench/templates/bioreactor-manual/system.json

@@ -62,7 +62,6 @@
       "bread0": {
         "kind": "bread",
         "provider_name": "bread0",
-        "require_live_session": true,
         "query_delay_us": 15000,
         "timeout_ms": 150,
         "retry_count": 3,

anolis_workbench/templates/mixed-bus-mock/system.json

@@ -53,7 +53,6 @@
       "bread0": {
         "kind": "bread",
         "provider_name": "bread0",
-        "require_live_session": false,
         "query_delay_us": 10000,
         "timeout_ms": 100,
         "retry_count": 2,

frontend/src/lib/contracts.ts

@@ -141,7 +141,6 @@ export interface SimProviderConfig extends UnknownRecord {
 export interface BreadProviderConfig extends UnknownRecord {
   kind: "bread";
   provider_name?: string;
-  require_live_session?: boolean;
   query_delay_us?: number;
   timeout_ms?: number;
   retry_count?: number;

tests/fixtures/bioreactor/provider-bread.bioreactor.yaml

@@ -3,7 +3,6 @@ provider:
 
 hardware:
   bus_path: /dev/i2c-1
-  require_live_session: true
   query_delay_us: 15000
   timeout_ms: 150
   retry_count: 3