Skip to content

docs: add .opencode/opencode.json as valid config location in precedence order#22416

Open
linchen1987 wants to merge 4 commits intoanomalyco:devfrom
linchen1987:doc-add-opencode-config-location
Open

docs: add .opencode/opencode.json as valid config location in precedence order#22416
linchen1987 wants to merge 4 commits intoanomalyco:devfrom
linchen1987:doc-add-opencode-config-location

Conversation

@linchen1987
Copy link
Copy Markdown

@linchen1987 linchen1987 commented Apr 14, 2026

Issue for this PR

Closes #

Type of change

  • Bug fix
  • New feature
  • Refactor / code improvement
  • Documentation

What does this PR do?

The config loading code at src/config/config.ts:1399-1408 already supports loading opencode.json from the .opencode/ directory, but this location was not documented in the precedence order section of the config docs.

Added .opencode/opencode.json as a new item (position 5) in the precedence list, between "Project config" and ".opencode directories". Updated all 18 locales: en, ar, bs, da, de, es, fr, it, ja, ko, nb, pl, pt-br, ru, th, tr, zh-cn, zh-tw.

How did you verify your code works?

  • Verified the code in src/config/config.ts loads .opencode/opencode.json
  • Confirmed all 18 locale files contain the new .opencode/opencode.json entry via grep
  • Verified no other content was changed

Screenshots / recordings

N/A - documentation only change.

Checklist

  • I have tested my changes locally
  • I have not included unrelated changes in this PR

…nce order

The code supports placing opencode.json inside the .opencode/ directory
(src/config/config.ts loads it), but this location was not documented.
Added it as a separate precedence item across all 18 locales.
@github-actions github-actions bot added needs:compliance This means the issue will auto-close after 2 hours. and removed needs:compliance This means the issue will auto-close after 2 hours. labels Apr 14, 2026
@github-actions
Copy link
Copy Markdown
Contributor

Thanks for updating your PR! It now meets our contributing guidelines. 👍

@linchen1987
Copy link
Copy Markdown
Author

Note on e2e (windows) failure

The e2e (windows) check timed out after 30 minutes with the error:

The action 'Run app e2e tests' has timed out after 30 minutes.

This is a CI infrastructure issue (Windows self-hosted runner timeout), not related to this PR's changes. This PR only modifies documentation files (18 locale config.mdx files) — no code changes.

Evidence:

Requesting a re-run of the failed job or a manual merge. Thanks!

@linchen1987
Copy link
Copy Markdown
Author

@kitlangton Could you help re-run the e2e (windows) job or merge this manually? It's a docs-only change (18 locale config files) and the Windows runner has timed out twice now due to infrastructure issues. All other checks pass. Thanks!

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