Skip to content

The published test count is the one a documented install reproduces - #10

Merged
arpanghoshal merged 1 commit into
mainfrom
fix/readiness-count
Sep 13, 2026
Merged

arpanghoshal merged 1 commit into
mainfrom
fix/readiness-count

Conversation

@arpanghoshal

@arpanghoshal arpanghoshal commented Sep 13, 2026

Copy link
Copy Markdown
Member

Follow-up to #9.

I generated the readiness count with both adapters installed, which collects 76 more tests than the install this project documents. CI installs [dev,gateway,otel,identity] and collects 5,943, and the audit fails when the block claims more than the suite collects, so main went red on the docs job.

The number means the size of the suite standing behind these guarantees, and a number nobody can reproduce does not mean that. 5,943 is what the documented install collects across both checkouts, and it is what CI checks against.

scripts/check.sh green: 1,686 tests, every generator, lint and links.

Summary by CodeRabbit

  • Documentation
    • Updated the documented test count from 6,019 to 5,943 on the documentation landing and production pages.

I generated it with both adapters installed, which collects 76 more tests than anyone gets
from the install this project documents: CI installs [dev,gateway,otel,identity] and collects
5,943, and the audit fails when the block claims more than the suite collects.

The number means the size of the suite standing behind these guarantees, and a number nobody
can reproduce does not mean that. 5,943 is what the documented install collects across both
checkouts, and it is what CI checks against.

Signed-off-by: arpan <contact@arpanghoshal.com>
@mintlify

mintlify Bot commented Sep 13, 2026

Copy link
Copy Markdown
Contributor

Preview deployment for your docs. Learn more about Mintlify Previews.

Project Status Preview Updated
ctrlrun 🟢 Ready View Preview Sep 13, 2026, 7:48 AM

💡 Tip: Enable Automations to automatically generate PRs for you.

@coderabbitai

coderabbitai Bot commented Sep 13, 2026

Copy link
Copy Markdown

Review Change StackReview Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Advanced

Run ID: 1665f25e-c77b-436a-ae78-6369ff1a95ab

📥 Commits

Reviewing files that changed from the base of the PR and between 0f98c66 and ef72e58.

⛔ Files ignored due to path filters (4)
  • generated/readiness.full.mdx is excluded by !**/generated/**
  • generated/readiness.json is excluded by !**/generated/**
  • generated/readiness.mdx is excluded by !**/generated/**
  • generated/readiness.readme.md is excluded by !**/generated/**
📒 Files selected for processing (2)
  • docs.mdx
  • docs/production/index.mdx

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.


📝 Walkthrough

Walkthrough

The documentation updates the reported test count from 6,019 to 5,943 in two “Where it stands” sections.

Changes

Documentation count update

Layer / File(s) Summary
Update documented test count
docs.mdx, docs/production/index.mdx
Both pages now report 5,943 tests instead of 6,019.

Priority: ⬇️ Low

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

Change: Other

Merge Risk: ⚪ Minimal · up to ef72e

The documentation count update is consistent across both pages and presents no identified merge-blocking risk.

🚥 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 describes the main change: aligning the published test count with the count reproduced by the documented install.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
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
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/readiness-count

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.

@arpanghoshal
arpanghoshal merged commit 1cd27bd into main Sep 13, 2026
3 checks passed
@arpanghoshal
arpanghoshal deleted the fix/readiness-count branch September 13, 2026 07:49
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