Skip to content

Conversation

@gocanto
Copy link
Collaborator

@gocanto gocanto commented Oct 14, 2025

Summary by CodeRabbit

  • New Features
    • Added page-specific titles and meta descriptions to About, Projects, and Resume pages, improving SEO and enhancing link previews when shared.
    • These pages now display more accurate titles in browser tabs and search results.
    • The homepage remains unchanged.
    • No visual or layout changes; improvements are limited to metadata for better discoverability and preview snippets.

@coderabbitai
Copy link

coderabbitai bot commented Oct 14, 2025

Caution

Review failed

The pull request is closed.

Walkthrough

Adds page-specific Title and Description to TemplateData for about, projects, and resume pages by passing an additional option to buildForPage using existing variadic options. Index page unchanged. No exported declarations modified.

Changes

Cohort / File(s) Summary
Page-specific SEO metadata injection
metal/cli/seo/generator.go
For about, projects, and resume pages, pass an extra option to buildForPage to set TemplateData.Title = TitleFor(web.Title) and TemplateData.Description = web.Excerpt. Index page flow unchanged; no error-handling changes.

Sequence Diagram(s)

sequenceDiagram
  autonumber
  actor User as CLI User
  participant CLI as CLI
  participant SEO as SEO Generator
  participant BFP as buildForPage
  participant TD as TemplateData

  User->>CLI: Run SEO generation
  CLI->>SEO: Generate pages
  Note over SEO: about, projects, resume
  SEO->>BFP: buildForPage(page, ...options)
  activate BFP
  BFP->>TD: Set Title = TitleFor(web.Title)
  BFP->>TD: Set Description = web.Excerpt
  deactivate BFP
  Note over SEO: index page\n(no extra options passed)
Loading

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Poem

A nibble of titles, a sprinkle of lore,
I hop through pages—now three get more.
About, projects, resume, neatly dressed,
With shiny excerpts and headings impressed.
Index sits calm, content at the door—
Thump-thump! SEO’s tuned like never before. 🐇✨

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch feat/enrich-seo-ii

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 3aa306e and 34bafc7.

📒 Files selected for processing (1)
  • metal/cli/seo/generator.go (3 hunks)

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@gocanto gocanto changed the title refine pages fix: Refine SEO pages title and description Oct 14, 2025
@gocanto gocanto marked this pull request as ready for review October 14, 2025 08:52
@gocanto gocanto merged commit 4ac0bb6 into main Oct 14, 2025
5 checks passed
@gocanto gocanto deleted the feat/enrich-seo-ii branch October 14, 2025 08:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants