Reports bot: correct MTD bucketing comment to match date-only FM field

541942f256a21297be28b683b6d19d69d04f0d33 · 2026-07-28 22:19:58 -0700 · Steve Abrams

Contrarian Hole 2, verified: the layout has no entry-timestamp, so MTD buckets
by the FM Date as entered, not a true Pacific-time instant. Fix the aspirational
'(Pacific Time)' comment rather than invent timezone logic the data can't support.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

Files touched

M scripts/sales-summary.mjs

Diff

commit 541942f256a21297be28b683b6d19d69d04f0d33
Author: Steve Abrams <steve@designerwallcoverings.com>
Date:   Tue Jul 28 22:19:58 2026 -0700

    Reports bot: correct MTD bucketing comment to match date-only FM field
    
    Contrarian Hole 2, verified: the layout has no entry-timestamp, so MTD buckets
    by the FM Date as entered, not a true Pacific-time instant. Fix the aspirational
    '(Pacific Time)' comment rather than invent timezone logic the data can't support.
    
    Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
---
 scripts/sales-summary.mjs | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/scripts/sales-summary.mjs b/scripts/sales-summary.mjs
index 150ca55..ceb0dc0 100644
--- a/scripts/sales-summary.mjs
+++ b/scripts/sales-summary.mjs
@@ -4,7 +4,10 @@
 // timestamp fields — only dates — so the window is implemented as a diff against a
 // ledger of invoice numbers already reported (data/sales-summary-ledger.json). That
 // catches late-day orders, backdated entries, and quote→booked conversions exactly.
-// MONTH-TO-DATE stays bucketed by invoice Date (Pacific Time).
+// MONTH-TO-DATE stays bucketed by the invoice's FM Date field. That field is
+// date-only (no timestamp on the layout), so a month-boundary invoice buckets by
+// the calendar date as entered, NOT by a true Pacific-time instant — negligible
+// for a single-office PT operation, and uncorrectable without an entry timestamp.
 //
 //   node scripts/sales-summary.mjs            # compute + POST to Slack + update ledger
 //   node scripts/sales-summary.mjs --dry-run  # compute + print only (no post, no ledger write)

← bbd7473 Reports bot: visible truncation warning + stop counting refu · back to Filemaker Mcp · chore: v0.2.4 (session close — MTD window fix + truncation g 26038de →