Skip to content

fix(notes): backfill summary timestamps - #27

Merged
Vann-Dev merged 2 commits into
mainfrom
kaito/fix-summary-updated-at
May 12, 2026
Merged

Vann-Dev merged 2 commits into
mainfrom
kaito/fix-summary-updated-at

Conversation

@Vann-Dev

Copy link
Copy Markdown
Owner

Summary

  • Backfill missing summaryUpdatedAt values for existing tasks with non-empty notes using taskUpdatedAt as deterministic fallback.
  • Make Notes page/API unread filtering use taskUpdatedAt fallback for any remaining legacy rows with null summary timestamps.
  • When marking a done note reviewed, repair a missing summaryUpdatedAt inside the same transaction before writing the done-status read marker.

Root cause

Agent/internal mutation paths now set summaryUpdatedAt when note text actually changes, but older task-card note work updated several done task notes before the metadata was consistently written. Those legacy rows had non-empty note and null summaryUpdatedAt, making read freshness less auditable.

Checks

  • PASS: corepack pnpm lint
  • PASS: DATABASE_URL=postgresql://user:pass@localhost:5432/db?schema=public AUTH_SECRET=placeholder corepack pnpm typecheck
  • PASS: DATABASE_URL=postgresql://user:pass@localhost:5432/db?schema=public AUTH_SECRET=placeholder corepack pnpm build

Build warnings only: existing Next.js middleware convention deprecation and jose Edge Runtime CompressionStream/DecompressionStream warnings.

@vercel

vercel Bot commented May 12, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
agent-bridge Error Error May 12, 2026 9:40pm

@Vann-Dev
Vann-Dev merged commit 72e4a54 into main May 12, 2026
2 of 3 checks passed
@Vann-Dev
Vann-Dev deleted the kaito/fix-summary-updated-at branch May 12, 2026 22:04

This branch was previously deployed

1 inactive deployment
Preview — 137fd071 Deployed May 12, 2026 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant