Skip to content

ci(release): stop publishing to crates.io - #11

Merged
senamakel merged 1 commit into
tinyhumansai:mainfrom
senamakel:release-no-cratesio
Aug 23, 2026
Merged

ci(release): stop publishing to crates.io#11
senamakel merged 1 commit into
tinyhumansai:mainfrom
senamakel:release-no-cratesio

Conversation

@senamakel

Copy link
Copy Markdown
Member

What

Removes the two cargo publish steps from the release workflow, and the cargo package --package tinydocs step that cannot work without them.

Why

The v0.1.14 release run (32628924101) failed at Package and publish the bus contract with:

error: no token found, please run `cargo login`

Neither tinydocs nor tinydocs-bus exists on crates.io, and neither is meant to: hosts consume this repository as a git submodule for the wire contract and load the module artifact this workflow builds. native-bundles has needs: publish, so a registry upload nobody reads was gating every module artifact in the release. Same conclusion tinywallet reached in tinyhumansai/tinywallet#25.

cargo package stays for tinydocs-bus — it is the cheap check that the manifest is packageable. It cannot stay for tinydocs, which depends on tinydocs-bus by path: packaging rewrites a path dependency to a registry version and then looks it up, so it fails on a crate that by design will never be in the index.

Verification

Re-cutting the release after this merges is the check.

The release workflow previously attempted to publish both the bus contract and the main crate to crates.io, but neither crate is actually published there — they are consumed as git submodules and the real artifact is the module bundle built later in the same job. The missing `CARGO_REGISTRY_TOKEN` caused these steps to fail, taking the entire release down with them and blocking the `native-bundles` job that depends on this workflow succeeding. The fix removes the `cargo publish` calls entirely and keeps only `cargo package` for the bus contract as a cheap manifest validation step, while dropping the packaging of the main crate because its path dependency on the bus contract makes it unpackageable without a registry lookup.
Auto-committed-on: dragonfly
Co-authored-by: Medulla <medulla@tinyhumans.ai>
@coderabbitai

Copy link
Copy Markdown

Warning

Review limit reached

Your included review limit has been reached.

You’re in a promotional period — use the checkbox below to run this review for free:

  • Run review for free

On-demand reviews are free for the next 28 days. After that, they cost $0.25 per reviewed file.

How can I continue?

Run this review now using the option above, or comment @coderabbitai review --use-credits.

You can also wait for the limit to reset (next review available in 11 minutes), then comment @coderabbitai review or push new commits to the PR.

An organization admin can change what happens after included review limits in Billing.

How do review limits work?

CodeRabbit enforces per-developer PR review limits within each organization.

For paid Pro and Pro+ reviews, CodeRabbit uses a developer's included PR review attempts over the past 7 days to set the current hourly allowance. At typical activity levels, the full plan allowance applies. Higher sustained activity can lower the allowance until earlier attempts leave the 7-day window.

Please refer docs for additional details.

Review details
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 56bb9686-dc8d-49e9-80be-20a2fd348a4b

📥 Commits

Reviewing files that changed from the base of the PR and between d17f7e3 and 507ad28.

📒 Files selected for processing (1)
  • .github/workflows/release.yml

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

Comment @coderabbitai help to get the list of available commands.

@tinysweepertinysweeperBot added the priority: p3 Whenever. Cosmetic, a nicety, or a cleanup with no user visible effect. label Aug 23, 2026

@tinysweepertinysweeperBot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

tinysweeper found nothing blocking. Approving.

$0.0000 · 0 in / 0 out · 111 embedded · openrouter/openai/text-embedding-3-small

@senamakel
senamakel merged commit 08c80b8 into tinyhumansai:mainAug 23, 2026
12 checks passed
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

priority: p3Whenever. Cosmetic, a nicety, or a cleanup with no user visible effect.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

@senamakel