@pharaoh-so/mcp 0.3.13 → 0.3.15
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/.claude-plugin/plugin.json +1 -1
- package/commands/plan.md +5 -0
- package/commands/review.md +5 -0
- package/dist/install-skills.d.ts +6 -2
- package/dist/install-skills.js +156 -42
- package/inspect-tools.json +24 -24
- package/package.json +2 -1
- package/skills/brainstorm/SKILL.md +1 -1
- package/skills/plan/SKILL.md +26 -24
- package/skills/audit-tests/SKILL.md +0 -89
- package/skills/debt/SKILL.md +0 -34
- package/skills/execute/SKILL.md +0 -58
- package/skills/explore/SKILL.md +0 -33
- package/skills/finish/SKILL.md +0 -80
- package/skills/health/SKILL.md +0 -37
- package/skills/investigate/SKILL.md +0 -35
- package/skills/onboard/SKILL.md +0 -33
- package/skills/orchestrate/SKILL.md +0 -145
- package/skills/pr/SKILL.md +0 -53
- package/skills/review-codex/SKILL.md +0 -81
- package/skills/review-receive/SKILL.md +0 -82
- package/skills/wiring/SKILL.md +0 -35
- package/skills/worktree/SKILL.md +0 -86
package/skills/wiring/SKILL.md
DELETED
|
@@ -1,35 +0,0 @@
|
|
|
1
|
-
---
|
|
2
|
-
name: wiring
|
|
3
|
-
prompt-name: validate-wiring
|
|
4
|
-
description: "Wiring validation for a codebase module using Pharaoh knowledge graph. Four-step pro-tier workflow: entry point reachability, caller verification via blast radius, test coverage per function, and dead code detection. Categorizes functions as CONNECTED, UNREACHABLE, DEAD, or UNTESTED. Produces a PASS or FAIL verdict with specific issues listed. Iron law: zero callers on a non-entry-point export means the code is not wired."
|
|
5
|
-
version: 0.2.0
|
|
6
|
-
homepage: https://pharaoh.so
|
|
7
|
-
user-invocable: true
|
|
8
|
-
metadata: {"emoji": "☥", "tags": ["wiring", "reachability", "dead-code", "pharaoh", "test-coverage", "entry-points", "validation"]}
|
|
9
|
-
---
|
|
10
|
-
|
|
11
|
-
# Validate Wiring
|
|
12
|
-
|
|
13
|
-
Verify that code in a module is properly connected. Uses `validate-wiring` — a 4-step pro-tier workflow that checks reachability from entry points, verifies functions have callers, confirms test coverage, and detects dead code. Produces a PASS / FAIL verdict with categorized findings. Requires Pro tools.
|
|
14
|
-
|
|
15
|
-
## When to Use
|
|
16
|
-
|
|
17
|
-
Invoke after adding new exports or functions, before merging code that adds a new module, or when dead code is suspected. Use it to answer "is this code actually being used?" with data instead of guesswork.
|
|
18
|
-
|
|
19
|
-
## Workflow
|
|
20
|
-
|
|
21
|
-
1. Call `check_reachability` for the target module to verify all exports are reachable from production entry points.
|
|
22
|
-
2. Call `get_blast_radius` for the target module to verify functions have callers.
|
|
23
|
-
3. Call `get_test_coverage` for the target module to check which functions are tested.
|
|
24
|
-
4. Call `get_unused_code` for the target module to detect any dead code.
|
|
25
|
-
|
|
26
|
-
Iron law: Zero callers on a non-entry-point export = the code is not wired.
|
|
27
|
-
|
|
28
|
-
## Output
|
|
29
|
-
|
|
30
|
-
A wiring report containing:
|
|
31
|
-
- **CONNECTED:** Functions reachable from entry points with test coverage
|
|
32
|
-
- **UNREACHABLE:** Exports with no path from entry points
|
|
33
|
-
- **DEAD:** Functions with zero callers (not entry points)
|
|
34
|
-
- **UNTESTED:** Reachable functions lacking test coverage
|
|
35
|
-
- **Verdict:** PASS (all exports wired and tested) or FAIL (with specific issues listed and recommended actions)
|
package/skills/worktree/SKILL.md
DELETED
|
@@ -1,86 +0,0 @@
|
|
|
1
|
-
---
|
|
2
|
-
name: worktree
|
|
3
|
-
prompt-name: git-worktree
|
|
4
|
-
description: "Set up isolated git worktrees for feature work. Smart directory selection with safety verification — checks for existing worktree directories, verifies gitignore, auto-detects project setup, and confirms clean test baseline before starting. Prevents accidentally committing worktree contents."
|
|
5
|
-
version: 0.2.0
|
|
6
|
-
homepage: https://pharaoh.so
|
|
7
|
-
user-invocable: true
|
|
8
|
-
metadata: {"emoji": "☥", "tags": ["git", "worktree", "isolation", "branching", "workspace"]}
|
|
9
|
-
---
|
|
10
|
-
|
|
11
|
-
# Git Worktrees
|
|
12
|
-
|
|
13
|
-
Create isolated workspaces sharing the same repository. Work on multiple branches simultaneously without switching.
|
|
14
|
-
|
|
15
|
-
## When to Use
|
|
16
|
-
|
|
17
|
-
Before starting feature work that needs isolation from the current workspace. Before executing implementation plans. Any time you need a clean branch without disturbing current work.
|
|
18
|
-
|
|
19
|
-
## Directory Selection
|
|
20
|
-
|
|
21
|
-
Follow this priority order:
|
|
22
|
-
|
|
23
|
-
1. **Check existing directories:** look for `.worktrees/` or `worktrees/` in the project root. If both exist, `.worktrees/` wins.
|
|
24
|
-
2. **Check project docs:** look for worktree directory preferences in CLAUDE.md or project config.
|
|
25
|
-
3. **Ask the user:** if no directory exists and no preference is documented.
|
|
26
|
-
|
|
27
|
-
## Safety Verification
|
|
28
|
-
|
|
29
|
-
For project-local worktree directories:
|
|
30
|
-
|
|
31
|
-
```bash
|
|
32
|
-
git check-ignore -q .worktrees 2>/dev/null
|
|
33
|
-
```
|
|
34
|
-
|
|
35
|
-
If the directory is NOT gitignored: add it to `.gitignore` and commit before creating worktrees. This prevents accidentally committing worktree contents to the repository.
|
|
36
|
-
|
|
37
|
-
Global directories (outside the project) don't need this check.
|
|
38
|
-
|
|
39
|
-
## Creation Steps
|
|
40
|
-
|
|
41
|
-
### 1. Create Worktree
|
|
42
|
-
|
|
43
|
-
```bash
|
|
44
|
-
git worktree add <path>/<branch-name> -b <branch-name>
|
|
45
|
-
cd <path>/<branch-name>
|
|
46
|
-
```
|
|
47
|
-
|
|
48
|
-
### 2. Install Dependencies
|
|
49
|
-
|
|
50
|
-
Auto-detect from project files:
|
|
51
|
-
|
|
52
|
-
| File | Command |
|
|
53
|
-
|------|---------|
|
|
54
|
-
| `package.json` + `pnpm-lock.yaml` | `pnpm install` |
|
|
55
|
-
| `package.json` | `npm install` |
|
|
56
|
-
| `Cargo.toml` | `cargo build` |
|
|
57
|
-
| `requirements.txt` | `pip install -r requirements.txt` |
|
|
58
|
-
| `go.mod` | `go mod download` |
|
|
59
|
-
|
|
60
|
-
### 3. Verify Clean Baseline
|
|
61
|
-
|
|
62
|
-
Run the project's test suite. If tests fail, report failures and ask whether to proceed or investigate. Don't start work on a broken baseline.
|
|
63
|
-
|
|
64
|
-
### 4. Report
|
|
65
|
-
|
|
66
|
-
```
|
|
67
|
-
Worktree ready at <full-path>
|
|
68
|
-
Tests passing (<N> tests, 0 failures)
|
|
69
|
-
Ready to implement <feature-name>
|
|
70
|
-
```
|
|
71
|
-
|
|
72
|
-
## Common Mistakes
|
|
73
|
-
|
|
74
|
-
| Mistake | Fix |
|
|
75
|
-
|---------|-----|
|
|
76
|
-
| Skipping gitignore verification | Always check before creating project-local worktrees |
|
|
77
|
-
| Assuming directory location | Follow priority: existing > docs > ask |
|
|
78
|
-
| Proceeding with failing baseline tests | Report failures, get permission first |
|
|
79
|
-
| Hardcoding setup commands | Auto-detect from project files |
|
|
80
|
-
|
|
81
|
-
## Red Flags
|
|
82
|
-
|
|
83
|
-
- Never create a worktree without verifying it's gitignored (project-local)
|
|
84
|
-
- Never skip baseline test verification
|
|
85
|
-
- Never proceed with failing tests without asking
|
|
86
|
-
- Never assume directory location when ambiguous
|