Flatten internal directory structure and update build configuration

- Move all Go files from subdirectories to internal/ root
- Update all package declarations to use 'internal' package
- Remove internal import statements between components
- Update type references to work with flattened structure
- Remove empty subdirectories after file moves
- Update CLAUDE.md to reflect new structure
- Add .gitignore for bin/ and .presentation files
- Update GitHub workflow to test flattened internal package
- Maintain all functionality while simplifying code organization

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>

Author: pythonandchips

.github/workflows/build.yml

@@ -18,7 +18,7 @@ jobs:
         go-version: '1.21'
 
     - name: Run tests
-      run: go test ./internal/controller ./internal/presentation ./internal/presenter ./internal/slide
+      run: go test ./internal
 
     - name: Run go vet
       run: go vet ./...

.gitignore

@@ -0,0 +1,2 @@
+bin/
+.presentation

CLAUDE.md

@@ -11,15 +11,12 @@ go build -o bin/git-presenter ./cmd/git-presenter
 
 ### Running Tests
 ```bash
-go test ./internal/controller ./internal/presentation ./internal/presenter ./internal/slide
+go test ./internal
 ```
 
 ### Testing Single Package
 ```bash
-go test ./internal/controller  # Controller tests
-go test ./internal/presentation  # Presentation tests
-go test ./internal/presenter  # Presenter tests
-go test ./internal/slide  # Slide tests
+go test ./internal  # All tests in internal package
 ```
 
 ### Installation
@@ -36,26 +33,26 @@ Git Presenter follows clean architecture principles with clear separation of con
 - Creates GitPresenter instance and delegates to it
 - Supports both interactive and command modes (`-c` flag)
 
-**Core Components**:
+**Core Components** (all in `internal/` package):
 
-1. **GitPresenter** (`internal/presenter/presenter.go`)
+1. **GitPresenter** (`internal/presenter.go`)
    - Main orchestrator that coordinates all components
    - Handles command routing and interactive mode
    - Manages the presentation lifecycle
 
-2. **Controller** (`internal/controller/controller.go`)
+2. **Controller** (`internal/controller.go`)
    - Manages presentation initialization and configuration
    - Interfaces with git repository using go-git library
    - Creates `.presentation` YAML file with slide metadata
    - Handles git operations for discovering commits
 
-3. **Presentation** (`internal/presentation/presentation.go`)
+3. **Presentation** (`internal/presentation.go`)
    - Core presentation logic and state management
    - Handles slide navigation (next, back, start, end, list)
    - Manages current slide tracking and status display
    - Provides interactive command execution
 
-4. **Slide** (`internal/slide/slide.go`)
+4. **Slide** (`internal/slide.go`)
    - Represents individual slides (git commits)
    - Handles git checkout operations to move between commits
    - Stores commit hash and message metadata
@@ -86,4 +83,4 @@ The codebase follows Test-Driven Development (TDD) with comprehensive test cover
 
 ## Testing Strategy
 
-Tests are organized by package and focus on unit testing individual components. When adding new features, ensure corresponding tests are added to maintain coverage.
+All tests are in the `internal/` package with a flattened structure. Tests focus on unit testing individual components. When adding new features, ensure corresponding tests are added to maintain coverage.

bin/git-presenter

@@ -3,7 +3,7 @@ package main
 import (
 	"fmt"
 	"os"
-	"git-presenter/internal/presenter"
+	"git-presenter/internal"
 )
 
 func main() {
@@ -26,7 +26,7 @@ func main() {
 		os.Exit(1)
 	}
 
-	gitPresenter := presenter.NewGitPresenter(currentDir, interactive)
+	gitPresenter := internal.NewGitPresenter(currentDir, interactive)
 
 	err = gitPresenter.Execute(command)
 	if err != nil {

cmd/git-presenter/main.go

@@ -1,4 +1,4 @@
-package controller
+package internal
 
 import (
 	"fmt"

internal/controller/controller.go internal/controller.gointernal/controller/controller.go renamed to internal/controller.go

@@ -1,4 +1,4 @@
-package controller
+package internal
 
 import (
 	"testing"

internal/controller/controller_test.go internal/controller_test.gointernal/controller/controller_test.go renamed to internal/controller_test.go

@@ -1,27 +1,26 @@
-package presentation
+package internal
 
 import (
 	"fmt"
 	"os/exec"
 	"strings"
-	"git-presenter/internal/slide"
 )
 
 type Presentation struct {
 	branch       string
-	slides       []*slide.Slide
-	currentSlide *slide.Slide
+	slides       []*Slide
+	currentSlide *Slide
 }
 
 func NewPresentation(config map[string]interface{}) *Presentation {
 	branch, _ := config["branch"].(string)
 
-	var slides []*slide.Slide
+	var slides []*Slide
 	if slidesData, ok := config["slides"].([]interface{}); ok {
 		for _, slideData := range slidesData {
 			if slideMap, ok := slideData.(map[string]interface{}); ok {
 				if slideInfo, ok := slideMap["slide"].(map[string]interface{}); ok {
-					slides = append(slides, slide.NewSlide(slideInfo))
+					slides = append(slides, NewSlide(slideInfo))
 				}
 			}
 		}
@@ -38,7 +37,7 @@ func NewPresentation(config map[string]interface{}) *Presentation {
 	return presentation
 }
 
-func (p *Presentation) findCurrentSlide() *slide.Slide {
+func (p *Presentation) findCurrentSlide() *Slide {
 	cmd := exec.Command("git", "rev-parse", "HEAD")
 	output, err := cmd.Output()
 	if err != nil {

internal/presentation/presentation.go internal/presentation.gointernal/presentation/presentation.go renamed to internal/presentation.go

@@ -1,4 +1,4 @@
-package presentation
+package internal
 
 import (
 	"testing"

…ternal/presentation/presentation_test.go internal/presentation_test.gointernal/presentation/presentation_test.go renamed to internal/presentation_test.go internal/presentation/presentation_test.go renamed to internal/presentation_test.go

@@ -1,26 +1,24 @@
-package presenter
+package internal
 
 import (
 	"bufio"
 	"fmt"
 	"os"
 	"strings"
-	"git-presenter/internal/controller"
-	"git-presenter/internal/presentation"
 )
 
 type GitPresenter struct {
 	currentDir   string
 	interactive  bool
-	controller   *controller.Controller
-	presentation *presentation.Presentation
+	controller   *Controller
+	presentation *Presentation
 }
 
 func NewGitPresenter(currentDir string, interactive bool) *GitPresenter {
 	return &GitPresenter{
 		currentDir:  currentDir,
 		interactive: interactive,
-		controller:  controller.NewController(currentDir),
+		controller:  NewController(currentDir),
 	}
 }
 
@@ -54,7 +52,7 @@ func (g *GitPresenter) startPresentation() error {
 		}
 	}
 
-	g.presentation = presentation.NewPresentation(configMap)
+	g.presentation = NewPresentation(configMap)
 
 	if g.interactive {
 		g.enterRunLoop()
@@ -88,7 +86,7 @@ func (g *GitPresenter) executeNavigationCommand(command string) error {
 			}
 		}
 
-		g.presentation = presentation.NewPresentation(configMap)
+		g.presentation = NewPresentation(configMap)
 	}
 
 	result := g.presentation.Execute(command)