docs(web): align marketing and docs with Corvus runtime identity#175
docs(web): align marketing and docs with Corvus runtime identity#175yacosta738 merged 5 commits intomainfrom
Conversation
|
Warning Rate limit exceeded
⌛ 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 (5)
📝 WalkthroughWalkthroughThis PR refactors documentation and marketing assets to emphasize a Rust-native runtime architecture. Changes include updating C4 architecture diagrams across English and Spanish guides (system-focused to runtime-focused), adding new cargo dependency and runtime core diagrams, substantially rewriting the marketing homepage with new hero/features/FAQ sections, and comprehensively updating global CSS styling with design tokens and layout improvements. Changes
Estimated code review effort🎯 4 (Complex) | ⏱️ ~60 minutes Possibly related PRs
Suggested reviewers
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 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 |
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 8
🤖 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/docs/src/content/docs/en/index.mdx`:
- Line 3: Update the frontmatter "description" metadata so it clearly modifies
the Rust-native runtime rather than the documentation; locate the description
key in the MDX frontmatter (description:) and rephrase to something like
"Secure, pluggable Rust-native runtime for autonomous AI agents, with companion
documentation" or similar wording that moves "secure, pluggable" to describe the
runtime rather than the docs, then verify the change in docs preview/SEO
snippets and that it remains technically accurate.
In `@clients/web/apps/docs/src/content/docs/es/guides/architecture.md`:
- Around line 121-124: Update the architecture table entries and linked diagram
filenames/labels to reflect the new Rust-native runtime: replace "Agent Core
KMP" and its link `component/agent-core-kmp.mmd` with a Rust-centric label such
as "Runtime Core" and a matching diagram file (e.g.,
`component/runtime-core.mmd`), change "Contenedores
Principales"/`container/system-containers.mmd` to "Contenedores del
Runtime"/`container/runtime-containers.mmd`, and adjust any Kotlin/KMP-specific
labels (like `system-containers`, `module-dependencies.mmd`) to Rust/Cargo
equivalents (e.g., `runtime-containers.mmd`, `cargo-dependencies.mmd`); apply
the same label and link updates to the corresponding English page to preserve
EN/ES parity and ensure the table entries C1, C2, C3 and the dependency row
point to the new Rust-native artifacts.
In `@clients/web/apps/docs/src/content/docs/es/index.mdx`:
- Around line 49-50: Update the LinkCard props so the card copy matches the
linked Spanish page: change the title prop on LinkCard to "Referencia de la CLI"
and localize the description prop to Spanish (e.g., replace "onboarding" and
"agents" with "incorporación" or "configuración" and "agentes") while keeping
the href="guides/cli-reference/"; modify the title and description values in the
LinkCard element to exactly mirror the target page terminology for ES parity.
- Around line 3-6: The Spanish copy lost accent marks in the frontmatter and
hero text; update the values for description and hero.tagline to restore accents
(e.g., "Documentación", "autónomos", "rápido") and also fix the other affected
strings referenced by lines 24-25 and 33-34 so all Spanish words include correct
diacritics; locate and edit the frontmatter keys (description, template,
hero.tagline) and any corresponding strings in the same file
(clients/web/apps/docs/src/content/docs/es/index.mdx) to use the normalized
accented forms.
In `@clients/web/apps/marketing/src/pages/index.astro`:
- Around line 60-61: The CTA uses the same ctaLabel ("Read the architecture")
but two different ctaHref values; update the ctaHref so both occurrences (the
objects containing ctaLabel and ctaHref) point to the same target URL (use the
architecture guide URL for both) so identical copy always navigates to the same
place—locate the objects with keys ctaLabel and ctaHref in index.astro and
replace the mismatched repository landing URL with the architecture guide URL.
- Line 124: The CTA anchor <a class="btn btn-primary btn-nav" href="#install"
data-track="cta_click" data-track-location="nav" data-track-label="start-now">
currently scrolls to another button instead of the installation section; fix by
either restoring the missing target element with id="install" and placing the
installation content there, or update the href on this anchor (and the other
occurrences with the same attributes/data-track values) to point to the correct
installation target (the actual section id or route used for install). Ensure
all anchors with href="#install" (and matching data-track attributes) are
consistent with the real install destination so they navigate to the intended
installation content.
- Around line 276-295: The pushMetric function currently returns early on any
localStorage error, preventing the window.dispatchEvent("corvus:analytics") from
firing; change the try/catch so persistence failures do not short-circuit the
event: in pushMetric, when reading/parsing metricsKey or writing to
localStorage, catch errors but do not return—fallback to an empty previous array
(or truncate safely) and proceed to setItem only if possible, then always call
window.dispatchEvent(new CustomEvent("corvus:analytics", { detail: payload }));
reference pushMetric and metricsKey and ensure malformed JSON is handled by
resetting previous to [] rather than returning (and add a regression test that
pre-populates metricsKey with invalid JSON to assert the event still fires).
In `@clients/web/apps/marketing/src/styles/global.css`:
- Around line 29-31: The global CSS enables smooth scrolling via the html {
scroll-behavior: smooth; } rule but the existing reduced-motion block (`@media`
(prefers-reduced-motion: reduce) ... ) does not override it; update the
reduced-motion media query to explicitly disable smooth scrolling by setting
scroll-behavior: auto (or scroll-behavior: unset/none depending on browser
support) on html (and/or :root) so anchors like `#faq` and `#install` do not animate
for users who prefer reduced motion.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: ASSERTIVE
Plan: Pro
Run ID: c347f278-9e9e-499c-b2e5-cff8500984a2
📒 Files selected for processing (13)
clients/web/apps/docs/src/content/docs/en/guides/architecture.mdclients/web/apps/docs/src/content/docs/en/guides/architecture/overview.mdclients/web/apps/docs/src/content/docs/en/guides/getting-started.mdclients/web/apps/docs/src/content/docs/en/guides/release.mdclients/web/apps/docs/src/content/docs/en/index.mdxclients/web/apps/docs/src/content/docs/es/guides/architecture.mdclients/web/apps/docs/src/content/docs/es/guides/architecture/overview.mdclients/web/apps/docs/src/content/docs/es/guides/getting-started.mdclients/web/apps/docs/src/content/docs/es/guides/release.mdclients/web/apps/docs/src/content/docs/es/index.mdxclients/web/apps/marketing/src/layouts/MarketingLayout.astroclients/web/apps/marketing/src/pages/index.astroclients/web/apps/marketing/src/styles/global.css
📜 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). (1)
- 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:
clients/web/apps/docs/src/content/docs/en/guides/release.mdclients/web/apps/docs/src/content/docs/es/guides/release.mdclients/web/apps/docs/src/content/docs/es/guides/getting-started.mdclients/web/apps/docs/src/content/docs/es/guides/architecture/overview.mdclients/web/apps/docs/src/content/docs/es/index.mdxclients/web/apps/docs/src/content/docs/es/guides/architecture.mdclients/web/apps/docs/src/content/docs/en/guides/architecture/overview.mdclients/web/apps/docs/src/content/docs/en/guides/getting-started.mdclients/web/apps/docs/src/content/docs/en/guides/architecture.mdclients/web/apps/docs/src/content/docs/en/index.mdx
**/*
⚙️ 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/apps/docs/src/content/docs/en/guides/release.mdclients/web/apps/docs/src/content/docs/es/guides/release.mdclients/web/apps/marketing/src/layouts/MarketingLayout.astroclients/web/apps/docs/src/content/docs/es/guides/getting-started.mdclients/web/apps/docs/src/content/docs/es/guides/architecture/overview.mdclients/web/apps/docs/src/content/docs/es/index.mdxclients/web/apps/docs/src/content/docs/es/guides/architecture.mdclients/web/apps/docs/src/content/docs/en/guides/architecture/overview.mdclients/web/apps/marketing/src/styles/global.cssclients/web/apps/marketing/src/pages/index.astroclients/web/apps/docs/src/content/docs/en/guides/getting-started.mdclients/web/apps/docs/src/content/docs/en/guides/architecture.mdclients/web/apps/docs/src/content/docs/en/index.mdx
🧠 Learnings (4)
📚 Learning: 2026-02-17T07:28:38.934Z
Learnt from: CR
Repo: dallay/corvus PR: 0
File: AGENTS.md:0-0
Timestamp: 2026-02-17T07:28:38.934Z
Learning: Applies to .agents/AGENTS.md : Document agent configurations and capabilities in AGENTS.md
Applied to files:
clients/web/apps/docs/src/content/docs/es/index.mdxclients/web/apps/docs/src/content/docs/en/guides/architecture/overview.mdclients/web/apps/docs/src/content/docs/en/index.mdx
📚 Learning: 2026-02-17T07:28:38.934Z
Learnt from: CR
Repo: dallay/corvus PR: 0
File: AGENTS.md:0-0
Timestamp: 2026-02-17T07:28:38.934Z
Learning: Applies to .agents/AGENTS.md : Maintain comprehensive agent metadata including name, description, purpose, and capabilities
Applied to files:
clients/web/apps/docs/src/content/docs/es/index.mdxclients/web/apps/docs/src/content/docs/en/index.mdx
📚 Learning: 2026-02-17T07:28:38.934Z
Learnt from: CR
Repo: dallay/corvus PR: 0
File: AGENTS.md:0-0
Timestamp: 2026-02-17T07:28:38.934Z
Learning: Applies to .agents/AGENTS.md : Include version information and compatibility details for agents
Applied to files:
clients/web/apps/docs/src/content/docs/es/index.mdxclients/web/apps/docs/src/content/docs/en/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,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/en/index.mdx
🪛 LanguageTool
clients/web/apps/docs/src/content/docs/es/guides/release.md
[grammar] ~12-~12: Agrega una palabra o signo.
Context: ...der publicar, asegúrate de tener: 1. Clave GPG configurada: Sigue la [Guía de Co...
(QB_NEW_ES_OTHER_ERROR_IDS_MISSING_OTHER)
[grammar] ~12-~12: Elimina la palabra o signo.
Context: ...ración GPG](./gpg-setup.md) para crear y configurar tu clave de firma 2. **Acceso...
(QB_NEW_ES_OTHER_ERROR_IDS_UNNECESSARY_SPACE)
[grammar] ~262-~262: Corrige la mayúscula.
Context: ...a features) ## Ver También - Guía de Configuración GPG - [GitHub Workflow...
(QB_NEW_ES_OTHER_ERROR_IDS_REPLACEMENT_ORTHOGRAPHY_UPPERCASE)
[grammar] ~262-~262: Cambia la palabra o signo.
Context: ...er También - Guía de Configuración GPG - [GitHub Workflows](https://github.com/dal...
(QB_NEW_ES_OTHER_ERROR_IDS_REPLACEMENT_SPACE)
clients/web/apps/docs/src/content/docs/es/guides/architecture/overview.md
[grammar] ~15-~15: Cambia la palabra o signo.
Context: ...es y sistemas externos. - Archivo: context/system-context.mmd - Formato: Mermaid - Descripción: Vi...
(QB_NEW_ES_OTHER_ERROR_IDS_REPLACEMENT_SPACE)
[grammar] ~25-~25: Elimina la palabra o signo.
Context: ...ntainer/system-containers.mmd) (Mermaid) - container/system-containers.puml (PlantUML) - Descripción: Muestra lo...
(QB_NEW_ES_OTHER_ERROR_IDS_UNNECESSARY_SPACE)
[grammar] ~26-~26: Cambia la palabra o signo.
Context: ...ainer/system-containers.puml) (PlantUML) - Descripción: Muestra los contenedores ...
(QB_NEW_ES_OTHER_ERROR_IDS_REPLACEMENT_SPACE)
[grammar] ~36-~36: Cambia la palabra o signo.
Context: ...os. ### Agent Core KMP - Archivo: component/agent-core-kmp.mmd - Descripción: Componentes internos del ...
(QB_NEW_ES_OTHER_ERROR_IDS_REPLACEMENT_SPACE)
[grammar] ~37-~37: Elimina la palabra o signo.
Context: ...ore: Dominio, Casos de Uso, Interfaces e Infraestructura. ## Dependencias entre ...
(QB_NEW_ES_OTHER_ERROR_IDS_UNNECESSARY_SPACE)
[grammar] ~40-~40: Corrige la mayúscula.
Context: ...Infraestructura. ## Dependencias entre Módulos Diagrama adicional mostrando las depende...
(QB_NEW_ES_OTHER_ERROR_IDS_REPLACEMENT_ORTHOGRAPHY_UPPERCASE)
[grammar] ~44-~44: Cambia la palabra o signo.
Context: ...e Gradle entre módulos. - Archivo: module-dependencies.mmd - Descripción: Muestra cómo los clientes...
(QB_NEW_ES_OTHER_ERROR_IDS_REPLACEMENT_SPACE)
🔇 Additional comments (5)
clients/web/apps/docs/src/content/docs/es/guides/release.md (2)
1-264: EN/ES parity maintained.The Spanish documentation changes mirror the English version exactly, maintaining proper localization consistency.
12-12: The target filegpg-setup.mdexists at the new locationclients/web/apps/docs/src/content/docs/es/guides/gpg-setup.md. The link change is valid.clients/web/apps/docs/src/content/docs/en/guides/release.md (1)
12-12: Link path change is valid. The gpg-setup.md file exists at the new relative path in the guides directory.clients/web/apps/docs/src/content/docs/en/index.mdx (1)
48-49: Good addition: the new CLI card points at an existing guide.Surfacing it here improves docs discoverability without introducing a dead link.
clients/web/apps/docs/src/content/docs/es/index.mdx (1)
27-29: Verify the “seguro por diseño” defaults before stating them as guaranteed.
"forman parte de la postura por defecto"turns pairing, sandboxing, workspace isolation, allowlists, and encrypted secrets into default security guarantees. Please confirm each one is actually enabled/configured by default in the product, or soften the wording to avoid over-promising the security posture.As per coding guidelines, "Verify technical accuracy and that docs stay aligned with code changes."
clients/web/apps/docs/src/content/docs/es/guides/architecture.md
Outdated
Show resolved
Hide resolved
Deploying corvus with
|
| Latest commit: |
dfbb2f2
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://adb9d631.corvus-42x.pages.dev |
| Branch Preview URL: | https://frontend.corvus-42x.pages.dev |
✅ Contributor ReportUser: @yacosta738
Contributor Report evaluates based on public GitHub activity. Analysis period: 2025-03-09 to 2026-03-09 |
There was a problem hiding this comment.
Actionable comments posted: 7
♻️ Duplicate comments (1)
clients/web/apps/docs/src/content/docs/es/guides/architecture.md (1)
121-121:⚠️ Potential issue | 🟠 MajorRetarget the C1 entry to the runtime context.
Line 121 still points to the old system-level artifact (
Sistema Completo/system-context.mmd) while the rest of the table has moved to runtime-specific diagrams. That keeps the Spanish architecture index out of sync with the new runtime positioning and likely with the English page as well.Suggested fix
-| C1 - Contexto | [Sistema Completo](./architecture/diagrams/context/system-context.mmd) | Vista de alto nivel del runtime y actores externos | `context/system-context.mmd` | +| C1 - Contexto | [Contexto del Runtime](./architecture/diagrams/context/runtime-context.mmd) | Vista de alto nivel del runtime y actores externos | `context/runtime-context.mmd` |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.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@clients/web/apps/docs/src/content/docs/es/guides/architecture.md` at line 121, The Spanish table row labeled "C1 - Contexto" still links to the old system-level artifact `context/system-context.mmd`; update that row to point to the runtime-specific diagram used by the rest of the table (replace the link and text to reference the runtime context artifact, e.g., `context/runtime-context.mmd` and a matching label like "Contexto del runtime" or the same text used on the English page), and verify the Spanish entry now matches the English page for parity.
🤖 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/docs/public/guides/architecture/diagrams/cargo-dependencies.mmd`:
- Line 21: The diagram currently shows "CORVUS --> ROBOT" implying a Cargo
dependency; update the Mermaid file so it accurately represents that corvus and
corvus-robot-kit are workspace members, not a dependency: remove the arrow
between CORVUS and ROBOT (or replace it with a sibling relationship line or a
note) and add a clarifying label like "workspace member (see
clients/agent-runtime/Cargo.toml)" referencing corvus-robot-kit to make it clear
they are co-members of the workspace rather than one depending on the other;
locate the diagram nodes named CORVUS and ROBOT in cargo-dependencies.mmd and
adjust accordingly.
In `@clients/web/apps/docs/src/content/docs/en/guides/architecture/overview.md`:
- Around line 40-45: Update the "## Module Dependencies" intro to replace the
stale "Gradle dependencies between modules" text with a Cargo-appropriate
description that matches the referenced cargo-dependencies.mmd and its
description; for example, change the sentence to mention "Cargo dependencies
between workspace modules" or similar so the heading and the referenced file
(`cargo-dependencies.mmd`) consistently describe a Cargo-based dependency
diagram.
In `@clients/web/apps/docs/src/content/docs/es/guides/architecture/overview.md`:
- Around line 40-45: La frase bajo el encabezado "## Dependencias entre Módulos"
sigue mencionando "dependencias de Gradle entre módulos" pero el diagrama es de
Cargo; actualizar ese texto para referirse a Cargo (por ejemplo "dependencias de
Cargo entre módulos" o "dependencias del workspace Cargo") y asegurarte de que
la descripción coincida con el archivo referenciado `cargo-dependencies.mmd`,
manteniendo el resto del bloque sin cambios para preservar la ruta y la
descripción del diagrama.
In `@clients/web/apps/marketing/src/pages/index.astro`:
- Around line 49-50: Replace the hardcoded docs links by introducing an
env-backed docs base URL and reusing it for installUrl and architectureUrl: read
the docs base from the same source used in
clients/web/apps/docs/astro.config.mjs (e.g. the env var or exported value) into
a local docsBaseUrl constant, then derive installUrl =
`${docsBaseUrl}/en/guides/getting-started/` and architectureUrl =
`${docsBaseUrl}/en/guides/architecture/`; update any other doc links on the page
to use docsBaseUrl so preview/staging environments point to the correct docs
instance.
- Line 261: The analytics label for the docs CTA is incorrect: update the
anchor's data-track-label value (currently "book-quick-call") to a label that
reflects the action (e.g., "explore-docs" or "open-docs") so that pushMetric
(the analytics handler reading data-track, data-track-location and
data-track-label) records the correct event; locate the <a> element with class
"btn btn-ghost" and data-track="cta_click" and change its data-track-label to
the new, accurate string.
- Around line 73-75: The CTA currently hardcodes ctaHref to
"https://docs.profiletailors.com"; change it to use the existing installUrl
variable so the "See the quick start" button navigates directly to the install
guide (update the object where ctaLabel and ctaHref are set to assign ctaHref:
installUrl). Ensure you reference the same installUrl defined above and keep the
ctaLabel as "See the quick start" so behavior matches the FAQ path.
In `@clients/web/apps/marketing/src/styles/global.css`:
- Around line 176-191: The nav hover rule is too broad and overrides button
styles (e.g., the install CTA uses <a class="btn btn-primary btn-nav">), so
update the selector in global.css to exclude link elements that are styled as
buttons (reference selectors: nav a, .btn-primary, .btn-nav). Replace the
existing nav a:hover rule with a scoped selector such as nav
a:not(.btn):not(.btn-primary):not(.btn-nav):hover (or use :not(.btn-nav) if you
only want to exclude that class) so button links keep their .btn-primary hover
styles while normal nav links keep the muted hover.
---
Duplicate comments:
In `@clients/web/apps/docs/src/content/docs/es/guides/architecture.md`:
- Line 121: The Spanish table row labeled "C1 - Contexto" still links to the old
system-level artifact `context/system-context.mmd`; update that row to point to
the runtime-specific diagram used by the rest of the table (replace the link and
text to reference the runtime context artifact, e.g.,
`context/runtime-context.mmd` and a matching label like "Contexto del runtime"
or the same text used on the English page), and verify the Spanish entry now
matches the English page for parity.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: ASSERTIVE
Plan: Pro
Run ID: 392c40a0-6eaf-4691-b806-38aa5faceef7
📒 Files selected for processing (13)
clients/agent-runtime/src/test_support.rsclients/web/apps/docs/public/guides/architecture/diagrams/cargo-dependencies.mmdclients/web/apps/docs/public/guides/architecture/diagrams/component/runtime-core.mmdclients/web/apps/docs/public/guides/architecture/diagrams/container/runtime-containers.mmdclients/web/apps/docs/public/guides/architecture/diagrams/container/runtime-containers.pumlclients/web/apps/docs/src/content/docs/en/guides/architecture.mdclients/web/apps/docs/src/content/docs/en/guides/architecture/overview.mdclients/web/apps/docs/src/content/docs/en/index.mdxclients/web/apps/docs/src/content/docs/es/guides/architecture.mdclients/web/apps/docs/src/content/docs/es/guides/architecture/overview.mdclients/web/apps/docs/src/content/docs/es/index.mdxclients/web/apps/marketing/src/pages/index.astroclients/web/apps/marketing/src/styles/global.css
💤 Files with no reviewable changes (1)
- clients/agent-runtime/src/test_support.rs
📜 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: pr-checks
- GitHub Check: sonar
- GitHub Check: pr-checks
- 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:
clients/web/apps/docs/src/content/docs/es/index.mdxclients/web/apps/docs/src/content/docs/en/index.mdxclients/web/apps/docs/src/content/docs/en/guides/architecture/overview.mdclients/web/apps/docs/src/content/docs/es/guides/architecture.mdclients/web/apps/docs/src/content/docs/en/guides/architecture.mdclients/web/apps/docs/src/content/docs/es/guides/architecture/overview.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:
clients/web/apps/docs/src/content/docs/es/index.mdxclients/web/apps/docs/src/content/docs/en/index.mdxclients/web/apps/docs/public/guides/architecture/diagrams/component/runtime-core.mmdclients/web/apps/docs/src/content/docs/en/guides/architecture/overview.mdclients/web/apps/docs/public/guides/architecture/diagrams/container/runtime-containers.pumlclients/web/apps/docs/src/content/docs/es/guides/architecture.mdclients/web/apps/docs/src/content/docs/en/guides/architecture.mdclients/web/apps/docs/public/guides/architecture/diagrams/cargo-dependencies.mmdclients/web/apps/docs/public/guides/architecture/diagrams/container/runtime-containers.mmdclients/web/apps/docs/src/content/docs/es/guides/architecture/overview.mdclients/web/apps/marketing/src/styles/global.cssclients/web/apps/marketing/src/pages/index.astro
🧠 Learnings (10)
📓 Common learnings
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/**/*.rs : Run `cargo fmt --all -- --check`, `cargo clippy --all-targets -- -D warnings`, and `cargo test` for code validation, or document which checks were skipped and why
📚 Learning: 2026-02-17T07:28:38.934Z
Learnt from: CR
Repo: dallay/corvus PR: 0
File: AGENTS.md:0-0
Timestamp: 2026-02-17T07:28:38.934Z
Learning: Applies to .agents/AGENTS.md : Maintain comprehensive agent metadata including name, description, purpose, and capabilities
Applied to files:
clients/web/apps/docs/src/content/docs/en/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,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/en/index.mdxclients/web/apps/docs/src/content/docs/es/guides/architecture.md
📚 Learning: 2026-02-17T07:28:38.934Z
Learnt from: CR
Repo: dallay/corvus PR: 0
File: AGENTS.md:0-0
Timestamp: 2026-02-17T07:28:38.934Z
Learning: Applies to .agents/AGENTS.md : Document agent configurations and capabilities in AGENTS.md
Applied to files:
clients/web/apps/docs/src/content/docs/en/index.mdx
📚 Learning: 2026-02-17T07:28:38.934Z
Learnt from: CR
Repo: dallay/corvus PR: 0
File: AGENTS.md:0-0
Timestamp: 2026-02-17T07:28:38.934Z
Learning: Applies to .agents/AGENTS.md : Include version information and compatibility details for agents
Applied to files:
clients/web/apps/docs/src/content/docs/en/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/**/Cargo.toml : Do not add heavy dependencies for minor convenience; justify new crate additions
Applied to files:
clients/web/apps/docs/src/content/docs/en/guides/architecture/overview.mdclients/web/apps/docs/src/content/docs/en/guides/architecture.mdclients/web/apps/docs/public/guides/architecture/diagrams/cargo-dependencies.mmd
📚 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/**/*.rs : Run `cargo fmt --all -- --check`, `cargo clippy --all-targets -- -D warnings`, and `cargo test` for code validation, or document which checks were skipped and why
Applied to files:
clients/web/apps/docs/src/content/docs/es/guides/architecture.md
📚 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/es/guides/architecture.md
📚 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/es/guides/architecture.md
📚 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/**/Cargo.toml : Preserve release-size profile assumptions in `Cargo.toml` and avoid adding heavy dependencies unless clearly justified
Applied to files:
clients/web/apps/docs/public/guides/architecture/diagrams/cargo-dependencies.mmd
🪛 LanguageTool
clients/web/apps/docs/src/content/docs/es/guides/architecture/overview.md
[grammar] ~25-~25: Elimina la palabra o signo.
Context: ...tainer/runtime-containers.mmd) (Mermaid) - container/runtime-containers.puml (PlantUML) - Descripción: Muestra lo...
(QB_NEW_ES_OTHER_ERROR_IDS_UNNECESSARY_SPACE)
[grammar] ~26-~26: Cambia la palabra o signo.
Context: ...iner/runtime-containers.puml) (PlantUML) - Descripción: Muestra los contenedores ...
(QB_NEW_ES_OTHER_ERROR_IDS_REPLACEMENT_SPACE)
[grammar] ~27-~27: Cambia la palabra o signo.
Context: ...runtime: CLI, gateway, servicios daemon, ejecución de tools, backends de memoria ...
(QB_NEW_ES_OTHER_ERROR_IDS_REPLACEMENT_OTHER)
[grammar] ~34-~34: Corrige la mayúscula.
Context: ...s componentes internos. ### Núcleo del Runtime - Archivo: [component/runtime-core.mmd...
(QB_NEW_ES_OTHER_ERROR_IDS_REPLACEMENT_ORTHOGRAPHY_UPPERCASE)
[grammar] ~36-~36: Cambia la palabra o signo.
Context: ... ### Núcleo del Runtime - Archivo: component/runtime-core.mmd - Descripción: Componentes internos del ...
(QB_NEW_ES_OTHER_ERROR_IDS_REPLACEMENT_SPACE)
[grammar] ~37-~37: Cambia la palabra o signo.
Context: ... del runtime: configuración, agent loop, providers, memoria, tools, canales, segu...
(QB_NEW_ES_OTHER_ERROR_IDS_REPLACEMENT_OTHER)
[grammar] ~40-~40: Corrige la mayúscula.
Context: ... observabilidad. ## Dependencias entre Módulos Diagrama adicional mostrando las depende...
(QB_NEW_ES_OTHER_ERROR_IDS_REPLACEMENT_ORTHOGRAPHY_UPPERCASE)
[grammar] ~44-~44: Cambia la palabra o signo.
Context: ...e Gradle entre módulos. - Archivo: cargo-dependencies.mmd - Descripción: Muestra la estructura del...
(QB_NEW_ES_OTHER_ERROR_IDS_REPLACEMENT_SPACE)
🔇 Additional comments (11)
clients/web/apps/docs/src/content/docs/en/index.mdx (2)
3-3: Description wording looks correct now.The metadata description properly attributes "secure, pluggable" to the runtime rather than the documentation.
48-49: CLI Reference guide exists.The path
guides/cli-reference/is valid — both English and Spanish versions are present in the docs structure.clients/web/apps/docs/public/guides/architecture/diagrams/component/runtime-core.mmd (1)
1-31: LGTM!The component diagram accurately represents the runtime core architecture with appropriate relationships between CLI, agent loop, and subsystems. Security policy enforcement at tool and channel boundaries is correctly depicted.
clients/web/apps/docs/public/guides/architecture/diagrams/container/runtime-containers.puml (1)
1-39: LGTM!Well-structured C4 container diagram with appropriate actors, external systems, and runtime containers. Good practice pinning the C4-PlantUML include to a specific version (v2.13.0).
clients/web/apps/docs/public/guides/architecture/diagrams/container/runtime-containers.mmd (1)
1-38: LGTM!The Mermaid version is consistent with the PlantUML diagram and uses valid flowchart syntax with appropriate styling.
clients/web/apps/docs/src/content/docs/en/guides/architecture.md (2)
34-58: Architecture narrative still references Kotlin/Spring Boot and Neo4j.The PR objectives state: "removed prior Kotlin/Spring Boot and Neo4j-centric messaging." However, lines 36-58 still describe:
- "Kotlin + Spring Boot + Coroutines/WebFlux" (line 38)
- "Neo4j (planned)" (line 44)
- "Rust (planned)" for sidecars (line 50)
This conflicts with the runtime-focused identity shift. Consider updating this section to align with the new Rust-native runtime positioning.
120-123: Diagram table updates look correct.The C4 diagram references are properly updated to point to the new runtime-focused diagrams.
clients/web/apps/docs/src/content/docs/es/index.mdx (2)
49-50: Nice fix on the Spanish CLI card.The title now matches the linked page (
Referencia de la CLI), and the description is localized enough to resolve the earlier EN/ES mismatch.
27-34: Security and runtime guarantees in the splash page are backed by existing documentation.The claims about default sandboxing, pairing, allowlists, encrypted secrets, providers, and runtime portability are substantiated by:
clients/web/apps/docs/src/content/docs/es/clients/agent-runtime/architecture.md: Detailed security architecture (Landlock, Bubblewrap, Firejail, pairing, policy, audit, secrets)clients/web/apps/docs/src/content/docs/es/intro/introduction.mdx: References "secure, sandboxed environment" and security subsystems as default featuresThe English splash page (
clients/web/apps/docs/en/index.mdx) uses identical language, confirming EN/ES parity. No changes needed.clients/web/apps/marketing/src/styles/global.css (1)
1036-1039: Nice reduced-motion override.Resetting
scroll-behaviorhere closes the smooth-scroll regression for users who opt out of motion.As per coding guidelines, "Look for behavioral regressions, missing tests, and contract breaks across modules."
clients/web/apps/marketing/src/pages/index.astro (1)
279-308: Nice hardening ofpushMetric.The split read/write
tryblocks keepcorvus:analyticsemission alive even when localStorage is unavailable or corrupted.As per coding guidelines, "Look for behavioral regressions, missing tests, and contract breaks across modules."
clients/web/apps/docs/public/guides/architecture/diagrams/cargo-dependencies.mmd
Outdated
Show resolved
Hide resolved
clients/web/apps/docs/src/content/docs/en/guides/architecture/overview.md
Show resolved
Hide resolved
clients/web/apps/docs/src/content/docs/es/guides/architecture/overview.md
Show resolved
Hide resolved
|
1 participant
This pull request updates the documentation and marketing site to improve clarity, consistency, and accuracy, especially regarding the architecture diagrams and the platform's positioning. The main changes include standardizing relative links for better navigation, updating product descriptions and feature highlights in both English and Spanish, and refining the site's typography.
Documentation improvements:
/guides/...) to relative (./...) paths in both English and Spanish guides, ensuring navigation works correctly regardless of the current location in the docs. [1] [2] [3] [4] [5] [6] [7] [8] [9] [10] [11] [12] [13] [14]Product positioning and feature highlights:
index.mdx) to emphasize the secure, pluggable, Rust-native, and portable nature of the runtime. [1] [2]Site styling:
Instrument Serifand adjust font weights, improving the site's typography.