clawlter/llm-wikipedia
Watch
1
0
Fork
You've already forked llm-wikipedia
 0
Code Issues Pull requests Projects Releases Packages 1 Wiki Activity Actions

fix: support health endpoint compatibility #4

Merged
clawlter merged 1 commit from fix/health-endpoint-compat into main 2026-06-10 10:23:56 -04:00
Conversation 0 Commits 1 Files changed 6 +59 -24
clawlter commented 2026-06-10 10:16:17 -04:00
Owner
Copy link

Summary

  • Restore the Dockerfile healthcheck to /api/health
  • Remove the temporary /api/ready route
  • Add /health as a compatibility alias for deployment platforms that probe that default path
  • Keep the mobile collapsed navigation label as LLM Wikipedia

Debugging note

The pasted HTML is a Next.js route miss, not a mounted-wiki health failure. In the current published image, /health returns that app 404 shape, while /api/health returns JSON. This change keeps the canonical Docker healthcheck on /api/health and also serves the same JSON at /health so infra defaults do not hit the app 404 page.

The supplied host mount /media/data/volumes/hermes_agent/runtime/wiki/brain:/data/wiki:ro does not exist inside this sandbox, so I could not verify that exact host path here. The container-side path /data/wiki is correct for the app.

Testing

  • npm test -- tests/health-endpoints.test.ts tests/wiki-layout.test.tsx
  • npm run lint
  • npm run typecheck
  • npm test
  • NEXT_TELEMETRY_DISABLED=1 LLM_WIKI_ROOT=/opt/hermes-runtime/repos/hermes-brain npm run build
  • npm audit --omit=dev
  • Browser QA at 390x844: collapsed mobile disclosure shows LLM Wikipedia
  • Local production server: /api/health and /health both return JSON 200 with 305 pages
  • Local Docker image: built-in healthcheck uses /api/health; container became healthy; /api/health and /health both returned JSON 200
## Summary - Restore the Dockerfile healthcheck to `/api/health` - Remove the temporary `/api/ready` route - Add `/health` as a compatibility alias for deployment platforms that probe that default path - Keep the mobile collapsed navigation label as `LLM Wikipedia` ## Debugging note The pasted HTML is a Next.js route miss, not a mounted-wiki health failure. In the current published image, `/health` returns that app 404 shape, while `/api/health` returns JSON. This change keeps the canonical Docker healthcheck on `/api/health` and also serves the same JSON at `/health` so infra defaults do not hit the app 404 page. The supplied host mount `/media/data/volumes/hermes_agent/runtime/wiki/brain:/data/wiki:ro` does not exist inside this sandbox, so I could not verify that exact host path here. The container-side path `/data/wiki` is correct for the app. ## Testing - npm test -- tests/health-endpoints.test.ts tests/wiki-layout.test.tsx - npm run lint - npm run typecheck - npm test - NEXT_TELEMETRY_DISABLED=1 LLM_WIKI_ROOT=/opt/hermes-runtime/repos/hermes-brain npm run build - npm audit --omit=dev - Browser QA at 390x844: collapsed mobile disclosure shows `LLM Wikipedia` - Local production server: `/api/health` and `/health` both return JSON 200 with 305 pages - Local Docker image: built-in healthcheck uses `/api/health`; container became healthy; `/api/health` and `/health` both returned JSON 200
fix: support health endpoint compatibility
All checks were successful
Build and publish container image / Build Docker image and publish package tags (pull_request) Successful in 1m43s
Details
CI / Validate formatting, linting, types, tests, and build (pull_request) Successful in 5m38s
Details
Build and publish container image / Build Docker image and publish package tags (push) Successful in 1m15s
Details
CI / Validate formatting, linting, types, tests, and build (push) Successful in 1m41s
Details
5c9af231dd
 
Restore the Docker healthcheck to the deep /api/health endpoint and add /health as a compatibility alias for deployment platforms that probe that path.
clawlter deleted branch fix/health-endpoint-compat 2026-06-10 10:23:56 -04:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
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
clawlter/llm-wikipedia!4
No description provided.