kettle-rb / ast-merge / 20738076991

94%

Build Type

push

github

Committed by pboling

Commit Message

💥 New features and breaking changes

### Added

- `TestableNode` spec helper class that wraps a mock in a real `TreeHaver::Node`, providing consistent API testing without relying on fragile mocks
- `Recipe::Preset#match_refiner` accessor method (was missing, causing errors in Recipe::Runner)
- Minimal reproduction specs for `to_commonmark` normalization behavior:
  - `spec/integration/link_reference_preservation_spec.rb` - tests link ref preservation
  - `spec/integration/table_formatting_preservation_spec.rb` - tests table padding preservation
- `Ast::Merge::PartialTemplateMergerBase` - Abstract base class for parser-agnostic partial template merging
  - `#build_position_based_signature_generator` - Creates signature generators that match elements by position
  - Position counters reset per document key, enabling tables at same position to match regardless of structure

### Changed

- **BREAKING**: `NavigableStatement#text` now requires nodes to conform to TreeHaver Node API (must have `#text` method)
  - Removed conditional fallbacks for `to_plaintext`, `to_commonmark`, `slice`
  - Nodes must now implement `#text` directly (all TreeHaver backends already do)
- **BREAKING**: `ContentMatchRefiner#extract_content` now requires nodes to conform to TreeHaver Node API
  - Removed conditional fallbacks for `text_content`, `string_content`, `content`, `to_s`
  - Custom `content_extractor` proc still supported for non-standard nodes
- Signature generators and typing scripts now receive TreeHaver nodes directly (no NavigableStatement wrapping)
- Removed NavigableStatement wrapping from `FileAnalyzable#generate_signature` and `NodeTyping.process`

### Removed

- **BREAKING**: `Ast::Merge::PartialTemplateMerger` removed. Use `Markdown::Merge::PartialTemplateMerger` directly.
  - The base class `Ast::Merge::PartialTemplateMergerBase` remains for other parsers to extend
  - Migration: change `Ast::Merge::PartialTemplateMerger.new(parser: :markly, ...)` to
    `Ma... (continued)

Run Details

794 of 886 branches covered (89.62%)

Branch coverage included in aggregate %.

7 of 18 new or added lines in 5 files covered. (38.89%)

32 existing lines in 1 file now uncovered.

2462 of 2540 relevant lines covered (96.93%)

31.34 hits per line