Skip to content

feat(cp): extend cp command to support local-to-Fabric and Fabric-to-local copy#73

Draft
Copilot wants to merge 1 commit intomainfrom
copilot/implement-cp-command-for-cli
Draft

feat(cp): extend cp command to support local-to-Fabric and Fabric-to-local copy#73
Copilot wants to merge 1 commit intomainfrom
copilot/implement-cp-command-for-cli

Conversation

Copy link

Copilot AI commented Mar 12, 2026

  • Understand codebase and existing cp local path handling
  • Add os.path.expanduser() in LocalPath.__init__ to normalize ~ paths
  • Add os.path.expanduser() in get_command_context local path fallback (fix bug with parenthesization)
  • Add ~ path examples to cp parser help text
  • Add 4 tests for expanduser support in test_cp_local.py
  • Run tests — all 35 pass (31 existing + 4 new)
  • Code review — no actionable issues
  • CodeQL — 0 security alerts

🔒 GitHub Advanced Security automatically protects Copilot coding agent pull requests. You can protect all pull requests by enabling Advanced Security for your repositories. Learn more about Advanced Security.

…local copy

- Add os.path.expanduser() in LocalPath.__init__ to normalize ~ paths
- Add os.path.expanduser() in get_command_context local path fallback
- Add ~ path examples to cp parser help text
- Add 4 tests for expanduser support in test_cp_local.py

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@ayeshurun ayeshurun force-pushed the copilot/implement-cp-command-for-cli branch from c7cc2ed to 767ddeb Compare March 17, 2026 11:49
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.

2 participants