intuit / intuit/infigraph

CI broken on main: cargo fmt + clippy failures block every open PR

Open
#15 3 comments 0 reactions 1 assignee Claimed by @johnintuit View on GitHub
Dominant language
Rust
Stars
88
Forks
16
PR merge metrics
No merged PRs in 30d

Description

main currently fails both CI gates (cargo fmt --all -- --check and cargo clippy --all-targets -- -D warnings), which blocks CI on every open PR regardless of what that PR actually changes — confirmed on PR #14, whose diff is doc-only (.md/.mdc files) and doesn't touch any of the failing files.

1. Formatting drift — cargo fmt --all -- --check

76 diffs across 12 files, all from recent remote/Neo4j/pgvector work:

- crates/infigraph-cli/src/info_commands.rs
- crates/infigraph-core/src/graph/neo4j_backend.rs
- crates/infigraph-core/src/lib.rs
- crates/infigraph-core/src/meta/postgres_store.rs
- crates/infigraph-core/src/multi/mod.rs
- crates/infigraph-core/tests/namespace_prefix.rs
- crates/infigraph-core/tests/neo4j_backend.rs
- crates/infigraph-core/tests/postgres_registry.rs
- crates/infigraph-docs/tests/remote_doc_embeddings.rs
- crates/infigraph-mcp/src/tools/groups.rs
- crates/infigraph-mcp/src/tools/search.rs

Fix: cargo fmt --all (mechanical, no logic change).

2. Clippy errors — cargo clippy --all-targets -- -D warnings

Two clippy::type_complexity errors that fail the build outright:

error: very complex type used. Consider factoring parts into `type` definitions
--> crates/infigraph-mcp/src/tools/search.rs:134:6
134 | ) -> Result<(Vec>, Vec<(String, Vec)>)> {

error: very complex type used. Consider factoring parts into `type` definitions
--> crates/infigraph-mcp/src/tools/search.rs:173:32
173 | fn get_search_data_remote() -> Result<(Vec>, Vec<(String, Vec)>)> {

Fix: introduce a type alias for the (Vec>, Vec<(String, Vec)>) tuple used by both signatures.

Impact

Any PR opened against current main — including ones with zero Rust changes — shows a red CI check unrelated to that PR's diff. Recommend fixing both as a small, isolated PR before/alongside merging anything else.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.