chore: pin rx4 to rotary harness SHA - #74
Conversation
Bugbot couldn't run - usage limit reachedBugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit. A user or team admin can review and increase usage limits in the Cursor dashboard. (requestId: serverGenReqId_2f9989f6-6f8b-46f3-ba84-ce20efdeecb0) |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 7bbb0337fe
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| # pulls in the fff/git2/notify stack), `mcp` (apollo has src/mcp_server.rs), | ||
| # `zkr-memory` (gates rx4's self_improve; apollo depends on zkr directly). | ||
| rx4 = { version = "0.6.4", default-features = false, features = ["ipc", "providers", "skills", "graph-memory"] } | ||
| rx4 = { git = "https://github.com/tschk/rotary.git", rev = "441ce524ef8b3ec2372227ba35b91fad0bc4389a", default-features = false, features = ["ipc", "providers", "skills", "graph-memory"] } |
There was a problem hiding this comment.
When preparing the repository's advertised crates.io package, Cargo rejects this manifest because every published dependency must specify a version requirement; cargo package reports that the git specification is removed from the packaged manifest. Since this entry now has only git and rev, the next crates.io release is blocked before compilation. Retain version = "0.6.5" alongside the git pin so local builds use the revision while the packaged crate resolves rx4 from crates.io.
AGENTS.md reference: AGENTS.md:L40-L42
Useful? React with 👍 / 👎.
1 participant
Summary
rx4totschk/rotary@441ce524ef8b3ec2372227ba35b91fad0bc4389a(feat/rx4-agent-harness), keepingdefault-features = falseand the existing feature set (ipc,providers,skills,graph-memory).Test plan
cargo check(git dep already fetched)Note
Medium Risk
Swaps the core agent-loop dependency to an unreleased git SHA, so apollo can pick up harness API/behavior changes without a crates.io release. No application code is changed in this PR.
Overview
Pins the
rx4agent-harness crate totschk/rotaryat441ce524instead of crates.io0.6.4. Feature flags stay the same (ipc,providers,skills,graph-memory; default features off).Cargo.lockfollows that git dep (reported as 0.6.5) and the usual transitive Windows/base64/getrandomversion shifts.Reviewed by Cursor Bugbot for commit 7bbb033. Configure here.