Skip to content

feat: add ccx move CLI subcommand for moving session project paths#59

Merged
gavin-jeong merged 2 commits into
masterfrom
feat-move-session
Jul 9, 2026
Merged

feat: add ccx move CLI subcommand for moving session project paths#59
gavin-jeong merged 2 commits into
masterfrom
feat-move-session

Conversation

@zzJinux

@zzJinux zzJinux commented Jul 7, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Expose session.MoveProject (already used by the TUI's m keybind) as a new ccx move CLI subcommand.
  • Default: moves the current tmux window's detected session's project path.
  • --session <id>: target a specific session by ID prefix.
  • --from <dir>: target a project directory directly by path (moves every session under it), without needing a session ID.

Test plan

  • go build ./... passes
  • ccx move <new-path> moves current tmux-detected session's project dir
  • ccx move --session <id> <new-path> moves by session ID
  • ccx move --from <old-dir> <new-path> moves by project dir path
  • ccx help shows updated usage

Expose session.MoveProject (already used by the TUI's `m` keybind) as a
CLI subcommand. Supports moving the current tmux-detected session, a
session by ID, or a project directory directly by path.

Claude-Session: LOCAL_992a1351-22c6-490b-a7b9-3518e4a2bcdb
@zzJinux zzJinux self-assigned this Jul 7, 2026
@zzJinux zzJinux requested a review from gavin-jeong July 7, 2026 18:18
@gavin-jeong gavin-jeong added the auto-review/pending Auto-review waiting for Atlantis plan / ArgoCD diff label Jul 7, 2026
@gavin-jeong

Copy link
Copy Markdown
Collaborator

🤖 Auto-review on hold — PR too young (224s), waiting. The PR review watcher will re-check automatically.

Add per-flag descriptions, a --help usage block, and clearer examples
in the top-level help output.

Claude-Session: LOCAL_992a1351-22c6-490b-a7b9-3518e4a2bcdb
@gavin-jeong

Copy link
Copy Markdown
Collaborator

🤖 Auto-review found potential issues (non-blocking):

The PR does not compile despite the "go build ./... passes" claim. Two errors in RunMove (internal/cli/cli.go): (1) session.FindSessionByID does not exist anywhere in the codebase — no such function is defined in the session package; (2) findSessionFile returns only (string, error) (2 values) but is called as _, sessID, err := findSessionFile(claudeDir) (3 values). The default/--session code paths also depend on sess.ProjectPath from the missing function. Fix by adding a real ID-lookup helper (or reuse LoadCachedSessions + prefix match) and correct the findSessionFile call to capture the file path and resolve the session ID from it, then re-verify with an actual build.

Will re-review automatically when new commits are pushed.

@gavin-jeong gavin-jeong added auto-review/changes-requested Auto-review found potential issues and removed auto-review/pending Auto-review waiting for Atlantis plan / ArgoCD diff labels Jul 7, 2026
@gavin-jeong gavin-jeong merged commit 74fd4aa into master Jul 9, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-review/changes-requested Auto-review found potential issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants