Conversation
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
WalkthroughA new "BMad Ecosystem" sidebar section was added to the Starlight configuration with four external links—BMad Method, BMad Builder, Creative Intelligence Suite, and Test Architect (TEA)—each configured to open in new browser tabs. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Poem
✨ Finishing Touches📝 Generate docstrings
🧪 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 |
🤖 Augment PR SummarySummary: Adds a new Changes:
🤖 Was this summary useful? React with 👍 or 👎 |
| label: 'BMad Ecosystem', | ||
| collapsed: false, | ||
| items: [ | ||
| { label: 'BMad Method', link: 'https://docs.bmad-method.org/', attrs: { target: '_blank' } }, |
There was a problem hiding this comment.
All these links use target: '_blank' but don’t set a rel (e.g., noopener), which can allow reverse-tabnabbing via window.opener when opening external sites. Consider ensuring a safe rel attribute is applied anywhere _blank is used.
Severity: medium
Other Locations
website/astro.config.mjs:112website/astro.config.mjs:113website/astro.config.mjs:117
🤖 Was this useful? React with 👍 or 👎, or 🚀 if it prevented an incident/outage.
Summary by CodeRabbit