feat(llm): send an explicit reasoning_effort for Kimi models that support it (GRAPHIFY_KIMI_EFFORT) - #2982
feat(llm): send an explicit reasoning_effort for Kimi models that support it (GRAPHIFY_KIMI_EFFORT)#2982Azeem1985 wants to merge 21 commits into
Graphify — worth a look
Worth a look — the grounded gate found no coupling regressions or blocking issues, but 5 advisory finding(s) below merit a look before merge.
Details
Graphify reviewed this change.
Worth a look — the grounded gate found no coupling regressions or blocking issues, but 5 advisory finding(s) below merit a look before merge.
Graphify review — findings
Adds an openai-cli extraction backend (Codex CLI via codex exec, stdin prompt, read-only sandbox, disabled MCP servers, serial by default) and forces it to concurrency 1 alongside ollama/claude-cli. Adds graphify extract --fallback-backend retry-on-second-backend, wraps the whole extract pipeline in the per-repo rebuild lock, and makes graphify watch --semantic run LLM extraction as a subprocess with forwarded --backend/--fallback-backend. Batches neo4j/falkordb --push into UNWIND queries with a new --batch-size (default 100), and sends an explicit reasoning_effort for the kimi backend.
Worth a look
- Crashed semantic pass with partial progress bypasses fallback and failure —
graphify/cli.py· Escalate · high- agreed by 2 of 2 members but NOT verified (no proof, no reproducing execution) — consensus is not a verdict; needs human review
- --no-cluster path now performs clustering and writes clustered graph schema —
graphify/cli.py· Escalate · high- agreed by 2 of 2 members but NOT verified (no proof, no reproducing execution) — consensus is not a verdict; needs human review
- Cargo introspection call lost its local import —
graphify/cli.py· Escalate · high- agreed by 2 of 2 members but NOT verified (no proof, no reproducing execution) — consensus is not a verdict; needs human review
- Non-Cargo extraction output path is nested under --cargo branch —
graphify/cli.py· Escalate · high- agreed by 2 of 2 members but NOT verified (no proof, no reproducing execution) — consensus is not a verdict; needs human review
- Indentation shift moves _extraction_incomplete / AST extraction under needs_llm block —
graphify/cli.py· Escalate · high- agreed by 2 of 2 members but NOT verified (no proof, no reproducing execution) — consensus is not a verdict; needs human review
Analysis details — impact, health, verification
Impact & health
Graphify review
Impact — 1360 functions depend on the 621 functions this change touches.
Health — this change adds coupling hotspots:
- new:
_rebuild_code()— 98 callers, 50 callees - new:
deduplicate_entities()— 63 callers, 21 callees - new:
build_merge()— 46 callers, 14 callees - new:
extract_files_direct()— 17 callers, 21 callees - new:
extract_corpus_parallel()— 26 callers, 11 callees - new:
_call_claude_cli()— 31 callers, 9 callees - new:
dispatch_command()— 2 callers, 121 callees - new:
_call_llm()— 11 callers, 21 callees - …and 24 more — each is listed as a finding
Verification — 1360 functions in the blast radius were not formally verified this run (proofs are advisory here).
Gate & verification
graphify gate
PASS — objectively clean (no health regressions, tests not run — proofs not run this pass (advisory)). Grounded, not self-assessed.
Advisory (not blocking):
- verification_scope: 1296 function(s) in the blast radius were not formally verified this run