change pipeline handling

internal/daemon/daemon.go

@@ -13,24 +13,29 @@ import (
 
 	"github.com/leonardotrapani/hyprvoice/internal/bus"
 	"github.com/leonardotrapani/hyprvoice/internal/notify"
+	"github.com/leonardotrapani/hyprvoice/internal/pipeline"
 )
 
-type Status string
+type Status = pipeline.Status
 
 const (
-	Idle         Status = "idle"
+	Idle         = pipeline.Idle
-	Recording    Status = "recording"
+	Recording    = pipeline.Recording
-	Transcribing Status = "transcribing"
+	Transcribing = pipeline.Transcribing
-	Injecting    Status = "injecting"
+	Injecting    = pipeline.Injecting
-	Completed    Status = "completed"
 )
 
 type Daemon struct {
-	mu       sync.Mutex
+	mu       sync.RWMutex
 	status   Status
 	notifier notify.Notifier
+
 	ctx    context.Context
 	cancel context.CancelFunc
+
+	pipeline       pipeline.Pipeline
+	pipelineCancel context.CancelFunc
+	statusCh       <-chan Status
 }
 
 func New(n notify.Notifier) *Daemon {
@@ -38,20 +43,57 @@ func New(n notify.Notifier) *Daemon {
 		n = notify.Desktop{}
 	}
 	ctx, cancel := context.WithCancel(context.Background())
-	return &Daemon{
+	d := &Daemon{
 		notifier: n,
 		ctx:      ctx,
 		cancel:   cancel,
 		status:   Idle,
 	}
+
+	return d
 }
 
 func (d *Daemon) Status() Status {
-	d.mu.Lock()
+	d.mu.RLock()
-	defer d.mu.Unlock()
+	defer d.mu.RUnlock()
 	return d.status
 }
 
+func (d *Daemon) startStatusReader(ctx context.Context, statusCh <-chan Status) {
+	go func() {
+		defer func() {
+			// Always clean up when this goroutine exits
+			d.mu.Lock()
+			d.status = Idle
+			d.statusCh = nil
+			d.pipeline = nil
+			d.pipelineCancel = nil
+			d.mu.Unlock()
+		}()
+
+		for {
+			select {
+			case status, ok := <-statusCh:
+				if !ok {
+					return // Channel closed
+				}
+
+				d.mu.Lock()
+				oldStatus := d.status
+				d.status = status
+				d.mu.Unlock()
+
+				if oldStatus != status {
+					log.Printf("Status changed: %s -> %s", oldStatus, status)
+				}
+
+			case <-ctx.Done():
+				return // Context cancelled
+			}
+		}
+	}()
+}
+
 func (d *Daemon) Run() error {
 	if err := bus.CheckExistingDaemon(); err != nil {
 		return err
@@ -80,36 +122,18 @@ func (d *Daemon) Run() error {
 
 	log.Printf("Daemon started, listening on socket")
 
-	// Accept connections in a goroutine
-	connCh := make(chan net.Conn)
-	errCh := make(chan error)
-
-	go func() {
 	for {
 		c, err := ln.Accept()
 		if err != nil {
-				errCh <- err
-				return
-			}
-			connCh <- c
-		}
-	}()
-
-	for {
-		select {
-		case <-d.ctx.Done():
-			log.Printf("Shutdown requested, exiting")
-			return nil
-		case c := <-connCh:
-			go d.handle(c)
-		case err := <-errCh:
-			// If context is cancelled, this is expected
 			if d.ctx.Err() != nil {
+				log.Printf("Shutdown requested")
 				return nil
 			}
 			log.Printf("Accept error: %v", err)
 			return fmt.Errorf("accept failed: %w", err)
 		}
+
+		go d.handle(c)
 	}
 }
 
@@ -129,35 +153,14 @@ func (d *Daemon) handle(c net.Conn) {
 	cmd := line[0]
 
 	switch cmd {
-	case 't': // toggle
+	case 't':
-		d.mu.Lock()
+		d.toggle()
-		defer d.mu.Unlock()
+	case 's':
-
+		status := d.Status()
-		switch d.status {
-		case Idle:
-			d.status = Recording
-
-			d.notifier.RecordingChanged(true)
-			log.Printf("Recording toggled: true")
-			fmt.Fprintf(c, "STATUS recording=%s\n", d.status)
-		default:
-			d.status = Idle
-
-			// TODO: trigger transcription
-
-			d.notifier.RecordingChanged(false)
-			log.Printf("Recording toggled: false")
-			fmt.Fprintf(c, "STATUS recording=%s\n", d.status)
-		}
-	case 's': // status
-		d.mu.Lock()
-		status := d.status
-		d.mu.Unlock()
-
 		fmt.Fprintf(c, "STATUS recording=%s\n", status)
-	case 'v': // protocol version
+	case 'v':
 		fmt.Fprintf(c, "STATUS proto=%s\n", bus.ProtoVer)
-	case 'q': // quit daemon
+	case 'q':
 		log.Printf("Shutdown requested")
 		fmt.Fprint(c, "OK quitting\n")
 		d.cancel()
@@ -166,3 +169,46 @@ func (d *Daemon) handle(c net.Conn) {
 		fmt.Fprintf(c, "ERR unknown=%q\n", cmd)
 	}
 }
+
+func (d *Daemon) toggle() {
+	d.mu.Lock()
+	defer d.mu.Unlock()
+
+	var notification func()
+
+	switch d.status {
+	case Idle:
+		ctx, cancel := context.WithCancel(d.ctx)
+		p := pipeline.New()
+		d.pipeline = p
+		d.pipelineCancel = cancel
+		d.statusCh = p.Run(ctx)
+		notification = d.notifier.RecordingStarted
+
+		// Start status reader for this pipeline
+		d.startStatusReader(ctx, d.statusCh)
+
+	case Recording:
+		if d.pipelineCancel != nil {
+			d.pipelineCancel() // Context cleanup handles the rest
+		}
+		notification = d.notifier.RecordingEnded
+
+	case Transcribing:
+		if d.pipeline != nil {
+			d.pipeline.Inject()
+		}
+		// No notification for injection start
+
+	case Injecting:
+		if d.pipelineCancel != nil {
+			d.pipelineCancel() // Context cleanup handles the rest
+		}
+		notification = d.notifier.RecordingEnded
+	}
+
+	// Send notification after releasing lock
+	if notification != nil {
+		go notification()
+	}
+}

internal/notify/notify.go

@@ -1,25 +1,35 @@
 package notify
 
 import (
-	"fmt"
 	"log"
 	"os/exec"
 )
 
 type Notifier interface {
-	RecordingChanged(on bool)
+	RecordingStarted()
+	RecordingEnded()
+	Transcribing()
 	Error(msg string)
 }
 
 type Desktop struct{}
 
-func (Desktop) RecordingChanged(on bool) {
+func (Desktop) RecordingStarted() {
-	state := "Stopped"
+	cmd := exec.Command("notify-send", "-a", "Hyprvoice", "Hyprvoice: Recording Started")
-	if on {
+	if err := cmd.Run(); err != nil {
-		state = "Started"
+		log.Printf("Failed to send notification: %v", err)
 	}
-	cmd := exec.Command("notify-send", "-a", "Hyprvoice",
+}
-		fmt.Sprintf("Hyprvoice: %s Recording", state))
+
+func (Desktop) RecordingEnded() {
+	cmd := exec.Command("notify-send", "-a", "Hyprvoice", "Hyprvoice: Recording Ended")
+	if err := cmd.Run(); err != nil {
+		log.Printf("Failed to send notification: %v", err)
+	}
+}
+
+func (Desktop) Transcribing() {
+	cmd := exec.Command("notify-send", "-a", "Hyprvoice", "Hyprvoice: Transcribing...")
 	if err := cmd.Run(); err != nil {
 		log.Printf("Failed to send notification: %v", err)
 	}
@@ -36,5 +46,7 @@ func (Desktop) Error(msg string) {
 // Useful in unit tests or headless builds.
 type Nop struct{}
 
-func (Nop) RecordingChanged(on bool) {}
+func (Nop) RecordingStarted() {}
+func (Nop) RecordingEnded()   {}
+func (Nop) Transcribing()     {}
 func (Nop) Error(msg string)  {}

internal/pipeline/pipeline.go

@@ -0,0 +1,88 @@
+package pipeline
+
+import (
+	"context"
+	"log"
+	"time"
+)
+
+type Status string
+
+const (
+	Idle         Status = "idle"
+	Recording    Status = "recording"
+	Transcribing Status = "transcribing"
+	Injecting    Status = "injecting"
+)
+
+type Pipeline interface {
+	Run(ctx context.Context) <-chan Status
+	Inject()
+}
+
+type pipeline struct {
+	injectCh chan struct{}
+}
+
+func New() Pipeline {
+	return &pipeline{
+		injectCh: make(chan struct{}, 1),
+	}
+}
+
+func (p *pipeline) Run(ctx context.Context) <-chan Status {
+	statusCh := make(chan Status, 1)
+	go p.run(ctx, statusCh)
+	return statusCh
+}
+
+
+func (p *pipeline) Inject() {
+	select {
+	case p.injectCh <- struct{}{}:
+	default:
+	}
+}
+
+func (p *pipeline) run(ctx context.Context, statusCh chan<- Status) {
+	defer close(statusCh)
+
+	// Start recording
+	log.Printf("Pipeline: Starting recording")
+	statusCh <- Recording
+
+	// Recording phase
+	select {
+	case <-time.After(2 * time.Second):
+		log.Printf("Pipeline: Recording complete, waiting for injection")
+		statusCh <- Transcribing
+	case <-ctx.Done():
+		log.Printf("Pipeline: Stopped during recording")
+		return
+	}
+
+	// Wait for injection or timeout
+	select {
+	case <-p.injectCh:
+		log.Printf("Pipeline: Injection started")
+		statusCh <- Injecting
+
+		// Injection work
+		select {
+		case <-time.After(1 * time.Second):
+			log.Printf("Pipeline: Injection complete")
+			statusCh <- Idle // Instead of Completed
+		case <-ctx.Done():
+			log.Printf("Pipeline: Stopped during injection")
+			return
+		}
+
+	case <-time.After(10 * time.Second): // use context timeout
+		log.Printf("Pipeline: Auto-timeout, completing")
+		statusCh <- Idle // Instead of Completed
+
+	case <-ctx.Done():
+		log.Printf("Pipeline: Stopped during transcription wait")
+		return
+	}
+}