Skip to content

Require explicit plaintext auth storage - #47

Merged
jlitola merged 1 commit into
mainfrom
golden-adasaurus
Apr 29, 2026
Merged

Require explicit plaintext auth storage#47
jlitola merged 1 commit into
mainfrom
golden-adasaurus

Conversation

@jlitola

@jlitola jlitola commented Apr 29, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Add config.toml auth storage mode with keychain default and explicit file mode for plaintext OAuth storage.
  • Stop silently downgrading keychain failures to plaintext files, while preserving legacy ~/.githits migration and logout cleanup.
  • Improve auth command dependency flow, storage error UX, atomic file writes, docs, and regression coverage.

Verification

  • bun test
  • bun run typecheck
  • bun run build
  • Built CLI smoke checks for help, auth status, file mode, env-token bypass, invalid config errors, and logout.

Default OAuth credential persistence now fails safely when the system keychain is unavailable, while explicit file mode supports headless environments with clear plaintext warnings.
@jlitola
jlitola merged commit b6e095c into main Apr 29, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant