[Improvement] Add contributor docs, AI assistant rules, and fix helm-lint

.claude/skills/bump/SKILL.md

@@ -0,0 +1,25 @@
+---
+name: bump
+description: Bump the chart version in charts/mlrun-ce/Chart.yaml (patch, minor, or rc)
+allowed-tools: Read(charts/mlrun-ce/Chart.yaml) Edit(charts/mlrun-ce/Chart.yaml) Read(charts/mlrun-ce/README.md) Edit(charts/mlrun-ce/README.md)
+disable-model-invocation: false
+---
+
+Bump the version in `charts/mlrun-ce/Chart.yaml`.
+
+Usage: /bump <patch|minor|rc>
+
+- `patch` — increment the patch digit: `0.11.0` → `0.11.1`
+- `minor` — increment the minor digit and reset patch: `0.11.3` → `0.12.0`
+- `rc` — increment the RC counter on the current version: `0.11.0-rc.34` → `0.11.0-rc.35`
+  - If the current version has no RC suffix, add `-rc.1`: `0.11.0` → `0.11.0-rc.1`
+
+Steps:
+1. Read the current version from `charts/mlrun-ce/Chart.yaml` (the `version:` field).
+2. Compute the new version according to the argument above.
+3. Show the user: "Bumping `<old>` → `<new>`" and ask for confirmation before writing.
+4. On confirmation, update the `version:` field in `charts/mlrun-ce/Chart.yaml` in-place.
+5. Remind the user: version bumps must be committed before opening a PR, and the PR title must follow `[Scope] description` format.
+6. Update the MLRun CE version under Version Matrix in `charts/mlrun-ce/README.md`.
+
+If no argument is given, show the current version and list the three options with the resulting version for each, then ask which to apply.

.claude/skills/pr/SKILL.md

