Skip to content

fix(opencode): ensure opencode.json is created in postStartCommand fo…#48

Merged
wcgomes merged 4 commits into
mainfrom
fix/opencode-poststart-config-volume
May 26, 2026
Merged

fix(opencode): ensure opencode.json is created in postStartCommand fo…#48
wcgomes merged 4 commits into
mainfrom
fix/opencode-poststart-config-volume

Conversation

@wcgomes

@wcgomes wcgomes commented May 26, 2026

Copy link
Copy Markdown
Owner

…r volume mount resilience

When /home/vscode/.config/opencode is a Docker volume mount, the build-time opencode.json created by install.sh is hidden by the empty volume. Move config creation to postStartCommand.sh so it runs at every container start.

  • Add directory creation, opencode.json write/merge, and chown to postStartCommand.sh
  • Fix detect_user fallback to match install.sh logic (root fallback instead of hardcoded vscode)
  • Add postStartCommand recreation test to all 6 test scripts
  • Document the decision in wiki/decisions/
  • Update wiki/ conventions, features, architecture, index

Test added 4 commits May 26, 2026 15:24
…r volume mount resilience

When /home/vscode/.config/opencode is a Docker volume mount, the
build-time opencode.json created by install.sh is hidden by the empty
volume. Move config creation to postStartCommand.sh so it runs at every
container start.

- Add directory creation, opencode.json write/merge, and chown to
  postStartCommand.sh
- Fix detect_user fallback to match install.sh logic (root fallback
  instead of hardcoded vscode)
- Add postStartCommand recreation test to all 6 test scripts
- Document the decision in wiki/decisions/
- Update wiki/ conventions, features, architecture, index
postStartCommand.sh was returning _REMOTE_USER/USERNAME without
validating the user exists in passwd. In CI, _REMOTE_USER=vscode
is set even in root-only images (debian:latest), causing the config
to be created in /home/vscode/ while tests check /root/.

Copy the same shell-validation logic from install.sh: if the env-var
user has */nologin, */false, or empty shell, fall back to the first
UID>=1000 user or root.
@wcgomes wcgomes merged commit b54be42 into main May 26, 2026
13 checks passed
@wcgomes wcgomes deleted the fix/opencode-poststart-config-volume branch May 26, 2026 17:06
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