Skip to content

build: Package referenced review evidence - #14

Merged
BenWestgate merged 1 commit into
reviewability-v1from
codex/v1-package-review-evidence
Sep 27, 2026
Merged

BenWestgate merged 1 commit into
reviewability-v1from
codex/v1-package-review-evidence

Conversation

@BenWestgate

@BenWestgate BenWestgate commented Sep 21, 2026 •

Copy link
Copy Markdown
Owner

What

Include documentation CSV and JSON files in source distributions.

Why

The packaged alignment benchmark report links to CSV and JSON evidence that the current manifest omits. This adapts the earlier review-evidence packaging fix onto the current target so source releases remain self-contained for review.

The branch was refreshed onto the current reviewability-v1 after the BSD-3-Clause packaging fix landed. The resulting diff is again one focused MANIFEST.in line: it preserves recursive-include LICENSES * and adds only *.csv/*.json to the existing documentation include.

Validation

  • python -m build --no-isolation
  • python -m twine check dist/*
  • python tools/verify_wheel_environment.py
  • confirmed both referenced evidence files are present in the rebuilt sdist
  • current head 9e4deb7 passes the full GitHub Actions matrix

@BenWestgate BenWestgate added gate: adversarial review Resolve, merge, or explicitly defer before the next full adversarial review. area: packaging/release Packaging, artifacts, compatibility, and release qualification. documentation Improvements or additions to documentation labels Sep 24, 2026
@BenWestgate
BenWestgate force-pushed the codex/v1-package-review-evidence branch from 5bb27d6 to cf1a599 Compare September 25, 2026 13:31

@BenWestgate BenWestgate left a comment

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

AI-generated review (Claude), posted at the maintainer's request.

ACK 9e4deb7

  • The only docs/**/*.{csv,json} files are the two benchmark evidence files the report links.
  • Merges cleanly with #54.

@BenWestgate
BenWestgate merged commit 6c0d763 into reviewability-v1 Sep 27, 2026
12 checks passed
@BenWestgate
BenWestgate deleted the codex/v1-package-review-evidence branch September 27, 2026 23:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: packaging/release Packaging, artifacts, compatibility, and release qualification. documentation Improvements or additions to documentation gate: adversarial review Resolve, merge, or explicitly defer before the next full adversarial review.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant