module: skip NODE_COMPILE_CACHE when policy is enabled

It might be worth designing a policy for the compilation cache. For
now, just skip the cache when policy is enabled.
PR-URL: #52577
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Chengzhong Wu <legendecas@gmail.com>

Author: joyeecheung

src/env.cc

@@ -1096,6 +1096,12 @@ void Environment::InitializeCompileCache() {
  dir_from_env.empty()) {
 return;
  }
+if (!options()->experimental_policy.empty()) {
+Debug(this,
+ DebugCategory::COMPILE_CACHE,
+"[compile cache] skipping cache because policy is enabled");
+return;
+ }
 auto handler = std::make_unique<CompileCacheHandler>(this);
 if (handler->InitializeDirectory(this, dir_from_env)) {
  compile_cache_handler_ = std::move(handler);

test/parallel/test-compile-cache-policy.js

@@ -0,0 +1,38 @@
+'use strict';
+
+// This tests NODE_COMPILE_CACHE is disabled when policy is used.
+
+constcommon=require('../common');
+if(!common.hasCrypto)
+common.skip('missing crypto');
+
+const{ spawnSyncAndAssert }=require('../common/child_process');
+constassert=require('assert');
+constfs=require('fs');
+consttmpdir=require('../common/tmpdir');
+constfixtures=require('../common/fixtures');
+
+{
+tmpdir.refresh();
+constdir=tmpdir.resolve('.compile_cache_dir');
+constscript=fixtures.path('policy','parent.js');
+constpolicy=fixtures.path(
+'policy',
+'dependencies',
+'dependencies-redirect-policy.json');
+spawnSyncAndAssert(
+process.execPath,
+['--experimental-policy',policy,script],
+{
+env: {
+ ...process.env,
+NODE_DEBUG_NATIVE: 'COMPILE_CACHE',
+NODE_COMPILE_CACHE: dir
+},
+cwd: tmpdir.path
+},
+{
+stderr: /skippingcachebecausepolicyisenabled/
+});
+assert(!fs.existsSync(dir));
+}