Skip to content

docs(trace): document the header control field - #556

Merged
HappyDevs1 merged 1 commit into
mainfrom
docs/trace-format-control-field
Sep 7, 2026
Merged

docs(trace): document the header control field#556
HappyDevs1 merged 1 commit into
mainfrom
docs/trace-format-control-field

Conversation

@HappyDevs1

@HappyDevs1 HappyDevs1 commented Sep 7, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • docs/trace-format.md's "Header line" section listed every optional header field (recording, redaction, session, mock, browser, apps) except control — the block copied from a spec's control: block, defined in crates/flowproof-trace/schema/trace-v1.schema.json and read by flowproof audit to report control coverage.
  • Adds a bullet describing control (id/title/description, id required) matching the schema.

Test plan

  • Docs-only change — no code affected.
  • Cross-checked field names/requiredness against crates/flowproof-trace/schema/trace-v1.schema.json.

The header's control block (crates/flowproof-trace/schema/trace-v1.schema.json)
copies the spec's control: block into every trace and is what `flowproof audit`
reads to report control coverage, but the trace-format header section never
mentioned it — a schema/doc drift CHARTER.md invariant 5 exists to prevent.

Claude-Session: https://claude.ai/code/session_014DBCXghioodAVbbyuyCCWW
Copilot AI lite review requested due to automatic review settings September 7, 2026 12:41

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@HappyDevs1
HappyDevs1 merged commit 74ad993 into main Sep 7, 2026
9 checks passed
@HappyDevs1
HappyDevs1 deleted the docs/trace-format-control-field branch September 7, 2026 13:25
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.

2 participants