Skip to content

chore: update slides and place on hero page#38690

Merged
mnkiefer merged 5 commits into
mainfrom
add-slides-to-hero
Jun 11, 2026
Merged

chore: update slides and place on hero page#38690
mnkiefer merged 5 commits into
mainfrom
add-slides-to-hero

Conversation

@mnkiefer

@mnkiefer mnkiefer commented Jun 11, 2026

Copy link
Copy Markdown
Collaborator
  • Updates how presentation slides are handled and displayed in the docs:
    • Removes the old static slides page and introduces a new interactive PDF slide deck preview on the main docs landing page
    • Simplifies slide build steps and updates dependencies
    • Adds Node.js engine requirement (>=22.13.0) for pdfjs-dist compatibility
    • Adds Playwright test coverage for slide preview (PDF loading, rendering, keyboard accessibility)

@mnkiefer mnkiefer self-assigned this Jun 11, 2026
Copilot AI review requested due to automatic review settings June 11, 2026 18:32
@mnkiefer

Copy link
Copy Markdown
Collaborator Author

@copilot Resolve conflicts and test docs build.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the docs site’s slide handling by removing the dedicated “Presentation Slides” docs page and instead surfacing a PDF slide deck preview directly on the docs landing page, while simplifying the slide build process and adjusting redirects.

Changes:

  • Remove the introduction/slides documentation page and add redirects to the docs homepage.
  • Add a new WorkflowHero component that renders a PDF slide preview (with basic carousel behavior) on the homepage.
  • Simplify slide build steps and swap Marp-based slide generation for copying a single PDF, adding pdfjs-dist as a dependency.
Show a summary per file
File Description
docs/src/content/docs/introduction/slides.mdx Removes the old static “Presentation Slides” page content.
docs/src/content/docs/index.mdx Adds the new homepage hero slide preview component.
docs/src/content/docs/agent-factory-status.mdx Updates the label text for the daily AIC consumption report.
docs/src/components/WorkflowHero.astro New interactive PDF slide-deck preview rendered on the homepage.
docs/package.json Simplifies slide build script, removes Marp CLI, adds pdfjs-dist.
docs/package-lock.json Locks dependency graph changes from removing Marp CLI and adding pdfjs-dist.
docs/astro.config.mjs Adds redirects from the old slides URLs to the docs homepage.

Copilot's findings

Tip

Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Files not reviewed (1)
  • docs/package-lock.json: Language not supported
  • Files reviewed: 6/11 changed files
  • Comments generated: 3

Comment thread docs/src/components/WorkflowHero.astro
Comment thread docs/package.json
Comment thread docs/src/components/WorkflowHero.astro
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@mnkiefer

Copy link
Copy Markdown
Collaborator Author

@copilot Address all review comments

Co-authored-by: mnkiefer <8320933+mnkiefer@users.noreply.github.com>

Copilot AI commented Jun 11, 2026

Copy link
Copy Markdown
Contributor

@copilot Resolve conflicts and test docs build.

Resolved by merging origin/main and fixing the lockfile conflict in commit acff070. I also ran make build-docs successfully after the merge.

…aywright test coverage

Co-authored-by: mnkiefer <8320933+mnkiefer@users.noreply.github.com>
@github-actions

Copy link
Copy Markdown
Contributor

@copilot review all comments and address unresolved review feedback. Please refresh the branch and rerun checks once that is done.

Generated by 👨‍🍳 PR Sous Chef · 65.7 AIC · ⌖ 1.82 AIC · ⊞ 17.3K ·

Co-authored-by: mnkiefer <8320933+mnkiefer@users.noreply.github.com>

Copilot AI commented Jun 11, 2026

Copy link
Copy Markdown
Contributor

@copilot Address all review comments

Addressed all review comments in commits 0de68f9 and 0bcf1c8:

  1. Added Node.js engine requirement (>=22.13.0) to docs/package.json for pdfjs-dist compatibility
  2. Added comprehensive Playwright test coverage for the slide preview feature, including:
    • PDF loading and rendering verification
    • Keyboard accessibility checks (tabindex, role, aria-label attributes)
    • Keyboard navigation tests (Enter and Space keys)
    • Error handling test for failed PDF loads

All validations passed with no review comments or security alerts.

@mnkiefer mnkiefer merged commit 6ee5825 into main Jun 11, 2026
15 checks passed
@mnkiefer mnkiefer deleted the add-slides-to-hero branch June 11, 2026 19:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants