IncludeCPP 3.3.11__py3-none-any.whl → 3.3.20__py3-none-any.whl

includecpp/__init__.py

@@ -1,7 +1,7 @@
 from .core.cpp_api import CppApi
 import warnings
 
-__version__ = "3.3.11"
+__version__ = "3.3.20"
 __all__ = ["CppApi"]
 
 # Module-level cache for C++ modules

includecpp/cli/commands.py

@@ -10,6 +10,22 @@ import urllib.error
 from pathlib import Path
 from .config_parser import CppProjectConfig
 
+
+def _is_experimental_enabled() -> bool:
+    """Check if experimental features (cppy, ai) are enabled."""
+    config_path = Path.home() / '.includecpp' / '.secret'
+    if config_path.exists():
+        try:
+            config = json.loads(config_path.read_text())
+            return config.get('experimental_features', False)
+        except:
+            pass
+    return False
+
+
+# Check once at module load time
+_EXPERIMENTAL_ENABLED = _is_experimental_enabled()
+
 # Unicode fallback for Windows terminals with limited encoding
 def _supports_unicode():
     """Check if terminal supports Unicode output."""
@@ -4816,7 +4832,8 @@ def fix(module_name, all_modules, exclude, undo, auto_fix, verbose, use_ai):
     click.echo(f"{'='*60}")
 
 
-@cli.group(invoke_without_command=True)
+# AI group - conditionally registered based on experimental_features setting
+@click.group(invoke_without_command=True)
 @click.option('--info', is_flag=True, help='Show AI status and usage')
 @click.pass_context
 def ai(ctx, info):
