fix: support norwegian characters øæå

Author: csvenke

internal/projects/filter.go

@@ -3,6 +3,7 @@ package projects
 import (
 	"sort"
 	"strings"
+	"unicode"
 )
 
 func Filter(projects []Project, query string) []Project {
@@ -44,9 +45,6 @@ const (
 	scoreWordBoundary = 3
 )
 
-// FuzzyScore calculates a fuzzy match score between query and target.
-// Returns 0 if no match, higher scores indicate better matches.
-// Exported for testing.
 func FuzzyScore(query string, target string) int {
 	if len(query) == 0 {
 		return 1
@@ -55,34 +53,37 @@ func FuzzyScore(query string, target string) int {
 		return 0
 	}
 
+	queryRunes := []rune(query)
+	targetRunes := []rune(target)
+
 	score := 0
 	qi := 0
 	prevMatch := -2
 
-	for ti := 0; ti < len(target) && qi < len(query); ti++ {
-		if len(target)-ti < len(query)-qi {
+	for ti := 0; ti < len(targetRunes) && qi < len(queryRunes); ti++ {
+		if len(targetRunes)-ti < len(queryRunes)-qi {
 			return 0
 		}
 
-		if target[ti] == query[qi] {
+		if targetRunes[ti] == queryRunes[qi] {
 			score += scoreMatch
 			if ti == prevMatch+1 {
 				score += scoreConsecutive
 			}
-			if ti == 0 || !isLetter(target[ti-1]) {
+			if ti == 0 || !isLetter(targetRunes[ti-1]) {
 				score += scoreWordBoundary
 			}
 			prevMatch = ti
 			qi++
 		}
 	}
 
-	if qi == len(query) {
+	if qi == len(queryRunes) {
 		return score
 	}
 	return 0
 }
 
-func isLetter(b byte) bool {
-	return (b >= 'a' && b <= 'z') || (b >= 'A' && b <= 'Z')
+func isLetter(r rune) bool {
+	return unicode.IsLetter(r)
 }

internal/projects/projects_test.go

@@ -445,6 +445,111 @@ func TestFilter_FuzzyPrefersWordBoundaryMatches(t *testing.T) {
 	}
 }
 
+func TestFilter_MatchesNorwegianCharacters(t *testing.T) {
+	projects := []Project{
+		{Name: "øl-project", Path: "/repos/øl-project"},
+		{Name: "særen", Path: "/repos/særen"},
+		{Name: "håland", Path: "/repos/håland"},
+	}
+
+	result := Filter(projects, "øl")
+	if len(result) != 1 {
+		t.Errorf("expected 1 match for 'øl', got %d", len(result))
+	}
+	if result[0].Name != "øl-project" {
+		t.Errorf("expected 'øl-project', got %q", result[0].Name)
+	}
+
+	result = Filter(projects, "sæ")
+	if len(result) != 1 {
+		t.Errorf("expected 1 match for 'sæ', got %d", len(result))
+	}
+	if result[0].Name != "særen" {
+		t.Errorf("expected 'særen', got %q", result[0].Name)
+	}
+
+	result = Filter(projects, "hå")
+	if len(result) != 1 {
+		t.Errorf("expected 1 match for 'hå', got %d", len(result))
+	}
+	if result[0].Name != "håland" {
+		t.Errorf("expected 'håland', got %q", result[0].Name)
+	}
+}
+
+func TestFilter_NorwegianCaseInsensitive(t *testing.T) {
+	projects := []Project{
+		{Name: "ØL-PROJECT", Path: "/repos/ØL-PROJECT"},
+		{Name: "SÆREN", Path: "/repos/SÆREN"},
+		{Name: "HÅLAND", Path: "/repos/HÅLAND"},
+	}
+
+	tests := []struct {
+		query  string
+		expect string
+	}{
+		{"øl", "ØL-PROJECT"},
+		{"øL", "ØL-PROJECT"},
+		{"ØL", "ØL-PROJECT"},
+		{"sæ", "SÆREN"},
+		{"SÆ", "SÆREN"},
+		{"hå", "HÅLAND"},
+		{"HÅ", "HÅLAND"},
+	}
+
+	for _, tt := range tests {
+		result := Filter(projects, tt.query)
+		if len(result) != 1 {
+			t.Errorf("query %q: expected 1 match, got %d", tt.query, len(result))
+			continue
+		}
+		if result[0].Name != tt.expect {
+			t.Errorf("query %q: expected %q, got %q", tt.query, tt.expect, result[0].Name)
+		}
+	}
+}
+
+func TestFilter_NorwegianFuzzyMatching(t *testing.T) {
+	projects := []Project{
+		{Name: "første-prosjekt", Path: "/repos/første-prosjekt"},
+		{Name: "anden-prosjekt", Path: "/repos/anden-prosjekt"},
+	}
+
+	// "fp" should match "første-prosjekt"
+	result := Filter(projects, "fp")
+	if len(result) != 1 {
+		t.Errorf("expected 1 match for 'fp', got %d", len(result))
+	}
+	if result[0].Name != "første-prosjekt" {
+		t.Errorf("expected 'første-prosjekt', got %q", result[0].Name)
+	}
+
+	// "ap" should match "anden-prosjekt"
+	result = Filter(projects, "ap")
+	if len(result) != 1 {
+		t.Errorf("expected 1 match for 'ap', got %d", len(result))
+	}
+	if result[0].Name != "anden-prosjekt" {
+		t.Errorf("expected 'anden-prosjekt', got %q", result[0].Name)
+	}
+}
+
+func TestFilter_NorwegianWordBoundary(t *testing.T) {
+	projects := []Project{
+		{Name: "xølx", Path: "/repos/xølx"},
+		{Name: "øl-project", Path: "/repos/øl-project"},
+	}
+
+	// "øl" at start should rank higher
+	result := Filter(projects, "øl")
+	if len(result) != 2 {
+		t.Fatalf("expected 2 matches, got %d", len(result))
+	}
+	if result[0].Name != "øl-project" {
+		t.Errorf("expected 'øl-project' first (word boundary), got %q", result[0].Name)
+	}
+}
+
 func TestScore_ScoreIsNonNegative(t *testing.T) {
 	properties := gopter.NewProperties(nil)
 

internal/tui/model.go

@@ -12,7 +12,7 @@ import (
 )
 
 type Icons struct {
-	Dir string
+	Dir  string
 	Term string
 }
 
@@ -98,7 +98,8 @@ func (m Model) Update(msg tea.Msg) (tea.Model, tea.Cmd) {
 
 		case key.Matches(msg, m.keys.Backspace):
 			if len(m.query) > 0 {
-				m.query = m.query[:len(m.query)-1]
+				runes := []rune(m.query)
+				m.query = string(runes[:len(runes)-1])
 				m.filtered = projects.Filter(m.projects, m.query)
 				m.cursor = 0
 			}
@@ -113,8 +114,8 @@ func (m Model) Update(msg tea.Msg) (tea.Model, tea.Cmd) {
 			return m, nil
 
 		default:
-			if len(msg.String()) == 1 {
-				m.query += msg.String()
+			if msg.Type == tea.KeyRunes {
+				m.query += string(msg.Runes)
 				m.filtered = projects.Filter(m.projects, m.query)
 				m.cursor = 0
 			}