astronomer / astro-cli / 38187824831

61%

main: 45%

Build Type

push

github

Committed by web-flow

Commit Message

Drop the redundant "(the encrypted vault)" from local env messages (#2298)

* Drop the redundant "(the encrypted vault)" from local env messages

A vault set printed "set connection x in vault (the encrypted vault)".
Encrypted is the default, so the parenthetical only repeated the scope.
The vault writer's Location is now empty when encrypted and "unencrypted"
for --plain, and messages add the parenthetical only when there is one,
so a .env store still names its path.

Co-Authored-By: Claude <noreply@anthropic.com>

* Keep a plain global set to one parenthetical

A --plain global set read "vault (global) (unencrypted)". A scope that
already ends in a parenthetical now takes the location inside it:
"vault (global, unencrypted)".

Co-Authored-By: Claude <noreply@anthropic.com>

* Name a value's store by what was written, not by --plain

Review follow-ups:

- A --plain delete of an encrypted global said "vault (global,
  unencrypted)": the mark followed the flag, which a delete carries too.
  The writer now marks itself unencrypted only after it stores a value
  plain.
- storeWhere spliced the qualifier in by checking for a trailing ")".
  Each store now names itself with Where(), so nothing parses a label.
- Location() meant a path for .env and a qualifier for the vault. It is
  gone; the one message that wanted the plaintext path uses DotenvPath.
- Tests cover the printed text for each store and the plain mark.

Co-Authored-By: Claude <noreply@anthropic.com>

---------

Co-authored-by: Claude <noreply@anthropic.com>

Coverage Stats

19 of 26 new or added lines in 2 files covered. (73.08%)

4 existing lines in 2 files now uncovered.

40071 of 69464 relevant lines covered (57.69%)

232.68 hits per line