fix(analytics): re-trigger chart publish for the unified-metrics image

[aleksdotbar]

What

Re-triggers the umbrella chart publish so the release references the analytics image built for the unified-metrics merge (#1656).

Why

That merge built and pushed the insight-analytics image, but publish-chart was skipped on the merge run because the same merge also triggered a descriptor-bump commit (publish-chart guards on bump-descriptors.committed != 'true'). The follow-up run that published the umbrella saw only descriptor changes, so changes.analytics was false and the analytics appVersion bump was never applied — the release kept referencing the pre-merge image.

This is a general release-pipeline defect (any backend service + descriptor change in one merge); tracked separately. This PR is the immediate recovery.

How

A doc comment on the analytics subchart Chart.yaml — a change scoped to src/backend/services/analytics/** with no descriptor change. On merge to main, changes.analytics is true and no descriptor commit intervenes, so publish-chart runs: rebuilds the image, bumps the subchart appVersion to the fresh tag, and republishes the umbrella. appVersion is left for the pipeline to own.

🤖 Generated with Claude Code

Summary by CodeRabbit

• Documentation
  • Added a note clarifying that the app version is updated automatically during releases and reflects the latest built image version after service changes.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 2dbd5f0f-1eb5-48fd-b24e-1571b58a2bd1

📥 Commits

Reviewing files that changed from the base of the PR and between f32edbf and 0c9fb78.

📒 Files selected for processing (1)

• src/backend/services/analytics/helm/Chart.yaml

---

📝 Walkthrough

Walkthrough

Added two explanatory comment lines to the analytics service Helm Chart.yaml documenting that the appVersion field is managed by the release pipeline and automatically updated to match the built image tag on relevant merges.

Changes

Chart Documentation Update

Layer / File(s)	Summary
Add appVersion management comments src/backend/services/analytics/helm/Chart.yaml	Two comment lines added explaining that appVersion is managed by the release pipeline and auto-bumped to the freshly built image tag on merges.

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

Possibly related PRs

• constructorfabric/insight#1298: CI workflow changes rebuild/attest chart and backend artifacts, driving the appVersion updates described in the new comments.

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly matches the main change: a targeted analytics chart update to re-trigger chart publishing for the unified-metrics image.
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 unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch fix/analytics-chart-republish

---

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.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

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