Make .distignore the single source of truth for the release ZIP

[erseco]

Why

.distignore and .gitattributes listed roughly the same paths — 41 rules in common, and two duplicated verbatim inside .gitattributes (README.md, phpunit.xml.dist) — even though they have different jobs:

• .distignore is the only list wp dist-archive reads (vendor/wp-cli/dist-archive-command/src/Dist_Archive_Command.php:77-84), so it alone decides what ships.
• .gitattributes export-ignore only shapes the source ZIP GitHub serves at archive/refs/heads/*.zip — the URL blueprint.json installs in Playground.

Maintaining them as mirrors is wrong in both directions and hid two bugs.

Bugs fixed

1. Unanchored rules were eating the bundled editor. wp dist-archive delegates matching to inmarelibero/gitignore-checker, which compiles every rule with the regex i flag and matches at any depth. So the bare build, README.md, CHANGELOG.md and package.json rules reached inside dist/static/ and silently stripped 16 files from the packaged editor. The LICENSE.txt rule already carried a comment about this exact trap; now every root-only rule is anchored.

2. test-results/ was shipping — Playwright output, gitignored, but dist-archive reads the working tree.

Verification

Built the ZIP before and after and diffed the file lists (3910 → 3926 entries):

> exelearning/LICENSE.txt
> exelearning/dist/static/libs/yjs/build/            (7 files)
> exelearning/dist/static/CHANGELOG.md
> exelearning/dist/static/libs/README.md
> exelearning/dist/static/libs/tinymce_5/CHANGELOG.md
> exelearning/dist/static/libs/tinymce_5/js/tinymce/langs/README.md
> exelearning/dist/static/app/common/mindmaps/README.md
> exelearning/dist/static/app/common/edicuatex/README.md
> exelearning/dist/static/app/common/fix_webm_duration/README.md
> exelearning/dist/static/app/common/fix_webm_duration/package.json
> exelearning/dist/static/files/perm/idevices/base/lomloe/README.md
< exelearning/test-results/
< exelearning/test-results/.last-run.json

Nothing else changed. make package itself is untouched.

Also in this PR

• .distignore loses ~35 rules inherited from WordPress/plugin-check that match nothing here (plugin-check.iml, behat.yml, .gherkin-lintrc, phpstan.neon, /phpcs-sniffs/…), and keeps a deliberate block of rules that must match at any depth (.git, node_modules, .DS_Store, .idea, sftp-config.json).
• LICENSE.txt now ships; it did not before.
• .gitattributes goes from 45 rules to 6. Rules for untracked paths (node_modules/, vendor/, dist/, artifacts/) were noise — git archive never sees them.
• Both files now state their scope in a header comment, so nobody re-syncs them by hand.

Context

wp-decker, wp-documentate and wp-autofirma carry the same two drifting files, and two of them ship a ZIP with no top-level directory (WordPress then installs them as decker-1.2.3/). ADR-0003 records the decision and SDD-0002 the cross-repository design; this is the first of four PRs and the only one that does not change the packaging command.

Follow-up (not here)

ci.yml:127-128 uses git checkout-index instead of git archive only because the blanket .* rule hid .phpcs.xml.dist. That rule is gone, so the workaround can be revisited — left out to keep this PR to packaging.

---

Añadido tras revisar la CI

check-editor-releases.yml no instalaba PHP ni Composer y sin embargo llama a make package, que necesita el comando i18n de WP-CLI para las traducciones de runtime y dist-archive para el ZIP, ambos en require-dev. El paso Create package fallaba en cuanto salía una versión nueva del editor. Es un fallo preexistente, no lo introduce esta PR; los pasos son los mismos que ya usa release.yml.

Comentario obsoleto en el guard de PHPCS. Decía que se usaba checkout-index porque .gitattributes excluía los dotfiles, incluido .phpcs.xml.dist — dejó de ser cierto al desaparecer la regla .*. checkout-index sigue siendo lo correcto, pero por una razón mejor: escribe el índice tal cual, sin filtrar por export-ignore, así que el guard no depende de lo que .gitattributes excluya en cada momento.

rm -f antes de archivar, porque --force no vacía el ZIP anterior (ver el commit correspondiente).

[github-actions]

Test in WordPress Playground

Test the plugin with the code from this branch:

ℹ️ The eXeLearning editor is fetched from the shared release and unpacked into the plugin when the playground boots, so the first load may take a few extra seconds. ELP upload, shortcode, Gutenberg block and preview work normally.

[codecov-commenter]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 73.47%. Comparing base (2d9fcb7) to head (87b13bd).

Additional details and impacted files

@@            Coverage Diff            @@
##               main      #86   +/-   ##
=========================================
  Coverage     73.47%   73.47%           
  Complexity      848      848           
=========================================
  Files            42       42           
  Lines          4339     4339           
=========================================
  Hits           3188     3188           
  Misses         1151     1151           

Flag	Coverage Δ
javascript	37.81% <ø> (ø)
php	85.39% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.