fix(docs): repair broken local links#399
Conversation
Restore valid local docs references and CSS imports so the offline link checker stops flagging false broken paths in the Corvus docs surfaces.
|
Warning Rate limit exceeded
Your organization is not enrolled in usage-based pricing. Contact your admin to enable usage-based pricing to continue reviews beyond the rate limit, or try again in 2 minutes and 54 seconds. ⌛ 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)
📝 WalkthroughWalkthroughThis PR updates documentation link paths throughout the codebase and converts CSS Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~5 minutes Possibly related PRs
Suggested labels
Suggested reviewers
🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 inconclusive)
✅ Passed checks (2 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-03 to 2026-04-03 |
![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 `@clients/web/apps/dashboard/src/style.css`:
- Around line 1-2: The import syntax is inconsistent: style.css uses bare-string
imports (`@import` "@corvus/shared/app-shell.css";) while the imported
app-shell.css still uses `@import` url("..."); — update the import statements in
app-shell.css to use the same bare-string syntax (e.g., `@import` "..." ) so both
files use the consistent form; ensure any occurrences of url("...") at the top
of app-shell.css that reference internal CSS modules are converted to the
bare-string `@import` "module-path" style to match the changed pattern.
🪄 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: 580b412e-3d17-415a-a772-d079046b7ffd
📒 Files selected for processing (11)
clients/agent-runtime/README.mdclients/web/apps/chat/src/style.cssclients/web/apps/dashboard/src/style.cssclients/web/apps/docs/src/content/docs/clients/agent-runtime/tools/core.mdclients/web/apps/docs/src/content/docs/clients/agent-runtime/tools/index.mdxclients/web/apps/docs/src/content/docs/es/clients/agent-runtime/tools/core.mdclients/web/apps/docs/src/content/docs/es/clients/agent-runtime/tools/index.mdxclients/web/apps/docs/src/styles/custom.cssclients/web/apps/marketing/src/styles/global.cssclients/web/packages/shared/tokens.cssmodules/cerebro/README.md
📜 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). (3)
- GitHub Check: pr-checks
- GitHub Check: sonar
- GitHub Check: Cloudflare Pages
🧰 Additional context used
📓 Path-based instructions (2)
**/*
⚙️ 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:
clients/web/packages/shared/tokens.cssclients/web/apps/docs/src/content/docs/es/clients/agent-runtime/tools/core.mdclients/web/apps/docs/src/content/docs/clients/agent-runtime/tools/index.mdxclients/web/apps/chat/src/style.cssclients/web/apps/dashboard/src/style.cssmodules/cerebro/README.mdclients/web/apps/marketing/src/styles/global.cssclients/web/apps/docs/src/content/docs/clients/agent-runtime/tools/core.mdclients/web/apps/docs/src/content/docs/es/clients/agent-runtime/tools/index.mdxclients/web/apps/docs/src/styles/custom.cssclients/agent-runtime/README.md
**/*.{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:
clients/web/apps/docs/src/content/docs/es/clients/agent-runtime/tools/core.mdclients/web/apps/docs/src/content/docs/clients/agent-runtime/tools/index.mdxmodules/cerebro/README.mdclients/web/apps/docs/src/content/docs/clients/agent-runtime/tools/core.mdclients/web/apps/docs/src/content/docs/es/clients/agent-runtime/tools/index.mdxclients/agent-runtime/README.md
🧠 Learnings (4)
📚 Learning: 2026-02-17T12:31:17.076Z
Learnt from: CR
Repo: dallay/corvus PR: 0
File: clients/agent-runtime/AGENTS.md:0-0
Timestamp: 2026-02-17T12:31:17.076Z
Learning: Applies to clients/agent-runtime/src/{security,gateway,tools,config}/**/*.rs : Do not silently weaken security policy or access constraints; keep default behavior secure-by-default with deny-by-default where applicable
Applied to files:
clients/web/apps/docs/src/content/docs/clients/agent-runtime/tools/index.mdx
📚 Learning: 2026-02-17T12:31:17.076Z
Learnt from: CR
Repo: dallay/corvus PR: 0
File: clients/agent-runtime/AGENTS.md:0-0
Timestamp: 2026-02-17T12:31:17.076Z
Learning: Applies to clients/agent-runtime/src/tools/**/*.rs : Implement `Tool` trait in `src/tools/` with strict parameter schema, validate and sanitize all inputs, and return structured `ToolResult` without panics in runtime path
Applied to files:
clients/web/apps/docs/src/content/docs/clients/agent-runtime/tools/index.mdx
📚 Learning: 2026-02-17T12:31:17.076Z
Learnt from: CR
Repo: dallay/corvus PR: 0
File: clients/agent-runtime/AGENTS.md:0-0
Timestamp: 2026-02-17T12:31:17.076Z
Learning: Applies to clients/agent-runtime/src/{security,gateway,tools}/**/*.rs : Treat `src/security/`, `src/gateway/`, `src/tools/` as high-risk surfaces and never broaden filesystem/network execution scope without explicit policy checks
Applied to files:
clients/web/apps/docs/src/content/docs/clients/agent-runtime/tools/index.mdx
📚 Learning: 2026-02-17T12:31:17.076Z
Learnt from: CR
Repo: dallay/corvus PR: 0
File: clients/agent-runtime/AGENTS.md:0-0
Timestamp: 2026-02-17T12:31:17.076Z
Learning: Applies to clients/agent-runtime/src/main.rs : Preserve CLI contract unless change is intentional and documented; prefer explicit errors over silent fallback for unsupported critical paths
Applied to files:
clients/web/apps/docs/src/content/docs/clients/agent-runtime/tools/core.mdclients/agent-runtime/README.md
🔇 Additional comments (11)
clients/web/apps/docs/src/content/docs/es/clients/agent-runtime/tools/core.md (1)
20-20: Valid relative link fix for Runtime section.Line 20 correctly points to a sibling
architecture.mdanchor and matches the corresponding English docs structure.
As per coding guidelines "**/*.{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."clients/web/apps/docs/src/content/docs/es/clients/agent-runtime/tools/index.mdx (1)
15-15: Security model link update looks correct.Line 15 now targets the architecture security-model section with a valid relative path and preserves ES/EN parity for this reference.
As per coding guidelines "**/*.{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."clients/agent-runtime/README.md (2)
102-102: Build-from-source migration link path is corrected and consistent.Line 102 uses a valid repo-relative location for the migration guide in the new docs location.
As per coding guidelines "**/*.{md,mdx}: Verify technical accuracy and that docs stay aligned with code changes."
389-389: Docs section now points to the same corrected migration target.Line 389 matches the updated migration-guide path, eliminating drift between README sections.
As per coding guidelines "**/*.{md,mdx}: Verify technical accuracy and that docs stay aligned with code changes."modules/cerebro/README.md (1)
21-21: Integration migration link fix is good.Line 21 correctly reflects the new migration-guide location and aligns with the related README updates.
As per coding guidelines "**/*.{md,mdx}: Verify technical accuracy and that docs stay aligned with code changes."clients/web/apps/docs/src/content/docs/clients/agent-runtime/tools/core.md (1)
18-18: Runtime link target correction is accurate.Line 18 now uses the correct relative jump to the architecture runtime anchor and stays in sync with the ES docs update.
As per coding guidelines "**/*.{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."clients/web/apps/docs/src/content/docs/clients/agent-runtime/tools/index.mdx (1)
15-15: Deny-by-default security link now points to the right section.Line 15 correctly references the architecture security-model anchor with a valid relative path and preserves EN/ES docs parity.
As per coding guidelines "**/*.{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."clients/web/apps/marketing/src/styles/global.css (1)
1-2: Syntax changes look correct.The updated
@importsyntax is valid CSS. Build tool compatibility verification requested inclients/web/apps/dashboard/src/style.cssapplies here as well.clients/web/apps/chat/src/style.css (1)
1-2: Syntax changes look correct.The updated
@importsyntax is valid CSS. Build tool compatibility verification requested inclients/web/apps/dashboard/src/style.cssapplies here as well.clients/web/apps/docs/src/styles/custom.css (1)
7-8: Syntax changes look correct.The updated
@importsyntax is valid CSS. Build tool compatibility verification requested inclients/web/apps/dashboard/src/style.cssapplies here as well.clients/web/packages/shared/tokens.css (1)
1-1: tokens.css is a documented compatibility alias during the rename to theme.css—no consolidation needed.The file is explicitly an alias (
./tokens.css→./theme.cssper package.json), as noted in the README. App-shell.css already imports theme.css, and no consumers currently import tokens.css directly, so there is no actual duplicate import risk. The temporary dual import in tokens.css is intentional during the transition period.
| @import "@corvus/shared/app-shell.css"; | ||
| @import "tailwindcss"; |
There was a problem hiding this comment.
🧩 Analysis chain
🏁 Script executed:
#!/bin/bash
# Description: Verify the import syntax is consistent and check build configuration
# Check if other CSS files use the old url() syntax
echo "=== Checking for remaining url() syntax in `@import` statements ==="
rg -n '@import\s+url\(' --type=css clients/web/
# Look for build tool configuration that might affect CSS import handling
echo -e "\n=== Checking build configuration files ==="
fd -e config.js -e config.ts -e json --exec echo "File: {}" \; --exec rg -l 'css|style' {} \;
# Check package.json for CSS/style related dependencies
echo -e "\n=== Checking CSS-related build dependencies ==="
fd package.json clients/web/ --exec jq -r '.devDependencies | keys[] | select(test("css|style|vite|webpack|postcss"))' {} \;Repository: dallay/corvus
Length of output: 5792
Ensure consistent @import syntax across CSS files that import each other.
Both @import "..." and @import url("...") syntaxes are valid and supported by Vite and PostCSS. However, the changed file in style.css now uses the bare string syntax while clients/web/packages/shared/app-shell.css (which it imports) still uses url() syntax. For maintainability, align the syntax in app-shell.css to match the new pattern.
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@clients/web/apps/dashboard/src/style.css` around lines 1 - 2, The import
syntax is inconsistent: style.css uses bare-string imports (`@import`
"@corvus/shared/app-shell.css";) while the imported app-shell.css still uses
`@import` url("..."); — update the import statements in app-shell.css to use the
same bare-string syntax (e.g., `@import` "..." ) so both files use the consistent
form; ensure any occurrences of url("...") at the top of app-shell.css that
reference internal CSS modules are converted to the bare-string `@import`
"module-path" style to match the changed pattern.
Deploying corvus with
|
| Latest commit: |
08b964a
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://f4b42c5e.corvus-42x.pages.dev |
| Branch Preview URL: | https://feature-dallay-211-docs-brok.corvus-42x.pages.dev |
|
1 participant
Restore valid local docs references and CSS imports so the offline link checker stops flagging false broken paths in the Corvus docs surfaces.