Skip to content

fix(compat): enforce required output formats - #33

Merged
Terastar-Paperclip merged 1 commit into
mainfrom
fix/qua-26-required-formats
Nov 9, 2026
Merged

Terastar-Paperclip merged 1 commit into
mainfrom
fix/qua-26-required-formats

Conversation

@Terastar-Paperclip

Copy link
Copy Markdown
Contributor

Change

Reject explicit SupportedFormats declarations that omit markdown or json before running the format contract. Previously the suite silently skipped required checks, so a nonconforming exporter could pass. Nil continues to declare the full surface; csv remains optional.

Exporters with an incomplete or misspelled declaration must correct it when updating this module. SkipDataPath continues to skip data-path checks but does not waive the required format declaration.

Validation

  • go test -count=1 ./... and go vet ./... in compat/
  • Child-process tests verify the actual fatal diagnostic for empty, single-format, and misspelled declarations; existing nil and partial-codec tests and an explicit full-set test cover valid declarations.

Refs QUA-26.

@Terastar-Paperclip
Terastar-Paperclip marked this pull request as ready for review November 9, 2026 23:42
@Terastar-Paperclip
Terastar-Paperclip enabled auto-merge (squash) November 9, 2026 23:42
@Terastar-Paperclip
Terastar-Paperclip merged commit 854f565 into main Nov 9, 2026
12 checks passed
@Terastar-Paperclip
Terastar-Paperclip deleted the fix/qua-26-required-formats branch November 9, 2026 23:45
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