Update version to v1.4.251 and commit

Update GitHub Workflow to Ignore Additional File Paths

.github/workflows/update-version-and-create-tag.yml

@@ -7,6 +7,8 @@ on:
     paths-ignore:
       - "data/patterns/**"
       - "**/*.md"
+      - "data/strategies/**"
+      - "cmd/generate_changelog/*.db"
 
 permissions:
   contents: write # Ensure the workflow has write permissions

CHANGELOG.md

@@ -1,6 +1,28 @@
 # Changelog
 
-## Unreleased
+## v1.4.249 (2025-07-16)
+
+### PR [#1617](https://github.com/danielmiessler/Fabric/pull/1617) by [ksylvan](https://github.com/ksylvan): Improve PR Sync Logic for Changelog Generator
+
+- Preserve PR numbers during version cache merges
+- Enhance changelog to associate PR numbers with version tags
+- Improve PR number parsing with proper error handling
+- Collect all PR numbers for commits between version tags
+- Associate aggregated PR numbers with each version entry
+
+## v1.4.248 (2025-07-16)
+
+### PR [#1616](https://github.com/danielmiessler/Fabric/pull/1616) by [ksylvan](https://github.com/ksylvan): Preserve PR Numbers During Version Cache Merges
+
+- Feat: enhance changelog to correctly associate PR numbers with version tags
+- Fix: improve PR number parsing with proper error handling
+- Collect all PR numbers for commits between version tags
+- Associate aggregated PR numbers with each version entry
+- Update cached versions with newly found PR numbers
+
+### Direct commits
+
+- Docs: reorganize v1.4.247 changelog to attribute changes to PR #1613
 
 ## v1.4.247 (2025-07-15)
 

cmd/fabric/version.go

@@ -1,3 +1,3 @@
 package main
 
-var version = "v1.4.248"
+var version = "v1.4.251"

cmd/generate_changelog/internal/changelog/generator.go

@@ -210,8 +210,26 @@ func (g *Generator) fetchPRs() error {
 		lastSync, _ = g.cache.GetLastPRSync()
 	}
 
+	// Check if we need to sync for missing PRs
+	missingPRs := false
+	for _, version := range g.versions {
+		for _, prNum := range version.PRNumbers {
+			if _, exists := g.prs[prNum]; !exists {
+				missingPRs = true
+				break
+			}
+		}
+		if missingPRs {
+			break
+		}
+	}
+
+	if missingPRs {
+		fmt.Fprintf(os.Stderr, "Full sync triggered due to missing PRs in cache.\n")
+	}
 	// If we have never synced or it's been more than 24 hours, do a full sync
-	needsSync := lastSync.IsZero() || time.Since(lastSync) > 24*time.Hour || g.cfg.ForcePRSync
+	// Also sync if we have versions with PR numbers that aren't cached
+	needsSync := lastSync.IsZero() || time.Since(lastSync) > 24*time.Hour || g.cfg.ForcePRSync || missingPRs
 
 	if !needsSync {
 		fmt.Fprintf(os.Stderr, "Using cached PR data (last sync: %s)\n", lastSync.Format("2006-01-02 15:04:05"))

nix/pkgs/fabric/version.nix

@@ -1 +1 @@
-"1.4.248"
+"1.4.251"