From e7699db95b4c9399a4231d1b07205449ae4f9e88 Mon Sep 17 00:00:00 2001
From: Shahar Epstein <60007259+shahar1@users.noreply.github.com>
Date: Fri, 13 Nov 2026 15:31:12 +0200
Subject: [PATCH] fix(pr-triage): address bundled scripts by skill directory

The pr_link.py and typed_decision_prefilter.py commands pointed at
<framework>/skills/pr-management-triage/scripts/. On a marketplace
install <framework> is ~/.cache/apache-magpie/framework/current, which
holds tools/ and no skills/, so both commands fail with "No such file".

Use <skill-dir>/scripts/, the placeholder isolated-setup-doctor already
uses, which resolves on every install method.

Signed-off-by: Shahar Epstein <60007259+shahar1@users.noreply.github.com>
Generated-by: Claude Code (Fable 5.2)
---
 plugins/magpie-pr-management/skills/pr-triage/SKILL.md    | 8 +++++---
 .../skills/pr-triage/typed-decision-prefilter.md          | 3 ++-
 2 files changed, 7 insertions(+), 4 deletions(-)

diff --git a/plugins/magpie-pr-management/skills/pr-triage/SKILL.md b/plugins/magpie-pr-management/skills/pr-triage/SKILL.md
index 7d9e6793c..a7a2df5d5 100644
--- a/plugins/magpie-pr-management/skills/pr-triage/SKILL.md
+++ b/plugins/magpie-pr-management/skills/pr-triage/SKILL.md
@@ -27,7 +27,7 @@ argument-hint: "[pr:N] [label:LBL] [author:LOGIN] [review-for-me] [stale] [repo:
 capability: capability:triage
 surface_hash: sha256:54dc4ba1be7b36d4
 license: Apache-2.0
-measured_tokens: 5151
+measured_tokens: 5172
 ---
 <!-- SPDX-License-Identifier: Apache-2.0
      https://www.apache.org/licenses/LICENSE-2.0 -->
@@ -247,14 +247,16 @@ terminal-bound PR reference instead of constructing OSC 8 sequences
 inside individual output paths:
 
 ```bash
-python3 <framework>/skills/pr-management-triage/scripts/pr_link.py \
+python3 <skill-dir>/scripts/pr_link.py \
   '<upstream>#NNN'
 
 # When the repository is obvious and only #NNN should be visible:
-python3 <framework>/skills/pr-management-triage/scripts/pr_link.py \
+python3 <skill-dir>/scripts/pr_link.py \
   --repo '<upstream>' '#NNN'
 ```
 
+`<skill-dir>` is the directory holding this `SKILL.md`; `<framework>/skills/` does not exist on a marketplace install.
+
 The helper accepts `<upstream>#NNN`, the full GitHub pull-request URL, or
 `#NNN` with `--repo <upstream>`. It preserves the visible form and always
 targets the canonical `https://github.com/<owner>/<repo>/pull/<N>` URL.
diff --git a/plugins/magpie-pr-management/skills/pr-triage/typed-decision-prefilter.md b/plugins/magpie-pr-management/skills/pr-triage/typed-decision-prefilter.md
index dba31ed59..fe172b1fa 100644
--- a/plugins/magpie-pr-management/skills/pr-triage/typed-decision-prefilter.md
+++ b/plugins/magpie-pr-management/skills/pr-triage/typed-decision-prefilter.md
@@ -9,10 +9,11 @@ When enabled via `enable_typed_decision_prefilter: true`
 (default `0.85`), run the helper alongside `triage classify` to evaluate classifier accuracy.
 For each classified PR, write its state to a scratch file and invoke:
 ```bash
-uv run --project <framework>/tools/typed-decision python3 <framework>/skills/pr-management-triage/scripts/typed_decision_prefilter.py \
+uv run --project <framework>/tools/typed-decision python3 <skill-dir>/scripts/typed_decision_prefilter.py \
   --file <scratch>/pr-<N>.json \
   --table-classification <label>
 ```
+`<skill-dir>` is the directory holding this skill's `SKILL.md`; `<framework>/skills/` does not exist on a marketplace install.
 - **Contract:**
   The decision table and Real-CI guard result remains authoritative in all cases.
   The shadow pass records predictions alongside the final table classification for telemetry.