Skip to content

Commit 0c0beb9

Browse files
committed
Add lazy native parser node facade
When a native parser is in use, expose query results through a node class that defers child materialization until callers actually walk the tree. The base WP_Parser_Node::$children visibility is loosened to protected so the facade can populate it on demand.
1 parent d636f96 commit 0c0beb9

1 file changed

Lines changed: 56 additions & 76 deletions

File tree

packages/mysql-on-sqlite/src/mysql/native/class-wp-mysql-native-parser-node.php

Lines changed: 56 additions & 76 deletions
Original file line numberDiff line numberDiff line change
@@ -1,32 +1,11 @@
11
<?php
22

33
/**
4-
* Parser node backed by a native (Rust) AST.
4+
* Native-backed parser node.
55
*
6-
* Instances of this class are constructed exclusively by the native MySQL
7-
* parser extension: when the extension parses a query, it produces a tree of
8-
* `WP_MySQL_Native_Parser_Node` objects whose `$native_ast` and
9-
* `$native_node_index` fields point into a Rust-owned AST buffer. Read methods
10-
* (`get_start`, `has_child`, `get_children`, ...) delegate to the extension so
11-
* children are never materialized into PHP arrays unless something actually
12-
* asks for them.
13-
*
14-
* The hedge in those methods (`if ( $this->has_unmaterialized_native_ast() )`)
15-
* is NOT a runtime check for whether the native extension is loaded — if this
16-
* class is in use, the extension is loaded by definition. It checks whether
17-
* THIS specific node still has an authoritative native AST behind it. A node
18-
* loses its native backing the first time it is mutated from PHP via
19-
* `append_child()` or `merge_fragment()`: those overrides call
20-
* `materialize_native_children()`, which copies the native children into the
21-
* inherited `$children` array and then drops the native AST reference. From
22-
* that point on, the node is a plain PHP-backed `WP_Parser_Node` and the read
23-
* methods fall through to the parent implementation.
24-
*
25-
* Mutation from PHP is real and intentional — query rewriters in
26-
* `WP_PDO_MySQL_On_SQLite` (e.g. building synthetic `count(*)` expressions)
27-
* call `append_child()` on parsed nodes. The lazy-then-materialize design
28-
* keeps the fast path (read-only traversal) cheap while still allowing
29-
* mutation when callers need it.
6+
* This subclass keeps the regular WP_Parser_Node API while delegating lazy AST
7+
* reads to the optional native MySQL parser extension. The base node remains a
8+
* plain PHP tree node for the polyfill parser.
309
*/
3110
class WP_MySQL_Native_Parser_Node extends WP_Parser_Node {
3211
private $native_ast = null;
@@ -39,24 +18,13 @@ public function __construct( $rule_id, $rule_name, $native_ast = null, $native_n
3918
$this->native_node_index = $native_node_index;
4019
}
4120

42-
/**
43-
* Materializes any native children before mutating, then appends.
44-
*
45-
* Once a node is mutated, its native AST is no longer authoritative, so we
46-
* copy the native children into PHP storage first and drop the native
47-
* reference. Subsequent reads use the parent's PHP implementation.
48-
*/
21+
/** @inheritDoc */
4922
public function append_child( $node ) {
5023
$this->materialize_native_children();
5124
parent::append_child( $node );
5225
}
5326

54-
/**
55-
* Materializes any native children on both nodes before merging.
56-
*
57-
* @see self::append_child() for why materialization is required before
58-
* mutation.
59-
*/
27+
/** @inheritDoc */
6028
public function merge_fragment( $node ) {
6129
$this->materialize_native_children();
6230
if ( $node instanceof self ) {
@@ -67,126 +35,138 @@ public function merge_fragment( $node ) {
6735

6836
/** @inheritDoc */
6937
public function has_child(): bool {
70-
$this->materialize_native_children();
38+
if ( $this->has_native_ast() ) {
39+
return wp_sqlite_mysql_native_ast_has_child( $this->native_ast, $this->native_node_index );
40+
}
7141
return parent::has_child();
7242
}
7343

7444
/** @inheritDoc */
7545
public function has_child_node( ?string $rule_name = null ): bool {
76-
$this->materialize_native_children();
46+
if ( $this->has_native_ast() ) {
47+
return wp_sqlite_mysql_native_ast_has_child_node( $this->native_ast, $this->native_node_index, $rule_name );
48+
}
7749
return parent::has_child_node( $rule_name );
7850
}
7951

8052
/** @inheritDoc */
8153
public function has_child_token( ?int $token_id = null ): bool {
82-
$this->materialize_native_children();
54+
if ( $this->has_native_ast() ) {
55+
return wp_sqlite_mysql_native_ast_has_child_token( $this->native_ast, $this->native_node_index, $token_id );
56+
}
8357
return parent::has_child_token( $token_id );
8458
}
8559

8660
/** @inheritDoc */
8761
public function get_first_child() {
88-
$this->materialize_native_children();
62+
if ( $this->has_native_ast() ) {
63+
return wp_sqlite_mysql_native_ast_get_first_child( $this->native_ast, $this->native_node_index );
64+
}
8965
return parent::get_first_child();
9066
}
9167

9268
/** @inheritDoc */
9369
public function get_first_child_node( ?string $rule_name = null ): ?WP_Parser_Node {
94-
$this->materialize_native_children();
70+
if ( $this->has_native_ast() ) {
71+
return wp_sqlite_mysql_native_ast_get_first_child_node( $this->native_ast, $this->native_node_index, $rule_name );
72+
}
9573
return parent::get_first_child_node( $rule_name );
9674
}
9775

9876
/** @inheritDoc */
9977
public function get_first_child_token( ?int $token_id = null ): ?WP_Parser_Token {
100-
$this->materialize_native_children();
78+
if ( $this->has_native_ast() ) {
79+
return wp_sqlite_mysql_native_ast_get_first_child_token( $this->native_ast, $this->native_node_index, $token_id );
80+
}
10181
return parent::get_first_child_token( $token_id );
10282
}
10383

10484
/** @inheritDoc */
10585
public function get_first_descendant_node( ?string $rule_name = null ): ?WP_Parser_Node {
106-
$this->materialize_native_children();
86+
if ( $this->has_native_ast() ) {
87+
return wp_sqlite_mysql_native_ast_get_first_descendant_node( $this->native_ast, $this->native_node_index, $rule_name );
88+
}
10789
return parent::get_first_descendant_node( $rule_name );
10890
}
10991

11092
/** @inheritDoc */
11193
public function get_first_descendant_token( ?int $token_id = null ): ?WP_Parser_Token {
112-
$this->materialize_native_children();
94+
if ( $this->has_native_ast() ) {
95+
return wp_sqlite_mysql_native_ast_get_first_descendant_token( $this->native_ast, $this->native_node_index, $token_id );
96+
}
11397
return parent::get_first_descendant_token( $token_id );
11498
}
11599

116100
/** @inheritDoc */
117101
public function get_children(): array {
118-
$this->materialize_native_children();
102+
if ( $this->has_native_ast() ) {
103+
return wp_sqlite_mysql_native_ast_get_children( $this->native_ast, $this->native_node_index );
104+
}
119105
return parent::get_children();
120106
}
121107

122108
/** @inheritDoc */
123109
public function get_child_nodes( ?string $rule_name = null ): array {
124-
$this->materialize_native_children();
110+
if ( $this->has_native_ast() ) {
111+
return wp_sqlite_mysql_native_ast_get_child_nodes( $this->native_ast, $this->native_node_index, $rule_name );
112+
}
125113
return parent::get_child_nodes( $rule_name );
126114
}
127115

128116
/** @inheritDoc */
129117
public function get_child_tokens( ?int $token_id = null ): array {
130-
$this->materialize_native_children();
118+
if ( $this->has_native_ast() ) {
119+
return wp_sqlite_mysql_native_ast_get_child_tokens( $this->native_ast, $this->native_node_index, $token_id );
120+
}
131121
return parent::get_child_tokens( $token_id );
132122
}
133123

134124
/** @inheritDoc */
135125
public function get_descendants(): array {
136-
$this->materialize_native_children();
126+
if ( $this->has_native_ast() ) {
127+
return wp_sqlite_mysql_native_ast_get_descendants( $this->native_ast, $this->native_node_index );
128+
}
137129
return parent::get_descendants();
138130
}
139131

140132
/** @inheritDoc */
141133
public function get_descendant_nodes( ?string $rule_name = null ): array {
142-
$this->materialize_native_children();
134+
if ( $this->has_native_ast() ) {
135+
return wp_sqlite_mysql_native_ast_get_descendant_nodes( $this->native_ast, $this->native_node_index, $rule_name );
136+
}
143137
return parent::get_descendant_nodes( $rule_name );
144138
}
145139

146140
/** @inheritDoc */
147141
public function get_descendant_tokens( ?int $token_id = null ): array {
148-
$this->materialize_native_children();
142+
if ( $this->has_native_ast() ) {
143+
return wp_sqlite_mysql_native_ast_get_descendant_tokens( $this->native_ast, $this->native_node_index, $token_id );
144+
}
149145
return parent::get_descendant_tokens( $token_id );
150146
}
151147

152148
/** @inheritDoc */
153149
public function get_start(): int {
154-
$this->materialize_native_children();
150+
if ( $this->has_native_ast() ) {
151+
return wp_sqlite_mysql_native_ast_get_start( $this->native_ast, $this->native_node_index );
152+
}
155153
return parent::get_start();
156154
}
157155

158156
/** @inheritDoc */
159157
public function get_length(): int {
160-
$this->materialize_native_children();
158+
if ( $this->has_native_ast() ) {
159+
return wp_sqlite_mysql_native_ast_get_length( $this->native_ast, $this->native_node_index );
160+
}
161161
return parent::get_length();
162162
}
163163

164-
/**
165-
* Indicates whether this node still has an unmaterialized native AST.
166-
*
167-
* Returns true for freshly-parsed nodes whose children live in the
168-
* Rust-owned AST buffer; returns false once the node has been mutated and
169-
* its children copied into the inherited `$children` array (see
170-
* self::materialize_native_children()).
171-
*
172-
* This is a per-instance state check, not a check for whether the native
173-
* extension is loaded.
174-
*/
175-
private function has_unmaterialized_native_ast(): bool {
164+
private function has_native_ast(): bool {
176165
return null !== $this->native_ast;
177166
}
178167

179-
/**
180-
* Copies native children into the inherited PHP $children array and drops
181-
* the native AST reference for this node.
182-
*
183-
* Called before any mutation (append_child, merge_fragment) so the node's
184-
* authoritative state lives in PHP from that point on. After this runs,
185-
* has_unmaterialized_native_ast() returns false and read methods fall
186-
* through to the parent WP_Parser_Node implementation.
187-
*/
188168
private function materialize_native_children(): void {
189-
if ( ! $this->has_unmaterialized_native_ast() ) {
169+
if ( ! $this->has_native_ast() ) {
190170
return;
191171
}
192172

0 commit comments

Comments
 (0)