Skip to content

Comments

Add ContextMemory integration guide#551

Closed
AkashKobal wants to merge 2 commits intosipeed:mainfrom
AkashKobal:patch-1
Closed

Add ContextMemory integration guide#551
AkashKobal wants to merge 2 commits intosipeed:mainfrom
AkashKobal:patch-1

Conversation

@AkashKobal
Copy link

Adds documentation for the ContextMemory CLI tool for persistent AI coding context memory.

npm:
https://www.npmjs.com/package/@akashkobal/contextmemory

GitHub:
https://github.com/AkashKobal/contextmemory

This guide explains how to use ContextMemory for persisting development context in picoclaw. It includes installation instructions, initialization steps, and usage examples.
Copilot AI review requested due to automatic review settings February 20, 2026 20:50
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds a new documentation page describing how contributors can optionally use the external ContextMemory CLI to persist/restore development context across AI-assisted coding sessions.

Changes:

  • Added docs/CONTEXTMEMORY.md integration guide (install/init/save/resume)
  • Documented optional MCP configuration for ContextMemory

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings February 20, 2026 20:56
@AkashKobal
Copy link
Author

@copilot open a new pull request to apply changes based on the comments in this thread

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 5 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines +91 to +94
## Optional MCP Integration

If using an MCP-compatible AI tool, add the following configuration:

Copy link

Copilot AI Feb 20, 2026

Choose a reason for hiding this comment

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

The MCP section says "add the following configuration" but doesn’t specify where (which config file / which MCP client(s), and whether this is per-user or per-repo). Please name the target config location(s) and any required restart/enable steps so contributors can apply this correctly.

Copilot uses AI. Check for mistakes.
Comment on lines +99 to +100
"command": "npx",
"args": ["-y", "@akashkobal/contextmemory", "mcp"]
Copy link

Copilot AI Feb 20, 2026

Choose a reason for hiding this comment

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

Using npx with -y and without a version pin will execute whatever package is currently published, which is a supply-chain risk and can reduce reproducibility. Prefer pinning an explicit version (and consider avoiding -y, or at least explaining the security tradeoff) in the recommended MCP command.

Suggested change
"command": "npx",
"args": ["-y", "@akashkobal/contextmemory", "mcp"]
"command": "contextmemory",
"args": ["mcp"]

Copilot uses AI. Check for mistakes.
Comment on lines +3 to +4
This guide explains how picoclaw contributors can optionally use ContextMemory to persist development context across AI-assisted coding sessions.

Copy link

Copilot AI Feb 20, 2026

Choose a reason for hiding this comment

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

Use consistent project naming: in prose, other docs refer to the project as "PicoClaw" (capitalized). Reserve picoclaw for CLI commands/paths, and update occurrences like "picoclaw contributors" accordingly (ideally throughout this document for consistency).

Copilot uses AI. Check for mistakes.
Comment on lines +44 to +55
This creates a `.contextmemory/` directory used to store development context locally.

Example structure:

```
.contextmemory/
├── context.json
├── history/
```

This directory is local to your development environment.

Copy link

Copilot AI Feb 20, 2026

Choose a reason for hiding this comment

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

Consider calling out that .contextmemory/ should not be committed. Adding a note here to include .contextmemory/ in .gitignore (similar to how .picoclaw/ is ignored) would help prevent accidental check-ins of local context/history.

Copilot uses AI. Check for mistakes.
```bash
npm install -g @akashkobal/contextmemory
```

Copy link

Copilot AI Feb 20, 2026

Choose a reason for hiding this comment

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

This repo’s exec tool blocks npm install -g by default (see docs/tools_configuration.md default deny patterns). It may be worth noting here that the install command should be run in a normal terminal, or that users must adjust exec deny patterns if attempting to run it via PicoClaw tooling.

Suggested change
Note: The picoclaw exec tool blocks `npm install -g` by default (see `docs/tools_configuration.md`). Run this install command in a normal terminal, or adjust the exec deny patterns if you need to run it via PicoClaw tooling.

Copilot uses AI. Check for mistakes.
@xiaket
Copy link
Collaborator

xiaket commented Feb 22, 2026

Going to close this for now. Not a real contribution.

@xiaket xiaket closed this Feb 22, 2026
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