From 18b2b9419488a274163a7eab6ce022a6cdc2e2eb Mon Sep 17 00:00:00 2001 From: curo1305 Date: Sun, 17 May 2026 18:09:41 +0200 Subject: [PATCH] refactor(vault): centralize _KEYS_FILE constant in vault/__init__.py reader.py and writer.py each independently computed the same path via pyra_home(). Single definition in __init__.py; both modules import it. Co-Authored-By: Claude Sonnet 4.6 --- src/pyra/vault/__init__.py | 3 +++ src/pyra/vault/reader.py | 6 ++---- src/pyra/vault/writer.py | 6 ++---- 3 files changed, 7 insertions(+), 8 deletions(-) diff --git a/src/pyra/vault/__init__.py b/src/pyra/vault/__init__.py index e69de29..c89cb4c 100644 --- a/src/pyra/vault/__init__.py +++ b/src/pyra/vault/__init__.py @@ -0,0 +1,3 @@ +from pyra.utils.paths import pyra_home + +_KEYS_FILE = pyra_home() / "vault" / "secrets" / "api_keys.json" diff --git a/src/pyra/vault/reader.py b/src/pyra/vault/reader.py index 816734c..1da8399 100644 --- a/src/pyra/vault/reader.py +++ b/src/pyra/vault/reader.py @@ -1,9 +1,7 @@ import json -from pathlib import Path -from pyra.utils.paths import pyra_home, safe_chmod - -_KEYS_FILE = pyra_home() / "vault" / "secrets" / "api_keys.json" +from pyra.vault import _KEYS_FILE +from pyra.utils.paths import safe_chmod def get_key(provider_id: str) -> str | None: diff --git a/src/pyra/vault/writer.py b/src/pyra/vault/writer.py index 36681ac..58c3fc9 100644 --- a/src/pyra/vault/writer.py +++ b/src/pyra/vault/writer.py @@ -1,9 +1,7 @@ import json -from pathlib import Path -from pyra.utils.paths import ensure_dir, pyra_home, safe_chmod - -_KEYS_FILE = pyra_home() / "vault" / "secrets" / "api_keys.json" +from pyra.vault import _KEYS_FILE +from pyra.utils.paths import ensure_dir, safe_chmod def set_key(provider_id: str, api_key: str) -> None: