submit mutation rejects current ccusage output: daily entries now use period instead of date (Invalid date format: undefined)

[dprvda]

Fixes #49.

Problem

Current ccusage --json emits each daily entry keyed by period (plus agent/metadata) instead of date. The submit mutation reads day.date, which is now undefined, so submissions fail with Invalid date format: undefined. Expected YYYY-MM-DD. None of the ingestion paths (packages/viberank-cli/cli.js, src/app/api/submit/route.ts, submit-to-viberank.sh) normalize this, so the fix lives at the single chokepoint they all funnel through: convex/submissions.ts.

Change (convex/submissions.ts, +19/−7)

1. Widen the daily args validator — date becomes optional and period/agent/metadata are accepted as optional. Convex v.object is strict, so without this it rejects the real payload before the handler runs.
2. Coalesce day.date ?? day.period in the validation loop and at every incoming-date read plus both stored-write paths (merge + insert), so dailyBreakdown.date is always a valid YYYY-MM-DD.

convex/schema.ts (stored date: v.string(), required) is intentionally left unchanged — period is normalized to date at ingestion.

Verification

• Type-neutral — typechecking the convex/ sources before and after the patch yields the same result; it introduces no new type errors. Please confirm with your usual convex deploy / tsc.
• End-to-end on a real failing cc.json (32 daily entries, all keyed by period): pre-change throws the exact reported error; post-change all 32 pass and store valid dates (range 2026-05-01..2026-06-01).

Coordination notes

• Overlaps "Token totals don't match. Please use official ccusage tool." #48 — a comment there independently reports the same missing-date crash and the same date = period fix; consider deduping.
• Fix/dashboard fixes #39 also edits convex/submissions.ts; flagging for potential merge-conflict coordination.

🤖 Generated with Claude Code

[vercel]

@dprvda is attempting to deploy a commit to the sculpt Team on Vercel.

A member of the Team first needs to authorize it.

[nikshepsvn]

Thanks for this — your diagnosis of the period vs date regression and the failing payload were spot on, and they directly informed the fix.

The one catch: production runs on Supabase, not the Convex backend this PR patches (Convex was a dormant fallback in the tree). So the real submission path never hit convex/submissions.ts — the fix had to land in the Supabase path. viberank v2 (#51) implements it there, normalizing every ccusage report shape at a single chokepoint, and also removes the dormant Convex backend entirely so this kind of wrong-target confusion can't happen again.

Closing as superseded by #51 — but genuinely appreciate you surfacing the root cause. 🙏