Skip to content

Remove constant_references from CoverageTracker#9

Merged
ngan merged 1 commit intomainfrom
np-remove-constant-references
Mar 12, 2026
Merged

Remove constant_references from CoverageTracker#9
ngan merged 1 commit intomainfrom
np-remove-constant-references

Conversation

@ngan
Copy link
Collaborator

@ngan ngan commented Mar 12, 2026

Summary

  • remove constant_references from FastCov::CoverageTracker and the native FastCov::Coverage implementation
  • delete the old Prism-based constant extractor and the benchmark/docs/specs that depended on it
  • keep coverage focused on line events, allocation tracing, and dynamic const_get tracking only

Testing

  • bundle exec rake compile
  • bundle exec rspec --fail-fast spec/lib/fast_cov/cache_spec.rb spec/lib/fast_cov/coverage/caching_spec.rb spec/lib/fast_cov/coverage/line_coverage_spec.rb
  • bundle exec rspec --fail-fast

Notes

This PR intentionally removes the existing constant-reference expansion path without adding StaticMap yet. The replacement static analysis API can be iterated on separately.

Co-authored-by: Codex <codex@openai.com>
@ngan ngan merged commit 7b15005 into main Mar 12, 2026
12 of 13 checks passed
@ngan ngan deleted the np-remove-constant-references branch March 12, 2026 06:02
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