# Contributing to cqs
Thank you for your interest in contributing to cqs!
## Development Setup
**Requires Rust 1.93+** (check with `rustc --version`)
1. Clone the repository:
```bash
git clone https://github.com/jamie8johnson/cqs
cd cqs
```
2. Build:
```bash
cargo build cargo build --features gpu-index ```
3. Run tests:
```bash
cargo test cargo test --features gpu-index ```
4. Initialize and index (for manual testing):
```bash
cargo run -- init
cargo run -- index
cargo run -- "your search query"
```
5. Set up pre-commit hook (recommended):
```bash
git config core.hooksPath .githooks
```
This runs `cargo fmt --check` before each commit.
## Code Style
- Run `cargo fmt` before committing
- No clippy warnings: `cargo clippy -- -D warnings`
- Add tests for new features
- Follow existing code patterns
### `_with_*` Function Naming Convention
Functions that accept pre-loaded resources use a `_with_<resource>` suffix:
| `_with_graph` | Pre-loaded call graph | `gather_with_graph()` |
| `_with_options` | Config struct parameter | `scout_with_options()` |
| `_with_embedding` | Pre-computed embedding | `suggest_placement_with_embedding()` |
| `_with_resources` | Pre-loaded embedder + graph | `task_with_resources()` |
Rules:
- The base function loads its own resources. The `_with_*` variant accepts them.
- Don't stack suffixes (`_with_graph_depth`). Add parameters to the existing `_with_*` function instead.
- If the `_with_*` variant has no external callers, fold it into the base function.
## Pull Request Process
1. Fork the repository and create a feature branch
2. Make your changes
3. Ensure all checks pass:
```bash
cargo test --features gpu-index
cargo clippy --features gpu-index -- -D warnings
cargo fmt --check
```
4. Update documentation if needed (README, CLAUDE.md)
5. Submit PR against `main`
## What to Contribute
### Good First Issues
- Look for issues labeled `good-first-issue`
- Documentation improvements
- Test coverage improvements
### Feature Ideas
- Additional language support (see `src/language/` for current list — 51 languages supported)
- Non-CUDA GPU support (ROCm for AMD, Metal for Apple Silicon)
- VS Code extension
- Performance improvements
- CLI enhancements
### Bug Reports
When reporting bugs, please include:
- cqs version (`cqs --version`)
- OS and architecture
- Steps to reproduce
- Expected vs actual behavior
## Architecture Overview
```
src/
cli/ - Command-line interface (clap)
mod.rs - Argument parsing, command dispatch
commands/ - Command implementations
mod.rs, query.rs, index.rs, stats.rs, graph.rs, init.rs, doctor.rs, notes.rs, reference.rs, similar.rs, explain.rs, diff.rs, drift.rs, trace.rs, impact.rs, impact_diff.rs, test_map.rs, context.rs, resolve.rs, dead.rs, gc.rs, gather.rs, project.rs, audit_mode.rs, read.rs, stale.rs, related.rs, where_cmd.rs, scout.rs, onboard.rs, convert.rs, review.rs, ci.rs, health.rs, suggest.rs, deps.rs, task.rs, blame.rs
chat.rs - Interactive REPL (wraps batch mode with rustyline)
batch/ - Batch mode: persistent Store + Embedder, stdin commands, JSONL output, pipeline syntax
mod.rs - BatchContext, vector index builder, main loop
commands.rs - BatchInput/BatchCmd parsing, dispatch router
handlers.rs - Handler functions (one per command)
pipeline.rs - Pipeline execution (pipe chaining via `|`)
types.rs - Output types (ChunkOutput, normalize_path)
config.rs - Configuration file loading
display.rs - Output formatting, result display
files.rs - File enumeration, lock files, path utilities
pipeline.rs - Multi-threaded indexing pipeline
signal.rs - Signal handling (Ctrl+C)
staleness.rs - Proactive staleness warnings for search results
watch.rs - File watcher for incremental reindexing
language/ - Tree-sitter language support
mod.rs - Language enum, LanguageRegistry, LanguageDef, ChunkType
rust.rs, python.rs, typescript.rs, javascript.rs, go.rs, c.rs, cpp.rs, java.rs, csharp.rs, fsharp.rs, powershell.rs, scala.rs, ruby.rs, bash.rs, hcl.rs, kotlin.rs, swift.rs, objc.rs, sql.rs, protobuf.rs, graphql.rs, php.rs, lua.rs, zig.rs, r.rs, yaml.rs, toml_lang.rs, elixir.rs, erlang.rs, gleam.rs, haskell.rs, julia.rs, ocaml.rs, css.rs, perl.rs, html.rs, json.rs, xml.rs, ini.rs, nix.rs, make.rs, latex.rs, solidity.rs, cuda.rs, glsl.rs, svelte.rs, razor.rs, vbnet.rs, vue.rs, aspx.rs, markdown.rs
store/ - SQLite storage layer (Schema v12, WAL mode)
mod.rs - Store struct, open/init, FTS5, RRF fusion
chunks.rs - Chunk CRUD, embedding_batches() for streaming
notes.rs - Note CRUD, note_embeddings(), brute-force search
calls.rs - Call graph storage and queries
types.rs - Type edge storage and queries
helpers.rs - Types, embedding conversion functions
migrations.rs - Schema migration framework
parser/ - Code parsing (tree-sitter + custom parsers, delegates to language/ registry)
mod.rs - Parser struct, parse_file(), parse_file_all(), supported_extensions()
types.rs - Chunk (incl. parent_type_name), CallSite, FunctionCalls, TypeRef, ParserError
chunk.rs - Chunk extraction, signatures, doc comments, parent type extraction
calls.rs - Call graph extraction, callee filtering
injection.rs - Multi-grammar injection (HTML→JS/CSS via set_included_ranges)
markdown.rs - Heading-based markdown parser, cross-reference extraction
embedder.rs - ONNX model (E5-base-v2), 769-dim embeddings
reranker.rs - Cross-encoder re-ranking (ms-marco-MiniLM-L-6-v2)
search.rs - Search algorithms, name matching, HNSW-guided search
math.rs - Vector math utilities (cosine similarity, SIMD)
hnsw/ - HNSW index with batched build, atomic writes
mod.rs - HnswIndex, LoadedHnsw (self_cell), HnswError, VectorIndex impl
build.rs - build(), build_batched() construction
search.rs - Nearest-neighbor search
persist.rs - save(), load(), checksum verification
safety.rs - Send/Sync and loaded-index safety tests
convert/ - Document-to-Markdown conversion (optional, "convert" feature)
mod.rs - ConvertOptions, convert_path(), format detection
html.rs - HTML → Markdown via fast_html2md
pdf.rs - PDF → Markdown via Python pymupdf4llm (shell out)
chm.rs - CHM → 7z extract → HTML → Markdown
naming.rs - Title extraction, kebab-case filename generation
cleaning.rs - Extensible tag-based cleaning rules (7 rules)
webhelp.rs - Web help site detection and multi-page merge
cagra.rs - GPU-accelerated CAGRA index (optional)
nl.rs - NL description generation, JSDoc parsing
note.rs - Developer notes with sentiment, rewrite_notes_file()
diff.rs - Semantic diff between indexed snapshots
drift.rs - Drift detection (semantic change magnitude between snapshots)
reference.rs - Multi-index: ReferenceIndex, load, search, merge
gather.rs - Smart context assembly (BFS call graph expansion)
structural.rs - Structural pattern matching on code chunks
project.rs - Cross-project search registry
audit.rs - Audit mode persistence and duration parsing
focused_read.rs - Focused read logic (extract type dependencies)
impact/ - Impact analysis (callers + affected tests + diff-aware)
mod.rs - Public API, re-exports
types.rs - Impact types (CallerDetail, RiskScore, etc.)
analysis.rs - suggest_tests, find_transitive_callers
diff.rs - analyze_diff_impact
bfs.rs - Reverse BFS traversal
format.rs - JSON/Mermaid formatting
hints.rs - compute_hints, risk scoring
related.rs - Co-occurrence analysis (shared callers, callees, types)
scout.rs - Pre-investigation dashboard (search + callers/tests + staleness + notes)
task.rs - Single-call implementation brief (scout + gather + impact + placement + notes)
onboard.rs - Guided codebase tour (entry point + call chain + callers + types + tests)
review.rs - Diff review (impact-diff + notes + risk scoring)
ci.rs - CI pipeline (review + dead code + gate logic)
where_to_add.rs - Placement suggestion (semantic search + pattern extraction)
diff_parse.rs - Unified diff parser for impact-diff
health.rs - Codebase quality snapshot (dead code, staleness, hotspots)
suggest.rs - Auto-suggest notes from code patterns
config.rs - Configuration file support
index.rs - VectorIndex trait (HNSW, CAGRA)
lib.rs - Public API
.claude/
skills/ - Claude Code skills (auto-discovered)
groom-notes/ - Interactive note review and cleanup
update-tears/ - Session state capture for context persistence
release/ - Version bump, changelog, publish workflow
audit/ - 14-category code audit with parallel agents
red-team/ - Adversarial security audit (attacker mindset, PoC-required)
pr/ - WSL-safe PR creation
cqs-bootstrap/ - New project setup with tears infrastructure
cqs/ - Unified CLI dispatcher (search, graph, quality, notes, infrastructure)
reindex/ - Rebuild index with before/after stats
docs-review/ - Check project docs for staleness
migrate/ - Schema version upgrades
troubleshoot/ - Diagnose common cqs issues
cqs-batch/ - Batch mode with pipeline syntax
cqs-plan/ - Task planning with templates
```
**Key design notes:**
- 769-dim embeddings (768 from E5-base-v2 + 1 sentiment dimension)
- HNSW index is chunk-only; notes use brute-force SQLite search (always fresh)
- Streaming HNSW build via `build_batched()` for memory efficiency
- Large chunks split by windowing (480 tokens, 64 overlap); notes capped at 10k entries
- Schema migrations allow upgrading indexes without full rebuild
- Skills in `.claude/skills/*/SKILL.md` are auto-discovered by Claude Code
## Adding Injection Rules (Multi-Grammar)
Files like HTML contain embedded languages (`<script>` → JS, `<style>` → CSS). cqs handles this via injection rules on `LanguageDef`.
**To add injection rules for a new host language:**
1. Define `InjectionRule` entries in the language's `LanguageDef` (`src/language/<lang>.rs`):
```rust
injections: &[
InjectionRule {
container_kind: "script_element", content_kind: "raw_text", target_language: "javascript", detect_language: Some(detect_fn), },
],
```
2. `container_kind` / `content_kind` must match the host grammar's node kinds (inspect with `tree-sitter parse`).
3. `target_language` must be a valid `Language` name with a grammar (validated at runtime in `find_injection_ranges`).
4. `detect_language` receives the container node and source — return `Some("typescript")` to override the default, `Some("_skip")` to skip the container entirely, or `None` for the default.
5. Injection is single-level only. Inner languages are not re-scanned for their own injections.
6. The two-phase flow in `parse_file` and `parse_file_relationships` automatically handles injection when `injections` is non-empty. No changes needed outside the language definition.
**Key files:** `src/language/mod.rs` (InjectionRule struct), `src/parser/injection.rs` (parsing logic), `src/language/html.rs` (reference implementation).
## Questions?
Open an issue for questions or discussions.