Skip to content

fix(data-audit): implement --format both, count validation, and seeded RNG#48

Open
BWM0223 wants to merge 1 commit into
lily-protocol:mainfrom
BWM0223:fix/data-audit-format
Open

fix(data-audit): implement --format both, count validation, and seeded RNG#48
BWM0223 wants to merge 1 commit into
lily-protocol:mainfrom
BWM0223:fix/data-audit-format

Conversation

@BWM0223

@BWM0223 BWM0223 commented Jun 25, 2026

Copy link
Copy Markdown

Summary

Fixes #38 - data_audit.ts broken --format both and input validation

Changes

  • --format both now correctly writes both JSON and CSV files
  • --json / --csv flags removed (consolidated into --format)
  • Count validation: non-negative integers only, clear argparse-style error
  • SeededRng class for deterministic output with same seed
  • Clean separation of validateOptions() and runAudit()

Acceptance Criteria

  • --format both creates both .json and .csv files
  • Negative counts fail with clear error
  • Same seed produces deterministic data

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.

[$50 BOUNTY] [Typescript] Fix data_audit.ts broken --format both and input validation

1 participant