Handle multiple rooms in calendar event notes (#211)

- Updated `reRoom` regex to be non-greedy to correctly identify the first room in a list for the Location field.
- Updated `cleanEvent` to find all NavigaTUM room IDs in the location string and generate navigation links for each of them in the event description.
- Added `TestMultipleRooms` to verify the fix.

Fixes #158

Co-authored-by: google-labs-jules[bot] <161369871+google-labs-jules[bot]@users.noreply.github.com>

Author: google-labs-jules[bot]

internal/app.go

@@ -337,7 +337,7 @@ var unneeded = []string{
 	"(Online)",
 }
 
-var reRoom = regexp.MustCompile("^(.*?),.*(\\d{4})\\.(?:\\d\\d|EG|UG|DG|Z\\d|U\\d)\\.\\d+")
+var reRoom = regexp.MustCompile("^(.*?),.*?(\\d{4})\\.(?:\\d\\d|EG|UG|DG|Z\\d|U\\d)\\.\\d+")
 
 // matches strings like: (5612.03.017), (5612.EG.017), (5612.EG.010B)
 var reNavigaTUM = regexp.MustCompile("\\(\\d{4}\\.[a-zA-Z0-9]{2}\\.\\d{3}[A-Z]?\\)")
@@ -381,9 +381,11 @@ func (a *App) cleanEvent(event *ics.VEvent) {
 			description = location + "\n" + description
 			event.SetLocation(building)
 		}
-		if roomID := reNavigaTUM.FindString(location); roomID != "" {
-			roomID = strings.Trim(roomID, "()")
-			description = fmt.Sprintf("https://nav.tum.de/room/%s\n%s", roomID, description)
+		if roomIDs := reNavigaTUM.FindAllString(location, -1); len(roomIDs) > 0 {
+			for _, roomID := range roomIDs {
+				roomID = strings.Trim(roomID, "()")
+				description = fmt.Sprintf("https://nav.tum.de/room/%s\n%s", roomID, description)
+			}
 		}
 	}
 	event.SetDescription(description)

internal/app_test.go

@@ -71,6 +71,60 @@ func TestDeduplication(t *testing.T) {
 	}
 }
 
+func TestMultipleRooms(t *testing.T) {
+	// Setup app with building replacements
+	app, err := newApp()
+	if err != nil {
+		t.Fatal(err)
+	}
+
+	// Create a dummy event with multiple rooms
+	event := ics.NewEvent("test-uid")
+
+	// Using 5508 and 5612 which are present in buildings.json
+	// 5508 -> Boltzmannstr. 15, 85748 Garching b. München
+	// 5612 -> Boltzmannstr. 3, 85748 Garching b. München
+
+	// Construct a location string with multiple rooms.
+	// RoomA, DescA (5508.01.001), RoomB, DescB (5612.01.001)
+
+	location := "RoomA, DescA (5508.01.001), RoomB, DescB (5612.01.001)"
+	event.SetProperty(ics.ComponentPropertyLocation, location)
+	event.SetProperty(ics.ComponentPropertySummary, "Test Event")
+	event.SetProperty(ics.ComponentPropertyDescription, "Original Description")
+	event.SetProperty(ics.ComponentPropertyStatus, "CONFIRMED")
+
+	app.cleanEvent(event)
+
+	desc := event.GetProperty(ics.ComponentPropertyDescription).Value
+	loc := event.GetProperty(ics.ComponentPropertyLocation).Value
+
+	// Check if both rooms are present in description or nav links
+	if !strings.Contains(desc, "5508.01.001") {
+		t.Errorf("Description should contain first room ID")
+	}
+	if !strings.Contains(desc, "5612.01.001") {
+		t.Errorf("Description should contain second room ID")
+	}
+
+	// Check if nav links are generated for both
+	// 5508.01.001 -> https://nav.tum.de/room/5508.01.001
+	// 5612.01.001 -> https://nav.tum.de/room/5612.01.001
+
+	if !strings.Contains(desc, "https://nav.tum.de/room/5508.01.001") {
+		t.Error("Missing nav link for first room")
+	}
+	if !strings.Contains(desc, "https://nav.tum.de/room/5612.01.001") {
+		t.Error("Missing nav link for second room")
+	}
+
+	// With non-greedy regex, the location should be the first building (5508)
+	expectedLoc := "Boltzmannstr. 15, 85748 Garching b. München"
+	if loc != expectedLoc {
+		t.Errorf("Location should be %s but is %s", expectedLoc, loc)
+	}
+}
+
 func TestNameShortening(t *testing.T) {
 	testData, app := getTestData(t, "nameshortening.ics")
 	calendar, err := app.getCleanedCalendar([]byte(testData), map[string]bool{})