test: cover dotenv parse, remove, and append helpers

[cursor]

Summary

Adds Vitest coverage for src/lib/dotenv-parse.ts, which backs dotenv handling in the secrets router and file workspace.

Risky behavior now covered

• Parsing: blank lines, # comments, CRLF vs LF, malformed lines (=bad, no =), quoted values, trimming around =.
• removeDotenvKey: removes all lines that parseDotenv would treat as the target key while preserving comments and structure; multiple definitions of the same key.
• appendDotenvKey: trailing-whitespace normalization, duplicate key detection (including keys that match after trim), validation errors (empty key, = in key, # prefix, newlines in key/value), and quoting/escaping for values that need it.

Test files added/updated

• Added: src/lib/dotenv-parse.test.ts

Why this reduces regression risk

The recent work on dotenv parsing, key removal, append UX, and error message formatting all flow through this module. These tests pin observable behavior for parsing and mutation of .env-style content without touching production code, so future edits are less likely to break secrets display, GitHub Action env picking, or vault file editing silently.

Validation

• npm run test (all 30 tests pass; deterministic, node environment).

  

[devactivity-app]

Pull Request Summary by devActivity

Metrics

Achievements

• @vitaliimelnychuk
  Earned XP: 15⭐
  Check Your Progress