Skip to content

Commit 4c0629d

Browse files
committed
docs(todo): restore cross cd local_path capability task in P3
- Restore P3 entry for adding cross cd to local_path capability - Document relationship with P2 refactor task (different approaches) - P2: refactor default behavior to local_path - P3: document alternative approaches (flags, separate command) - Both tasks are valid and complementary
1 parent 6c8d1ff commit 4c0629d

File tree

1 file changed

+23
-0
lines changed

1 file changed

+23
-0
lines changed

TODO.md

Lines changed: 23 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -124,6 +124,29 @@
124124
- **Priority Rationale:** Low priority - UX improvement, not a bug
125125
- **Status:** Documented for future implementation
126126

127+
- [ ] **Add `cross cd` to local_path capability** - Currently only changes to worktree
128+
- **Issue:** `cross cd` currently opens a shell in the WORKTREE (hidden `.git/cross/worktrees/`)
129+
- **Desired Behavior:**
130+
- Provide ability to change directory to LOCAL_PATH (the actual patched directory in main repo)
131+
- Options:
132+
1. Add `--local` flag: `cross cd --local [patch]` → changes to local_path
133+
2. Add separate command: `cross path [patch]` → outputs local_path for use with shell `cd $(cross path patch)`
134+
3. Make `cd` default to local_path, add `--worktree` flag for old behavior (covered in P2 refactor)
135+
- **Current Implementation:**
136+
- Justfile (lines 761-797): `cd` target opens shell in worktree directory
137+
- Go (lines 681-730): Same behavior - opens shell in worktree
138+
- Rust: Similar behavior
139+
- **Relationship to P2 Task:**
140+
- P2 refactor will change default behavior to local_path
141+
- This task documents alternative approaches (flags, separate command)
142+
- May be resolved by P2 implementation or kept as additional options
143+
- **Effort:** 2-3 hours
144+
- **Files:** `Justfile.cross`, `src-go/main.go`, `src-rust/src/main.rs`, `test/017_cd_local.sh`
145+
- **Complexity:** LOW - Add path resolution and output logic
146+
- **Impact:** MEDIUM - Improves workflow for users editing patched files
147+
- **Priority Rationale:** Low priority - workaround exists (manually navigate to patch), P2 refactor may address this
148+
- **Status:** Not yet implemented, may be covered by P2 refactor
149+
127150
### Completed Enhancements
128151

129152
## Known Issues (To FIX)

0 commit comments

Comments
 (0)