Skip to content

Conversation

@QaidVoid
Copy link
Member

@QaidVoid QaidVoid commented Dec 29, 2025

πŸ€– 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-config breaking changes

--- failure constructible_struct_adds_field: externally-constructible struct adds field ---

Description:
A pub struct constructible with a struct literal has a new pub field. Existing struct literals must be updated to include the new field.
        ref: https://doc.rust-lang.org/reference/expressions/struct-expr.html
       impl: https://github.com/obi1kenobi/cargo-semver-checks/tree/v0.45.0/src/lints/constructible_struct_adds_field.ron

Failed in:
  field PackageOptions.entrypoint in /tmp/.tmp7qqMMh/soar/crates/soar-config/src/packages.rs:89
  field PackageOptions.update in /tmp/.tmp7qqMMh/soar/crates/soar-config/src/packages.rs:108
  field ResolvedPackage.entrypoint in /tmp/.tmp7qqMMh/soar/crates/soar-config/src/packages.rs:182
  field ResolvedPackage.update in /tmp/.tmp7qqMMh/soar/crates/soar-config/src/packages.rs:188

--- failure enum_variant_added: enum variant added on exhaustive enum ---

Description:
A publicly-visible enum without #[non_exhaustive] has a new variant.
        ref: https://doc.rust-lang.org/cargo/reference/semver.html#enum-variant-new
       impl: https://github.com/obi1kenobi/cargo-semver-checks/tree/v0.45.0/src/lints/enum_variant_added.ron

Failed in:
  variant ConfigError:Custom in /tmp/.tmp7qqMMh/soar/crates/soar-config/src/error.rs:114

⚠ soar-registry breaking changes

--- failure struct_pub_field_missing: pub struct's pub field removed or renamed ---

Description:
A publicly-visible struct has at least one public field that is no longer available under its prior name. It may have been renamed or removed entirely.
        ref: https://doc.rust-lang.org/cargo/reference/semver.html#item-remove
       impl: https://github.com/obi1kenobi/cargo-semver-checks/tree/v0.45.0/src/lints/struct_pub_field_missing.ron

Failed in:
  field rank of struct RemotePackage, previously in file /tmp/.tmpL8mIDj/soar-registry/src/package.rs:87
  field pkg of struct RemotePackage, previously in file /tmp/.tmpL8mIDj/soar-registry/src/package.rs:90
  field version_upstream of struct RemotePackage, previously in file /tmp/.tmpL8mIDj/soar-registry/src/package.rs:107
  field rank of struct RemotePackage, previously in file /tmp/.tmpL8mIDj/soar-registry/src/package.rs:87
  field pkg of struct RemotePackage, previously in file /tmp/.tmpL8mIDj/soar-registry/src/package.rs:90
  field version_upstream of struct RemotePackage, previously in file /tmp/.tmpL8mIDj/soar-registry/src/package.rs:107

⚠ soar-db breaking changes

--- failure struct_missing: pub struct removed or renamed ---

Description:
A publicly-visible struct cannot be imported by its prior path. A `pub use` may have been removed, or the struct itself may have been renamed or removed entirely.
        ref: https://doc.rust-lang.org/cargo/reference/semver.html#item-remove
       impl: https://github.com/obi1kenobi/cargo-semver-checks/tree/v0.45.0/src/lints/struct_missing.ron

Failed in:
  struct soar_db::schema::metadata::packages::dsl::version_upstream, previously in file /tmp/.tmpL8mIDj/soar-db/src/schema/metadata.rs:17
  struct soar_db::schema::metadata::packages::columns::version_upstream, previously in file /tmp/.tmpL8mIDj/soar-db/src/schema/metadata.rs:17
  struct soar_db::schema::metadata::packages::version_upstream, previously in file /tmp/.tmpL8mIDj/soar-db/src/schema/metadata.rs:17

--- failure struct_pub_field_missing: pub struct's pub field removed or renamed ---

Description:
A publicly-visible struct has at least one public field that is no longer available under its prior name. It may have been renamed or removed entirely.
        ref: https://doc.rust-lang.org/cargo/reference/semver.html#item-remove
       impl: https://github.com/obi1kenobi/cargo-semver-checks/tree/v0.45.0/src/lints/struct_pub_field_missing.ron

Failed in:
  field version_upstream of struct NewPackage, previously in file /tmp/.tmpL8mIDj/soar-db/src/models/metadata.rs:188
  field version_upstream of struct PackageListing, previously in file /tmp/.tmpL8mIDj/soar-db/src/models/metadata.rs:141
  field version_upstream of struct Package, previously in file /tmp/.tmpL8mIDj/soar-db/src/models/metadata.rs:16

⚠ soar-package breaking changes

--- failure function_parameter_count_changed: pub fn parameter count changed ---

Description:
A publicly-visible function now takes a different number of parameters.
        ref: https://doc.rust-lang.org/cargo/reference/semver.html#fn-change-arity
       impl: https://github.com/obi1kenobi/cargo-semver-checks/tree/v0.45.0/src/lints/function_parameter_count_changed.ron

Failed in:
  soar_package::formats::common::integrate_package now takes 8 parameters instead of 7, in /tmp/.tmp7qqMMh/soar/crates/soar-package/src/formats/common.rs:332
  soar_package::integrate_package now takes 8 parameters instead of 7, in /tmp/.tmp7qqMMh/soar/crates/soar-package/src/formats/common.rs:332

⚠ soar-core breaking changes

--- failure constructible_struct_adds_field: externally-constructible struct adds field ---

Description:
A pub struct constructible with a struct literal has a new pub field. Existing struct literals must be updated to include the new field.
        ref: https://doc.rust-lang.org/reference/expressions/struct-expr.html
       impl: https://github.com/obi1kenobi/cargo-semver-checks/tree/v0.45.0/src/lints/constructible_struct_adds_field.ron

Failed in:
  field InstallTarget.entrypoint in /tmp/.tmp7qqMMh/soar/crates/soar-core/src/package/install.rs:82
  field UrlPackage.is_ghcr in /tmp/.tmp7qqMMh/soar/crates/soar-core/src/package/url.rs:23

--- failure struct_pub_field_missing: pub struct's pub field removed or renamed ---

Description:
A publicly-visible struct has at least one public field that is no longer available under its prior name. It may have been renamed or removed entirely.
        ref: https://doc.rust-lang.org/cargo/reference/semver.html#item-remove
       impl: https://github.com/obi1kenobi/cargo-semver-checks/tree/v0.45.0/src/lints/struct_pub_field_missing.ron

Failed in:
  field version_upstream of struct Package, previously in file /tmp/.tmpL8mIDj/soar-core/src/database/models.rs:37
Changelog

soar-utils

0.1.2 - 2026-01-09

βš™οΈ Miscellaneous Tasks

  • Update Cargo.toml dependencies - (0000000)

soar-config

0.2.0 - 2026-01-09

⛰️ Features

  • (install) Add entrypoint option and executable discovery fallbacks - (b77cffd)
  • (update) Allow updating remote URL packages (#137) - (af13bb6)

soar-dl

0.7.2 - 2026-01-09

⛰️ Features

  • (apply) Allow applying ghcr packages - (06e2b73)

🚜 Refactor

  • (error) Don't override error messages - (e44342f)

soar-registry

0.2.0 - 2026-01-09

⛰️ Features

  • (apply) Allow applying ghcr packages - (06e2b73)

🚜 Refactor

  • (error) Don't override error messages - (e44342f)
  • (query) Update query field icons - (695a427)

soar-db

0.3.0 - 2026-01-09

🚜 Refactor

  • (error) Don't override error messages - (e44342f)
  • (query) Update query field icons - (695a427)

soar-package

0.2.0 - 2026-01-09

⛰️ Features

  • (apply) Allow applying ghcr packages - (06e2b73)
  • (install) Add entrypoint option and executable discovery fallbacks - (b77cffd)

🚜 Refactor

  • (error) Don't override error messages - (e44342f)

soar-core

0.11.0 - 2026-01-09

⛰️ Features

  • (apply) Allow applying ghcr packages - (06e2b73)
  • (install) Add entrypoint option and executable discovery fallbacks - (b77cffd)
  • (update) Allow updating remote URL packages (#137) - (af13bb6)

🚜 Refactor

  • (error) Don't override error messages - (e44342f)
  • (query) Update query field icons - (695a427)

soar-cli

0.9.2 - 2026-01-09

⛰️ Features

  • (apply) Allow applying ghcr packages - (06e2b73)
  • (install) Add entrypoint option and executable discovery fallbacks - (b77cffd)
  • (update) Allow updating remote URL packages (#137) - (af13bb6)

🚜 Refactor

  • (error) Don't override error messages - (e44342f)
  • (query) Update query field icons - (695a427)


This PR was generated with release-plz.

Summary by CodeRabbit

Release Notes

  • New Features

    • Added support for applying ghcr packages
    • Enhanced installation with entrypoint options and executable discovery fallbacks
    • Enabled updating packages from remote URLs
  • Refactor

    • Improved error message handling
    • Updated query field icons

✏️ Tip: You can customize this high-level summary in your review settings.

@QaidVoid QaidVoid added the release New Release (Probably Automated) label Dec 29, 2025
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Dec 29, 2025

πŸ“ Walkthrough

Walkthrough

Workspace 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; changelogs document GHCR apply feature and refactors (error messages, query icons).

Changes

Cohort / File(s) Summary
Root metadata
CHANGELOG.md, Cargo.toml
Root CHANGELOG adds v0.9.2 entry; workspace dependency versions bumped to match crate releases.
soar-cli
crates/soar-cli/Cargo.toml
Package version bumped 0.9.1 β†’ 0.9.2.
soar-core
crates/soar-core/Cargo.toml, crates/soar-core/CHANGELOG.md
Package version bumped 0.10.0 β†’ 0.11.0; CHANGELOG adds v0.11.0 features (GHCR apply, install improvements, update remote URL packages) and refactors (don't override error messages; update query icons).
soar-db
crates/soar-db/Cargo.toml, crates/soar-db/CHANGELOG.md
Package version bumped 0.2.0 β†’ 0.3.0; CHANGELOG adds v0.3.0 refactor notes (don't override error messages; update query icons).
soar-dl
crates/soar-dl/Cargo.toml, crates/soar-dl/CHANGELOG.md
Package version bumped 0.7.1 β†’ 0.7.2; CHANGELOG adds v0.7.2 feature (apply GHCR packages) and refactor (don't override error messages).
soar-package
crates/soar-package/Cargo.toml, crates/soar-package/CHANGELOG.md
Package version bumped 0.1.1 β†’ 0.2.0; CHANGELOG adds v0.2.0 features (GHCR apply; install entrypoint option and executable discovery fallbacks) and refactor (don't override error messages).
soar-registry
crates/soar-registry/Cargo.toml, crates/soar-registry/CHANGELOG.md
Package version bumped 0.1.1 β†’ 0.2.0; CHANGELOG adds v0.2.0 feature (apply GHCR packages) and refactors (don't override error messages; update query icons); older entry updated with a misc dependencies note.
soar-config
crates/soar-config/Cargo.toml, crates/soar-config/CHANGELOG.md
Package version bumped 0.1.1 β†’ 0.2.0; CHANGELOG adds install entrypoint option and executable discovery fallbacks, plus update remote URL packages.
soar-utils
crates/soar-utils/Cargo.toml, crates/soar-utils/CHANGELOG.md
Package version bumped 0.1.1 β†’ 0.1.2; CHANGELOG adds v0.1.2 misc task noting updated Cargo.toml dependencies.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 A hop, a tag, a tiny cheerβ€”
Versions nudged and notes appear,
GHCR tricks and errors spared,
Icons tweaked with gentle care,
I nibble bytes and twitch my ear!

πŸš₯ Pre-merge checks | βœ… 2 | ❌ 1
❌ Failed checks (1 inconclusive)
Check name Status Explanation Resolution
Title check ❓ Inconclusive The title 'chore: release' is generic and vague; it describes the type of change (release management) but lacks specificity about the actual release scope or version changes across multiple crates. Consider a more descriptive title like 'chore: release multiple crates with version bumps' to better convey the scope of the release to reviewers scanning the history.
βœ… Passed checks (2 passed)
Check name Status Explanation
Description Check βœ… Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage βœ… Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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


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 and usage tips.

Copy link
Contributor

@coderabbitai coderabbitai bot left a 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

πŸ“₯ Commits

Reviewing files that changed from the base of the PR and between e44342f and ca39b1c.

β›” Files ignored due to path filters (1)
  • Cargo.lock is excluded by !**/*.lock
πŸ“’ Files selected for processing (13)
  • CHANGELOG.md
  • Cargo.toml
  • crates/soar-cli/Cargo.toml
  • crates/soar-core/CHANGELOG.md
  • crates/soar-core/Cargo.toml
  • crates/soar-db/CHANGELOG.md
  • crates/soar-db/Cargo.toml
  • crates/soar-dl/CHANGELOG.md
  • crates/soar-dl/Cargo.toml
  • crates/soar-package/CHANGELOG.md
  • crates/soar-package/Cargo.toml
  • crates/soar-registry/CHANGELOG.md
  • crates/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.

@QaidVoid QaidVoid force-pushed the release-plz-2025-12-29T16-57-22Z branch from ca39b1c to 66609f3 Compare January 7, 2026 15:11
@codecov
Copy link

codecov bot commented Jan 7, 2026

Codecov Report

βœ… All modified and coverable lines are covered by tests.

πŸ“’ Thoughts on this report? Let us know!

Copy link
Contributor

@coderabbitai coderabbitai bot left a 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

πŸ“₯ Commits

Reviewing files that changed from the base of the PR and between ca39b1c and 66609f3.

β›” Files ignored due to path filters (1)
  • Cargo.lock is excluded by !**/*.lock
πŸ“’ Files selected for processing (13)
  • CHANGELOG.md
  • Cargo.toml
  • crates/soar-cli/Cargo.toml
  • crates/soar-core/CHANGELOG.md
  • crates/soar-core/Cargo.toml
  • crates/soar-db/CHANGELOG.md
  • crates/soar-db/Cargo.toml
  • crates/soar-dl/CHANGELOG.md
  • crates/soar-dl/Cargo.toml
  • crates/soar-package/CHANGELOG.md
  • crates/soar-package/Cargo.toml
  • crates/soar-registry/CHANGELOG.md
  • crates/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.

Comment on lines 2 to 7
## [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))
Copy link
Contributor

Choose a reason for hiding this comment

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

⚠️ Potential issue | πŸ”΄ Critical

🧩 Analysis chain

🏁 Script executed:

# First, let's check the CHANGELOG.md file
cat -n crates/soar-db/CHANGELOG.md | head -20

Repository: 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 rust

Repository: 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.rs

Repository: 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.rs

Repository: 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.rs

Repository: 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


⚠️ Missing breaking change documentation in CHANGELOG for soar-db v0.3.0.

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.

@QaidVoid QaidVoid force-pushed the release-plz-2025-12-29T16-57-22Z branch from 66609f3 to d078eee Compare January 7, 2026 16:36
@QaidVoid QaidVoid force-pushed the release-plz-2025-12-29T16-57-22Z branch from d078eee to 222278a Compare January 9, 2026 16:32
Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 2

πŸ€– Fix all issues with AI agents
In @crates/soar-registry/CHANGELOG.md:
- Around line 2-12: Add a "Breaking Changes" subsection to the v0.2.0 changelog
entry documenting that struct RemotePackage had three public fields removed;
explicitly list the removed fields `rank`, `pkg`, and `version_upstream` and
state that this is a breaking API change so downstream consumers are warned.
Ensure the new subsection appears under the v0.2.0 header and uses a clear label
such as "⚠️ Breaking Changes" with a short bullet describing the removed fields
from RemotePackage.

In @crates/soar-utils/CHANGELOG.md:
- Line 6: Replace the placeholder commit hash "0000000" used in the CHANGELOG.md
entry (e.g., the line "Update Cargo.toml dependencies - ([0000000](...))") with
the real 7-character commit hash used in this PR (verify and use the correct
hash such as af13bb6 or 222278a), and do the same replacement for the other
affected crate changelogs (soar-utils, soar-registry, soar-config) so the GitHub
links are valid; ensure each changelog link uses the exact 7-character commit id
in the markdown link.
🧹 Nitpick comments (1)
crates/soar-utils/CHANGELOG.md (1)

4-6: Consider adding more context to the changelog entry.

The "Update Cargo.toml dependencies" entry is generic and doesn't explain what specifically changed in soar-utils or why the version was bumped. For a patch release (0.1.1 β†’ 0.1.2), the changelog should clearly describe the functional or dependency changes involved.

πŸ“œ Review details

Configuration used: defaults

Review profile: CHILL

Plan: Pro

πŸ“₯ Commits

Reviewing files that changed from the base of the PR and between d078eee and 222278a.

β›” Files ignored due to path filters (1)
  • Cargo.lock is excluded by !**/*.lock
πŸ“’ Files selected for processing (17)
  • CHANGELOG.md
  • Cargo.toml
  • crates/soar-cli/Cargo.toml
  • crates/soar-config/CHANGELOG.md
  • crates/soar-config/Cargo.toml
  • crates/soar-core/CHANGELOG.md
  • crates/soar-core/Cargo.toml
  • crates/soar-db/CHANGELOG.md
  • crates/soar-db/Cargo.toml
  • crates/soar-dl/CHANGELOG.md
  • crates/soar-dl/Cargo.toml
  • crates/soar-package/CHANGELOG.md
  • crates/soar-package/Cargo.toml
  • crates/soar-registry/CHANGELOG.md
  • crates/soar-registry/Cargo.toml
  • crates/soar-utils/CHANGELOG.md
  • crates/soar-utils/Cargo.toml
βœ… Files skipped from review due to trivial changes (1)
  • crates/soar-core/Cargo.toml
🚧 Files skipped from review as they are similar to previous changes (13)
  • crates/soar-cli/Cargo.toml
  • crates/soar-package/CHANGELOG.md
  • crates/soar-registry/Cargo.toml
  • crates/soar-utils/Cargo.toml
  • crates/soar-db/CHANGELOG.md
  • crates/soar-config/Cargo.toml
  • crates/soar-package/Cargo.toml
  • crates/soar-dl/CHANGELOG.md
  • CHANGELOG.md
  • Cargo.toml
  • crates/soar-dl/Cargo.toml
  • crates/soar-db/Cargo.toml
  • crates/soar-config/CHANGELOG.md
⏰ 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 (1)
crates/soar-core/CHANGELOG.md (1)

2-13: Changelog entry is correct and properly formatted.

All commit hashes exist in the repository, version numbering is appropriate (0.11.0 follows 0.10.0), the date is accurate, and the format is consistent with existing changelog entries. The features and refactors documented align with the PR scope.

Comment on lines +2 to +12
## [0.2.0](https://github.com/pkgforge/soar/compare/soar-registry-v0.1.1...soar-registry-v0.2.0) - 2026-01-09

### ⛰️ Features

- *(apply)* Allow applying ghcr packages - ([06e2b73](https://github.com/pkgforge/soar/commit/06e2b73fce7f4189527b8868bb9adfe14d0600cc))

### 🚜 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))

Copy link
Contributor

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🟠 Major

🧩 Analysis chain

🏁 Script executed:

# First, locate and examine the struct RemotePackage to verify the breaking changes
find . -name "*.rs" -type f | xargs grep -l "struct RemotePackage" | head -5

Repository: pkgforge/soar

Length of output: 94


🏁 Script executed:

# Check the CHANGELOG.md file in full to understand context
cat -n crates/soar-registry/CHANGELOG.md | head -50

Repository: pkgforge/soar

Length of output: 1238


🏁 Script executed:

# Look for changelog patterns in other soar crates
fd "CHANGELOG.md" --type f | head -10

Repository: pkgforge/soar

Length of output: 285


🏁 Script executed:

# Search for any documentation of breaking changes in the commits or recent changes
git log --oneline --all -- crates/soar-registry/CHANGELOG.md | head -10

Repository: pkgforge/soar

Length of output: 139


🏁 Script executed:

# Examine the struct RemotePackage definition
cat -n crates/soar-registry/src/package.rs | head -100

Repository: pkgforge/soar

Length of output: 3542


🏁 Script executed:

# Check other crate changelogs for breaking changes patterns
cat crates/soar-core/CHANGELOG.md | grep -A 10 "Breaking\|⚠️" | head -30

Repository: pkgforge/soar

Length of output: 39


🏁 Script executed:

# Check soar-package changelog
cat crates/soar-package/CHANGELOG.md | grep -A 10 "Breaking\|⚠️" | head -30

Repository: pkgforge/soar

Length of output: 39


🏁 Script executed:

# Search for any mentions of RemotePackage changes in recent commits
git log --all --oneline --grep="RemotePackage\|breaking\|Breaking" | head -20

Repository: pkgforge/soar

Length of output: 108


🏁 Script executed:

# Get more of the package.rs file to find RemotePackage struct definition
wc -l crates/soar-registry/src/package.rs

Repository: pkgforge/soar

Length of output: 96


🏁 Script executed:

# Read the RemotePackage struct definition
sed -n '100,250p' crates/soar-registry/src/package.rs

Repository: pkgforge/soar

Length of output: 4497


🏁 Script executed:

# Search for mentions of rank, pkg, version_upstream fields
rg "rank|pkg|version_upstream" crates/soar-registry/src/package.rs -C 3

Repository: pkgforge/soar

Length of output: 1945


🏁 Script executed:

# Check git history for this file to see what changed between versions
git log --oneline --all -- crates/soar-registry/src/package.rs | head -20

Repository: pkgforge/soar

Length of output: 255


🏁 Script executed:

# Check if there's a PR description or PR file
git log --all --oneline | grep -i "0.2.0\|release" | head -10

Repository: pkgforge/soar

Length of output: 394


🏁 Script executed:

# Check git diff between v0.1.1 and v0.2.0 tags or commits for RemotePackage
git tag | grep "soar-registry" | sort -V

Repository: pkgforge/soar

Length of output: 39


🏁 Script executed:

# Look for any version tags
git log --all --oneline --decorate | grep -i "soar-registry\|0.2.0\|0.1.1" | head -20

Repository: pkgforge/soar

Length of output: 148


🏁 Script executed:

# Check the actual current state of the code - are there ANY removed fields somewhere?
rg "pub.*rank|pub.*version_upstream" crates/soar-registry/ -t rust

Repository: pkgforge/soar

Length of output: 39


🏁 Script executed:

# Check the release commit to see if it includes any documentation of changes
git show 5831e60 | head -100

Repository: pkgforge/soar

Length of output: 2337


🏁 Script executed:

# Check if there's any documentation file mentioning breaking changes
find . -name "*.md" -o -name "*.txt" | xargs grep -l "breaking\|BREAKING" 2>/dev/null | head -10

Repository: pkgforge/soar

Length of output: 71


🏁 Script executed:

# Search entire repo for any mention of fields: rank, pkg (as field name), version_upstream
rg "\.rank|\.pkg(?![_a-z])|\.version_upstream" crates/soar-registry/ -t rust

Repository: pkgforge/soar

Length of output: 313


🏁 Script executed:

# Check the diff between init commit and current to see if RemotePackage changed
git show 21070db:crates/soar-registry/src/package.rs 2>/dev/null | grep -A 50 "struct RemotePackage"

Repository: pkgforge/soar

Length of output: 1568


Document breaking API changes in the changelog.

The v0.2.0 release removes three public fields from struct RemotePackage: rank, pkg, and version_upstream. These are breaking changes that affect any downstream consumers using these fields. The current changelog entry omits this critical information.

Add a "Breaking Changes" section to the v0.2.0 changelog entry:

Suggested changelog addition
### ⚠️ Breaking Changes

- Struct `RemotePackage`: removed public fields (`rank`, `pkg`, `version_upstream`)
πŸ€– Prompt for AI Agents
In @crates/soar-registry/CHANGELOG.md around lines 2 - 12, Add a "Breaking
Changes" subsection to the v0.2.0 changelog entry documenting that struct
RemotePackage had three public fields removed; explicitly list the removed
fields `rank`, `pkg`, and `version_upstream` and state that this is a breaking
API change so downstream consumers are warned. Ensure the new subsection appears
under the v0.2.0 header and uses a clear label such as "⚠️ Breaking Changes"
with a short bullet describing the removed fields from RemotePackage.


### βš™οΈ Miscellaneous Tasks

- Update Cargo.toml dependencies - ([0000000](https://github.com/pkgforge/soar/commit/0000000))
Copy link
Contributor

Choose a reason for hiding this comment

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

⚠️ Potential issue | πŸ”΄ Critical

🧩 Analysis chain

🏁 Script executed:

cat -n crates/soar-utils/CHANGELOG.md

Repository: pkgforge/soar

Length of output: 922


🏁 Script executed:

git log --oneline | head -20

Repository: pkgforge/soar

Length of output: 79


🏁 Script executed:

git rev-parse HEAD

Repository: pkgforge/soar

Length of output: 97


🏁 Script executed:

fd CHANGELOG.md --type f

Repository: pkgforge/soar

Length of output: 285


🏁 Script executed:

for file in crates/*/CHANGELOG.md CHANGELOG.md; do
  if [ -f "$file" ]; then
    echo "=== $file ==="
    head -20 "$file" | grep -E "commit/[0-9a-f]{7}" || echo "No commit hashes in first 20 lines"
  fi
done

Repository: pkgforge/soar

Length of output: 5826


🏁 Script executed:

git log --all --oneline --grep="Update Cargo.toml dependencies" 2>/dev/null | head -10

Repository: pkgforge/soar

Length of output: 39


🏁 Script executed:

git log --all --oneline soar-utils | head -20

Repository: pkgforge/soar

Length of output: 247


🏁 Script executed:

git log --all --oneline -20 --decorate

Repository: pkgforge/soar

Length of output: 1223


🏁 Script executed:

git show HEAD --stat

Repository: pkgforge/soar

Length of output: 9081


Placeholder commit hashes in multiple CHANGELOG.md files must be replaced before merge.

The hash 0000000 appears in at least three crate changelogs (soar-utils, soar-registry, soar-config) with entries like "Update Cargo.toml dependencies" and "Updated the following local packages". This is a placeholder that will create broken GitHub links and should not be merged.

Verify which actual commit should be referenced in these changelog entries (likely af13bb6 or 222278a) and update all affected changelogs with the correct 7-character commit hash before this PR is merged.

πŸ€– Prompt for AI Agents
In @crates/soar-utils/CHANGELOG.md at line 6, Replace the placeholder commit
hash "0000000" used in the CHANGELOG.md entry (e.g., the line "Update Cargo.toml
dependencies - ([0000000](...))") with the real 7-character commit hash used in
this PR (verify and use the correct hash such as af13bb6 or 222278a), and do the
same replacement for the other affected crate changelogs (soar-utils,
soar-registry, soar-config) so the GitHub links are valid; ensure each changelog
link uses the exact 7-character commit id in the markdown link.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release New Release (Probably Automated)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants