MokoConsulting/MokoGitea-APP
1
0
Fork 0
Code Issues 171 Pull Requests 1 Releases 96 Wiki Activity

fix: wiki API sub-page support and content response (#606, #607) #608

Merged
jmiller merged 1 commits from fix/606-607-wiki-api-bugs into main 2026-06-11 22:17:02 +00:00
Conversation 0 Commits 1 Files Changed 2
+25 -13
jmiller commented 2026-06-11 21:56:59 +00:00
Owner
Copy Link
Copy Source

Summary

Fixes two wiki API bugs:

#606: Sub-pages with path separators return 404

  • Changed wiki API routes from {pageName} to wildcard * to capture slashes in page paths (e.g. mcp/fleet-overview)
  • Changed ListWikiPages to use ListEntriesRecursiveFast() instead of ListEntries() so pages in subdirectories are included
  • Fixed pagination to count only regular files, not directories

#607: content_base64 empty in POST/PATCH responses

  • Added error logging in wikiContentsByEntry to diagnose empty content responses
  • The route fix for #606 may also resolve cases where path mismatches caused content lookup failures

Changes

  • routers/api/v1/api.go: Wiki routes use wildcard instead of {pageName}
  • routers/api/v1/repo/wiki.go: Recursive listing, wildcard path param, error logging

MCP Compatibility

The MCP tool gitea_wiki_page_get already constructs URLs with the page name in the path — slashes will now work naturally with the wildcard route.

Closes #606, Closes #607

## Summary Fixes two wiki API bugs: ### #606: Sub-pages with path separators return 404 - Changed wiki API routes from `{pageName}` to wildcard `*` to capture slashes in page paths (e.g. `mcp/fleet-overview`) - Changed `ListWikiPages` to use `ListEntriesRecursiveFast()` instead of `ListEntries()` so pages in subdirectories are included - Fixed pagination to count only regular files, not directories ### #607: content_base64 empty in POST/PATCH responses - Added error logging in `wikiContentsByEntry` to diagnose empty content responses - The route fix for #606 may also resolve cases where path mismatches caused content lookup failures ## Changes - `routers/api/v1/api.go`: Wiki routes use wildcard instead of `{pageName}` - `routers/api/v1/repo/wiki.go`: Recursive listing, wildcard path param, error logging ## MCP Compatibility The MCP tool `gitea_wiki_page_get` already constructs URLs with the page name in the path — slashes will now work naturally with the wildcard route. Closes #606, Closes #607
jmiller added 1 commit 2026-06-11 21:57:04 +00:00
fix: wiki API sub-page support and content response (#606, #607)
Generic: Project CI / Tests (pull_request) Blocked by required conditions
Details
Universal: PR Check / Build RC Package (pull_request) Blocked by required conditions
Details
Universal: PR Check / Report Issues (pull_request) Blocked by required conditions
Details
Branch Policy Check / Verify merge target (pull_request) Failing after 4s
Details
Generic: Repo Health / Scripts governance (pull_request) Blocked by required conditions
Details
Universal: PR Check / Branch Policy (pull_request) Failing after 4s
Details
Generic: Repo Health / Repository health (pull_request) Blocked by required conditions
Details
Generic: Repo Health / Report Issues (pull_request) Blocked by required conditions
Details
Generic: Repo Health / Site Health (pull_request) Has been skipped
Details
Generic: Repo Health / Access control (pull_request) Successful in 4s
Details
Universal: PR Check / Validate PR (pull_request) Failing after 14s
Details
Generic: Repo Health / Scripts governance (push) Blocked by required conditions
Details
Generic: Repo Health / Repository health (push) Blocked by required conditions
Details
Generic: Repo Health / Report Issues (push) Blocked by required conditions
Details
Generic: Repo Health / Site Health (push) Has been skipped
Details
Generic: Repo Health / Access control (push) Successful in 3s
Details
Branch Cleanup / Delete merged branch (pull_request) Has been skipped
Details
Universal: Build & Release / Build & Release Pipeline (pull_request) Has been skipped
Details
RC Revert / Rename rc/ back to dev/ (pull_request) Has been skipped
Details
Universal: Auto Version Bump / Version Bump (push) Failing after 12s
Details
Generic: Project CI / Lint & Validate (pull_request) Successful in 44s
Details
Universal: Build & Release / Promote to RC (pull_request) Successful in 44s
Details
Universal: Secret Scanning / Gitleaks Secret Scan (pull_request) Successful in 57s
Details
PR RC Release / Build RC Release (pull_request) Failing after 54s
Details
3a7b07590f 
- Change wiki API routes from {pageName} to wildcard (*) to support
  pages with path separators (e.g. mcp/fleet-overview)
- Use ListEntriesRecursiveFast() in ListWikiPages to include pages
  in subdirectories, not just root-level files
- Add error logging in wikiContentsByEntry for diagnosing empty
  content_base64 responses
- Fix pagination to count only regular files, not directories

Co-Authored-By: Moko Consulting <hello@mokoconsulting.tech>
jmiller closed this pull request 2026-06-11 21:57:34 +00:00
jmiller deleted branch fix/606-607-wiki-api-bugs 2026-06-11 21:57:39 +00:00
jmiller reopened this pull request 2026-06-11 22:16:35 +00:00
jmiller merged commit 5893a8b9dd into main 2026-06-11 22:17:02 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
breaking-change
Breaking API or behavior change
 ci-cd
CI/CD pipeline changes
 config
Configuration changes
 dependencies
Dependency updates
 deploy-failure
Deployment failed
 docker
Docker/container changes
 documentation
Documentation changes
 good first issue
Good for newcomers
 health-check
Repo health check result
 help wanted
Extra attention needed
 mokostandards
Related to MokoStandards framework
 push-failure
Git push operation failed
 security
Security vulnerability or hardening
 size/l
200-500 lines changed
 size/m
50-200 lines changed
 size/s
10-50 lines changed
 size/xl
500+ lines changed
 size/xs
< 10 lines changed
 standards-drift
Deviates from MokoStandards
 standards-update
MokoStandards compliance update
 sync-failure
Sync or mirror failed
 tech-debt
Technical debt and TODO/FIXME items
 upstream
upstream
Inherited from upstream Gitea
 work-in-progress
Draft or incomplete work
No labels
Priority -
Type -
Status —
Priority —
Type —
Milestone
No items
No Milestone
Assignees
Clear assignees
jmiller (Jonathan Miller) moko-deploy (Moko Deploy Bot)
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: MokoConsulting/MokoGitea-APP#608
Delete Branch "fix/606-607-wiki-api-bugs"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?