Skip to content

docs(self-hosting): run the setup doctor from the Docker image - #155

Merged
charlesrhoward merged 1 commit into
mainfrom
docs/doctor-in-image
Oct 16, 2026
Merged

charlesrhoward merged 1 commit into
mainfrom
docs/doctor-in-image

Conversation

@charlesrhoward

@charlesrhoward charlesrhoward commented Oct 15, 2026 •

Copy link
Copy Markdown
Contributor

Documents Mogplex/mogplex#655:

  • Quickstart and Operations: Docker installs run the doctor inside the image with docker compose exec app node scripts/doctor.cjs [--live]. No checkout needed.
  • Operations: /api/health counts an OpenAI-compatible endpoint that lists models as configured models.

Merge order: after Mogplex/mogplex#655 merges and ships in a release image. Until then, scripts/doctor.cjs is not in v0.2.0.

pnpm types:check passes.


View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need.

@vercel

vercel Bot commented Oct 15, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
mogplex-docs Ready Ready Preview Oct 15, 2026 11:41pm UTC

Request Review

@mogplex

mogplex Bot commented Oct 15, 2026

Copy link
Copy Markdown

Mogplex PR Review

Status: No material issues found

The docs changes accurately document running the setup doctor inside the Docker image and clarify the health endpoint behavior. The updates are clear, consistent, and match the stated intent.

  • Added Docker-specific doctor commands in quickstart and operations, including the live flag. Commands use docker compose exec app node scripts/doctor.cjs as stated.
  • Clarified that /api/health counts models as configured when an AI Gateway key is present or an OpenAI-compatible endpoint lists at least one model. This matches the PR description.
  • Kept upgrade steps consistent, adding Docker alternative for running the doctor after migrations.
  • No code changes, so no security/correctness/architecture/performance/test concerns. The note about needing the script in the released image is a release/merge-order point, not a content issue in the docs.

View check run

@charlesrhoward
charlesrhoward merged commit ea1644a into main Oct 16, 2026
7 checks passed
@charlesrhoward
charlesrhoward deleted the docs/doctor-in-image branch October 16, 2026 00:55

This branch was successfully deployed

1 active deployment
Preview — 513669e0 Deployed Oct 15, 2026 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant