raidboss: sort r9s bat positions for consistency (#985)

When tracking the bat positions it is possible for the different lines
to arrive in an arbitrary order. This results in the callout being
inconsistent even when the bats are in the same position. For example it
might call out "away from bats S/N" instead of always consistently
saying "away from bats N/S".

Use sort() to make sure the bat callouts are consistent and always
display the directions in the same order.

---------

Co-authored-by: valarnin <valarnin@gmail.com>

Author: jacob-keller

resources/util.ts

@@ -263,6 +263,18 @@ const output16Dir: DirectionOutput16[] = [
 const outputCardinalDir: DirectionOutputCardinal[] = ['dirN', 'dirE', 'dirS', 'dirW'];
 const outputIntercardDir: DirectionOutputIntercard[] = ['dirNE', 'dirSE', 'dirSW', 'dirNW'];
 
+const compareDirectionOutput = (a: DirectionOutput16, b: DirectionOutput16): number => {
+  const getIndex = (n: DirectionOutput16) => {
+    const index = output16Dir.indexOf(n);
+    // Values outside of output16Dir (i.e. 'unknown') sort last
+    if (index < 0)
+      return output16Dir.length;
+    return index;
+  };
+
+  return getIndex(a) - getIndex(b);
+};
+
 const outputStrings16Dir: OutputStrings = {
   dirN: Outputs.dirN,
   dirNNE: Outputs.dirNNE,
@@ -375,6 +387,7 @@ export const Directions = {
   output16Dir: output16Dir,
   outputCardinalDir: outputCardinalDir,
   outputIntercardDir: outputIntercardDir,
+  compareDirectionOutput: compareDirectionOutput,
   outputStrings16Dir: outputStrings16Dir,
   outputStrings8Dir: outputStrings8Dir,
   outputStringsCardinalDir: outputStringsCardinalDir,

ui/raidboss/data/07-dt/raid/r9s.ts

@@ -172,7 +172,7 @@ const triggerSet: TriggerSet<Data> = {
       durationSeconds: 5.5,
       suppressSeconds: 1,
       infoText: (data, _matches, output) => {
-        const [dir1, dir2] = data.bats.inner;
+        const [dir1, dir2] = data.bats.inner.sort(Directions.compareDirectionOutput);
 
         return output.away!({
           dir1: output[dir1 ?? 'unknown']!(),
@@ -202,7 +202,7 @@ const triggerSet: TriggerSet<Data> = {
       durationSeconds: 3.4,
       suppressSeconds: 1,
       infoText: (data, _matches, output) => {
-        const [dir1, dir2, dir3] = data.bats.middle;
+        const [dir1, dir2, dir3] = data.bats.middle.sort(Directions.compareDirectionOutput);
 
         return output.away!({
           dir1: output[dir1 ?? 'unknown']!(),