Skip to content

fix(extensions): show directory-based project plugins#1535

Merged
src-opn merged 4 commits into
different-ai:devfrom
Pagecran:fix/1105-local-plugin-visibility
May 4, 2026
Merged

fix(extensions): show directory-based project plugins#1535
src-opn merged 4 commits into
different-ai:devfrom
Pagecran:fix/1105-local-plugin-visibility

Conversation

@pascalandr
Copy link
Copy Markdown
Contributor

Summary

Fixes #1105 by updating the Extensions settings store to include all project-scoped plugins returned by openworkClient.listPlugins(...), deduped by spec, instead of only showing plugins whose source is config.

Testing

  • git diff --check upstream/dev...fix/1105-local-plugin-visibility OK
  • pnpm --filter @openwork/app typecheck could not provide a clean signal in this environment due local TypeScript install issues (react-router-dom, lucide-react, zustand, @tanstack/react-virtual missing) plus an existing streamdown / React types mismatch

Notes

  • This is the React port of the local pre-cutover fix.

@vercel
Copy link
Copy Markdown
Contributor

vercel Bot commented Apr 23, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
openwork-landing Ready Ready Preview, Comment, Open in v0 Apr 30, 2026 8:16am

@vercel
Copy link
Copy Markdown
Contributor

vercel Bot commented Apr 23, 2026

@pascalandr is attempting to deploy a commit to the Different AI Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions
Copy link
Copy Markdown
Contributor

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

Copy link
Copy Markdown
Collaborator

@src-opn src-opn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Requesting changes.

Finding:

  • apps/app/src/react-app/domains/settings/state/extensions-store.ts:1306 and apps/app/src/react-app/domains/settings/pages/plugins-view.tsx:250-267: directory-discovered project plugins are now added to pluginList as bare strings, but pluginList does not preserve each item's source. The Plugins UI renders every entry with the same Enabled state and Remove action, and removePlugin() only removes config-backed plugins from opencode.json / DELETE /workspace/:id/plugins/:name. For a dir.project item, clicking Remove will either report Plugin not found or refresh back to the same item, which makes the new visible plugin look removable when it is not. Please preserve plugin metadata in state or otherwise render directory plugins read-only / source-aware, and only enable removal for config-backed plugins.

Verification performed:

  • git diff --check origin/dev...HEAD
  • byte-level scan of the changed patch for non-ASCII/control/Bidi/BOM characters
  • pnpm install --frozen-lockfile
  • pnpm --filter @openwork/app typecheck
  • pnpm --filter @openwork/app build

@pascalandr
Copy link
Copy Markdown
Contributor Author

Made directory plugins visible but read-only, and kept Remove only for config plugins.

@src-opn src-opn merged commit 4a81b15 into different-ai:dev May 4, 2026
9 of 13 checks passed
benjaminshafii pushed a commit that referenced this pull request May 15, 2026
* fix(extensions): show directory-based project plugins

* fix(extensions): make discovered plugins read-only
benjaminshafii pushed a commit that referenced this pull request May 15, 2026
* fix(extensions): show directory-based project plugins

* fix(extensions): make discovered plugins read-only
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.

[Bug]: Extensions UI does not display plugins that come from local plugin directories

2 participants