Skip to content

test: add Kibana Knip conformance fuzzing - #17

Merged
longlho merged 1 commit into
mainfrom
codex/knip-kibana-conformance
May 18, 2026
Merged

longlho merged 1 commit into
mainfrom
codex/knip-kibana-conformance

Conversation

@longlho

@longlho longlho commented May 18, 2026

Copy link
Copy Markdown
Contributor

Summary

  • add a Kibana/Knip conformance harness that copies the Bazel-fetched Kibana fixture, injects synthetic unused TS files, disables Knip framework plugins, and compares the core TS graph
  • document Codescythe's intentional difference from Knip: Codescythe reports every unreachable file in a dead subgraph, while Knip may report only the dead root
  • add //benchmarks:kibana_conformance_test as a functional_test Bazel target and split GitHub Actions into normal and functional Bazel test lanes
  • refresh the benchmark docs with current Kibana numbers

Test Plan

  • node --experimental-transform-types benchmarks/run.ts --fixture kibana --samples 3 --warmups 1 --skip-build
  • node --experimental-transform-types benchmarks/kibana_conformance.ts --skip-build --fuzz-files 16
  • bazel test //benchmarks:kibana_conformance_test
  • bazel test //... --build_tests_only --test_tag_filters=functional_test
  • bazel test //... --build_tests_only --test_tag_filters=-functional_test
  • node --experimental-transform-types benchmarks/run.ts --help
  • node --experimental-transform-types benchmarks/kibana_conformance.ts --help
  • git diff --check

@longlho
longlho force-pushed the codex/knip-kibana-conformance branch from 320458e to b867f26 Compare May 18, 2026 21:11
@longlho
longlho force-pushed the codex/knip-kibana-conformance branch from b867f26 to 5758ac9 Compare May 18, 2026 21:16
@longlho
longlho merged commit c870aca into main May 18, 2026
10 checks passed
@longlho
longlho deleted the codex/knip-kibana-conformance branch May 18, 2026 21:29
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