Skip to content

[codex] Prepare Push MD for WordPress.org submission#51

Merged
artpi merged 2 commits into
trunkfrom
codex/push-md-wporg-submission
May 20, 2026
Merged

[codex] Prepare Push MD for WordPress.org submission#51
artpi merged 2 commits into
trunkfrom
codex/push-md-wporg-submission

Conversation

@artpi
Copy link
Copy Markdown
Contributor

@artpi artpi commented May 20, 2026

Summary

  • Build Push MD as a readable WordPress.org submission zip with a bundled php-toolkit/ runtime instead of a PHAR.
  • Prune development/test/docs/vendor noise from the built zip and add release checks for forbidden submission files.
  • Rename Push MD-owned globals/hooks to push_md_*, replace the YAML/front matter dependency path with a small parser, and harden Plugin Check findings in the bundled toolkit path.
  • Remove the activation redirect and expose Push MD through an explicit plugin-row action link instead.

Why

This prepares Push MD for WordPress.org review, where the submitted zip must be complete, human-readable, GPL-compatible, and clean under Plugin Check across every bundled PHP file.

Validation

  • vendor/bin/phpunit components/Markdown/Tests/ components/Filesystem/Tests/ components/Git/Tests/
  • vendor/bin/phpunit plugins/push-md/Tests/
  • bash bin/test-push-md-git-actions.sh
  • bash bin/build-plugins.sh
  • bash bin/inspect-push-md-zip.sh dist/plugins/push-md.zip
  • php -l plugins/push-md/push-md.php
  • vendor/bin/phpcs -d memory_limit=1G plugins/push-md/push-md.php
  • composer lint still exits with the existing repo-wide warning baseline outside the touched Push MD submission files.

Reviewer Notes

  • The WordPress.org artifact is generated with bash bin/build-plugins.sh; dist/plugins/push-md.zip contains push-md/ plus readable push-md/php-toolkit/ runtime files.
  • The runtime bundle does not require Composer install on the target WordPress site and does not ship as a PHAR.
  • CommonMark remains bundled for Markdown parsing; the YAML/front matter dependency path was removed for this submission pass.

@artpi artpi marked this pull request as ready for review May 20, 2026 12:01
@artpi artpi self-assigned this May 20, 2026
@github-actions
Copy link
Copy Markdown
Contributor

✅ WordPress Plugin Check Report

✅ Status: Passed

📊 Report

All checks passed! No errors or warnings found.


🤖 Generated by WordPress Plugin Check Action • Learn more about Plugin Check

@artpi artpi merged commit f8aaa3d into trunk May 20, 2026
24 checks passed
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