From c34e87ec1c2935cb4f9fd79afb0211cb6756ed65 Mon Sep 17 00:00:00 2001
From: kina0630 <kiakiana0630@gmail.com>
Date: Mon, 15 Jun 2026 19:10:33 +0800
Subject: [PATCH] fix(helm): use venv python for kb index job

---
 coding-agent/helm/templates/cronjob-kb-index.yaml | 2 +-
 coding-agent/tests/deploy/test_helm_chart.py      | 4 +++-
 2 files changed, 4 insertions(+), 2 deletions(-)

diff --git a/coding-agent/helm/templates/cronjob-kb-index.yaml b/coding-agent/helm/templates/cronjob-kb-index.yaml
index b4a705fdb..8cd37546e 100644
--- a/coding-agent/helm/templates/cronjob-kb-index.yaml
+++ b/coding-agent/helm/templates/cronjob-kb-index.yaml
@@ -51,7 +51,7 @@ spec:
 
                   {{- range .Values.kb.index.repos }}
                   git clone --depth 1 --branch {{ $.Values.kb.index.gitRef | quote }} {{ required "kb.index.repos[].url is required" .url | quote }} "$workdir/{{ required "kb.index.repos[].name is required" .name }}"
-                  kb index "$workdir/{{ required "kb.index.repos[].name is required" .name }}" --corpus {{ required "kb.index.repos[].corpus is required" .corpus | quote }}
+                  /app/.venv/bin/python -m coding_agent kb index "$workdir/{{ required "kb.index.repos[].name is required" .name }}" --corpus {{ required "kb.index.repos[].corpus is required" .corpus | quote }}
                   {{- end }}
               {{- if or .Values.agent.secretRef .Values.kb.index.gitSecretName }}
               envFrom:
diff --git a/coding-agent/tests/deploy/test_helm_chart.py b/coding-agent/tests/deploy/test_helm_chart.py
index 73dfc9ff7..171e18a66 100644
--- a/coding-agent/tests/deploy/test_helm_chart.py
+++ b/coding-agent/tests/deploy/test_helm_chart.py
@@ -1,6 +1,7 @@
 from __future__ import annotations
 
 import os
+import re
 import shutil
 import subprocess
 import tomllib
@@ -677,7 +678,8 @@ def test_helm_kb_index_cronjob_renders_pvc_env_and_netpol_labels() -> None:
     script = "\n".join(container["command"])
     assert "git clone --depth 1 --branch \"main\"" in script
     assert "<forgejo-host>/<owner>/sre.git" in script
-    assert "kb index" in script
+    assert "/app/.venv/bin/python -m coding_agent kb index" in script
+    assert re.search(r"(^|\n)\s*kb\s+index(\s|$)", script) is None
     assert "--corpus \"sre\"" in script