fix: return proper resolution reasons in the JS provider

[Datron]

Problem

The resolution reason states are missing in the JS provider

Solution

Use resolution reason states in the reason field

Summary by CodeRabbit

• Refactor
  • Standardized resolution reason values in the feature flag provider across all resolution paths.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[semanticdiff-com]

Review changes with  

Changed Files

File	Status
clients/javascript/open-feature-provider/superposition-provider.ts	52% smaller

[coderabbitai]

Walkthrough

Resolution reasons in the Superposition provider were refactored from string literals to the StandardResolutionReasons enum. The logic routes different conditions—stale status, default value matches, and targeting matches—to corresponding enum values. No functional behavior changed.

Changes

Cohort / File(s)	Summary
Resolution Reason Enum Refactoring clients/javascript/open-feature-provider/superposition-provider.ts	Replaced string-based resolution reasons with StandardResolutionReasons enum values. Not-ready/stale/error paths now use StandardResolutionReasons.DEFAULT or .ERROR; normal resolution paths use .STALE, .DEFAULT, or .TARGETING_MATCH based on status and value comparison.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Poem

🐰 A hop, a skip, through reasons so fine,
From strings into enums, a structured design!
Resolution clarity, standardized true,
No breaking of function, just making it new. ✨

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The PR title accurately describes the main change: replacing string-based resolution reasons with StandardResolutionReasons enum in the JS provider.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

• 📝 Generate docstrings

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}