Skip to content

Align Runway generators with skeleton App\ layout - #714

Merged
n0nag0n merged 1 commit into
masterfrom
feat/align-runway-commands-with-skeleton
Aug 7, 2026
Merged

Align Runway generators with skeleton App\ layout#714
n0nag0n merged 1 commit into
masterfrom
feat/align-runway-commands-with-skeleton

Conversation

@n0nag0n

@n0nag0n n0nag0n commented Aug 7, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • make:controller: generates App\Controller\… under app/Controller/ (matches flightphp/skeleton layout) instead of app\controllers.
  • ai:generate-instructions: writes only root AGENTS.md (no Copilot/Cursor/Gemini/Windsurf dumps); prefers existing AGENTS.md for context (falls back to legacy copilot file); defaults templating to Twig; LLM prompt encodes App\…, Engine injection, SimplePdo, and AGENTS.md-only conventions.
  • composer.json suggest: recommend Twig (keep Latte as optional).
  • CI/local gates: clear stale PHPStan baseline ignore for parseStringClassAndMethod; fix PHPCS issues (string concat, Loader @template order, SimplePdo cast spacing, test heredocs) so tests/run_all_tests.sh is green on PHP 7.4 / 8.2 / 8.3.

Test plan

  • bash tests/run_all_tests.sh (PHPUnit + PHPStan + PHPCS on available PHP versions)
  • Unit tests for ControllerCommand and AiGenerateInstructionsCommand
  • Manually: php runway make:controller Foo in a skeleton-style app → file at app/Controller/FooController.php with namespace App\Controller
  • Manually: php runway ai:generate-instructions → only updates AGENTS.md

make:controller now generates App\Controller under app/Controller/, and
ai:generate-instructions writes only AGENTS.md with Twig/SimplePdo
conventions. Clear a stale PHPStan baseline entry and resolve PHPCS
issues so multi-PHP tests/phpstan/phpcs pass cleanly.
@n0nag0n
n0nag0n merged commit ee03274 into master Aug 7, 2026
21 checks passed
@n0nag0n
n0nag0n deleted the feat/align-runway-commands-with-skeleton branch August 7, 2026 13:44
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