Conversation
|
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: f6e8de8b6e
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
- Remove hardcoded author map from Info/index.tsx; the wrapped Info component already renders resolved author data from Docusaurus - Replace client-side new Date().toLocaleDateString() with metadata.formattedDate in BlogListPage to avoid timezone-based off-by-one day shifts for viewers west of UTC Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com> Signed-off-by: Angie Jones <jones.angie@gmail.com>
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 230b332a69
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
- Use imageURL (Docusaurus camelCase) alongside image_url when detecting and rendering author avatars so resolved authors show their photos - Exclude only the single promoted featured post from the grid, so additional featured posts remain visible - Bump postsPerPage from 20 to 22 to fill 3-column grid rows evenly with 1 featured post + 21 grid posts Signed-off-by: Angie Jones <jones.angie@gmail.com> Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com> Signed-off-by: Angie Jones <jones.angie@gmail.com>
230b332 to
4bd894d
Compare
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 4bd894dc93
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| {posts.map((post, index) => { | ||
| const url = useBaseUrl(post.content.metadata.permalink); | ||
| const imageUrl = post.content.frontMatter.image ? useBaseUrl(post.content.frontMatter.image) : null; |
There was a problem hiding this comment.
Move useBaseUrl calls out of the posts map
useBaseUrl is a React hook (it calls other hooks internally), but it is invoked inside posts.map(...) here, so the number/order of hook calls depends on how many posts are rendered. If this component re-renders with a different page payload (for example, navigating across paginated blog pages where item counts differ), React can throw a hooks-order error (Rendered more/fewer hooks than expected) and break the blog list view. Resolve URLs with useBaseUrlUtils().withBaseUrl(...) or by moving each card into a child component that calls hooks at the top level.
Useful? React with 👍 / 👎.
Copy automate-taste.png and goose-maintains-goose.png to /static/img/blog/ and add image: frontmatter so they render as thumbnails in the blog list grid. Signed-off-by: Angie Jones <jones.angie@gmail.com> Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com> Signed-off-by: Angie Jones <jones.angie@gmail.com>
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 35767d64c9
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| const url = useBaseUrl(post.content.metadata.permalink); | ||
| const imageUrl = post.content.frontMatter.image ? useBaseUrl(post.content.frontMatter.image) : null; | ||
| const title = post.content.metadata.title; | ||
| const formattedDate = post.content.metadata.formattedDate; |
There was a problem hiding this comment.
Render post dates from an existing metadata field
The new list renderer reads post.content.metadata.formattedDate, but blog list item metadata does not provide formattedDate (it provides date), so the date text is empty in the rendered cards. In the generated output this leaves <div class="postDate"> blank for every post, which removes publication dates from the blog index and featured section.
Useful? React with 👍 / 👎.
| --- | ||
| title: "Why Tool Descriptions Aren’t Enough" | ||
| description: "I thought better tool descriptions would solve everything. They didn’t. Here’s what finally made MCP sampling click for me." | ||
| image: /img/blog/tool-descriptions-banner.png |
There was a problem hiding this comment.
Refresh hardcoded social image URLs after banner renames
This post switched to the new /img/blog/tool-descriptions-banner.png asset, but the hardcoded <head> tags later in the same file still point to .../assets/images/blogbanner-...png. After the rename, that old asset path no longer exists in the build output, so og:image/twitter:image resolve to 404 and social previews break for this post (and similarly renamed posts in this commit).
Useful? React with 👍 / 👎.
useBaseUrl was being called inside posts.map() in BlogPostGrid, violating React's Rules of Hooks. Extracted each card into a BlogPostCard component so hooks are called at the top level. Signed-off-by: Angie Jones <jones.angie@gmail.com> Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com> Signed-off-by: Angie Jones <jones.angie@gmail.com>
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 37da898e90
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
Fixes og:image/twitter:image resolution for social previews. Signed-off-by: Angie Jones <jones.angie@gmail.com> Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com> Signed-off-by: Angie Jones <jones.angie@gmail.com>
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 9bf6bbb137
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| description: "goose v1.25.0 brings macOS sandboxing, a unified summon extension, rich MCP app UIs, agentic CLI upgrades, and SLSA build provenance." | ||
| authors: | ||
| - debbie | ||
| image: /img/blog/goose-v1-25-0.png |
There was a problem hiding this comment.
Update social meta image URL to moved banner path
This post now declares /img/blog/goose-v1-25-0.png as its banner, but the hardcoded <head> metadata later in the file still references the old hashed assets/images/banner-...png URL for og:image/twitter:image. Because banner.png was moved out of the post directory in this commit, that legacy hashed asset is no longer produced, so social shares for this article will point to a missing image unless the manual meta tags are updated (or removed in favor of front matter).
Useful? React with 👍 / 👎.
Summary
Testing
Visual change, manually tested.
Screenshots/Demos (for UX changes)
Before:

After:
