From 3d88f8e2fcafa43d1224d267e5b14fa4ab77cf2c Mon Sep 17 00:00:00 2001 From: Kayvan Sylvan Date: Sat, 3 Jan 2026 23:23:19 -0800 Subject: [PATCH] fix: update `SetLastPRSync` to use version date instead of current time - Change last_pr_sync to use versionDate instead of time.Now() - Ensure future runs fetch PRs merged after the version date - Add clarifying comments explaining the sync timing logic --- cmd/generate_changelog/internal/changelog/processing.go | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/cmd/generate_changelog/internal/changelog/processing.go b/cmd/generate_changelog/internal/changelog/processing.go index 560df64b..bb888cd1 100644 --- a/cmd/generate_changelog/internal/changelog/processing.go +++ b/cmd/generate_changelog/internal/changelog/processing.go @@ -291,8 +291,9 @@ func (g *Generator) CreateNewChangelogEntry(version string) error { fmt.Fprintf(os.Stderr, "Warning: Failed to update last_processed_tag: %v\n", err) } - // Update last_pr_sync to current time - if err := g.cache.SetLastPRSync(time.Now()); err != nil { + // Update last_pr_sync to the version date (not current time) + // This ensures future runs will fetch PRs merged after this version + if err := g.cache.SetLastPRSync(versionDate); err != nil { fmt.Fprintf(os.Stderr, "Warning: Failed to update last_pr_sync: %v\n", err) } }