Skip to content

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

Draft
ayeshurun with Copilot wants to merge 9 commits into
mainfrom
copilot/implement-cp-command-for-cli
Draft

feat(cp): extend cp command to support local-to-Fabric and Fabric-to-local copy#73
ayeshurun with Copilot wants to merge 9 commits into
mainfrom
copilot/implement-cp-command-for-cli

Conversation

Copilot AI commented Mar 12, 2026

Copy link
Copy Markdown
  • 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.

ayeshurun and others added 2 commits March 17, 2026 12:14
This script benchmarks the startup performance of the CLI by measuring module import times, CLI invocation times, and heavy dependency loading. It allows comparisons against a baseline branch or tag.
…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