bpsai-pair 0.2.1__tar.gz → 0.2.2__tar.gz

{bpsai_pair-0.2.1 → bpsai_pair-0.2.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: bpsai-pair
-Version: 0.2.1
+Version: 0.2.2
 Summary: CLI for AI pair-coding workflow
 Author: BPS AI Software
 Requires-Python: >=3.9

{bpsai_pair-0.2.1 → bpsai_pair-0.2.2}/bpsai_pair/__init__.py

@@ -2,7 +2,7 @@
 bpsai_pair package
 """
 
-__version__ = "0.2.1"
+__version__ = "0.2.2"
 
 # Make modules available at package level
 from . import cli

{bpsai_pair-0.2.1 → bpsai_pair-0.2.2}/bpsai_pair/data/cookiecutter-paircoder/{{cookiecutter.project_slug}}/.agentpackignore

@@ -1,6 +1,8 @@
 # Default agent pack exclusions
 .git/
 .venv/
+venv/
+env/
 __pycache__/
 node_modules/
 dist/
@@ -10,3 +12,8 @@ build/
 *.tgz
 *.tar.gz
 *.zip
+.env
+.env.*
+*.pyc
+.DS_Store
+Thumbs.db

{bpsai_pair-0.2.1 → bpsai_pair-0.2.2}/bpsai_pair/ops.py

@@ -17,12 +17,12 @@ import json
 
 class GitOps:
     """Git operations helper."""
-    
+
     @staticmethod
     def is_repo(path: Path) -> bool:
         """Check if path is a git repo."""
         return (path / ".git").exists()
-    
+
     @staticmethod
     def is_clean(path: Path) -> bool:
         """Check if working tree is clean."""
@@ -58,7 +58,7 @@ class GitOps:
             return True
         except:
             return False
-    
+
     @staticmethod
     def current_branch(path: Path) -> str:
         """Get current branch name."""
@@ -69,7 +69,7 @@ class GitOps:
             text=True
         )
         return result.stdout.strip() if result.returncode == 0 else ""
-    
+
     @staticmethod
     def create_branch(path: Path, branch: str, from_branch: str = "main") -> bool:
         """Create and checkout a new branch."""
@@ -81,10 +81,10 @@ class GitOps:
         )
         if check.returncode != 0:
             return False
-        
+
         # Checkout source branch
         subprocess.run(["git", "checkout", from_branch], cwd=path, capture_output=True)
-        
+
         # Pull if upstream exists
         upstream = subprocess.run(
             ["git", "rev-parse", "--abbrev-ref", "--symbolic-full-name", "@{u}"],
@@ -93,7 +93,7 @@ class GitOps:
         )
         if upstream.returncode == 0:
             subprocess.run(["git", "pull", "--ff-only"], cwd=path, capture_output=True)
-        
+
         # Create new branch
         result = subprocess.run(
             ["git", "checkout", "-b", branch],
@@ -101,13 +101,13 @@ class GitOps:
             capture_output=True
         )
         return result.returncode == 0
-    
+
     @staticmethod
     def add_commit(path: Path, files: List[Path], message: str) -> bool:
         """Add files and commit."""
         for f in files:
             subprocess.run(["git", "add", str(f)], cwd=path, capture_output=True)
-        
+
         result = subprocess.run(
             ["git", "commit", "-m", message],
             cwd=path,
@@ -118,19 +118,19 @@ class GitOps:
 
 class ProjectTree:
     """Generate project tree snapshots."""
-    
+
     @staticmethod
     def generate(root: Path, excludes: Optional[Set[str]] = None) -> str:
         """Generate a tree structure of the project."""
         if excludes is None:
             excludes = {
-                '.git', '.venv', 'venv', '__pycache__', 
+                '.git', '.venv', 'venv', '__pycache__',
                 'node_modules', 'dist', 'build', '.mypy_cache',
                 '.pytest_cache', '.tox', '*.egg-info', '.DS_Store'
             }
-        
+
         tree_lines = []
-        
+
         def should_skip(path: Path) -> bool:
             name = path.name
             for pattern in excludes:
@@ -139,20 +139,20 @@ class ProjectTree:
                 if name == pattern:
                     return True
             return False
-        
+
         def walk_dir(dir_path: Path, prefix: str = ""):
             items = sorted(dir_path.iterdir(), key=lambda x: (x.is_file(), x.name))
             items = [i for i in items if not should_skip(i)]
-            
+
             for i, item in enumerate(items):
                 is_last = i == len(items) - 1
                 current = "└── " if is_last else "├── "
                 tree_lines.append(f"{prefix}{current}{item.name}")
-                
+
                 if item.is_dir():
                     extension = "    " if is_last else "│   "
                     walk_dir(item, prefix + extension)
-        
+
         tree_lines.append(".")
         walk_dir(root)
         return "\n".join(tree_lines)
@@ -160,7 +160,7 @@ class ProjectTree:
 
 class ContextPacker:
     """Package context files for AI agents."""
-    
+
     @staticmethod
     def read_ignore_patterns(ignore_file: Path) -> Set[str]:
         """Read patterns from .agentpackignore file."""
@@ -170,42 +170,41 @@ class ContextPacker:
                 for line in f:
                     line = line.strip()
                     if line and not line.startswith('#'):
-                        patterns.add(line.rstrip('/'))
+                        patterns.add(line)
         else:
             # Default patterns
             patterns = {
                 '.git', '.venv', '__pycache__', 'node_modules',
-                'dist', 'build', '*.log', '*.bak', '*.tgz', 
+                'dist', 'build', '*.log', '*.bak', '*.tgz',
                 '*.tar.gz', '*.zip', '.env*'
             }
         return patterns
-    
+
     @staticmethod
     def should_exclude(path: Path, patterns: Set[str]) -> bool:
         """Check if path should be excluded based on patterns."""
-        path_str = str(path).replace('\\', '/')
-        
+        from pathlib import PurePath
+
+        p = PurePath(path.as_posix())
+
         for pattern in patterns:
-            # Handle wildcards
-            if '*' in pattern:
-                import fnmatch
-                if fnmatch.fnmatch(path.name, pattern):
+            # Handle directory patterns (ending with /)
+            if pattern.endswith('/'):
+                dir_pattern = pattern.rstrip('/')
+                # Check if this is the directory itself or if it's inside the directory
+                if path.is_dir() and p.match(dir_pattern):
+                    return True
+                if any(parent.match(dir_pattern) for parent in p.parents):
                     return True
-                if fnmatch.fnmatch(path_str, pattern):
+            # Handle file/general patterns
+            else:
+                if p.match(pattern):
                     return True
-            # Handle directory patterns
-            elif pattern.endswith('/'):
-                if path.is_dir() and path.name == pattern[:-1]:
+                if any(parent.match(pattern) for parent in p.parents):
                     return True
-            # Exact match
-            elif path.name == pattern:
-                return True
-            # Path contains pattern
-            elif pattern in path_str.split('/'):
-                return True
-        
+
         return False
-    
+
     @staticmethod
     def pack(
         root: Path,
@@ -217,33 +216,33 @@ class ContextPacker:
         # Default files to include
         context_files = [
             root / "context" / "development.md",
-            root / "context" / "agents.md", 
+            root / "context" / "agents.md",
             root / "context" / "project_tree.md",
         ]
-        
+
         # Add directory_notes if it exists
         dir_notes = root / "context" / "directory_notes"
         if dir_notes.exists():
             for note in dir_notes.rglob("*.md"):
                 context_files.append(note)
-        
+
         # Add extra files
         if extra_files:
             for extra in extra_files:
                 extra_path = root / extra
                 if extra_path.exists():
                     context_files.append(extra_path)
-        
+
         # Filter out non-existent files
         context_files = [f for f in context_files if f.exists()]
-        
+
         if dry_run:
             return context_files
-        
+
         # Read ignore patterns
         ignore_file = root / ".agentpackignore"
         patterns = ContextPacker.read_ignore_patterns(ignore_file)
-        
+
         # Create tarball
         with tarfile.open(output, "w:gz") as tar:
             for file_path in context_files:
@@ -251,13 +250,13 @@ class ContextPacker:
                 if not ContextPacker.should_exclude(file_path, patterns):
                     arcname = file_path.relative_to(root)
                     tar.add(file_path, arcname=str(arcname))
-        
+
         return context_files
 
 
 class FeatureOps:
     """Operations for feature branch management."""
-    
+
     @staticmethod
     def create_feature(
         root: Path,
@@ -271,17 +270,17 @@ class FeatureOps:
         # Check if working tree is clean
         if not force and not GitOps.is_clean(root):
             raise ValueError("Working tree not clean. Commit or stash changes, or use --force")
-        
+
         # Create branch
         branch_name = f"{branch_type}/{name}"
         if not GitOps.create_branch(root, branch_name):
             raise ValueError(f"Failed to create branch {branch_name}")
-        
+
         # Ensure context directory structure
         context_dir = root / "context"
         context_dir.mkdir(exist_ok=True)
         (context_dir / "directory_notes").mkdir(exist_ok=True)
-        
+
         # Update or create development.md
         dev_file = context_dir / "development.md"
         if not dev_file.exists():
@@ -301,7 +300,7 @@ class FeatureOps:
         else:
             # Update existing file
             content = dev_file.read_text()
-            
+
             # Update Primary Goal
             if primary_goal:
                 import re
@@ -315,7 +314,7 @@ class FeatureOps:
                     f'Overall goal is: {primary_goal}',
                     content
                 )
-            
+
             # Update Phase
             if phase:
                 content = re.sub(
@@ -328,16 +327,16 @@ class FeatureOps:
                     f'Next action will be: {phase}',
                     content
                 )
-            
+
             # Update Last action
             content = re.sub(
                 r'Last action was:.*',
                 f'Last action was: Created feature branch {branch_name}',
                 content
             )
-            
+
             dev_file.write_text(content)
-        
+
         # Create agents.md if missing
         agents_file = context_dir / "agents.md"
         if not agents_file.exists():
@@ -360,7 +359,7 @@ This project uses a **Context Loop**. Always keep these fields current:
 Run `bpsai-pair pack --out agent_pack.tgz` and upload to your session.
 """
             agents_file.write_text(agents_content)
-        
+
         # Generate project tree
         tree_file = context_dir / "project_tree.md"
         tree_content = f"""# Project Tree (snapshot)
@@ -371,29 +370,29 @@ _Generated: {datetime.now(timezone.utc).isoformat()}Z_
 ```
 """
         tree_file.write_text(tree_content)
-        
+
         # Commit changes
         GitOps.add_commit(
             root,
             [dev_file, agents_file, tree_file],
             f"feat(context): start {branch_name} — Primary Goal: {primary_goal or 'TBD'}"
         )
-        
+
         return True
 
 
 class LocalCI:
     """Cross-platform local CI runner."""
-    
+
     @staticmethod
     def run_python_checks(root: Path) -> dict:
         """Run Python linting, formatting, and tests."""
         results = {}
-        
+
         # Check if Python project
         if not ((root / "pyproject.toml").exists() or (root / "requirements.txt").exists()):
             return results
-        
+
         # Try to run ruff
         try:
             subprocess.run(["ruff", "format", "--check", "."], cwd=root, check=True)
@@ -401,46 +400,46 @@ class LocalCI:
             results["ruff"] = "passed"
         except:
             results["ruff"] = "failed or not installed"
-        
+
         # Try to run mypy
         try:
             subprocess.run(["mypy", "."], cwd=root, check=True)
             results["mypy"] = "passed"
         except:
             results["mypy"] = "failed or not installed"
-        
+
         # Try to run pytest
         try:
             subprocess.run(["pytest", "-q"], cwd=root, check=True)
             results["pytest"] = "passed"
         except:
             results["pytest"] = "failed or not installed"
-        
+
         return results
-    
+
     @staticmethod
     def run_node_checks(root: Path) -> dict:
         """Run Node.js linting, formatting, and tests."""
         results = {}
-        
+
         if not (root / "package.json").exists():
             return results
-        
+
         # Try npm commands
         try:
             subprocess.run(["npm", "run", "lint"], cwd=root, check=True)
             results["eslint"] = "passed"
         except:
             results["eslint"] = "failed or not configured"
-        
+
         try:
             subprocess.run(["npm", "test"], cwd=root, check=True)
             results["npm test"] = "passed"
         except:
             results["npm test"] = "failed or not configured"
-        
+
         return results
-    
+
     @staticmethod
     def run_all(root: Path) -> dict:
         """Run all applicable CI checks."""

{bpsai_pair-0.2.1 → bpsai_pair-0.2.2}/bpsai_pair.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: bpsai-pair
-Version: 0.2.1
+Version: 0.2.2
 Summary: CLI for AI pair-coding workflow
 Author: BPS AI Software
 Requires-Python: >=3.9

{bpsai_pair-0.2.1 → bpsai_pair-0.2.2}/pyproject.toml

@@ -1,7 +1,7 @@
 [project]
 readme = "README.md"
 name = "bpsai-pair"
-version = "0.2.1"
+version = "0.2.2"
 description = "CLI for AI pair-coding workflow"
 requires-python = ">=3.9"
 dependencies = [ "typer>=0.12", "rich>=13.7", "pyyaml>=6.0",]

{bpsai_pair-0.2.1 → bpsai_pair-0.2.2}/tests/test_ops.py

@@ -2,6 +2,7 @@
 from pathlib import Path
 import subprocess
 import sys
+import tarfile
 
 import pytest
 
@@ -68,3 +69,29 @@ def test_context_packer_patterns():
     assert ops.ContextPacker.should_exclude(Path(".git"), patterns) == True
     assert ops.ContextPacker.should_exclude(Path("test.log"), patterns) == True
     assert ops.ContextPacker.should_exclude(Path("main.py"), patterns) == False
+
+
+def test_pack_respects_directory_ignore(tmp_path):
+    """Ensure directory patterns exclude contents from pack."""
+    root = tmp_path
+
+    context_dir = root / "context"
+    context_dir.mkdir()
+    (context_dir / "development.md").write_text("dev")
+    (context_dir / "agents.md").write_text("agents")
+    (context_dir / "project_tree.md").write_text("tree")
+
+    private_dir = root / "private"
+    private_dir.mkdir()
+    (private_dir / "secret.txt").write_text("shh")
+
+    (root / ".agentpackignore").write_text("private/\n")
+
+    output = root / "pack.tgz"
+    ops.ContextPacker.pack(root, output, extra_files=["private/secret.txt"])
+
+    with tarfile.open(output, "r:gz") as tar:
+        names = tar.getnames()
+
+    assert "context/development.md" in names
+    assert "private/secret.txt" not in names