chore: fix forgejo docs drift against shipped tool schemas #279

Merged

david merged 1 commit from chore/2026-09-18/fix-forgejo-docs-drift into main

2026-09-18 06:32:43 +00:00

david commented

2026-09-18 06:31:54 +00:00

Owner

Summary

Fixes doc drift found during the issue-121 final consistency pass: DESIGN.md and IMPLEMENTATION_PLAN.md made parameter claims that don't match the shipped tool schemas in extensions/forgejo/src/index.ts. README was already consistent; no code changes.

Changes

extensions/forgejo/docs/DESIGN.md

Issues table — forgejo_issue_create: removed nonexistent ref param (shipped: title, body?, assignees?, labels?, milestone?, due_date?, closed?)
PRs table — forgejo_pr_create: removed nonexistent draft? param
PRs table — forgejo_pr_merge: method now lists all 6 shipped strategies (merge/rebase/rebase-merge/squash/fast-forward-only/manually-merged) — previously only 4, contradicting the README
PRs table — forgejo_pr_search: removed nonexistent q?/head? params (shipped: state?, labels?, base?, page?, limit?)
API endpoints table — List/search PRs: head → base_branch (the endpoint has no head filter; see searchPullRequests() in src/pulls.ts)

extensions/forgejo/docs/IMPLEMENTATION_PLAN.md

Phase 3.1: removed draft from the createPullRequest param list
Phase 3.8: query string corrected to ?state=&labels=&base_branch=&page=&limit= (dropped nonexistent head)

Verification

Tool names/params re-diffed against grep 'name: "forgejo_' src/index.ts (37 tools: 20 issues + 11 PRs + 1 detect + 5 actions)
npm run check — pass
npm test — 259/259 pass (11 suites)

Completes the last open acceptance criterion of issue #121 (docs consistent with shipped code).

## Summary Fixes doc drift found during the issue-121 final consistency pass: DESIGN.md and IMPLEMENTATION_PLAN.md made parameter claims that don't match the shipped tool schemas in `extensions/forgejo/src/index.ts`. README was already consistent; no code changes. ## Changes **`extensions/forgejo/docs/DESIGN.md`** - Issues table — `forgejo_issue_create`: removed nonexistent `ref` param (shipped: `title`, `body?`, `assignees?`, `labels?`, `milestone?`, `due_date?`, `closed?`) - PRs table — `forgejo_pr_create`: removed nonexistent `draft?` param - PRs table — `forgejo_pr_merge`: method now lists all 6 shipped strategies (`merge/rebase/rebase-merge/squash/fast-forward-only/manually-merged`) — previously only 4, contradicting the README - PRs table — `forgejo_pr_search`: removed nonexistent `q?`/`head?` params (shipped: `state?`, `labels?`, `base?`, `page?`, `limit?`) - API endpoints table — List/search PRs: `head` → `base_branch` (the endpoint has no `head` filter; see `searchPullRequests()` in `src/pulls.ts`) **`extensions/forgejo/docs/IMPLEMENTATION_PLAN.md`** - Phase 3.1: removed `draft` from the `createPullRequest` param list - Phase 3.8: query string corrected to `?state=&labels=&base_branch=&page=&limit=` (dropped nonexistent `head`) ## Verification - Tool names/params re-diffed against `grep 'name: "forgejo_' src/index.ts` (37 tools: 20 issues + 11 PRs + 1 detect + 5 actions) - `npm run check` — pass - `npm test` — 259/259 pass (11 suites) Completes the last open acceptance criterion of issue #121 (docs consistent with shipped code).

david added 1 commit

2026-09-18 06:31:54 +00:00