fix: resolve docs, plugin loading, and localized layout issues#23603
Open
LakshmanTurlapati wants to merge 1 commit intoanomalyco:devfrom
Open
fix: resolve docs, plugin loading, and localized layout issues#23603LakshmanTurlapati wants to merge 1 commit intoanomalyco:devfrom
LakshmanTurlapati wants to merge 1 commit intoanomalyco:devfrom
Conversation
github-actions
bot
added
needs:compliance
Contributor
|
Thanks for updating your PR! It now meets our contributing guidelines. 👍 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Issue for this PR
Closes #23566
Closes #23535
Closes #23547
Closes #23553
Closes #4208
Type of change
What does this PR do?
This keeps the scope limited to the five linked issues.
It fixes the public docs and landing-page copy so they no longer imply LSP is always enabled by default, and clarifies that
config.json,opencode.json, andopencode.jsoncare optional config files users create when they need them.It also fixes auto-discovered local plugins loaded through symlinked plugin directories by preserving the discovered path and making sure the plugin can still resolve
@opencode-ai/pluginfrom the config-side dependency location.For the UI issues, it lets the
New workspacesidebar button wrap instead of overflowing in narrower localized sidebars, and removes the fixed-width review action button that caused the TurkishExpand alllabel to overflow.I also updated one app test to avoid the Bun-incompatible fake-timer path so the local app test suite runs cleanly in this environment.
How did you verify your code works?
I verified the changed areas with targeted package-level checks:
cd packages/app && bun typecheckcd packages/app && bun test:unitcd packages/ui && bun typecheckcd packages/opencode && bun test test/util/glob.test.ts test/plugin/loader-shared.test.ts --timeout 30000cd packages/opencode && bun typecheckcd packages/web && bunx prettier --check src/content/docs/lsp.mdx src/content/docs/providers.mdx src/content/i18n/en.jsonI also verified the PR branch diff only contains the expected milestone files.
One local environment blocker remains outside the source changes:
cd packages/web && bunx astro checknode_modulescannot be loaded on this machine.Screenshots / recordings
No screenshots included.
The UI changes are small layout containment fixes in the sidebar and review panel.
Checklist