Conversation
nmem-cli-based plugin replaces the old prompt package. Skills: read-working-memory, search-memory, save-thread, distill-memory, status. Progressive loading — each skill injects context only when triggered. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- integrations.json: Codex entry updated to type plugin, v0.1.0 - README: Codex row now points to plugin install - codex-prompts: deprecation notice added - check-integration skill: Codex install command updated Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- plugin.json: developerName → Nowledge Labs, description leads with user outcome, defaultPrompts use natural language, keywords reflect what users search for - SKILL.md descriptions: remove "Uses nmem CLI" implementation detail, lead with what the user gets - README: rewritten to lead with the promise (remember across tools), verify step uses natural language - AGENTS.md: tighter, more operational Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Replace em dashes with colons, commas, periods, or sentence restructuring throughout all plugin copy. Follows feedback that em dashes read as AI-generated rather than human-polished. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
/docs/features/working-memory, /docs/features/search, and /docs/features/threads were phantom URLs (SPA returns 200 but renders 404 client-side). Fixed to real pages: - Working Memory -> /docs/advanced-features#working-memory - Search -> /docs/search-relevance - Threads -> /docs/threads Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
📝 WalkthroughWalkthroughThis PR migrates the Nowledge Mem integration from a prompt-based to a native Codex plugin architecture. Changes include updating the integration registry, adding a plugin manifest and skill documentation, introducing installation via plugin directory copy instead of remote scripts, and marking the previous prompt-based package as deprecated. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~12 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 inconclusive)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Member
Author
|
bugbot run |
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 2
🧹 Nitpick comments (1)
README.md (1)
36-36: Clarify the fifth skill in the Codex row description.Line 36 says “five composable skills” but the sentence enumerates only four capability areas. Consider explicitly mentioning
statusfor consistency and scanning clarity.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@README.md` at line 36, The README table row for "Codex Plugin (nowledge-mem-codex-plugin)" claims "five composable skills" but only lists four; update that cell to explicitly include the missing "status" skill (e.g., "Working Memory, routed recall, real session save, distillation, and status") so the phrase "five composable skills" matches the enumerated items and improves scannability.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@nowledge-mem-codex-prompts/README.md`:
- Around line 3-4: The blockquote in README.md contains an extra blank line that
triggers markdownlint MD028; edit the block that starts with "**Deprecated**:
This custom prompts package..." to remove the empty line so the entire
deprecation notice is a single continuous blockquote line (no blank line between
sentences) to satisfy the no-blanks-blockquote rule.
In `@nowledge-mem-npx-skills/skills/check-integration/SKILL.md`:
- Line 44: The Codex CLI table row in SKILL.md ("Codex CLI" entry) omits the
`.agents/plugins/nowledge-mem` install target that integrations.json declares as
canonical; update the install instruction in that table cell to include both
install locations (e.g., `~/.codex/plugins/nowledge-mem` and
`~/.agents/plugins/nowledge-mem`) so the Markdown matches the registry. Verify
against the `integrations.json` entry for the Codex integration to ensure naming
and paths exactly match the source-of-truth.
---
Nitpick comments:
In `@README.md`:
- Line 36: The README table row for "Codex Plugin (nowledge-mem-codex-plugin)"
claims "five composable skills" but only lists four; update that cell to
explicitly include the missing "status" skill (e.g., "Working Memory, routed
recall, real session save, distillation, and status") so the phrase "five
composable skills" matches the enumerated items and improves scannability.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: f4a83855-b143-404f-88d5-19d6ff12e658
📒 Files selected for processing (13)
README.mdintegrations.jsonnowledge-mem-codex-plugin/.codex-plugin/plugin.jsonnowledge-mem-codex-plugin/AGENTS.mdnowledge-mem-codex-plugin/CHANGELOG.mdnowledge-mem-codex-plugin/README.mdnowledge-mem-codex-plugin/skills/distill-memory/SKILL.mdnowledge-mem-codex-plugin/skills/read-working-memory/SKILL.mdnowledge-mem-codex-plugin/skills/save-thread/SKILL.mdnowledge-mem-codex-plugin/skills/search-memory/SKILL.mdnowledge-mem-codex-plugin/skills/status/SKILL.mdnowledge-mem-codex-prompts/README.mdnowledge-mem-npx-skills/skills/check-integration/SKILL.md
Comment on lines
+3
to
+4
| > **Deprecated**: This custom prompts package has been superseded by the **[Codex Plugin](../nowledge-mem-codex-plugin/)**. The plugin provides the same capabilities as composable Codex skills with proper plugin lifecycle support. See the [migration guide](https://mem.nowledge.co/docs/integrations/codex-cli#migrating-from-custom-prompts) for details. | ||
|
|
There was a problem hiding this comment.
Fix markdownlint MD028 in deprecation notice.
Line 4 introduces a blank line inside the blockquote, which triggers no-blanks-blockquote.
Suggested markdown fix
> **Deprecated**: This custom prompts package has been superseded by the **[Codex Plugin](../nowledge-mem-codex-plugin/)**. The plugin provides the same capabilities as composable Codex skills with proper plugin lifecycle support. See the [migration guide](https://mem.nowledge.co/docs/integrations/codex-cli#migrating-from-custom-prompts) for details.
-
+🧰 Tools
🪛 markdownlint-cli2 (0.22.0)
[warning] 4-4: Blank line inside blockquote
(MD028, no-blanks-blockquote)
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@nowledge-mem-codex-prompts/README.md` around lines 3 - 4, The blockquote in
README.md contains an extra blank line that triggers markdownlint MD028; edit
the block that starts with "**Deprecated**: This custom prompts package..." to
remove the empty line so the entire deprecation notice is a single continuous
blockquote line (no blank line between sentences) to satisfy the
no-blanks-blockquote rule.
| | **Alma** | Running inside Alma; `~/.config/alma/` exists | In Alma: Settings > Plugins > Marketplace, search "Nowledge Mem" | [Guide](https://mem.nowledge.co/docs/integrations/alma) | | ||
| | **Droid** | Running inside Droid (Factory) | Add nowledge-co/community marketplace, install nowledge-mem@nowledge-community | [Guide](https://mem.nowledge.co/docs/integrations/droid) | | ||
| | **Codex CLI** | Running as Codex CLI agent; `~/.codex/` exists | `curl -fsSL https://raw.githubusercontent.com/nowledge-co/community/main/nowledge-mem-codex-prompts/install.sh \| bash` | [Guide](https://mem.nowledge.co/docs/integrations/codex-cli) | | ||
| | **Codex CLI** | Running as Codex CLI agent; `~/.codex/` exists | Copy `nowledge-mem-codex-plugin` to `~/.codex/plugins/nowledge-mem` | [Guide](https://mem.nowledge.co/docs/integrations/codex-cli) | |
There was a problem hiding this comment.
Align Codex install path with registry source of truth.
Line 44 omits the .agents/plugins/nowledge-mem install location that is present in integrations.json, so this table can drift from the declared canonical source. Please include both targets in the install instruction.
Based on learnings: Maintain integrations.json as the single source of truth for all Nowledge Mem integrations, tracking capabilities, versions, install commands, transport, tool naming, and thread save methods.
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@nowledge-mem-npx-skills/skills/check-integration/SKILL.md` at line 44, The
Codex CLI table row in SKILL.md ("Codex CLI" entry) omits the
`.agents/plugins/nowledge-mem` install target that integrations.json declares as
canonical; update the install instruction in that table cell to include both
install locations (e.g., `~/.codex/plugins/nowledge-mem` and
`~/.agents/plugins/nowledge-mem`) so the Markdown matches the registry. Verify
against the `integrations.json` entry for the Codex integration to ensure naming
and paths exactly match the source-of-truth.
This was referenced Apr 5, 2026
Merged
Merged
This was referenced Apr 16, 2026
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.
Note
Low Risk
Mostly adds documentation/metadata and a new Codex plugin package; the main risk is incorrect integration registry/install instructions affecting setup and discovery.
Overview
Adds a new Codex native plugin (
nowledge-mem-codex-plugin) with aplugin.jsonmanifest, five composable skills (read-working-memory,search-memory,save-thread,distill-memory,status), and accompanyingREADME.md/AGENTS.md/CHANGELOG.md.Updates the integration surfaces (
integrations.json, top-levelREADME.md, and the npxcheck-integrationskill) to switch Codex from the legacynowledge-mem-codex-promptsinstall flow to the new plugin-based install, including a migration/deprecation note for the old prompts package.Written by Cursor Bugbot for commit c6f09c8. This will update automatically on new commits. Configure here.
Summary by CodeRabbit
Release Notes
New Features
Documentation
Deprecated