Skip to content

Add plugin CLI command guide to integrations - #1534

Open
ishaanxgupta wants to merge 1 commit into
mainfrom
feat/plugin-command-modal
Open

Add plugin CLI command guide to integrations#1534
ishaanxgupta wants to merge 1 commit into
mainfrom
feat/plugin-command-modal

Conversation

@ishaanxgupta

@ishaanxgupta ishaanxgupta commented Aug 18, 2026

Copy link
Copy Markdown
Contributor

Summary

  • add an “Install plugins with one command” action beside the Plugins section
image image

@ishaanxgupta
ishaanxgupta marked this pull request as ready for review August 18, 2026 16:02
@claude

claude Bot commented Aug 18, 2026

Copy link
Copy Markdown

Claude finished @ishaanxgupta's task in 1m 14s —— View job


Code Review Complete

  • Get PR diff and understand the changes
  • Review for bugs, security issues, and logic errors
  • Check existing review comments
  • Submit review summary

Overview: This PR adds a modal displaying plugin CLI commands (install, login, uninstall) accessible via a button next to the Plugins section header.

Issues found: None — this looks good to ship.

The code correctly references all plugin IDs that exist in PLUGIN_CATALOG, properly reuses the existing InstallSteps component, has good accessibility attributes, and extends SectionRail cleanly with an optional labelSlot prop.

Score: 10/10

@claude claude Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Overview: Adds a modal displaying plugin CLI commands (install, login, uninstall) accessible via a button next to the Plugins section header.

Issues found: None — this looks good to ship.

The code:

  • Correctly references all four plugin IDs (claude_code, cursor, codex, opencode) which all exist in PLUGIN_CATALOG
  • Properly reuses the existing InstallSteps component for consistent UI
  • Has good accessibility with aria-haspopup, aria-expanded, and screen-reader-only DialogTitle
  • Extends SectionRail cleanly with an optional labelSlot prop
  • Uses proper state management for the modal

Score: 10/10

@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Updated (UTC)
✅ Deployment successful!
View logs
supermemory-mcp 277ca1a Aug 18 2026, 04:02 PM

@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Preview URL Updated (UTC)
✅ Deployment successful!
View logs
supermemory-app 277ca1a Commit Preview URL

Branch Preview URL
Aug 18 2026, 04:04 PM

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