docs(readme): align Docker auth example with GOG_HOME pattern#830
docs(readme): align Docker auth example with GOG_HOME pattern#830WadydX wants to merge 1 commit into
Conversation
The README Docker example mounted only ~/.config/gogcli, but file-keyring tokens are stored under the data directory. Users following the example would lose auth state after the container exited. Update the example to use GOG_HOME=/persist/gogcli with a persistent state volume, matching docs/install.md and docs/paths.md. Closes openclaw#828
|
ClawSweeper review: did not complete due to Codex infrastructure failure. Reviewed June 16, 2026, 9:16 AM ET / 13:16 UTC. Summary Reproducibility: unclear. The review failed before ClawSweeper could establish a reproduction path. Review metrics: none identified. Merge readiness This is a ClawSweeper/Codex infrastructure failure, not a PR readiness or patch-quality verdict. Risk before merge
Maintainer options:
Next step before merge
Review detailsBest possible solution: Retry the Codex review after fixing the execution failure. Do we have a high-confidence way to reproduce the issue? Unclear. The review failed before ClawSweeper could establish a reproduction path. Is this the best way to solve the issue? Unclear. Retry the review first so ClawSweeper can evaluate the actual issue and fix direction. AGENTS.md: unclear because the file could not be read completely. Codex review notes: model internal, reasoning high; reviewed against f371f43eaef3. Evidence reviewedWhat I checked:
Likely related people:
How this review workflow works
|
Summary
Aligns the README Docker auth example with the
GOG_HOMEpattern already documented indocs/install.mdanddocs/paths.md.Closes #828
Problem
The README Docker example mounts only
-v gogcli-config:/home/gog/.config/gogcli, but file-keyring tokens are stored under the data directory ($GOG_DATA_DIR/keyring/), not the config directory. Users following the README completeauth addinside the container, then lose their tokens when the container exits because the data directory was never persisted.Change
GOG_HOMEvolume (gogcli-state:/persist/gogcli)-e GOG_HOME=/persist/gogclito thedocker runcommandgogcli-configtogogcli-statefor clarityThis matches the exact example already shown in
docs/install.md(line 30–43).Verification
docs/install.mdalready uses this pattern:docker volume create gogcli-state+GOG_HOME=/persist/gogclidocs/paths.mdconfirms data (including file-keyring entries) lives under the data directory