dev-cmd/test-bot: make --skip-recursive-dependents default

[cho-m]

---

• Have you followed our Contributing guidelines?
• Have you checked for other open Pull Requests for the same change?
• Have you explained what your changes do? Performance claims (e.g. "this is faster") must include Hyperfine benchmarks.
• Have you explained why you'd like these changes included, not just what they do?
• For bug fixes, have you given step-by-step brew commands to reproduce the bug?
• Have you written new tests (excluding integration tests)? Here's an example.
• Have you successfully run brew lgtm (style, typechecking and tests) locally?

---

• AI was used to generate or assist with generating this PR.

---

This PR makes --skip-recursive-dependents default as we have made indirect linkage into explicit dependencies. Something we've discussed before to reduce time to finish tests.

Added a deprecation period with --no flag in case there are any specific issues that are found either in Homebrew/core or external taps. If everything looks good, we can rip out the recursive dependents logic from test-bot

Partner PR to:

• workflows: skip recursive dependents by default homebrew-core#291338

---

Looking at args.skip_recursive_dependents?.inspect, [no-] switch defaults to nil (though brew typecheck --update doesn't change test_bot_cmd.rbi) so adding nil handling logic.

[Copilot]

Pull request overview

Updates brew test-bot to treat “skip recursive dependents” as the default behavior, reflecting the shift from implicit to explicit dependencies and aiming to reduce overall test-bot runtime. It also introduces a tri-state flag handling path (true/false/nil) to support the new default behavior.

Changes:

• Make --skip-recursive-dependents effectively default-on by treating an unset CLI value (nil) as true.
• Update test-bot CLI option to a --[no-]skip-recursive-dependents switch to allow opting back into recursive dependents during a transition period.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.

File	Description
Library/Homebrew/test_bot/formulae_dependents.rb	Defaults “skip recursive dependents” to enabled when the CLI flag is unset.
Library/Homebrew/dev-cmd/test-bot.rb	Changes the CLI switch to --[no-]skip-recursive-dependents and marks it deprecated.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

[MikeMcQuaid]

Thanks!