Skip to content

refactor(cli): remove migrated Organon commands - #629

Merged
birdmanmandbir merged 5 commits into
mainfrom
refactor/remove-project-list
Jul 17, 2026
Merged

refactor(cli): remove migrated Organon commands#629
birdmanmandbir merged 5 commits into
mainfrom
refactor/remove-project-list

Conversation

@birdmanmandbir

@birdmanmandbir birdmanmandbir commented Jul 15, 2026

Copy link
Copy Markdown
Contributor

Summary

  • remove all duplicated TTAL project commands, including ttal project resolve --json
  • remove all TTAL user Git commands: push, tag, pull, and pr
  • remove unused daemon routes for push, tag, pull, PR creation/modification, and CI inspection
  • retain only task-lifecycle PR lookup, validation, merge, and mergeability operations
  • update documentation, worker prompts, agent templates, and runtime guidance to use project, og git, and og pr

Verification

  • go test ./...
  • go build ./...
  • verified ttal project is rejected and standalone project remains the project interface

make ci remains blocked by 51 existing lint findings outside this change.

@birdmanmandbir birdmanmandbir changed the title refactor(project): remove migrated project commands refactor(cli): remove migrated Organon commands Jul 15, 2026
@birdmanmandbir

Copy link
Copy Markdown
Contributor Author

Triage Update

Fixed

  • Development run example used task get — changed it to the TTAL-native agent list in Makefile and CI_CD_SETUP.md (43fb13d).

False Positive

  • cmd/review_context.go has no callers — cmd/comment_new.go calls loadConfigAndCoderRuntime (line 371) and writeReviewFile (line 406). The helpers were moved from deleted cmd/pr.go because comment notifications still need them. They are package-private, not exported.
  • parseModifyArgs changed = to : — the helper was moved verbatim from cmd/project.go in origin/main; that implementation already accepted only field:value. cmd/agent_test.go covers the retained syntax.
  • Imports cause side effects when only writeReviewFile is needed — Go imports are compiled per package; config.Load() is invoked only by loadConfigAndCoderRuntime, not at import time. Splitting this small, cohesive shared helper file has no practical benefit.

Remaining

  • No code changes remain for the false-positive findings. Verified with go test ./cmd, go build ./..., and make help.

@birdmanmandbir
birdmanmandbir merged commit aa0a023 into main Jul 17, 2026
6 checks passed
@birdmanmandbir
birdmanmandbir deleted the refactor/remove-project-list branch July 17, 2026 09:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant