Enum `FlagSource` not useable as value

[jhoermann]

Problem
Since the enum FlagSource is only exported as Type from types.d.ts and not as value it can't be used in the code e.g. like this.

import { FlagSource } from 'flagsmith'

if (source === FlagSource.None) {
    ...
}

Directly comparing with the string "NONE" should be avoided (see https://typescript-eslint.io/rules/no-unsafe-enum-comparison/).
So the only workaround is using it like this:

import type { FlagSource } from 'flagsmith/types'

if (source === ('NONE' as FlagSource.NONE)) {
    ...
}

Which is not ideal.

Possible Solutions

1. Export the enum as value e.g. from the flagsmith core file
2. Get rid of the enum and use string literal types (https://www.typescriptcourse.com/string-literal-unions-over-enums)

[jhoermann]

Fix MR in progress: #349

[talissoncosta]

Thanks a lot for raising this issue and for the detailed context, @jhoermann — that was super helpful 🙌

This has now been fixed and merged. FlagSource is exported as a value and can be used directly in code (e.g., FlagSource.None) without the workaround.

The fix will be included in the next release. I’ll update here once it’s live.

Appreciate you taking the time to report this and suggest possible solutions!