Skip to content

chore: refresh the generated endpoint counts so CI is green on main - #1144

Open
dmazhukov wants to merge 1 commit into
rohitg00:mainfrom
dmazhukov:chore/refresh-generated-endpoint-counts
Open

chore: refresh the generated endpoint counts so CI is green on main#1144
dmazhukov wants to merge 1 commit into
rohitg00:mainfrom
dmazhukov:chore/refresh-generated-endpoint-counts

Conversation

@dmazhukov

@dmazhukov dmazhukov commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

Summary

CI is currently red on main itself, so every open PR inherits two failures it did not cause.
This refreshes the four places that carry a hardcoded endpoint count.

An endpoint landed in #1132 / #1136 without the generated docs being refreshed alongside it:

  • npm run skills:checkDRIFT: plugin/skills/agentmemory-rest-api/REFERENCE.md (says 118, generator produces 119)
  • test/consistency.test.ts > documented REST endpoint counts match registered API paths → README says 129 endpoints on port, 130 are registered

AGENTS.md and the boot banner in src/index.ts carry the same number and the consistency
test checks all three, so all four files move together.

Verification

On a clean checkout of main:

npm run skills:check   →  DRIFT: … REFERENCE.md (AUTOGEN:rest out of date)
npx vitest run test/consistency.test.ts  →  1 failed | 15 passed

With this change:

npm run skills:check   →  Skill lint passed: 15 skills checked.
npx vitest run --exclude test/integration.test.ts  →  1546 passed | 1 skipped
npm run build          →  clean

REFERENCE.md is the generator's own output (npm run skills:gen), not hand-edited.

Noticed while working on #1142; keeping it separate so that PR stays one logical change.

Summary by CodeRabbit

  • Documentation

    • Updated REST API documentation to reflect the current total of 130 endpoints.
    • Updated the API reference to report 119 registered endpoints.
  • Chores

    • Updated the application startup information to display the current endpoint count.

Signed-off-by: Dmitrii Zhukov <dmitry0983@gmail.com>
@vercel

vercel Bot commented Aug 3, 2026

Copy link
Copy Markdown

@dmazhukov is attempting to deploy a commit to the rohitg00's projects Team on Vercel.

A member of the Team first needs to authorize it.

@coderabbitai

coderabbitai Bot commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 9bec0f4a-742f-4d19-b83f-9da1bff10005

📥 Commits

Reviewing files that changed from the base of the PR and between 5023cf3 and 48bacdf.

📒 Files selected for processing (4)
  • AGENTS.md
  • README.md
  • plugin/skills/agentmemory-rest-api/REFERENCE.md
  • src/index.ts

📝 Walkthrough

Walkthrough

The change synchronizes REST endpoint counts in AGENTS.md, README.md, the generated API reference, and the startup log.

Changes

REST endpoint count synchronization

Layer / File(s) Summary
Update documented and reported counts
AGENTS.md, README.md, plugin/skills/agentmemory-rest-api/REFERENCE.md, src/index.ts
Documentation and startup logging now report 130 REST endpoints. The generated reference reports 119 registered endpoints.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Possibly related PRs

Suggested reviewers: rohitg00, honor2030, somsamantray

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly summarizes the endpoint count updates and their purpose of resolving CI failures.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches 💡 1
🛠️ Fix failing CI checks 💡
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

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