Minor optimizations: pre-allocate maps in event paths, consolidate metrics

[Copilot]

Changes

Map pre-allocation in hot paths

• event.go: make(map[string]string, 6) for event headers (exact size known)
• outbox.go: make(map[Header]string, len(outboxRecord.Headers)) (dynamic sizing)

Consolidated duplicate metric recording (timeout.go)

// Before: recorded twice (error path + success path)
if err != nil {
    metrics.ProcessLatency.WithLabelValues(...).Observe(...)
    return err
}
metrics.ProcessLatency.WithLabelValues(...).Observe(...)

// After: single call
metrics.ProcessLatency.WithLabelValues(...).Observe(...)
if err != nil { return err }

Benchmarks added (performance_test.go)

• Map allocation patterns
• Metric label caching
• String operations

Context

Profiling revealed core workflow already well-optimized:

• 98% of allocations in test infrastructure (memrecordstore), not production code
• Run pooling via sync.Pool already implemented
• No spinning issues—all loops use proper blocking/delays

Changes prevent minor inefficiencies and improve code clarity.

---

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
100.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud