GeezerVison

Pull request overview

This PR optimizes cached artifact serving by reducing database round-trips for cache hits and lowering per-request allocations during response streaming, while keeping integrity verification and hit accounting intact.

Changes:

• Replace multiple cache-hit DB lookups with a single indexed join query (GetCachedArtifact).
• Reduce allocations in artifact serving (pooled 32 KiB copy buffer; avoid fmt.Sprintf formatting; avoid PURL parsing for metrics).
• Add benchmarks and a database test to validate the new cached-artifact lookup path.

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 1 comment.

			if err := p.DB.ClearArtifactCache(versionPURL, filename); err != nil {
				p.Logger.Warn("failed to clear corrupt artifact from cache", "error", err)
			}
		})
	p.recordCacheHit(ecosystem, versionPURL, filename)
	return result, nil

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.