-
Notifications
You must be signed in to change notification settings - Fork 0
fix(plugin): refresh marketplace and add OMP support #443
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,21 @@ | ||
| { | ||
| "$schema": "https://anthropic.com/claude-code/marketplace.schema.json", | ||
| "name": "engram", | ||
| "version": "6.46.2", | ||
| "description": "Persistent memory for Claude Code and Oh My Pi", | ||
| "owner": { | ||
| "name": "thebtf" | ||
| }, | ||
| "plugins": [ | ||
| { | ||
| "name": "engram", | ||
| "description": "Persistent shared memory with MCP integration", | ||
| "version": "6.46.2", | ||
| "author": { | ||
| "name": "thebtf" | ||
| }, | ||
| "source": "./plugin/engram", | ||
| "category": "productivity" | ||
| } | ||
| ] | ||
| } |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -245,22 +245,32 @@ before proxying browser routes. | |
| <!-- redoc:start:installation --> | ||
| ## Installation | ||
|
|
||
| ### Plugin Install (recommended) | ||
| ### Plugin install (recommended) | ||
|
|
||
| The plugin registers the MCP server, hooks, and slash commands automatically. | ||
| The marketplace plugin registers the MCP server, skills, and slash commands in | ||
| Claude Code and Oh My Pi. Claude Code also activates the bundled lifecycle hooks; | ||
| OMP 17.x does not execute Claude `hooks.json`, so automatic capture and context | ||
| injection remain Claude-only. | ||
|
|
||
| ```bash | ||
| # Set environment variables first | ||
| ENGRAM_URL=http://your-server:37777 | ||
| ENGRAM_TOKEN=engram_your_workstation_keycard | ||
| ``` | ||
| Install the marketplace plugin first, then run `/engram:setup` to create the | ||
| universal `~/.engram/config.json` configuration and restart the host. | ||
|
Comment on lines
+255
to
+256
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
For a fresh Claude Code or OMP installation following this new recommended flow, AGENTS.md reference: AGENTS.md:L30-L39 Useful? React with 👍 / 👎. |
||
|
|
||
| Claude Code: | ||
|
|
||
| ``` | ||
| /plugin marketplace add thebtf/engram-marketplace | ||
| /plugin install engram | ||
| ``` | ||
|
|
||
| Restart Claude Code. Everything is configured. | ||
| Oh My Pi: | ||
|
|
||
| ```bash | ||
| omp plugin marketplace add thebtf/engram-marketplace | ||
| omp plugin install engram@engram | ||
|
Comment on lines
+268
to
+269
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
When an OMP user runs this newly documented command, it clones AGENTS.md reference: AGENTS.md:L21-L22 Useful? React with 👍 / 👎. |
||
| ``` | ||
|
|
||
| Restart the agent host after configuration. | ||
|
|
||
|
|
||
| ### Docker Compose | ||
|
|
||
|
|
||
This file was deleted.
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,3 +1,7 @@ | ||
| --- | ||
| description: Diagnose Engram MCP connectivity and subsystem health | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
When an OMP user discovers the newly advertised AGENTS.md reference: AGENTS.md:L30-L39 Useful? React with 👍 / 👎. |
||
| --- | ||
|
|
||
| # Engram Doctor | ||
|
|
||
| Diagnose Engram connectivity and health. Tests the actual MCP connection, not just environment variables. | ||
|
|
||
This file was deleted.
This file was deleted.
This file was deleted.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
When an OMP user invokes the newly exposed
/engram:issue, its mandatory tracked-project check runscurlwith$ENGRAM_AUTH_ADMIN_TOKENagainsthttp://$ENGRAM_HOST/api/issues/tracked-projects(plugin/engram/commands/issue.md), but the OMP setup added here only writesserver_url/api_tokento~/.engram/config.json,ENGRAM_HOSThas no live definition elsewhere in the repository, and workstation operator tokens are explicitly forbidden. A correctly configured OMP installation therefore fails before triage; rewrite this check against the live worker-auth/config path or do not advertise this slash command to OMP.AGENTS.md reference: AGENTS.md:L32-L39
Useful? React with 👍 / 👎.