Skip to content

Commit

Permalink
Move the default implied on date for nodejs-compat-v2
Browse files Browse the repository at this point in the history
Push back the date where nodejs-compat implies nodejs-compat-v2
  • Loading branch information
jasnell committed Aug 27, 2024
1 parent bee639d commit ff7872b
Show file tree
Hide file tree
Showing 2 changed files with 11 additions and 4 deletions.
13 changes: 10 additions & 3 deletions src/workerd/io/compatibility-date-test.c++
Original file line number Diff line number Diff line change
Expand Up @@ -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,"
Expand Down Expand Up @@ -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,"
Expand Down Expand Up @@ -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);
}

Expand Down
2 changes: 1 addition & 1 deletion src/workerd/io/compatibility-date.capnp
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down

0 comments on commit ff7872b

Please sign in to comment.