docs: fix broken links and lychee resolution errors#474
Conversation
- Update lychee.toml to use unanchored regex patterns for internal CSS imports and Tailwind directives to correctly handle local file URI resolution. - Add @fontsource to Lychee exclusions to ignore font package references in CSS. - Correct the version comparison link in CHANGELOG.md from v1.0.0...v2.0.0 to v0.5.0...v2.0.0 as v1.0.0 does not exist.
|
👋 Jules, reporting for duty! I'm here to lend a hand with this pull request. When you start a review, I'll add a 👀 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down. I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job! For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with New to Jules? Learn more at jules.google/docs. For security, I will only act on instructions from the user who triggered this task. |
Deploying corvus with
|
| Latest commit: |
da135ad
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://8e53e046.corvus-42x.pages.dev |
| Branch Preview URL: | https://fix-broken-links-12950876164.corvus-42x.pages.dev |
|
Note Reviews pausedIt looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the Use the following commands to manage reviews:
Use the checkboxes below for quick actions:
📝 WalkthroughWalkthroughUpdated changelog compare range to start from Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related PRs
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 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 |
✅ Contributor ReportUser: @yacosta738
Contributor Report evaluates based on public GitHub activity. Analysis period: 2025-04-10 to 2026-04-10 |
![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 the current code and only fix it if needed.
Inline comments:
In `@lychee.toml`:
- Around line 45-47: The exclusion regexes '.*@corvus/shared/.*',
'.*tailwindcss.*', and '.*@fontsource.*' are too broad and are matching external
https links; update them to only match npm package import paths by anchoring and
constraining the patterns to package-name formats (e.g., require leading
`@/package` segments and/or start/end anchors so they match only package
specifiers like `@corvus/shared/`... , tailwindcss (package), or
`@fontsource/`<pkg>), replacing the loose '.*' wrappers with precise anchors and
path separators so legitimate URLs (like https://tailwindcss.com/...) are no
longer excluded.
🪄 Autofix (Beta)
✅ Autofix completed
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: ASSERTIVE
Plan: Pro
Run ID: ac67e311-1800-464a-90d6-4521845d2452
📒 Files selected for processing (2)
CHANGELOG.mdlychee.toml
📜 Review details
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (4)
- GitHub Check: sonar
- GitHub Check: pr-checks
- GitHub Check: submit-gradle
- GitHub Check: Cloudflare Pages
🧰 Additional context used
📓 Path-based instructions (2)
**/*.{md,mdx}
⚙️ CodeRabbit configuration file
**/*.{md,mdx}: Verify technical accuracy and that docs stay aligned with code changes.
For user-facing docs, check EN/ES parity or explicitly note pending translation gaps.
Files:
CHANGELOG.md
**/*
⚙️ CodeRabbit configuration file
**/*: Security first, performance second.
Validate input boundaries, auth/authz implications, and secret management.
Look for behavioral regressions, missing tests, and contract breaks across modules.
Files:
CHANGELOG.mdlychee.toml
🔇 Additional comments (1)
CHANGELOG.md (1)
23-23: Compare-link base correction looks correct.The
v0.5.0...v2.0.0range aligns with the broken-link fix objective and is the right changelog metadata update.As per coding guidelines, "Verify technical accuracy and that docs stay aligned with code changes."
|
Note Autofix is a beta feature. Expect some limitations and changes as we gather feedback and continue to improve it. Fixes Applied SuccessfullyFixed 1 file(s) based on 1 unresolved review comment. Files modified:
Commit: The changes have been pushed to the Time taken: |
Fixed 1 file(s) based on 1 unresolved review comment. Co-authored-by: CodeRabbit <noreply@coderabbit.ai>
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@lychee.toml`:
- Line 47: The exclude regex '^@fontsource/' in lychee.toml doesn't match
imports like '@fontsource-variable/space-grotesk'; update the pattern to cover
both variants by replacing '^@fontsource/' with a regex that accepts the
optional "-variable" segment (for example '^@fontsource(-variable)?/'), so the
exclude will match both '@fontsource/...' and '@fontsource-variable/...'.
🪄 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: 45ec23c3-743f-44e5-a093-08de84bc1d2c
📒 Files selected for processing (1)
lychee.toml
📜 Review details
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (4)
- GitHub Check: sonar
- GitHub Check: pr-checks
- GitHub Check: submit-gradle
- GitHub Check: Cloudflare Pages
🧰 Additional context used
📓 Path-based instructions (1)
**/*
⚙️ CodeRabbit configuration file
**/*: Security first, performance second.
Validate input boundaries, auth/authz implications, and secret management.
Look for behavioral regressions, missing tests, and contract breaks across modules.
Files:
lychee.toml
Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com> Signed-off-by: Yuniel Acosta Pérez <33158051+yacosta738@users.noreply.github.com>
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@lychee.toml`:
- Around line 45-49: The lychee exclusion list currently uses the pattern
'^@corvus/shared/' which only matches subpaths and is duplicated; update the
pattern to match both the bare package and subpaths (e.g., change
'^@corvus/shared/' to a single pattern like '^@corvus/shared(?:$|/)') and remove
the duplicate entries (also dedupe any repeated '^tailwindcss(/|$)' lines) so
the bare imports referenced in useGateway.ts, useConfig.ts, useCostGovernance.ts
and marketing index.astro are correctly excluded.
🪄 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: 6ea0338b-6aaf-415d-8a0e-4ae6d834ecbe
📒 Files selected for processing (1)
lychee.toml
📜 Review details
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (4)
- GitHub Check: submit-gradle
- GitHub Check: sonar
- GitHub Check: pr-checks
- GitHub Check: Cloudflare Pages
🧰 Additional context used
📓 Path-based instructions (1)
**/*
⚙️ CodeRabbit configuration file
**/*: Security first, performance second.
Validate input boundaries, auth/authz implications, and secret management.
Look for behavioral regressions, missing tests, and contract breaks across modules.
Files:
lychee.toml
Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com> Signed-off-by: Yuniel Acosta Pérez <33158051+yacosta738@users.noreply.github.com>
|
1 participant
This PR fixes all 21 broken link errors reported by Lychee.
Key changes:
lychee.tomlto handle localfile://URI resolution for internal package imports. The previous anchored patterns failed when Lychee converted CSS imports into absolute file paths. Added@fontsourceto exclusions.v0.5.0instead of the non-existentv1.0.0.Fixes #464
PR created automatically by Jules for task 12950876164370050822 started by @yacosta738