Dormant Core.VirtualTags engine decision, ScriptSandbox reference caching, single drain-gate predicate (archreview r3 item 9) #603

Closed
opened 2026-10-08 02:31:11 -04:00 by dohertj2 · 1 comment
Owner

Oldest open Highs. ~1,100-line Core.VirtualTags engine dormant with no retire/sanction decision; ScriptSandbox.Build rebuilds the BCL reference set every compile; the alarm drain gate's "replicated" test (PeerAddress || SyncListenPort) disagrees with LocalDb's (PeerAddress || ApiKey).
Sources: archreview/00-OVERALL.md:120 (02/U1, 02/P1, 02/S15); src/Server/ZB.MOM.WW.OtOpcUa.Runtime/ServiceCollectionExtensions.cs:180-185 vs LocalDbSetup.ReplicationIsConfigured.

  • Decide the engine (retiring closes 02/S1, S10, half of S3, C1, U7-b)
  • Cache the sandbox reference set statically
  • Use ReplicationIsConfigured as the single predicate

Filed 2026-10-08 by the pending/deferred-work sweep after the #562–#591 issue sweep.

Oldest open Highs. ~1,100-line `Core.VirtualTags` engine dormant with no retire/sanction decision; `ScriptSandbox.Build` rebuilds the BCL reference set every compile; the alarm drain gate's "replicated" test (`PeerAddress || SyncListenPort`) disagrees with LocalDb's (`PeerAddress || ApiKey`). Sources: archreview/00-OVERALL.md:120 (02/U1, 02/P1, 02/S15); src/Server/ZB.MOM.WW.OtOpcUa.Runtime/ServiceCollectionExtensions.cs:180-185 vs `LocalDbSetup.ReplicationIsConfigured`. - [ ] Decide the engine (retiring closes 02/S1, S10, half of S3, C1, U7-b) - [ ] Cache the sandbox reference set statically - [ ] Use `ReplicationIsConfigured` as the single predicate _Filed 2026-10-08 by the pending/deferred-work sweep after the #562–#591 issue sweep._
Author
Owner

Resolved on master (merge e9e35baa).

  • Engine decision: retired. VirtualTagEngine/Source/Definition and TimerTriggerScheduler deleted (a8703c6a); the scheduler was welded to the engine and had no production consumer.
  • ScriptSandbox now caches the BCL + pinned metadata references statically (1b37b2f2).
  • The alarm drain gate and table registration share LocalDbSetup.ReplicationIsConfigured (PeerAddress || ApiKey) (4f6231fa). Behaviour change noted in review: a node with only a SyncListenPort is no longer gated (it shares nothing, fail-closed), and an ApiKey-only passive node now is. The passive half of a pair was further drain-gated on its mesh partner in 3be8a121 (review R6, merge 31346f1e).

Closed 2026-10-08 by the hardware-available issue sweep (master d898262c).

Resolved on master (merge e9e35baa). - [x] Engine decision: **retired**. `VirtualTagEngine`/Source/Definition and `TimerTriggerScheduler` deleted (a8703c6a); the scheduler was welded to the engine and had no production consumer. - [x] `ScriptSandbox` now caches the BCL + pinned metadata references statically (1b37b2f2). - [x] The alarm drain gate and table registration share `LocalDbSetup.ReplicationIsConfigured` (PeerAddress || ApiKey) (4f6231fa). Behaviour change noted in review: a node with only a SyncListenPort is no longer gated (it shares nothing, fail-closed), and an ApiKey-only passive node now is. The passive half of a pair was further drain-gated on its mesh partner in 3be8a121 (review R6, merge 31346f1e). *Closed 2026-10-08 by the hardware-available issue sweep (master d898262c).*
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: dohertj2/lmxopcua#603