Address PR comments

v4-client-js/examples/cancel_example.ts

@@ -21,6 +21,14 @@ async function test(): Promise<void> {
   console.log(client);
   const subaccount = new Subaccount(wallet, 0);
 
+  /*
+  Note this example places a stateful order.
+  Programmatic traders should generally not use stateful orders for following reasons:
+  - Stateful orders received out of order by validators will fail sequence number validation and be dropped.
+  - Stateful orders have worse time priority since they are only matched after they are included on the block.
+  - Stateful order rate limits are more restrictive than Short-Term orders, specifically max 2 per block / 20 per 100 blocks.
+  - Stateful orders can only be canceled after they’ve been included in a block.
+  */
   const longTermOrderClientId = randomInt(MAX_CLIENT_ID);
   try {
     // place a long term order

v4-client-js/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@dydxprotocol/v4-client-js",
-  "version": "0.38.1",
+  "version": "0.38.2",
   "description": "General client library for the new dYdX system (v4 decentralized)",
   "main": "build/src/index.js",
   "scripts": {

v4-client-js/src/clients/composite-client.ts

@@ -5,7 +5,7 @@ import { Order_ConditionType, Order_TimeInForce } from '@dydxprotocol/v4-proto/s
 import Long from 'long';
 import protobuf from 'protobufjs';
 
-import { isStatefulOrder } from '../lib/validation';
+import { isStatefulOrder, verifyOrderFlags } from '../lib/validation';
 import { OrderFlags } from '../types';
 import {
   DYDX_DENOM,
@@ -519,6 +519,10 @@ export class CompositeClient {
     const market = marketsResponse.markets[marketId];
     const clobPairId = market.clobPairId;
 
+    if (!verifyOrderFlags(orderFlags)) {
+      throw new Error('Invalid order flags: ' + orderFlags);
+    }
+
     let goodTilBlockTime;
     if (isStatefulOrder(orderFlags)) {
       if (goodTilTimeInSeconds === 0) {

v4-client-js/src/lib/validation.ts

@@ -131,6 +131,11 @@ function verifyNumberIsUint32(num: number): boolean {
   return num >= 0 && num <= MAX_UINT_32;
 }
 
+export function verifyOrderFlags(orderFlags: OrderFlags): boolean {
+  return orderFlags === OrderFlags.SHORT_TERM ||
+    orderFlags === OrderFlags.LONG_TERM || orderFlags === OrderFlags.CONDITIONAL;
+}
+
 export function isStatefulOrder(orderFlags: OrderFlags): boolean {
   return orderFlags === OrderFlags.LONG_TERM || orderFlags === OrderFlags.CONDITIONAL;
 }