Skip to content

fix(pr-triage): address bundled scripts by skill directory #2063

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.

Sign up for GitHub

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

Open
shahar1 wants to merge 1 commit into
base: main
Choose a base branch
from
Open

fix(pr-triage): address bundled scripts by skill directory #2063

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 5 additions & 3 deletions plugins/magpie-pr-management/skills/pr-triage/SKILL.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -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 -->
Expand Down Expand Up @@ -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.
Expand Down
3 changes: 2 additions & 1 deletion plugins/magpie-pr-management/skills/pr-triage/typed-decision-prefilter.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -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.
Expand Down