Skip to content

Move public API implementation files#273

Merged
adamziel merged 1 commit into
trunkfrom
codex-native-api-file-split
May 16, 2026
Merged

Move public API implementation files#273
adamziel merged 1 commit into
trunkfrom
codex-native-api-file-split

Conversation

@adamziel
Copy link
Copy Markdown
Collaborator

@adamziel adamziel commented May 16, 2026

What it does

Moves the existing public API implementation files to *php* sidecar filenames with no line changes.

This is intentionally only a file-move PR. It does not add public loaders, native wrapper classes, or behavior changes; those stay in the stacked PR #271.

Rationale

This gives reviewers a clean baseline where GitHub shows the large implementation files as renames before #271 adds the native integration and loader changes.

Implementation

Moved these files at 100% similarity:

  • components/HTML/class-wp-html-tag-processor.php -> components/HTML/class-wp-html-php-tag-processor.php
  • components/HTML/class-wp-html-processor.php -> components/HTML/class-wp-html-php-processor.php
  • components/XML/class-xmlprocessor.php -> components/XML/class-phpxmlprocessor.php
  • components/DataLiberation/URL/class-urlintextprocessor.php -> components/DataLiberation/URL/class-phpurlintextprocessor.php

The class names are deliberately unchanged in this PR so the diff remains move-only.

Testing instructions

composer dump-autoload
php -l components/HTML/class-wp-html-php-tag-processor.php components/HTML/class-wp-html-php-processor.php components/XML/class-phpxmlprocessor.php components/DataLiberation/URL/class-phpurlintextprocessor.php
vendor/bin/phpunit components/XML/Tests/
vendor/bin/phpunit components/DataLiberation/Tests/URLInTextProcessorTest.php
vendor/bin/phpunit components/DataLiberation/Tests/MarkupProcessorConsumerTest.php components/DataLiberation/Tests/HTMLEntityReaderTest.php
vendor/bin/phpunit components/Markdown/Tests/MarkdownConsumerTest.php
git diff origin/trunk...HEAD --check

@adamziel adamziel force-pushed the codex-native-api-file-split branch from 615f8e8 to 2da700f Compare May 16, 2026 16:20
@adamziel adamziel changed the title Split public API implementations into sidecars Move public API implementation files May 16, 2026
@adamziel adamziel force-pushed the codex-native-api-file-split branch from 2da700f to 121895f Compare May 16, 2026 16:47
@adamziel adamziel marked this pull request as ready for review May 16, 2026 16:47
@adamziel adamziel merged commit 7c92cf0 into trunk May 16, 2026
29 checks passed
@adamziel adamziel deleted the codex-native-api-file-split branch May 16, 2026 16:47
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