dohertj2/ScadaBridge
1
0
Fork 0
Code Issues 2 Pull Requests 1 Actions Packages Projects Releases Wiki Activity

R2-06: Edge Integrations round-2 fixes (6 tasks) #7

Closed
dohertj2 wants to merge 0 commits from r2-plan06 into main
pull from: r2-plan06
merge into: dohertj2:main
Conversation 1 Commits - Files Changed -
dohertj2 commented 2026-07-13 10:02:33 -04:00
Owner
Copy Link
Copy Source

Round-2 architecture-review fixes — PLAN-R2-06 (Edge Integrations), all 6 tasks. Source: archreview/06-edge-integrations.md round-2 findings.

TDD throughout; per-task pathspec commits; targeted + affected-consumer suites green (full-suite sweep deferred to the round-2 integration merge).

  • T1 db0285e2 fix(inbound): wire the compiled-handler cache as the IScriptArtifactChangeBus ApiMethod consumer (closes round-1 U2 partial; pairs with R2-05 T5 publisher)
  • T2 9668600c docs(inbound): correct bus-consumer claims in tracker + invalidation-contract doc + CLAUDE.md
  • T3 cc94b3a6 fix(esg): park deterministic ArgumentException (path-template/verb) failures instead of retrying to exhaustion
  • T4 b41b3b1b fix(esg): honor the declared response charset in ReadBodyBoundedAsync (UTF-8 fallback)
  • T5 fb9dc7ad fix(inbound): typed SITE_UNREACHABLE on AskTimeoutException at the router await (delete message-substring sniff)
  • T6 91bde0fa fix(inbound): 415 guard + lenient JSON parse cover chunked (no Content-Length) bodies

Note: T2 edits 00-MASTER-TRACKER.md/CLAUDE.md — reconcile with sibling round-2 PRs at merge. Part of the round-2 initiative (8 plans / 86 tasks).

Round-2 architecture-review fixes — **PLAN-R2-06 (Edge Integrations)**, all 6 tasks. Source: `archreview/06-edge-integrations.md` round-2 findings. TDD throughout; per-task pathspec commits; targeted + affected-consumer suites green (full-suite sweep deferred to the round-2 integration merge). - **T1** `db0285e2` fix(inbound): wire the compiled-handler cache as the `IScriptArtifactChangeBus` ApiMethod consumer (closes round-1 U2 partial; pairs with R2-05 T5 publisher) - **T2** `9668600c` docs(inbound): correct bus-consumer claims in tracker + invalidation-contract doc + CLAUDE.md - **T3** `cc94b3a6` fix(esg): park deterministic `ArgumentException` (path-template/verb) failures instead of retrying to exhaustion - **T4** `b41b3b1b` fix(esg): honor the declared response charset in `ReadBodyBoundedAsync` (UTF-8 fallback) - **T5** `fb9dc7ad` fix(inbound): typed `SITE_UNREACHABLE` on `AskTimeoutException` at the router await (delete message-substring sniff) - **T6** `91bde0fa` fix(inbound): 415 guard + lenient JSON parse cover chunked (no Content-Length) bodies Note: T2 edits `00-MASTER-TRACKER.md`/`CLAUDE.md` — reconcile with sibling round-2 PRs at merge. Part of the round-2 initiative (8 plans / 86 tasks).
dohertj2 added 6 commits 2026-07-13 10:02:34 -04:00
fix(inbound): wire the compiled-handler cache as the IScriptArtifactChangeBus ApiMethod consumer (plan R2-06 T1) db0285e2de
docs(inbound): correct bus-consumer claims in tracker + invalidation-contract doc + CLAUDE.md (plan R2-06 T2) 9668600c38
fix(esg): park buffered calls that fail deterministically with ArgumentException (path template/verb) instead of retrying to exhaustion (plan R2-06 T3) cc94b3a68c
fix(esg): honor the declared response charset in ReadBodyBoundedAsync with UTF-8 fallback (plan R2-06 T4) b41b3b1b31
fix(inbound): type SITE_UNREACHABLE on AskTimeoutException at the router await instead of message-substring sniffing (plan R2-06 T5) fb9dc7ad14
fix(inbound): 415 guard and lenient JSON parse cover chunked (no Content-Length) bodies (plan R2-06 T6) 91bde0fae6
dohertj2 commented 2026-07-13 11:48:35 -04:00
Author
Owner
Copy Link
Copy Source

Merged to main via the r2-integration fast-forward (main @ 1930f19b, pushed to origin). All commits from this branch are in main; the two integration-surfaced test fixes (IntegrationTests NodeName host-boot config, compile-cache assertion relax) landed on main as well. Closing — content fully integrated.

Merged to main via the r2-integration fast-forward (main @ 1930f19b, pushed to origin). All commits from this branch are in main; the two integration-surfaced test fixes (IntegrationTests NodeName host-boot config, compile-cache assertion relax) landed on main as well. Closing — content fully integrated.
dohertj2 closed this pull request 2026-07-13 11:48:35 -04:00

Pull request closed

This pull request cannot be reopened because the branch was deleted.
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
No items
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
dohertj2 (dohertj2)
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: dohertj2/ScadaBridge#7
Delete Branch "r2-plan06"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?