raidboss: fix user triggerset order

[Souma-Sumire]

#992

[jacob-keller]

If I understand correctly, this moves the triggers without an ID to the end so that they'll override anything before them?

[Souma-Sumire]

Yes, Since built-in triggers always have IDs, the current load order is:
[Built-in -> User (w/ ID) -> User (w/o ID)]

However, if we want to respect the user's original file loading order:
[Built-in -> User A (w/ ID) -> User B (w/o ID) -> User C (w/ ID)]

It might be better to do this instead:

    const lastVersionOfId = new Map<string, typeof this.triggerSets[number]>();
    for (const set of this.triggerSets) {
      if (set.id !== undefined) {
        const existing = lastVersionOfId.get(set.id);
        if (existing !== undefined) {
          console.log(
            `Overriding trigger set id '${set.id}' from '${existing.filename}' with '${set.filename}'`,
          );
        }
        lastVersionOfId.set(set.id, set);
      }
    }

    this.triggerSets = this.triggerSets.filter((set) => {
      if (set.id === undefined)
        return true;
      return lastVersionOfId.get(set.id) === set;
    });
    this.triggerSetsById = Object.fromEntries(lastVersionOfId);

[Souma-Sumire]

I'll push this version to stay consistent with the original load order.