Rewrite with better semantics and readability

[tshemsedinov]

Code readability is terrible, forEach has different semantics, long expressions are totally unreadable

be-pkg-utils/src/session.ts

Lines 3 to 15 in e969374

	export function profileFeaturesToList(features: UserFeatures): ProfileFeature[] {
	const featuresList: ProfileFeature[] = []
	Object.entries(features).forEach(([key, value]: [string, unknown]) => value && featuresList.push(<ProfileFeature>key))
	return featuresList.filter((feature) => {
	if (feature !== ProfileFeature.office) {
	return true
	}
	return features?.[feature]?.status === DiiaOfficeStatus.ACTIVE
	})
	}

Please consider at least:

export function profileFeaturesToList(features: UserFeatures): ProfileFeature[] {
    const featuresList: ProfileFeature[] = []

    const entries = Object.entries(features)
    for (const [key, value]: [string, unknown] of entries) {
      if (!value) continue
      featuresList.push(<ProfileFeature>key))
    }

    return featuresList.filter((feature) => {
        if (feature !== ProfileFeature.office) return true
        return features?.[feature]?.status === DiiaOfficeStatus.ACTIVE
    })
}

[bohdansec]

У коді, який ви надали, є синтаксична помилка: ви додали зайву закриваючу дужку після <ProfileFeature>key

Також, у вашому коді є багато частин, які можна покращити. Наприклад, я рекомендую використовувати оператор as для приведення типів замість застарілого синтаксису з кутовими дужками, який ви використовували у вашому прикладі <ProfileFeature>key. Цей підхід є більш сучасним, безпечним і бажаним у TypeScript.

[vird]

#25