Review remediation: Lane Modbus #741

Open
opened 2026-10-22 06:05:19 -04:00 by dohertj2 · 0 comments
Owner

From the 2026-10-21 Fable system review. Findings R0x:ID refer to docs/reviews/2026-10-21/0x-*.md (location, evidence, failure scenario, fix recipe). Plan: docs/superpowers/plans/2026-10-22-review-remediation.md.

Lane Modbus

  • Deadband baselines on (value,status) (R07:H2).
  • Transport self-heals with a throttled in-gate connect (P3).
  • Dispose the failed TcpClient, one in-gate attempt, DNS under the timeout (L3).
  • Wire Family/MelsecSubFamily/DisableFC23 (U1): the editor + factory route the address through ModbusAddressParser and persist the resolved region/offset; DisableFC23 gates multi-write.
  • Coalescing on by default (gap 8), plus coils/DI/BitInRegister folding (U2).
  • String[]/BitInRegister[] writes (U3).
  • Region/type validation at the factory (P6).
  • Exception codes 01/02/03 are refusals (H4).
  • Write status mapping (H7); PDU length check (H9).
From the 2026-10-21 Fable system review. Findings `R0x:ID` refer to `docs/reviews/2026-10-21/0x-*.md` (location, evidence, failure scenario, fix recipe). Plan: `docs/superpowers/plans/2026-10-22-review-remediation.md`. ### Lane Modbus - Deadband baselines on `(value,status)` (`R07:H2`). - Transport self-heals with a throttled in-gate connect (`P3`). - Dispose the failed `TcpClient`, one in-gate attempt, DNS under the timeout (`L3`). - **Wire Family/MelsecSubFamily/DisableFC23** (`U1`): the editor + factory route the address through `ModbusAddressParser` and persist the resolved region/offset; `DisableFC23` gates multi-write. - Coalescing on by default (gap 8), plus coils/DI/BitInRegister folding (`U2`). - String[]/BitInRegister[] writes (`U3`). - Region/type validation at the factory (`P6`). - Exception codes 01/02/03 are refusals (`H4`). - Write status mapping (`H7`); PDU length check (`H9`).
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: dohertj2/lmxopcua#741