Tie break top-scoring bandit actions by action name (#119)

* tie break top-scoring actions by action name * bump version number
Eppo-exp · Aug 8, 2024 · 90dabbc · 90dabbc
1 parent ae6bfde
commit 90dabbc
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 2 deletions.
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "@eppo/js-client-sdk-common",
-  "version": "4.0.1",
+  "version": "4.0.2",
   "description": "Eppo SDK for client-side JavaScript applications (base for both web and react native)",
   "main": "dist/index.js",
   "files": [

diff --git a/src/bandit-evaluator.ts b/src/bandit-evaluator.ts
@@ -144,7 +144,12 @@ export class BanditEvaluator {
     let currTopScore: number | null = null;
     let currTopAction: string | null = null;
     actionScoreEntries.forEach(([actionKey, actionScore]) => {
-      if (currTopScore === null || actionScore > currTopScore) {
+      if (
+        currTopScore === null ||
+        currTopAction === null ||
+        actionScore > currTopScore ||
+        (actionScore === currTopScore && actionKey < currTopAction)
+      ) {
         currTopScore = actionScore;
         currTopAction = actionKey;
       }