Skip to content

feat(nova): add hermes plugin instructions#832

Merged
graphite-app[bot] merged 1 commit intomainfrom
04-06-feat_nova_add_hermes_plugin_instructions
Apr 6, 2026
Merged

feat(nova): add hermes plugin instructions#832
graphite-app[bot] merged 1 commit intomainfrom
04-06-feat_nova_add_hermes_plugin_instructions

Conversation

@MaheshtheDev
Copy link
Copy Markdown
Member

No description provided.

@cloudflare-workers-and-pages
Copy link
Copy Markdown

cloudflare-workers-and-pages bot commented Apr 6, 2026

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-app 0638f65 Apr 06 2026, 11:22 PM

Copy link
Copy Markdown
Member Author


How to use the Graphite Merge Queue

Add the label Main to this PR to add it to the merge queue.

You must have a Graphite account in order to use the merge queue. Sign up using this link.

An organization admin has enabled the Graphite Merge Queue in this repository.

Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue.

This stack of pull requests is managed by Graphite. Learn more about stacking.

@MaheshtheDev MaheshtheDev marked this pull request as ready for review April 6, 2026 21:05
@claude
Copy link
Copy Markdown

claude bot commented Apr 6, 2026

Claude finished @MaheshtheDev's task in 1m 9s —— View job


PR Review Complete

  • Fetching PR diff
  • Analyzing changes for bugs and security issues
  • Checking surrounding code context
  • Submitting review

Overview: Adds Hermes plugin integration across the web app, including plugin info, icons, and a new ?plugins=true URL parameter for direct panel access.

Issues found: None — this looks good to ship.

The changes are well-structured:

  • Type safety is maintained by importing and reusing IntegrationParamValue consistently
  • The new pluginsPanelParam boolean param provides a clean way to deep-link to the plugins panel
  • SVG is safe (no script tags or event handlers)
  • The useEffect in integrations-view.tsx properly handles the new state alongside existing integration param logic
  • All plugin catalog entries follow the established pattern

Score: 10/10

Copy link
Copy Markdown

@claude claude bot left a comment

Choose a reason for hiding this comment

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

Overview: Adds Hermes plugin integration across the web app, including plugin info, icons, and a new ?plugins=true URL parameter for direct panel access.

Issues found: None — this looks good to ship.

The changes are well-structured:

  • Type safety is maintained by importing and reusing IntegrationParamValue consistently
  • The new pluginsPanelParam boolean param provides a clean way to deep-link to the plugins panel
  • SVG is safe (no script tags or event handlers)
  • The useEffect in integrations-view.tsx properly handles the new state alongside existing integration param logic
  • All plugin catalog entries follow the established pattern

Score: 10/10

@graphite-app
Copy link
Copy Markdown

graphite-app bot commented Apr 6, 2026

Merge activity

@graphite-app graphite-app bot force-pushed the 04-06-feat_nova_add_hermes_plugin_instructions branch from 60a8953 to 0638f65 Compare April 6, 2026 23:18
@graphite-app graphite-app bot merged commit 0638f65 into main Apr 6, 2026
7 of 9 checks passed
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.

2 participants