Point codedb_remote at api.wiki.codes#327
Conversation
Benchmark Regression ReportThresholds: 10.00% and 50,000 ns absolute delta
|
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 32b5e3f82c
ℹ️ 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".
| const scope = getStr(args, "scope") orelse query orelse "runtime"; | ||
| if (!std.mem.eql(u8, scope, "runtime") and !std.mem.eql(u8, scope, "all")) { | ||
| out.appendSlice(alloc, "error: scope must be 'runtime' or 'all'") catch {}; |
There was a problem hiding this comment.
Treat empty query as absent for score/cves scope default
handleRemote derives scope via getStr(args, "scope") orelse query orelse "runtime", so if a client includes query:"" (common in generic MCP forms) and omits scope, the empty query is selected and then rejected by the runtime/all validation. That contradicts the documented default-to-runtime behavior and makes otherwise valid score/cves requests fail based only on an unused empty field; empty query should be ignored for this fallback path.
Useful? React with 👍 / 👎.
Summary
Verification