From 276be58fbe4f9398a63d399e8f3d8f6f363cd532 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Nicol=C3=B2=20Boschi?= Date: Fri, 5 Jun 2026 10:26:12 +0200 Subject: [PATCH] ci(obs): enable consolidation dedup (0.97) in the observation benchmark The obs benchmark exists to track consolidation's near-duplicate observation rate, but the job ran with dedup disabled (default threshold 1.0), so it measured the baseline rather than the shipped feature. Enable it at 0.97 in the job env. Safe here because the benchmark runs on embedded pg0 (Postgres); the merge path is Postgres-only, which is why the feature stays opt-in globally. --- .github/workflows/perf-test.yml | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/.github/workflows/perf-test.yml b/.github/workflows/perf-test.yml index 2987d053e..a4f4dec4d 100644 --- a/.github/workflows/perf-test.yml +++ b/.github/workflows/perf-test.yml @@ -223,6 +223,10 @@ jobs: HINDSIGHT_API_LLM_VERTEXAI_SERVICE_ACCOUNT_KEY: /tmp/gcp-credentials.json HINDSIGHT_API_LLM_MODEL: google/gemini-2.5-flash-lite HINDSIGHT_API_ENABLE_OBSERVATIONS: "true" + # Exercise create+update semantic dedup (the benchmark's whole point). Safe here because + # the benchmark runs on embedded pg0 (Postgres); the merge path is Postgres-only, which is + # why the feature stays opt-in / disabled by default rather than enabled globally. + HINDSIGHT_API_CONSOLIDATION_DEDUP_THRESHOLD: "0.97" steps: - uses: actions/checkout@v6 with: