Merge pull request #2 from ryanmac/fix/installer-pyenv-poetry-detection

Fix pyenv/Poetry detection in universal installer

Author: ryanmac

README.md

@@ -21,9 +21,9 @@
 
 **One command. Instant AI coordination.**
 
-**Prerequisites for all options:** Git, Python 3.9+, curl (for one-liner), and tar. Run from the root of an existing Git repository.
+**Prerequisites for all options:** Git, Python 3.9-3.12, curl (for one-liner), and tar. Run from the root of an existing Git repository. **If using pyenv, ensure your active Python version (e.g., via `pyenv shell 3.12.x`) has Poetry installed if you prefer it; otherwise, the script falls back to pip.**
 
-### Option 1: Universal One-Liner (Recommended - No Cloning Required)
+### **Option 1: Universal One-Liner (Recommended - No Cloning Required)**
 Run this in your existing project's root directory to download and install Conductor-Score directly:
 
 ```bash
@@ -33,8 +33,9 @@ bash <(curl -fsSL https://raw.githubusercontent.com/ryanmac/conductor-score/main
 - This method avoids cloning the full Conductor-Score repo and is ideal for integrating into existing projects without repository pollution.
 - The script will prompt before overwriting any existing installation.
 - **Security best practice:** Review the script at the raw URL before running.
+- **Pyenv users:** If Poetry install fails, switch to the Python version that has Poetry installed (e.g., `pyenv shell 3.10.13`) and re-run.
 
-### Option 2: Poetry (Manual Clone)
+### Option 2: Poetry (For Cloned Repo)
 ```bash
 # Clone the repository
 git clone https://github.com/ryanmac/conductor-score.git
@@ -45,7 +46,7 @@ poetry install
 poetry run python setup.py
 ```
 
-### Option 3: Pip + Virtual Environment
+### Option 3: Pip + Virtual Environment (For Cloned Repo)
 ```bash
 # Clone the repository
 git clone https://github.com/ryanmac/conductor-score.git
@@ -62,7 +63,7 @@ pip install -r requirements.txt
 python setup.py
 ```
 
-### Option 4: One-Command Install Script (from cloned repo)
+### Option 4: One-Command Install Script (For Cloned Repo)
 ```bash
 # From the repository directory:
 ./install.sh
@@ -344,7 +345,7 @@ python .conductor/scripts/validate-config.py
 ## Development Setup
 
 ### Prerequisites
-- Python 3.9+
+- Python 3.9-3.12
 - Git
 - GitHub CLI (optional, for issue integration)
 
@@ -375,7 +376,7 @@ poetry run black --check .conductor/scripts/ setup.py
 ### CI/CD
 The project uses GitHub Actions for continuous integration:
 - **Linting**: flake8 and black formatting checks
-- **Testing**: pytest on multiple Python versions (3.9, 3.10, 3.11)
+- **Testing**: pytest on multiple Python versions (3.9, 3.10, 3.11, 3.12)
 - **Security**: safety vulnerability scanning
 - **Platforms**: Ubuntu and macOS
 

conductor-init.sh

@@ -33,9 +33,9 @@ if ! command -v git >/dev/null 2>&1; then
     exit 1
 fi
 
-# Check for Python 3.9+
-if ! command -v python3 >/dev/null 2>&1 || ! python3 -c "import sys; exit(0 if sys.version_info >= (3,9) else 1)"; then
-    echo -e "${RED}❌ Error: Python 3.9+ is required. Please install Python 3.9 or higher.${NC}"
+# Check for Python 3.9-3.12
+if ! command -v python3 >/dev/null 2>&1 || ! python3 -c "import sys; exit(0 if sys.version_info >= (3,9) and sys.version_info < (3,13) else 1)"; then
+    echo -e "${RED}❌ Error: Python 3.9-3.12 is required. Please install Python 3.9, 3.10, 3.11, or 3.12.${NC}"
     exit 1
 fi
 
@@ -51,6 +51,11 @@ if ! command -v tar >/dev/null 2>&1; then
     exit 1
 fi
 
