Replace LLM engineering loop with AI engineering loop on homepage - #3494
Conversation
Update the homepage sidebar TOC label and section header to use AI engineering loop wording. Co-authored-by: annabellscha <annabellscha@users.noreply.github.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
@claude review |
There was a problem hiding this comment.
LGTM — straightforward copy change with no logic or structural impact.
What was reviewed: Confirmed both occurrences (Rive section heading and the homepage TOC label in HomeAside) are updated consistently from "LLM engineering loop" to "AI engineering loop", and that the anchor id/URL (#llm-engineering-loop) is left unchanged, so no existing links break.
Extended reasoning...
Overview
This PR changes two string literals: the homepage Rive section heading (RiveSection.tsx) from "The full LLM engineering loop" to "The full AI engineering loop", and the matching "On this page" sidebar TOC label (HomeAside.tsx) from "LLM engineering loop" to "AI engineering loop". The section id/anchor (llm-engineering-loop) is intentionally left unchanged to preserve existing link stability, which is a good practice call.
Security risks
None. This is purely user-facing copy text with no logic, data handling, or routing changes.
Level of scrutiny
Minimal scrutiny is warranted — this is a size:XS, purely cosmetic text change in two non-critical marketing/homepage components, following an existing pattern (both files already had these strings hardcoded). No CODEOWNER-sensitive paths, no auth/crypto, no build config.
Other factors
Verified the two edits are consistent with each other and with the PR description (anchor kept the same, only the visible label text changed). No test coverage concerns since there's no behavior change. The bug hunting system reported no issues, which aligns with the trivial nature of the diff.
Summary
Section anchor
#llm-engineering-loopis unchanged for existing link stability.Screenshots
Sidebar TOC:
Sidebar shows AI engineering loop
Section header:
Header shows The full AI engineering loop
Test plan
To show artifacts inline, enable in settings.
Linear Issue: LF-89
Greptile Summary
The PR consistently renames the homepage’s “LLM engineering loop” display text to “AI engineering loop” while preserving the existing stable anchor.
Confidence Score: 5/5
The PR appears safe to merge.
The two display-text changes are consistent, and the existing section anchor remains unchanged for link compatibility.
Reviews (1): Last reviewed commit: "Replace LLM engineering loop with AI eng..." | Re-trigger Greptile