Polish repository for crates.io publish and public release#41
Conversation
- Fill in workspace and per-crate Cargo metadata: descriptions, authors, repository, homepage, readme, keywords, categories, rust-version. Pin inter-workspace path deps with version = "0.1.0" so cargo publish accepts them. - Bump unicode-segmentation to 1.13.2 in Cargo.lock (1.13.0 was yanked). - Add crate-root //! docs to tempyr-core, tempyr-index, tempyr-render, tempyr-interview, tempyr-linear, and tempyr-mcp so each crate has a meaningful crates.io front page. - Generalize sample fixtures away from the maintainer's name (caleb -> alice; task-b uses bob to keep the --owner filter test meaningful). Authorship attribution in LICENSE-MIT and docs/ author lines is preserved; the redaction tests in tempyr-journal still test the real path string by design. - Add CI / license / Rust / status badges and an "experimental side project" disclaimer to the README. - Drop the stub CHANGELOG.md until there's a real release to describe. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughThis PR updates workspace and crate metadata (adds workspace rust-version and expanded package fields), adds crate-level documentation and a new docs/spec, declares public modules and an error type for the linear crate, and standardizes many test fixtures by changing frontmatter owner values from ChangesWorkspace & Crate Metadata, Documentation, and Test Fixtures
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~25 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. Review rate limit: 2/5 reviews remaining, refill in 28 minutes and 22 seconds. Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@README.md`:
- Line 6: The status badge link "[](`#status`)"
points to a non-existent anchor; either remove the "(`#status`)" link target from
that badge or add a real "Status" heading (for example add a "## Status"
heading) above the existing blockquote so the anchor exists and the badge link
resolves correctly.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: ASSERTIVE
Plan: Pro
Run ID: 65952ac0-6af3-488b-a983-96f6b4d70fda
⛔ Files ignored due to path filters (1)
Cargo.lockis excluded by!**/*.lock
📒 Files selected for processing (34)
CHANGELOG.mdCargo.tomlREADME.mdcrates/tempyr-cli/Cargo.tomlcrates/tempyr-cli/src/commands/dispatch.rscrates/tempyr-cli/src/commands/migrate.rscrates/tempyr-cli/tests/integration.rscrates/tempyr-core/Cargo.tomlcrates/tempyr-core/src/graph.rscrates/tempyr-core/src/lib.rscrates/tempyr-core/src/node.rscrates/tempyr-core/src/ops.rscrates/tempyr-core/src/validate.rscrates/tempyr-index/Cargo.tomlcrates/tempyr-index/src/embeddings.rscrates/tempyr-index/src/fts.rscrates/tempyr-index/src/hybrid.rscrates/tempyr-index/src/incremental.rscrates/tempyr-index/src/indexer.rscrates/tempyr-index/src/lib.rscrates/tempyr-interview/Cargo.tomlcrates/tempyr-interview/src/lib.rscrates/tempyr-interview/src/session.rscrates/tempyr-journal-index/Cargo.tomlcrates/tempyr-journal/Cargo.tomlcrates/tempyr-linear/Cargo.tomlcrates/tempyr-linear/src/lib.rscrates/tempyr-mcp/Cargo.tomlcrates/tempyr-mcp/src/lib.rscrates/tempyr-render/Cargo.tomlcrates/tempyr-render/src/collector.rscrates/tempyr-render/src/formatter.rscrates/tempyr-render/src/lib.rsdocs/graphspec.md
💤 Files with no reviewable changes (1)
- CHANGELOG.md
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Summary
A pre-publication polish pass over the workspace. Nothing in the runtime behavior changes; this is metadata, docs, and sample-fixture cleanup.
description,authors,repository,homepage,readme,keywords,categories,rust-version = 1.89. Each crate gets its owndescriptionand inherits the rest.tempyr-clioverrideskeywords/categoriesso library crates aren't mis-tagged as command-line utilities. Inter-workspace path deps now carryversion = "0.1.0"socargo publishaccepts them;cargo publish --dry-run -p tempyr-coresucceeds.//!headers totempyr-core,tempyr-index,tempyr-render,tempyr-interview,tempyr-linear,tempyr-mcp(the other two already had them) so each crate has a meaningful front page on crates.io.unicode-segmentation1.13.0 → 1.13.2 inCargo.lock.calebwithaliceacross test fixtures and the README quick-start.test_list_by_statusneeded two distinct owners, sotask-busesbob.LICENSE-MITcopyright,docs/author lines, and thetempyr-journalredaction tests (which deliberately use the real path string) are intentionally left as-is.CHANGELOG.md— no point keeping it until there's a real release to describe.Test plan
cargo build --workspace --lockedcargo test --workspace --locked(all 18 test groups pass)cargo clippy --workspace --all-targets --locked -- -D warningscargo fmt --checkcargo audit(only pre-allowed warnings remain)cargo publish --dry-run -p tempyr-core --allow-dirty --no-verify🤖 Generated with Claude Code
Summary by CodeRabbit
New Features
Documentation
Chores