[EtaCassiopeia]

Summary

rift 0.23.0 (rift#1495, PR achird-labs/rift#1529) reads an XPath-selected element as Mountebank's nodeValue does: as its first child node's text, or "undefined" when it has no children or starts with a child element. So //a on <a>x<b/>y</a> is x, not xy.

PredicateEvaluator backs Imposter.recorded(RequestMatch), but it still read getTextContent(), the element's whole text. On the 0.23.0 pin, recorded(match) could therefore disagree with what the engine's xpath stub matched and with what verify counted.

• Elements: an element reads as its first child's value, or undefined. Text and attribute nodes read their own value.
• Parsing: the parser coalesces, so text, references and CDATA form one node, as the engine sees them.
• String-valued selectors: a selector that yields a string, number or boolean, such as string(//a) (the engine's documented way to read an element's whole text) or count(//a), is evaluated to that one value. It used to be refused as an invalid selector. A malformed selector still throws InvalidDefinition.

Tests

• PredicateEvaluatorTest:

  • <a>x<b/>y</a> reads x (and not xy);
  • <a><b/>z</a> and <a/> read undefined;
  • text, references and CDATA form one node;
  • text(), @id and string(//a) read their own values.

  Both new tests were red before the fix. The existing XXE and malformed-selector tests still pass.

• XPathRecordedMatchIT (conformance, spawn and embedded lanes, live rift 0.23.0): two XML bodies go through an imposter whose stub matches body == "x" at //a. The engine matches only the first, and recorded(match) agrees, for both x and undefined.

Verification

The core and conformance unit tests pass against the 0.23.0 corpus, and the IT passes on both lanes against the rift 0.23.0 binary and dylib.

rift-scala: no change. It inherits the evaluator through rift-java.

Closes #330

🤖 Generated with Claude Code