Skip to content

fix(core): skip dependency install in read-only config dirs#12128

Merged
rekram1-node merged 1 commit into
anomalyco:devfrom
shantur:bugfix/readonly-config-dir
Feb 4, 2026
Merged

fix(core): skip dependency install in read-only config dirs#12128
rekram1-node merged 1 commit into
anomalyco:devfrom
shantur:bugfix/readonly-config-dir

Conversation

@shantur
Copy link
Copy Markdown
Contributor

@shantur shantur commented Feb 4, 2026

What does this PR do?

Prevents config bootstrap from writing to read-only OPENCODE_CONFIG_DIR and adds coverage for read-only and writable dirs.
Fixes bug introduced with #9675

How did you verify your code works?

Added tests

Prevents config bootstrap from writing to read-only OPENCODE_CONFIG_DIR and adds coverage for read-only and writable dirs.
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Feb 4, 2026

Thanks for your contribution!

This PR doesn't have a linked issue. All PRs must reference an existing issue.

Please:

  1. Open an issue describing the bug/feature (if one doesn't exist)
  2. Add Fixes #<number> or Closes #<number> to this PR description

See CONTRIBUTING.md for details.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Feb 4, 2026

The following comment was made by an LLM, it may be inaccurate:

No duplicate PRs found

@rekram1-node rekram1-node merged commit 0d38e69 into anomalyco:dev Feb 4, 2026
6 checks passed
01luyicheng pushed a commit to 01luyicheng/opencode that referenced this pull request Feb 5, 2026
Colin4k1024 pushed a commit to Colin4k1024/opencode that referenced this pull request Feb 10, 2026
schneiderlo pushed a commit to schneiderlo/opencode that referenced this pull request Feb 17, 2026
avion23 pushed a commit to avion23/opencode that referenced this pull request Mar 4, 2026
xywsxp pushed a commit to xywsxp/opencode that referenced this pull request Apr 24, 2026
Rwanbt pushed a commit to Rwanbt/opencode that referenced this pull request May 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants