Skip to content

landing: auth indicator lock icons and grouped per-agent details - #516

Merged
777genius merged 1 commit into
mainfrom
fix/landing-auth-indicator-icons
Oct 25, 2026
Merged

777genius merged 1 commit into
mainfrom
fix/landing-auth-indicator-icons

Conversation

@777genius

Copy link
Copy Markdown
Owner

Summary

Closed-lock icon for authentication required, open-lock for no authentication required, "?" for unknown. Distinct wavy-lock for varies. Popover restructured into a grouped list by policy instead of prose wall.

Testing

  • red-first: extended unit test suite (5/5 green) + browser spec (240/240 green)
  • i18n parity check passes for 9 locales
  • lint: 13 pre-existing warnings, 0 errors
  • vue-tsc: 0 typescript errors

Live Preview

  • Branch: fix/landing-auth-indicator-icons → CI deploy to Pages
  • Browse the landing page and click auth indicators on plugin cards

@coderabbitai

coderabbitai Bot commented Oct 25, 2026

Copy link
Copy Markdown

Warning

Review limit reached

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Next included review available in 21 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

Learn how review limits work.

Review configuration:

Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: c618a2c6-1f33-4800-8d94-3964d4b3a011
Commits

Reviewing files that changed from the base of the PR and between a6beceb and 1f986cb.

Files selected for processing: 13
  • landing/components/registry/AuthenticationIndicator.vue
  • landing/locales/ar.json
  • landing/locales/en.json
  • landing/locales/es.json
  • landing/locales/fr.json
  • landing/locales/hi.json
  • landing/locales/pt.json
  • landing/locales/ru.json
  • landing/locales/uk.json
  • landing/locales/zh.json
  • landing/tests/authentication.test.ts
  • landing/tests/browser/authentication-indicator.spec.ts
  • landing/utils/authentication.ts
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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.

@codecov

codecov Bot commented Oct 25, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 62.81%. Comparing base (62fb629) to head (1f986cb).
⚠️ Report is 29 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #516      +/-   ##
==========================================
+ Coverage   62.59%   62.81%   +0.21%     
==========================================
  Files        1619     1648      +29     
  Lines       94313    96716    +2403     
==========================================
+ Hits        59039    60754    +1715     
- Misses      26837    27279     +442     
- Partials     8437     8683     +246     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@777genius
777genius merged commit 8e7be58 into main Oct 25, 2026
38 of 41 checks passed
@777genius
777genius deleted the fix/landing-auth-indicator-icons branch October 25, 2026 11:11
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