+# **Improved: Check for pyenv and suggest version switch if Poetry fails later**
+if command -v pyenv >/dev/null 2>&1; then
+    echo -e "${YELLOW}⚠️ pyenv detected. Ensure your active Python version has Poetry installed if using it.${NC}"
+fi
+
 # Check for existing installation
 if [ -d ".conductor" ]; then
     echo -e "${YELLOW}⚠️ Existing .conductor directory found.${NC}"
@@ -127,15 +132,21 @@ echo ""
 # Step 3: Install Dependencies
 echo -e "${YELLOW}📦 Installing dependencies...${NC}"
 
-# Prefer Poetry if available, otherwise use pip + venv
-if command -v poetry >/dev/null 2>&1; then
-    echo "🎵 Poetry detected. Using Poetry for installation."
+# **Improved: Check if Poetry is functional before using it**
+POETRY_AVAILABLE=false
+if command -v poetry >/dev/null 2>&1 && poetry --version >/dev/null 2>&1; then
+    POETRY_AVAILABLE=true
+fi
+
+# Prefer Poetry if available and functional, otherwise use pip + venv
+if $POETRY_AVAILABLE; then
+    echo "🎵 Poetry detected and functional. Using Poetry for installation."
     poetry install || {
-        echo -e "${RED}❌ Poetry install failed.${NC}"
+        echo -e "${RED}❌ Poetry install failed. If using pyenv, try switching versions (e.g., pyenv shell 3.10.13) and re-run.${NC}"
         exit 1
     }
 else
-    echo "📦 Poetry not found. Using pip and virtual environment."
+    echo "📦 Poetry not found or not functional. Using pip and virtual environment."
     python3 -m venv .venv || {
         echo -e "${RED}❌ Failed to create virtual environment.${NC}"
         exit 1
@@ -158,7 +169,7 @@ echo ""
 echo -e "${YELLOW}🔧 Running automatic setup...${NC}"
 
 # Run setup.py with --auto flag
-if command -v poetry >/dev/null 2>&1; then
+if $POETRY_AVAILABLE; then
     poetry run python setup.py --auto || {
         echo -e "${RED}❌ Setup failed.${NC}"
         exit 1

pyproject.toml

@@ -7,7 +7,7 @@ readme = "README.md"
 packages = [{include = "conductor_score"}]
 
 [tool.poetry.dependencies]
-python = "^3.9"
+python = "^3.9,<3.13"
 pyyaml = "^6.0"
 requests = "^2.28.0"
 
@@ -25,7 +25,7 @@ build-backend = "poetry.core.masonry.api"
 
 [tool.black]
 line-length = 88
-target-version = ['py39']
+target-version = ['py39', 'py310', 'py311', 'py312']
 
 [tool.flake8]
 max-line-length = 88

setup.py

@@ -565,7 +565,7 @@ def create_github_workflows(self):
       - name: Setup Python
         uses: actions/setup-python@v4
         with:
-          python-version: '3.9'
+          python-version: '3.12'
 
       - name: Convert Issue to Task
         run: |
@@ -587,7 +587,7 @@ def create_github_workflows(self):
       - name: Setup Python
         uses: actions/setup-python@v4
         with:
-          python-version: '3.9'
+          python-version: '3.12'
 
       - name: Install dependencies
         run: |
@@ -639,7 +639,7 @@ def create_github_workflows(self):
       - name: Setup Python
         uses: actions/setup-python@v4
         with:
-          python-version: '3.9'
+          python-version: '3.12'
 
       - name: Install dependencies
         run: |

tests/test_basic.py

@@ -6,8 +6,9 @@
 
 
 def test_python_version():
-    """Test that we're running Python 3.9+"""
-    assert sys.version_info >= (3, 9), f"Python 3.9+ required, got {sys.version}"
+    """Test that we're running Python 3.9-3.12"""
+    assert sys.version_info >= (3, 9), f"Python 3.9-3.12 required, got {sys.version}"
+    assert sys.version_info < (3, 13), f"Python 3.13+ not yet supported, got {sys.version}"
 
 
 def test_dependencies():