@@ -5306,7 +5323,7 @@ def ai_edit(task, module_name, files, all_modules, exclude, think_mode, think_tw
         question = None  # Continuation doesn't ask more questions
 
     if not changes:
-        verbose.status("No changes needed", success=True)
+        verbose.status("No changes needed", phase='complete')
         verbose.end(success=True, message="No changes required")
         return
 
@@ -5520,7 +5537,7 @@ def ai_generate(task, files, think_mode, think_twice, think_three, use_websearch
     verbose.api_call(endpoint='chat/completions', tokens_out=len(str(changes)) // 4 if changes else len(response) // 4)
 
     if not changes:
-        verbose.status("No file changes needed", success=True)
+        verbose.status("No file changes needed", phase='complete')
         # Show response if any
         if response:
             _safe_echo(f"\n{response[:2000]}")
@@ -5598,7 +5615,7 @@ def ai_generate(task, files, think_mode, think_twice, think_three, use_websearch
             shell=True, capture_output=True, text=True, cwd=project_root
         )
         if result.returncode == 0:
-            verbose.status("Plugin generated", success=True)
+            verbose.status("Plugin generated", phase='complete')
         else:
             verbose.error(f"Plugin error: {result.stderr[:100]}")
             verbose.end(success=False, message="Plugin generation failed")
@@ -5611,7 +5628,7 @@ def ai_generate(task, files, think_mode, think_twice, think_three, use_websearch
             shell=True, capture_output=True, text=True, cwd=project_root
         )
         if result.returncode == 0:
-            verbose.status("Build successful", success=True)
+            verbose.status("Build successful", phase='complete')
         else:
             verbose.error(f"Build error: {result.stderr[:100]}")
 
@@ -5719,7 +5736,7 @@ def ai_optimize(module_name, files, agent_task, auto_confirm, no_verbose):
 
     verbose.phase('analyzing', 'Analyzing code for optimization opportunities')
     for fp in files_to_optimize.keys():
-        verbose.status(f"Scanning {Path(fp).name}", success=True)
+        verbose.status(f"Scanning {Path(fp).name}", phase='analyzing')
 
     verbose.phase('thinking', 'AI is analyzing and planning optimizations')
     verbose.api_call(endpoint='chat/completions', tokens_in=total_lines * 4)
@@ -5734,7 +5751,7 @@ def ai_optimize(module_name, files, agent_task, auto_confirm, no_verbose):
     verbose.api_call(endpoint='chat/completions', tokens_out=len(str(changes)) // 4 if changes else 0)
 
     if not changes:
-        verbose.status("No optimizations needed", success=True)
+        verbose.status("No optimizations needed", phase='complete')
         verbose.end(success=True, message="Code is already optimal")
         return
 
@@ -5787,7 +5804,7 @@ def ai_optimize(module_name, files, agent_task, auto_confirm, no_verbose):
                         click.secho("CONFIRM REQUIRED:", fg='yellow', bold=True)
                         click.echo(f"\n{confirm}\n")
                         if not click.confirm("Apply this change?"):
-                            verbose.status(f"Skipped: {change['file']}", success=False)
+                            verbose.status(f"Skipped: {change['file']}", phase='warning')
                             changes = [c for c in changes if c != change]
         if not click.confirm("Apply all changes?"):
             verbose.warning("Changes aborted by user")
@@ -5848,11 +5865,59 @@ def settings():
         click.echo("  includecpp ai key <YOUR_KEY>")
 
 
+@click.command()
+@click.option('--path', '-p', type=click.Path(), default=None,
+              help='Path to project directory (default: current directory)')
+def project(path):
+    """Open the Project Interface with CodeMaker.
+
+    Opens a professional visual mindmap tool for planning and designing
+    your C++ project structure.
+
+    Features:
+    - Visual node-based system design
+    - Class, Function, Object, and Definition nodes
+    - Connectable nodes with bezier curves
+    - Pan/zoom navigation (middle mouse button)
+    - Auto-save functionality
+    - .ma map file management
+
+    Controls:
+    - Right-click canvas: Create new nodes
+    - Right-click node: Edit/Connect/Delete
+    - Middle mouse + drag: Pan view
+    - Middle mouse + scroll: Zoom in/out
+    - ESC: Cancel connection mode
+    - DELETE: Remove selected nodes
+
+    Requires: pip install PyQt6
+    """
+    from pathlib import Path as PathLib
+    project_path = PathLib(path) if path else PathLib.cwd()
+
+    if not project_path.exists():
+        click.secho(f"Path does not exist: {project_path}", fg='red', err=True)
+        return
+
+    try:
+        from ..core.project_ui import show_project, PYQT_AVAILABLE
+        if not PYQT_AVAILABLE:
+            click.secho("PyQt6 not installed.", fg='red', err=True)
+            click.echo("Install with: pip install PyQt6")
+            return
+        success, msg = show_project(str(project_path))
+        if not success:
+            click.secho(msg, fg='red', err=True)
+    except Exception as e:
+        click.secho(f"Error opening project interface: {e}", fg='red', err=True)
+
+
 # ============================================================================
-# CPPY - Code Conversion Tools
+# CPPY - Code Conversion Tools (Experimental)
 # ============================================================================
 
-@cli.group(invoke_without_command=True)
+# CPPY group - conditionally registered based on experimental_features setting
+@click.group(invoke_without_command=True)
 @click.pass_context
 def cppy(ctx):
     """Code conversion tools for Python <-> C++.
@@ -6358,6 +6423,38 @@ def _convert_with_ai(content: str, module_name: str, source_file,
                 return _convert_to_python(content, module_name, source_file, output_dir, verbose)
 
         ai_verbose.api_call(endpoint='chat/completions', tokens_out=len(response) // 4)
+
+        # Check if AI needs clarification on unconvertible modules
+        if 'CLARIFICATION_NEEDED:' in response:
+            ai_verbose.phase('clarification', 'AI needs clarification')
+            clarification_match = re.search(r'CLARIFICATION_NEEDED:\n((?:- .+\n?)+)', response)
+            if clarification_match:
+                questions = clarification_match.group(1).strip()
+                click.echo("\n" + "=" * 60)
+                click.secho("AI NEEDS CLARIFICATION:", fg='yellow', bold=True)
+                click.echo(questions)
+                click.echo("=" * 60)
+                user_input = click.prompt("Your response (or 'skip' to use defaults)")
+                if user_input.lower() != 'skip':
+                    # Re-run AI with user's clarification
+                    clarified_prompt = ai_prompt + f"\n\nUSER CLARIFICATION:\n{user_input}\n\nNow convert with this clarification:"
+                    success, response, _ = ai_manager.generate(
+                        task=clarified_prompt,
+                        files={str(source_file): content},
+                        project_root=project_root,
+                        think=think,
+                        think_twice=think_twice,
+                        think_three=think_three,
+                        use_websearch=use_websearch,
+                        skip_tool_execution=True
+                    )
+                    if not success:
+                        ai_verbose.warning("Clarified conversion failed, using standard converter")
+                        if to_cpp:
+                            return _convert_to_cpp(content, module_name, source_file, output_dir, no_header, namespace, verbose)
+                        else:
+                            return _convert_to_python(content, module_name, source_file, output_dir, verbose)
+
         ai_verbose.phase('parsing', 'Extracting converted code from AI response')
 
         # Parse AI response
@@ -6367,7 +6464,7 @@ def _convert_with_ai(content: str, module_name: str, source_file,
         converted_code = _extract_ai_converted_code(response, to_cpp, module_name, namespace)
 
         if not converted_code:
-            ai_verbose.status("Using hybrid conversion (AI + standard)", success=True)
+            ai_verbose.status("Using hybrid conversion (AI + standard)", phase='complete')
             # Fall back to standard conversion with AI enhancements
             if to_cpp:
                 converter = PythonToCppConverter()
@@ -6460,22 +6557,95 @@ def _convert_with_ai(content: str, module_name: str, source_file,
     return result
 
 
+def _get_includecpp_readme() -> str:
+    """Load IncludeCPP README.md for AI context."""
+    try:
+        import importlib.resources
+        import includecpp
+        readme_path = Path(includecpp.__file__).parent.parent / 'README.md'
+        if readme_path.exists():
+            readme = readme_path.read_text(encoding='utf-8')
+            # Truncate to first sections (keep it focused)
+            sections = readme.split('# Changelog')[0]  # Everything before changelog
+            return sections[:8000]  # Limit to ~8k chars
+    except Exception:
+        pass
+    return ""
+
 def _build_cppy_ai_prompt(source: str, mode: str, module_name: str, namespace: str, analysis: dict, no_header: bool = False) -> str:
     """Build AI prompt with IncludeCPP-specific instructions."""
 
+    # Load README for AI context (dynamically, not hardcoded)
+    readme_context = _get_includecpp_readme()
+    includecpp_docs = ""
+    if readme_context:
+        includecpp_docs = f'''
+=== INCLUDECPP DOCUMENTATION (from README.md) ===
+{readme_context}
+=== END DOCUMENTATION ===
+'''
+
+    # Critical file extension rules
+    file_rules = f'''
+CRITICAL FILE EXTENSION RULES:
+- C++ implementation file MUST be: {module_name}.cpp (NOT .cp, NOT .cc, NOT .cxx)
+- C++ header file MUST be: {module_name}.h (NOT .hpp, NOT .hh)
+- Python file MUST be: {module_name}.py
+FOLLOW THESE EXTENSIONS EXACTLY.
+'''
+
+    # Unconvertible module handling
+    unconvertible_info = ""
+    for item, reason, line in getattr(analysis.get('converter', None), 'unconvertible', []) if isinstance(analysis, dict) else []:
+        unconvertible_info += f"- Line {line}: {item} ({reason})\n"
+
+    unconvertible_guidance = ""
+    if unconvertible_info:
+        unconvertible_guidance = f'''
+UNCONVERTIBLE MODULES DETECTED:
+{unconvertible_info}
+
+For unconvertible modules (tkinter, pygame, ursina, etc.):
+1. COMMENT OUT: Add /* UNCONVERTIBLE: tkinter */ comment
+2. ALTERNATIVE: Suggest C++ alternatives (Qt for GUI, SDL2 for games, etc.)
+3. SKIP: Remove the code with explanation comment
+
+If you need clarification, output:
+CLARIFICATION_NEEDED:
+- <what you need clarified>
+'''
+
     if mode == 'py_to_cpp':
         direction = "Python to C++"
         source_lang = "python"
         target_lang = "cpp"
 
+        # CRITICAL: No pybind11 - IncludeCPP handles bindings automatically
+        includecpp_note = '''
+**CRITICAL - IncludeCPP HANDLES BINDINGS AUTOMATICALLY:**
+- DO NOT include pybind11 headers
+- DO NOT write PYBIND11_MODULE macros
+- DO NOT use py:: namespace or py::object
+- Just write CLEAN C++ code in namespace includecpp
+- IncludeCPP will auto-generate Python bindings via: includecpp plugin <name> <file.cpp>
+
+The user runs:
+1. includecpp cppy convert file.py --cpp  (generates .cpp)
+2. includecpp plugin mymod file.cpp       (auto-generates pybind11 bindings)
+3. includecpp rebuild                     (compiles to Python module)
+'''
+
         if no_header:
             # No header mode - put everything in .cpp
             specific_rules = f'''
 INCLUDECPP-SPECIFIC REQUIREMENTS:
 1. ALL code MUST be wrapped in: namespace {namespace} {{ ... }}
-2. DO NOT create a header file - put ALL code in the .cpp file only
-3. Include declarations at the top of the .cpp file (no separate .h)
-4. For Python features without C++ equivalent, use pybind11 wrappers
+2. **CRITICAL: DO NOT create a header file** - --no-h flag is set
+3. Put ALL declarations AND implementations in the .cpp file only
+4. NO separate .h file should be created
+{includecpp_note}
+
+{file_rules}
 
 TYPE CONVERSIONS:
 - int -> int
@@ -6487,8 +6657,14 @@ TYPE CONVERSIONS:
 - Set[T] -> std::unordered_set<T>
 - Optional[T] -> std::optional<T>
 - Callable -> std::function<R(Args...)>
-- Any -> auto or py::object
-'''
+- bytes -> std::vector<uint8_t>
+
+GENERIC/TEMPLATE FUNCTIONS:
+- For functions that accept any type (like choices), use C++ templates:
+  template<typename T> T getRandomChoice(const std::vector<T>& choices);
+- NOT: auto getRandomChoice(const std::vector<auto>& choices)  // INVALID!
+
+{unconvertible_guidance}'''
         else:
             # Normal mode with header
             specific_rules = f'''
@@ -6497,22 +6673,30 @@ INCLUDECPP-SPECIFIC REQUIREMENTS:
 2. Create BOTH .cpp implementation AND .h header file
 3. Use #include "{module_name}.h" at top of .cpp file
 4. Header guard: #ifndef {module_name.upper()}_H / #define / #endif
-5. For Python features without C++ equivalent, use pybind11 wrappers:
-   - Generators: Use callback pattern or py::iterator
-   - Dynamic attributes: Use py::object
-   - Context managers: Use RAII pattern
-   - Duck typing: Use templates with concepts
+{includecpp_note}
+
+{file_rules}
 
-PYBIND11 WRAPPER PATTERN (for unconvertible features):
+GENERIC/TEMPLATE FUNCTIONS:
+- For functions that accept any type, use C++ templates:
+  template<typename T> T getRandomChoice(const std::vector<T>& choices);
+- NOT: auto getRandomChoice(const std::vector<auto>& choices)  // INVALID!
+
+TEMPLATE EXAMPLE (for generic functions):
 ```cpp
-#include <pybind11/pybind11.h>
-namespace py = pybind11;
+// In header (.h):
+template<typename T>
+T getRandomChoice(const std::vector<T>& choices);
 
-// For Python module calls that can't be converted:
-py::object call_python_func(const std::string& module, const std::string& func) {{
-    py::module_ mod = py::module_::import(module.c_str());
-    return mod.attr(func.c_str())();
+// In implementation (.cpp):
+template<typename T>
+T Manager::getRandomChoice(const std::vector<T>& choices) {{
+    // implementation
 }}
+// Explicit instantiations for common types:
+template int Manager::getRandomChoice<int>(const std::vector<int>&);
+template double Manager::getRandomChoice<double>(const std::vector<double>&);
+template std::string Manager::getRandomChoice<std::string>(const std::vector<std::string>&);
 ```
 
 TYPE CONVERSIONS:
@@ -6520,13 +6704,14 @@ TYPE CONVERSIONS:
 - float -> double
 - str -> std::string
 - bool -> bool
+- bytes -> std::vector<uint8_t>
 - List[T] -> std::vector<T>
 - Dict[K,V] -> std::unordered_map<K,V>
 - Set[T] -> std::unordered_set<T>
 - Optional[T] -> std::optional<T>
 - Callable -> std::function<R(Args...)>
-- Any -> auto or py::object
-'''
+
+{unconvertible_guidance}'''
     else:
         direction = "C++ to Python"
         source_lang = "cpp"
@@ -6559,37 +6744,60 @@ TYPE CONVERSIONS:
     # Build output format based on mode and no_header flag
     if mode == 'py_to_cpp':
         if no_header:
-            output_format = f'''OUTPUT FORMAT (NO HEADER - .cpp only):
-1. Output ONLY the .cpp file content:
+            output_format = f'''OUTPUT FORMAT (--no-h flag is set, NO HEADER FILE):
+**CRITICAL**: Output ONLY a single .cpp file. NO .h file allowed.
+
+Output the implementation file (extension MUST be .cpp):
 ```cpp
 // {module_name}.cpp
-<full cpp content with all declarations and implementations>
+#include <string>
+#include <vector>
+// ... other includes ...
+
+namespace {namespace} {{
+
+// All declarations AND implementations go here
+// No forward declarations needed since everything is in one file
+
+}} // namespace {namespace}
 ```
 
-IMPORTANT: Do NOT output a header file. Put everything in .cpp.'''
+REMEMBER:
+- File extension is .cpp (NOT .cp, NOT .cc)
+- DO NOT create a .h header file
+- Put ALL code in the single .cpp file'''
         else:
             output_format = f'''OUTPUT FORMAT (with header):
-1. First output the .cpp file content:
+1. First output the implementation file (extension MUST be .cpp):
 ```cpp
 // {module_name}.cpp
-<full cpp content>
+#include "{module_name}.h"
+// ... implementations ...
 ```
 
-2. Then the header file (REQUIRED):
+2. Then output the header file (extension MUST be .h):
 ```cpp
 // {module_name}.h
-<full header content>
-```'''
+#ifndef {module_name.upper()}_H
+#define {module_name.upper()}_H
+// ... declarations ...
+#endif // {module_name.upper()}_H
+```
+
+REMEMBER:
+- Implementation file extension is .cpp (NOT .cp, NOT .cc)
+- Header file extension is .h (NOT .hpp, NOT .hh)'''
     else:
         output_format = f'''OUTPUT FORMAT:
-Output the Python file:
+Output the Python file (extension MUST be .py):
 ```python
 # {module_name}.py
 <full python content>
 ```'''
 
     prompt = f'''Convert the following {direction}.
-
+You are IncludeCPP's AI code converter. Follow all requirements precisely.
+{includecpp_docs}
 MODULE NAME: {module_name}
 NAMESPACE: {namespace}
 {specific_rules}
@@ -6602,6 +6810,10 @@ SOURCE CODE ({source_lang}):
 
 {output_format}
 
+If you need clarification about how to convert specific modules or patterns:
+CLARIFICATION_NEEDED:
+- <specific question about conversion approach>
+
 At the end, list any API changes:
 API_CHANGES:
 - <change description>
@@ -6921,5 +7133,17 @@ def cppy_types():
     click.echo("=" * 60)
 
 
+# ============================================================================
+# Conditional Registration of Experimental Commands
+# ============================================================================
+# AI and CPPY commands are only available when experimental features are enabled
+# Enable via: includecpp settings -> "Enable Experimental Features" checkbox
+
+if _EXPERIMENTAL_ENABLED:
+    cli.add_command(ai)
+    cli.add_command(cppy)
+    cli.add_command(project)
+
+
 if __name__ == '__main__':
     cli()

includecpp/core/ai_integration.py

@@ -1,11 +1,33 @@
 import json
 import os
+import sys
 import stat
 import requests
 from pathlib import Path
 from datetime import datetime
 from typing import Optional, Dict, List, Tuple, Any
 
+
+def _supports_unicode():
+    """Check if terminal supports Unicode output."""
+    if sys.platform == 'win32':
+        try:
+            '✓✗❌'.encode(sys.stdout.encoding or 'utf-8')
+            return True
+        except (UnicodeEncodeError, LookupError, AttributeError):
+            return False
+    return True
+
+
+_UNICODE_OK = _supports_unicode()
+
+# Unicode symbols with ASCII fallbacks
+SYM_CHECK = '✓' if _UNICODE_OK else '[OK]'
+SYM_CROSS = '✗' if _UNICODE_OK else '[X]'
+SYM_ERROR = '❌' if _UNICODE_OK else '[ERR]'
+SYM_ARROW = '→' if _UNICODE_OK else '->'
+SYM_BULLET = '•' if _UNICODE_OK else '*'
+
 MODELS = {
     'gpt-3.5-turbo': {'context': 16385, 'endpoint': 'gpt-3.5-turbo'},
     'gpt-4-turbo': {'context': 128000, 'endpoint': 'gpt-4-turbo'},
@@ -1759,29 +1781,29 @@ class AIVerboseOutput:
         'white': '\033[37m',
     }
 
-    # Status icons and messages
+    # Status icons and messages - with ASCII fallbacks for Windows
     PHASES = {
-        'init': ('⚙', 'cyan', 'Initializing'),
-        'context': ('📋', 'blue', 'Building context'),
-        'thinking': ('🧠', 'magenta', 'Thinking'),
-        'planning': ('📝', 'yellow', 'Planning'),
-        'analyzing': ('🔍', 'cyan', 'Analyzing'),
-        'generating': ('✨', 'green', 'Generating'),
-        'writing': ('📄', 'blue', 'Writing'),
-        'editing': ('✏️', 'yellow', 'Editing'),
-        'reading': ('👁', 'cyan', 'Reading'),
-        'searching': ('🔎', 'blue', 'Searching'),
-        'executing': ('⚡', 'magenta', 'Executing'),
-        'converting': ('🔄', 'cyan', 'Converting'),
-        'optimizing': ('⚡', 'green', 'Optimizing'),
-        'websearch': ('🌐', 'blue', 'Web searching'),
-        'parsing': ('📊', 'cyan', 'Parsing response'),
-        'applying': ('💾', 'green', 'Applying changes'),
-        'complete': ('✅', 'green', 'Complete'),
-        'error': ('❌', 'red', 'Error'),
-        'warning': ('⚠️', 'yellow', 'Warning'),
-        'waiting': ('⏳', 'dim', 'Waiting for API'),
-        'tool': ('🔧', 'cyan', 'Running tool'),
+        'init': ('*' if not _UNICODE_OK else '⚙', 'cyan', 'Initializing'),
+        'context': ('*' if not _UNICODE_OK else '📋', 'blue', 'Building context'),
+        'thinking': ('*' if not _UNICODE_OK else '🧠', 'magenta', 'Thinking'),
+        'planning': ('*' if not _UNICODE_OK else '📝', 'yellow', 'Planning'),
+        'analyzing': ('>' if not _UNICODE_OK else '🔍', 'cyan', 'Analyzing'),
+        'generating': ('+' if not _UNICODE_OK else '✨', 'green', 'Generating'),
+        'writing': ('>' if not _UNICODE_OK else '📄', 'blue', 'Writing'),
+        'editing': ('>' if not _UNICODE_OK else '✏️', 'yellow', 'Editing'),
+        'reading': ('>' if not _UNICODE_OK else '👁', 'cyan', 'Reading'),
+        'searching': ('>' if not _UNICODE_OK else '🔎', 'blue', 'Searching'),
+        'executing': ('!' if not _UNICODE_OK else '⚡', 'magenta', 'Executing'),
+        'converting': ('~' if not _UNICODE_OK else '🔄', 'cyan', 'Converting'),
+        'optimizing': ('!' if not _UNICODE_OK else '⚡', 'green', 'Optimizing'),
+        'websearch': ('@' if not _UNICODE_OK else '🌐', 'blue', 'Web searching'),
+        'parsing': ('>' if not _UNICODE_OK else '📊', 'cyan', 'Parsing response'),
+        'applying': ('+' if not _UNICODE_OK else '💾', 'green', 'Applying changes'),
+        'complete': ('[OK]' if not _UNICODE_OK else '✅', 'green', 'Complete'),
+        'error': ('[ERR]' if not _UNICODE_OK else '❌', 'red', 'Error'),
+        'warning': ('[!]' if not _UNICODE_OK else '⚠️', 'yellow', 'Warning'),
+        'waiting': ('...' if not _UNICODE_OK else '⏳', 'dim', 'Waiting for API'),
+        'tool': ('#' if not _UNICODE_OK else '🔧', 'cyan', 'Running tool'),
     }
 
     def __init__(self, enabled: bool = True, use_colors: bool = True):
@@ -1893,7 +1915,9 @@ class AIVerboseOutput:
         pct = int((current / total) * 100) if total > 0 else 0
         bar_width = 30
         filled = int(bar_width * current / total) if total > 0 else 0
-        bar = "█" * filled + "░" * (bar_width - filled)
+        fill_char = '#' if not _UNICODE_OK else '█'
+        empty_char = '-' if not _UNICODE_OK else '░'
+        bar = fill_char * filled + empty_char * (bar_width - filled)
         label_str = f" {label}" if label else ""
         print(f"\r{indent}  [{self._color(bar, 'cyan')}] {pct}%{label_str}", end='', flush=True)
         if current >= total:
@@ -1917,10 +1941,10 @@ class AIVerboseOutput:
             return
         indent = self._get_indent()
         if success:
-            icon = "✓"
+            icon = SYM_CHECK
             color = 'green'
         else:
-            icon = "✗"
+            icon = SYM_CROSS
             color = 'red'
         msg = message[:60] + "..." if message and len(message) > 60 else (message or "")
         print(f"{indent}    {self._color(icon, color)} {msg}")
@@ -2064,15 +2088,26 @@ class AIVerboseOutput:
         if not self.enabled:
             return
         indent = self._get_indent()
-        ops = {
-            'read': ('👁', 'Reading'),
-            'write': ('📝', 'Writing'),
-            'edit': ('✏️', 'Editing'),
-            'delete': ('🗑', 'Deleting'),
-            'create': ('📁', 'Creating'),
-        }
-        icon, label = ops.get(operation, ('📄', operation))
-        status = self._color('✓', 'green') if success else self._color('✗', 'red')
+        if _UNICODE_OK:
+            ops = {
+                'read': ('👁', 'Reading'),
+                'write': ('📝', 'Writing'),
+                'edit': ('✏️', 'Editing'),
+                'delete': ('🗑', 'Deleting'),
+                'create': ('📁', 'Creating'),
+            }
+            default_icon = '📄'
+        else:
+            ops = {
+                'read': ('>', 'Reading'),
+                'write': ('>', 'Writing'),
+                'edit': ('>', 'Editing'),
+                'delete': ('x', 'Deleting'),
+                'create': ('+', 'Creating'),
+            }
+            default_icon = '>'
+        icon, label = ops.get(operation, (default_icon, operation))
+        status = self._color(SYM_CHECK, 'green') if success else self._color(SYM_CROSS, 'red')
         # Truncate path if too long
         display_path = path if len(path) <= 40 else "..." + path[-37:]
         print(f"{indent}  {icon} {label}: {display_path} {status}")