fix(website): anonymise reviewer testimonials and drop architecture from non-home nav#74
Conversation
…rom non-home nav Reviewer names removed from public-facing surfaces. Each validation card now renders the reviewer's role as the headline and a generic affiliation as the subline. Internal name field is kept in config for our own records but never rendered. Identifying organisation strings (EAS, INRIA PRIVATICS, Moca Network, etc.) replaced with generic descriptors so the config alone cannot be used to identify a reviewer. Validation intro paragraph reworded to drop EAS-founder and INRIA identifiers. Architecture removed from every non-home surface: footer link replaced with Offline verify, docs guides sidebar entry removed, llms.txt entry removed, eu-compliance-walkthrough cross-link removed. The architecture diagram on the landing page (ConsolePreviewSection) is preserved. The architecture.mdx file remains on disk but is unlinked. Changelog mention of named reviewers also anonymised.
|
Warning Rate limit exceeded
You’ve run out of usage credits. Purchase more in the billing tab. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Pro Run ID: 📒 Files selected for processing (1)
Note
|
| Layer / File(s) | Summary |
|---|---|
Testimonial Configuration & Schema website/src/lib/config.tsx |
Five highlight entries (id: 1, 2, 3, 5, 6) updated with anonymous: true as const and adjusted role/company metadata; documentation clarifies names kept on file but hidden publicly. |
Anonymous Rendering Implementation website/src/components/sections/v2/validation.tsx |
New displayName and displaySubline helper functions encode logic: when anonymous, show role + company; otherwise show name + role. Figcaption rendering unified to conditionally display single subline value. "Reviewed by" descriptive text updated with anonymous-aware wording. |
Documentation & Navigation Updates website/content/docs/guides/eu-compliance-walkthrough.mdx, website/public/llms.txt, website/src/components/sections/v2/footer-v2.tsx, website/content/docs/guides/meta.json |
Architecture Overview link removed from docs; offline-verify guide added to footer; base-l2-anchor added to guides metadata. |
Changelog & Release Notes website/content/docs/project/changelog.mdx |
Version 0.2.4 testimonials entry updated to note anonymous reviewer validation with names kept on file. |
Estimated code review effort
🎯 2 (Simple) | ⏱️ ~12 minutes
Possibly related PRs
- VibeTensor/attestix#62: Modifies guides metadata (meta.json) and testimonial configuration in website/src/lib/config.tsx with related anonymous/guide page changes.
- VibeTensor/attestix#3: Updates the same siteConfig object in website/src/lib/config.tsx.
🚥 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 and concisely summarizes the two main changes: anonymizing reviewer testimonials and removing the architecture link from non-home navigation surfaces. |
| 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. |
✏️ Tip: You can configure your own custom pre-merge checks in the settings.
✨ Finishing Touches
🧪 Generate unit tests (beta)
- Create PR with unit tests
- Commit unit tests in branch
fix/anonymise-reviewers
Tip
💬 Introducing Slack Agent: The best way for teams to turn conversations into code.
Slack Agent is built on CodeRabbit's deep understanding of your code, so your team can collaborate across the entire SDLC without losing context.
- Generate code and open pull requests
- Plan features and break down work
- Investigate incidents and troubleshoot customer tickets together
- Automate recurring tasks and respond to alerts with triggers
- Summarize progress and report instantly
Built for teams:
- Shared memory across your entire org—no repeating context
- Per-thread sandboxes to safely plan and execute work
- Governance built-in—scoped access, auditability, and budget controls
One agent for your entire SDLC. Right inside Slack.
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 help to get the list of available commands and usage tips.
Deploying attestix with
|
| Latest commit: |
30a9294
|
| Status: | ⚡️ Build in progress... |
![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 `@website/src/components/sections/v2/validation.tsx`:
- Around line 60-62: Compute displaySubline(h) once and reuse it instead of
calling it twice: call displaySubline(h) into a local variable (e.g., const
subline = displaySubline(h)) in the component/render scope that contains the
JSX, then use that variable for the conditional check and for rendering. Update
the JSX that currently references displaySubline(h) twice to reference the new
variable (subline) for both the boolean check and the rendered value.
🪄 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: Path: .coderabbit.yaml
Review profile: ASSERTIVE
Plan: Pro
Run ID: 0e470909-e0b2-4fed-880b-10f454a234c0
📒 Files selected for processing (7)
website/content/docs/guides/eu-compliance-walkthrough.mdxwebsite/content/docs/guides/meta.jsonwebsite/content/docs/project/changelog.mdxwebsite/public/llms.txtwebsite/src/components/sections/v2/footer-v2.tsxwebsite/src/components/sections/v2/validation.tsxwebsite/src/lib/config.tsx
💤 Files with no reviewable changes (2)
- website/public/llms.txt
- website/content/docs/guides/eu-compliance-walkthrough.mdx
ascender1729
dismissed
coderabbitai[bot]’s stale review
Addressed in 30a9294 — displaySubline call is now memoised.
1 participant
Summary
namefield kept in config for our records but never rendered.architecture.mdxremains on disk but is unlinked.Test plan
npm run devinwebsite/, open/, confirm validation cards show role-as-headline and no real names/docs/guides, confirm Architecture is not in the sidebarcurl -Iagainst/docs/guides/architecturestill resolves (file kept on disk for any inbound bookmark)Summary by CodeRabbit
Documentation
Updates