Skip to content

Commit f8a04c3

Browse files
Fix missing references - issue #200
Root cause 1: SKILL-STRUCTURE-AND-ROUTING.md referenced but doesn't exist - SkillSystem.md is the canonical skill structure document - Updated 4 references in CONSTITUTION.md and BrightData/SKILL.md - Changed SKILL-STRUCTURE-AND-ROUTING.md → SkillSystem.md Root cause 2: skill-workflow-notification script missing from PAI - Script exists in Kai (~/.claude/Tools/SkillWorkflowNotification) - Copied SkillWorkflowNotification to PAI .claude/Tools/ - Updated SKILL.md and SkillSystem.md with correct script path - Fixed lowercase tools/ → Tools/ (proper Title Case) - Enhanced workflow routing instructions with dual notification pattern Files changed: - .claude/Skills/CORE/CONSTITUTION.md (3 references updated) - .claude/Skills/BrightData/SKILL.md (1 reference updated) - .claude/Skills/CORE/SKILL.md (path and instructions updated) - .claude/Skills/CORE/SkillSystem.md (path and instructions updated) - .claude/Tools/SkillWorkflowNotification (copied from Kai) Fixes #200
1 parent 90c99b9 commit f8a04c3

File tree

5 files changed

+45
-13
lines changed

5 files changed

+45
-13
lines changed

.claude/Skills/BrightData/SKILL.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -149,7 +149,7 @@ Skill Response:
149149
---
150150

151151
**Related Documentation:**
152-
- `${PAI_DIR}/Skills/CORE/SKILL-STRUCTURE-AND-ROUTING.md` - Canonical structure guide
152+
- `${PAI_DIR}/Skills/CORE/SkillSystem.md` - Canonical structure guide
153153
- `${PAI_DIR}/Skills/CORE/CONSTITUTION.md` - Overall Kai philosophy
154154

155155
**Last Updated:** 2025-11-23

.claude/Skills/CORE/CONSTITUTION.md

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -457,7 +457,7 @@ Level 4: Workflow Invocation (Which specific procedure?)
457457
Execution
458458
```
459459

460-
**For complete routing guide, see:** `${PAI_DIR}/Skills/CORE/SKILL-STRUCTURE-AND-ROUTING.md`
460+
**For complete routing guide, see:** `${PAI_DIR}/Skills/CORE/SkillSystem.md`
461461

462462
---
463463

@@ -1317,7 +1317,7 @@ read ${PAI_DIR}/Skills/CORE/SKILL.md
13171317
4. Reference files for deep dives
13181318
5. Assets/examples for templates
13191319

1320-
**Reference:** `${PAI_DIR}/Skills/CORE/SKILL-STRUCTURE-AND-ROUTING.md`
1320+
**Reference:** `${PAI_DIR}/Skills/CORE/SkillSystem.md`
13211321

13221322
### Adding Workflows
13231323

@@ -1486,7 +1486,7 @@ voice_id: [ElevenLabs voice ID]
14861486
## Related Documentation
14871487

14881488
**For implementation details, see:**
1489-
- Skill structure patterns: `SKILL-STRUCTURE-AND-ROUTING.md`
1489+
- Skill structure patterns: `SkillSystem.md`
14901490
- CLI-First detailed guide: `cli-first-architecture.md`
14911491
- MCP strategy full details: `mcp-strategy.md`
14921492
- Testing comprehensive guide: `TESTING.md`

.claude/Skills/CORE/SKILL.md

Lines changed: 11 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -9,13 +9,19 @@ description: PAI (Personal AI Infrastructure) - Your AI system core. AUTO-LOADS
99

1010
## Workflow Routing
1111

12-
**When executing a workflow, call the notification script via Bash:**
12+
**When executing a workflow, do BOTH of these:**
1313

14-
```bash
15-
${PAI_DIR}/tools/skill-workflow-notification WorkflowName CORE
16-
```
14+
1. **Call the notification script** (for observability tracking):
15+
```bash
16+
~/.claude/Tools/SkillWorkflowNotification WORKFLOWNAME SKILLNAME
17+
```
18+
19+
2. **Output the text notification** (for user visibility):
20+
```
21+
Running the **WorkflowName** workflow from the **SKILLNAME** skill...
22+
```
1723

18-
This emits the notification AND enables dashboards to detect workflow activations.
24+
This ensures workflows appear in the observability dashboard AND the user sees the announcement.
1925

2026
| Action | Trigger | Behavior |
2127
|--------|---------|----------|

.claude/Skills/CORE/SkillSystem.md

Lines changed: 12 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -59,11 +59,19 @@ description: [What it does]. USE WHEN [intent triggers using OR]. [Additional ca
5959

6060
## Workflow Routing
6161

62-
**When executing a workflow, call the notification script via Bash:**
62+
**When executing a workflow, do BOTH of these:**
6363

64-
```bash
65-
${PAI_DIR}/tools/skill-workflow-notification WorkflowName SkillName
66-
```
64+
1. **Call the notification script** (for observability tracking):
65+
```bash
66+
~/.claude/Tools/SkillWorkflowNotification WORKFLOWNAME SKILLNAME
67+
```
68+
69+
2. **Output the text notification** (for user visibility):
70+
```
71+
Running the **WorkflowName** workflow from the **SKILLNAME** skill...
72+
```
73+
74+
This ensures workflows appear in the observability dashboard AND the user sees the announcement.
6775
6876
| Workflow | Trigger | File |
6977
|----------|---------|------|
Lines changed: 18 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,18 @@
1+
#!/bin/bash
2+
# skill-workflow-notification - Called when a skill workflow is invoked
3+
# This script exists primarily as a signal for the observability system
4+
# which detects workflow invocations by watching for calls to this script.
5+
#
6+
# Usage: skill-workflow-notification WORKFLOWNAME SKILLNAME
7+
#
8+
# Example: skill-workflow-notification GIT CORE
9+
#
10+
# The observability system parses Bash tool calls for this pattern:
11+
# /skill-workflow-notification\s+(\w+)\s+(\w+)
12+
# and extracts workflow and skill names for display.
13+
14+
WORKFLOW_NAME="${1:-unknown}"
15+
SKILL_NAME="${2:-unknown}"
16+
17+
# Output confirmation (captured by observability)
18+
echo "Workflow invoked: ${WORKFLOW_NAME} from ${SKILL_NAME}"

0 commit comments

Comments
 (0)