@@ -0,0 +1,108 @@
+---
+name: pr
+description: Analyze branch changes and generate a fully filled PR description ready to paste into GitHub
+allowed-tools: Bash(git diff*) Bash(git log*)
+disable-model-invocation: false
+---
+
+Analyze the current branch changes and generate a fully filled PR description ready to paste into GitHub.
+
+## Steps
+
+1. **Gather context** — run these in parallel:
+   - `git diff upstream/development...HEAD` — full diff against the base branch
+   - `git log upstream/development..HEAD --oneline` — commit list
+   - `git diff upstream/development...HEAD --name-only` — changed files
+
+2. **Analyze the diff** carefully:
+   - What components or templates were changed? (check which `templates/` subdirs, `values.yaml` sections, `requirements.yaml`, `Chart.yaml`)
+   - Were any new values keys added? Do they need to be reflected in the three install-mode values files?
+   - Were any Secrets, ConfigMaps, or port numbers changed? (potential breaking changes)
+   - Was `Chart.yaml` version bumped? If not, flag it.
+   - Were `requirements.yaml` or `requirements.lock` changed?
+   - Does `charts/mlrun-ce/README.md` need updating (new NodePort, new component, new install step)?
+
+3. **Detect breaking changes** — flag as breaking if any of:
+   - A value key was renamed or removed
+   - A Secret or ConfigMap name changed
+   - A NodePort number changed
+   - A sub-chart was upgraded with a major version bump
+   - The storage credentials structure changed
+   - Any hook annotation or hook-weight changed in a way that affects upgrade order
+
+4. Provide an optional PR title following the `[Scope] description` format, where Scope is one of: `['feature', 'fix', 'docs', 'improvement', 'revert', 'breaking', 'ci']`. For example: `[Feature] Add Redis support to mlrun-ce`.
+5. **Fill the PR template** — produce the complete filled template below. Be specific and concrete; do not use placeholder text.
+
+---
+
+Apply these checklist rules before writing the output:
+- `[x]` — you can confirm this item is satisfied from the diff alone
+- `[ ]` — requires human action, judgment, or external system access
+
+Specific rules:
+- "tested" → always `[ ]`
+- "documentation PR" → always `[ ]`
+- "QA tests / Jira ticket" → always `[ ]`
+- "installation verified" → always `[ ]`
+- `Chart.yaml` version bump → `[x]` if diff shows version changed, otherwise `[ ]` and add to Warnings
+- Multi-namespace values files → `[x]` if all three are in the diff OR the change has no effect on install-mode values; `[ ]` with a note if a new value was added only to `values.yaml`
+- README update → `[x]` if `charts/mlrun-ce/README.md` is in the diff OR no new NodePorts/components were added; otherwise `[ ]`
+
+Output exactly this structure with real content (no placeholder text):
+
+```markdown
+### 📝 Description
+<2-4 sentences: what changed, why, and what it affects>
+
+---
+
+### 🛠️ Changes Made
+<concrete bullet list — file paths, value keys, resource names>
+
+---
+
+### ✅ Checklist
+- [ ] I have tested the changes in this PR
+- [ ] I confirmed whether my changes require a change in documentation and if so, I created another PR in MLRun for the relevant documentation.
+- [ ] I confirmed whether my changes require changes in QA tests, for example: credentials changes, resources naming change and if so, I updated the relevant Jira ticket for QA.
+- [ ] I increased the Chart version in `charts/mlrun-ce/Chart.yaml`.
+- [ ] I confirmed that the installation works both on a local Docker Desktop environment and on a real cluster when using the required [prerequisites](https://docs.mlrun.org/en/stable/install-mlrun-ce/kubernetes-install.html#prerequisites).
+  - [ ] If installation issues were found, I updated the relevant Jira ticket with the issue and steps to reproduce, or updated the prerequisites documentation if the issue is related to missing or outdated prerequisites.
+- [ ] If needed, update https://github.com/mlrun/ce/blob/development/charts/mlrun-ce/README.md with the relevant installation instructions and version Matrix.
+- [ ] If needed, update the following values files for multi namespace support:
+  - [ ] [Admin values](https://github.com/mlrun/ce/blob/development/charts/mlrun-ce/admin_installation_values.yaml)
+  - [ ] [User values Node Port](https://github.com/mlrun/ce/blob/development/charts/mlrun-ce/non_admin_installation_values.yaml)
+  - [ ] [User values ClusterIP](https://github.com/mlrun/ce/blob/development/charts/mlrun-ce/non_admin_cluster_ip_installation_values.yaml)
+
+---
+
+### 🧪 Testing
+<what was tested: lint, helm template dry-run, Kind cluster, manual — based on nature of changes>
+
+---
+
+### 🔗 References
+- Ticket link:
+- External links:
+- Design docs links (Optional):
+
+---
+
+### 🚨 Breaking Changes?
+
+- [ ] Yes (explain below)
+- [ ] No
+
+<if breaking: bullet list of what downstream consumers must change — value keys to rename, Secrets to recreate, ports to update>
+
+---
+
+### 🔍️ Additional Notes
+<follow-up tasks, known issues, affected areas — omit if nothing to add>
+```
+
+Then replace each `[ ]` with `[x]` on items you can confirm from the diff, following the rules above.
+
+After outputting the filled template, add a short **"Warnings"** section (outside the template) listing anything that needs human attention before opening the PR (missing version bump, unsynced values files, potential breaking changes, etc.).
+
+Between every sentence that ends with a `.` add two new lines to make it more readable.

.cursorrules

@@ -0,0 +1,64 @@
+# MLRun Community Edition — Cursor Rules
+
+## Source of Truth
+
+`AGENTS.md` is the authoritative reference for this project. Read it before making any suggestions. It covers architecture, design patterns, template conventions, component dependencies, how to add new components, and common debugging scenarios.
+
+`CONTRIBUTING.md` covers the development workflow, commit format, and PR process.
+
+## Preferred Response Patterns
+<!-- Mirrors CLAUDE.md — duplicated here because Cursor has no @file import support -->
+
+- Values changes: show `--set` flags or a patch values file overlay, not edits to `values.yaml` directly
+- New templates: show the complete file including the `{{- if .Values.<component>.enabled }}` guard and `include "mlrun-ce.common.labels"` call
+- Service references within templates: use `{{ .Release.Namespace }}`, never hardcode namespace strings
+- After any `requirements.yaml` change: remind the user to run `make helm-update-dependencies` and commit `requirements.lock`
+- If a change affects the default installation, remind the user to update `values.yaml` and all three install-mode values files (`admin_installation_values.yaml`, `non_admin_installation_values.yaml`, `non_admin_cluster_ip_installation_values.yaml`) with the appropriate default
+- If a change adds a new component, changes a component version, or changes the installation process, remind the user to update `charts/mlrun-ce/README.md`
+
+## Common Tasks (Claude Code has `/bump`, `/pr` skills for these)
+
+When a user asks you to help with the following tasks, use the commands below — these are the manual equivalents of the Claude Code skills defined in `.claude/skills/`.
+
+**Render chart templates**
+```bash
+# Full chart
+helm template mlrun charts/mlrun-ce -f charts/mlrun-ce/values.yaml
+
+# Single template or directory
+helm template mlrun charts/mlrun-ce -f charts/mlrun-ce/values.yaml \
+  --show-only templates/<path>
+
+# With a values overlay (e.g. non-admin install)
+helm template mlrun charts/mlrun-ce \
+  -f charts/mlrun-ce/values.yaml \
+  -f charts/mlrun-ce/non_admin_installation_values.yaml
+```
+
+**Bump Chart version** (`/bump` in Claude Code)
+Read the current version from `charts/mlrun-ce/Chart.yaml` and increment:
+- `patch` — `0.11.0` → `0.11.1`
+- `minor` — `0.11.3` → `0.12.0`
+- `rc` — `0.11.0-rc.34` → `0.11.0-rc.35` (or append `-rc.1` if no suffix)
+
+Always show the old → new version before writing and confirm with the user.
+
+**Generate PR description** (`/pr` in Claude Code)
+Run `git diff upstream/development...HEAD`, `git log upstream/development..HEAD --oneline`, and `git diff upstream/development...HEAD --name-only`, then fill in `.github/pull_request_template.md` based on the changes. Check `[x]` on checklist items confirmable from the diff; leave `[ ]` on items requiring human action. Flag missing version bumps, unsynced values files, and breaking changes.
+
+## Do Not Suggest
+
+- `helm upgrade --install` without running `make helm-update-dependencies` first
+- Adding a new sub-chart to `requirements.yaml` for custom resources — add templates to `charts/mlrun-ce/templates/<component>/` instead
+- `kubectl apply` for resources managed by this chart
+- `Chart.yaml` apiVersion v2 dependency blocks (this chart uses apiVersion v1 + `requirements.yaml`)
+- Creating a second credentials Secret — mount the existing `storage-credentials` Secret via `envFrom`
+- Hardcoding namespace names in templates — use `{{ .Release.Namespace }}`
+- Using `kafka.enabled + strimzi-kafka-operator.enabled` as a combined condition — the template guard is only `kafka.enabled`; Strimzi is a prerequisite, not a co-guard
+- Treating `seaweedfs-s3-config` as a SeaweedFS dependency — SeaweedFS *creates* it; Pipelines and MLRun *consume* it
+
+## Workflow (from CONTRIBUTING.md)
+
+- Fork-based workflow: PRs target `upstream/development`, not `origin/development`
+- Branch naming: `<scope>/<short-description-or-ticket>` — e.g. `feature/add-redis-support`, `fix/CE-111`
+- Always bump `charts/mlrun-ce/Chart.yaml` version before opening a PR

.gitignore

@@ -8,3 +8,6 @@ charts/mlrun-ce/charts/*
 charts/mlrun-ce/mlrun-ce-*.tgz
 # MLRun project directories created by test scripts
 otlp-pro/
+
+# Claude Code local settings (machine-specific, not for commit)
+.claude/settings.local.json