sindresorhus/type-fest

`NonEmptyObject` fails for objects with dynamic properties

Open

#821 aperta il 27 feb 2024

Vedi su GitHub
 (3 commenti) (1 reazione) (0 assegnatari)TypeScript (471 fork)batch import
buggood first issuehelp wanted

Metriche repository

Star
 (12.328 star)
Metriche merge PR
 (Merge medio 4g 18h) (9 PR mergiate in 30 g)

Descrizione

The following code is considered as valid by Typescript, although I was expecting it to complain that the filter foo in commonArguments was empty.

import type { NonEmptyObject } from "type-fest"

interface CommonArguments {
  [filter: string]: NonEmptyObject<{ [argument: string]: string | number | undefined }>
}

export const commonArguments: CommonArguments = {
  foo: {}
}

Guida contributor

Direzione di ricerca
Il problema è che il tipo `NonEmptyObject` non rifiuta correttamente gli oggetti con firme di indice quando l'oggetto è vuoto. Il problema risiede probabilmente nella definizione del tipo che utilizza un tipo condizionale che verifica se tutte le chiavi sono opzionali o simili. Indaga l'implementazione corrente nel file sorgente (probabilmente `source/non empty object.d.ts`). Considera di modificare la logica per gestire le firme di indice controllando se l'oggetto ha chiavi note o se è assegnabile a `Record<string, never>`. Esamina il tracker delle issue di TypeScript per pattern simili. Controlla PR o commenti collegati per ulteriore contesto.
Tech stack
typescript
Dominio
developer experience
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Abbastanza chiara
Prerequisiti
understanding of TypeScript conditional typesfamiliarity with type fest utilities
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
40