docs/ci: add production DNP3 engineering contract and PR build gate - #1
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 7df21545a7
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| branches: | ||
| - master | ||
| push: | ||
| branches: | ||
| - master |
There was a problem hiding this comment.
Trigger the build gate for the main branch
The repository's active integration branch is main—the reviewed commit's parent is the main tip, and the existing Pages workflow also listens to main—but both event filters select only master. Consequently, pull requests targeting main and pushes to main never schedule this job, so the advertised application build gate provides no protection; include main or target the actual default branch.
AGENTS.md reference: AGENTS.md:L20-L24
Useful? React with 👍 / 👎.
Summary
Strengthens DNPTester governance without changing protocol/runtime behavior.
Changes
Dnp3MasterTester/AGENTS.mdwith root-cause-first workflow, typed Result/Try failure handling, bounded diagnostics, explicit state ownership, UI backpressure, resource lifecycle, and regression disciplineKey protections
No product feature behavior is intentionally changed.