Repository navigation
Conversation
|
The following comment was made by an LLM, it may be inaccurate: |
![opencode-agent[bot]](https://avatars.githubusercontent.com/in/1549082?s=60&v=4){kind=small}
There was a problem hiding this comment.
The behaviour this describes matches the code. I checked it against the source instead of running it, since the change is docs-only. serve reads OPENCODE_PASSWORD and ignores the password saved by service set password. It only prints server password when the env var is unset. The basic-auth username is opencode, the data directory is /root/.local/share/opencode for the image's root user, and the image's entrypoint is opencode, so serve has to be passed.
Public API: unchanged (docs only).
The main problem: the example is built around the 2.0.0 image, but releases have moved well past it, and /api/info exists from at least v2.0.9 through the current v2.0.32 and dev. Pinning 2.0.0 sends readers to an old image with an older health check, and it adds caveats specific to that version in two places (see inline comments). Please use a current or placeholder version tag with the standard /api/info check, and drop the 2.0.0 caveats, or reduce them to a short note about older images.
Related: #45856
|
Updated the example and Intro to the verified Website typecheck, generated-reference check, full build/link validation, and the root check pass again for |
|
The two earlier inline points on this PR are addressed in
Could a maintainer approve the pending fork workflow runs so these required checks can execute? The fork verification linked in each PR is separate from these upstream gates. |
|
Following up on the pending fork-CI approval for this PR and #57300. Both patches still apply cleanly to today's Could a maintainer approve the pending runs? Here are the check and test runs for this PR, and check and test for #57300. Existing review points are resolved; prior fork verification remains separate from these upstream gates. |
Issue for this PR
Closes #55558
Type of change
What does this PR do?
Document a foreground Docker server with
serve, an explicit container bind address,OPENCODE_PASSWORD, and persistent data/project mounts. Explain thatservice set passwordonly configures the managed service. Use the current V2 image tag (2.0.32) and its standard/api/infocheck.How did you verify your code works?
Cross-checked the current source and the
v2.0.32tag, and confirmed that the public Docker tag exists. Local Bash syntax and documentation links pass. On Linux with Node 24 / Bun 1.4.2, websitebun typecheck,bun run check:generated,bun run build, and rootbun run checkall pass for the revised head in fork CI. The container itself has not been run.AI assistance was used to prepare and check this documentation change.
Screenshots / recordings
N/A — documentation only.
Checklist