Skip to content

Fix assertion during lexical path normalization - #376

Draft
aryansk wants to merge 1 commit into
apple:mainfrom
aryansk:codex/issue-283-path-normalization
Draft

Fix assertion during lexical path normalization#376
aryansk wants to merge 1 commit into
apple:mainfrom
aryansk:codex/issue-283-path-normalization

Conversation

@aryansk

@aryansk aryansk commented Aug 8, 2026

Copy link
Copy Markdown

Fixes #283

What changed

  • Avoid reparsing mutable compacted storage while asserting the parent-directory invariant during lexical normalization.
  • Add a regression case for foo/../../../bar.

Validation

  • swift test --filter FilePathSyntaxTest — 6 tests passed.

No documentation or release-note changes are required.

@aryansk

aryansk commented Aug 9, 2026

Copy link
Copy Markdown
Author

Thanks @merosm for the approvals on this change — I appreciate the review and the confirmation that the Swift System updates are ready for maintainer handling.

@aryansk

aryansk commented Aug 9, 2026

Copy link
Copy Markdown
Author

Thanks again @merosm — I appreciate the additional approval and the continued review of this change. The PR remains available for maintainer handling.

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.

Assertion failure in FilePath.lexicallyNormalized()

2 participants