From cea34dc7ef61b16b9d65ccc788b78dac914bc9b0 Mon Sep 17 00:00:00 2001 From: Teknium Date: Mon, 13 Apr 2026 04:35:13 -0700 Subject: [PATCH] fix: follow-up for salvaged PR #8939 - Move test file to tests/hermes_cli/ (consistent with test layout) - Remove unused imports (os, pytest) from test file - Update _sanitize_env_lines docstring: now used on read + write paths --- hermes_cli/config.py | 2 +- tests/{ => hermes_cli}/test_env_sanitize_on_load.py | 3 --- 2 files changed, 1 insertion(+), 4 deletions(-) rename tests/{ => hermes_cli}/test_env_sanitize_on_load.py (99%) diff --git a/hermes_cli/config.py b/hermes_cli/config.py index 487708388..f30e663e5 100644 --- a/hermes_cli/config.py +++ b/hermes_cli/config.py @@ -2413,7 +2413,7 @@ def load_env() -> Dict[str, str]: def _sanitize_env_lines(lines: list) -> list: - """Fix corrupted .env lines before writing. + """Fix corrupted .env lines before reading or writing. Handles two known corruption patterns: 1. Concatenated KEY=VALUE pairs on a single line (missing newline between diff --git a/tests/test_env_sanitize_on_load.py b/tests/hermes_cli/test_env_sanitize_on_load.py similarity index 99% rename from tests/test_env_sanitize_on_load.py rename to tests/hermes_cli/test_env_sanitize_on_load.py index c29f5ecc5..6ac7c2cef 100644 --- a/tests/test_env_sanitize_on_load.py +++ b/tests/hermes_cli/test_env_sanitize_on_load.py @@ -1,12 +1,9 @@ """Tests for .env sanitization during load to prevent token duplication (#8908).""" -import os import tempfile from pathlib import Path from unittest.mock import patch -import pytest - def test_load_env_sanitizes_concatenated_lines(): """Verify load_env() splits concatenated KEY=VALUE pairs.