Add homepage field to package.json#664
Conversation
Tests (1081) + build pass - per upstream commit 3551241. Signed-off-by: Hermes Agent <root@okwn.cc>
|
Someone is attempting to deploy a commit to the rohitg00's projects Team on Vercel. A member of the Team first needs to authorize it. |
📝 WalkthroughWalkthroughThis PR updates project metrics and metadata across documentation files and package configuration. Test count references are refreshed from 950+ to 1081+, MCP tool availability from 51 to 53, REST endpoint count from 124 to 130, and a homepage field is added to package.json. ChangesDocumentation and metadata updates
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 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 |
There was a problem hiding this comment.
Actionable comments posted: 2
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@package.json`:
- Line 85: Update package.json so the "homepage" field matches the canonical
repository URL referenced by "repository.url" (i.e., use the GitHub URL for
rohitg00/agentmemory). Edit the "homepage" value to the corresponding
https://github.com/rohitg00/agentmemory (or change "repository.url" to
okwn/agentmemory if that is the intended canonical repo) to ensure both fields
are consistent.
In `@README.md`:
- Line 444: Update README count references to be consistent: search for numeric
mentions of "tools" and "endpoints" (e.g., the string "`@agentmemory/mcp`" /
"AGENTMEMORY_URL", occurrences of "51 tools", and "124 endpoints") and replace
stale values so they match the canonical metrics used in the updated lines —
specifically ensure the MCP description reflects "53 tools" when AGENTMEMORY_URL
points at a running agentmemory server (fallback "7 tools" locally) and update
any "124 endpoints" references to the current endpoint count used elsewhere in
the README.
🪄 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: 076543e4-faee-45d1-acbe-423b7d501f98
📒 Files selected for processing (3)
AGENTS.mdREADME.mdpackage.json
| "node": ">=20.0.0" | ||
| } | ||
| }, | ||
| "homepage": "https://github.com/okwn/agentmemory" |
There was a problem hiding this comment.
Homepage URL appears inconsistent with the package repository metadata.
homepage points to okwn/agentmemory, while repository.url points to rohitg00/agentmemory. If this package is published from the main repo, homepage should likely match that canonical URL.
Proposed fix
- "homepage": "https://github.com/okwn/agentmemory"
+ "homepage": "https://github.com/rohitg00/agentmemory"📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| "homepage": "https://github.com/okwn/agentmemory" | |
| "homepage": "https://github.com/rohitg00/agentmemory" |
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@package.json` at line 85, Update package.json so the "homepage" field matches
the canonical repository URL referenced by "repository.url" (i.e., use the
GitHub URL for rohitg00/agentmemory). Edit the "homepage" value to the
corresponding https://github.com/rohitg00/agentmemory (or change
"repository.url" to okwn/agentmemory if that is the intended canonical repo) to
ensure both fields are consistent.
| The Codex plugin ships from the same `plugin/` directory as the Claude Code plugin. It registers: | ||
|
|
||
| - `@agentmemory/mcp` as an MCP server (proxies all 51 tools when `AGENTMEMORY_URL` points at a running agentmemory server; falls back to 7 tools locally when no server is reachable) | ||
| - `@agentmemory/mcp` as an MCP server (proxies all 53 tools when `AGENTMEMORY_URL` points at a running agentmemory server; falls back to 7 tools locally when no server is reachable) |
There was a problem hiding this comment.
Refresh all README count references to avoid conflicting docs.
Some count references were updated, but stale values remain (e.g., Line 821/Line 823 still mention 51 tools, Line 1244 still says 124 endpoints). Please make README counts consistent everywhere with the new metrics.
As per coding guidelines: "README.md: When adding or removing MCP tools or REST endpoints, update tool and endpoint counts in the documentation".
Also applies to: 466-466, 491-491, 1071-1071, 1278-1278
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@README.md` at line 444, Update README count references to be consistent:
search for numeric mentions of "tools" and "endpoints" (e.g., the string
"`@agentmemory/mcp`" / "AGENTMEMORY_URL", occurrences of "51 tools", and "124
endpoints") and replace stale values so they match the canonical metrics used in
the updated lines — specifically ensure the MCP description reflects "53 tools"
when AGENTMEMORY_URL points at a running agentmemory server (fallback "7 tools"
locally) and update any "124 endpoints" references to the current endpoint count
used elsewhere in the README.
Summary by CodeRabbit