@pkmn/engine 0.1.0-dev.d716e940 → 0.1.0-dev.d97c6e03

package/src/lib/common/protocol.zig

@@ -7,7 +7,7 @@ const options = @import("./options.zig");
 const assert = std.debug.assert;
 const print = std.debug.print;
 
-const enabled = options.log;
+const trace = options.trace;
 
 const Player = data.Player;
 const ID = data.ID;
@@ -230,7 +230,7 @@ pub fn Log(comptime Writer: type) type {
         pub const Error = Writer.Error;
 
         pub fn move(self: Self, source: ID, m: anytype, target: ID, from: anytype) Error!void {
-            if (!enabled) return;
+            if (!trace) return;
             assert(m != .None);
             try self.writer.writeAll(&.{
                 @enumToInt(ArgType.Move),
@@ -253,7 +253,7 @@ pub fn Log(comptime Writer: type) type {
         }
 
         pub fn switched(self: Self, ident: ID, pokemon: anytype) Error!void {
-            if (!enabled) return;
+            if (!trace) return;
             try self.writer.writeAll(&.{ @enumToInt(ArgType.Switch), @bitCast(u8, ident) });
             try self.writer.writeAll(&.{ @enumToInt(pokemon.species), pokemon.level });
             try self.writer.writeIntNative(u16, pokemon.hp);
@@ -262,7 +262,7 @@ pub fn Log(comptime Writer: type) type {
         }
 
         pub fn cant(self: Self, ident: ID, reason: Cant) Error!void {
-            if (!enabled) return;
+            if (!trace) return;
             assert(reason != .Disable);
             try self.writer.writeAll(&.{
                 @enumToInt(ArgType.Cant),
@@ -272,7 +272,7 @@ pub fn Log(comptime Writer: type) type {
         }
 
         pub fn disabled(self: Self, ident: ID, m: anytype) Error!void {
-            if (!enabled) return;
+            if (!trace) return;
             try self.writer.writeAll(&.{
                 @enumToInt(ArgType.Cant),
                 @bitCast(u8, ident),
@@ -282,20 +282,20 @@ pub fn Log(comptime Writer: type) type {
         }
 
         pub fn faint(self: Self, ident: ID, done: bool) Error!void {
-            if (!enabled) return;
+            if (!trace) return;
             try self.writer.writeAll(&.{ @enumToInt(ArgType.Faint), @bitCast(u8, ident) });
             if (done) try self.writer.writeByte(@enumToInt(ArgType.None));
         }
 
         pub fn turn(self: Self, num: u16) Error!void {
-            if (!enabled) return;
+            if (!trace) return;
             try self.writer.writeByte(@enumToInt(ArgType.Turn));
             try self.writer.writeIntNative(u16, num);
             try self.writer.writeByte(@enumToInt(ArgType.None));
         }
 
         pub fn win(self: Self, player: Player) Error!void {
-            if (!enabled) return;
+            if (!trace) return;
             try self.writer.writeAll(&.{
                 @enumToInt(ArgType.Win),
                 @enumToInt(player),
@@ -304,12 +304,12 @@ pub fn Log(comptime Writer: type) type {
         }
 
         pub fn tie(self: Self) Error!void {
-            if (!enabled) return;
+            if (!trace) return;
             try self.writer.writeAll(&.{ @enumToInt(ArgType.Tie), @enumToInt(ArgType.None) });
         }
 
         pub fn damage(self: Self, ident: ID, pokemon: anytype, reason: Damage) Error!void {
-            if (!enabled) return;
+            if (!trace) return;
             assert(@enumToInt(reason) <= @enumToInt(Damage.LeechSeed));
             try self.writer.writeAll(&.{ @enumToInt(ArgType.Damage), @bitCast(u8, ident) });
             try self.writer.writeIntNative(u16, pokemon.hp);
@@ -324,7 +324,7 @@ pub fn Log(comptime Writer: type) type {
             reason: Damage,
             source: ID,
         ) Error!void {
-            if (!enabled) return;
+            if (!trace) return;
             assert(@enumToInt(reason) == @enumToInt(Damage.RecoilOf));
             try self.writer.writeAll(&.{ @enumToInt(ArgType.Damage), @bitCast(u8, ident) });
             try self.writer.writeIntNative(u16, pokemon.hp);
@@ -337,7 +337,7 @@ pub fn Log(comptime Writer: type) type {
         }
 
         pub fn heal(self: Self, ident: ID, pokemon: anytype, reason: Heal) Error!void {
-            if (!enabled) return;
+            if (!trace) return;
             assert(reason != .Drain);
             try self.writer.writeAll(&.{ @enumToInt(ArgType.Heal), @bitCast(u8, ident) });
             try self.writer.writeIntNative(u16, pokemon.hp);
@@ -346,7 +346,7 @@ pub fn Log(comptime Writer: type) type {
         }
 
         pub fn drain(self: Self, source: ID, pokemon: anytype, target: ID) Error!void {
-            if (!enabled) return;
+            if (!trace) return;
             try self.writer.writeAll(&.{ @enumToInt(ArgType.Heal), @bitCast(u8, source) });
             try self.writer.writeIntNative(u16, pokemon.hp);
             try self.writer.writeIntNative(u16, pokemon.stats.hp);
@@ -358,7 +358,7 @@ pub fn Log(comptime Writer: type) type {
         }
 
         pub fn status(self: Self, ident: ID, value: u8, reason: Status) Error!void {
-            if (!enabled) return;
+            if (!trace) return;
             assert(reason != .From);
             try self.writer.writeAll(&.{
                 @enumToInt(ArgType.Status),
@@ -369,7 +369,7 @@ pub fn Log(comptime Writer: type) type {
         }
 
         pub fn statusFrom(self: Self, ident: ID, value: u8, m: anytype) Error!void {
-            if (!enabled) return;
+            if (!trace) return;
             try self.writer.writeAll(&.{
                 @enumToInt(ArgType.Status),
                 @bitCast(u8, ident),
@@ -380,7 +380,7 @@ pub fn Log(comptime Writer: type) type {
         }
 
         pub fn curestatus(self: Self, ident: ID, value: u8, reason: CureStatus) Error!void {
-            if (!enabled) return;
+            if (!trace) return;
             try self.writer.writeAll(&.{
                 @enumToInt(ArgType.CureStatus),
                 @bitCast(u8, ident),
@@ -390,7 +390,7 @@ pub fn Log(comptime Writer: type) type {
         }
 
         pub fn boost(self: Self, ident: ID, reason: Boost, num: i8) Error!void {
-            if (!enabled) return;
+            if (!trace) return;
             assert(num != 0 and (reason != .Rage or num > 0));
             try self.writer.writeAll(&.{
                 @enumToInt(ArgType.Boost),
@@ -401,12 +401,12 @@ pub fn Log(comptime Writer: type) type {
         }
 
         pub fn clearallboost(self: Self) Error!void {
-            if (!enabled) return;
+            if (!trace) return;
             try self.writer.writeAll(&.{@enumToInt(ArgType.ClearAllBoost)});
         }
 
         pub fn fail(self: Self, ident: ID, reason: Fail) Error!void {
-            if (!enabled) return;
+            if (!trace) return;
             try self.writer.writeAll(&.{
                 @enumToInt(ArgType.Fail),
                 @bitCast(u8, ident),
@@ -415,17 +415,17 @@ pub fn Log(comptime Writer: type) type {
         }
 
         pub fn miss(self: Self, source: ID) Error!void {
-            if (!enabled) return;
+            if (!trace) return;
             try self.writer.writeAll(&.{ @enumToInt(ArgType.Miss), @bitCast(u8, source) });
         }
 
         pub fn hitcount(self: Self, ident: ID, num: u8) Error!void {
-            if (!enabled) return;
+            if (!trace) return;
             try self.writer.writeAll(&.{ @enumToInt(ArgType.HitCount), @bitCast(u8, ident), num });
         }
 
         pub fn prepare(self: Self, source: ID, m: anytype) Error!void {
-            if (!enabled) return;
+            if (!trace) return;
             try self.writer.writeAll(&.{
                 @enumToInt(ArgType.Prepare),
                 @bitCast(u8, source),
@@ -434,12 +434,12 @@ pub fn Log(comptime Writer: type) type {
         }
 
         pub fn mustrecharge(self: Self, ident: ID) Error!void {
-            if (!enabled) return;
+            if (!trace) return;
             try self.writer.writeAll(&.{ @enumToInt(ArgType.MustRecharge), @bitCast(u8, ident) });
         }
 
         pub fn activate(self: Self, ident: ID, reason: Activate) Error!void {
-            if (!enabled) return;
+            if (!trace) return;
             try self.writer.writeAll(&.{
                 @enumToInt(ArgType.Activate),
                 @bitCast(u8, ident),
@@ -448,12 +448,12 @@ pub fn Log(comptime Writer: type) type {
         }
 
         pub fn fieldactivate(self: Self) Error!void {
-            if (!enabled) return;
+            if (!trace) return;
             try self.writer.writeAll(&.{@enumToInt(ArgType.FieldActivate)});
         }
 
         pub fn start(self: Self, ident: ID, reason: Start) Error!void {
-            if (!enabled) return;
+            if (!trace) return;
             assert(@enumToInt(reason) < @enumToInt(Start.TypeChange));
             try self.writer.writeAll(&.{
                 @enumToInt(ArgType.Start),
@@ -463,7 +463,7 @@ pub fn Log(comptime Writer: type) type {
         }
 
         pub fn typechange(self: Self, source: ID, types: anytype, target: ID) Error!void {
-            if (!enabled) return;
+            if (!trace) return;
             try self.writer.writeAll(&.{
                 @enumToInt(ArgType.Start),
                 @bitCast(u8, source),
@@ -474,7 +474,7 @@ pub fn Log(comptime Writer: type) type {
         }
 
         pub fn startEffect(self: Self, ident: ID, reason: Start, m: anytype) Error!void {
-            if (!enabled) return;
+            if (!trace) return;
             assert(@enumToInt(reason) > @enumToInt(Start.TypeChange));
             try self.writer.writeAll(&.{
                 @enumToInt(ArgType.Start),
@@ -485,7 +485,7 @@ pub fn Log(comptime Writer: type) type {
         }
 
         pub fn end(self: Self, ident: ID, reason: End) Error!void {
-            if (!enabled) return;
+            if (!trace) return;
             try self.writer.writeAll(&.{
                 @enumToInt(ArgType.End),
                 @bitCast(u8, ident),
@@ -494,27 +494,27 @@ pub fn Log(comptime Writer: type) type {
         }
 
         pub fn ohko(self: Self) Error!void {
-            if (!enabled) return;
+            if (!trace) return;
             try self.writer.writeAll(&.{@enumToInt(ArgType.OHKO)});
         }
 
         pub fn crit(self: Self, ident: ID) Error!void {
-            if (!enabled) return;
+            if (!trace) return;
             try self.writer.writeAll(&.{ @enumToInt(ArgType.Crit), @bitCast(u8, ident) });
         }
 
         pub fn supereffective(self: Self, ident: ID) Error!void {
-            if (!enabled) return;
+            if (!trace) return;
             try self.writer.writeAll(&.{ @enumToInt(ArgType.SuperEffective), @bitCast(u8, ident) });
         }
 
         pub fn resisted(self: Self, ident: ID) Error!void {
-            if (!enabled) return;
+            if (!trace) return;
             try self.writer.writeAll(&.{ @enumToInt(ArgType.Resisted), @bitCast(u8, ident) });
         }
 
         pub fn immune(self: Self, ident: ID, reason: Immune) Error!void {
-            if (!enabled) return;
+            if (!trace) return;
             try self.writer.writeAll(&.{
                 @enumToInt(ArgType.Immune),
                 @bitCast(u8, ident),
@@ -523,7 +523,7 @@ pub fn Log(comptime Writer: type) type {
         }
 
         pub fn transform(self: Self, source: ID, target: ID) Error!void {
-            if (!enabled) return;
+            if (!trace) return;
             try self.writer.writeAll(&.{
                 @enumToInt(ArgType.Transform),
                 @bitCast(u8, source),
@@ -532,12 +532,12 @@ pub fn Log(comptime Writer: type) type {
         }
 
         pub fn laststill(self: Self) Error!void {
-            if (!enabled) return;
+            if (!trace) return;
             try self.writer.writeAll(&.{@enumToInt(ArgType.LastStill)});
         }
 
         pub fn lastmiss(self: Self) Error!void {
-            if (!enabled) return;
+            if (!trace) return;
             try self.writer.writeAll(&.{@enumToInt(ArgType.LastMiss)});
         }
     };
@@ -838,7 +838,7 @@ pub fn expectLog(
     actual: []const u8,
     offset: usize,
 ) !void {
-    if (!enabled) return;
+    if (!trace) return;
     const color = color: {
         if (std.process.hasEnvVarConstant("ZIG_DEBUG_COLOR")) {
             break :color true;

package/src/lib/gen1/README.md

@@ -28,36 +28,35 @@ list of additional reference [**resources**](#resources).
 
 The following information is required to simulate a Generation I Pokémon battle:
 
-| pkmn                                    | Pokémon Red (pret)                 | Pokémon Showdown                       |
-| --------------------------------------- | ---------------------------------- | -------------------------------------- |
-| `battle.seed`                           | `Random{Add,Sub}`                  | `battle.seed`                          |
-| `battle.turn`                           | -                                  | `battle.turn`                          |
-| `battle.last_damage`                    | `Damage`                           | `battle.lastDamage`                    |
-| `side.{active,pokemon}`                 | `PlayerMonNumber`/`BattleMon`      | `side.active`                          |
-| `side.team`                             | `PartyMons`                        | `side.pokemon`                         |
-| `side.last_used_move`                   | `PlayerUsedMove`                   | `pokemon.lastMove`                     |
-| `side.last_selected_move`               | `PlayerSelectedMove`               | `side.lastSelectedMove`                |
-| `battle.last_moves.{p1,p2}_index`       | `PlayerMoveListIndex`              | -                                      |
-| `battle.last_moves.{p1,p2}_counterable` | -                                  | `side.lastMove`                        |
-| `side.order`                            | -                                  | `pokemon.position`                     |
-| `{pokemon,active}.moves`                | `{party,battle}_struct.{Moves,PP}` | `pokemon.{baseMoveSlots,moveSlots}`    |
-| `pokemon.hp`                            | `{party,battle}_struct.HP`         | `pokemon.hp`                           |
-| `{pokemon,active}.status`               | `{party,battle}_struct.Status`     | `pokemon.status`                       |
-| `{pokemon,active}.level`                | `PlayerMonUnmodifiedLevel`         | `pokemon.level`                        |
-| `pokemon.species`                       | `party_struct.Species`             | `pokemon.baseSpecies`                  |
-| `pokemon.stats`                         | `box_struct.Stats`                 | `pokemon.baseStoredStats`              |
-| `active.stats`                          | `battle_struct.Stats`              | `pokemon.modifiedStats`                |
-| `volatiles.transform`                   | `PlayerMonUnmodified*`             | `pokemon.storedStats`                  |
-| `active.species`                        | `battle_struct.Species`            | `pokemon.species`                      |
-| `{pokemon,active}.types`                | `{party,battle}_struct.Type`       | `pokemon.types`                        |
-| `active.boosts`                         | `PlayerMon*Mod`                    | `pokemon.boosts`                       |
-| `active.volatiles`                      | `PlayerBattleStatus{1,2,3}`        | `pokemon.volatiles`                    |
-| `volatiles.state`                       | `PlayerBideAccumulatedDamage`      | `volatiles.bide.totalDamage`           |
-| `volatiles.attacks`                     | `PlayerNumAttacksLeft`             | `volatiles.{bide,lockedmove}.duration` |
-| `volatiles.confusion`                   | `PlayerConfusedCounter`            | `volatiles.confusion.duration`         |
-| `volatiles.toxic`                       | `PlayerToxicCounter`               | `volatiles.residualdmg.counter`        |
-| `volatiles.substitute`                  | `PlayerSubstituteHP`               | `volatiles.substitute.hp`              |
-| `volatiles.disabled_{move,duration}`    | `PlayerDisabledMove{,Number}`      | `volatiles.disable.{move,time}`        |
+| pkmn                                 | Pokémon Red (pret)                 | Pokémon Showdown                       |
+| ------------------------------------ | ---------------------------------- | -------------------------------------- |
+| `battle.seed`                        | `Random{Add,Sub}`                  | `battle.seed`                          |
+| `battle.turn`                        | -                                  | `battle.turn`                          |
+| `battle.last_damage`                 | `Damage`                           | `battle.lastDamage`                    |
+| `side.{active,pokemon}`              | `PlayerMonNumber`/`BattleMon`      | `side.active`                          |
+| `side.team`                          | `PartyMons`                        | `side.pokemon`                         |
+| `side.last_used_move`                | `PlayerUsedMove`                   | `pokemon.lastMove`                     |
+| `side.last_selected_move`            | `PlayerSelectedMove`               | `side.lastSelectedMove`                |
+| `battle.last_selected_indexes`       | `PlayerMoveListIndex`              | `side.lastMove`                        |
+| `side.order`                         | -                                  | `pokemon.position`                     |
+| `{pokemon,active}.moves`             | `{party,battle}_struct.{Moves,PP}` | `pokemon.{baseMoveSlots,moveSlots}`    |
+| `pokemon.hp`                         | `{party,battle}_struct.HP`         | `pokemon.hp`                           |
+| `{pokemon,active}.status`            | `{party,battle}_struct.Status`     | `pokemon.status`                       |
+| `{pokemon,active}.level`             | `PlayerMonUnmodifiedLevel`         | `pokemon.level`                        |
+| `pokemon.species`                    | `party_struct.Species`             | `pokemon.baseSpecies`                  |
+| `pokemon.stats`                      | `box_struct.Stats`                 | `pokemon.baseStoredStats`              |
+| `active.stats`                       | `battle_struct.Stats`              | `pokemon.modifiedStats`                |
+| `volatiles.transform`                | `PlayerMonUnmodified*`             | `pokemon.storedStats`                  |
+| `active.species`                     | `battle_struct.Species`            | `pokemon.species`                      |
+| `{pokemon,active}.types`             | `{party,battle}_struct.Type`       | `pokemon.types`                        |
+| `active.boosts`                      | `PlayerMon*Mod`                    | `pokemon.boosts`                       |
+| `active.volatiles`                   | `PlayerBattleStatus{1,2,3}`        | `pokemon.volatiles`                    |
+| `volatiles.state`                    | `PlayerBideAccumulatedDamage`      | `volatiles.bide.totalDamage`           |
+| `volatiles.attacks`                  | `PlayerNumAttacksLeft`             | `volatiles.{bide,lockedmove}.duration` |
+| `volatiles.confusion`                | `PlayerConfusedCounter`            | `volatiles.confusion.duration`         |
+| `volatiles.toxic`                    | `PlayerToxicCounter`               | `volatiles.residualdmg.counter`        |
+| `volatiles.substitute`               | `PlayerSubstituteHP`               | `volatiles.substitute.hp`              |
+| `volatiles.disabled_{move,duration}` | `PlayerDisabledMove{,Number}`      | `volatiles.disable.{move,time}`        |
 
 - Pokémon Showdown [does not implement the correct Generation I RNG](#rng) and as such its `seed` is
   different
@@ -69,9 +68,6 @@ The following information is required to simulate a Generation I Pokémon battle
   Counter and is not set and cleared in the correct locations. `Pokemon.lastMove` matches
   `PlayerUsedMove` more often so is what the engine attempts to model, despite the implications for
   Counter)
-- `battle.last_moves` doesn't map precisely to cartridge concepts because it is used to track desyncs
-  which instead are represented on the cartridge as discrepancies between the two separate battle states
-  that exist in the link battle.
 - Pokémon Showdown does not implement the [partial-trapping move Mirror Move
   glitch](https://glitchcity.wiki/Partial_trapping_move_Mirror_Move_link_battle_glitch) and as such
   does not need to keep track of a player's last selected move index (`PlayerMoveListIndex`)
@@ -102,8 +98,8 @@ The following information is required to simulate a Generation I Pokémon battle
 store general information about the battle. Unlike in Pokémon Showdown there is a distinction
 between the data structure for the "active" Pokémon and its party members (see below).
 
-Due to layout constraints, details about the last moves for a side are stored in what would
-otherwise be the padding bytes of`Battle` (the `last_moves` field) instead of in `Side`.
+Due to layout constraints, the index of the last selected move for a side is stored in what would
+otherwise be the padding bytes of`Battle` (the `last_selected_indexes` field) instead of in `Side`.
 
 ### `Pokemon` / `ActivePokemon`
 
@@ -202,7 +198,7 @@ above, PP information isn't strictly necessary in Generation I so is dropped. `M
 a special sentinel value to indicate `null`. Move PP data is only included for testing and is not
 necessary for the actual engine implementation.
 
-In order to workaround various [Pokémon Showdown bugs](#bugs) and to support its protocol in logs,
+In order to workaround various [Pokémon Showdown bugs](#bugs) and to support its protocol in traces,
 additional information is stored in `Move.Data` (`targets`) about what Pokémon Showdown believes the
 `Move.Target` to be (despite the concept of targeting not existing until Generation III when Double
 battles were introduced). More specifically, a move's "targeting" status is required in various
@@ -335,33 +331,30 @@ does not produce an equivalent to Pokémon Showdown's `|request|` protocol messa
 must be gleaned through the `Battle` state). Useful size and offset information can be found in
 the [`layout.json`](../../data/layout.json) which exists to simplify writing driver code.
 
-Documentation of the wire protocol used for protocol message logging when `-Dlog` is enabled can be
-found in [PROTOCOL.md](../../../docs/PROTOCOL.md). Note that the
+Documentation of the wire protocol used for logging traces when `-Dtrace` is enabled can be found in
+[PROTOCOL.md](../../../docs/PROTOCOL.md). Note that the
 [`pkmn-debug`](../../../README.md#pkmn-debug) tool exists to display the binary protocol and battle
 data in the browser for ease of debugging.
 
 ### `Battle`
 
-| Start   | End     | Data                | Description                                             |
-| ------- | ------- | ------------------- | ------------------------------------------------------- |
-| 0       | 184     | [`sides[0]`](#side) | Player 1's side                                         |
-| 184     | 368     | [`sides[1]`](#side) | Player 2's side                                         |
-| 368     | 370     | `turn`              | The current turn number                                 |
-| 370     | 372     | `last_damage`       | The last damage dealt by either side                    |
-| 372     | 373-376 | `last_moves`        | The slot index of the last selected moves for each side |
-| 373-376 | 384     | `rng`               | The RNG state                                           |
+| Start   | End     | Data                    | Description                                             |
+| ------- | ------- | ----------------------- | ------------------------------------------------------- |
+| 0       | 184     | [`sides[0]`](#side)     | Player 1's side                                         |
+| 184     | 368     | [`sides[1]`](#side)     | Player 2's side                                         |
+| 368     | 370     | `turn`                  | The current turn number                                 |
+| 370     | 372     | `last_damage`           | The last damage dealt by either side                    |
+| 372     | 373-376 | `last_selected_indexes` | The slot index of the last selected moves for each side |
+| 373-376 | 384     | `rng`                   | The RNG state                                           |
 
 - the current `turn` is 2 bytes, written in native-endianness
-- `last_moves` layout depends on whether or not Pokémon Showdown compatibility mode is enabled
-  (`-Dshowdown`). In either case it stores the last selected move index and last executed move for
-  each player (required to detect desyncs). In the case of the move index, 0 is used as an empty
-  value and 1-4 represent actual moves:
-  - if `showdown` is enabled byte 372 is used to store the last selected move index of `side[0]` and
-    byte 373 is used to store whether or not the side's last executed move would have been considered
-    "counterable"; and bytes 374 and 375 store the same information for `side[1]`
-  - otherwise the first 3 bits of byte 372 stores the last selected move index of `side[0]` and the
-    next bit stores whether that side's last executed move would have been considered "counterable",
-    with the same information for `side[1]`'s being stored in the second 4 bits
+- `last_selected_indexes` layout depends on whether or not Pokémon Showdown compatibility mode is
+  enabled (`-Dshowdown`). In either case it stores a move "slot", where 0 is used as an empty value and
+  1-4 represent actual moves:
+  - if `showdown` is enabled bytes 372 and 373 are used to store the last selected move index of
+    `side[0]` and bytes 374 and 375 store the last selected move index of `side[1]`
+  - otherwise byte 372 stores the last selected move index of `side[0]` in its first 4 bits and
+    `side[1]`'s in its second 4 bits
 - the `rng` depends on whether or not Pokémon Showdown compatibility mode is enabled (`-Dshowdown`):
   - if `showdown` is enabled, the RNG state begins on byte 376 and consists of a 64-bit seed,
     written in native-endianness
@@ -492,14 +485,12 @@ Beyond the general bugs listed above, several move effects are implemented incor
 Showdown. Some of these moves are [too fundamentally broken to be implemented](#unimplementable) by
 the pkmn engine, but the following moves have their broken behavior preserved in `-Dshowdown` mode:
 
-- **Bide**: Bide's damage can overflow if OHKO moves are involved because OHKO moves work by setting
-  the damage to 65535 - Pokémon Showdown doesn't implement this overflow and instead lets Bide's
-  damage grow unbounded. Additionally, if the opponent faints after Bide inflicts damage on Pokémon
-  Showdown residual damage incorrectly still gets applied to Bide's user.
-- **Counter**: On Pokémon Showdown choices made while sleeping (which should not have been
-  registered) can erroneously cause Counter to trigger Desync Clause Mod behavior.
+- **Bide**: If the opponent faints after Bide inflicts damage on Pokémon Showdown residual damage
+  incorrectly still gets applied to Bide's user.
 - **Leech Seed**: Leech Seed fails to heal its source side if a seeded target faints due to
-  recoil/crash damage on Pokémon Showdown.
+  recoil/crash damage on Pokémon Showdown. Pokémon Showdown also updates the battle's last damage
+  field with the uncapped Leech Seed damage if the residual damage is applied which has consequences
+  for Bide and Counter.
 - **Pay Day**: Pay Day should still scatter coins if it hits (but doesn't break) and opponent's
   Substitute but doesn't on Pokémon Showdown.
 - **Flinch**: Flinching does not get cleared during move selection on Pokémon Showdown and is
@@ -540,13 +531,9 @@ the pkmn engine, but the following moves have their broken behavior preserved in
   glitch](https://glitchcity.wiki/Substitute_%C2%BC_HP_glitch) also fails in many cases due to
   Pokémon Showdown implementing the health check based on floating point division instead of integer
   divison like on the cartridge (meaning the Substitute 1/4 glitch will only occur if the Pokémon's
-  maximum HP is evenly divisible by 4). Substitute also incorrectly blocks Dream Eater on
+  maximum HP is evenly divisible by 4). Finally, Substitute also incorrectly blocks Dream Eater on
   Pokémon Showdown and will incorrectly still heal 1 HP for any draining moves if the attack does 0
-  damage. Finally, Pokémon Showdown uses a `subFainted` field to track whether a Substitute was
-  broken in order to know when to nullify a move's effect, only it doesn't get cleared at the end of
-  the turn and can result in incorrect behavior on subsequent turns with the moves Mirror Move and
-  Metronome that invoke `runMove` (which is where `subFainted` gets cleared) on the user but skips
-  calling it for the eventual true target.
+  damage.
 
 In addition to numerous cases where Pokémon Showdown uses the wrong type of message (e.g. `|-fail`
 vs. `|-miss|` vs. `|-immune|`, e.g. in the case of Leech Seed) which are not documented here, Pokémon
@@ -616,9 +603,9 @@ the correct control flow):
   still gets a turn under the [trapping sleep
   glitch](https://glitchcity.wiki/Trapping_move_and_sleep_glitch), Wrap does 0 damage against
   Ghost-type Pokémon instead of properly respecting immunity, and binding effects are handled in the
-  wrong order in the code which results in either out of order messaging or, more consequentially,
-  Pokémon using Rage/Bide or Thrashing/Charging moves being incorrectly forced to skip a turn when
-  the Pokémon using the Binding move switches.
+  wrong order in the code.
+- **Counter**: On Pokémon Showdown choices made while sleeping (which should not have been
+  registered) can erroneously cause Counter to trigger Desync Clause Mod behavior.
 - **Mimic**: Pokémon Showdown checks that the user of Mimic has Mimic in one of their move slots,
   which means Mimic legally called via Metronome or Mirror Move will only work if the user also has
   Mimic (and the moved mimicked by Mimic called via Metronome / Mirror Move will erroneously
@@ -626,27 +613,19 @@ the correct control flow):
   because Pokémon Showdown deducts PP based on a move's name instead of slot, if Mimic copies a move
   the Pokémon already knows, PP deduction for using either the original move of the mimicked move
   will instead deduct PP for whichever appears at the lower move slot index and the PP will be
-  allowed to go negative (effectively allowing for infinite PP use). Pokémon Showdown also doesn't
-  deduct PP from using Transform if it was copied by Mimic.
-- **Mirror Move**: As covered above, both Substitute and Binding moves misbehave when used via
-  Mirror Move (though Pokémon Showdown has its own weird behavior and does not implement the
-  [partial trapping move Mirror Move
+  allowed to go negative (effectively allowing for infinite PP use).
+- **Mirror Move**: Binding moves misbehave when used via Mirror Move (though Pokémon Showdown has
+  its own weird behavior and does not implement the [partial trapping move Mirror Move
   glitch](https://glitchcity.wiki/Partial_trapping_move_Mirror_Move_link_battle_glitch) that exists
-  on the cartridge). Additionally, Pokémon Showdown sets the last used move every turn a Pokémon is
-  Thrashing instead of just on the turn it is actually selected meaning Mirror Move will sometimes
-  successfully mirror a Thrashing move when it should fail. Finally, if Mirror Move copies
-  Struggle it should not deduct PP but on Pokémon Showdown it does.
-- **Metronome**: In addition to the issues with binding moves and Substitute, Metronome and Mirror
-  Move cannot mutually call each other more than 3 times in a row without causing the Pokémon
-  Showdown simulator to crash due to defensive safety checks that do not exist on the cartridge.
-- **Transform**: Transform screws up the effect of Disable, because on Pokémon Showdown, Disable
+  on the cartridge).
+- **Metronome**: In addition to the issues with binding moves, Metronome and Mirror Move cannot
+  mutually call each other more than 3 times in a row without causing the Pokémon Showdown simulator
+  to crash due to defensive safety checks that do not exist on the cartridge.
+- **Transform**:  Transform screws up the effect of Disable, because on Pokémon Showdown, Disable
   prevents moves of a given *name* from being used (e.g. "Water Gun") as opposed to moves in a
   specific *slot* (e.g. the 2nd move slot), and a Pokémon's moves can change after Transform (this
-  is not an issue with Disable + Mimic because Mimic happens to replace the same slot). Transform
-  also has a bad interaction with Pokémon Showdown's buggy Haze implementation, as Haze on Pokémon
-  Showdown does not copy unmodified stats, leaving the Transformed Pokémon with incorrect stats.
-  Furthermore, transforming and then using Mirror Move / Metronome can result in [glitchy behavior
-  and
+  is not an issue with Disable + Mimic because Mimic happens to replace the same slot). Furthermore,
+  transforming and then using Mirror Move / Metronome can result in [glitchy behavior and
   softlocks](https://pkmn.cc/bulba/Transform_glitches#Transform_.2B_Mirror_Move.2FMetronome_PP_error)
   which Pokémon Showdown does not implement.
 

package/src/lib/gen1/data.zig

@@ -55,8 +55,8 @@ pub fn Battle(comptime RNG: anytype) type {
         turn: u16 = 0,
         /// The last damage dealt by either side.
         last_damage: u16 = 0,
-        /// Details about the last move selected/used by each player required to detect desyncs.
-        last_moves: MoveDetails = .{},
+        /// The slot index of the last selected moves for each side
+        last_selected_indexes: MoveIndexes = .{},
         /// The pseudo random number generator.
         rng: RNG,
 
@@ -76,7 +76,7 @@ pub fn Battle(comptime RNG: anytype) type {
         }
 
         /// Returns the result of applying Player 1's choice `c1` and Player 2's choice `c2` to the
-        /// battle, optionally writing protocol message logs to `log` if `options.log` is enabled.
+        /// battle, optionally writing protocol logs to `log` if `options.trace` is enabled.
         pub fn update(self: *Self, c1: Choice, c2: Choice, log: anytype) !Result {
             return mechanics.update(self, c1, c2, log);
         }
@@ -201,19 +201,17 @@ pub const MoveSlot = extern struct {
     }
 };
 
-/// Details required to detect desyncs based on the move last selected/executed by players.
-pub const MoveDetails = packed struct {
-    /// The move slot index of the last move selected in the battle menu by Player 1
-    p1_index: if (showdown) u8 else u3 = 0,
-    /// Whether the last move executed by Player 1 was counterable.
-    p1_counterable: if (showdown) u8 else u1 = 0,
-    /// The move slot index of the last move selected in the battle menu by Player 2
-    p2_index: if (showdown) u8 else u3 = 0,
-    /// Whether the last move executed by Player 2 was counterable.
-    p2_counterable: if (showdown) u8 else u1 = 0,
+const uindex = if (showdown) u16 else u4;
+
+/// Representation of the move slot indexes for each side in a Generation I Pokémon battle.
+pub const MoveIndexes = packed struct {
+    /// Player 1's move index.
+    p1: uindex = 0,
+    /// Player 1's move index.
+    p2: uindex = 0,
 
     comptime {
-        assert(@sizeOf(MoveDetails) == if (showdown) 4 else 1);
+        assert(@sizeOf(MoveIndexes) == if (showdown) 4 else 1);
     }
 };
 

package/src/lib/gen1/helpers.zig

@@ -55,12 +55,17 @@ pub const Battle = struct {
     }
 
     pub fn random(rand: *PSRNG, opt: Options) data.Battle(data.PRNG) {
-        return .{
+        var p1: u4 = 0;
+        var p2: u4 = 0;
+        var battle: data.Battle(data.PRNG) = .{
             .rng = prng(rand),
             .turn = 0,
             .last_damage = 0,
             .sides = .{ Side.random(rand, opt), Side.random(rand, opt) },
         };
+        battle.last_selected_indexes.p1 = p1;
+        battle.last_selected_indexes.p2 = p2;
+        return battle;
     }
 };
 
@@ -206,10 +211,16 @@ pub const Pokemon = struct {
     }
 };
 
+// cat src/test/showdown/blocklist.json | jq '."1".moves'
 fn blocked(m: Move) bool {
-    // Binding moves are borked but only via Mirror Move / Metronome which are already blocked
+    if (Move.get(m).effect == .Binding) return true;
     return switch (m) {
-        .Mimic, .Metronome, .MirrorMove, .Transform => true,
+        .Counter,
+        .Mimic,
+        .Metronome,
+        .MirrorMove,
+        .Transform,
+        => true,
         else => false,
     };
 }