From 3a4ae44500dbac4a073d79a6cd89fc89a794822d Mon Sep 17 00:00:00 2001 From: Defame1297 Date: Sun, 5 Jul 2026 19:08:35 +0000 Subject: [PATCH] fix(gitea): correct create-vs-update param scoping in gitea-prs docs references/pull-requests.md wrongly scoped pull_request_write's milestone param to "update" only, and reviewers/team_reviewers to add_reviewers/remove_reviewers only. The canonical create example in docs/research/docs/gitea/examples.md sets milestone and reviewers directly on "create", and api-reference.md lists these params generically without method scoping. Corrects both to reflect that they're settable on create, with add_reviewers/remove_reviewers reserved for adjusting an already-open PR. Bumps gitea-prs skill version 0.1.0 -> 0.1.1 for the docs fix. Co-Authored-By: Claude Sonnet 5 Claude-Session: https://claude.ai/code/session_01VwtxcDXuLZxYzWnT2FaoZU --- plugins/gitea/skills/gitea-prs/SKILL.md | 2 +- plugins/gitea/skills/gitea-prs/references/pull-requests.md | 6 +++--- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/plugins/gitea/skills/gitea-prs/SKILL.md b/plugins/gitea/skills/gitea-prs/SKILL.md index dd3b0a3..ee2c449 100644 --- a/plugins/gitea/skills/gitea-prs/SKILL.md +++ b/plugins/gitea/skills/gitea-prs/SKILL.md @@ -20,7 +20,7 @@ metadata: - gitea-mcp-slim-go - context7-websites-gitea - context7-gitea-tea-cli - version: "0.1.0" + version: "0.1.1" allowed-tools: mcp__gitea__list_pull_requests mcp__gitea__pull_request_read mcp__gitea__pull_request_write mcp__gitea__pull_request_review_write --- diff --git a/plugins/gitea/skills/gitea-prs/references/pull-requests.md b/plugins/gitea/skills/gitea-prs/references/pull-requests.md index 58fbdf8..93986e6 100644 --- a/plugins/gitea/skills/gitea-prs/references/pull-requests.md +++ b/plugins/gitea/skills/gitea-prs/references/pull-requests.md @@ -56,14 +56,14 @@ List responses trim PRs down to summary fields — `head`/`base` are bare ref st - `base` (string, required for `"create"`) — target branch - `assignee` (string, optional) — single login - `assignees` (array of strings, optional) — login names -- `milestone` (number, optional, for `"update"`) — milestone ID, never a title +- `milestone` (number, optional) — milestone ID, never a title; settable on both `"create"` and `"update"` - `state` (string, optional, for `"update"`) — `"open"` | `"closed"` (no `"all"` — unlike issue state filters) - `allow_maintainer_edit` (boolean, optional, for `"update"`) - `labels` (array of numbers, optional) — label IDs, never names — resolve via `gitea-labels-milestones` first - `deadline` (string, optional) — ISO 8601 - `remove_deadline` (boolean, optional) -- `reviewers` (array of strings, optional, for `"add_reviewers"`/`"remove_reviewers"`) — login names -- `team_reviewers` (array of strings, optional, for `"add_reviewers"`/`"remove_reviewers"`) +- `reviewers` (array of strings, optional) — login names; settable directly on `"create"`, or use `"add_reviewers"`/`"remove_reviewers"` to adjust reviewers on an already-open PR +- `team_reviewers` (array of strings, optional) — same as `reviewers`: settable on `"create"`, or via `"add_reviewers"`/`"remove_reviewers"` post-creation - `draft` (boolean, optional, for `"create"`) — prepends `"WIP:"` to the title (see Gotcha) Merge-specific parameters (`merge_style`, `delete_branch`, `force_merge`, `merge_when_checks_succeed`, `head_commit_id`, `message` as merge commit message) are covered in `references/merging.md`.