fix: handle missing SUMMARY without crashing transform

Normalize missing event titles to an empty string before the view
transform runs, so event.title.search() no longer throws when an ICS
entry has no SUMMARY. Add a regression test for events without a title.

Ref #465.

Author: KristjanESPERANTO

node_helper.js

@@ -143,7 +143,7 @@ module.exports = NodeHelper.create({
       ev.calendarId = calendar.uid;
       ev.location = item.location;
       ev.description = item.description;
-      ev.title = item.summary;
+      ev.title = item.summary || "";
       ev.isRecurring = item.isRecurring;
       ev.attendees = item.attendees || [];
       ev.categories = item.categories || [];

test/node-helper-parser.test.js

@@ -54,6 +54,17 @@ const ICS_TWO_EVENTS = joinIcs(
   "END:VCALENDAR"
 );
 
+const ICS_EVENT_WITHOUT_SUMMARY = joinIcs(
+  "BEGIN:VCALENDAR",
+  "VERSION:2.0",
+  "BEGIN:VEVENT",
+  "UID:evt-no-summary",
+  `DTSTART:${dtDay}T140000Z`,
+  `DTEND:${dtDay}T150000Z`,
+  "END:VEVENT",
+  "END:VCALENDAR"
+);
+
 const makeCalendar = (overrides = {}) => ({
   uid: 0,
   name: "test-cal",
@@ -148,4 +159,14 @@ describe("node_helper – parser() event filtering", () => {
     const [, events] = sendSocketNotification.mock.calls[0].arguments;
     assert.equal(events.length, 1);
   });
+
+  it("normalizes missing event summaries to an empty title", () => {
+    const {ctx, sendSocketNotification} = makeCtx();
+
+    ctx.parser(makeCalendar(), ICS_EVENT_WITHOUT_SUMMARY);
+
+    const [, events] = sendSocketNotification.mock.calls[0].arguments;
+    assert.equal(events.length, 1);
+    assert.equal(events[0].title, "");
+  });
 });