dsh-plugins/dsh-gitea
Watch
2
0
Fork
You've already forked dsh-gitea
 0
Code Issues 5 Pull requests Projects Releases 4 Packages 1 Wiki Activity Actions

fix(wiki): treat an empty wiki's 404 as an empty page list #13

Merged
dsh merged 1 commit from fix/9-wiki-empty-list into master 2026-10-23 22:36:55 +00:00
Conversation 0 Commits 1 Files changed 2 +76 -1
dsh commented 2026-10-23 22:36:45 +00:00
Owner

What

gitea_wiki {action:"list"} treats an enabled-but-empty wiki as an error:

gitea_wiki failed: The target couldn't be found.

Forgejo's GET /repos/{owner}/{repo}/wiki/pages answers 404 when the wiki has no pages (there is no empty list to return). listWikiPages() now maps only that 404 to { ok: true, status: 404, data: [] }.

Auth (401), permission (403) and server (5xx) failures still surface as errors — the mapping is not a blanket swallow.

Verification

  • New test/wiki.test.mjs: mocked-fetch coverage for 404 → empty success, 200 → pages unchanged, 401/403/500 → still errors.
  • npm run check passes (49 tests, leak scan clean).

Acceptance (from #9)

  • Unit test with a mocked fetch 404: gitea_wiki list resolves ok: true with an empty list and no error text.
  • 401/403/5xx keep failing as before.
  • Live check on ml-lab/tardits after merge/reload (tracked in #9).
  • Live check that pvlb-infra/t14-deeps still lists all pages.

Closes #9.

## What `gitea_wiki {action:"list"}` treats an enabled-but-empty wiki as an error: ``` gitea_wiki failed: The target couldn't be found. ``` Forgejo's `GET /repos/{owner}/{repo}/wiki/pages` answers **404** when the wiki has no pages (there is no empty list to return). `listWikiPages()` now maps only that 404 to `{ ok: true, status: 404, data: [] }`. Auth (`401`), permission (`403`) and server (`5xx`) failures still surface as errors — the mapping is not a blanket swallow. ## Verification - New `test/wiki.test.mjs`: mocked-fetch coverage for 404 → empty success, 200 → pages unchanged, 401/403/500 → still errors. - `npm run check` passes (49 tests, leak scan clean). ## Acceptance (from #9) - [x] Unit test with a mocked `fetch` 404: `gitea_wiki list` resolves `ok: true` with an empty list and no error text. - [x] 401/403/5xx keep failing as before. - [ ] Live check on `ml-lab/tardits` after merge/reload (tracked in #9). - [ ] Live check that `pvlb-infra/t14-deeps` still lists all pages. Closes #9. 
Forgejo answers 404 for an enabled-but-empty wiki. Map only that 404 to a successful empty list; 401/403/5xx still fail.
dsh merged commit 913013aa7c into master 2026-10-23 22:36:55 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
area/dsh
DSH plugin surface: host/client, settings, updater

  
area/forgejo
Forgejo API, issues, wiki, CI plumbing

  
experiment
Exploratory: measure before committing to a design.

  
priority/later
Not scheduled; revisit when the data justifies it.

  
priority/next
Do after current work; no urgency

  
priority/now
Do next; something is broken or time-boxed

  
type/bug
Defect: wrong, missing or misleading behaviour

  
type/task
Work item: feature, chore, docs, refactor

No labels
area/dsh
area/forgejo
experiment
priority/later
priority/next
priority/now
type/bug
type/task
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
dsh-plugins/dsh-gitea!13
No description provided.