* to activate the hourly trigger.
*/
function buildSetupSource(triggerFn) {
- if (!/^[a-zA-Z_][a-zA-Z0-9_]*$/.test(triggerFn)) {
+ if (!/^[a-zA-Z_]\w*$/.test(triggerFn)) {
throw new Error(
'Invalid trigger function name: ' + JSON.stringify(triggerFn)
)
@@ -1189,14 +1188,11 @@
3Configure your scripts
'' +
'
diff --git a/gas-installer/src/index.js b/gas-installer/src/index.js
index b3bfa6e7..cb73276c 100644
--- a/gas-installer/src/index.js
+++ b/gas-installer/src/index.js
@@ -21,8 +21,8 @@ const APPS_SCRIPT_API_BASE = 'https://script.googleapis.com/v1'
*/
function getFileType(filename) {
if (!filename || typeof filename !== 'string') return null
- if (/\.gs$/.test(filename)) return 'SERVER_JS'
- if (/\.html$/.test(filename)) return 'HTML'
+ if (filename.endsWith('.gs')) return 'SERVER_JS'
+ if (filename.endsWith('.html')) return 'HTML'
return null
}
@@ -42,8 +42,7 @@ function filterGithubItems(items) {
return items
.filter(
(item) =>
- item &&
- item.type === 'file' &&
+ item?.type === 'file' &&
typeof item.name === 'string' &&
item.download_url
)
diff --git a/scripts/check-coverage.js b/scripts/check-coverage.js
index 87025646..f7e701b5 100644
--- a/scripts/check-coverage.js
+++ b/scripts/check-coverage.js
@@ -7,8 +7,8 @@
* its minimum threshold, or 0 if all pass.
*/
-const fs = require('fs')
-const path = require('path')
+const fs = require('node:fs')
+const path = require('node:path')
const coveragePath = path.join(process.cwd(), 'coverage', 'coverage-final.json')
@@ -44,9 +44,9 @@ for (const filePath in coverage) {
if (fileCoverage.b) {
for (const key in fileCoverage.b) {
const branches = fileCoverage.b[key]
- for (let i = 0; i < branches.length; i++) {
+ for (const hitCount of branches) {
totals.branches.total++
- if (branches[i] > 0) totals.branches.covered++
+ if (hitCount > 0) totals.branches.covered++
}
}
}
diff --git a/src/calendar-to-briefing-doc/src/index.js b/src/calendar-to-briefing-doc/src/index.js
index a5fcd8d6..517962b8 100644
--- a/src/calendar-to-briefing-doc/src/index.js
+++ b/src/calendar-to-briefing-doc/src/index.js
@@ -95,7 +95,7 @@ function detectConflicts(eventTuples) {
* @returns {string}
*/
function formatConflictWarning(conflicts, formatTime) {
- if (!conflicts || !conflicts.length) return ''
+ if (!conflicts?.length) return ''
return conflicts
.map((c) => {
const aLabel = c.a.calendarName
@@ -249,7 +249,7 @@ function formatBriefing(title, groupedEvents, formatTime) {
* @param {string} body - The full briefing text
*/
function emailBriefing(gmailApp, recipients, subject, body) {
- if (!recipients || !recipients.length) return
+ if (!recipients?.length) return
for (const recipient of recipients) {
gmailApp.sendEmail(recipient, subject, body)
}
@@ -312,7 +312,7 @@ function generateBriefingForConfig(
getDateKey,
formatTime
) {
- if (!config.emailRecipients || !config.emailRecipients.length) {
+ if (!config.emailRecipients?.length) {
throw new Error('emailRecipients is required')
}
diff --git a/src/calendar-to-briefing-doc/tests/index.test.js b/src/calendar-to-briefing-doc/tests/index.test.js
index 50dcc0cc..81e8f48e 100644
--- a/src/calendar-to-briefing-doc/tests/index.test.js
+++ b/src/calendar-to-briefing-doc/tests/index.test.js
@@ -408,37 +408,22 @@ describe('emailBriefing', () => {
// ── shouldRunNow ─────────────────────────────────────────────────────────────
describe('shouldRunNow', () => {
- it('returns true on the configured weekly day and hour', () => {
+ it.each([
+ [
+ 'configured weekly day and hour (Monday 7 AM)',
+ '2025-01-13T07:00:00',
+ true,
+ ],
+ ['wrong day (Tuesday 7 AM)', '2025-01-14T07:00:00', false],
+ ['wrong hour (Monday 8 AM)', '2025-01-13T08:00:00', false],
+ ])('returns %s -> %s', (_desc, nowIso, expected) => {
const config = {
scheduleFrequency: 'weekly',
scheduleDay: 'MONDAY',
scheduleHour: 7,
}
- // Monday at 7 AM
- const now = new Date('2025-01-13T07:00:00')
- expect(shouldRunNow(config, now, null)).toBe(true)
- })
-
- it('returns false on wrong day', () => {
- const config = {
- scheduleFrequency: 'weekly',
- scheduleDay: 'MONDAY',
- scheduleHour: 7,
- }
- // Tuesday at 7 AM
- const now = new Date('2025-01-14T07:00:00')
- expect(shouldRunNow(config, now, null)).toBe(false)
- })
-
- it('returns false on wrong hour', () => {
- const config = {
- scheduleFrequency: 'weekly',
- scheduleDay: 'MONDAY',
- scheduleHour: 7,
- }
- // Monday at 8 AM
- const now = new Date('2025-01-13T08:00:00')
- expect(shouldRunNow(config, now, null)).toBe(false)
+ const now = new Date(nowIso)
+ expect(shouldRunNow(config, now, null)).toBe(expected)
})
it('defaults to weekly Monday 7 AM when schedule fields are missing', () => {
diff --git a/src/calendar-to-sheets/src/index.js b/src/calendar-to-sheets/src/index.js
index 68f6f2ca..24b32fd7 100644
--- a/src/calendar-to-sheets/src/index.js
+++ b/src/calendar-to-sheets/src/index.js
@@ -32,7 +32,7 @@ function rowsToMap(rows) {
const m = new Map()
for (let i = 0; i < rows.length; i++) {
const r = rows[i]
- if (!r || !r[0]) continue
+ if (!r?.[0]) continue
m.set(r[0], { rowIndex: i + 2, values: r }) // assume header at row 1
}
return m
@@ -51,7 +51,8 @@ function rowsEqual(a, b) {
// Handle Date comparison (a is ISO string, b is Date object from sheet)
if (typeof valA === 'string' && valB instanceof Date) {
const dateA = new Date(valA)
- if (!isNaN(dateA) && dateA.getTime() === valB.getTime()) continue
+ if (!Number.isNaN(dateA.getTime()) && dateA.getTime() === valB.getTime())
+ continue
}
// Handle sanitized formula comparison (a has leading ', b does not)
@@ -260,7 +261,10 @@ async function syncCalendarToSheet(
const toDelete = computeDeletableRows(existingMap, desiredMap, start, end)
// delete from bottom to top
console.log('[syncCalendarToSheet] Deleting rows:', toDelete.length)
- toDelete.sort((a, b) => b - a).forEach((r) => sheet.deleteRow(r))
+ toDelete
+ .slice()
+ .sort((a, b) => b - a)
+ .forEach((r) => sheet.deleteRow(r))
console.log('[syncCalendarToSheet] Sync complete')
}
diff --git a/src/deploy/index.js b/src/deploy/index.js
index a056fab0..4d3217c3 100644
--- a/src/deploy/index.js
+++ b/src/deploy/index.js
@@ -100,7 +100,7 @@ function buildProjectContent(files) {
throw new Error('Each file must have a non-empty string name')
}
if (typeof f.source !== 'string') {
- throw new Error('Each file must have a string source')
+ throw new TypeError('Each file must have a string source')
}
}
return {
diff --git a/src/gas-utils.js b/src/gas-utils.js
index 142eccb0..1129223c 100644
--- a/src/gas-utils.js
+++ b/src/gas-utils.js
@@ -10,7 +10,6 @@ function getCleanBody(text) {
/^\s*On\s.+\swrote:/m,
/^\s*From:\s.+\sSent:\s+/m,
/^\s*_{10,}/m,
- /^\s*From:\s+.+<.+@.+>/m,
/confidentiality notice/im,
]
@@ -19,10 +18,8 @@ function getCleanBody(text) {
const match = text.match(pattern)
if (match) {
// Prefer splitting at the start of the line containing the match
- const lineStart =
- text.lastIndexOf('\n', match.index) === -1
- ? 0
- : text.lastIndexOf('\n', match.index) + 1
+ const nlIndex = text.lastIndexOf('\n', match.index)
+ const lineStart = nlIndex === -1 ? 0 : nlIndex + 1
if (splitIndex === -1 || lineStart < splitIndex) {
splitIndex = lineStart
}
@@ -53,7 +50,7 @@ function getFileHash(blob) {
bytes = blob
} else if (blob && typeof blob.getBytes === 'function') {
bytes = Buffer.from(blob.getBytes())
- } else if (blob && blob.bytes) {
+ } else if (blob?.bytes) {
bytes = Buffer.from(blob.bytes)
} else {
throw new TypeError('Unsupported blob type')
diff --git a/src/gmail-to-drive-by-labels/src/index.js b/src/gmail-to-drive-by-labels/src/index.js
index c757739f..24941721 100644
--- a/src/gmail-to-drive-by-labels/src/index.js
+++ b/src/gmail-to-drive-by-labels/src/index.js
@@ -231,7 +231,7 @@ function processMessageToDoc(message, body, folder, options = {}) {
let currentIndex = 0
// Insert subject
- const subjectText = 'Subject: ' + (subject ? subject : '(No Subject)')
+ const subjectText = 'Subject: ' + (subject || '(No Subject)')
const headingPara = body.insertParagraph(currentIndex++, subjectText)
// Try to set heading style (GAS only)
@@ -535,7 +535,7 @@ function processLabelGroup(config, services, helperFns) {
// This ensures the most recent emails appear first.
let currentIndex = 0
- const subjectText = 'Subject: ' + (subject ? subject : '(No Subject)')
+ const subjectText = 'Subject: ' + (subject || '(No Subject)')
const headingPara = body.insertParagraph(currentIndex++, subjectText)
// Try to set heading, fallback to bold if Doc is busy
@@ -634,7 +634,7 @@ function processLabelGroup(config, services, helperFns) {
'_HHmmss'
)
// Insert timestamp before the file extension
- let newName = fileName.replace(/(\.[\w\d_-]+)$/i, timeTag + '$1')
+ let newName = fileName.replace(/(\.[\w-]+)$/i, timeTag + '$1')
// Fallback if regex fails (files without extension)
if (newName === fileName) newName += timeTag
diff --git a/src/gmail-to-drive-by-labels/tests/gas-utils.test.js b/src/gmail-to-drive-by-labels/tests/gas-utils.test.js
index 23245f57..64137f63 100644
--- a/src/gmail-to-drive-by-labels/tests/gas-utils.test.js
+++ b/src/gmail-to-drive-by-labels/tests/gas-utils.test.js
@@ -17,6 +17,20 @@ describe('getCleanBody', () => {
expect(getCleanBody(input)).toBe('Line1')
})
+ test('cuts off at Outlook-style From:/Sent: reply header', () => {
+ const input =
+ 'My reply\nFrom: Jane Doe\nSent: Monday, Jan 1, 2020 9:00 AM\nQuoted'
+ expect(getCleanBody(input)).toBe('My reply')
+ })
+
+ test('does not truncate at standalone From: line (forwarded content should be preserved)', () => {
+ const input =
+ 'Please see the forwarded email below:\nFrom: Jane Doe