But CLI: Use modern apply for top-level but apply

[krlvi]

Summary

• route top-level but apply through command::branch::apply
• make the modern apply command available in legacy builds while preserving shell output
• remove the unused legacy apply implementation and module export

Verification

• cargo test -p but apply
• cargo fmt --check --all
• cargo clippy -p but --all-targets

[Copilot]

Pull request overview

This PR routes the legacy-only top-level but apply command through the modern command::branch::apply implementation, aligning behavior across builds while removing the now-redundant legacy apply code.

Changes:

• Switch legacy top-level but apply to call command::branch::apply (modern implementation).
• Make the modern apply implementation available under feature = "legacy" by removing legacy-only gating in command::branch.
• Remove the unused legacy apply module and its export.

Reviewed changes

Copilot reviewed 6 out of 7 changed files in this pull request and generated no comments.

Show a summary per file

File	Description
crates/but/src/lib.rs	Routes top-level legacy Subcommands::Apply through command::branch::apply using setup::init_ctx.
crates/but/src/command/legacy/branch/mod.rs	Removes the legacy apply module export.
crates/but/src/command/legacy/branch/apply.rs	Deletes the legacy apply implementation.
crates/but/src/command/branch/mod.rs	Exposes apply in legacy builds by removing cfg(not(feature = "legacy")) gating.
crates/but/src/command/branch/apply.rs	Adjusts shell output to emit the full reference name (matching legacy shell output expectations).

[chatgpt-codex-connector]

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 2472d7fe21

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

[chatgpt-codex-connector]

Preserve legacy branch-name resolution for top-level apply

In legacy builds, routing Subcommands::Apply through command::branch::apply drops the old command::legacy::branch::apply::resolve_reference_to_apply behavior, which handled non-exact inputs (for example resolving remote-only names and interactive/auto selection for partial matches). The new path relies on repo.find_reference(branch_name) exact resolution, so previously valid but apply <name> inputs now fail with “reference did not exist.” This is a compatibility regression in the legacy command path.

Useful? React with 👍 / 👎.

[estib-vega]

[slarse]

Nice!