Add requirement traceability validation#2
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: dff6fa84e3
ℹ️ 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".
| for requirement_id in id_pattern.findall(line): | ||
| if requirement_id not in valid_ids: |
There was a problem hiding this comment.
Reject malformed security IDs in specs
The traceability check only validates tokens that already match \b[A-Z][A-Z0-9]+-\d{3}\b, so malformed references in ## Security Requirements (for example AUTHN-01 or lowercase IDs) are silently ignored and the check still passes. This defeats the purpose of enforcing spec-to-requirement traceability, because a typo can remove a requirement from validation without triggering CI.
Useful? React with 👍 / 👎.
Summary
scripts/check-requirement-traceability.shto validate spec security requirement IDs againstcontext/06-security-requirements.md.Validation
chmod +x scripts/check-requirement-traceability.shscripts/check-security-methodology.shpassed.scripts/verify-methodology.shpassed.scripts/check-requirement-traceability.shpassed.