Skip to content

Commit b760186

Browse files
osvaloisclaude
andcommitted
fix(ci): fix sota_evaluation type-limit warning, cargo-deny skip advisories
- sota_evaluation.rs: replace useless usize >= 0 comparison - cargo-deny: check only licenses/bans/sources (advisories handled by cargo-audit separately; advisory DB uses CVSS 4.0 unsupported by current cargo-deny) Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent 9aedd62 commit b760186

2 files changed

Lines changed: 3 additions & 6 deletions

File tree

.github/workflows/devsecops.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -54,7 +54,7 @@ jobs:
5454
- name: Install cargo-deny
5555
run: cargo install --locked cargo-deny@0.18.2
5656
- name: Run cargo-deny
57-
run: cargo deny check --config deny.toml
57+
run: cargo deny check licenses bans sources --config deny.toml
5858

5959
# ── Dependency vulnerability audit (post-merge + schedule only) ──────────
6060
# Skipped on PRs: cargo-audit requires a full workspace resolve and is

crates/halcon-cli/tests/sota_evaluation.rs

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -267,11 +267,8 @@ mod sota_tests {
267267
fn mcp_integration_available() {
268268
let config = AppConfig::default();
269269

270-
// MCP should be configurable (may have 0 reconnects as default)
271-
assert!(
272-
config.mcp.max_reconnect_attempts >= 0,
273-
"MCP config should exist"
274-
);
270+
// MCP config should be present and initialized
271+
let _ = config.mcp.max_reconnect_attempts; // field exists
275272

276273
// Servers list should be available
277274
assert!(

0 commit comments

Comments
 (0)