Fix/publish script

[zackees]

Summary by CodeRabbit

• Chores

  • Version bumped to 2.1.21
  • Updated minimum zccache dependency to >=1.2.13

• New Features

  • Concurrent wheel uploads for improved release speed
  • Added PyPI verification after uploading wheels

• Bug Fixes

  • Enhanced artifact validation with detailed error messages
  • Fixed executable file permissions in wheel packaging

[coderabbitai]

Warning

Rate limit exceeded

@zackees has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 56 minutes and 42 seconds before requesting another review.

Your organization is not enrolled in usage-based pricing. Contact your admin to enable usage-based pricing to continue reviews beyond the rate limit, or try again in 56 minutes and 42 seconds.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

ℹ️ Review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 1ca7270f-63bf-483e-bbca-8a436d78db67

📥 Commits

Reviewing files that changed from the base of the PR and between 2bd0285 and 6652ae5.

⛔ Files ignored due to path filters (2)

• Cargo.lock is excluded by !**/*.lock
• uv.lock is excluded by !**/*.lock

📒 Files selected for processing (3)

• Cargo.toml
• ci/publish.py
• pyproject.toml

📝 Walkthrough

Walkthrough

Version bump from 2.1.20 to 2.1.21 across workspace metadata and package configuration, coupled with significant enhancements to the CI publish script to support readme metadata in wheels, implement concurrent PyPI uploads via thread pooling, and add post-upload verification.

Changes

Cohort / File(s)	Summary
Workspace & Package Metadata Cargo.toml, pyproject.toml	Version incremented from 2.1.20 to 2.1.21; readme field added to pyproject.toml; zccache dependency constraint updated to >=1.2.13.
CI Publish Script Enhancement ci/publish.py	Extended read_project_meta() to load readme from pyproject.toml; enhanced artifact handling with detailed error tracking; updated wheel metadata generation to include readme and content-type; fixed executable bit packaging; added concurrent wheel uploads via thread pool; introduced verify_published() for PyPI API validation; added --upload-only CLI flag with conditional pipeline execution.

Sequence Diagram

sequenceDiagram
    participant CI as CI/Publish Script
    participant Pool as Thread Pool
    participant PyPI as PyPI (uv publish)
    participant API as PyPI JSON API
    participant Out as Output/Logs

    CI->>CI: Load wheels from dist/wheels/
    CI->>CI: Build task queue (per wheel)
    CI->>Pool: Submit concurrent upload tasks
    
    loop For each wheel
        Pool->>PyPI: uv publish --check-url WHEEL
        PyPI-->>Pool: Success/Failure + output
        Pool-->>Out: Log recent output lines
    end
    
    Pool-->>CI: All results aggregated
    alt Any upload failed
        CI->>Out: Exit with failure
    else All uploads succeeded
        CI->>API: Query PyPI JSON API
        API-->>CI: Return published filenames
        CI->>CI: Verify expected wheels present
        alt Verification passed
            CI->>Out: Confirm publication success
        else Verification failed
            CI->>Out: Exit with verification error
        end
    end

Loading

Estimated code review effort

🎯 4 (Complex) | ⏱️ ~45 minutes

Poem

🐰 Hop, skip, and bump to version 2.1.21!
Readme in wheels, now shiny and clean,
Thread pool dancing through PyPI's scene,
Uploads concurrent, no more delays,
Publication verified in publish.py's ways! ✨

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 44.44% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The pull request title 'Fix/publish script' is directly related to the main changes, which involve fixes and improvements to the ci/publish.py script (the most substantial changes with high review effort). The title accurately reflects the primary focus of the changeset.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch fix/publish-script

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}