Skip to content

Fail D1 sync when submitter metadata is missing#23

Merged
bradvin merged 1 commit into
mainfrom
fix-submitter-sync
Mar 29, 2026
Merged

Fail D1 sync when submitter metadata is missing#23
bradvin merged 1 commit into
mainfrom
fix-submitter-sync

Conversation

@bradvin
Copy link
Copy Markdown
Owner

@bradvin bradvin commented Mar 29, 2026

Summary

  • derive tool submitters from every tool PR, not only approved ones
  • keep content validation submitter-agnostic for authors and PR validation
  • fail D1 sync generation when a tool is missing PR-derived submitter metadata
  • add regression coverage for missing submitter mappings during sync

Verification

  • npm test
  • npm run validate:content
  • node ./scripts/generate-d1-sync-sql.mjs

@bradvin bradvin merged commit 67a0c49 into main Mar 29, 2026
1 check passed
@bradvin bradvin deleted the fix-submitter-sync branch March 29, 2026 11:09
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.

1 participant