Merge pull request #22 from jlowin/print

don't copy when printing to screen

Author: jlowin

src/copychat/cli.py

@@ -433,8 +433,8 @@ def main(
             error_console.print(
                 f"Output {'appended' if append else 'written'} to [green]{outfile}[/]"
             )
-        # Handle clipboard only if not writing to file
-        else:
+        # Only use clipboard if not writing to file AND not just printing to stdout
+        elif not print_output or append:
             if append:
                 try:
                     existing_clipboard = pyperclip.paste()
@@ -444,13 +444,24 @@ def main(
                         "[yellow]Warning: Could not read clipboard for append[/]"
                     )
 
-            pyperclip.copy(result)
-            # Calculate total lines outside the f-string
-            total_lines = sum(f.content.count("\n") + 1 for f in format_result.files)
-            error_console.print(
-                f"{'Appended' if append else 'Copied'} to clipboard "
-                f"(~{format_result.total_tokens:,} tokens, {total_lines:,} lines)"
-            )
+            try:
+                pyperclip.copy(result)
+                # Calculate total lines outside the f-string
+                total_lines = sum(
+                    f.content.count("\n") + 1 for f in format_result.files
+                )
+                error_console.print(
+                    f"{'Appended' if append else 'Copied'} to clipboard "
+                    f"(~{format_result.total_tokens:,} tokens, {total_lines:,} lines)"
+                )
+            except Exception as e:
+                error_console.print(
+                    f"[yellow]Warning: Could not copy to clipboard: {str(e)}[/]"
+                )
+                if not print_output:
+                    # If clipboard failed and we're not printing, show the content
+                    error_console.print("[cyan]Content would have been:[/]")
+                    print(result)
 
         # Print to stdout only if explicitly requested
         if print_output:

tests/test_cli.py

@@ -4,7 +4,7 @@
 import re
 from pathlib import Path
 
-runner = CliRunner(mix_stderr=False)
+runner = CliRunner()
 
 
 def strip_ansi(text: str) -> str:
@@ -259,7 +259,7 @@ def mock_copy(text):
 
 def test_cli_github_item_basic(monkeypatch):
     """Basic test for GitHub item handling that doesn't rely on internal implementation."""
-    runner = CliRunner(mix_stderr=False)
+    runner = CliRunner()
 
     # Instead of mocking complex internals, just provide a simple mock for the scan_directory function
     # so it returns a known result when the CLI processes a GitHub item