chore(avm): add SET serialize comments and simplify signature (#4476)

yarn-project/simulator/src/avm/opcodes/instruction.ts

@@ -1,4 +1,4 @@
-import { assert } from 'console';
+import { strict as assert } from 'assert';
 
 import type { AvmContext } from '../avm_context.js';
 import { BufferCursor } from '../serialization/buffer_cursor.js';
@@ -39,7 +39,7 @@ export abstract class Instruction {
    * @returns The serialized instruction.
    */
   public serialize(this: any): Buffer {
-    assert(this instanceof Instruction);
+    assert(!!this.constructor.wireFormat, 'wireFormat must be defined on the class');
     return serialize(this.constructor.wireFormat, this);
   }
 
@@ -50,12 +50,15 @@ export abstract class Instruction {
    * @param buf Buffer to read from.
    * @returns Constructed instance of Class.
    */
-  public static deserialize<T extends { new (...args: any[]): InstanceType<T>; wireFormat: OperandType[] }>(
-    this: T,
-    buf: BufferCursor | Buffer,
-  ): InstanceType<T> {
+  public static deserialize(this: InstructionConstructor, buf: BufferCursor | Buffer): Instruction {
+    assert(!!this.wireFormat, 'wireFormat must be defined on the instruction class');
     const res = deserialize(buf, this.wireFormat);
-    const args = res.slice(1) as ConstructorParameters<T>; // Remove opcode.
+    const args = res.slice(1); // Remove opcode.
     return new this(...args);
   }
 }
+
+type InstructionConstructor = {
+  new (...args: any[]): Instruction;
+  wireFormat?: OperandType[];
+};

yarn-project/simulator/src/avm/opcodes/memory.ts

@@ -43,6 +43,7 @@ export class Set extends Instruction {
     super();
   }
 
+  /** We need to use a custom serialize function because of the variable length of the value. */
   public serialize(): Buffer {
     const format: OperandType[] = [
       ...Set.wireFormatBeforeConst,
@@ -52,10 +53,8 @@ export class Set extends Instruction {
     return serialize(format, this);
   }
 
-  public static deserialize<T extends { new (...args: any[]): InstanceType<T> }>(
-    this: T,
-    buf: BufferCursor | Buffer,
-  ): InstanceType<T> {
+  /** We need to use a custom deserialize function because of the variable length of the value. */
+  public static deserialize(this: typeof Set, buf: BufferCursor | Buffer): Set {
     if (buf instanceof Buffer) {
       buf = new BufferCursor(buf);
     }