Repository navigation
Showcase enterprise readiness on the landing page - #70883
Conversation
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Visual Regression Check
Chrome could not launch in the sandbox (
|
|
✅ Test Quality Sentinel completed test quality analysis. Test Quality SentinelOutcome: The agent completed and reported no action. A no-op output alone does not verify that all required work was completed or that publication was possible. Agent messages are omitted from this comment. The category uses structured run status, not interpretation of agent prose.
|
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
There was a problem hiding this comment.
Impeccable review — critique + audit
Reviewed the new EnterpriseReady.astro landing section (443 lines) and its wiring into index.mdx.
No blocking issues found. The component follows the existing design system closely:
- Reuses LandingSection, Octicon, and the shared --aw-* design tokens (all referenced tokens are defined in tokens.css; verified no undefined custom properties).
- BASE_URL handling, aw-card/aw-link/aw-badge class usage, and the max-width: 29.99rem mobile breakpoint match the conventions used in ForOrganizations.astro, HowItWorks.astro, and WorkflowsMore.astro.
- All linked doc pages exist: self-hosted-runners, arc-dind-copilot-agent, integrity, agent-runtimes, open-telemetry, steps-jobs.
- Preview/prototype labeling (Cloud Hypervisor, NVX, Central Agentic Ops) matches the caveats already documented in reference/agent-runtimes.md.
- Reduced-motion is respected via
@media(prefers-reduced-motion: no-preference) guarding the animated spans/packet, with a reduce fallback disabling the hover transform. - Figures use aria-label plus figcaption, and status badges use text labels (not color-only) for the preview/research-prototype distinctions.
Minor, non-blocking observations
- The external link to https://github.com/githubnext/gh-cao (card and prose) has no target="_blank", so the lack of rel="noopener" is not a security concern — just flagging it was checked.
- The three runtime cards (Docker, Cloud Hypervisor, NVX) all link to the same /reference/agent-runtimes/ anchor; this mirrors existing patterns elsewhere and each card has distinct accessible link text, so it's fine as-is.
No inline comments posted — no changed lines rose to the level of a required fix.
🧵 Reviewed using Impeccable skills by Impeccable Skills Reviewer · copilot · sonnet50 · 65.6 AIC · ⌖ 13.7 AIC · ⊞ 8.2K
There was a problem hiding this comment.
🟡 Changes recommended
The added sections invalidate existing slideshow tests, and automatic animations need accessibility corrections.
3 open issues
What changed in this PR
Makes enterprise deployment and integrations easier to discover on the gh-aw landing page.
Changes:
- Illustrates runner options, isolation runtimes, integrity filtering, and Central Agentic Ops.
- Adds GitHub Actions and OpenTelemetry diagrams with responsive, theme-aware styling.
| File | Description |
|---|---|
docs/src/content/docs/index.mdx |
Adds the new sections to the landing page. |
docs/src/components/landing/EnterpriseReady.astro |
Implements deployment and integration illustrations, links, and animations. |
Review effort: Max
| <HowItWorks /> | ||
| <CostControl /> | ||
| <ForOrganizations /> | ||
| <EnterpriseReady /> |
| animation: aw-packet 3s ease-in-out infinite; | ||
| } | ||
| .aw-span { | ||
| animation: aw-trace 5s var(--aw-ease-out) var(--delay) infinite; |
| } | ||
| .aw-span { | ||
| animation: aw-trace 5s var(--aw-ease-out) var(--delay) infinite; | ||
| } |
…ns reuse Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
pelikhan
changed the title
|
@copilot run pr-finisher skill |
Visual Regression Check
Error: Chrome failed to launch in the sandbox:
|
|
🎉 This pull request is included in a new release. Release: |
Summary
Make enterprise deployment capabilities easier to discover on the landing page with a graphical Enterprise ready section matching the existing design.
Reuse the existing section component and design tokens, with responsive layouts, light/dark themes, accessible documentation links, and reduced-motion support. Cloud Hypervisor and NVX are explicitly labeled preview runtimes with host eligibility caveats. Only the enterprise section is added; the separate compatibility section and Central Agentic Ops content have been removed.
Screenshots
Enterprise ready
Mobile layout in dark mode
Validation
make agent-report-progresscd docs && npm exec -- astro build(all internal links valid)git diff --check