Use Literal type for expiry status instead of magic strings

[f3rdy]

Description

Status strings "expired", "warning", "ok" are used as bare string literals in keys.py and compared in cli.py. Typos would not be caught by mypy.

Proposed Solution

from typing import Literal

ExpiryStatus = Literal["expired", "warning", "ok"]

Use ExpiryStatus in ExpiryWarning.status and all comparisons.

Files

• src/vaultctl/keys.py (lines 100, 102, 104)
• src/vaultctl/cli.py (lines 337, 361)

Priority

P2

[f3rdy]

Resolved in master.