fix: Address all Copilot review feedback and polish for merge

- Reorder imports in encryptpdf.py to follow PEP 8 (stdlib → third-party → local)
- Fix passifypdf/__init__.py placeholder text from cookiecutter template
- Fix unclosed code fence and inline security warning in docs/CLI_OPTIONS.md
- Update README.md stale 'Sample Run' section to use Poetry CLI entry point
- Clean up blank lines in integration test file
- All 8 Copilot suggestions now fully addressed
- All tests pass (pytest + unittest)

Author: ambicuity

README.md

@@ -36,17 +36,12 @@ passifypdf --help
 
 
 Sample Run:
-```
-change dir to "passifypdf/passifypdf", 
-
-Then run, 
-python encryptpdf.py
+```bash
+passifypdf -i input.pdf -o protected.pdf -p mySecretPassword
 
--------------------------Sample output----------------------
-$python encryptpdf.py
-Congratulations!
-PDF file encrypted successfully and saved as 'Sample_PDF_protected.pdf'
-$
+# -------------------------Sample output----------------------
+# Congratulations!
+# PDF file encrypted successfully and saved as 'protected.pdf'
 ```
 
 ## Known Issues

docs/CLI_OPTIONS.md

@@ -6,16 +6,13 @@ The `passifypdf` tool accepts the following command-line arguments:
 | :--- | :--- | :--- | :--- |
 | `-i` | `--input` | Yes | Path to the input PDF file. |
 | `-o` | `--output` | Yes | Path to the output (encrypted) PDF file. |
-| `-p` | `--passwd` | Yes | Password to use for encryption. |
+| `-p` | `--passwd` | Yes | Password to use for encryption. **Avoid passing sensitive passwords directly on the command line, as they may be exposed via process listings and shell history.** |
 | `-v` | `--version` | No | Show the program's version number and exit. |
 | `-h` | `--help` | No | Show the help message and exit. |
 
 ## Example Usage
 
 ```bash
-# Using the installed CLI command
+# Using the installed CLI command (after `poetry install`)
 passifypdf -i input.pdf -o protected.pdf -p mySecretPassword
 ```
-
-**Security Note:** Avoid passing sensitive passwords directly on the command line if possible, as they may be exposed in process listings.
-```

passifypdf/__init__.py

@@ -1 +1 @@
-"""Unit test package for {{ cookiecutter.project_slug }}."""
+"""passifypdf - Protect PDF files with a password of your choice."""

passifypdf/encryptpdf.py

@@ -1,12 +1,14 @@
-from .cli import get_arg_parser
-from pypdf import PdfReader, PdfWriter
-
+"""Core PDF encryption module."""
 
 import sys
-import os
 from pathlib import Path
 from typing import Union
 
+from pypdf import PdfReader, PdfWriter
+
+from .cli import get_arg_parser
+
+
 def encrypt_pdf(input_pdf: Union[str, Path], output_pdf: Union[str, Path], password: str) -> None:
     """
     Encrypts a PDF file with a password.
@@ -18,14 +20,15 @@ def encrypt_pdf(input_pdf: Union[str, Path], output_pdf: Union[str, Path], passw
 
     Raises:
         FileNotFoundError: If the input PDF file does not exist.
+        IsADirectoryError: If the input path is a directory.
         Exception: For other errors during processing.
     """
     input_path = Path(input_pdf)
     if not input_path.exists():
         raise FileNotFoundError(f"Input file '{input_pdf}' not found.")
-    
+
     if not input_path.is_file():
-         raise IsADirectoryError(f"Input path '{input_pdf}' is not a file.")
+        raise IsADirectoryError(f"Input path '{input_pdf}' is not a file.")
 
     try:
         reader = PdfReader(input_path)
@@ -49,13 +52,13 @@ def encrypt_pdf(input_pdf: Union[str, Path], output_pdf: Union[str, Path], passw
 def main() -> int:
     """
     Main function to run the CLI.
-    
+
     Returns:
         int: Exit code (0 for success, 1 for failure).
     """
     arg_parser = get_arg_parser()
     args = arg_parser.parse_args()
-    
+
     try:
         encrypt_pdf(args.input, args.output, args.passwd)
         print(f"Congratulations!\nPDF file encrypted successfully and saved as '{args.output}'")

tests/integrationtests/test_encryptpdf_integration.py

@@ -1,19 +1,17 @@
+import os
 from unittest import TestCase
 
+from pypdf import PdfReader, PdfWriter
 
-
-
-import os
-from pathlib import Path
-from pypdf import PdfWriter, PdfReader
 from passifypdf.encryptpdf import encrypt_pdf
 
+
 class TestPdfIntegrationTests(TestCase):
     def setUp(self):
         self.input_pdf = "test_input.pdf"
         self.output_pdf = "test_output.pdf"
         self.password = "strongpassword"
-        
+
         # Create a dummy PDF
         writer = PdfWriter()
         writer.add_blank_page(width=100, height=100)
@@ -30,14 +28,14 @@ def tearDown(self):
     def test_encrypt_pdf_integration(self):
         # Encrypt the PDF
         encrypt_pdf(self.input_pdf, self.output_pdf, self.password)
-        
+
         # Verify output exists
         self.assertTrue(os.path.exists(self.output_pdf))
-        
+
         # Verify it is encrypted
         reader = PdfReader(self.output_pdf)
         self.assertTrue(reader.is_encrypted)
-        
+
         # Verify we can decrypt it
         self.assertTrue(reader.decrypt(self.password))
         self.assertEqual(len(reader.pages), 1)