Skip to content

Commit 149dc91

Browse files
ryanmacclaude
andcommitted
fix: Clarify token requirements and Python dependency scope
Based on user feedback, this commit addresses confusion about: 1. GitHub Token Requirements: - Emphasize that NO CONDUCTOR_GITHUB_TOKEN setup is needed - Workflows use GitHub's built-in authentication - Add clear messaging throughout installation process 2. Python Dependency Clarification: - Python is only needed for conductor scripts, NOT for CI/CD - Code Conductor does NOT add Python test/lint workflows - Language-agnostic workflows only 3. Documentation Improvements: - Add comprehensive FAQ addressing common concerns - Update installer with clearer messaging - Enhance README to highlight no-token setup This ensures users understand that Code Conductor: - Uses GitHub's built-in auth (no manual tokens) - Doesn't impose language-specific CI/CD - Only requires Python for orchestration scripts 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>
1 parent 4ed1c81 commit 149dc91

File tree

4 files changed

+115
-4
lines changed

4 files changed

+115
-4
lines changed

README.md

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,13 @@ One command. 60 seconds. Done.
2626
bash <(curl -fsSL https://raw.githubusercontent.com/ryanmac/code-conductor/main/conductor-init.sh)
2727
```
2828

29-
The installer auto-detects your stack (React, Python, Go, etc.) and configures everything. No GitHub tokens. No complex setup.
29+
The installer auto-detects your stack (React, Python, Go, etc.) and configures everything. **No GitHub tokens needed. No complex setup.**
30+
31+
**What you get:**
32+
- ✅ GitHub Actions workflows that use built-in authentication
33+
- ✅ No manual token creation or repository secrets required
34+
- ✅ AI code reviews work automatically on all PRs
35+
- ✅ Language-agnostic setup (no Python CI/CD added to non-Python projects)
3036

3137
**Or let Claude Code install it for you:**
3238
```
@@ -65,6 +71,7 @@ gh issue create --label "conductor:task" --title "Add dark mode toggle"
6571
- [Stack Support](docs/STACK_SUPPORT.md) - Works with React, Vue, Python, Go, and more
6672
- [Architecture](docs/ARCHITECTURE.md) - How it all works under the hood
6773
- [AI Code Review](docs/AI_CODE_REVIEW.md) - Smart, opt-in PR reviews
74+
- [FAQ](docs/FAQ.md) - Common questions about tokens, Python, and workflows
6875
- [Troubleshooting](docs/TROUBLESHOOTING.md) - Common issues and solutions
6976

7077
🚀 **[Power User Prompts](CLAUDE_CODE_PROMPT.md)** - Game-changing automation with Claude Code

conductor-init.sh

Lines changed: 18 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,9 @@ echo "=========================================="
1717
echo "This script will install Code Conductor into your current Git repository."
1818
echo "It will download necessary files and run the setup automatically."
1919
echo ""
20+
echo -e "${GREEN}✨ No GitHub Token Setup Required!${NC}"
21+
echo "Code Conductor uses GitHub's built-in authentication - no manual token needed."
22+
echo ""
2023

2124
# Step 1: Prerequisite Checks
2225
echo -e "${YELLOW}🔍 Checking prerequisites...${NC}"
@@ -36,6 +39,8 @@ fi
3639
# Check for Python 3.9-3.12
3740
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
3841
echo -e "${RED}❌ Error: Python 3.9-3.12 is required. Please install Python 3.9, 3.10, 3.11, or 3.12.${NC}"
42+
echo -e "${YELLOW}Note: Python is only needed for conductor scripts, NOT for your project's CI/CD.${NC}"
43+
echo -e "${YELLOW}Code Conductor does NOT add Python-specific workflows to non-Python projects.${NC}"
3944
exit 1
4045
fi
4146

@@ -323,6 +328,13 @@ if [ "$IS_UPGRADE" = false ]; then
323328

324329
echo -e "${GREEN}✅ Setup complete.${NC}"
325330
echo ""
331+
332+
# Emphasize no token requirement
333+
echo -e "${GREEN}🔐 GitHub Integration Status:${NC}"
334+
echo " ✅ Workflows configured to use GitHub's built-in token"
335+
echo " ✅ No CONDUCTOR_GITHUB_TOKEN setup required"
336+
echo " ✅ AI code reviews will work automatically on PRs"
337+
echo ""
326338
else
327339
echo -e "${GREEN}✅ Skipping setup - existing configuration preserved.${NC}"
328340
echo ""
@@ -583,7 +595,7 @@ case "$ENV_CHOICE" in
583595
echo " • Conductor will handle task claiming and worktree setup automatically"
584596
echo " • Use the built-in terminal for git operations"
585597
echo " • AI code reviews happen automatically on PRs"
586-
echo " • No GitHub token setup needed—uses built-in authentication"
598+
echo -e "${GREEN}No GitHub token setup needed—uses built-in authentication${NC}"
587599
echo ""
588600
echo "📚 Learn more: https://conductor.build"
589601
;;
@@ -661,7 +673,7 @@ if [ "$IS_UPGRADE" = true ]; then
661673
echo " • Role definitions (.conductor/roles/)"
662674
echo " • GitHub workflows (.github/workflows/)"
663675
echo " • Setup and configuration tools"
664-
echo " • Token configuration (no manual setup needed)"
676+
echo -e "${GREEN}Token configuration (uses GitHub's built-in auth)${NC}"
665677
echo ""
666678
echo "✅ What was preserved:"
667679
echo " • Your project configuration (.conductor/config.yaml)"
@@ -698,6 +710,8 @@ elif [ "$ENV_CHOICE" != "1" ]; then
698710
echo " ✅ AI code-reviewer for all PRs"
699711
echo " ✅ Specialized roles: ${CONFIGURED_ROLES}"
700712
echo " ✅ Demo tasks ready to claim"
713+
echo -e " ${GREEN}✅ No GitHub token setup required${NC}"
714+
echo -e " ${GREEN}✅ No Python CI/CD workflows added${NC}"
701715
echo ""
702716
echo -e "${YELLOW}Quick Start Commands:${NC}"
703717
echo -e " 📋 View tasks: ${GREEN}./conductor tasks${NC}"
@@ -725,7 +739,8 @@ else
725739
echo " ✅ AI code-reviewer for all PRs"
726740
echo " ✅ Specialized roles: ${CONFIGURED_ROLES}"
727741
echo " ✅ Demo tasks ready in Conductor"
728-
echo " ✅ No GitHub token setup required"
742+
echo -e " ${GREEN}✅ No GitHub token setup required${NC}"
743+
echo -e " ${GREEN}✅ No Python CI/CD workflows added${NC}"
729744
echo ""
730745
echo "📚 Documentation: https://github.com/ryanmac/code-conductor"
731746
echo "🐛 Report issues: https://github.com/ryanmac/code-conductor/issues"

docs/FAQ.md

Lines changed: 88 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,88 @@
1+
# Frequently Asked Questions
2+
3+
## Do I need to set up a CONDUCTOR_GITHUB_TOKEN?
4+
5+
**No!** Code Conductor uses GitHub's built-in authentication (`${{ github.token }}`) for all workflows. This token is automatically available in GitHub Actions with no setup required.
6+
7+
You only need a Personal Access Token (PAT) if you want:
8+
- Higher API rate limits (5,000/hour vs 1,000/hour)
9+
- Cross-repository access
10+
- Ability to trigger other workflows
11+
12+
For detailed token information, see [.conductor/GITHUB_TOKEN_SETUP.md](.conductor/GITHUB_TOKEN_SETUP.md).
13+
14+
## Why does Code Conductor require Python?
15+
16+
Python is required **only for the conductor orchestration scripts**, not for your project's CI/CD. Here's what this means:
17+
18+
-**Required**: Python 3.9-3.12 to run conductor commands
19+
-**NOT added**: Python test runners (pytest)
20+
-**NOT added**: Python linters (flake8, black)
21+
-**NOT added**: Python security scanners (safety)
22+
-**NOT added**: Any Python-specific CI/CD workflows
23+
24+
Your project's existing CI/CD remains unchanged. Code Conductor only adds three language-agnostic GitHub workflows for task orchestration.
25+
26+
## Does Code Conductor add Python CI/CD to my JavaScript/TypeScript/Go project?
27+
28+
**No!** Code Conductor is language-agnostic. It detects your project's technology stack and configures roles accordingly, but it does NOT add language-specific CI/CD workflows.
29+
30+
The only workflows added are:
31+
1. `conductor.yml` - Task orchestration and health checks
32+
2. `conductor-cleanup.yml` - Stale worktree cleanup
33+
3. `pr-review.yml` - AI code reviews (if enabled)
34+
35+
These workflows only run conductor scripts and do not test, lint, or build your code.
36+
37+
## What if I see Python test failures in my CI?
38+
39+
If you're seeing Python test failures like:
40+
```
41+
============================= test session starts ==============================
42+
collected 0 items
43+
============================ no tests ran in 0.02s =============================
44+
Error: Process completed with exit code 5.
45+
```
46+
47+
This is NOT from Code Conductor. Check if:
48+
1. You accidentally copied CI workflows from the Code Conductor repository itself
49+
2. You have existing Python CI workflows in your project
50+
3. Another tool added Python-specific workflows
51+
52+
Code Conductor's template workflows do not include any Python testing commands.
53+
54+
## Can I use Code Conductor without installing Python?
55+
56+
Currently, Python is required because the conductor scripts are written in Python. We chose Python for:
57+
- Maximum compatibility across all platforms
58+
- Easy installation via standard package managers
59+
- Reliable YAML and JSON processing
60+
- Strong GitHub API support
61+
62+
Future versions may offer alternative implementations, but Python remains the most universal choice.
63+
64+
## Do I need to manage Python dependencies for my non-Python project?
65+
66+
No, the Python dependencies are isolated to Code Conductor's functionality:
67+
- `pyyaml` - For configuration file handling
68+
- Standard library modules only
69+
70+
These are only used by conductor scripts and do not affect your project's dependencies.
71+
72+
## How do I know which workflows were added by Code Conductor?
73+
74+
Code Conductor only adds workflows from its templates:
75+
- `.github/workflows/conductor.yml`
76+
- `.github/workflows/conductor-cleanup.yml`
77+
- `.github/workflows/pr-review.yml` (if code-reviewer role is enabled)
78+
79+
Any other workflows (especially those running pytest, flake8, etc.) are from other sources.
80+
81+
## Can I remove the Python dependency from Code Conductor?
82+
83+
The conductor scripts require Python to run. However, you can:
84+
1. Run agents from the Conductor app (macOS) which handles Python internally
85+
2. Use containerized environments where Python is pre-installed
86+
3. Install Python in your CI environment only (not locally)
87+
88+
The Python requirement is minimal and doesn't affect your project's runtime or deployment.

docs/README.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,7 @@ Welcome to the comprehensive Code Conductor documentation. This directory contai
88
- [**Installation Guide**](INSTALLATION.md) - All installation methods and options
99
- [**Usage Guide**](USAGE.md) - How to use Code Conductor effectively
1010
- [**Stack Support**](STACK_SUPPORT.md) - Supported technologies and frameworks
11+
- [**FAQ**](FAQ.md) - Frequently asked questions and clarifications
1112

1213
### Core Concepts
1314
- [**Architecture**](ARCHITECTURE.md) - System design and components

0 commit comments

Comments
 (0)