Repository navigation
fix(scip): index converges with the discovery census and reports a summary - #1674
Merged
Merged
Conversation
…t a summary (#1673) User-reported: a mixed C#/other repo indexed via scip-dotnet finished with no report, and every rerun said "repository '.' has only 32 of 46 files indexed. Continuing." — forever. - Minimal-File-node fallbacks on every shortfall path in the SCIP pipeline: supplement-parse errors, supplement-parse crashes, and failed SCIP-covered writes all still record the file, so the graph count converges with the discovery census by construction and reruns reach "already indexed". - The SCIP path now populates the CLI index summary (files by extension, function/class/CALLS counts), and the builder resets the summary before a SCIP run so a stale Tree-sitter summary from the same process can't print. - estimate_processing_time accepts cgcignore_path and the resume check passes the context's — the two sides of the "N of M" comparison now count the same set. - The "Continuing" branch names up to 5 missing files, so a genuine gap is actionable instead of an eternal loop. Regression tests drive the pipeline with a fake scip module against a real embedded database, including a simulated parser crash.
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
github-project-automation
Bot
moved this from Backlog tasks
to Done
in CGC Progress Board
Contributor
|
Hi! 👋 Join our CodeGraphContext Discord channel to collaborate: https://discord.gg/dR4QY32uYQ |
Contributor
🔍 PR Code Graph Analysisfix(scip): index converges with the discovery census and reports a summary (#1674) 📊 Interactive VisualizationView the blast radius graph: PR Reviewer Dashboard 📦 ArtifactsThe graph JSON has been uploaded as a build artifact: Generated by CodeGraphContext using FalkorDB Lite |
This branch was successfully deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fixes the user-reported scip-dotnet loop: no summary after indexing, and "repository '.' has only 32 of 46 files indexed. Continuing." on every rerun, forever.
estimate_processing_timenow takes the context'scgcignore_path, so both sides of the "N of M" comparison count the same file set.Regression tests drive the full pipeline with a fake scip module against a real embedded database — including a simulated parser crash — asserting graph-count ≥ census and a populated summary. Unit suite 1420 passed / 7 skipped; full integration gate 89 passed / 0 failed.
Fixes #1673