perf: avoid array conversion in getMetricsAsJSON

mcollina · claude · jdmarshall · commit a943565ffc99 · 2026-01-03T00:24:15.000-08:00
Replace this.getMetricsAsArray() with direct iteration over this._metrics.values() to eliminate unnecessary Array.from() conversion. Performance improvement: ~1.3% faster (3,216 -> 3,259 calls/sec) 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
diff --git a/lib/registry.js b/lib/registry.js
@@ -137,7 +137,7 @@ class Registry {
 
 		const promises = [];
 
-		for (const metric of this.getMetricsAsArray()) {
+		for (const metric of this._metrics.values()) {
 			promises.push(metric.get());
 		}
 

Original file line number	Diff line number	Diff line change
`@@ -137,7 +137,7 @@ class Registry {`
`137`	`137`
`138`	`138`	`const promises = [];`
`139`	`139`
`140`		`- for (const metric of this.getMetricsAsArray()) {`
	`140`	`+ for (const metric of this._metrics.values()) {`
`141`	`141`	`promises.push(metric.get());`
`142`	`142`	`}`
`143`	`143`