feat: add '/cp' command to copy last response to clipboard #2049
Conversation
PR Change SummaryAdded a new command to the clai package that allows users to copy the last response to the clipboard.
Modified Files
How can I customize these reviews?Check out the Hyperlint AI Reviewer docs for more information on how to customize the review. If you just want to ignore it on this PR, you can add the Note specifically for link checks, we only check the first 30 links in a file and we cache the results for several hours (for instance, if you just added a page, you might experience this). Our recommendation is to add |
There was a problem hiding this comment.
Pull Request Overview
Adds a new /cp command to copy the last chat response to the clipboard, including dependency updates, CLI handling, documentation, and tests.
- Introduces
pyperclipas a new CLI dependency - Implements
/cpinhandle_slash_commandand updates auto-suggestions - Updates docs, adds unit tests for the new behavior
Reviewed Changes
Copilot reviewed 5 out of 6 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| tests/test_cli.py | Imported pyperclip, added tests for /cp command |
| pydantic_ai_slim/pyproject.toml | Added pyperclip>=1.9.0 to the cli dependencies |
| pydantic_ai_slim/pydantic_ai/_cli.py | Imported pyperclip, updated help and handler for /cp, modified auto-suggestions |
| docs/cli.md | Documented the /cp command |
| clai/README.md | Documented the /cp command |
Comments suppressed due to low confidence (1)
tests/test_cli.py:215
- [nitpick] Consider adding a test case where
messages[-1].partscontains only non-text parts to cover the 'No text content to copy.' branch and ensure that no clipboard write occurs.
def test_handle_slash_command_copy():
07pepa
force-pushed
the
add-coppy-to-clipboard-comand
branch
6 times, most recently
from
b32f526 to
620e858
Compare
07pepa
force-pushed
the
add-coppy-to-clipboard-comand
branch
from
620e858 to
ccc9422
Compare
| except IndexError: | ||
| console.print('[dim]No output available to copy.[/dim]') | ||
| else: | ||
| text_to_copy = ''.join(part.content for part in parts if part.part_kind == 'text') |
There was a problem hiding this comment.
I think we'll want to put 1 or 2 newlines in between text from different parts, as they may not join cleanly.
|
@07pepa Thanks, nice feature! |
This feature is for
claipackage.It adds
/cpcommand (analogus to linux cp command) to clai that coppies model output to clipboard as is.TL DR what was done:
claidependency:pyperclipclaifor comand/cpto store data to clipboard