Skip to content

[codex] Prepare showcase for senior Symfony review#3

Open
roadynet wants to merge 4 commits into
mainfrom
codex/showcase-safety-audit
Open

[codex] Prepare showcase for senior Symfony review#3
roadynet wants to merge 4 commits into
mainfrom
codex/showcase-safety-audit

Conversation

@roadynet

@roadynet roadynet commented Jun 15, 2026

Copy link
Copy Markdown
Owner

Summary

  • Add Composer-based reproducible verification with composer verify and a separate composer smoke:public live check.
  • Add a repeatable tests/showcase_audit.py public-safety audit for env-file policy, obvious secret patterns, private-code path leakage, local Markdown links, screenshots, wording consistency, and phpunit.xml parsing.
  • Run the Portfolio Audit workflow through Composer so local review and CI use the same entrypoint.
  • Add PHPStan level: max for the public PHP examples and lock the toolchain through composer.lock.
  • Add a Senior Symfony review guide, ADRs, and a security/privacy boundary document.
  • Add PHPUnit coverage for the public next-due-question example so the showcase demonstrates testable business rules beyond scheduling and parsing.
  • Document public live readiness evidence for the canonical public demo routes and security headers.

Validation

  • Fresh temp-copy CI simulation:
    • composer validate --strict --no-check-publish
    • composer install --no-interaction --prefer-dist --no-progress
    • composer verify
    • composer smoke:public
  • composer verify covers PHP syntax lint, PHPStan, PHPUnit, and public showcase audit.
  • PHPUnit result: 7 tests, 10 assertions, OK.
  • PHPStan result: no errors.
  • Public live smoke: /health, /login, /register, /impressum, /dsgvo all 200 with CSP, nosniff, and frame protection.
  • git diff --check passed.

Notes

  • No demo credentials are published.
  • .env.example remains placeholder-only.
  • The private application repository remains separate from this public showcase.
  • Required CI intentionally does not call the live smoke command because production/network availability should not block documentation-only PRs.

@roadynet roadynet changed the title [codex] Harden public showcase safety checks [codex] Prepare showcase for senior Symfony review Jun 15, 2026
@roadynet roadynet marked this pull request as ready for review July 2, 2026 04:42
@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, you can upgrade your account or add credits to your account and enable them for code reviews in your settings.

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