Drop the Buffer/Uint8Array injection workarounds

Both protobuf data converter features reached into the Node realm to overwrite
a global inside the workflow sandbox: binary_protobuf replaced Uint8Array, and
json_protobuf replaced Buffer. They date back to #286 and reference an unnamed
SDK bug around how `bytes` fields cross the sandbox boundary.

The SDK now normalizes protobufjs's `Buffer` allocations to `Uint8Array` when
decoding, so neither injection is needed; both features pass without them.

Author: mjameswh

features/data_converter/binary_protobuf/feature.ts

@@ -2,11 +2,6 @@ import * as assert from 'assert';
 import { Feature } from '@temporalio/harness';
 import * as proto from '@temporalio/proto';
 
-// Inject Buffer and Uint8Array from the node context to the workflow context to workaround SDK bug
-// TODO(antlai-temporal) Remove when SDK bug is fixed
-const g = globalThis as any;
-g.Uint8Array = g.constructor.constructor('return globalThis.Uint8Array')();
-
 const expectedResult = proto.temporal.api.common.v1.DataBlob.create({
   encodingType: proto.temporal.api.enums.v1.EncodingType.ENCODING_TYPE_UNSPECIFIED,
   data: new Uint8Array([0xde, 0xad, 0xbe, 0xef]),

features/data_converter/json_protobuf/feature.ts

@@ -9,11 +9,6 @@ import { decode } from '@temporalio/common/lib/encoding';
 const patched = patchProtobufRoot(proto) as any;
 const dataBlobType = patched.lookupType('temporal.api.common.v1.DataBlob');
 
-// Inject Buffer and Uint8Array from the node context to the workflow context to workaround SDK bug
-// TODO(antlai-temporal) Remove workaround when SDK bug is fixed
-const g = globalThis as any;
-g.Buffer = g.constructor.constructor('return globalThis.Buffer')();
-
 const expectedResult = proto.temporal.api.common.v1.DataBlob.create({
   encodingType: proto.temporal.api.enums.v1.EncodingType.ENCODING_TYPE_UNSPECIFIED,
   data: new Uint8Array([0xde, 0xad, 0xbe, 0xef]),