Skip to content

[rosary-d19eca] chore(meta): crate metadata + README hygiene#367

Open
jamestexas wants to merge 3 commits into
mainfrom
chore/crate-readme-hygiene
Open

[rosary-d19eca] chore(meta): crate metadata + README hygiene#367
jamestexas wants to merge 3 commits into
mainfrom
chore/crate-readme-hygiene

Conversation

@jamestexas

Copy link
Copy Markdown
Contributor

What

Gets the crate/README story coherent (bead rosary-d19eca):

  • [workspace.package] inheritance — edition/license/repository/homepage defined once, inherited by all three crates. Homepage now points at https://docs.rosary.bot (matching the GitHub repo homepage) instead of the repo URL.
  • publish = false on all three crates. Audit finding: the crates.io rosary name is already taken by an unrelated crate; bdr, rosary-crypto, and rsry are free (checked 2026-07-15). Distribution today is binary + OCI image, so publishing is now an explicit decision rather than an accident waiting on cargo publish.
  • Full registry metadata (readme, keywords, categories) on every crate, so a future publish flip is one line.
  • bdr license split documented — Apache-2.0 vs the workspace AGPL is deliberate (liberally-licensed decomposition core); now stated in the manifest and the new crates/bdr/README.md.
  • Tool-count drift fixed: handlers register 41 MCP tools but README/CLAUDE.md/docs said 40 — rsry_expand_ref was missing from both tool tables. Counts and tables updated across README.md, CLAUDE.md, docs/ARCHITECTURE.md, docs/GETTING_STARTED.md.
  • README badges: CI + license.

Verification

task check passes through contract/rules/compile/lint/test locally; the local smells failure is the documented darwin phantom (dead_code/god_file flood CI never emits — CI enforces only dup-def/long_file/long_file_rosary, and this diff adds no symbols or source lines). Tool count verified against src/serve/handlers/mod.rs (41 unique rsry_* registrations).

🤖 Generated with Claude Code

https://claude.ai/code/session_019azu4MnUTnEHN3V5grqM2E

jamestexas and others added 3 commits July 15, 2026 11:04
…ublishing, fix tool-count drift

- Add [workspace.package] (edition/license/repository/homepage) and inherit
  it in all three crates; homepage now points at docs.rosary.bot.
- publish = false on rosary/bdr/rosary-crypto: the crates.io `rosary` name
  belongs to an unrelated crate (bdr + rosary-crypto + rsry are free as of
  2026-07-15); flip deliberately if a publish story lands.
- Add readme/keywords/categories metadata; document that bdr's Apache-2.0
  (vs workspace AGPL) is deliberate.
- New crates/bdr/README.md (crypto already had one).
- Fix stale "40 tools" → 41 across README/CLAUDE.md/docs and add the missing
  rsry_expand_ref to both tool tables.
- README: CI + license badges.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_019azu4MnUTnEHN3V5grqM2E
…t + bdr Apache-2.0 license text

Fresh-eyes review findings on #367:
- docs/ARCHITECTURE.md:273 heading said "MCP Tools (40)" — the one count
  the first pass's grep pattern couldn't match.
- crates/bdr declared license = "Apache-2.0" (pre-existing) with no license
  text on disk; add the canonical LICENSE so the SPDX claim is backed.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_019azu4MnUTnEHN3V5grqM2E
The Apache-2.0 in crates/bdr/Cargo.toml predated this PR but was never
intentional — user confirmed the whole repo is AGPL. Inherit the workspace
license, drop the Apache LICENSE file added in 64aec0f, fix the README claim.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_019azu4MnUTnEHN3V5grqM2E
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