chore(release): Update version to v1.4.347

Merge pull request #1875 from ksylvan/kayvan/modernize-part2-loops
modernize: update benchmarks to use b.Loop and refactor map copying
2026-01-09 22:38:10 -05:00 · 2025-12-16 06:51:40 +00:00 · 2025-12-15 22:48:59 -08:00 · 2025-12-15 22:46:45 -08:00 · 2025-12-15 22:40:55 -08:00
6 changed files with 14 additions and 8 deletions
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -1,5 +1,13 @@
 # Changelog

+## v1.4.347 (2025-12-16)
+
+### PR [#1875](https://github.com/danielmiessler/Fabric/pull/1875) by [ksylvan](https://github.com/ksylvan): modernize: update benchmarks to use b.Loop and refactor map copying
+
+- Updated benchmark loops to use cleaner `b.Loop()` syntax
+- Removed unnecessary `b.ResetTimer()` call in token benchmark
+- Used `maps.Copy` for merging variables in patterns handler
+
 ## v1.4.346 (2025-12-16)

 ### PR [#1874](https://github.com/danielmiessler/Fabric/pull/1874) by [ksylvan](https://github.com/ksylvan): refactor: replace interface{} with any across codebase
--- a/cmd/fabric/version.go
+++ b/cmd/fabric/version.go
@@ -1,3 +1,3 @@
 package main

-var version = "v1.4.346"
+var version = "v1.4.347"
--- a/cmd/generate_changelog/changelog.db
+++ b/cmd/generate_changelog/changelog.db
--- a/internal/plugins/ai/anthropic/oauth_test.go
+++ b/internal/plugins/ai/anthropic/oauth_test.go
@@ -416,7 +416,7 @@ func TestGetValidTokenWithValidToken(t *testing.T) {

 // Benchmark tests
 func BenchmarkGeneratePKCE(b *testing.B) {
-	for i := 0; i < b.N; i++ {
+	for b.Loop() {
 		_, _, err := generatePKCE()
 		if err != nil {
 			b.Fatal(err)
@@ -427,8 +427,7 @@ func BenchmarkGeneratePKCE(b *testing.B) {
 func BenchmarkTokenIsExpired(b *testing.B) {
 	token := createTestToken("access", "refresh", 3600)

-	b.ResetTimer()
-	for i := 0; i < b.N; i++ {
+	for b.Loop() {
 		token.IsExpired(5)
 	}
 }
--- a/internal/server/patterns.go
+++ b/internal/server/patterns.go
@@ -1,6 +1,7 @@
 package restapi

 import (
+	"maps"
 	"net/http"

 	"github.com/danielmiessler/fabric/internal/plugins/db/fsdb"
@@ -74,9 +75,7 @@ func (h *PatternsHandler) ApplyPattern(c *gin.Context) {
 			variables[key] = values[0]
 		}
 	}
-	for key, value := range request.Variables {
-		variables[key] = value
-	}
+	maps.Copy(variables, request.Variables)

 	pattern, err := h.patterns.GetApplyVariables(name, variables, request.Input)
 	if err != nil {
--- a/nix/pkgs/fabric/version.nix
+++ b/nix/pkgs/fabric/version.nix
@@ -1 +1 @@
-"1.4.346"
+"1.4.347"
Author	SHA1	Message	Date
github-actions[bot]	005d43674f	chore(release): Update version to v1.4.347	2025-12-16 06:51:40 +00:00
Kayvan Sylvan	3a69437790	Merge pull request #1875 from ksylvan/kayvan/modernize-part2-loops modernize: update benchmarks to use b.Loop and refactor map copying	2025-12-15 22:48:59 -08:00
Changelog Bot	b057f52ca6	chore: incoming 1875 changelog entry	2025-12-15 22:46:45 -08:00
Kayvan Sylvan	dccdfbac8c	test: update benchmarks to use b.Loop and refactor map copying # CHANGES - update benchmark loops to use cleaner `b.Loop()` syntax - remove unnecessary `b.ResetTimer()` call in token benchmark - use `maps.Copy` for merging variables in patterns handler	2025-12-15 22:40:55 -08:00