-
Notifications
You must be signed in to change notification settings - Fork 284
fix: correct script path references in skills and agents #138
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -56,14 +56,14 @@ To create a new MCP chain script and wrap it as a skill: | |
| Copy the multi-tool-pipeline template: | ||
|
|
||
| ```bash | ||
| cp $CLAUDE_PROJECT_DIR/scripts/multi_tool_pipeline.py $CLAUDE_PROJECT_DIR/scripts/my_pipeline.py | ||
| cp $CLAUDE_OPC_DIR/scripts/multi_tool_pipeline.py $CLAUDE_OPC_DIR/scripts/my_pipeline.py | ||
| ``` | ||
|
|
||
| Reference the template pattern: | ||
|
|
||
| ```bash | ||
| cat $CLAUDE_PROJECT_DIR/.claude/skills/multi-tool-pipeline/SKILL.md | ||
| cat $CLAUDE_PROJECT_DIR/scripts/multi_tool_pipeline.py | ||
| cat $CLAUDE_CC_DIR/.claude/skills/multi-tool-pipeline/SKILL.md | ||
| cat $CLAUDE_OPC_DIR/scripts/multi_tool_pipeline.py | ||
| ``` | ||
|
|
||
| ### Step 2: Customize the Script | ||
|
|
@@ -140,8 +140,8 @@ Add to `.claude/skills/skill-rules.json`: | |
| For full details, read: | ||
|
|
||
| ```bash | ||
| cat $CLAUDE_PROJECT_DIR/.claude/rules/skill-development.md | ||
| cat $CLAUDE_PROJECT_DIR/.claude/rules/mcp-scripts.md | ||
| cat $CLAUDE_CC_DIR/.claude/rules/skill-development.md | ||
| cat $CLAUDE_CC_DIR/.claude/rules/mcp-scripts.md | ||
|
Comment on lines
+143
to
+144
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 🧩 Analysis chain🏁 Script executed: #!/bin/bash
# Verify reference documentation files exist
echo "=== Searching for skill-development.md ==="
fd --type f --full-path '.claude/rules/skill-development.md$' 2>&1 || echo "Not found"
echo ""
echo "=== Searching for mcp-scripts.md ==="
fd --type f --full-path '.claude/rules/mcp-scripts.md$' 2>&1 || echo "Not found"
echo ""
echo "=== All files under .claude/rules directory ==="
fd --type f '.claude/rules/' 2>&1 || echo "Directory not found"Repository: parcadei/Continuous-Claude-v3 Length of output: 584 🏁 Script executed: #!/bin/bash
# Broader search to understand the structure
echo "=== Checking if .claude/rules directory exists ==="
fd . '.claude/rules/' 2>&1 | head -20
echo ""
echo "=== Finding SKILL.md file ==="
fd --type f 'SKILL.md$' 2>&1
echo ""
echo "=== All files under .claude directory ==="
fd . '.claude/' 2>&1 | head -30
echo ""
echo "=== Searching for skill-development or skill development files ==="
fd --type f -i 'skill.*development' 2>&1
echo ""
echo "=== Searching for mcp-scripts files ==="
fd --type f -i 'mcp.*scripts' 2>&1Repository: parcadei/Continuous-Claude-v3 Length of output: 1560 🏁 Script executed: #!/bin/bash
# More comprehensive search for SKILL.md
echo "=== Recursive search for any SKILL.md file ==="
find . -type f -name 'SKILL.md' 2>/dev/null
echo ""
echo "=== Searching in skill-developer directory ==="
fd . 'skill-developer' 2>&1 | head -30
echo ""
echo "=== All markdown files in .claude ==="
find .claude -type f -name '*.md' 2>/dev/null | sortRepository: parcadei/Continuous-Claude-v3 Length of output: 17131 🏁 Script executed: #!/bin/bash
# Check the specific lines in skill-developer SKILL.md
echo "=== Lines around 143-144 in ./.claude/skills/skill-developer/SKILL.md ==="
sed -n '140,150p' ./.claude/skills/skill-developer/SKILL.md
echo ""
echo "=== Total line count ==="
wc -l ./.claude/skills/skill-developer/SKILL.mdRepository: parcadei/Continuous-Claude-v3 Length of output: 468 Create missing reference documentation files or update paths to existing documentation. The referenced files do not exist at the specified paths:
Either create these files in 🤖 Prompt for AI Agents |
||
| ``` | ||
|
|
||
| ## Quick Checklist | ||
|
|
@@ -157,7 +157,7 @@ cat $CLAUDE_PROJECT_DIR/.claude/rules/mcp-scripts.md | |
| Look at existing skills for patterns: | ||
|
|
||
| ```bash | ||
| ls $CLAUDE_PROJECT_DIR/.claude/skills/ | ||
| cat $CLAUDE_PROJECT_DIR/.claude/skills/commit/SKILL.md | ||
| cat $CLAUDE_PROJECT_DIR/.claude/skills/firecrawl-scrape/SKILL.md | ||
| ls $CLAUDE_CC_DIR/.claude/skills/ | ||
| cat $CLAUDE_CC_DIR/.claude/skills/commit/SKILL.md | ||
| cat $CLAUDE_CC_DIR/.claude/skills/firecrawl-scrape/SKILL.md | ||
| ``` | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -836,8 +836,10 @@ async def run_setup_wizard() -> None: | |
| else: | ||
| console.print(" Skipped integration installation") | ||
|
|
||
| # Set CLAUDE_OPC_DIR environment variable for skills to find scripts | ||
| console.print(" Setting CLAUDE_OPC_DIR environment variable...") | ||
| # Set CLAUDE_OPC_DIR and CLAUDE_CC_DIR environment variables | ||
| # CLAUDE_OPC_DIR = opc/ directory (Python scripts, MCP runtime) | ||
| # CLAUDE_CC_DIR = repository root (contains .claude/scripts/, .claude/skills/) | ||
| console.print(" Setting environment variables...") | ||
| shell_config = None | ||
| shell = os.environ.get("SHELL", "") | ||
| if "zsh" in shell: | ||
|
|
@@ -846,21 +848,32 @@ async def run_setup_wizard() -> None: | |
| shell_config = Path.home() / ".bashrc" | ||
|
|
||
| opc_dir = _project_root # Use script location, not cwd (robust if invoked from elsewhere) | ||
| cc_dir = opc_dir.parent # Repository root (Continuous-Claude-v3/) | ||
| if shell_config and shell_config.exists(): | ||
| content = shell_config.read_text() | ||
| export_line = f'export CLAUDE_OPC_DIR="{opc_dir}"' | ||
| changed = False | ||
| export_opc = f'export CLAUDE_OPC_DIR="{opc_dir}"' | ||
| export_cc = f'export CLAUDE_CC_DIR="{cc_dir}"' | ||
| if "CLAUDE_OPC_DIR" not in content: | ||
| with open(shell_config, "a") as f: | ||
| f.write(f"\n# Continuous-Claude OPC directory (for skills to find scripts)\n{export_line}\n") | ||
| console.print(f" [green]OK[/green] Added CLAUDE_OPC_DIR to {shell_config.name}") | ||
| f.write(f"\n# Continuous-Claude directories (for skills to find scripts)\n{export_opc}\n{export_cc}\n") | ||
| changed = True | ||
| elif "CLAUDE_CC_DIR" not in content: | ||
| with open(shell_config, "a") as f: | ||
| f.write(f"\n# Continuous-Claude root directory\n{export_cc}\n") | ||
| changed = True | ||
| if changed: | ||
| console.print(f" [green]OK[/green] Added CLAUDE_OPC_DIR and CLAUDE_CC_DIR to {shell_config.name}") | ||
| else: | ||
| console.print(f" [dim]CLAUDE_OPC_DIR already in {shell_config.name}[/dim]") | ||
| console.print(f" [dim]CLAUDE_OPC_DIR and CLAUDE_CC_DIR already in {shell_config.name}[/dim]") | ||
| elif sys.platform == "win32": | ||
| console.print(" [yellow]NOTE[/yellow] Add to your environment:") | ||
| console.print(f' set CLAUDE_OPC_DIR="{opc_dir}"') | ||
| console.print(f' set CLAUDE_CC_DIR="{cc_dir}"') | ||
|
Comment on lines
870
to
+872
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Bug: The setup wizard provides Suggested FixDetect if the user is on Windows and provide PowerShell-specific instructions using Prompt for AI AgentDid we get this right? 👍 / 👎 to inform future reviews. |
||
| else: | ||
| console.print(" [yellow]NOTE[/yellow] Add to your shell config:") | ||
| console.print(f' export CLAUDE_OPC_DIR="{opc_dir}"') | ||
| console.print(f' export CLAUDE_CC_DIR="{cc_dir}"') | ||
|
|
||
| # Step 8: Math Features (Optional) | ||
| console.print("\n[bold]Step 9/13: Math Features (Optional)[/bold]") | ||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🧩 Analysis chain
🏁 Script executed:
Repository: parcadei/Continuous-Claude-v3
Length of output: 371
🏁 Script executed:
Repository: parcadei/Continuous-Claude-v3
Length of output: 1559
🏁 Script executed:
Repository: parcadei/Continuous-Claude-v3
Length of output: 100
🏁 Script executed:
Repository: parcadei/Continuous-Claude-v3
Length of output: 55
🏁 Script executed:
Repository: parcadei/Continuous-Claude-v3
Length of output: 1059
🏁 Script executed:
Repository: parcadei/Continuous-Claude-v3
Length of output: 2337
🏁 Script executed:
Repository: parcadei/Continuous-Claude-v3
Length of output: 55
🏁 Script executed:
Repository: parcadei/Continuous-Claude-v3
Length of output: 1214
Multiple referenced files do not exist; instructions are broken.
The path updates use correct environment variables (confirmed:
CLAUDE_CC_DIRandCLAUDE_OPC_DIRare defined in wizard.py), but the referenced files are missing:$CLAUDE_CC_DIR/.claude/skills/multi-tool-pipeline/SKILL.md— directory does not exist$CLAUDE_CC_DIR/.claude/rules/skill-development.md— file does not exist (actual rules are: agent-memory-recall.md, agent-model-selection.md, claim-verification.md, etc.)$CLAUDE_CC_DIR/.claude/rules/mcp-scripts.md— file does not existcommit/SKILL.mdandfirecrawl-scrape/SKILL.md— neither directory existsAdditionally, line 85 has a duplicate heading: "### Step 2" should be "### Step 3".
Users following these instructions will encounter file-not-found errors. Either create the missing template/example files or correct the references.
🤖 Prompt for AI Agents