test(e2e): the fixture's shared-metrics answer carries offer_version, so interactive suites stop on no prompt

Since #133571 (b5025017d5, teknium1, 2026-10-05) a shared-metrics "No thanks" without
`offer_version` reads as `reask` and the interactive CLI/TUI offers the prompt once more. The
fake-provider home every e2e suite builds (`write_hermes_home`) wrote exactly that shape
(`enabled: false` only), so every PTY/tmux scenario now opens on the consent picker, never
reaches its ready status bar and times out: 61 failures in the e2e lane, the Windows unicode
CLI test, and the e2e-upgrade interactive-CLI entrypoint on main since the merge.

The fixture now records a complete current answer (`send: false`, `offer_version` imported
from the consent module so the next version bump cannot strand it again).
This commit is contained in:
kshitijk4poor authored and Teknium committed 2026-10-06 03:29:28 -07:00
1 parent 32859234d4
commit 22fcbee492
1 file changed
+6 -1
+6 -1
View File
@@ -32,6 +32,8 @@ from http.server import BaseHTTPRequestHandler, ThreadingHTTPServer
from pathlib import Path
from typing import Any, Callable, Union
from hermes_cli.observability.shared_metrics_consent import OFFER_VERSION
MODEL_ID = "fake-model"
@@ -430,10 +432,13 @@ def write_hermes_home(
" context_length: 128000\n"
"agent:\n"
" api_max_retries: 1\n"
# Answered, so an interactive chat never stops on the one-time shared-metrics offer.
# Answered at the current offer version, so an interactive chat never stops on the
# shared-metrics offer (a "No thanks" without offer_version is re-asked).
"telemetry:\n"
" shared_metrics:\n"
" enabled: false\n"
" send: false\n"
f" offer_version: {OFFER_VERSION}\n"
+ extra_config,
encoding="utf-8",
)