Skip to content

Cap report schema summary work - #3284

Merged
Widthdom merged 2 commits into
mainfrom
fix-issue3146
Jun 5, 2026
Merged

Cap report schema summary work#3284
Widthdom merged 2 commits into
mainfrom
fix-issue3146

Conversation

@Widthdom

@Widthdom Widthdom commented Jun 5, 2026

Copy link
Copy Markdown
Owner

Summary

  • Cap cdidx report --db schema table enumeration at 64 entries and omit additional tables from schema.txt.
  • Cap displayed table names at 96 characters and report bounded row counts with a 1000-row scan limit per table.
  • Update report documentation and add a bilingual changelog fragment.

Validation

  • dotnet format CodeIndex.sln --verify-no-changes
  • dotnet build CodeIndex.sln -c Release -p:UseSharedCompilation=false
  • dotnet build
  • dotnet test tests/CodeIndex.Tests/CodeIndex.Tests.csproj -c Release -p:UseSharedCompilation=false --filter FullyQualifiedName~ReportCommandRunnerTests
  • dotnet test tests/CodeIndex.Tests/CodeIndex.Tests.csproj -c Release -p:UseSharedCompilation=false --filter FullyQualifiedName~ExtractorPluginRegistryTests.LoadPluginAssemblies_RetainsCapDiagnosticWhenCandidatesAlsoFailToLoad
  • dotnet run --project tools/CodeIndex.Changelog -- check
  • dotnet ./src/CodeIndex/bin/Debug/net8.0/cdidx.dll status --check --json

Full dotnet test CodeIndex.sln -c Release -p:UseSharedCompilation=false was attempted, but I stopped it after a long no-output run. It had reported one unrelated ExtractorPluginRegistryTests regex-timeout failure first; that specific test passed on isolated rerun for both target frameworks.

Documentation / Changelog

  • Updated USER_GUIDE.md report bundle text in English and Japanese.
  • Added changelog.d/unreleased/3146.security.md.

Review

Adversarial review found one documentation wording issue (no user content was too strong because schema summaries still include table names). That was fixed; no blocking/actionable issues remained after re-review.

Follow-up Candidates

  • None.

Fixes #3146

@Widthdom
Widthdom marked this pull request as ready for review June 5, 2026 03:15
@Widthdom
Widthdom merged commit f246ff4 into main Jun 5, 2026
10 checks passed
@Widthdom
Widthdom deleted the fix-issue3146 branch June 6, 2026 18:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

report schema summary should cap table scans on arbitrary DBs

1 participant