labels: prevent deps from getting "lib / src" label

Author: phillipj

lib/node-labels.js

@@ -104,7 +104,11 @@ function matchSubSystemsByRegex (rxLabelsMap, filepathsChanged, limitLib) {
 
     for (var i = 0; i < mappedSubSystems.length; ++i) {
       const mappedSubSystem = mappedSubSystems[i]
-      if (limitLib && jsSubsystemList.includes(mappedSubSystem)) {
+      const considerLimiting = hasJsSubsystemFilechanges(filepathsChanged) &&
+                                limitLib &&
+                                jsSubsystemList.includes(mappedSubSystem)
+
+      if (considerLimiting) {
         if (jsLabelCount.length >= 4) {
           for (const jsLabel of jsLabelCount) {
             delete map[jsLabel]
@@ -157,4 +161,8 @@ function matchesAnExclusiveLabel (filepath) {
   return mappedSubSystemsForFile(exclusiveLabelsMap, filepath) !== undefined
 }
 
+function hasJsSubsystemFilechanges (filepaths) {
+  return filepaths.some((filepath) => filepath.startsWith('lib/'))
+}
+
 exports.resolveLabels = resolveLabels

test/node-labels.test.js

@@ -157,6 +157,21 @@ tap.test('label: "lib / src" when 5 or more JS sub-systems have been changed', (
   t.end()
 })
 
+// https://github.com/nodejs/node/pull/7488 wrongfully labelled with "lib / src"
+tap.test('label: not "lib / src" when only deps have been changed', (t) => {
+  const labels = nodeLabels.resolveLabels([
+    'deps/v8/test/cctest/interpreter/bytecode_expectations/ArrayLiterals.golden',
+    'deps/v8/test/cctest/interpreter/bytecode_expectations/ArrayLiteralsWide.golden',
+    'deps/v8/test/cctest/interpreter/bytecode_expectations/AssignmentsInBinaryExpression.golden',
+    'deps/v8/test/cctest/interpreter/bytecode_expectations/BasicBlockToBoolean.golden',
+    'deps/v8/test/cctest/interpreter/bytecode_expectations/BasicLoops.golden'
+  ])
+
+  t.same(labels, ['v8'])
+
+  t.end()
+})
+
 tap.test('label: "JS sub-systems when less than 5 sub-systems have changed', (t) => {
   const labels = nodeLabels.resolveLabels([
     'lib/assert.js',