Skip to content

Commit edd838b

Browse files
sdsrssclaude
andcommitted
fix(mcp): clippy 1.95 unnecessary_sort_by on v0.11.4 rollup (v0.11.5)
CI on v0.11.4 failed with clippy::unnecessary_sort_by on src/mcp/server/tools.rs:503-504 (sort_by(|a, b| b.0.cmp(&a.0))). Local clippy 0.1.91 didn't catch it; CI's stable 1.95 did. Applied the exact suggested rewrite: caller_entries.sort_by_key(|e| std::cmp::Reverse(e.0)); callee_entries.sort_by_key(|e| std::cmp::Reverse(e.0)); Functional behavior identical. v0.11.4 tag + release remain pointing at the failing commit (released artifacts are not retroactively edited). Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
1 parent 0eb5a9e commit edd838b

File tree

12 files changed

+37
-18
lines changed

12 files changed

+37
-18
lines changed

.claude-plugin/marketplace.json

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5,14 +5,14 @@
55
},
66
"metadata": {
77
"description": "AST knowledge graph plugin for Claude Code — semantic search, call graph, HTTP tracing, impact analysis",
8-
"version": "0.11.4"
8+
"version": "0.11.5"
99
},
1010
"plugins": [
1111
{
1212
"name": "code-graph-mcp",
1313
"source": "./claude-plugin",
1414
"description": "AST knowledge graph for intelligent code navigation — auto-indexes your codebase and provides semantic search, call graph traversal, HTTP route tracing, and impact analysis via MCP tools",
15-
"version": "0.11.4",
15+
"version": "0.11.5",
1616
"author": {
1717
"name": "sdsrs"
1818
},

CHANGELOG.md

Lines changed: 19 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,24 @@
11
# Changelog
22

3+
## v0.11.5 — Hotfix: clippy 1.95 parity (`unnecessary_sort_by`)
4+
5+
`-D warnings` on stable clippy 1.95 flagged the two `sort_by(|a, b| b.0.cmp(&a.0))`
6+
calls added in v0.11.4 rollup. Local clippy (0.1.91, ~4 months behind stable)
7+
accepted them. Functional behavior unchanged.
8+
9+
### Fix
10+
11+
- `src/mcp/server/tools.rs:503-504`: `sort_by(|a, b| b.0.cmp(&a.0))`
12+
`sort_by_key(|e| std::cmp::Reverse(e.0))` (applied exactly as clippy suggested).
13+
14+
### Why v0.11.4 shipped red
15+
16+
Local pre-push ran `cargo clippy --all-targets -- -D warnings` — passed on 0.1.91.
17+
CI uses `dtolnay/rust-toolchain@stable` which pulls whatever's latest
18+
(1.95.0 at ship time), catching `clippy::unnecessary_sort_by` which landed post-0.1.91.
19+
Functional code from v0.11.4 is unaffected; only the `-D warnings` gate broke.
20+
v0.11.4 tag + release left pointing at the failing commit as a historical artifact.
21+
322
## v0.11.4 — Integration-friction fixes: ast_search hint + acronym expansion + call graph rollup
423

524
Integration-test pass against Claude Code found three specific friction points

Cargo.lock

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

Cargo.toml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
[package]
22
name = "code-graph-mcp"
3-
version = "0.11.4"
3+
version = "0.11.5"
44
edition = "2021"
55

66
[features]

claude-plugin/.claude-plugin/plugin.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44
"author": {
55
"name": "sdsrs"
66
},
7-
"version": "0.11.4",
7+
"version": "0.11.5",
88
"keywords": [
99
"code-graph",
1010
"ast",

npm/darwin-arm64/package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
{
22
"name": "@sdsrs/code-graph-darwin-arm64",
3-
"version": "0.11.4",
3+
"version": "0.11.5",
44
"description": "code-graph-mcp binary for macOS ARM64",
55
"license": "MIT",
66
"repository": {

npm/darwin-x64/package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
{
22
"name": "@sdsrs/code-graph-darwin-x64",
3-
"version": "0.11.4",
3+
"version": "0.11.5",
44
"description": "code-graph-mcp binary for macOS x64",
55
"license": "MIT",
66
"repository": {

npm/linux-arm64/package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
{
22
"name": "@sdsrs/code-graph-linux-arm64",
3-
"version": "0.11.4",
3+
"version": "0.11.5",
44
"description": "code-graph-mcp binary for Linux ARM64",
55
"license": "MIT",
66
"repository": {

npm/linux-x64/package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
{
22
"name": "@sdsrs/code-graph-linux-x64",
3-
"version": "0.11.4",
3+
"version": "0.11.5",
44
"description": "code-graph-mcp binary for Linux x64",
55
"license": "MIT",
66
"repository": {

npm/win32-x64/package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
{
22
"name": "@sdsrs/code-graph-win32-x64",
3-
"version": "0.11.4",
3+
"version": "0.11.5",
44
"description": "code-graph-mcp binary for Windows x64",
55
"license": "MIT",
66
"repository": {

0 commit comments

Comments
 (0)