Skip to content

Removed web-based UI#44

Merged
wham merged 2 commits intomainfrom
copilot/remove-ui-command-and-assets
Dec 20, 2025
Merged

Removed web-based UI#44
wham merged 2 commits intomainfrom
copilot/remove-ui-command-and-assets

Conversation

Copy link
Contributor

Copilot AI commented Dec 20, 2025

The UI command provided a web-based search interface that duplicated MCP functionality. This removes the command, web server implementation, and associated static assets.

Changes

Code removal:

  • Removed RunUIServer() function and HTTP handlers
  • Removed ui command case from CLI switch
  • Removed embedded assets (indexHTML, htmxJS) and unused imports (embed, html, html/template)
  • Updated help text

Documentation:

  • Removed UI command sections from README.md and main.md spec
  • Removed UI screenshot reference

Assets:

  • Deleted docs/ui.png, index.html, htmx.min.js (~381KB total)

Preserved:

  • SearchEngine and SearchResult types remain - used by MCP search tool

Result

$ github-brain ui
Unknown command: ui
Use github-brain -h for help

$ github-brain -h
Commands:
  login  Authenticate with GitHub
  pull   Pull GitHub repositories and discussions
  mcp    Start the MCP server

Total: 621 lines removed, 3 files deleted.


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

- Remove UI command documentation from README.md
- Remove UI command section from main.md spec
- Remove UI command case from main.go
- Remove RunUIServer function from main.go
- Remove embedded HTML and JavaScript assets (index.html, htmx.min.js)
- Remove unused imports (embed, html, html/template)
- Remove docs/ui.png screenshot
- Update help text to exclude UI command

Co-authored-by: wham <448809+wham@users.noreply.github.com>
Copilot AI changed the title [WIP] Remove UI command and unused assets Remove UI command and web server implementation Dec 20, 2025
Copilot AI requested a review from wham December 20, 2025 01:46
@wham wham changed the title Remove UI command and web server implementation Removed web-based UI Dec 20, 2025
@wham wham added the minor label Dec 20, 2025
@wham wham marked this pull request as ready for review December 20, 2025 02:25
@wham wham merged commit 7cbb9b5 into main Dec 20, 2025
6 checks passed
@wham wham deleted the copilot/remove-ui-command-and-assets branch December 20, 2025 02:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants