Skip to content

Conversation

@pushpak1300
Copy link
Member

This update introduces first class Sail support across the guidelines.
When a project installs Boost with Sail enabled, agent instructions now automatically switch to Sail specific commands rather than native CLI commands.

Example guidelines

Before
- Run the minimum number of tests needed. Use php artisan test with a filename or filter.

After (Sail selected during boost:install)
- Run the minimum number of tests needed. Use vendor/bin/sail artisan test with a filename or filter.

Alongside these dynamic command replacements, this MR adds dedicated Sail usage guidance so LLM agents can reliably work inside Sail environments.

## Laravel Sail

- This project runs inside Laravel Sail's Docker containers. You must run commands through Sail.
- Start services with `vendor/bin/sail up -d` and stop them using `vendor/bin/sail stop`.
- Open the app in a browser with `vendor/bin/sail open`.
- Prefix PHP, Artisan, Composer and Node commands with `vendor/bin/sail`. For example:
  - `vendor/bin/sail artisan migrate`
  - `vendor/bin/sail composer install`
  - `vendor/bin/sail npm run dev`
  - `vendor/bin/sail php script.php`
- View all Sail commands by running `vendor/bin/sail` with no arguments.

Signed-off-by: Pushpak Chhajed <[email protected]>
Signed-off-by: Pushpak Chhajed <[email protected]>
@pushpak1300 pushpak1300 marked this pull request as draft November 4, 2025 13:44
Signed-off-by: Pushpak Chhajed <[email protected]>
Signed-off-by: Pushpak Chhajed <[email protected]>
@pushpak1300 pushpak1300 marked this pull request as ready for review November 4, 2025 17:16
pushpak1300 and others added 4 commits November 4, 2025 22:46
Signed-off-by: Pushpak Chhajed <[email protected]>

# Conflicts:
#	all.php
#	src/Console/InstallCommand.php
Signed-off-by: Pushpak Chhajed <[email protected]>
Signed-off-by: Pushpak Chhajed <[email protected]>
Copy link
Contributor

@joetannenbaum joetannenbaum left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Close, perhaps a bit more tweaking based on my comments. Let me know what you think.

pushpak1300 and others added 11 commits November 7, 2025 18:22
Signed-off-by: Pushpak Chhajed <[email protected]>
Signed-off-by: Pushpak Chhajed <[email protected]>
Signed-off-by: Pushpak Chhajed <[email protected]>
Signed-off-by: Pushpak Chhajed <[email protected]>
Signed-off-by: Pushpak Chhajed <[email protected]>
Signed-off-by: Pushpak Chhajed <[email protected]>
Signed-off-by: Pushpak Chhajed <[email protected]>
Signed-off-by: Pushpak Chhajed <[email protected]>
Copy link
Contributor

@joetannenbaum joetannenbaum left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, did some minor refactoring in a couple of places, let me know you object to any of the changes.

Let's try to internalize some of these smaller detail-oriented nits for future PRs:

Image

@pushpak1300 pushpak1300 merged commit 7e676d7 into main Nov 14, 2025
18 checks passed
@pushpak1300 pushpak1300 deleted the sail_guidline_support branch November 14, 2025 06:21
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.

3 participants