Skip to content

chore(deps): bump lru from 0.18.2 to 0.18.3 - #5876

Open
dependabot[bot] wants to merge 3 commits into
mainfrom
dependabot/cargo/lru-0.18.3
Open

chore(deps): bump lru from 0.18.2 to 0.18.3#5876
dependabot[bot] wants to merge 3 commits into
mainfrom
dependabot/cargo/lru-0.18.3

Conversation

@dependabot

@dependabot dependabot Bot commented on behalf of github Sep 4, 2026

Copy link
Copy Markdown
Contributor

Bumps lru from 0.18.2 to 0.18.3.

Changelog

Sourced from lru's changelog.

v0.18.3 - 2026-08-27

  • Add sparse constructor.
Commits

Dependabot compatibility score

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot show <dependency name> ignore conditions will show all of the ignore conditions of the specified dependency
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

Bumps [lru](https://github.com/jeromefroe/lru-rs) from 0.18.2 to 0.18.3.
- [Changelog](https://github.com/jeromefroe/lru-rs/blob/master/CHANGELOG.md)
- [Commits](jeromefroe/lru-rs@0.18.2...0.18.3)

---
updated-dependencies:
- dependency-name: lru
  dependency-version: 0.18.3
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
@dependabot dependabot Bot added dependencies Pull requests that update a dependency file rust Pull requests that update rust code labels Sep 4, 2026
@dependabot
dependabot Bot requested a review from Hmbown as a code owner September 4, 2026 21:16
@dependabot dependabot Bot added dependencies Pull requests that update a dependency file rust Pull requests that update rust code labels Sep 4, 2026
@github-actions

github-actions Bot commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

Thanks @dependabot[bot] for taking the time to contribute.

This repository is observing a maintainer-managed PR intake gate in dry-run mode, so this pull request is staying open. This note helps maintainers prepare the allowlist before any enforcement is considered.

Please read CONTRIBUTING.md for the expected contribution shape. A maintainer can grant recurring PR access by commenting /lgtm on a pull request.

@claude

claude Bot commented Sep 5, 2026

Copy link
Copy Markdown
Contributor

Claude finished @Hmbown's task in 36s —— View job


Reviewing PR #5876 — chore(deps): bump lru 0.18.2 → 0.18.3

  • Gather context (diff, changed files)
  • Understand request (dependabot lockfile bump — review for correctness/safety/reuse/tests/security)
  • Execute review
  • Post final summary

Summary

This PR only updates Cargo.lock (version + checksum for the lru crate, 0.18.2 → 0.18.3). No application source files are touched.

  • Correctness: N/A — no code changes.
  • Provider/model/route safety: N/A — unrelated to routing/provider logic.
  • Reuse/simplification: N/A.
  • Tests: N/A — a patch-level lockfile bump doesn't need new tests.
  • Security: Upstream changelog for 0.18.3 only adds a sparse() constructor (see jeromefroe/lru-rs CHANGELOG) — purely additive, no known advisories.

I checked the three consumers of LruCache in this repo (crates/tui/src/tools/web/cache.rs, crates/tui/src/regex_cache.rs, crates/tui/src/llm_response_cache.rs) — none use anything affected by this bump.

Verdict: LGTM, safe to merge as a routine dependency update. No changes requested.

Note: this repo's PR intake gate is in dry-run mode per the earlier bot comment — a maintainer may still want to /lgtm this to keep future dependabot PRs from getting the same notice.

@codewhale-agent codewhale-agent Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Codewhale review

Dependabot updates Cargo.lock to raise lru from 0.18.2 to 0.18.3. The patch release adds a sparse constructor and a lint fix, with no source changes in this PR.

Findings

  • [INFO] Lockfile-only update; confirm manifest compatibility (Cargo.lock:3020)
    Only Cargo.lock is changed. This is expected if the Cargo.toml requirement (e.g. lru = "0.18" or lru = "0.18.2") already permits 0.18.3. Verify that the manifest does not pin an exact older version and rely on CI for build/test validation.

Assessment

Safe semver-compatible patch bump. No application code or tests are affected; the lockfile changes are appropriate for the 0.18.3 release.


Advisory review by Codewhale (codewhale review --pr 5876 --post, head 4e145338883218b7a9c0cba6e73d36579f10120e). Line-specific findings are also posted as inline review comments; mechanical fixes arrive as committable suggestions you can apply from the Files tab. CODEOWNERS approval still governs merge.

Comment thread Cargo.lock
name = "lru"
version = "0.18.2"
version = "0.18.3"
source = "registry+https://github.com/rust-lang/crates.io-index"

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

[INFO] Lockfile-only update; confirm manifest compatibility

Only Cargo.lock is changed. This is expected if the Cargo.toml requirement (e.g. lru = "0.18" or lru = "0.18.2") already permits 0.18.3. Verify that the manifest does not pin an exact older version and rely on CI for build/test validation.

@claude

claude Bot commented Sep 5, 2026

Copy link
Copy Markdown
Contributor

Claude finished @Hmbown's task in 41s —— View job


Reviewing PR #5876 — chore(deps): bump lru 0.18.2 → 0.18.3

  • Gather context (diff, manifest, lockfile)
  • Understand request (dependabot lockfile bump — review for correctness/safety/reuse/tests/security)
  • Execute review
  • Post final summary

Summary

Lockfile-only change (Cargo.lock:3018-3021, lru 0.18.2 → 0.18.3, checksum updated). No source files touched.

  • Correctness: N/A — no code changes.
  • Manifest compatibility (resolves the earlier INFO finding): crates/tui/Cargo.toml:103 declares lru = "0.18", with a comment noting lru >= 0.18.2 is required to fix RUSTSEC-2026-0253 (panic-unsafe pop()). 0.18.3 satisfies both the semver range and the RUSTSEC floor, so this is a safe, in-range patch bump.
  • Provider/model/route safety: N/A — unrelated.
  • Reuse/simplification: N/A. Confirmed only one lru entry in Cargo.lock (no duplicate/diamond version pulled in transitively).
  • Tests: N/A for a patch bump; no new public API is consumed.
  • Security: Upstream 0.18.3 changelog only adds an LruCache::sparse() constructor and a lint fix — purely additive, no advisories, and it stays above the RUSTSEC-2026-0253 floor already pinned in crates/tui/Cargo.toml.

Checked the repo's LruCache consumers (crates/tui/src/tools/web/cache.rs, crates/tui/src/regex_cache.rs, crates/tui/src/llm_response_cache.rs) — none touch the new sparse() constructor or anything else affected by this release.

Verdict: LGTM, safe to merge as a routine dependency update. No changes requested.

@codewhale-agent codewhale-agent Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Codewhale review

This PR bumps the lru crate from 0.18.2 to 0.18.3 in Cargo.lock. The only change is the version and checksum; no source code changes are included.

Assessment

Looks correct: Cargo.lock is consistent with the described dependency upgrade and no code changes or tests are required for this lockfile-only update.


Advisory review by Codewhale (codewhale review --pr 5876 --post, head 9e81f9bf9227b330930bf1956afee46a9249e4b8). Line-specific findings are also posted as inline review comments; mechanical fixes arrive as committable suggestions you can apply from the Files tab. CODEOWNERS approval still governs merge.

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

Labels

dependencies Pull requests that update a dependency file rust Pull requests that update rust code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant