Bug Fix

Author: Nisarg38

check_duplicate_versions.py

@@ -11,12 +11,20 @@
 
 
 def get_file_hash(file_path: Path) -> str:
-    """Calculate MD5 hash of a file."""
-    md5_hash = hashlib.md5()
-    with open(file_path, 'rb') as f:
-        for chunk in iter(lambda: f.read(4096), b''):
-            md5_hash.update(chunk)
-    return md5_hash.hexdigest()
+    """Calculate MD5 hash of a file.
+    
+    Returns:
+        Hash string on success, None on failure (logs error)
+    """
+    try:
+        md5_hash = hashlib.md5()
+        with open(file_path, 'rb') as f:
+            for chunk in iter(lambda: f.read(4096), b''):
+                md5_hash.update(chunk)
+        return md5_hash.hexdigest()
+    except (OSError, IOError) as e:
+        print(f"⚠️  Error reading file {file_path}: {e}")
+        return None
 
 
 def find_duplicate_versions(prompts_dir: Path) -> Dict[str, Dict[str, List[str]]]:
@@ -43,7 +51,9 @@ def find_duplicate_versions(prompts_dir: Path) -> Dict[str, Dict[str, List[str]]
 
         for version_file in version_files:
             file_hash = get_file_hash(version_file)
-            hash_map[file_hash].append(version_file.name)
+            # Skip files that couldn't be read
+            if file_hash is not None:
+                hash_map[file_hash].append(version_file.name)
 
         # Only include agents with duplicates
         duplicates = {h: files for h, files in hash_map.items() if len(files) > 1}
@@ -79,9 +89,9 @@ def print_report(duplicates: Dict[str, Dict[str, List[str]]]):
 
 def main():
     """Main entry point."""
-    # Find prompts directory
-    script_dir = Path(__file__).parent
-    prompts_dir = script_dir / "prompts"
+    # Find prompts directory (at repository root)
+    repo_root = Path(__file__).parent
+    prompts_dir = repo_root / "prompts"
 
     if not prompts_dir.exists():
         print(f"❌ Error: Prompts directory not found at {prompts_dir}")

prompts/CodeReviewer/versions/v006.md

@@ -0,0 +1,5 @@
+You are a code review assistant specialized in {{programming_language}}. Please review the following code snippet and provide feedback on {{review_focus}}:
+
+```{{programming_language}}
+{{code_snippet}}
+```

prompts/ComplexCodeReviewer/versions/v005.md

@@ -0,0 +1,7 @@
+You are a code review assistant with active tools: {{active_tools}}. Specialized in {{programming_language}}. Review the code with {{severity}} scrutiny focusing on {{review_focus}}:
+
+```{{programming_language}}
+{{code_snippet}}
+```
+
+Provide feedback in: 'Summary', 'Critical Issues', 'Improvements', 'Positives'.

prompts/SimpleChat/versions/v006.md

@@ -0,0 +1 @@
+You are a helpful AI assistant named {{assistant_name}}. Your goal is to provide clear and concise answers to {{user_name}}'s questions.

prompts/TemplateDemo/versions/v005.md

@@ -0,0 +1,16 @@
+You are creating a {{content_type}} about {{theme}}.
+
+{% if difficulty == 'beginner' %}
+Keep it simple and accessible for beginners.
+{% elif difficulty == 'intermediate' %}
+Include some advanced concepts but explain them clearly.
+{% else %}
+Don't hold back on technical details and advanced concepts.
+{% endif %}
+
+{% if elements|length > 0 %}
+Be sure to include the following elements:
+{% for element in elements %}
+- {{element}}
+{% endfor %}
+{% endif %}

prompts/simple_chat/versions/v005.md

@@ -0,0 +1,11 @@
+You are a {{personality}} assistant specialized in {{domain}}.
+
+Your role is to provide helpful, accurate, and engaging responses to user questions and requests. Always maintain a professional and friendly tone while adapting to the user's needs.
+
+Key guidelines:
+- Be concise but thorough in your explanations
+- Ask clarifying questions when needed
+- Provide examples when helpful
+- Stay focused on the {{domain}} domain when specified
+
+How can I help you today?

src/promptix/tools/cli.py

@@ -457,10 +457,17 @@ def main():
     """
     try:
         # Handle the case where user runs OpenAI commands directly
-        if len(sys.argv) > 1 and sys.argv[1] not in ['studio', 'agent', 'openai', '--help', '--version']:
-            # This looks like an OpenAI command, redirect
-            Config.validate()
-            sys.exit(openai_main())
+        # Check if first arg is a flag (starts with '-') or a recognized top-level command
+        if len(sys.argv) > 1:
+            first_arg = sys.argv[1]
+            # List of recognized top-level commands
+            top_level_commands = ['studio', 'agent', 'openai', 'version', 'hooks']
+            
+            # Don't redirect if it's a flag or a recognized command
+            if not first_arg.startswith('-') and first_arg not in top_level_commands:
+                # This looks like an OpenAI command, redirect
+                Config.validate()
+                sys.exit(openai_main())
 
         cli()
 

src/promptix/tools/hook_manager.py

@@ -309,17 +309,17 @@ def main():
                              help='Overwrite existing hook')
 
     # Uninstall command
-    uninstall_cmd = subparsers.add_parser('uninstall', help='Uninstall pre-commit hook')
+    subparsers.add_parser('uninstall', help='Uninstall pre-commit hook')
 
     # Enable/disable commands
-    enable_cmd = subparsers.add_parser('enable', help='Enable disabled hook')
-    disable_cmd = subparsers.add_parser('disable', help='Disable hook temporarily')
+    subparsers.add_parser('enable', help='Enable disabled hook')
+    subparsers.add_parser('disable', help='Disable hook temporarily')
 
     # Status command
-    status_cmd = subparsers.add_parser('status', help='Show hook status')
+    subparsers.add_parser('status', help='Show hook status')
 
     # Test command
-    test_cmd = subparsers.add_parser('test', help='Test hook without committing')
+    subparsers.add_parser('test', help='Test hook without committing')
 
     args = parser.parse_args()