Skip to content

bug: ConfigCache.has() inflates hit/miss stats by calling get() internally #497

@nikolasdehor

Description

@nikolasdehor

Bug

ConfigCache.has() delega internamente para get(), que incrementa hits ou misses como efeito colateral.

Problema

Uma simples verificação de existência (has()) infla as estatísticas do cache. Qualquer código que faça if (cache.has(key)) { cache.get(key) } conta 2 hits em vez de 1.

Localização

config-cache.js — método has() chama this.get(key):

has(key) {
  return this.get(key) !== undefined; // incrementa hits/misses!
}

Correção

Reimplementar has() para verificar diretamente no Map sem tocar nas métricas.

Metadata

Metadata

Assignees

No one assigned

    Labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions