Skip to content

Fix suggest_improvement storage safety - #2734

Merged
Widthdom merged 6 commits into
mainfrom
codex/fix-issue1614-1796-1921-1940
May 31, 2026
Merged

Fix suggest_improvement storage safety#2734
Widthdom merged 6 commits into
mainfrom
codex/fix-issue1614-1796-1921-1940

Conversation

@Widthdom

Copy link
Copy Markdown
Owner

Summary

  • Sanitize suggest_improvement GitHub issue titles before submission.
  • Redact common credential/token patterns before persisting or submitting suggestions.
  • Validate the resolved .cdidx suggestion-store directory is writable before taking the store lock, and report cdidx_dir in tool payloads.
  • Prune stale or over-limit local suggestions into .cdidx/suggestions-*.archive.jsonl, with config/env controls.

Validation

  • dotnet run --project tools/CodeIndex.Changelog -- check
  • dotnet test tests/CodeIndex.Tests/CodeIndex.Tests.csproj --filter "FullyQualifiedName~GitHubIssueReporterTests|FullyQualifiedName~SuggestionStoreTests|FullyQualifiedName~CdidxConfigFileTests|FullyQualifiedName~McpServerTests.SuggestImprovement" -p:UseSharedCompilation=false
  • dotnet ./src/CodeIndex/bin/Debug/net8.0/cdidx.dll index . --changed-between origin/main HEAD --json
  • dotnet ./src/CodeIndex/bin/Debug/net8.0/cdidx.dll status --check --json remains stale only because the full workspace scan hit E013_INDEX_EXTRACTION_STALLED in SymbolExtractor.JavaScriptTypeScriptSupport.cs; changed files are matched after the diff index.

Documentation And Changelog

  • USER_GUIDE.md
  • changelog.d/unreleased/1614.fixed.md
  • changelog.d/unreleased/1796.fixed.md
  • changelog.d/unreleased/1921.fixed.md
  • changelog.d/unreleased/1940.fixed.md

Review

  • Codex adversarial review found two actionable issues.
  • Fixed: duplicate prune paths now persist the pruned live store.
  • Fixed: writability probing now uses a unique CreateNew temp file and preserves any existing .write_probe.

Follow-Up Candidates

Fixes #1614
Fixes #1796
Fixes #1921
Fixes #1940

@Widthdom
Widthdom marked this pull request as ready for review May 31, 2026 06:15
@Widthdom
Widthdom merged commit 03f5099 into main May 31, 2026
10 checks passed
@Widthdom
Widthdom deleted the codex/fix-issue1614-1796-1921-1940 branch May 31, 2026 06:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

1 participant