Skip to content

feat: support PHP-only app releases - #10

Merged
ddespinoza merged 3 commits into
mainfrom
fix/184-php-app-packaging
Oct 31, 2026
Merged

ddespinoza merged 3 commits into
mainfrom
fix/184-php-app-packaging

Conversation

@ddespinoza

Copy link
Copy Markdown
Contributor

Summary

Closes #9. Required by APS-Conecta/gestion#239, which blocks APS-Conecta/gestion#184.

pack app
  default: require the built browser bundle
  php-only: require runtime PHP; refuse frontend manifests and bundles
  archive → checksums → existing signing and verification

Adds an explicit php-only workflow input, disabled by default. Server apps supply their PHP check through the existing build input.

Evidence

  • Before: the extended existing packer self-test reports 31 checks, 3 failures.
    After: 31 checks, 0 failures, including PHP-only packaging and frontend refusal cases.
  • ShellCheck, actionlint, zizmor and the community checker pass. The existing browser-app fixture runs in CI.
  • Standards and Spec reviews: no findings; final delta makes the same refusal condition explicit for ShellCheck.

Merge Danger

Door: two-way

Blast Radius: publishing

Existing callers keep the default browser-bundle requirement. Artifact identity, deterministic packaging and signature checks retain their existing behavior.

root added 3 commits October 31, 2026 04:11
Keep existing frontend bundle checks enabled by default. A server app must opt in, contain runtime PHP classes, and have no frontend manifest or bundle.

Assisted-by: codex:GPT-6
@ddespinoza
ddespinoza marked this pull request as ready for review October 31, 2026 04:20
@ddespinoza
ddespinoza merged commit b2d2739 into main Oct 31, 2026
4 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.

Publicar aplicaciones PHP sin un paquete JavaScript ficticio

1 participant