Skip to content

Add Unforgit plugin#145

Open
MiguelMedeiros wants to merge 1 commit into
cursor:mainfrom
MiguelMedeiros:add-unforgit-plugin
Open

Add Unforgit plugin#145
MiguelMedeiros wants to merge 1 commit into
cursor:mainfrom
MiguelMedeiros:add-unforgit-plugin

Conversation

@MiguelMedeiros

@MiguelMedeiros MiguelMedeiros commented Jul 10, 2026

Copy link
Copy Markdown

Summary

Add Unforgit as a Cursor plugin with:

  • MCP server registration for unforgit-mcp
  • Cursor memory rules
  • Unforgit memory skill
  • install/setup docs and logo

Unforgit is a Git-backed repository memory system for AI coding agents. It lets agents store and recall durable project knowledge such as decisions, conventions, gotchas, and reusable playbooks.

Verification

  • npm install --no-save ajv ajv-formats
  • node scripts/validate-plugins.mjs
  • git diff --check

Note

Low Risk
Documentation and plugin scaffolding only; no changes to core product runtime beyond a new optional MCP integration users must install separately.

Overview
Registers Unforgit in the Cursor plugin marketplace and adds a full unforgit/ plugin package so agents can use Git-backed repository memory via MCP.

The new plugin wires unforgit-mcp through mcp.json, ships unforgit-memory rules and a matching skill (recall-before-work, save durable notes, curation tools), and documents global CLI install plus unforgit init --ide cursor. Metadata lives in plugin.json (v0.9.3, MIT, logo path); marketplace.json gains one new plugin entry.

Reviewed by Cursor Bugbot for commit 1fa2412. Bugbot is set up for automated code reviews on this repo. Configure here.

@cursor cursor 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.

Cursor Bugbot has reviewed your changes using high effort and found 4 potential issues.

Fix All in Cursor

Bugbot Autofix is ON, but it could not run because the branch was deleted or merged before autofix could start.

Reviewed by Cursor Bugbot for commit 1fa2412. Configure here.

Comment thread unforgit/mcp.json
"args": []
}
}
}

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

MCP binary appears unpublished

High Severity · Logic Bug

The plugin's core functionality depends on a global unforgit CLI (exposing unforgit-mcp) and references to unforgit.com and its GitHub repository. These package, binary, site, and repository targets do not appear to exist publicly. This prevents the MCP server from starting and makes the documented setup steps unusable, even if the plugin installs successfully.

Additional Locations (1)
Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit 1fa2412. Configure here.

- Search: `unforgit_recall({ "query": "...", "k": 5 })`
- Save: `unforgit_add({ "text": "...", "type": "semantic", "tags": ["decision"] })`
- Health: `unforgit_health()`
- Sync status: `unforgit_sync_status()`

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Rule missing activation frontmatter

High Severity · Logic Bug

unforgit-memory.mdc has no YAML frontmatter (description, alwaysApply, or globs), so Cursor cannot choose an activation mode. The intended “recall first for substantive work” path will not run automatically, and the rule may show as incorrectly formatted or only apply when manually referenced.

Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit 1fa2412. Configure here.

- Search: `unforgit_recall({ "query": "...", "k": 5 })`
- Save: `unforgit_add({ "text": "...", "type": "semantic", "tags": ["decision"] })`
- Health: `unforgit_health()`
- Sync status: `unforgit_sync_status()`

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Identical rule and skill content

Low Severity · Code Quality

The rule body is a near copy of skills/unforgit-memory/SKILL.md, including the line “Use this skill whenever…”. Shipping both as-is duplicates guidance, risks double injection when both load, and makes future edits easy to diverge.

Additional Locations (1)
Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit 1fa2412. Configure here.

# Unforgit Memory

Use this skill whenever work can benefit from durable project memory or when the user asks you to remember, recall, curate, or inspect repository knowledge.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Rule missing required frontmatter

Medium Severity · Logic Bug

unforgit-memory.mdc has no YAML frontmatter (description, alwaysApply), unlike every other rule in this repo and the create-plugin scaffold/review checklist. Without that metadata Cursor cannot classify activation, so the memory guidance may never attach. The body also opens with “Use this skill,” which is skill wording left in a rule file.

Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit 1fa2412. Configure here.

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