Add file-based fallback for token storage when dbus unavailable

[Copilot]

Refactored keyring implementation per feedback

• Extract keyring logic to internal/keyring/keyring.go
• Create internal/keyring/keyring_linux.go with file-based storage
• Create internal/keyring/keyring_darwin.go with system keyring
• Create internal/keyring/keyring_windows.go with system keyring
• Update internal/config/config.go to use new keyring package
• Add tests for keyring package
• Remove old fallback implementation
• All tests pass

Implementation Details

New Package Structure

• internal/keyring/keyring.go: Defines Provider interface and package-level Set/Get functions
• internal/keyring/keyring_linux.go: Linux-specific implementation using file storage only (no dbus dependency)
• internal/keyring/keyring_darwin.go: macOS implementation using system keyring
• internal/keyring/keyring_windows.go: Windows implementation using system keyring

Linux File Storage

On Linux, tokens are stored in ~/.config/jira-cli/token as JSON:

{
  "host1.atlassian.net": "token1",
  "host2.atlassian.net": "token2"
}

• File permissions: 0600 (owner read/write only)
• Directory permissions: 0700

Platform Selection

Go build tags automatically select the correct implementation:

• Linux: Uses file-based storage
• macOS/Windows: Uses system keyring via go-keyring

Testing

All tests pass (11 total):

• 7 existing tests (unchanged)
• 4 new keyring tests

Original prompt

The key-chain we use does not work on linux if you do not have dbus installed. When that happens, lets fall back to use a file.

---

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

[alexec]

@copilot Nope. Extract the key-ring logic to a new file (internal/keyring/keyring.go) and internal/keyring/keyring_linux.go which just uses files for the moment.

[Copilot]

@copilot Nope. Extract the key-ring logic to a new file (internal/keyring/keyring.go) and internal/keyring/keyring_linux.go which just uses files for the moment.

Done. Refactored to use platform-specific implementations:

• internal/keyring/keyring.go - interface and common code
• internal/keyring/keyring_linux.go - file-based storage (no dbus)
• internal/keyring/keyring_darwin.go - system keyring for macOS
• internal/keyring/keyring_windows.go - system keyring for Windows

Commit: 6f2ced0