Revert "add --enable-experimental-byob-streams flag to turn the feature on"

This reverts commit dcd472a.

Author: Guy Bedford

c-dependencies/js-compute-runtime/c-at-e-world/c_at_e_world_component_type.o

@@ -115,15 +115,12 @@ bool init_js() {
   if (!js::UseInternalJobQueues(cx) || !JS::InitSelfHostedCode(cx))
     return false;
 
-  bool ENABLE_EXPERIMENTAL_BYOB_STREAMS =
-      std::string(std::getenv("ENABLE_EXPERIMENTAL_BYOB_STREAMS")) == "1";
-
   // TODO: check if we should set a different creation zone.
   JS::RealmOptions options;
   options.creationOptions()
       .setStreamsEnabled(true)
-      .setReadableByteStreamsEnabled(ENABLE_EXPERIMENTAL_BYOB_STREAMS)
-      .setBYOBStreamReadersEnabled(ENABLE_EXPERIMENTAL_BYOB_STREAMS)
+      .setReadableByteStreamsEnabled(true)
+      .setBYOBStreamReadersEnabled(true)
       .setReadableStreamPipeToEnabled(true)
       .setWritableStreamsEnabled(true)
       .setWeakRefsEnabled(JS::WeakRefSpecifier::EnabledWithoutCleanupSome);

c-dependencies/js-compute-runtime/js-compute-runtime.cpp

@@ -9,4 +9,4 @@ name = "byob"
 service_id = ""
 
 [scripts]
-  build = "node ../../../../js-compute-runtime-cli.js --enable-experimental-byob-streams"
+  build = "node ../../../../js-compute-runtime-cli.js"

integration-tests/js-compute/fixtures/byob/fastly.toml.in

@@ -5,7 +5,7 @@ import { printVersion } from "./src/printVersion.js";
 import { printHelp } from "./src/printHelp.js";
 import { addSdkMetadataField } from "./src/addSdkMetadataField.js";
 
-const {enableExperimentalByobStreams, wasmEngine, input, component, output, version, help} = await parseInputs(process.argv.slice(2))
+const {wasmEngine, input, component, output, version, help} = await parseInputs(process.argv.slice(2))
 
 if (version) {
   await printVersion();
@@ -19,7 +19,7 @@ if (version) {
   // it could be that the user is using an older version of js-compute-runtime
   // and a newer version does support the platform they are using.
   const {compileApplicationToWasm} = await import('./src/compileApplicationToWasm.js')
-  await compileApplicationToWasm(input, output, wasmEngine, enableExperimentalByobStreams);
+  await compileApplicationToWasm(input, output, wasmEngine);
   if (component) {
     const {compileComponent} = await import('./src/component.js');
     await compileComponent(output);

js-compute-runtime-cli.js

@@ -8,7 +8,7 @@ import { precompile } from "./precompile.js";
 import { bundle } from "./bundle.js";
 import { containsSyntaxErrors } from "./containsSyntaxErrors.js";
 
-export async function compileApplicationToWasm(input, output, wasmEngine, enableExperimentalByobStreams = false) {
+export async function compileApplicationToWasm(input, output, wasmEngine) {
   try {
     if (!(await isFile(input))) {
       console.error(
@@ -99,7 +99,6 @@ export async function compileApplicationToWasm(input, output, wasmEngine, enable
         shell: true,
         encoding: "utf-8",
         env: {
-          ENABLE_EXPERIMENTAL_BYOB_STREAMS: enableExperimentalByobStreams ? '1' : '0'
         }
       }
     );

src/compileApplicationToWasm.js

@@ -7,7 +7,6 @@ export async function parseInputs(cliInputs) {
   const __dirname = dirname(fileURLToPath(import.meta.url));
 
   let component = false;
-  let enableExperimentalByobStreams = false;
   let customEngineSet = false;
   let wasmEngine = join(__dirname, "../js-compute-runtime.wasm");
   let customInputSet = false;
@@ -21,10 +20,6 @@ export async function parseInputs(cliInputs) {
       case "--": {
         break loop;
       }
-      case "--enable-experimental-byob-streams": {
-        enableExperimentalByobStreams = true;
-        break;
-      }
       case "-V":
       case "--version": {
         return { version: true };
@@ -93,5 +88,5 @@ export async function parseInputs(cliInputs) {
       }
     }
   }
-  return { wasmEngine, component, input, output, enableExperimentalByobStreams };
+  return { wasmEngine, component, input, output };
 }

src/parseInputs.js

@@ -10,4 +10,4 @@ inputs=(
 )
 
 cat "${inputs[@]}" > "${script_dir}/wpt-test-runner.js"
-node "${script_dir}/../../js-compute-runtime-cli.js" --enable-experimental-byob-streams "${script_dir}/wpt-test-runner.js" wpt-runtime.wasm
+node "${script_dir}/../../js-compute-runtime-cli.js" "${script_dir}/wpt-test-runner.js" wpt-runtime.wasm