Fix tenants:seed command name

[lucasmichot]

Fixes #1474.

Replaces the version_compare check in Seed::__construct() with a configure() override that calls setName('tenants:seed'). configure() always runs after the command name is set by Symfony's Command constructor, regardless of whether the installed Laravel version's SeedCommand sets its name via $name (pre-13.24) or $signature (13.24+), so this works across versions without checking app()->version().

configure() is also the general place to add, remove, or tweak a parent command's arguments/options ($this->addArgument(), $this->addOption(), or $this->getDefinition()->remove*()), since it's guaranteed to run after the parent's own definition is built and before the command executes. Worth keeping in mind for other commands here that extend Laravel's built-in ones and need to adjust their inherited signature.

Summary by CodeRabbit

• Bug Fixes

  • Fixed tenant database seeding command registration so it no longer replaces Laravel’s built-in database seeding command.
  • Preserved separate access to both tenant seeding and standard database seeding commands.

• Tests

  • Added coverage confirming each seeding command resolves and operates independently.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: 17341ece-7cd0-4fa7-8ac7-2c385bac7675

📥 Commits

Reviewing files that changed from the base of the PR and between e0990a4 and c747612.

📒 Files selected for processing (2)

• src/Commands/Seed.php
• tests/CommandsTest.php

---

📝 Walkthrough

Walkthrough

Seed now preserves inherited Laravel seed command configuration, adds tenant options only when needed, and explicitly registers the command as tenants:seed. Tests verify that db:seed and tenants:seed remain separately registered.

Changes

Tenant seed command

Layer / File(s)	Summary
Command initialization and naming src/Commands/Seed.php	Seed initializes SeedCommand, conditionally adds tenant options, preserves inherited configuration, and sets the command name to tenants:seed.
Command registration validation tests/CommandsTest.php	Tests verify that db:seed uses Laravel’s SeedCommand and tenants:seed uses the tenancy Seed command.

Estimated code review effort: 2 (Simple) | ~10 minutes

Sequence Diagram(s)

sequenceDiagram
  participant Artisan
  participant SeedCommand
  participant TenantSeed
  Artisan->>SeedCommand: register db:seed
  Artisan->>TenantSeed: configure tenant options
  TenantSeed->>TenantSeed: set name to tenants:seed
  Artisan-->>Artisan: retain both command registrations

Loading

Possibly related PRs

• archtechx/tenancy#1476: Both PRs modify src/Commands/Seed.php for Laravel command initialization and naming.
• archtechx/tenancy#1477: Both PRs preserve distinct tenants:seed and db:seed registrations.

Suggested reviewers: stancl, lukinovec

Poem

A rabbit checks the seed command’s name,
Laravel’s db:seed stays the same.
Tenant options join the queue,
tenants:seed comes through.
Two commands hop without a fight. 🐇

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly identifies the primary change: fixing the tenants:seed command name.
Linked Issues check	✅ Passed	The changes set tenants:seed explicitly and test that Laravel's db:seed command remains intact, meeting issue #1474.
Out of Scope Changes check	✅ Passed	The implementation and regression test directly address the linked issue without unrelated code changes.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 86.64%. Comparing base (e0990a4) to head (c747612).

Additional details and impacted files

@@             Coverage Diff              @@
##             master    #1478      +/-   ##
============================================
- Coverage     86.65%   86.64%   -0.01%     
- Complexity     1220     1221       +1     
============================================
  Files           186      186              
  Lines          3589     3588       -1     
============================================
- Hits           3110     3109       -1     
  Misses          479      479              

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[coderabbitai]

Note

GitHub couldn't provide a complete incremental comparison for this pull request, so CodeRabbit is performing a full review instead. This review may take a little longer.