From 00bcee1cb82ac24bc05050e8047469de2c717a94 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Olivier=20Fl=C3=BCckiger?= Date: Wed, 19 Aug 2026 07:30:06 +0000 Subject: [PATCH] src: remove --harmony-import-attributes MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Import attributes have been enabled by default since V8 M123 and are now standard in JavaScript. Remove the hardcoded injection of the flag in Node.js so that the flag can be deprecated and removed upstream in V8. Refs: https://chromium-review.googlesource.com/c/v8/v8/+/8260695 Signed-off-by: Olivier Flückiger --- src/node.cc | 7 ------- 1 file changed, 7 deletions(-) diff --git a/src/node.cc b/src/node.cc index 00282634419d..ce0beb021361 100644 --- a/src/node.cc +++ b/src/node.cc @@ -759,13 +759,6 @@ static ExitCode ProcessGlobalArgsInternal(std::vector* args, return ExitCode::kInvalidCommandLineArgument2; } - // TODO(aduh95): remove this when the harmony-import-attributes flag - // is removed in V8. - if (std::ranges::find(v8_args, "--no-harmony-import-attributes") == - v8_args.end()) { - v8_args.emplace_back("--harmony-import-attributes"); - } - if (!per_process::cli_options->per_isolate->max_old_space_size_percentage .empty()) { v8_args.emplace_back(