docs: refresh current project stats#330
Conversation
|
@honor2030 is attempting to deploy a commit to the rohitg00's projects Team on Vercel. A member of the Team first needs to authorize it. |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
✅ Files skipped from review due to trivial changes (1)
📝 WalkthroughWalkthroughUpdates project statistics documentation across AGENTS.md and README.md. AGENTS.md "Current Stats" section upgrades from v0.8.9 to v0.9.11 with revised tool, endpoint, and test counts. README badges and inline comments update test totals from 827 to 877 and REST endpoints from 104 to 107. ChangesProject Statistics Update
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Possibly related PRs
🚥 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 |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 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 `@README.md`:
- Line 910: The README badge showing "104 endpoints" is stale; update the badge
text/count in README.md to match the current number of REST endpoints (change
"104 endpoints" to "107 endpoints") and ensure the same source-of-truth changes
were applied: verify triggers/api.ts has the new endpoints registered and
index.ts contains the updated endpoint count in its log line so all three places
(triggers/api.ts, index.ts, and README.md) remain consistent.
🪄 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: 49b8c316-a7c6-4a38-be93-0ebf82ca45bc
⛔ Files ignored due to path filters (2)
assets/tags/light/stat-tests.svgis excluded by!**/*.svgassets/tags/stat-tests.svgis excluded by!**/*.svg
📒 Files selected for processing (2)
AGENTS.mdREADME.md
| | Custom plugin systems | `iii worker add <name>` | | ||
|
|
||
| **118 source files · ~21,800 LOC · 800 tests · 123 functions · 34 KV scopes** — all on three primitives. No `agentmemory plugin install`. The plugin system is iii itself. | ||
| **118 source files · ~21,800 LOC · 877 tests · 123 functions · 34 KV scopes** — all on three primitives. No `agentmemory plugin install`. The plugin system is iii itself. |
There was a problem hiding this comment.
Stats refresh is incomplete: REST endpoint badge is still stale.
After this update, README still shows 104 endpoints at Line 155, while the document now states 107 endpoints at Line 1021. Please update the badge text/count to keep the public metrics consistent.
Based on learnings: "When adding REST endpoints, update: triggers/api.ts (endpoint registration), index.ts (endpoint count in log line), and README.md (endpoint count)".
🤖 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 910, The README badge showing "104 endpoints" is stale;
update the badge text/count in README.md to match the current number of REST
endpoints (change "104 endpoints" to "107 endpoints") and ensure the same
source-of-truth changes were applied: verify triggers/api.ts has the new
endpoints registered and index.ts contains the updated endpoint count in its log
line so all three places (triggers/api.ts, index.ts, and README.md) remain
consistent.
1 participant
Summary
AGENTS.mdstats from v0.8.9 to v0.9.11Evidence
PATH=/opt/homebrew/bin:$PATH npx tsx -e "import { getAllTools } from './src/mcp/tools-registry.ts'; console.log('mcp_tools', getAllTools().length);"→mcp_tools 51src/index.tsendpoint log currently reports107 RESTPATH=/opt/homebrew/bin:$PATH npm test -- --reporter=basic→81 passed,877 passedTest Plan
PATH=/opt/homebrew/bin:$PATH npm test -- --reporter=basicpython3stale-stats check for old699,827,800,v0.8.9,44 MCP tools,104 REST endpointsgit diff --checkRefs docs hygiene follow-up; does not change runtime behavior.
Summary by CodeRabbit
Release Notes