-
Notifications
You must be signed in to change notification settings - Fork 11
chore: release #133
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. Weβll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
chore: release #133
Conversation
π WalkthroughWalkthroughWorkspace crate versions bumped and changelogs added: soar-cli 0.9.2, soar-core 0.11.0, soar-db 0.3.0, soar-dl 0.7.2, soar-package 0.2.0, soar-registry 0.2.0, soar-config 0.2.0, soar-utils 0.1.2; multiple CHANGELOGs note GHCR apply feature and refactor items about error messages and query icons. Changes
Estimated code review effortπ― 2 (Simple) | β±οΈ ~10 minutes Poem
π₯ Pre-merge checks | β 2 | β 1β Failed checks (1 inconclusive)
β Passed checks (2 passed)
βοΈ Tip: You can configure your own custom pre-merge checks in the settings. π Recent review detailsConfiguration used: defaults Review profile: CHILL Plan: Pro β Files ignored due to path filters (1)
π Files selected for processing (17)
β Files skipped from review due to trivial changes (5)
π§ Files skipped from review as they are similar to previous changes (9)
β° Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (2)
π Additional comments (3)
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. Comment |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 0
π§Ή Nitpick comments (1)
crates/soar-registry/CHANGELOG.md (1)
12-12: Placeholder commit hash in existing entry.The 0.1.1 changelog entry references commit
0000000, which appears to be a placeholder. While this is pre-existing and not introduced by this PR, consider updating it to the actual commit hash for historical accuracy.
π Review details
Configuration used: defaults
Review profile: CHILL
Plan: Pro
β Files ignored due to path filters (1)
Cargo.lockis excluded by!**/*.lock
π Files selected for processing (13)
CHANGELOG.mdCargo.tomlcrates/soar-cli/Cargo.tomlcrates/soar-core/CHANGELOG.mdcrates/soar-core/Cargo.tomlcrates/soar-db/CHANGELOG.mdcrates/soar-db/Cargo.tomlcrates/soar-dl/CHANGELOG.mdcrates/soar-dl/Cargo.tomlcrates/soar-package/CHANGELOG.mdcrates/soar-package/Cargo.tomlcrates/soar-registry/CHANGELOG.mdcrates/soar-registry/Cargo.toml
β° Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (2)
- GitHub Check: test
- GitHub Check: coverage
π Additional comments (13)
crates/soar-cli/Cargo.toml (1)
3-3: LGTM!Version bump to 0.9.2 is consistent with the release PR objectives and the API-compatible patch increment for the error message refactor.
crates/soar-db/Cargo.toml (1)
3-3: LGTM!Version bump to 0.2.1 aligns with the workspace-wide release for the error message refactor.
crates/soar-db/CHANGELOG.md (1)
2-6: LGTM!Changelog entry correctly documents the 0.2.1 release with the refactor commit reference and proper comparison link.
crates/soar-core/CHANGELOG.md (1)
2-6: LGTM!Changelog entry for 0.10.1 is well-formatted and consistent with the workspace-wide release pattern.
crates/soar-dl/Cargo.toml (1)
3-3: LGTM!Version bump to 0.7.2 is consistent with the coordinated workspace release.
crates/soar-registry/CHANGELOG.md (1)
2-6: LGTM!Changelog entry for 0.1.2 correctly documents the refactor with the appropriate commit reference.
crates/soar-package/Cargo.toml (1)
3-3: LGTM!Version bump to 0.1.2 aligns with the coordinated workspace release.
crates/soar-core/Cargo.toml (1)
3-3: LGTM!Version bump to 0.10.1 is consistent with the workspace-wide release. Internal workspace dependencies will correctly resolve to the updated versions.
CHANGELOG.md (1)
2-7: LGTM! Changelog entry is well-formatted.The new version 0.9.2 changelog entry follows the conventional format with proper date, category, commit reference, and version comparison link. The refactor note aligns with the referenced commit.
crates/soar-dl/CHANGELOG.md (1)
2-7: LGTM! Changelog entry is consistent and properly formatted.The version 0.7.2 entry uses the correct crate-specific tag format and matches the refactor documented across all workspace crates in this release.
crates/soar-package/CHANGELOG.md (1)
2-7: LGTM! Changelog entry follows the correct format.The version 0.1.2 entry is properly formatted with the crate-specific tag prefix and documents the same refactor as other workspace crates.
crates/soar-registry/Cargo.toml (1)
3-3: LGTM! Version bump is correct.The version bump to 0.1.2 aligns with the PR objectives and represents an API-compatible patch release.
Cargo.toml (1)
51-55: LGTM! Workspace dependency versions are correctly synchronized.All workspace crate versions have been properly updated to match their individual package version bumps:
- soar-core: 0.10.0 β 0.10.1
- soar-db: 0.2.0 β 0.2.1
- soar-dl: 0.7.1 β 0.7.2
- soar-package: 0.1.1 β 0.1.2
- soar-registry: 0.1.1 β 0.1.2
These changes align with the PR objectives and maintain API compatibility across the workspace.
ca39b1c to
66609f3
Compare
Codecov Reportβ All modified and coverable lines are covered by tests. π’ Thoughts on this report? Let us know! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 1
π€ Fix all issues with AI agents
In @crates/soar-db/CHANGELOG.md:
- Around line 2-7: The changelog is missing a Breaking Changes section: add a
new "### β οΈ Breaking Changes" entry for v0.3.0 that explicitly states the public
API structs NewPackage, PackageListing, and Package no longer include the
version_upstream field; explain the change, list the affected symbols
(NewPackage, PackageListing, Package, version_upstream), provide a brief
migration note instructing consumers to remove uses of version_upstream or map
it from their own metadata before constructing these structs, and include a
short example description of how to adapt callers (e.g., stop setting
version_upstream or populate an alternative field) so users upgrading from 0.2.0
to 0.3.0 have clear guidance.
π Review details
Configuration used: defaults
Review profile: CHILL
Plan: Pro
β Files ignored due to path filters (1)
Cargo.lockis excluded by!**/*.lock
π Files selected for processing (13)
CHANGELOG.mdCargo.tomlcrates/soar-cli/Cargo.tomlcrates/soar-core/CHANGELOG.mdcrates/soar-core/Cargo.tomlcrates/soar-db/CHANGELOG.mdcrates/soar-db/Cargo.tomlcrates/soar-dl/CHANGELOG.mdcrates/soar-dl/Cargo.tomlcrates/soar-package/CHANGELOG.mdcrates/soar-package/Cargo.tomlcrates/soar-registry/CHANGELOG.mdcrates/soar-registry/Cargo.toml
β Files skipped from review due to trivial changes (2)
- crates/soar-core/Cargo.toml
- crates/soar-core/CHANGELOG.md
π§ Files skipped from review as they are similar to previous changes (8)
- Cargo.toml
- crates/soar-dl/Cargo.toml
- crates/soar-package/CHANGELOG.md
- crates/soar-registry/CHANGELOG.md
- crates/soar-registry/Cargo.toml
- crates/soar-package/Cargo.toml
- CHANGELOG.md
- crates/soar-cli/Cargo.toml
β° Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (2)
- GitHub Check: coverage
- GitHub Check: test
π Additional comments (3)
crates/soar-db/Cargo.toml (2)
3-3: LGTM! Version bump correctly reflects breaking changes.The minor version bump from 0.2.0 to 0.3.0 appropriately follows semantic versioning for pre-1.0 releases, where breaking changes increment the minor version. This aligns with the documented breaking changes in the PR objectives (field and struct removals).
16-26: Dependency management approach is appropriate.Using workspace-managed dependencies (
workspace = true) is the correct approach for coordinating version updates across the monorepo, especially when soar-registry (line 24) is also receiving breaking changes in this release.crates/soar-dl/CHANGELOG.md (1)
2-10: β Changelog entry for soar-dl v0.7.2 is well-formatted.The changelog correctly documents an API-compatible release (0.7.1 β 0.7.2) with feature and refactor entries. The structure follows the established format with proper section headers, emojis, scopes, and commit references.
| ## [0.3.0](https://github.com/pkgforge/soar/compare/soar-db-v0.2.0...soar-db-v0.3.0) - 2026-01-07 | ||
|
|
||
| ### π Refactor | ||
|
|
||
| - *(error)* Don't override error messages - ([e44342f](https://github.com/pkgforge/soar/commit/e44342f3c23b9cdbe23df2739bcf04bde4138025)) | ||
| - *(query)* Update query field icons - ([695a427](https://github.com/pkgforge/soar/commit/695a427ef6a4874cb212cdceed192f94150c5548)) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
π§© Analysis chain
π Script executed:
# First, let's check the CHANGELOG.md file
cat -n crates/soar-db/CHANGELOG.md | head -20Repository: pkgforge/soar
Length of output: 1081
π Script executed:
# Check the Cargo.toml for version information
cat crates/soar-db/Cargo.toml | grep -A 2 "^\[package\]"Repository: pkgforge/soar
Length of output: 101
π Script executed:
# Search for the struct definitions mentioned in the review
rg "struct NewPackage|struct PackageListing|struct Package" crates/soar-db/src --type rustRepository: pkgforge/soar
Length of output: 635
π Script executed:
# Check the NewPackage struct definition
sed -n '/pub struct NewPackage/,/^}/p' crates/soar-db/src/models/metadata.rsRepository: pkgforge/soar
Length of output: 1348
π Script executed:
# Check the PackageListing struct definition
sed -n '/pub struct PackageListing/,/^}/p' crates/soar-db/src/models/metadata.rsRepository: pkgforge/soar
Length of output: 195
π Script executed:
# Check the Package struct definition
sed -n '/pub struct Package/,/^}/p' crates/soar-db/src/models/metadata.rsRepository: pkgforge/soar
Length of output: 1620
π Script executed:
# Search for any mention of version_upstream in the codebase to verify if it was removed
rg "version_upstream" crates/soar-db/Repository: pkgforge/soar
Length of output: 162
The version_upstream field has been removed from the public API structs (NewPackage, PackageListing, Package), but the changelog entry only lists refactors without documenting any breaking changes. Consumers upgrading from 0.2.0 to 0.3.0 need explicit visibility into this breaking API change before upgrading.
Add a ### β οΈ Breaking Changes section documenting the removed field:
Suggested structure
## [0.3.0](https://github.com/pkgforge/soar/compare/soar-db-v0.2.0...soar-db-v0.3.0) - 2026-01-07
+### β οΈ Breaking Changes
+
+- Removed `version_upstream` field from `NewPackage`, `PackageListing`, and `Package` structs
+
### π Refactorπ€ Prompt for AI Agents
In @crates/soar-db/CHANGELOG.md around lines 2 - 7, The changelog is missing a
Breaking Changes section: add a new "### β οΈ Breaking Changes" entry for v0.3.0
that explicitly states the public API structs NewPackage, PackageListing, and
Package no longer include the version_upstream field; explain the change, list
the affected symbols (NewPackage, PackageListing, Package, version_upstream),
provide a brief migration note instructing consumers to remove uses of
version_upstream or map it from their own metadata before constructing these
structs, and include a short example description of how to adapt callers (e.g.,
stop setting version_upstream or populate an alternative field) so users
upgrading from 0.2.0 to 0.3.0 have clear guidance.
66609f3 to
d078eee
Compare
π€ New release
soar-utils: 0.1.1 -> 0.1.2 (β API compatible changes)soar-config: 0.1.1 -> 0.2.0 (β API breaking changes)soar-dl: 0.7.1 -> 0.7.2 (β API compatible changes)soar-registry: 0.1.1 -> 0.2.0 (β API breaking changes)soar-db: 0.2.0 -> 0.3.0 (β API breaking changes)soar-package: 0.1.1 -> 0.2.0 (β API breaking changes)soar-core: 0.10.0 -> 0.11.0 (β API breaking changes)soar-cli: 0.9.1 -> 0.9.2β
soar-configbreaking changesβ
soar-registrybreaking changesβ
soar-dbbreaking changesβ
soar-packagebreaking changesβ
soar-corebreaking changesChangelog
soar-utilssoar-configsoar-dlsoar-registrysoar-dbsoar-packagesoar-coresoar-cliThis PR was generated with release-plz.
Summary by CodeRabbit
New Features
Refactor
βοΈ Tip: You can customize this high-level summary in your review settings.