Skip to content

Perf: dedupe import resolution#58

Merged
longlho merged 1 commit into
mainfrom
codex/dedupe-import-resolution
May 23, 2026
Merged

Perf: dedupe import resolution#58
longlho merged 1 commit into
mainfrom
codex/dedupe-import-resolution

Conversation

@longlho
Copy link
Copy Markdown
Contributor

@longlho longlho commented May 23, 2026

Summary

  • dedupe duplicate import, dynamic import, glob, member-use, and re-export records after parsing
  • group static imports by source during graph inspection so each source resolves once per file

Benchmark

  • Kibana profiled run: total 14.68s -> 11.48s; graph inspect 7.31s -> 5.88s; resolver calls 1,789,125 -> 1,035,474
  • Kibana one-shot benchmark: codescythe 13,160ms; knip 34,506ms
  • Kibana Codescythe-only benchmark: 12,286.5ms +/-3.25% over 5 samples, 1 warmup

Tests

  • bazel test //crates/codescythe:codescythe_test //benchmarks:kibana_fixture_test
  • git diff --check

@longlho longlho enabled auto-merge (squash) May 23, 2026 04:05
@longlho longlho merged commit 64bbdb0 into main May 23, 2026
1 check passed
@longlho longlho deleted the codex/dedupe-import-resolution branch May 23, 2026 04:06
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.

1 participant