Skip to content

Commit 374faa6

Browse files
committed
refactor(skills): migrate pr-risk-assessment from CLAUDE_CONFIG_DIR to FULLSEND_SKILLS_DIR
Replace runtime-specific CLAUDE_CONFIG_DIR path references with the runtime-neutral FULLSEND_SKILLS_DIR env var so pr-risk-assessment scripts work on any supported runtime, not just Claude. - Update script invocation path in SKILL.md and risk-assessment.md - Update header comment in risk-tier1.sh - Add FULLSEND_SKILLS_DIR to SANDBOX_RUNTIME_VARS in reachability test - Remove resolved KNOWN_GAPS entry for pr-risk-assessment:CLAUDE_CONFIG_DIR AI-Assisted: true Assisted-by: claude-opus-4-6 Generated with AI assistance (claude-opus-4-6) Signed-off-by: Jay Flowers <jay.flowers@gmail.com>
1 parent 27cb4e8 commit 374faa6

4 files changed

Lines changed: 4 additions & 7 deletions

File tree

‎scripts/skill-env-reachability-test.sh‎

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -40,6 +40,7 @@ SANDBOX_RUNTIME_VARS=(
4040
FULLSEND_OUTPUT_DIR
4141
FULLSEND_OUTPUT_SCHEMA
4242
FULLSEND_ROLE
43+
FULLSEND_SKILLS_DIR
4344
GOOGLE_APPLICATION_CREDENTIALS
4445
PATH
4546
)
@@ -131,10 +132,6 @@ declare -A KNOWN_GAPS=(
131132
["github-forge:ISSUE_NUMBER"]="#1664 — forge skill derives ISSUE_NUMBER locally from ISSUE_URL; not in fix/retro/review/prioritize/triage env.sandbox"
132133
["gitlab-forge:ISSUE_NUMBER"]="#1664 — forge skill derives ISSUE_NUMBER locally from ISSUE_URL; not in fix/retro/review/prioritize/triage env.sandbox"
133134
["issue-labels/gitlab:ISSUE_NUMBER"]="#1664 — label skill derives ISSUE_NUMBER locally from ISSUE_URL; review/triage harnesses do not supply it"
134-
# pr-risk-assessment scripts hard-reference the runtime-specific config
135-
# dir (CLAUDE_CONFIG_DIR on claude; CODEX_HOME/PI_CODING_AGENT_DIR on
136-
# codex/pi). No runtime-neutral skill path exists yet.
137-
["pr-risk-assessment:CLAUDE_CONFIG_DIR"]="#1672 — runtime-specific config dir; needs runtime-neutral skill path"
138135
# Skills below reference these vars as ${VAR} expansions without
139136
# assigning them, and no delivering harness declares them in
140137
# env.sandbox.

‎skills/pr-review/sub-agents/risk-assessment.md‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@ review verdicts.
2424

2525
1. Run the Tier 1 metadata script:
2626
```bash
27-
bash "${CLAUDE_CONFIG_DIR}/skills/pr-risk-assessment/scripts/risk-tier1.sh"
27+
bash "${FULLSEND_SKILLS_DIR}/pr-risk-assessment/scripts/risk-tier1.sh"
2828
```
2929
Capture the KEY=VALUE output. Each line is one signal.
3030

‎skills/pr-risk-assessment/SKILL.md‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -204,7 +204,7 @@ response directly.
204204

205205
1. **Run Tier 1 script:**
206206
```bash
207-
bash "${CLAUDE_CONFIG_DIR}/skills/pr-risk-assessment/scripts/risk-tier1.sh"
207+
bash "${FULLSEND_SKILLS_DIR}/pr-risk-assessment/scripts/risk-tier1.sh"
208208
```
209209
Capture KEY=VALUE output. Parse each line and store signals.
210210

‎skills/pr-risk-assessment/scripts/risk-tier1.sh‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22
# risk-tier1.sh — Compute deterministic Tier 1 metadata signals for PR risk assessment.
33
#
44
# Run by the risk-assessment sub-agent inside the sandbox:
5-
# bash "${CLAUDE_CONFIG_DIR}/skills/pr-risk-assessment/scripts/risk-tier1.sh"
5+
# bash "${FULLSEND_SKILLS_DIR}/pr-risk-assessment/scripts/risk-tier1.sh"
66
#
77
# Required env vars: PR_NUMBER, REPO_FULL_NAME, GH_TOKEN
88
# Output: KEY=VALUE pairs on stdout, one per line.

0 commit comments

Comments
 (0)