[mstdokumaci]

Summary by CodeRabbit

• Bug Fixes

  • Improved import scanning accuracy, including aliases, chained paths, escaped paths, sorting, and import-block boundaries.
  • Excluded invalid and nested imports from analysis.

• Refactor

  • Internalized implementation details and compatibility declarations, reducing the public API surface.

• Tests

  • Added comprehensive coverage for AST scanning and import classification.
  • Updated existing tests to use the current import-analysis behavior.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: c119e9d1-4fce-4b80-bcf9-e3f625822d39

📥 Commits

Reviewing files that changed from the base of the PR and between 1e953d1 and a0dde33.

📒 Files selected for processing (6)

• build.zig
• src/ast_scan.zig
• src/ast_scan_test.zig
• src/compat.zig
• src/zsort.zig
• src/zsort_test.zig

---

📝 Walkthrough

Walkthrough

The change internalizes implementation exports, adds comprehensive AST scanner tests, updates zsort tests to use ast_scan, and adds the AST scanner test target to the test and check build steps.

Changes

AST scanner API and validation

Layer / File(s)	Summary
Internalize implementation exports src/ast_scan.zig, src/zsort.zig, src/compat.zig	AST scanner helpers, zsort implementation declarations, and selected compatibility aliases are no longer publicly exported.
Cover AST scanning and update consumers src/ast_scan_test.zig, src/zsort_test.zig	New tests cover import classification, boundaries, aliases, paths, sorting, and invalid imports. Existing zsort tests now use ast_scan APIs and constants.
Wire AST scanner tests into the build build.zig	The AST scanner test target runs with the test step and compiles with the check step.

Estimated code review effort: 3 (Moderate) | ~20 minutes

Possibly related PRs

• mstdokumaci/zsort#4: Modifies the same AST-scanning symbols and adds related tests.
• mstdokumaci/zsort#5: Changes ast_scan.zig and its import-analysis tests.
• mstdokumaci/zsort#6: Overlaps in build.zig and src/zsort.zig changes.

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly describes the main change of moving AST scan tests into a separate file.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch reorg-tests

---

_{Comment @coderabbitai help to get the list of available commands.}