🐛 Fixed
Critical Dependency Validation Fix:
- Fixed
spec-kitty agent workflow implementnot validating WP dependencies before creating workspaces- Bug: WP with single dependency could create workspace without
--baseflag - Impact: Workspace branched from main instead of dependency branch (silent correctness bug)
- Fix: Added shared validation utility that errors when single dependency but no
--baseprovided - Example:
WP06depends onWP04→ command now errors and suggests--base WP04 - Created
src/specify_cli/core/implement_validation.pywithvalidate_and_resolve_base() - Agent commands now delegate to top-level commands (no more legacy script calls)
- Bug: WP with single dependency could create workspace without
Fixed Broken Agent Commands:
- Fixed
spec-kitty agent feature acceptcalling non-existentscripts/tasks/tasks_cli.py- Now delegates to top-level
accept()command
- Now delegates to top-level
- Fixed
spec-kitty agent feature mergecalling non-existentscripts/tasks/tasks_cli.py- Now delegates to top-level
merge()command - Parameter mapping:
keep_branch→delete_branch(inverted logic)
- Now delegates to top-level
Critical Merge Workflow Fix:
- Fixed merge failing when main branch lacks upstream tracking (Issue reported post-0.13.2 release)
- 0.13.2 only checked if remote EXISTS, but not if branch TRACKS it
- Added
has_tracking_branch()function to check upstream tracking - Merge now skips pull if: (1) no remote OR (2) no upstream tracking
- Affects users with local-only repos or repos where main doesn't track origin/main
Testing & Prevention:
- Added 22 new tests for dependency validation and agent command wrappers
- Unit tests:
test_implement_validation.py(11 tests) - Integration tests:
test_agent_command_wrappers.py(11 tests)
- Unit tests:
- Added
TestMigrationRegistryCompletenesstest (prevents 0.13.2-style release blocker)- Verifies all m_*.py migration files are imported in init.py
- Prevents silent bugs where migrations exist but never run
- Added integration tests for merge with untracked branches
- Added unit tests for
has_tracking_branch()function
Documentation:
- Added
src/specify_cli/cli/commands/agent/README.md(wrapper pattern documentation)- Dependency validation best practices
- Parameter mapping guidelines
- Common pitfalls and examples
- Updated RELEASE_CHECKLIST.md with mandatory migration registry verification