Skip to content

Add pull-files and pull-db commands#279

Draft
adamziel wants to merge 13 commits into
trunkfrom
adamziel/pull-files-db
Draft

Add pull-files and pull-db commands#279
adamziel wants to merge 13 commits into
trunkfrom
adamziel/pull-files-db

Conversation

@adamziel

Copy link
Copy Markdown
Collaborator

Summary

Adds two higher-level commands:

  • pull-files, which runs the same retrying file-download stage used by pull
  • pull-db, which runs the same retrying database-download stage used by pull

The full pull pipeline now delegates its file and database stages through those higher-level methods instead of calling the lower-level files-pull / db-pull handlers directly. Command help and option grouping include the new commands.

Tests

  • PATH="$PWD/.context/test-bin:$PATH" php -n vendor/bin/phpunit --configuration tests/phpunit.xml tests/Import/PullFilterOptionTest.php tests/Import/PullStartModeTest.php tests/Import/LegacyCommandAliasTest.php tests/Import/OnlyCliParseTest.php --testdox --colors=never
  • php -n -l packages/reprint-importer/src/import.php
  • php -n -l packages/reprint-importer/src/lib/pull/class-pull.php
  • php -n -l tests/Import/PullFilterOptionTest.php
  • PATH="$PWD/.context/test-bin:$PATH" php -n /Users/cloudnik/.local/bin/composer analyze

Notes

  • composer test is not clean in this workspace because the MySQL-backed tests cannot connect to 127.0.0.1 (SQLSTATE[HY000] [2002] Connection refused).
  • composer lint:php still reports existing PHPCS issues across the repository unrelated to this change.

@github-actions

github-actions Bot commented Jun 26, 2026

Copy link
Copy Markdown
Contributor

Pull pipeline performance — large-directory

Site: large-directory · 2,000+ plus targeted file-transfer scenarios files · 10,000 posts · 25,000 postmeta · PHP 8.5.7

trunk baseline unavailable — showing PR numbers only.

Stage Wall time Resume attempts Status Details
playground-sqlite-db-download 7.45 s 1 condition=db-download in PHP.wasm
runtime=php.wasm 8.3
wp_mysql_parser=enabled
mode=lexer
native_lexer=verified
native_token_stream=WP_MySQL_Native_Token_Stream
native_token_count=18
native_parser=selected
playground-sqlite-db-apply 2.82 s 1 condition=db-apply to SQLite in PHP.wasm
runtime=php.wasm 8.3
wp_mysql_parser=enabled
mode=parser
native_lexer=verified
native_token_stream=WP_MySQL_Native_Token_Stream
native_token_count=18
native_parser=verified
native_ast=WP_MySQL_Native_Parser_Node
sqlite_driver_parser=verified
Total 10.27 s

Numbers carry runner noise; treat single-run deltas as directional, not authoritative.

📈 Trunk performance history — commit-by-commit timeline.

@adamziel adamziel force-pushed the adamziel/pull-files-db branch from ebc0676 to 2aa5410 Compare June 26, 2026 14:45
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