Skip to content

perf: probe visible/inventory paths with OsStr order, not Path components - #835

Merged
jonathanong merged 2 commits into
execute-plan/79dd0742-pr-4-reduce-nodeid-clones-in-selector-edge-appendfrom
execute-plan/79dd0742-pr-5-normalized-path-membership-without-pathcmp
Sep 5, 2026
Merged

perf: probe visible/inventory paths with OsStr order, not Path components#835
jonathanong merged 2 commits into
execute-plan/79dd0742-pr-4-reduce-nodeid-clones-in-selector-edge-appendfrom
execute-plan/79dd0742-pr-5-normalized-path-membership-without-pathcmp

fix: address review feedback for perf: probe visible/inventory paths …

b887520
Select commit
Loading
Failed to load commit list.
CodSpeed HQ / CodSpeed Performance Analysis succeeded Sep 5, 2026 in 0s

Performance Gate Passed

⚠️ Different runtime environments detected

Some benchmarks with significant performance changes were compared across different runtime environments,
which may affect the accuracy of the results.

Open the report in CodSpeed to investigate

⚡ 5 improved benchmarks
✅ 79 untouched benchmarks
⏩ 4 skipped benchmarks1

Performance Changes

Mode Benchmark BASE HEAD Efficiency
Memory extract 694.4 KB 213.6 KB ×3.3
Simulation extract 12 ms 10.4 ms +15.46%
Simulation facts/imports_and_symbols 1.9 ms 1.6 ms +12.9%
Memory facts/imports_and_symbols 129.1 KB 115.9 KB +11.43%
Simulation extract 9 ms 8.2 ms +10.4%

Tip

Curious why performance improved? Comment @codspeedbot explain why performance improved on this PR, or directly use the CodSpeed MCP with your agent.


Comparing execute-plan/79dd0742-pr-5-normalized-path-membership-without-pathcmp (b887520) with execute-plan/79dd0742-pr-4-reduce-nodeid-clones-in-selector-edge-append (d6a55b7)

Open in CodSpeed

Footnotes

  1. 4 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports.