docs: v5.2.0 — extract and formally reference every TC18 SHOULD/MAY clause - #139
Merged
Merged
Conversation
…lause Mirrors c-RCP's and cpp-RCP's own identical audits. Individually classified all 51 non-legal SHOULD/MAY occurrences in TC18. 7 already-implemented optional capabilities got a real tc18 citation added to their existing requirement entry (0/608 requirements had one before this pass). The non-testable lines are individually cited in new docs/TC18-NON-NORMATIVE-CLAUSES.md. Honestly flags 4 MAY-described capabilities as genuinely uncertain rather than papering over them with a citation — most notably a possible 32-bit-vs-48-bit time-domain conflation in the Timed-request readiness check (REQ-TIME-002/003), the same bug class found and fixed this session in go-RCP's conditional-request envelopes. Not fixed here; recorded for its own dedicated investigation. No code behavior changed. Signed-off-by: Matt <47545907+SoundMatt@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
tc18citation added to their existing requirement entry:REQ-PWR-001,REQ-LIFE-001,REQ-SPI-005,REQ-ADC-007,REQ-TRIG-001,REQ-SEQ-001,REQ-SAFEMEAS-001. This crate had 0/608 requirements citing TC18 before this pass.docs/TC18-NON-NORMATIVE-CLAUSES.md.REQ-TIME-002/REQ-TIME-003's Timed-request readiness check usesAvtpTimestamp(32-bit, ~4.3s rollover) for what TC18 §11.2.2.5 defines as a 48-bit, 3.25-day-rolloverpresentation_time— the same class of time-domain conflation bug found and fixed this session in go-RCP's conditional-request envelopes. Not fixed here — recorded for its own dedicated investigation, not silently resolved with a citation.Test plan
.fusa-reqs.jsonvalid JSON, surgical diffcargo build --release: cleancargo test --release: 1107+32 tests passed, 0 failedrsfusa check(fresh v0.3.17 build matching CI's exact pin): 0 errorsrsfusa trace --req-coverage 100: gate passed