Skip to content

Conversation

@Jordonbc
Copy link
Contributor

@Jordonbc Jordonbc commented Feb 7, 2026

No description provided.

Jordonbc and others added 20 commits December 31, 2025 19:31
Add guideline requiring a short (<=72 chars) title, a blank line, and optional body to AGENTS.md files for agents.
Bumps the actions-minor-patch group with 1 update: [github/codeql-action](https://github.com/github/codeql-action).


Updates `github/codeql-action` from 4.31.9 to 4.31.10
- [Release notes](https://github.com/github/codeql-action/releases)
- [Changelog](https://github.com/github/codeql-action/blob/main/CHANGELOG.md)
- [Commits](github/codeql-action@5d4e8d1...cdefb33)

---
updated-dependencies:
- dependency-name: github/codeql-action
  dependency-version: 4.31.10
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: actions-minor-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps the cargo-minor-patch group with 1 update: [thiserror](https://github.com/dtolnay/thiserror).


Updates `thiserror` from 2.0.17 to 2.0.18
- [Release notes](https://github.com/dtolnay/thiserror/releases)
- [Commits](dtolnay/thiserror@2.0.17...2.0.18)

---
updated-dependencies:
- dependency-name: thiserror
  dependency-version: 2.0.18
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: cargo-minor-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps the actions-minor-patch group with 2 updates: [actions/checkout](https://github.com/actions/checkout) and [github/codeql-action](https://github.com/github/codeql-action).


Updates `actions/checkout` from 6.0.1 to 6.0.2
- [Release notes](https://github.com/actions/checkout/releases)
- [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md)
- [Commits](actions/checkout@8e8c483...de0fac2)

Updates `github/codeql-action` from 4.31.10 to 4.31.11
- [Release notes](https://github.com/github/codeql-action/releases)
- [Changelog](https://github.com/github/codeql-action/blob/main/CHANGELOG.md)
- [Commits](github/codeql-action@cdefb33...19b2f06)

---
updated-dependencies:
- dependency-name: actions/checkout
  dependency-version: 6.0.2
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: actions-minor-patch
- dependency-name: github/codeql-action
  dependency-version: 4.31.11
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: actions-minor-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps the actions-minor-patch group with 1 update: [github/codeql-action](https://github.com/github/codeql-action).


Updates `github/codeql-action` from 4.31.11 to 4.32.0
- [Release notes](https://github.com/github/codeql-action/releases)
- [Changelog](https://github.com/github/codeql-action/blob/main/CHANGELOG.md)
- [Commits](github/codeql-action@19b2f06...b20883b)

---
updated-dependencies:
- dependency-name: github/codeql-action
  dependency-version: 4.32.0
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: actions-minor-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Added badges for nightly, dev, and stable workflows.
@chatgpt-codex-connector
Copy link

💡 Codex Review

OpenVCS-Core/Cargo.toml

Lines 33 to 37 in c12bf17

# The VCS trait and VCS-related error type.
vcs = ["dep:thiserror", "backend-registry"]
# Backend discovery via the `BACKENDS` registry (link-time registration).
backend-registry = ["dep:linkme"]

P2 Badge Enforce vcs dependency for backend-registry feature

The backend-registry feature no longer pulls in vcs, but src/backend_descriptor.rs is still compiled under feature = "backend-registry" and references Vcs/Result types that only exist when feature = "vcs" is enabled. This makes features = ["backend-registry"] a broken combination that now fails to compile, which is a regression from the previous feature wiring. Either add vcs back to the feature dependencies or gate the module/types behind cfg(feature = "vcs") to keep feature combinations valid.


let msg = {
let mut lock = stdin
.lock()
.map_err(|_| PluginError::message("stdin lock poisoned"))?;
receive_message(&mut *lock).ok_or_else(|| PluginError::message("host closed stdin"))?

P1 Badge Make host call timeout enforceable with blocking stdin

call_host checks the deadline before reading, but then calls receive_message on a blocking BufRead, so if the host never sends anything the function will block indefinitely and the timeout can never be observed. This means OPENVCS_PLUGIN_HOST_TIMEOUT_MS is ineffective whenever stdin is idle, leading to plugins hanging forever instead of timing out. Consider using non-blocking I/O, a separate reader thread, or a read with timeout so the deadline is actually enforced.

ℹ️ 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".

@Jordonbc Jordonbc merged commit 883a259 into Stable Feb 7, 2026
4 checks passed
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