Skip to content

Preserve logical input paths - #306

Merged
1sgtpepper merged 3 commits into
mainfrom
fix/logical-display-paths
Aug 11, 2026
Merged

Preserve logical input paths#306
1sgtpepper merged 3 commits into
mainfrom
fix/logical-display-paths

Conversation

@1sgtpepper

@1sgtpepper 1sgtpepper commented Aug 4, 2026

Copy link
Copy Markdown
Owner

Summary

Diagnostics, graph spans, and baseline identities now preserve caller-visible lexical
input paths without resolving symlink targets. Absolute inputs are working-directory
relative by default, explicitly requested absolute paths retain their spelling,
incompatible native roots fail without leaking a path, and read-error details omit the
operating-system filename.

This PR is stacked on #322.

Linked issue

Fixes #239.

Validation

Exact-head GitHub Actions passed on efae340666fcea6288f3214699e643e3e634d72a
(CI run 31420710000):
CI/quality, CI/docs, CI/package, CI/public regression replay,
CI/self-check, CI/test (3.11) through CI/test (3.13), aggregate CI/test,
Docs/build-docs, and codecov/patch.

@1sgtpepper
1sgtpepper force-pushed the fix/logical-display-paths branch from 5046045 to 14b507c Compare August 6, 2026 21:02
@codecov

codecov Bot commented Aug 7, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@1sgtpepper 1sgtpepper added bug Something isn't working area:reporters Text, JSON, GitHub, SARIF reporting. area:project Project mode, ordering, and baselines. needs design Not ready for implementation. needs fixture Needs example input/output coverage. blocked Waiting on another issue or release. labels Aug 7, 2026
@1sgtpepper
1sgtpepper force-pushed the fix/logical-display-paths branch 13 times, most recently from 6a9881f to 82c2084 Compare August 9, 2026 11:42
@1sgtpepper
1sgtpepper changed the base branch from main to fix/protect-analysis-inputs August 10, 2026 09:05
@1sgtpepper
1sgtpepper force-pushed the fix/protect-analysis-inputs branch from cf25dde to 0fbdea4 Compare August 10, 2026 09:06
@1sgtpepper
1sgtpepper force-pushed the fix/logical-display-paths branch from f3e3988 to 921b712 Compare August 10, 2026 09:06
@1sgtpepper 1sgtpepper removed the needs fixture Needs example input/output coverage. label Aug 10, 2026
@1sgtpepper
1sgtpepper force-pushed the fix/logical-display-paths branch 2 times, most recently from 9db480b to 2c635e9 Compare August 10, 2026 09:21
@1sgtpepper
1sgtpepper changed the base branch from fix/protect-analysis-inputs to test/public-regression-contract August 10, 2026 18:36
@1sgtpepper
1sgtpepper force-pushed the fix/logical-display-paths branch from 2c635e9 to 1d68347 Compare August 10, 2026 18:37
@1sgtpepper 1sgtpepper removed needs design Not ready for implementation. blocked Waiting on another issue or release. labels Aug 10, 2026
@1sgtpepper
1sgtpepper marked this pull request as ready for review August 10, 2026 19:35
@1sgtpepper
1sgtpepper force-pushed the fix/logical-display-paths branch from 226691c to b3600cc Compare August 11, 2026 15:26
@1sgtpepper
1sgtpepper changed the base branch from test/public-regression-contract to main August 11, 2026 15:26
@1sgtpepper
1sgtpepper merged commit 7594387 into main Aug 11, 2026
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:project Project mode, ordering, and baselines. area:reporters Text, JSON, GitHub, SARIF reporting. bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Preserve logical input paths in default diagnostics

1 participant