alerts() returns undefined for state

[cshields]

First, thanks for the library, this is great!

I noticed when pulling prom.alerts() that our alert state is always coming back undefined (rather than 'firing' or 'pending').

I'm not confident enough to try a patch and PR but I think this is a mixed array call:

prometheus-query-js/src/types.ts

Line 229 in f1a2776

ResponseType[obj['state']],

You may have intended for that to reference TargetState?

prometheus-query-js/src/types.ts

Line 299 in f1a2776

export type TargetState = 'active' | 'dropped' | 'any';

and if so TargetState should also include 'firing' and 'pending'

cheers!

[samber]

I have no time for writing a patch. But feel free to clone the repository.

You can use the following script for testing: https://github.com/samber/prometheus-query-js/blob/master/examples/nodejs/app.ts