Skip to content

fix(ui): Consolidate GitHub settings into General + add gh CLI tool section#326

Merged
eyelock merged 1 commit into
developfrom
fix/github-settings-move
May 14, 2026
Merged

fix(ui): Consolidate GitHub settings into General + add gh CLI tool section#326
eyelock merged 1 commit into
developfrom
fix/github-settings-move

Conversation

@eyelock
Copy link
Copy Markdown
Owner

@eyelock eyelock commented May 14, 2026

Summary

The GitHub settings tab held a single setting (Remote PR feed cap), so this consolidates it into the General → Git section directly under Protected Branches and removes the dedicated tab. Reduces visual clutter in Settings.

Also adds a gh entry to the Tools tab — both a Status row and a full tool section mirroring tmux — so users can see at a glance whether the GitHub CLI is installed and authenticated. Remote PR features depend on it, so it belongs alongside the other first-class tool integrations.

Changes

  • Remove gitHub SettingsTab case and delete SettingsGitHubView.swift
  • Move PR-feed-cap Stepper into SettingsGeneralView Git section
  • Add gh status row to Tools status section + new gh detail section (version, path, auth state, brew install hint, re-check)
  • GhCliProbe gains a version property parsed from gh --version
  • Extract Settings.gh.* into Strings+Gh.swift to keep the outer enum under SwiftLint's type-body-length limit
  • Drop obsolete settings.tab.github / settings.github.section.feed keys from all 40 locales; add 13 new settings.gh.* keys (kept verbatim in English per the existing tool-settings convention)

Testing

  • make check passes
  • Localization validated (all 40 locales, 1026 keys, zero NEEDS TRANSLATION markers)
  • Manual settings UI walkthrough

🤖 Generated with Claude Code

…ection

The GitHub settings tab held a single setting (Remote PR feed cap), so
move it under General → Git, directly below Protected Branches, and drop
the dedicated tab to reduce visual clutter.

Add a `gh` entry to the Tools tab — both a Status row and a full tool
section mirroring tmux — so users can see at a glance whether the GitHub
CLI is installed and authenticated. Remote PR features depend on it, so
it belongs alongside the other first-class tool integrations.

- GhCliProbe gains a `version` property parsed from `gh --version`
- Settings.gh.* strings extracted into Strings+Gh.swift to keep the
  outer enum under SwiftLint's type-body-length limit

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
@eyelock eyelock merged commit 4955e84 into develop May 14, 2026
7 checks passed
@eyelock eyelock deleted the fix/github-settings-move branch May 14, 2026 00:07
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