From 917977ff9b1d6a68457c75d739d24494140fdf28 Mon Sep 17 00:00:00 2001 From: James M Snell Date: Mon, 26 Aug 2024 17:37:19 -0700 Subject: [PATCH] Move the default implied on date for nodejs-compat-v2 Push back the date where nodejs-compat implies nodejs-compat-v2 --- src/workerd/io/compatibility-date-test.c++ | 13 ++++++++++--- src/workerd/io/compatibility-date.capnp | 2 +- 2 files changed, 11 insertions(+), 4 deletions(-) diff --git a/src/workerd/io/compatibility-date-test.c++ b/src/workerd/io/compatibility-date-test.c++ index 6f2c9d5ef34..535e4e6339d 100644 --- a/src/workerd/io/compatibility-date-test.c++ +++ b/src/workerd/io/compatibility-date-test.c++ @@ -179,7 +179,7 @@ KJ_TEST("compatibility flag parsing") { CompatibilityDateValidation::FUTURE_FOR_TEST, true, false); // nodejs_compat implies nodejs_compat_v2 on or after 2024-08-05 - expectCompileCompatibilityFlags("2024-09-02", {"nodejs_compat"}, + expectCompileCompatibilityFlags("2024-09-23", {"nodejs_compat"}, "(formDataParserSupportsFiles = true," " fetchRefusesUnknownProtocols = true," " esiIncludeIsVoidTag = false," @@ -236,7 +236,7 @@ KJ_TEST("compatibility flag parsing") { " allowCustomPorts = true," " internalWritableStreamAbortClearsQueue = true)", {}, CompatibilityDateValidation::FUTURE_FOR_TEST, false, false); - expectCompileCompatibilityFlags("2024-09-01", {"nodejs_compat"}, + expectCompileCompatibilityFlags("2024-09-22", {"nodejs_compat"}, "(formDataParserSupportsFiles = true," " fetchRefusesUnknownProtocols = true," " esiIncludeIsVoidTag = false," @@ -289,7 +289,14 @@ KJ_TEST("compatibility flag parsing") { " fetchStandardUrl = true," " nodeJsCompatV2 = false," " globalFetchStrictlyPublic = false," - " newModuleRegistry = false)", + " newModuleRegistry = false," + " cacheOptionEnabled = false," + " kvDirectBinding = false," + " allowCustomPorts = true," + " increaseWebsocketMessageSize = false," + " internalWritableStreamAbortClearsQueue = true," + " pythonWorkersDevPyodide = false," + " nodeJsZlib = false)", {}, CompatibilityDateValidation::FUTURE_FOR_TEST, false, false); } diff --git a/src/workerd/io/compatibility-date.capnp b/src/workerd/io/compatibility-date.capnp index 24851cf1d6c..c863303d80b 100644 --- a/src/workerd/io/compatibility-date.capnp +++ b/src/workerd/io/compatibility-date.capnp @@ -486,7 +486,7 @@ struct CompatibilityFlags @0x8f8c1b68151b6cef { nodeJsCompatV2 @50 :Bool $compatEnableFlag("nodejs_compat_v2") $compatDisableFlag("no_nodejs_compat_v2") - $impliedByAfterDate(name = "nodeJsCompat", date = "2024-09-02"); + $impliedByAfterDate(name = "nodeJsCompat", date = "2024-09-23"); # Implies nodeJSCompat with the following additional modifications: # * Node.js Compat built-ins may be imported/required with or without the node: prefix # * Node.js Compat the globals Buffer and process are available everywhere