SonarCloud: cognitive complexity (S3776)

[don-petry]

SonarCloud: cognitive complexity (S3776)

4 open SonarCloud finding(s) in petry-projects/google-app-scripts, worst severity CRITICAL. Generated by the org SonarCloud Audit.

View in SonarCloud →

Findings

Rule	Count	Representative message
javascript:S3776	4	Refactor this function to reduce its Cognitive Complexity from 19 to the 15 allowed.

Affected files

• src/gmail-to-drive-by-labels/src/index.js (2)
• deploy/index.html (1)
• src/calendar-to-sheets/src/index.js (1)

Priority

Labeled priority:critical — mapped from this bucket's worst SonarCloud severity (CRITICAL).

Acceptance

• All findings in this workstream resolved to zero in SonarCloud
• No behavior change; CI green
• Real fixes (no blanket NOSONAR unless a confirmed false positive, noted inline)

---

Idempotent issue — updated automatically each audit run; auto-closed when the finding count reaches zero.

[don-petry]

Dev-Lead Implementation Plan

Issue: #427 — SonarCloud: cognitive complexity (S3776)

Scope

Behavior-preserving refactors to reduce cognitive complexity below the 15 threshold for the 4 flagged functions, verified by the existing test suite (no behavior change).

Findings (from SonarCloud API)

File:line	Function	Complexity
src/calendar-to-sheets/src/index.js:109	syncCalendarToSheet	25
src/gmail-to-drive-by-labels/src/index.js:125	processMessageToDoc attachment loop	33
src/gmail-to-drive-by-labels/src/index.js:690	rebuildDoc	17
deploy/index.html:1846	handleDeploy	19

Implementation Checklist

• src/calendar-to-sheets/src/index.js: extract upsertRows, insertNewRows, computeRowsToDelete helpers out of syncCalendarToSheet
• src/gmail-to-drive-by-labels/src/index.js: extract attachment dedup/rename helpers out of the processMessageToDoc attachment loop
• src/gmail-to-drive-by-labels/src/index.js: extract state-machine phase helpers out of rebuildDoc
• deploy/index.html: extract resolveDeployProjectId + buildDeployErrorHtml helpers out of handleDeploy

Tests to Write

• Behavior-preserving refactor: no new behavior, so no new tests required (Phase 2 exception). The existing 301 Jest tests + Playwright deploy/tests/ui.spec.js validate behavior preservation. Any newly extracted helper stays covered via its existing callers to maintain the 99% line / 85% branch coverage thresholds.

Test Command

npm test (Jest, includes coverage thresholds via npm test -- --coverage)

[don-petry]

Dev-Lead: Implementation Complete

Behavior-preserving refactors that reduce cognitive complexity (S3776) below the 15 threshold for all 4 flagged functions. No behavior change — validated by the existing test suite (Phase 2 TDD exception applies to a pure refactor).

Plan Execution

• src/calendar-to-sheets/src/index.js — syncCalendarToSheet (25→~5): extracted upsertRows, insertNewRows, shouldDeleteRow, computeRowsToDelete
• src/gmail-to-drive-by-labels/src/index.js — processMessageToDoc attachment loop (33→~5): extracted isDuplicateAttachment, resolveAttachmentFileName
• src/gmail-to-drive-by-labels/src/index.js — rebuildDoc (17→~5): extracted clearDocPhase, moveEmailsPhase (null-sentinel preserves original early-return/fall-through control flow)
• deploy/index.html — handleDeploy (19→~8): extracted resolveDeployProjectId, buildDeployErrorHtml

Verification

• npm test — 301/301 passing
• npm test -- --coverage — exit 0, global thresholds met (branches 90.22 ≥ 85, functions 98.14 ≥ 95, lines 99.37 ≥ 99, statements 98.93 ≥ 95); extracted helpers at 100% function coverage
• npm run check (prettier --check + eslint) — exit 0 (only pre-existing unused-var warnings in untouched test files)
• bash .dev-lead/scripts/dev-lead-lint.sh — all checks passed
• npx prettier --write . applied
• node --check on both refactored index.js files — OK

Test Results

Test Suites: 12 passed, 12 total
Tests:       301 passed, 301 total
Snapshots:   0 total

----------------------------------|---------|----------|---------|---------|
File                              | % Stmts | % Branch | % Funcs | % Lines |
----------------------------------|---------|----------|---------|---------|
All files                         |   98.93 |    90.22 |   98.14 |   99.37 |
 src/calendar-to-sheets/src       |     100 |      100 |     100 |     100 |
  index.js                        |     100 |      100 |     100 |     100 |
 src/deploy                       |     100 |      100 |     100 |     100 |
 src/gmail-to-drive-by-labels/src |   98.53 |    90.56 |     100 |   99.09 |
  index.js                        |   98.53 |    90.56 |     100 |   99.09 |
----------------------------------|---------|----------|---------|---------|

Files Changed

• src/calendar-to-sheets/src/index.js: split syncCalendarToSheet into upsert/insert/delete helpers
• src/gmail-to-drive-by-labels/src/index.js: split the attachment loop and rebuildDoc state machine into helpers
• deploy/index.html: split handleDeploy into project-resolution and error-html helpers

Notes

• Pure refactor: no NOSONAR suppressions used. All original console.log/console.error calls and produced values/HTML are preserved verbatim, so runtime behavior is unchanged.
• deploy/index.html is not part of Jest coverage (covered by the Playwright deploy/tests/ui.spec.js e2e); its inline JS was validated via prettier parse + syntax review. Extracted functions live in the same <script> scope and are hoisted.
• The similar attachment loop in processLabelGroup was not flagged by SonarCloud and was left untouched to avoid scope creep.

[don-petry]

SonarCloud Audit (2026-07-14): still open — 4 finding(s), worst severity CRITICAL.