chore: deprecate --ai flag in favor of --integration on specify init

[mnriem]

Description

Implements the deprecation of the legacy --ai flag on specify init in favor of the modern --integration flag, as requested in #2169.

Changes

• Adds a prominent red deprecation warning panel when --ai is used
• Displays the equivalent --integration command
• Handles special case of generic integration with --commands-dir mapping
• Warning is positioned above "Next Steps" for visibility
• The --ai flag continues to function (non-breaking change)

Testing

• Added comprehensive test coverage for the deprecation behavior
• All existing tests pass (1420+ tests)
• Manual CLI verification confirms the warning renders correctly

Fixes

Fixes #2169

[Copilot]

Pull request overview

Deprecates the legacy specify init --ai flag in favor of the newer --integration flow by emitting a prominent Rich warning panel (while keeping --ai functional for now), and adds tests to validate the warning content and placement.

Changes:

• Add helper functions to generate a --integration equivalent and a Rich deprecation warning message for --ai.
• Print a red “Deprecation Warning” panel when --ai is used, positioned before the “Next Steps” panel.
• Add CLI-level tests covering the warning output (including the generic integration --commands-dir mapping).

Show a summary per file

File	Description
src/specify_cli/__init__.py	Builds and prints the --ai deprecation warning and suggested --integration equivalent.
tests/integrations/test_cli.py	Adds tests asserting the deprecation warning content/ordering for --ai usage (including generic mapping).

Copilot's findings

Tip

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

• Files reviewed: 2/2 changed files
• Comments generated: 2

[mnriem]

Review Feedback Addressed

I've addressed both comments from the code review:

1. Reuse existing strip_ansi helper ✅

   • Updated _normalize_cli_output() in tests to use the existing strip_ansi function from tests/conftest.py instead of duplicating the ANSI escape pattern regex
   • This ensures consistency and benefits from the more comprehensive escape-sequence pattern used elsewhere

2. Properly escape paths with spaces ✅

   • Added import shlex to the imports
   • Updated the --integration-options command suggestion to use shlex.quote(ai_commands_dir) when building the replacement command
   • This ensures that paths containing spaces (or special characters) will be correctly parsed by the generic integration's shlex.split() call

The changes maintain the deprecation warning functionality while addressing the technical concerns raised in the review.

[Copilot]

Pull request overview

Deprecates the legacy specify init --ai flag in favor of --integration by emitting a visible deprecation warning while keeping --ai functional for backward compatibility (per #2169).

Changes:

• Add Rich “Deprecation Warning” panel when --ai is used, including the equivalent --integration ... command (with special-case mapping for generic + --ai-commands-dir → --integration-options).
• Ensure the warning is rendered above the “Next Steps” panel.
• Add CLI-level tests verifying the warning content and its placement in output.

Show a summary per file

File	Description
tests/integrations/test_cli.py	Adds normalization helper + new tests asserting --ai deprecation warning content/ordering and generic mapping behavior.
src/specify_cli/__init__.py	Builds and prints a deprecation warning panel for legacy --ai, including an --integration equivalent command string.

Copilot's findings

Tip

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

• Files reviewed: 2/2 changed files
• Comments generated: 0 new