fix: ensure idempotent project-relative path rewriting in CommandRegistrar

[darion-yaphet]

Description

Refactors CommandRegistrar.rewrite_project_relative_paths() from multi-step string replacements and sequential regex passes into a unified single-pass regex callback.

Key improvements:

1. Single-pass regex callback: Replaces the fragile pattern of three sequential string replacements followed by three re.sub passes and trailing .replace(".specify/.specify/", ".specify/") cleanup with a clean, unified regex match callback.
2. Deterministic routing: Consistently routes deeper parent-relative references (../../, ../../../) to root .specify/<target>/ while preserving extension-local scoping for top-level scripts when extension_id is supplied.
3. Expanded delimiter & assignment support: Recognizes Markdown punctuation enclosures ([], (), {}, <>), quotes, backticks, and option/variable assignment delimiters (=, e.g., --template=../../templates/spec.md and SCRIPT=../../scripts/bash/run.sh).
4. Idempotency guard: Naturally guards already-normalized .specify/ paths from duplicate prefixing across repeated passes without needing post-hoc string stripping.

Testing

• Ran existing tests with pytest tests/test_extensions.py (542 passed)
• Ran idempotency, delimiter, and assignment test suite with pytest tests/test_extensions.py -k "test_rewrite_project_relative_paths" (5 passed)
• Ran linter check with ruff check src/specify_cli/agents.py tests/test_extensions.py (0 errors)

AI Disclosure

• I did not use AI assistance for this contribution
• I did use AI assistance (fill in the disclosure below)

AI disclosure: Implemented with Google Antigravity using Gemini 3.8 Flash (High) in human-supervised mode for code refactoring, test case generation, and addressing review feedback.

[mnriem]

The delimiter and deeper-parent-path improvements have useful regression coverage. Before merge, please preserve parent-relative paths following =: --template=../../templates/spec.md and SCRIPT=../../scripts/bash/run.sh were rewritten previously but are now left unchanged. Add regression cases for these alongside the existing idempotency checks.

Please also clarify the description: the supplied samples already remain stable under repeated rewriting on the old code; the demonstrated improvements are routing and delimiter handling. Finally, complete this PR’s AI disclosure with the tool, model, mode/settings, and extent of assistance—the checkbox alone does not describe the contribution.

Drafted for @mnriem by GitHub Copilot (model: GPT-6 Astra; comment drafting).

[Copilot]

🟢 Approval recommended

The refactor preserves routing behavior and includes focused regression coverage.

Pull request overview

Refactors project-relative path rewriting into an idempotent, single-pass implementation.

Changes:

• Consolidates path normalization into one regex callback.
• Adds regression coverage for idempotency, delimiters, and invalid inputs.

File summaries

File	Description
src/specify_cli/agents.py	Implements unified path rewriting.
tests/test_extensions.py	Adds comprehensive regression tests.

Review details

• Files reviewed: 2/2 changed files
• Comments generated: 0
• Review effort level: Balanced (auto)

Note

Copilot is running an experiment and ran this review at Balanced.

---

💡 Configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

[darion-yaphet]

Thanks for the thorough review and catching the = assignment case!

I have addressed the feedback in commit f6ecaa9:

1. Preserved = prefix: Added = to the boundary delimiter class ((^|[\s"'([{<=])), ensuring CLI option flags (like --template=../../templates/spec.md) and environment variable assignments (like SCRIPT=../../scripts/bash/run.sh`) are properly rewritten.
2. Added regression coverage: Added test cases for = followed by parent-relative and extension-scoped paths in tests/test_extensions.py, asserting correct transformation and idempotency across repeated passes.
3. Clarified PR description: Clarified in the description that the improvements focus on the single-pass regex architecture, deeper parent-path routing, and expanded delimiter/assignment handling.
4. Completed AI disclosure: Updated the PR body with the full AI disclosure (tool, model, supervised mode, and extent of assistance).

Drafted on behalf of @darion-yaphet by Google Antigravity (model: Gemini 3.8 Flash (High), human-supervised; comment drafting and code refinement).