feat: implement issue #442 — SonarCloud modernization (1/3): src/ app code

.gitignore

@@ -402,3 +402,4 @@ test-results/
 playwright-report/
 # compliance-ci-trigger
 # ci-trigger-298
+.dev-lead/

AGENTS.md

@@ -62,6 +62,17 @@ if (typeof module !== 'undefined' && module.exports) {
 
 ---
 
+## Pull request reviews
+
+- When addressing PR review comments, **always mark each resolved comment thread as Resolved** on GitHub after the fix is pushed. Use the GitHub GraphQL API via `gh api graphql` with the `resolveReviewThread` mutation:
+  ```bash
+  gh api graphql -f query='mutation { resolveReviewThread(input: {threadId: "PRRT_..."}) { thread { id isResolved } } }'
+  ```
+  Retrieve thread IDs first with a `reviewThreads` query on the pull request.
+- Resolve all addressed threads in one pass after pushing the fix commit, not one at a time during implementation.
+
+---
+
 ## Code style & commits
 
 - ALWAYS run `npx prettier --write .` before committing — pre-commit hooks do not run in agent sessions, so formatting must be applied manually.

src/calendar-to-briefing-doc/src/index.js

@@ -122,7 +122,7 @@ function formatConflictWarning(conflicts, formatTime) {
 function groupEventsByDay(events, getDateKey) {
   const map = new Map()
   for (const item of events) {
-    const evt = item && item.event ? item.event : item
+    const evt = item?.event ?? item
     const key = getDateKey(evt.getStartTime())
     if (!map.has(key)) map.set(key, [])
     map.get(key).push(item)
@@ -213,7 +213,7 @@ function formatBriefing(title, groupedEvents, formatTime) {
     sections.push(dayLabel)
 
     // Detect if items are tuples or plain events
-    const isTuple = items.length > 0 && items[0] && items[0].event
+    const isTuple = items.length > 0 && items[0]?.event
     const tuples = isTuple
       ? items
       : items.map((e) => ({ event: e, calendarName: null }))
@@ -266,7 +266,7 @@ function emailBriefing(gmailApp, recipients, subject, body) {
  */
 function shouldRunNow(config, now, lastRunMs) {
   const hour = now.getHours()
-  const scheduleHour = config.scheduleHour != null ? config.scheduleHour : 7
+  const scheduleHour = config.scheduleHour ?? 7
 
   if (hour !== scheduleHour) return false