Skip to content

feat(server): project committed event notification values - #455

Merged
jscott3201 merged 1 commit into
devfrom
feat/pr-0404-notification-parameters-values
Aug 29, 2026
Merged

feat(server): project committed event notification values#455
jscott3201 merged 1 commit into
devfrom
feat/pr-0404-notification-parameters-values

Conversation

@jscott3201

Copy link
Copy Markdown
Owner

Summary

  • project committed built-in intrinsic and Event Enrollment outputs into the seven reachable structured NotificationParameters alternatives
  • carry one validated payload snapshot through confirmed, unconfirmed, and retry delivery without rereading mutable source state
  • omit AckRequired, From State, and Event Values from ACK notifications while accepting the reduced ACK envelope on decode

Behavior

  • preserves intrinsic committed Message Text and keeps Event Enrollment message-absent
  • uses the effective reliability event type for transitions into and out of fault
  • builds reliability property values from explicit typed properties in deterministic family-specific order
  • commits local transition history but suppresses frame emission when required projection data is malformed or missing
  • leaves legacy object implementations and public transition APIs unchanged

Validation

  • cargo fmt --all -- --check
  • cargo clippy --workspace --exclude rusty-bacnet --all-targets --locked
  • cargo test --workspace --exclude rusty-bacnet --locked
  • cargo check -p rusty-bacnet --tests --locked
  • python3 scripts/generate-conformance-docs.py --check
  • bash .github/scripts/check-file-size.sh
  • bash .github/scripts/check-no-secrets.sh
  • git diff --check
  • focused and package suites for bacnet-services and bacnet-server

No README, Python, dependencies, CI, recipient routing policy, or support claims are changed.

Closes #135

@jscott3201
jscott3201 merged commit d563290 into dev Aug 29, 2026
16 checks passed
@jscott3201
jscott3201 deleted the feat/pr-0404-notification-parameters-values branch August 30, 2026 02:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Populate EventNotification message text and structured event values

1 participant