docs(notediscovery): sync 0.28.1 default

[mberlofa]

Summary

• Sync NoteDiscovery docs with the chart default image tag 0.28.1.
• Add the pinned image tag control to the playground defaults.
• Document the 0.28.x large-vault indexing/storage path notes for operators.

Validation

• make site-sync-check CHART=notediscovery
• npm run format:check
• npm run lint
• npm run build
• make release-check REPO=site
• make attribution-check REPO=site

Summary by CodeRabbit

• New Features

  • Added an Image Tag setting to the Qdrant playground configuration to simplify selecting the container version.

• Documentation

  • Updated NoteDiscovery chart documentation to reference the latest pinned container image tag.
  • Added an Upgrade Notes section highlighting changes in the new version and guidance to keep storage mount paths stable during upgrades.

[coderabbitai]

📝 Walkthrough

Walkthrough

Adds an Image Tag field to the qdrant playground config. Updates NoteDiscovery docs to pin image version 0.28.1 and adds upgrade notes about release changes and storage-path behavior.

Changes

Config and Docs Update

Layer / File(s)	Summary
Qdrant playground config field src/data/playground-configs.ts	Adds an "Image Tag" text field (image.tag) with default 0.28.1 to the qdrant chart's General section, before the HTTP Port field.
NoteDiscovery documentation update src/pages/docs/charts/notediscovery.mdx	Bumps the pinned image reference to 0.28.1 and adds an Upgrade Notes section describing release changes and storage-path behavior during upgrades.

Estimated code review effort: 1 (Trivial) | ~5 minutes

Possibly related PRs

• helmforgedev/site#351: Also changes the qdrant playground configuration, so it’s adjacent to the same chart-config area.

Suggested labels: documentation, enhancement

Suggested reviewers: None identified.

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title is concise and relevant, but it only mentions the NoteDiscovery docs and omits the playground default update.
Description check	✅ Passed	The description summarizes the change and validation, but it does not follow the template's exact Quality Gates and Related Issue sections.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch docs/notediscovery-0.28.1

---

_{Comment @coderabbitai help to get the list of available commands.}

[coderabbitai]

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@src/data/playground-configs.ts`:
- Around line 5156-5162: The Image Tag control is currently defined in
langflow-specific playground config even though its default and description are
NoteDiscovery-specific. Move the image.tag field from the langflow configuration
in playground-configs to the NoteDiscovery chart/config section, and ensure the
NoteDiscovery playground uses the same key with its matching metadata so the
control appears under the correct chart.

🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

• Push a commit to this branch (recommended)
• Create a new PR with the fixes

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: ccbaede8-b0dc-4773-8825-6809243385c5

📥 Commits

Reviewing files that changed from the base of the PR and between 24e68e4 and e6ca35f.

📒 Files selected for processing (2)

• src/data/playground-configs.ts
• src/pages/docs/charts/notediscovery.mdx

[coderabbitai]

♻️ Duplicate comments (1)

src/data/playground-configs.ts (1)

8747-8753: 🎯 Functional Correctness | 🟠 Major | ⚡ Quick win

Misplaced Image Tag field: added to qdrant, but description/default are NoteDiscovery-specific

This field's label, default (0.28.1), and description ("Pinned NoteDiscovery image tag") all reference NoteDiscovery, not Qdrant. Meanwhile, the notediscovery chart's General section (lines 8936-8962) has no image.tag field. This mirrors a previously flagged and supposedly-fixed issue where this same field was misplaced under langflow; it appears to have resurfaced under qdrant instead of landing in notediscovery.

🐛 Proposed fix: move the field to the notediscovery chart

   qdrant: [
     {
       name: 'General',
       fields: [
-        {
-          label: 'Image Tag',
-          key: 'image.tag',
-          type: 'text',
-          default: '0.28.1',
-          description: 'Pinned NoteDiscovery image tag',
-        },
         {
           label: 'HTTP Port',
           key: 'app.port',

   notediscovery: [
     {
       name: 'General',
       fields: [
+        {
+          label: 'Image Tag',
+          key: 'image.tag',
+          type: 'text',
+          default: '0.28.1',
+          description: 'Pinned NoteDiscovery image tag',
+        },
         {
           label: 'HTTP Port',
           key: 'app.port',

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@src/data/playground-configs.ts` around lines 8747 - 8753, The Image Tag field
is misplaced in the qdrant config even though its label, default, and
description are NoteDiscovery-specific. Move the `image.tag` entry out of the
qdrant section in `playground-configs` and add it to the `notediscovery` chart’s
General configuration, keeping the existing NoteDiscovery
label/default/description there. Use the `image.tag` field definition and the
qdrant/notediscovery chart sections to relocate it correctly.

🤖 Prompt for all review comments with AI agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Duplicate comments:
In `@src/data/playground-configs.ts`:
- Around line 8747-8753: The Image Tag field is misplaced in the qdrant config
even though its label, default, and description are NoteDiscovery-specific. Move
the `image.tag` entry out of the qdrant section in `playground-configs` and add
it to the `notediscovery` chart’s General configuration, keeping the existing
NoteDiscovery label/default/description there. Use the `image.tag` field
definition and the qdrant/notediscovery chart sections to relocate it correctly.

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: b54bb50d-bb41-4caa-8b40-5ec4cd237097

📥 Commits

Reviewing files that changed from the base of the PR and between e6ca35f and 7570d20.

📒 Files selected for processing (1)

• src/data/playground-configs.ts