Skip to content

Remove remote section and all remote account management#3

Open
kitanoyoru wants to merge 1 commit into
masterfrom
remove-remote-section
Open

Remove remote section and all remote account management#3
kitanoyoru wants to merge 1 commit into
masterfrom
remove-remote-section

Conversation

@kitanoyoru

Copy link
Copy Markdown
Owner

Summary

Removes the Remote section and all related remote account management from kssh, keeping the app focused on its core purpose: SSH keys, Git profiles, and GPG keys.

Changes

Deleted files (10)

  • Models: RemoteAccount.swift, RemoteUser.swift, RemoteProfileDetail.swift, ContributionGraph.swift
  • Services: GitHubService.swift, GitLabService.swift, BitbucketService.swift, RemoteKeyError.swift
  • Utils: NetrcReader.swift, KeychainManager.swift (only used by remote)

Modified files (5)

  • StatusViewModel.swift — removed ~200 lines of remote state, resolution, and CRUD
  • MenuBarView.swift — removed ~500 lines of remote UI (section, detail screens, provider badges, contribution graph)
  • SettingsStore.swift — removed ~250 lines of account CRUD, persistence, and migration
  • SettingsView.swift — removed Accounts tab
  • ksshTests.swift — removed 5 remote test suites (~180 lines)

Result

  • Builds cleanly
  • All 56 remaining tests pass
  • -2,690 lines deleted, +4 lines changed

- Remove Remote model types (RemoteAccount, RemoteUser, RemoteProfileDetail, ContributionGraph)
- Remove remote HTTP services (GitHub, GitLab, Bitbucket services + RemoteKeyError)
- Remove NetrcReader and KeychainManager utilities (only used by remote)
- Strip all remote state, resolution, and CRUD from StatusViewModel
- Remove remote section UI, screens, and provider badges from MenuBarView
- Strip account persistence and migration from SettingsStore
- Remove Accounts tab from SettingsView
- Remove remote-related tests (RemoteAccount, ContributionGraph, RemoteUser, KeyNormalization, RemoteAddKeyRequest, NetrcReader)
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