refactor(uniformbuilder): sort weapon qual plates by catalog awardPriority

[SyniRon]

Closes #242

Built on #241, which merged while this was in progress. Rebased onto main, so the diff is this change alone.

What changed

• Every weapon qual entry in AWARD_CATALOG carries awardPriority, its rank in the S1 Uniforms SOP weapon order. All three entries for a weapon carry the same number. Ranks 9 and 10 are left open for Carbine and Auto Rifle, which the SOP lists and MILPAC does not award yet. The section comment says so.
• WeaponQual records the rank as each qual is filed and sortQuals sorts by it. The hand list weaponOrder and its indexOf sort are gone. The level arrays are still arrays of tags, because the canvas names the plate file after the tag.
• Catalog test: one row per tag checks the three entries carry one number. A missing or disagreeing entry prints the set it broke.
• Canvas test: the reverse-SOP test holds all ten weapons and is the one place a person types out the SOP order. A guard beside it reads the catalog for which weapons exist, order aside, and goes red when a weapon qual entry lands with no row in that list.
• The "stacks above Mk-82" family, its count guard and its comment block are gone.

Why

Adding a weapon took three edits that had to agree by hand: the catalog entries, the slot in weaponOrder, and the plate file. The plate had a test. The slot did not, and no canvas test could see a missing slot for the last SOP weapon, because last is where the SOP puts it. #229 was one instance of that class. Ribbons, medals, unit citations and tabs already sort by awardPriority from the catalog. Weapon quals were the one ordered type that did not.

What a member sees on the uniform does not change.

For the reviewer

• Mutations run against the finished branch, each restored afterwards. Pistol given Rifle's number: the reverse-SOP test fails. Pistol and Aeroweapons swapped: the same test fails. Pistol Expert loses its priority: the catalog row for pistol fails with [null,7]. An eleventh weapon added with Mk-82's number and no row in the held list: the guard fails naming Carbine Expert.
• The brief's criterion "no two tags share a priority fails a catalog test" is struck through on Derive the weapon qual sort order from the award catalog #242. The tie fails the canvas test instead, because the held list is reversed and a stable sort keeps a tied pair reversed. With the guard, every catalog weapon is in that list, so a catalog row would be a structural copy that cannot go red alone.
• The weaponQualPlates.js comment named the deleted list, so it is reworded. No code change there.
• Checks: npm test exit 0 (215 sprite assertions, 46 catalog, 29 canvas, 13 numerals, server). npm run format:check clean.

This was generated by AI