Open
Conversation
✅ Deploy Preview for gardener-docs ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
gardener-prow
bot
added
do-not-merge/needs-kind
Contributor
|
Note Reviews pausedIt looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the Use the following commands to manage reviews:
Use the checkboxes below for quick actions:
📝 WalkthroughWalkthroughReworks blog data loader to expose authors, tags, previews, and robust UTC date parsing; adds client-side tag filtering and BlogPostMeta component; integrates metadata normalization utilities; supplies a tag-inference script; and applies tags/authors across many blog posts and docs. Changes
Sequence Diagram(s)sequenceDiagram
autonumber
participant Reader
participant Browser
participant Theme as "VitePress Theme\n(BlogIndex / BlogPostMeta)"
participant Loader as ".vitepress/data/blog.data.ts"
participant FS as "Markdown Files\n(website/blog/*.md)"
Reader->>Browser: Open /blog
Browser->>Theme: render BlogIndex
Theme->>Loader: request posts (render: true)
Loader->>FS: read markdown files & frontmatter
FS-->>Loader: raw frontmatter + content
Loader->>Loader: normalize authors/tags, extract preview, parse/format dates
Loader-->>Theme: Post[] (title, url, authors, tags, preview, date)
Theme-->>Browser: render filtered posts UI
Browser->>Theme: user toggles tag / types query
Theme->>Theme: compute filteredPosts (selectedTags, tagQuery)
Theme-->>Browser: update displayed posts
Browser->>Theme: open post
Theme->>Browser: render BlogPostMeta (date, authors, tags)
Estimated code review effort🎯 4 (Complex) | ⏱️ ~60 minutes Possibly related PRs
Suggested labels
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
📝 Coding Plan
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. Comment Tip Migrating from UI to YAML configuration.Use the |
n-boshnakov
changed the title
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
Actionable comments posted: 17
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (2)
website/blog/2018/06/06.11-shared-storage-with-s3-backend.md (1)
20-20:⚠️ Potential issue | 🟡 MinorConsider a more appropriate transition word.
The word "However" typically signals a contrast or contradiction, but this sentence introduces an additional use case rather than contrasting with the previous content. Consider using "Additionally", "Furthermore", or "For example" instead for better clarity.
✍️ Suggested improvement
-However, it can be used to import and parse large amounts of data into a database. +Additionally, it can be used to import and parse large amounts of data into a database.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@website/blog/2018/06/06.11-shared-storage-with-s3-backend.md` at line 20, Replace the transitional adverb "However" at the start of the sentence "However, it can be used to import and parse large amounts of data into a database." with a more appropriate connector such as "Additionally," or "Furthermore," (or "For example," if you intend to illustrate a use case) so the sentence reads e.g. "Additionally, it can be used to import and parse large amounts of data into a database." This change improves cohesion without altering the sentence content.website/blog/2018/12/12.25-gardener-cookies.md (1)
17-17:⚠️ Potential issue | 🔴 CriticalFix CSS syntax error in margin property.
The inline CSS contains invalid syntax:
margin: 0, autouses a comma instead of a space. This will not render as intended.🐛 Proposed fix
-<img style="width: 50%; height: auto; margin: 0, auto" src="./images/slider/cookie-00.jpg"/> +<img style="width: 50%; height: auto; margin: 0 auto" src="./images/slider/cookie-00.jpg"/>🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@website/blog/2018/12/12.25-gardener-cookies.md` at line 17, The inline img element uses an invalid CSS value "margin: 0, auto"; update the style on the <img> (the tag with src="./images/slider/cookie-00.jpg") to use a valid margin declaration "margin: 0 auto" (and keep the rest: "width: 50%; height: auto") so the image is properly centered.
♻️ Duplicate comments (1)
website/blog/2022/02/02.17-gardener-community-meeting-february.md (1)
36-37:⚠️ Potential issue | 🟠 MajorHugo shortcode syntax not converted.
This file still contains Hugo shortcode syntax
{{< youtube ... >}}which should be converted to VitePress component format per the PR objectives.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@website/blog/2022/02/02.17-gardener-community-meeting-february.md` around lines 36 - 37, The file contains a Hugo shortcode "{{< youtube id=\"U1VvyQiE3Jg\" title=\"Gardenctl-v2\" >}}" that wasn't converted to VitePress format; replace that Hugo shortcode with the VitePress YouTube component (use the YouTube component with id and title props) so the embedded video uses the new VitePress component system and renders correctly.
🧹 Nitpick comments (4)
website/blog/2018/06/06.11-auditing-kubernetes-for-secure-setup.md (1)
9-12: Tags implementation looks good!The YAML syntax is correct and the tags are properly formatted. The
technical-deep-diveandsecuritytags align well with the post's content about security auditing and penetration testing.The
networkingtag is more tangential—while the post mentions "network air gap," the primary focus is on security auditing rather than networking concepts. Consider whether this tag adds value for users filtering by networking topics.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@website/blog/2018/06/06.11-auditing-kubernetes-for-secure-setup.md` around lines 9 - 12, The tags list includes a tangential "networking" tag; review the post's focus and either remove "networking" from the YAML tags array or replace it with a more relevant tag (e.g., "compliance" or "audit") by editing the tags block where "technical-deep-dive", "security", and "networking" are defined so the metadata accurately reflects the article's primary focus.website/documentation/contribute/documentation/blog-tags.md (1)
1-59: LGTM! Clear and comprehensive tag taxonomy.The documentation provides a well-structured reference for blog tagging with clear categorization and consistent formatting. The lowercase kebab-case convention is explicitly stated and followed throughout.
Consider enhancing the documentation with:
- Brief usage examples for ambiguous tags (e.g., when to use
technical-deep-divevs.tutorial)- Guidance on the recommended number of tags per post
- Clarification on whether multiple content type tags can be used together
📝 Optional documentation enhancement
## Tag Format Use lowercase kebab-case (`tag-name`) for all tags. + +## Tag Selection Guidelines + +- Choose 2-5 tags per post for optimal discoverability +- Include at least one content type tag +- Add technical domain tags that represent major topics (not every minor mention) +- Multiple content type tags can be used when content serves multiple purposes (e.g., `tutorial` + `case-study`) ## Content Type Tags🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@website/documentation/contribute/documentation/blog-tags.md` around lines 1 - 59, Add short clarifying guidance and examples to the blog tag reference: under "Content Type Tags" add a one-sentence example for when to use `technical-deep-dive` versus `tutorial` and an explicit statement about whether multiple content-type tags (e.g., `feature-announcement` + `release-notes`) are allowed; add a small "Tag usage" subsection recommending a maximum number of tags per post (e.g., 3–5) and a one-line rule-of-thumb for selecting primary vs. secondary tags; place these additions near the existing Content Type Tags block so readers can quickly map tag names like `case-study`, `community-event`, `milestone` to concrete use cases.website/blog/2025/10/10-08-efs-filestore-csi-drivers.md (1)
11-18: Consider reducing the number of tags for better focus.This post has 7 tags, which is quite comprehensive but may dilute the effectiveness of tag-based filtering. While all tags have some relevance, consider focusing on the 3-4 most essential ones:
- Core tags:
technical-deep-dive,storage,provider-aws,provider-gcp- Optional/tangential:
security(briefly mentioned),networking(not the primary focus),node-management(indirectly related)Fewer, more focused tags typically provide better discoverability and user experience.
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@website/blog/2025/10/10-08-efs-filestore-csi-drivers.md` around lines 11 - 18, The frontmatter "tags" list is too broad (7 items); reduce it to 3–4 focused tags by keeping the core ones and removing tangential ones: update the YAML "tags" value to include "technical-deep-dive", "storage", and either or both "provider-aws" and "provider-gcp" as applicable, and remove or make optional "security", "networking", and "node-management" so tag-based filtering is more effective.website/blog/2018/12/12.25-gardener-cookies.md (1)
9-11: Consider whether these tags accurately reflect the content.The tags
technical-deep-diveandsecurityseem misaligned with a blog post about a cookie recipe. These tags would typically be used for technical documentation or security-related features, which may confuse users filtering or searching for such content.If this post is intended to be light-hearted/off-topic content, consider using tags like
communityorfuninstead, or removing tags entirely.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@website/blog/2018/12/12.25-gardener-cookies.md` around lines 9 - 11, The frontmatter tags currently list technical-deep-dive and security which don't match a cookie recipe; update the tags array in the post's frontmatter (the tags key) to more appropriate values—e.g., replace technical-deep-dive and security with community or fun, or remove the tags entirely—so the tags accurately reflect this light-hearted/off-topic blog post.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In @.vitepress/config.mts:
- Around line 305-308: The config sets
'data-modal-override-open-selector-search' to '#local-search' but that ID does
not exist, so the Kapa widget's search button and Cmd/Ctrl+K binding won't find
an element; either add an element with id="local-search" in the theme/layout
where the search modal is rendered (e.g., the header or search component) or
change the selector value in the config to the actual element's selector (the
real search button/modal ID/class) so 'data-modal-override-open-selector-search'
points to a valid DOM node.
In @.vitepress/data/blog.data.ts:
- Around line 55-99: The filter currently computes hasDate but never uses it and
the filename regex only matches MM.DD- patterns, causing undated posts to be
treated as newest; update the .filter callback to include hasDate (return
!isIndex && isNotFolder && hasDate) and broaden the filename regex used both
where hasDate is set and in the .map (replace /\/(\d{4})\/(\d{2})\.(\d{2})-/
with a pattern that accepts either a dot or dash between month and day, e.g.
/\/(\d{4})\/(\d{2})(?:[.-])(\d{2})-/), ensuring the same regex is used for the
initial hasDate computation and the url.match before calling formatDate.
- Around line 209-221: normalizeAuthor currently extracts a GitHub login but
still uses the original string as the display name, causing URLs to render as
names; update the string branch in normalizeAuthor to set name to the extracted
login when getGitHubLogin(trimmed) returns a non-empty login (fall back to
trimmed without leading '@' only if login is empty), and continue to use
toGitHubAvatar(login) for avatar resolution so display name, login and avatar
are consistent.
In @.vitepress/theme/components/BlogIndex.vue:
- Around line 28-61: selectedTags is being populated from window.location.search
at module setup time causing SSR vs client hydration mismatches; change it to
initialize selectedTags as an empty ref and defer calling
parseSelectedTagsFromUrl() to an onMounted() callback. Specifically: import
onMounted from 'vue', set const selectedTags = ref<string[]>([]) (leave tagQuery
as-is), and inside onMounted() call selectedTags.value =
parseSelectedTagsFromUrl() so no window-dependent parsing runs during server
render.
In @.vitepress/theme/components/BlogPostMeta.vue:
- Around line 45-76: The date-only ISO strings are parsed as UTC instants and
toLocaleDateString in toDisplayDate converts them to the viewer's local timezone
causing off-by-one-day for some users; update formatDate/toDisplayDate so that
when the input is a date-only string (e.g., parsed from formatDate's new
Date(trimmed) where trimmed matches /^\d{4}-\d{2}-\d{2}$/) you format using the
date's UTC components (getUTCFullYear/getUTCMonth/getUTCDate) or explicitly
build a Date at UTC midnight and format those UTC fields into the "Month D,
YYYY" string instead of relying on toLocaleDateString on a local-time Date; keep
toDisplayDate or add a toDisplayDateUTC helper and call it from formatDate for
date-only inputs and leave non-date-only behavior unchanged.
In `@hack/apply-blog-tags.mjs`:
- Around line 18-41: The taxonomy provider slugs in TAXONOMY_ORDER are
inconsistent with the front-matter (they use 'aws', 'azure', 'gcp',
'openstack'); update those entries to 'provider-aws', 'provider-azure',
'provider-gcp', 'provider-openstack' so they match the post front-matter; also
make the same replacement in the other provider tag list later in the file (the
provider tag array used around lines 87-93) so the script won't create duplicate
provider buckets when run.
In `@tmp-blog-page.html`:
- Around line 1-15: Delete the temporary development artifact tmp-blog-page.html
(it contains dev-only paths like "/@vite/client" and
"/@fs/app/node_modules/vitepress/dist/client/app/index.js") and remove any
references to it; instead add tmp-blog-page.html (or the tmp-*.html pattern) to
.gitignore to prevent committing dev-only files, and verify the build
configuration and project entry points do not rely on the id="app" placeholder
or these Vite dev paths before removing.
In `@website/blog/2018/12/12.22-cookies-are-dangerous.md`:
- Around line 9-11: The post's front-matter tags currently list
technical-deep-dive and security which are incorrect for a baking/holiday piece;
update the YAML tags block (the "tags:" list) to either remove tags entirely or
replace them with appropriate labels such as "community", "culture", "holiday",
"food", or "team" to reflect the edible cookies content so users filtering by
technical/security tags are not misled.
In `@website/blog/2022/03/03.23-gardener-community-meeting-march.md`:
- Around line 40-43: Replace the Hugo shortcode usage "{{< youtube
id="nV_JI8YWwY4" title="Deploying and Developing Gardener Locally (Without Any
External Infrastructure!)" >}}" with the VitePress component syntax for
embedding YouTube (e.g., the project's YouTube or Video component) so the video
renders after the "Recording" heading; update the line that contains the youtube
shortcode to call the VitePress component (passing id="nV_JI8YWwY4" and title as
props) consistent with other VitePress component usages in the repo.
In `@website/blog/2022/06/06.17-gardener-community-meeting-june.md`:
- Line 36: Several blog files (including
website/blog/2022/06/06.17-gardener-community-meeting-june.md) still contain
Hugo YouTube shortcodes like {{< youtube id="nG2FRYL05mc" title="..." >}}; run
the automated converter by executing the post-processing script to replace these
with the VitePress component format (<YouTubeVideo videoId="nG2FRYL05mc" />).
Specifically run node post-processing/part-1.js --youtube which invokes the
replaceYouTubeShortcodes() function to transform all occurrences across affected
posts; verify the resulting files contain the <YouTubeVideo videoId="..."/>
component and commit the changes.
In `@website/blog/2022/10/10.06-gardener-community-meeting-october.md`:
- Line 38: The Hugo youtube shortcode "{{< youtube id="46nCdVA-rsc"
title="Cilium / Isovalent Presentation" >}}" was not converted to the VitePress
component; replace this shortcode with the project's VitePress YouTube component
(e.g., the component used across the repo such as <YouTube .../>), passing the
same id ("46nCdVA-rsc") and title ("Cilium / Isovalent Presentation") props so
the video renders correctly; apply the same fix to the other blog files that
still contain Hugo youtube shortcodes.
In `@website/blog/2022/10/10.20-gardener-community-meeting-october-2.md`:
- Around line 34-35: Replace the Hugo shortcode {{< youtube id="5a_A3furzlg"
title="Get more computing power in Gardener with CRI-resource-manager" >}} with
the VitePress component equivalent (e.g., <YouTube id="5a_A3furzlg" title="Get
more computing power in Gardener with CRI-resource-manager" /> or the project's
defined <YouTube> component syntax), ensuring proper self-closing or explicit
opening/closing tags per the VitePress setup; apply the same replacement to the
other blog files that still contain Hugo shortcodes (06.17, 02.17, 10.06, 03.23)
so all shortcodes are converted consistently.
In `@website/blog/2024/11/11-09-demo.md`:
- Around line 23-25: Frontmatter includes an inappropriate tag: the
"observability" tag in the "tags:" block should be replaced with a more relevant
tag; update the frontmatter where "tags:" is defined to remove "observability"
and add a suitable alternative such as "getting-started", "tutorial", or
"community" (choose one consistent with your site's taxonomy) so the post about
the Gardener demo environment is correctly classified.
In
`@website/blog/2025/05/05-19-gardener-dashboard-180-streamlined-credentials-enhanced-cluster-views-and-real-time-updates.md`:
- Around line 11-15: Remove the incorrect "cost-optimization" tag from the tags
frontmatter block (the tags: list that currently includes feature-announcement,
cost-optimization, security, dashboard) so the post isn't surfaced under
cost-focused filters; if you intend to keep it, instead add a cost-specific
section to the article that justifies keeping "cost-optimization".
In
`@website/blog/2025/06/06-18-enhanced-internal-traffic-management-l7-load-balancing-for-kube-apiservers-in-gardener.md`:
- Around line 11-14: The front-matter tags list contains an incorrect "storage"
tag; remove the "storage" entry from the tags array in the article metadata (the
YAML block starting with "tags:") so only relevant tags remain (e.g., keep
"feature-announcement" and "networking") and ensure the YAML remains valid after
removing the "- storage" line.
In `@website/blog/2025/09/09-10-explicit-internal-dns-configuration-for-seeds.md`:
- Around line 11-15: The front-matter tags list includes a misleading
provider-specific tag "provider-aws"; remove "provider-aws" from the tags array
in the post's front-matter (the YAML block that currently lists tags:
technical-deep-dive, security, networking, provider-aws) so the article remains
provider-agnostic—simply delete the provider-aws entry and ensure the tags
remain valid YAML (comma-separated list or YAML sequence) with the other tags
unchanged.
In `@website/documentation/resources/videos/livecheck-readiness.md`:
- Line 8: The markdown uses the Hugo YouTube shortcode {{< youtube
id="mxEvAPQRwhw" title="Readiness != Liveness" >}} but the post-processing step
in part-1.js (invoked with the --youtube flag) is not guaranteed to run
automatically; either ensure the post-processing script is invoked as part of
the standard build pipeline (add the part-1.js --youtube invocation to the build
step in CI/npm scripts) or replace the shortcode in this file and the other
affected markdowns with the VitePress YouTube component syntax before merging so
the pages render correctly without manual post-processing.
---
Outside diff comments:
In `@website/blog/2018/06/06.11-shared-storage-with-s3-backend.md`:
- Line 20: Replace the transitional adverb "However" at the start of the
sentence "However, it can be used to import and parse large amounts of data into
a database." with a more appropriate connector such as "Additionally," or
"Furthermore," (or "For example," if you intend to illustrate a use case) so the
sentence reads e.g. "Additionally, it can be used to import and parse large
amounts of data into a database." This change improves cohesion without altering
the sentence content.
In `@website/blog/2018/12/12.25-gardener-cookies.md`:
- Line 17: The inline img element uses an invalid CSS value "margin: 0, auto";
update the style on the <img> (the tag with src="./images/slider/cookie-00.jpg")
to use a valid margin declaration "margin: 0 auto" (and keep the rest: "width:
50%; height: auto") so the image is properly centered.
---
Duplicate comments:
In `@website/blog/2022/02/02.17-gardener-community-meeting-february.md`:
- Around line 36-37: The file contains a Hugo shortcode "{{< youtube
id=\"U1VvyQiE3Jg\" title=\"Gardenctl-v2\" >}}" that wasn't converted to
VitePress format; replace that Hugo shortcode with the VitePress YouTube
component (use the YouTube component with id and title props) so the embedded
video uses the new VitePress component system and renders correctly.
---
Nitpick comments:
In `@website/blog/2018/06/06.11-auditing-kubernetes-for-secure-setup.md`:
- Around line 9-12: The tags list includes a tangential "networking" tag; review
the post's focus and either remove "networking" from the YAML tags array or
replace it with a more relevant tag (e.g., "compliance" or "audit") by editing
the tags block where "technical-deep-dive", "security", and "networking" are
defined so the metadata accurately reflects the article's primary focus.
In `@website/blog/2018/12/12.25-gardener-cookies.md`:
- Around line 9-11: The frontmatter tags currently list technical-deep-dive and
security which don't match a cookie recipe; update the tags array in the post's
frontmatter (the tags key) to more appropriate values—e.g., replace
technical-deep-dive and security with community or fun, or remove the tags
entirely—so the tags accurately reflect this light-hearted/off-topic blog post.
In `@website/blog/2025/10/10-08-efs-filestore-csi-drivers.md`:
- Around line 11-18: The frontmatter "tags" list is too broad (7 items); reduce
it to 3–4 focused tags by keeping the core ones and removing tangential ones:
update the YAML "tags" value to include "technical-deep-dive", "storage", and
either or both "provider-aws" and "provider-gcp" as applicable, and remove or
make optional "security", "networking", and "node-management" so tag-based
filtering is more effective.
In `@website/documentation/contribute/documentation/blog-tags.md`:
- Around line 1-59: Add short clarifying guidance and examples to the blog tag
reference: under "Content Type Tags" add a one-sentence example for when to use
`technical-deep-dive` versus `tutorial` and an explicit statement about whether
multiple content-type tags (e.g., `feature-announcement` + `release-notes`) are
allowed; add a small "Tag usage" subsection recommending a maximum number of
tags per post (e.g., 3–5) and a one-line rule-of-thumb for selecting primary vs.
secondary tags; place these additions near the existing Content Type Tags block
so readers can quickly map tag names like `case-study`, `community-event`,
`milestone` to concrete use cases.
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: b6be578c-987f-4709-9128-40d6b391e834
📒 Files selected for processing (96)
.vitepress/config.mts.vitepress/data/blog.data.ts.vitepress/theme/components/BlogIndex.vue.vitepress/theme/components/BlogPostMeta.vue.vitepress/theme/index.tshack/apply-blog-tags.mjshack/preview.shtmp-blog-page.htmlwebsite/blog/2018/06/06.11-anti-patterns.mdwebsite/blog/2018/06/06.11-auditing-kubernetes-for-secure-setup.mdwebsite/blog/2018/06/06.11-big-things-come-in-small-packages.mdwebsite/blog/2018/06/06.11-hardening-the-gardener-community-setup.mdwebsite/blog/2018/06/06.11-kubernetes-is-available-in-docker-for-mac-17-12-ce.mdwebsite/blog/2018/06/06.11-namespace-isolation.mdwebsite/blog/2018/06/06.11-namespace-scope.mdwebsite/blog/2018/06/06.11-readwritemany-dynamically-provisioned-persistent-volumes-using-amazon-efs.mdwebsite/blog/2018/06/06.11-shared-storage-with-s3-backend.mdwebsite/blog/2018/06/06.11-watching-logs-of-several-pods.mdwebsite/blog/2018/07/07.11-hibernate-a-cluster-to-save-money.mdwebsite/blog/2018/12/12.22-cookies-are-dangerous.mdwebsite/blog/2018/12/12.25-gardener-cookies.mdwebsite/blog/2019/05/05.24-cluster-api-machine-abstractions-kubecon-talk.mdwebsite/blog/2019/06/06.11-feature-flags-in-kubernetes-applications.mdwebsite/blog/2019/06/06.11-organizing-access-using-kubeconfig-files.mdwebsite/blog/2020/05/05.11-new-website-same-green-flower.mdwebsite/blog/2020/08/08.06-gardener-v1.8.0-released.mdwebsite/blog/2020/09/09.11-gardener-v1.9-and-v1.10-released.mdwebsite/blog/2020/10/10.19-gardener-integrates-with-kubevirt.mdwebsite/blog/2020/10/10.19-shoot-reconciliation-details.mdwebsite/blog/2020/11/11.04-gardener-v1.11-and-v1.12-released.mdwebsite/blog/2020/11/11.20-case-study-migrating-etcd-volumes-in-production.mdwebsite/blog/2020/11/11.23-gardener-v1.13-released.mdwebsite/blog/2020/12/12.03-stackit-kubernetes-engine-with-gardener.mdwebsite/blog/2021/01/01.25-machine-controller-manager.mdwebsite/blog/2021/02/02.01-happy-anniversary-gardener.mdwebsite/blog/2021/09/09.12-navigating-cloud-native-security.mdwebsite/blog/2022/02/02.17-gardener-community-meeting-february.mdwebsite/blog/2022/03/03.23-gardener-community-meeting-march.mdwebsite/blog/2022/06/06.17-gardener-community-meeting-june.mdwebsite/blog/2022/10/10.06-gardener-community-meeting-october.mdwebsite/blog/2022/10/10.20-gardener-community-meeting-october-2.mdwebsite/blog/2023/03/03-27-high-availability-and-zone-outage-toleration.mdwebsite/blog/2024/04/04-05-kubecon-cloudnativecon-europe-2024-highlights.mdwebsite/blog/2024/04/04-18-spinkube-gardener-shoot-cluster.mdwebsite/blog/2024/04/04-22-gardener's-registry-cache-extension-another-cost-saving-win-and-more.mdwebsite/blog/2024/05/05-21-innovation-unleashed-a-deep-dive-into-the-5th-gardener-community-hackathon.mdwebsite/blog/2024/10/10-24-gardener-kubecon-cloudnativecon-na-2024-announcement.mdwebsite/blog/2024/11/11-06-promcon-eu-2024.mdwebsite/blog/2024/11/11-09-demo.mdwebsite/blog/2024/12/12-08-unleashing-potential-highlights-from-the-6th-gardener-community-hackathon.mdwebsite/blog/2025/03/03-18-gardener-kubecon-cloudnativecon-europe-2025-announcement.mdwebsite/blog/2025/04/04-17-leaner-clusters-lower-bills.mdwebsite/blog/2025/05/05-12-gardener-neonephos.mdwebsite/blog/2025/05/05-19-enhanced-network-flexibility-gardener-now-supports-cidr-overlap-for-non-ha-shoots.mdwebsite/blog/2025/05/05-19-enhanced-node-management-introducing-in-place-updates-in-gardener.mdwebsite/blog/2025/05/05-19-gardener-dashboard-180-streamlined-credentials-enhanced-cluster-views-and-real-time-updates.mdwebsite/blog/2025/05/05-21-fine-tuning-kube-proxy-readiness-ensuring-accurate-health-checks-during-node-scale-down.mdwebsite/blog/2025/05/05-21-new-in-gardener-forceful-redeployment-of-gardenlets-for-enhanced-operational-control.mdwebsite/blog/2025/05/05-21-streamlined-node-onboarding-introducing-gardenadm-token-and-gardenadm-join.mdwebsite/blog/2025/06/06-17-taking-gardener-to-the-next-level-highlights-from-the-7th-gardener-community-hackathon-in-schelklingen.mdwebsite/blog/2025/06/06-18-enabling-seamless-ipv4-to-dual-stack-migration-for-kubernetes-clusters-on-gcp.mdwebsite/blog/2025/06/06-18-enhanced-extension-management-introducing-autoenable-and-clustercompatibility.mdwebsite/blog/2025/06/06-18-enhanced-internal-traffic-management-l7-load-balancing-for-kube-apiservers-in-gardener.mdwebsite/blog/2025/06/06-18-gardener-enhances-observability-with-opentelemetry-integration-for-logging.mdwebsite/blog/2025/06/06-25-enhanced-health-checks-for-node-rolling-updates.mdwebsite/blog/2025/06/06-25-enhancing-meltdown-protection-with-dependency-watchdog-annotations.mdwebsite/blog/2025/06/06-25-improving-credential-management-for-seed-backups.mdwebsite/blog/2025/06/06-25-introducing-gardenadm-bootstrap-for-autonomous-shoots.mdwebsite/blog/2025/06/06-30-getting-started-with-opentelemetry-on-gardener-shoot-cluster.mdwebsite/blog/2025/07/07-16-enhancing-data-protection-with-immutable-backup-buckets.mdwebsite/blog/2025/07/07-30-enhanced-network-flexibility-cidr-overlap-now-supported-for-ha-shoots.mdwebsite/blog/2025/08/08-04-cluster-api-provider-gardener.mdwebsite/blog/2025/08/08-13-keeping-track-of-your-resources-with-inventory.mdwebsite/blog/2025/08/08-27-enabling-node-local-dns-without-node-rollouts.mdwebsite/blog/2025/08/08-27-new-emergency-brake-for-gardener-shoot-reconciliations.mdwebsite/blog/2025/09/09-05-ipv6-update.mdwebsite/blog/2025/09/09-10-explicit-internal-dns-configuration-for-seeds.mdwebsite/blog/2025/09/09-10-modernizing-gardeners-logging-stack-with-opentelemetry.mdwebsite/blog/2025/10/10-08-efs-filestore-csi-drivers.mdwebsite/blog/2025/10/10-22-extensible-advertised-addresses-for-shoots.mdwebsite/blog/2025/10/10-22-useunifiedhttpproxy-feature-gate.mdwebsite/blog/2025/10/10-27-unifying-dns-behavior-custom-coredns-configurations-now-supported-in-node-local-dns.mdwebsite/blog/2025/11/11-12-gardener-ai-conformance.mdwebsite/blog/2025/11/11-13-promcon-eu-2025.mdwebsite/blog/2026/01/01-28-introducing-automated-credential-rotation.mdwebsite/blog/2026/02/02-18-enhanced-security-for-helm-deployments-gardener-adds-custom-ca-support-for-oci-registries.mdwebsite/blog/2026/02/02-18-seamlessly-switch-calicos-overlay-network-in-gardener.mdwebsite/blog/2026/02/02-18-simplify-multi-cluster-configuration-with-static-manifest-propagation.mdwebsite/blog/index.mdwebsite/documentation/contribute/documentation/blog-tags.mdwebsite/documentation/resources/videos/in-out-networking.mdwebsite/documentation/resources/videos/livecheck-readiness.mdwebsite/documentation/resources/videos/microservices-in_kubernetes.mdwebsite/documentation/resources/videos/namespace.mdwebsite/documentation/resources/videos/small-container.mdwebsite/documentation/resources/videos/why-kubernetes.md
💤 Files with no reviewable changes (1)
- website/blog/index.md
...rdener-dashboard-180-streamlined-credentials-enhanced-cluster-views-and-real-time-updates.md
Show resolved
Hide resolved
...18-enhanced-internal-traffic-management-l7-load-balancing-for-kube-apiservers-in-gardener.md
Outdated
Show resolved
Hide resolved
website/blog/2025/09/09-10-explicit-internal-dns-configuration-for-seeds.md
Outdated
Show resolved
Hide resolved
Contributor
There was a problem hiding this comment.
Actionable comments posted: 4
🧹 Nitpick comments (1)
website/blog/2018/12/12.22-cookies-are-dangerous.md (1)
9-10: Significant improvement: misleading technical tags removed.The previous critical issue has been resolved—
technical-deep-diveandsecuritytags have been replaced withcommunity-event. Whilecommunity-eventis defined as "Community meeting, conference, or hackathon updates" (per the blog tag reference documentation inwebsite/documentation/contribute/documentation/blog-tags.md), this tag is the closest fit among the available content-type tags for this informal Christmas team event post.Optional refinement: If the tagging system evolves to include a
team-cultureorinformaltag, this post might be a good candidate for reclassification.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@website/blog/2018/12/12.22-cookies-are-dangerous.md` around lines 9 - 10, Update the post's YAML frontmatter tags to reflect the event nature by removing technical tags and using the community-focused tag value currently set as "community-event"; in the frontmatter ensure the "tags:" key contains only appropriate content-type tags (replace "technical-deep-dive" and "security" with "community-event"), and consider adding a future "team-culture" or "informal" tag when the tagging taxonomy expands.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In @.vitepress/data/blog.data.ts:
- Around line 261-280: The loader currently only reads rawAuthor.avatar; update
the avatar resolution so it honors authors[].image too by computing avatar =
asNonEmptyString(rawAuthor.image) || asNonEmptyString(rawAuthor.avatar) ||
toGitHubAvatar(login). Change the avatar assignment (where avatar is declared)
to check rawAuthor.image first, then rawAuthor.avatar, then fallback to
toGitHubAvatar(login) and keep using asNonEmptyString(...) to validate values;
no other logic changes needed.
In @.vitepress/theme/components/BlogIndex.vue:
- Around line 63-65: The selectedTags state is initialized from
parseSelectedTagsFromUrl in onMounted but not written back when the user changes
filters, so toggleTag() and clearFilters() must also update the URL query;
locate the selectedTags state and add a single reactive sync (either by adding a
watch on selectedTags or by updating the query inside toggleTag() and
clearFilters()) that writes the current tag set to the query string (e.g.,
update the "tag" param with the joined tags or remove it when empty) using the
router (useRouter().replace) or history.replaceState so the address always
reflects the active tag set; ensure parseSelectedTagsFromUrl still reads that
same query on mount.
In @.vitepress/theme/components/BlogPostMeta.vue:
- Around line 35-36: The code is currently passing non-ISO, human-readable
strings (e.g., frontmatter.value.newsSubtitle) into formatDate which parses them
with local-time new Date and then formats to UTC, causing off-by-one-day issues
for timezones ahead of UTC; update the logic around formatDate (and the call
site using frontmatter.value.publishdate / frontmatter.value.date /
frontmatter.value.newsSubtitle) to detect non-ISO inputs and either (a) use the
already pre-formatted human-readable string as-is (return newsSubtitle
unchanged) or (b) normalize inputs to a proper ISO date before parsing; ensure
formatDate only parses ISO strings (or remove UTC conversion for non-ISO display
strings) so calendar dates aren’t shifted for users in UTC+ timezones.
In `@hack/apply-blog-tags.mjs`:
- Around line 122-128: The fallback tag 'technical-deep-dive' is being applied
based only on inferredTags before merge, which can cause duplicate content-type
tags; instead, remove the current check and after computing mergedTags =
mergeTags(inferredTags, existingTags) add the fallback only if mergedTags.length
=== 0 so the fallback is added only when neither heuristics nor frontmatter
produced any tags. Update the code around inferTags, normalizeTags and mergeTags
usage so the conditional check for adding 'technical-deep-dive' occurs after
mergeTags returns the final mergedTags.
---
Nitpick comments:
In `@website/blog/2018/12/12.22-cookies-are-dangerous.md`:
- Around line 9-10: Update the post's YAML frontmatter tags to reflect the event
nature by removing technical tags and using the community-focused tag value
currently set as "community-event"; in the frontmatter ensure the "tags:" key
contains only appropriate content-type tags (replace "technical-deep-dive" and
"security" with "community-event"), and consider adding a future "team-culture"
or "informal" tag when the tagging taxonomy expands.
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: a00bf0f1-e3cd-4c22-aad1-4ce0c7381013
📒 Files selected for processing (11)
.vitepress/data/blog.data.ts.vitepress/theme/components/BlogIndex.vue.vitepress/theme/components/BlogPostMeta.vuehack/apply-blog-tags.mjshack/preview.shwebsite/blog/2018/12/12.22-cookies-are-dangerous.mdwebsite/blog/2018/12/12.25-gardener-cookies.mdwebsite/blog/2024/11/11-09-demo.mdwebsite/blog/2025/05/05-19-enhanced-network-flexibility-gardener-now-supports-cidr-overlap-for-non-ha-shoots.mdwebsite/blog/2025/06/06-18-enhanced-internal-traffic-management-l7-load-balancing-for-kube-apiservers-in-gardener.mdwebsite/blog/2025/09/09-10-explicit-internal-dns-configuration-for-seeds.md
🚧 Files skipped from review as they are similar to previous changes (5)
- hack/preview.sh
- website/blog/2025/05/05-19-enhanced-network-flexibility-gardener-now-supports-cidr-overlap-for-non-ha-shoots.md
- website/blog/2025/09/09-10-explicit-internal-dns-configuration-for-seeds.md
- website/blog/2024/11/11-09-demo.md
- website/blog/2018/12/12.25-gardener-cookies.md
Contributor
There was a problem hiding this comment.
Actionable comments posted: 4
🧹 Nitpick comments (1)
website/documentation/contribute/documentation/style-guide/_index.md (1)
98-117: Consider adding field descriptions and examples.While the YAML template provides a good starting point, contributors would benefit from:
- Brief descriptions of what each field is used for (e.g., "
publishdatecontrols the post's publish date and sorting order")- A concrete example with real values instead of just placeholders
- Guidance on the
newsSubtitlefield format (it appears to be a date string based on the example, but the relationship topublishdateis unclear)🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@website/documentation/contribute/documentation/style-guide/_index.md` around lines 98 - 117, Add short descriptions for each YAML field (title, newsSubtitle, publishdate, authors, tags) and include a concrete example with real values to replace placeholders; explicitly state the expected format for newsSubtitle (e.g., "Month DD, YYYY") and clarify its relationship to publishdate (publishdate is the machine-readable ISO date used for sorting/URL generation, newsSubtitle is a human-readable display string), and show one full example block demonstrating these formats and a filled authors entry (avatar, login, email, name).
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@website/blog/2018/12/12.25-gardener-cookies.md`:
- Line 16: The <img> element in the blog post (the tag shown as <img
style="width: 50%; height: auto; margin: 0 auto"
src="./images/slider/cookie-00.jpg"/>) is missing an alt attribute; add a
concise, descriptive alt text (e.g., alt="Decorative cookies on a plate" or a
description appropriate to the image) to the tag so screen readers receive a
meaningful description.
- Around line 4-8: The front-matter authors block includes unsupported fields;
remove the email and image keys from the authors entry so only supported fields
remain (e.g. keep name and avatar, and add login if needed); locate the authors
block in the markdown (the authors: entry containing name, email, avatar, image)
and delete the email and image lines to match the shape expected by the blog
data importer (which only uses name, avatar, and login).
In `@website/documentation/contribute/documentation/style-guide/_index.md`:
- Line 109: The YAML-style frontmatter line contains a syntax error: the field
"name" currently reads `name: <author-name>>`; remove the extra closing angle
bracket so the line reads `name: <author-name>` to restore valid syntax (locate
the "name" entry in the frontmatter of the documentation/style-guide file and
edit that line).
- Around line 105-109: The author template in
website/documentation/contribute/documentation/style-guide/_index.md includes an
unused email field; update the template to remove the email key and add a short
note listing which Author fields are required vs optional based on the Author
interface in .vitepress/data/blog.data.ts (Author: name required; avatar?
optional; login? optional) so contributors see the correct schema and required
fields.
---
Nitpick comments:
In `@website/documentation/contribute/documentation/style-guide/_index.md`:
- Around line 98-117: Add short descriptions for each YAML field (title,
newsSubtitle, publishdate, authors, tags) and include a concrete example with
real values to replace placeholders; explicitly state the expected format for
newsSubtitle (e.g., "Month DD, YYYY") and clarify its relationship to
publishdate (publishdate is the machine-readable ISO date used for sorting/URL
generation, newsSubtitle is a human-readable display string), and show one full
example block demonstrating these formats and a filled authors entry (avatar,
login, email, name).
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: b3288f87-e53b-4c74-9d3f-99234aeebe9c
📒 Files selected for processing (5)
website/blog/2018/06/06.11-shared-storage-with-s3-backend.mdwebsite/blog/2018/12/12.25-gardener-cookies.mdwebsite/blog/2025/10/10-08-efs-filestore-csi-drivers.mdwebsite/documentation/contribute/documentation/blog-tags.mdwebsite/documentation/contribute/documentation/style-guide/_index.md
🚧 Files skipped from review as they are similar to previous changes (3)
- website/blog/2025/10/10-08-efs-filestore-csi-drivers.md
- website/documentation/contribute/documentation/blog-tags.md
- website/blog/2018/06/06.11-shared-storage-with-s3-backend.md
| For a team event during the Christmas season we decided to completely reinterpret the topic `cookies`. :-) | ||
|
|
||
| <img style="width: 50%; height: auto; margin: 0, auto" src="./images/slider/cookie-00.jpg"/> | ||
| <img style="width: 50%; height: auto; margin: 0 auto" src="./images/slider/cookie-00.jpg"/> |
Contributor
There was a problem hiding this comment.
Add alt text to the changed image.
The raw HTML image on Line 16 has no alt, so screen readers get no description.
Suggested fix
-<img style="width: 50%; height: auto; margin: 0 auto" src="./images/slider/cookie-00.jpg"/>
+<img style="width: 50%; height: auto; margin: 0 auto" src="./images/slider/cookie-00.jpg" alt="Green tea matcha cookies"/>📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
Suggested change
| <img style="width: 50%; height: auto; margin: 0 auto" src="./images/slider/cookie-00.jpg"/> | |
| <img style="width: 50%; height: auto; margin: 0 auto" src="./images/slider/cookie-00.jpg" alt="Green tea matcha cookies"/> |
🧰 Tools
🪛 markdownlint-cli2 (0.21.0)
[warning] 16-16: Images should have alternate text (alt text)
(MD045, no-alt-text)
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@website/blog/2018/12/12.25-gardener-cookies.md` at line 16, The <img> element
in the blog post (the tag shown as <img style="width: 50%; height: auto; margin:
0 auto" src="./images/slider/cookie-00.jpg"/>) is missing an alt attribute; add
a concise, descriptive alt text (e.g., alt="Decorative cookies on a plate" or a
description appropriate to the image) to the tag so screen readers receive a
meaningful description.
Comment on lines
+105
to
+109
| authors: | ||
| - avatar: <author-profile-picture-link> | ||
| login: <author-GitHub-name> | ||
| email: <author-email> | ||
| name: <author-name>> |
Contributor
There was a problem hiding this comment.
🧩 Analysis chain
🏁 Script executed:
#!/bin/bash
# Description: Check how author fields (especially email) are consumed in the blog data pipeline and components
# Search for email field usage in blog-related files
echo "=== Searching for 'email' field in blog data and components ==="
rg -n --type=ts --type=vue -C3 'email' -g '*blog*'
# Search for Author interface/type definitions
echo -e "\n=== Searching for Author type definitions ==="
ast-grep --pattern $'interface Author {
$$$
}'
# Check how authors are processed in blog data
echo -e "\n=== Checking blog data processing ==="
fd -e ts -e js 'blog.data' --exec cat {}Repository: gardener/documentation
Length of output: 260
🏁 Script executed:
# Search for Author definition in all file types
rg -n "interface Author\|type Author\|class Author" --type-list | head -20
# More general search for "Author"
rg -n "Author" -g "*.ts" -g "*.js" -g "*.tsx" -g "*.jsx" -C 2 | head -50
# Look for author-related configurations
fd -e ts -e js -e json -e yml -e yaml | xargs grep -l -i "author" 2>/dev/null | head -20Repository: gardener/documentation
Length of output: 857
🏁 Script executed:
# Check the directory structure of the documentation
find website/documentation/contribute -type f -name "*.md" | head -20
# Look at the actual file mentioned in the review
head -120 website/documentation/contribute/documentation/style-guide/_index.md | tail -20Repository: gardener/documentation
Length of output: 1746
🏁 Script executed:
# Search for email field across all file types
rg -n "email" -C 2 | head -80
# Look for blog or author processing logic
fd -type f \( -name "*.ts" -o -name "*.js" \) -path "*blog*" -o -name "*author*"Repository: gardener/documentation
Length of output: 7746
🏁 Script executed:
# Search for blog-related TypeScript/JavaScript files
fd -type f \( -name "*.ts" -o -name "*.js" \) | xargs grep -l "author" -i | head -30Repository: gardener/documentation
Length of output: 236
🏁 Script executed:
# Look for blog data or config files
find . -type f \( -name "*.ts" -o -name "*.js" -o -name "*.json" \) | xargs grep -l "blog" 2>/dev/null | grep -v node_modules | head -20Repository: gardener/documentation
Length of output: 187
🏁 Script executed:
# Search for more blog posts to see what author fields are actually used
find website/blog -name "*.md" -o -name "*.yml" | head -5 | xargs head -20Repository: gardener/documentation
Length of output: 4963
🏁 Script executed:
# Look for type definitions or interfaces related to authors
rg -n "interface|type|class" | grep -i "author" | head -20Repository: gardener/documentation
Length of output: 3955
🏁 Script executed:
# Search in website directory specifically for author-related code
find website -type f \( -name "*.ts" -o -name "*.js" -o -name "*.tsx" \) -exec grep -l "author" {} \;Repository: gardener/documentation
Length of output: 48
🏁 Script executed:
cat -n ./.vitepress/data/blog.data.tsRepository: gardener/documentation
Length of output: 16506
Remove the email field from the author template and document which fields are required vs. optional.
The Author interface in .vitepress/data/blog.data.ts (lines 40-44) only defines name (required), avatar? (optional), and login? (optional). The email field in the template is not consumed by the blog system—it's silently ignored during author normalization. Additionally, the template should clarify which fields are required versus optional to guide contributors accurately.
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@website/documentation/contribute/documentation/style-guide/_index.md` around
lines 105 - 109, The author template in
website/documentation/contribute/documentation/style-guide/_index.md includes an
unused email field; update the template to remove the email key and add a short
note listing which Author fields are required vs optional based on the Author
interface in .vitepress/data/blog.data.ts (Author: name required; avatar?
optional; login? optional) so contributors see the correct schema and required
fields.
website/documentation/contribute/documentation/style-guide/_index.md
Outdated
Show resolved
Hide resolved
Contributor
There was a problem hiding this comment.
Actionable comments posted: 2
🧹 Nitpick comments (1)
.vitepress/theme/components/BlogPostMeta.vue (1)
87-199: Extract the metadata normalizers instead of maintaining another copy.
normalizeAuthors(),normalizeAuthor(),getGitHubLogin(),TAG_ALIASES, anduniqueNonEmpty()are duplicated here and in.vitepress/data/blog.data.ts, and the copies already differ (normalizeAuthors()dedupes byname + loginthere but only bynamehere). A small shared helper module would keep the cards and post page aligned the next time author or tag handling changes.Also applies to: 201-249
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In @.vitepress/theme/components/BlogPostMeta.vue around lines 87 - 199, This file duplicates metadata-normalizer logic already present in .vitepress/data/blog.data.ts; extract normalizeAuthors, normalizeAuthor, getGitHubLogin, TAG_ALIASES, and uniqueNonEmpty into a single shared helper module (e.g., vitepress/helpers/metadata.ts), update both .vitepress/theme/components/BlogPostMeta.vue and .vitepress/data/blog.data.ts to import those functions, and ensure the canonical dedupe behaviour (use name+login dedupe as used in blog.data.ts) is preserved so both consumers share identical normalization logic and outputs.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In @.vitepress/data/blog.data.ts:
- Line 75: The code at the authors normalization step only reads
frontmatter.authors as an array and drops singleton or legacy author fields;
update the logic around normalizeAuthors/normalizeAuthor to accept string/object
singletons and fall back to frontmatter.author when authors is missing—i.e., if
frontmatter.authors is not an array, pass frontmatter.authors (string/object) or
frontmatter.author into normalizeAuthors (or call normalizeAuthor directly) so
normalizeAuthors(normalizeAuthor(frontmatter.author/ frontmatter.authors))
returns a consistent array; also mirror this fix in BlogPostMeta.vue where
authors are read so both index and post page use the same normalized authors
output.
In @.vitepress/theme/components/BlogIndex.vue:
- Around line 31-58: In parseSelectedTagsFromUrl(), canonicalize each query tag
value before adding to selected: after trimming each raw value (in function
parseSelectedTagsFromUrl), call the existing canonicalizeTag(...) helper on the
normalized value, skip empty results, and perform the case-insensitive
dedupe/contain check against the canonicalized form (not the raw value) so
legacy slugs like "aws" or "neonephos" map to their normalized tags (e.g.,
"provider-aws") before populating selected.
---
Nitpick comments:
In @.vitepress/theme/components/BlogPostMeta.vue:
- Around line 87-199: This file duplicates metadata-normalizer logic already
present in .vitepress/data/blog.data.ts; extract normalizeAuthors,
normalizeAuthor, getGitHubLogin, TAG_ALIASES, and uniqueNonEmpty into a single
shared helper module (e.g., vitepress/helpers/metadata.ts), update both
.vitepress/theme/components/BlogPostMeta.vue and .vitepress/data/blog.data.ts to
import those functions, and ensure the canonical dedupe behaviour (use
name+login dedupe as used in blog.data.ts) is preserved so both consumers share
identical normalization logic and outputs.
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: 049111b6-667e-4b88-b860-c3454a5dfa92
📒 Files selected for processing (4)
.vitepress/data/blog.data.ts.vitepress/theme/components/BlogIndex.vue.vitepress/theme/components/BlogPostMeta.vuehack/apply-blog-tags.mjs
Contributor
There was a problem hiding this comment.
Actionable comments posted: 1
♻️ Duplicate comments (3)
website/blog/2018/12/12.25-gardener-cookies.md (2)
15-15:⚠️ Potential issue | 🟡 MinorAdd alt text to the changed image.
Line 15 still uses a raw
<img>withoutalt, so screen readers get no description. Use a short descriptivealtvalue, oralt=""if the image is purely decorative.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@website/blog/2018/12/12.25-gardener-cookies.md` at line 15, The <img> element with src="./images/slider/cookie-00.jpg" is missing an alt attribute; update that <img> tag (the one with style="width: 50%; height: auto; margin: 0 auto") to include a short descriptive alt (e.g., alt="Gardener holding cookies" or an appropriate description) or alt="" if it is purely decorative so screen readers get correct behavior.
4-7:⚠️ Potential issue | 🟠 MajorRemove the unused
The blog metadata pipeline only preserves
name,avatar, andlogin, so Line 6 just publishes personal contact data without affecting rendering.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@website/blog/2018/12/12.25-gardener-cookies.md` around lines 4 - 7, Remove the unused front-matter field "email" from the blog post's header (the YAML block containing authors) — the metadata pipeline only preserves name, avatar and login, so delete the "email: andreas.herz@sap.com" entry under the authors block to avoid publishing personal contact data while keeping the existing name and avatar entries unchanged.website/documentation/contribute/documentation/style-guide/_index.md (1)
105-109:⚠️ Potential issue | 🟡 MinorKeep the template aligned with the supported author schema.
The example still lists
name,avatar, andlogin. That encourages unused PII in front matter and makes the optional fields look required. Please dropnameis required.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@website/documentation/contribute/documentation/style-guide/_index.md` around lines 105 - 109, The authors front-matter example includes an unsupported `email` field; update the example under `authors:` to remove the `email:` line and adjust the surrounding text to state that only `name` is required while `avatar` and `login` are optional, ensuring the example matches the normalizer which exposes only `name`, `avatar`, and `login`.
🧹 Nitpick comments (1)
.vitepress/data/blog.data.ts (1)
45-48: AddincludeSrc: trueto expose raw markdown for preview extraction.The loader currently requests
excerptandrender(which providespage.html), butgetSourceMarkdown()readspage.src. To populatepage.src, setincludeSrc: true.Suggested change
export default createContentLoader('blog/**/*.md', { excerpt: true, + includeSrc: true, render: true,Also applies to: lines 65–65, 197–204
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In @.vitepress/data/blog.data.ts around lines 45 - 48, The createContentLoader call in the blog data loader needs includeSrc: true so page.src is populated for getSourceMarkdown() — update the createContentLoader options (the object passed to createContentLoader in this file where transform(raw): Post[] is defined) to add includeSrc: true alongside excerpt and render; make the same addition to the other createContentLoader invocations referenced (the blocks around lines with similar loaders) so page.src is available for preview extraction and getSourceMarkdown() can read raw markdown.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In @.vitepress/theme/components/BlogPostMeta.vue:
- Around line 31-33: The computed dateText currently only uses
frontmatter.publishdate/date/newsSubtitle and therefore drops filename-derived
dates; change dateText (in BlogPostMeta.vue) to use the same fallback logic as
the index loader by deriving raw from frontmatter.publishdate ||
frontmatter.date || extractDateFromBlogUrl(frontmatter.url) (or the shared
helper used by .vitepress/data/blog.data.ts), then pass that raw into
formatDate; import and reuse the existing extractDateFromBlogUrl or the shared
helper instead of relying solely on frontmatter to ensure filename-based dates
appear on the post page.
---
Duplicate comments:
In `@website/blog/2018/12/12.25-gardener-cookies.md`:
- Line 15: The <img> element with src="./images/slider/cookie-00.jpg" is missing
an alt attribute; update that <img> tag (the one with style="width: 50%; height:
auto; margin: 0 auto") to include a short descriptive alt (e.g., alt="Gardener
holding cookies" or an appropriate description) or alt="" if it is purely
decorative so screen readers get correct behavior.
- Around line 4-7: Remove the unused front-matter field "email" from the blog
post's header (the YAML block containing authors) — the metadata pipeline only
preserves name, avatar and login, so delete the "email: andreas.herz@sap.com"
entry under the authors block to avoid publishing personal contact data while
keeping the existing name and avatar entries unchanged.
In `@website/documentation/contribute/documentation/style-guide/_index.md`:
- Around line 105-109: The authors front-matter example includes an unsupported
`email` field; update the example under `authors:` to remove the `email:` line
and adjust the surrounding text to state that only `name` is required while
`avatar` and `login` are optional, ensuring the example matches the normalizer
which exposes only `name`, `avatar`, and `login`.
---
Nitpick comments:
In @.vitepress/data/blog.data.ts:
- Around line 45-48: The createContentLoader call in the blog data loader needs
includeSrc: true so page.src is populated for getSourceMarkdown() — update the
createContentLoader options (the object passed to createContentLoader in this
file where transform(raw): Post[] is defined) to add includeSrc: true alongside
excerpt and render; make the same addition to the other createContentLoader
invocations referenced (the blocks around lines with similar loaders) so
page.src is available for preview extraction and getSourceMarkdown() can read
raw markdown.
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: 28888f70-9fe4-4c73-b2ad-0a465508c811
📒 Files selected for processing (8)
.vitepress/data/blog.data.ts.vitepress/shared/blogMetadata.ts.vitepress/theme/components/BlogIndex.vue.vitepress/theme/components/BlogPostMeta.vuewebsite/blog/2018/12/12.25-gardener-cookies.mdwebsite/blog/2020/05/05.11-new-website-same-green-flower.mdwebsite/blog/2024/11/11-06-promcon-eu-2024.mdwebsite/documentation/contribute/documentation/style-guide/_index.md
🚧 Files skipped from review as they are similar to previous changes (2)
- website/blog/2020/05/05.11-new-website-same-green-flower.md
- website/blog/2024/11/11-06-promcon-eu-2024.md
Comment on lines
+31
to
+33
| const dateText = computed(() => { | ||
| const raw = frontmatter.value.publishdate || frontmatter.value.date || frontmatter.value.newsSubtitle | ||
| return formatDate(raw) |
Contributor
There was a problem hiding this comment.
Use the same filename-date fallback as the index loader.
.vitepress/data/blog.data.ts already derives dates from publishdate, date, or extractDateFromBlogUrl(url), but dateText only reads front matter here. Posts that rely on filename dates will show a date in the index/archive and then lose it on the post page. Reuse the same fallback here, ideally via a shared helper.
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In @.vitepress/theme/components/BlogPostMeta.vue around lines 31 - 33, The
computed dateText currently only uses frontmatter.publishdate/date/newsSubtitle
and therefore drops filename-derived dates; change dateText (in
BlogPostMeta.vue) to use the same fallback logic as the index loader by deriving
raw from frontmatter.publishdate || frontmatter.date ||
extractDateFromBlogUrl(frontmatter.url) (or the shared helper used by
.vitepress/data/blog.data.ts), then pass that raw into formatDate; import and
reuse the existing extractDateFromBlogUrl or the shared helper instead of
relying solely on frontmatter to ensure filename-based dates appear on the post
page.
BoHristova
previously approved these changes
Mar 20, 2026
Contributor
BoHristova
left a comment
BoHristova
left a comment
There was a problem hiding this comment.
Awesome work! Thank you, @n-boshnakov!
/lgtm
|
LGTM label has been added. DetailsGit tree hash: e9ce20f51165d1af128137c25cf0129aadfa1571 |
gardener-prow
bot
added
the
approved
- added a tagging system to the blog posts in order to satisfy Apeiro requirement - added tags to existing blog posts - improved the view of the blogs - added author name + picture for each post - converted Hugo shortcode syntax inside Markdown to the Vitepress component format in order to fix build issues
n-boshnakov
force-pushed
the
n-boshnakov/updated-blog-structure
branch
from
111fbd4 to
1d6a1dc
Compare
gardener-prow
bot
removed
lgtm
Contributor
Author
|
The PR has been rebased with the latest changes on the main branch. On mobile view, the blog preview stays inside the screen: |
BoHristova
approved these changes
Mar 20, 2026
Contributor
BoHristova
left a comment
BoHristova
left a comment
There was a problem hiding this comment.
Thank you for the complete revamp! The blog sections looks sooooooo much better now!
/lgtm
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: BoHristova The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
|
LGTM label has been added. DetailsGit tree hash: 3d1ed4daaaafbca602b6e637dc80b702157798fd |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
How to categorize this PR?
/kind enhancement
/kind cleanup
What this PR does / why we need it:
This PR updates the blogs visualization. Major changes:
Which issue(s) this PR fixes:
Fixes #882
Special notes for your reviewer:
View of changes:
Summary by CodeRabbit
New Features
Documentation
Content