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

Surface inbound API method compile errors to operators (api-method validate / recompile) #75

New Issue
Open
opened 2026-11-06 15:24:47 -05:00 by dohertj2 · 0 comments
dohertj2 commented 2026-11-06 15:24:47 -05:00
Owner
Copy Link
Copy Source

When an inbound method's script fails to compile, callers get a generic Script compilation failed for this method (correct for the data plane). The real Roslyn diagnostic goes only to the central log, which on the wonder boxes means remote log access. There is no api-method validate, and the _knownBadMethods cache in InboundScriptExecutor cannot be observed or reset per method. A direct DB edit keeps serving the failure until a full-replace UpdateApiMethod or a restart.

Suggested fix:

  • api-method validate --id backed by a role-gated management ValidateApiMethodCommand that returns diagnostics (mirror template validate).
  • LastCompileError + IsKnownBad on api-method get.
  • A lightweight api-method recompile --id (RecompileApiMethodCommand) to re-run CompileAndRegister without re-sending the entity.

Current state: still open. A grep of src/ for ValidateApiMethod|RecompileApiMethod|LastCompileError|IsKnownBad returns nothing.

Source: docs/known-issues/2026-06-25-inbound-api-compile-error-not-client-visible.md (Status: OPEN, items 1 and 2)

When an inbound method's script fails to compile, callers get a generic `Script compilation failed for this method` (correct for the data plane). The real Roslyn diagnostic goes only to the central log, which on the wonder boxes means remote log access. There is no `api-method validate`, and the `_knownBadMethods` cache in `InboundScriptExecutor` cannot be observed or reset per method. A direct DB edit keeps serving the failure until a full-replace `UpdateApiMethod` or a restart. Suggested fix: - `api-method validate --id` backed by a role-gated management `ValidateApiMethodCommand` that returns diagnostics (mirror `template validate`). - `LastCompileError` + `IsKnownBad` on `api-method get`. - A lightweight `api-method recompile --id` (`RecompileApiMethodCommand`) to re-run `CompileAndRegister` without re-sending the entity. **Current state:** still open. A grep of `src/` for `ValidateApiMethod|RecompileApiMethod|LastCompileError|IsKnownBad` returns nothing. Source: `docs/known-issues/2026-06-25-inbound-api-compile-error-not-client-visible.md` (Status: OPEN, items 1 and 2)
dohertj2 added the type:featurepriority:p2migrated labels 2026-11-06 15:24:47 -05:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
blocked
Blocked by dependency or external hold
 cross-repo
Needs coordinated change in another family repo (wire contract etc.)
 migrated
Migrated from a markdown backlog (2026-11 standardization)
 priority:p0
Must-have / blocking
 priority:p1
High priority
 priority:p2
Normal priority
 priority:p3
Low priority / nice to have
 type:bug
Defect in shipped behaviour
 type:debt
Technical debt / hygiene
 type:deploy-gate
Rollout/deploy/migration step not yet done on an environment
 type:docs
Documentation work
 type:feature
Feature implementation
 type:followup
Follow-up deferred from a plan, PR or gate
 type:infra
Infrastructure and scaffolding
 type:review-finding
Unfixed finding from an architecture/code review
 type:test
Test work
No Label migrated priority:p2 type:feature
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#75
Delete Branch "%!s()"

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?