Skip to content

Isolate unreadable marker fixtures across target frameworks - #5035

Merged
Widthdom merged 2 commits into
mainfrom
fix-issue5019
Aug 2, 2026
Merged

Isolate unreadable marker fixtures across target frameworks#5035
Widthdom merged 2 commits into
mainfrom
fix-issue5019

Conversation

@Widthdom

@Widthdom Widthdom commented Aug 2, 2026

Copy link
Copy Markdown
Owner

Summary

  • Move the unreadable MCP marker-fingerprint fixture to a unique temporary workspace outside repository and target-framework output trees.
  • Temporarily scope the process current directory to that workspace so MCP path authorization remains valid, then restore it in finally.
  • Document the fixture-isolation contract in both English and Japanese and add a bilingual changelog fragment.

Root cause

The fixture previously lived beneath the test assembly's current directory. During a default multi-target run, one target framework could recursively enumerate another target's bin tree while the latter intentionally held its secret directory unreadable. Filtering bin / obj after recursive enumeration was too late to prevent the access failure.

Validation

  • dotnet build CodeIndex.sln -c Release -p:UseSharedCompilation=false --no-restore
    • succeeded with 0 warnings and 0 errors
  • Focused unreadable-marker and source-policy tests under net8.0 and net9.0
    • passed, including an adversarial concurrent cross-target run
  • dotnet test --no-restore --nologo -p:UseSharedCompilation=false
  • dotnet run --project tools/CodeIndex.Changelog -- check
    • validated all 105 fragments
  • cdidx status --check --json
    • workspace/index matched; graph data complete
  • Codex adversarial review against the final origin/main..HEAD
    • No blocking/actionable issues found.

Documentation and changelog

  • Updated TESTING_GUIDE.md in English and Japanese.
  • Added changelog.d/unreleased/5019.internal.md.
  • Did not edit CHANGELOG.md.

Follow-up

Fixes #5019

@Widthdom
Widthdom marked this pull request as ready for review August 2, 2026 00:48
@Widthdom
Widthdom merged commit 3d28e67 into main Aug 2, 2026
11 checks passed
@Widthdom
Widthdom deleted the fix-issue5019 branch August 2, 2026 00:48
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.

Isolate unreadable marker fixtures from cross-target full test runs

1 participant