Skip to content

chore: clean up#257

Merged
theantichris merged 6 commits intomainfrom
chore/clean-up
Feb 26, 2026
Merged

chore: clean up#257
theantichris merged 6 commits intomainfrom
chore/clean-up

Conversation

@theantichris
Copy link
Owner

No description provided.

@theantichris theantichris self-assigned this Feb 26, 2026
Copilot AI review requested due to automatic review settings February 26, 2026 22:44
@github-project-automation github-project-automation bot moved this to Backlog in theantichris Feb 26, 2026
@theantichris theantichris moved this from Backlog to In review in theantichris Feb 26, 2026
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR performs a comprehensive cleanup of the codebase, focusing on renaming packages for better clarity and removing unused files. The changes rename the theme package to style to better reflect its styling/rendering purpose, and rename the ui package to tui (Terminal UI) to be more specific about its terminal-based interface implementation. Additionally, the Tool interface is consolidated from a separate file into registry.go, and several unused documentation and test files are removed.

Changes:

  • Renamed theme package to style across 7 files and updated all import references
  • Renamed ui package to tui across 22 files and updated all import references
  • Consolidated Tool interface from internal/tool/tool.go into internal/tool/registry.go
  • Removed unused test scripts and documentation tape/gif files
  • Simplified CLAUDE.md documentation to reflect the new package structure
  • Inlined constants in cmd/root.go for better code organization

Reviewed changes

Copilot reviewed 41 out of 45 changed files in this pull request and generated no comments.

Show a summary per file
File Description
style/style.go Package declaration updated from theme to style
style/render.go Package declaration updated from theme to style
style/json.go Package declaration updated from theme to style
style/glyph.go Package declaration updated from theme to style
style/glamour.go Package declaration updated from theme to style
style/fang.go Package declaration updated from theme to style
style/color.go Package declaration updated from theme to style
main.go Import updated to use style package instead of theme
internal/tui/*.go All 22 files updated with package name from ui to tui
internal/tui/stream.go Import updated to use style package
internal/tui/chat_stream.go Import updated to use style package
internal/tui/chat_stream_test.go Import updated to use style package
internal/tui/chat_command.go Import updated to use style package
internal/tui/chat_command_test.go Import updated to use style package
internal/tui/chat.go Import updated to use style package
internal/tool/tool.go File deleted - interface moved to registry.go
internal/tool/registry.go Tool interface definition added from deleted tool.go
cmd/root.go Imports updated to use style and tui, constants inlined
cmd/chat.go Import updated to use tui package
test_features.sh Deleted unused test script
documentation/*.tape Deleted unused VHS tape files for GIF generation
documentation/*.gif Deleted generated GIF files
AGENTS.md Deleted duplicate documentation file
CLAUDE.md Simplified and updated to reflect new package names and structure

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@theantichris theantichris merged commit 8b8d69f into main Feb 26, 2026
9 checks passed
@theantichris theantichris deleted the chore/clean-up branch February 26, 2026 22:47
@github-project-automation github-project-automation bot moved this from In review to Done in theantichris Feb 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants