@natlibfi/marc-record-validators-melinda 11.6.6 → 11.6.7-alpha.1

package/src/field-structure.js

@@ -104,8 +104,8 @@ export default function (config) {
         configMatchesSpec(val, key, confSpec);
 
         // Concat all excluded elements to array
-        if (confSpec[key].excl) { // eslint-disable-line functional/no-conditional-statements
-          excluded.push(...confSpec[key].excl); // eslint-disable-line functional/immutable-data
+        if (confSpec[key].excl) {
+          excluded.push(...confSpec[key].excl);
         }
       });
 
@@ -132,9 +132,9 @@ export default function (config) {
     }
 
     // Check subfields/dependencies recursively
-    if (key === 'subfields' || key === 'dependencies') { // eslint-disable-line functional/no-conditional-statements
+    if (key === 'subfields' || key === 'dependencies') {
       forEach(data, ([, subObj]) => {
-        if (typeof subObj === 'object') { // eslint-disable-line functional/no-conditional-statements
+        if (typeof subObj === 'object') {
           forEach(subObj, ([subKey, subVal]) => {
             configMatchesSpec(subVal, subKey, key === 'subfields' ? subSpec : depSpec);
           });
@@ -198,7 +198,7 @@ export default function (config) {
       // Check that subfield stuff
       if (confField === 'subfields') {
         const strict = confObj.strict || false; // Defaults to false
-        let elementsTotal = 0; // eslint-disable-line functional/no-let
+        let elementsTotal = 0;
         const valid = !Object.entries(confObj.subfields).some(([key, val]) => {
           const matching = recordSubObj.subfields.filter(({code}) => code === key);
           elementsTotal += matching.length; // Calculate amount of record objects matching all confObj objects

package/src/fields-present.js

@@ -15,7 +15,7 @@ export default function (tagPatterns) {
       return result ? undefined : pattern;
     });
     const isEmpty = missingFields.every(index => index === undefined);
-    let errorMessage = ['The following tag patterns are not present in the record tag field: ']; // eslint-disable-line functional/no-let
+    let errorMessage = ['The following tag patterns are not present in the record tag field: '];
     errorMessage = errorMessage.concat(missingFields).join(' ');
 
     return isEmpty ? {valid: true, messages: []} : {valid: false, messages: [errorMessage]};

package/src/fields-present.spec.js

@@ -51,7 +51,7 @@ describe('fields-present', () => {
       expect(result).to.eql({valid: true, messages: []});
     });
     it('Finds the record valid', async () => {
-      const tagPatterns = [/^(020|022|024)$/u]; // eslint-disable-line
+      const tagPatterns = [/^(020|022|024)$/u];
       const validator = await validatorFactory(tagPatterns);
       const record = new MarcRecord({
         fields: [

package/src/fix-33X.js

@@ -376,7 +376,7 @@ export default function () {
     nvdebug(`FIX ${description}...`);
     const catLang = getCatalogingLanguage(record) || 'fin';
     const fields = getRelevantFields(record);
-    fields.forEach(f => fixField(f, catLang));
+    fields.forEach(f => fixField(f, catLang)); // eslint-disable-line array-callback-return
     nvdebug(` GOT ${fields.length}...`);
     // FFS: we actually need newFields array here! Videogame, for example, might be
     // 336 ## ‡a kaksiulotteinen liikkuva kuva ‡b tdi ‡2 rdacontent
@@ -395,7 +395,7 @@ export default function () {
     }
     const originalStrings = fields.map(f => fieldToString(f));
     const clonedFields = fields.map(f => clone(f));
-    clonedFields.forEach(f => fixField(f, catLang));
+    clonedFields.forEach(f => fixField(f, catLang)); // eslint-disable-line array-callback-return
     const modifiedStrings = clonedFields.map(f => fieldToString(f));
 
     const changes = originalStrings.map((str, i) => `'${str}' => '${modifiedStrings[i]}'`);
@@ -467,9 +467,9 @@ export default function () {
       return;
     }
 
-    subfieldA.value = newTerm; // eslint-disable-line functional/immutable-data
-    field.subfields.push({'code': 'b', 'value': code}); // eslint-disable-line functional/immutable-data
-    field.subfields.push(tagToSubfield2(field.tag)); // eslint-disable-line functional/immutable-data
+    subfieldA.value = newTerm;
+    field.subfields.push({'code': 'b', 'value': code});
+    field.subfields.push(tagToSubfield2(field.tag));
 
     function codeToTerm(code) {
       if (field.tag === '336') {

package/src/fix-country-codes.js

@@ -44,9 +44,9 @@ export default function () {
       return res;
     }
 
-    res.message.push(`Modify 008/15-17: '${originalCountryCode}' => '${modifiedCountryCode}'`); // eslint-disable-line functional/immutable-data
+    res.message.push(`Modify 008/15-17: '${originalCountryCode}' => '${modifiedCountryCode}'`);
 
-    res.valid = false; // eslint-disable-line functional/immutable-data
+    res.valid = false;
     return res;
   }
 
@@ -54,7 +54,7 @@ export default function () {
     const originalCountryCode = getCountryCodeFromField008(field008);
     const modifiedCountryCode = deprecatedCountryCodeToCurrentCountryCode(originalCountryCode);
     if (originalCountryCode !== modifiedCountryCode && modifiedCountryCode.length === 3) {
-      field008.value = `${field008.value.substring(0, 15)}${modifiedCountryCode}${field008.value.substring(18)}`; // eslint-disable-line functional/immutable-data
+      field008.value = `${field008.value.substring(0, 15)}${modifiedCountryCode}${field008.value.substring(18)}`;
       return;
     }
   }

package/src/fix-language-codes.js

@@ -25,7 +25,7 @@ export default function () {
     fixLanguageField008(field008);
 
     const f041 = record.get('041');
-    f041.forEach(f => fixField041(f));
+    f041.forEach(f => fixField041(f)); // eslint-disable-line array-callback-return
 
     return res;
   }
@@ -37,9 +37,10 @@ export default function () {
     validateField008(record, res);
 
     const f041 = record.get('041');
-    f041.forEach(f => validateField041(f, res));
 
-    res.valid = !(res.message.length >= 1); // eslint-disable-line functional/immutable-data
+    f041.forEach(f => validateField041(f, res)); // eslint-disable-line array-callback-return
+
+    res.valid = !(res.message.length >= 1);
     return res;
   }
 
@@ -56,8 +57,8 @@ export default function () {
       return;
     }
 
-    res.message.push(`Modify 008/35-37: '${originalLanguage}' => '${modifiedLanguage}'`); // eslint-disable-line functional/immutable-data
-    res.valid = false; // eslint-disable-line functional/immutable-data
+    res.message.push(`Modify 008/35-37: '${originalLanguage}' => '${modifiedLanguage}'`);
+    res.valid = false;
   }
 
 
@@ -78,7 +79,7 @@ export default function () {
       return;
     }
     const msg = `${originalString} => ${modifiedString}`;
-    res.message.push(msg); // eslint-disable-line functional/immutable-data
+    res.message.push(msg);
 
   }
 
@@ -87,7 +88,7 @@ export default function () {
     const originalLanguage = getLanguageFromField008(field008);
     const modifiedLanguage = deprecatedLanguageToCurrentLanguage(originalLanguage);
     if (originalLanguage !== modifiedLanguage && modifiedLanguage.length === 3) {
-      field008.value = `${field008.value.substring(0, 35)}${modifiedLanguage}${field008.value.substring(38)}`; // eslint-disable-line functional/immutable-data
+      field008.value = `${field008.value.substring(0, 35)}${modifiedLanguage}${field008.value.substring(38)}`;
       return;
     }
   }
@@ -97,13 +98,13 @@ export default function () {
       return;
     }
 
-    field.subfields.forEach(sf => fixField041Subfield(sf));
+    field.subfields.forEach(sf => fixField041Subfield(sf)); // eslint-disable-line array-callback-return
 
     function fixField041Subfield(subfield) {
       if (!isRelevantField041SubfieldCode(subfield)) {
         return;
       }
-      subfield.value = deprecatedLanguageToCurrentLanguage(subfield.value); // eslint-disable-line functional/immutable-data
+      subfield.value = deprecatedLanguageToCurrentLanguage(subfield.value);
     }
   }
 

package/src/fixRelatorTerms.js

@@ -42,12 +42,12 @@ export default function () {
       fieldFixRelatorTerms(field, language, language);
       const clonedFieldAsString = fieldToString(clonedField);
       const fieldAsString = fieldToString(field);
-      if (fieldAsString !== clonedFieldAsString) { // eslint-disable-line functional/no-conditional-statements
-        res.message.push(`${fieldAsString} => ${clonedFieldAsString}`); // eslint-disable-line functional/immutable-data
+      if (fieldAsString !== clonedFieldAsString) {
+        res.message.push(`${fieldAsString} => ${clonedFieldAsString}`);
       }
     });
 
-    res.valid = !(res.message.length >= 1); // eslint-disable-line functional/immutable-data
+    res.valid = !(res.message.length >= 1);
     return res;
   }
 }
@@ -139,7 +139,7 @@ function subfieldHandleRelatorTermAbbreviation(subfield, language) {
       const hit = `${finnishAbbreviations[lcValue]}${punc}`;
       nvdebug(`Relator hit: ${hit}`, debugDev);
       // NB! 'esitt.' => 'esittäjä'
-      subfield.value = hit; // eslint-disable-line functional/immutable-data
+      subfield.value = hit;
       return;
     }
   }
@@ -157,7 +157,7 @@ function fieldHandleRelatorTermAbbreviations(field, language) {
   }
 
   const originalValue = fieldToString(field);
-  field.subfields.forEach(sf => subfieldHandleRelatorTermAbbreviation(sf, language));
+  field.subfields.forEach(sf => subfieldHandleRelatorTermAbbreviation(sf, language)); // eslint-disable-line array-callback-return
   const modifiedValue = fieldToString(field);
   if (modifiedValue === originalValue) {
     return;
@@ -212,7 +212,7 @@ function subfieldTranslateRelatorTerm(subfield, fromLanguage, toLanguage) {
   if (subfield.code !== 'e') {
     return;
   }
-  subfield.value = translateRelatorTerm(subfield.value, fromLanguage, toLanguage); // eslint-disable-line functional/immutable-data
+  subfield.value = translateRelatorTerm(subfield.value, fromLanguage, toLanguage);
 }
 
 export function fieldFixRelatorTerms(field, fromLanguage, toLanguage) {
@@ -222,7 +222,7 @@ export function fieldFixRelatorTerms(field, fromLanguage, toLanguage) {
   }
   fieldHandleRelatorTermAbbreviations(field, fromLanguage);
 
-  field.subfields.forEach(sf => subfieldTranslateRelatorTerm(sf, fromLanguage, toLanguage));
+  field.subfields.forEach(sf => subfieldTranslateRelatorTerm(sf, fromLanguage, toLanguage)); // eslint-disable-line array-callback-return
 }
 
 
@@ -230,7 +230,7 @@ export function recordFixRelatorTerms(record, defaultToLanguage = null, defaultF
   const fromLanguage = defaultFromLanguage ? defaultFromLanguage : getCatalogingLanguage(record);
   const toLanguage = defaultToLanguage ? defaultToLanguage : getCatalogingLanguage(record);
 
-  record.fields.forEach(field => translateField(field, fromLanguage, toLanguage));
+  record.fields.forEach(field => translateField(field, fromLanguage, toLanguage)); // eslint-disable-line array-callback-return
 
   function translateField(field, from, to) {
     fieldFixRelatorTerms(field, from, to);

package/src/fixed-fields.js

@@ -11,11 +11,11 @@ export default function (configuration) {
 
   function validate(record) {
     const messages = configuration.reduce((messages, item) => {
-      let results; // eslint-disable-line functional/no-let
+      let results;
 
-      if (item.leader) { // eslint-disable-line functional/no-conditional-statements
+      if (item.leader) {
         results = validateField(record.leader, item);
-      } else { // eslint-disable-line functional/no-conditional-statements
+      } else {
         results = record.get(item.tag)
           .map(f => validateField(f.value, item, f.tag));
       }

package/src/identical-fields.js

@@ -23,13 +23,13 @@ export default function () {
   function fix(record) {
     record.fields
       .filter(tag => !uniqWith(record.fields).includes(tag))
-      .forEach(tag => record.removeField(tag));
+      .forEach(tag => record.removeField(tag)); // eslint-disable-line array-callback-return
   }
 
   function uniqWith(fields) {
     return fields.reduce((uniq, field) => {
-      if (!uniq.some(f => MarcRecord.isEqual(f, field))) { // eslint-disable-line functional/no-conditional-statements
-        uniq.push(field); // eslint-disable-line functional/immutable-data
+      if (!uniq.some(f => MarcRecord.isEqual(f, field))) {
+        uniq.push(field);
       }
 
       return uniq;

package/src/indicator-fixes.js

@@ -25,7 +25,7 @@ export default function () {
 
     validateRecord(record, res);
 
-    res.valid = res.message.length < 1; // eslint-disable-line functional/immutable-data
+    res.valid = res.message.length < 1;
     return res;
   }
 
@@ -35,22 +35,22 @@ export default function () {
     const clonedFields = JSON.parse(JSON.stringify(record.fields));
     recordNormalizeIndicators(record);
 
-    record.fields.forEach((field, index) => compareFields(field, index));
+    record.fields.forEach((field, index) => compareFields(field, index)); // eslint-disable-line array-callback-return
 
     function compareFields(field, index) {
       const origFieldAsString = fieldToString(clonedFields[index]);
       //const clonedFieldAsString = fieldToString(field);
-      if (clonedFields[index].ind1 !== field.ind1) { // eslint-disable-line functional/no-conditional-statements
+      if (clonedFields[index].ind1 !== field.ind1) {
         //nvdebug(`FIX IND1: '${clonedFields[index].ind1}' => '${field.ind1}': ${clonedFieldAsString}`);
-        res.message.push(`Expected IND1 for '${origFieldAsString}' is '${field.ind1}'`); // eslint-disable-line functional/immutable-data
+        res.message.push(`Expected IND1 for '${origFieldAsString}' is '${field.ind1}'`);
       }
-      if (clonedFields[index].ind2 !== field.ind2) { // eslint-disable-line functional/no-conditional-statements
+      if (clonedFields[index].ind2 !== field.ind2) {
         //nvdebug(`FIX IND2: '${clonedFields[index].ind2}' => '${field.ind2}': ${clonedFieldAsString}`);
-        res.message.push(`Expected IND2 for '${origFieldAsString}' is '${field.ind2}'`); // eslint-disable-line functional/immutable-data
+        res.message.push(`Expected IND2 for '${origFieldAsString}' is '${field.ind2}'`);
       }
     }
     // Validator should not change the original record:
-    record.fields = clonedFields; // eslint-disable-line functional/immutable-data
+    record.fields = clonedFields;
     return;
   }
 }
@@ -133,7 +133,7 @@ function normalizeNonFilingIndicator1(field, languages = []) {
     return;
   }
 
-  field.ind1 = determineNonFilingIndicatorValue(field, languages); // eslint-disable-line functional/immutable-data
+  field.ind1 = determineNonFilingIndicatorValue(field, languages);
 }
 
 function normalizeNonFilingIndicator2(field, languages = []) {
@@ -141,7 +141,7 @@ function normalizeNonFilingIndicator2(field, languages = []) {
     return;
   }
 
-  field.ind2 = determineNonFilingIndicatorValue(field, languages); // eslint-disable-line functional/immutable-data
+  field.ind2 = determineNonFilingIndicatorValue(field, languages);
 }
 
 const fiktiivisenAineistonLisaluokatFI = ['Eläimet', 'Erotiikka', 'Erä', 'Fantasia', 'Historia', 'Huumori', 'Jännitys', 'Kauhu', 'Novellit', 'Romantiikka', 'Scifi', 'Sota', 'Urheilu', 'Uskonto'];
@@ -158,7 +158,7 @@ function normalize084Indicator1(field) {
 
   // https://marc21.kansalliskirjasto.fi/bib/05X-08X.htm#084 and https://finto.fi/ykl/fi/page/fiktioluokka
   if (field.ind1 !== '9' && containsFiktiivisenAineistonLisaluokka(field) && field.subfields.some(sf => sf.code === '2' && sf.value === 'ykl')) {
-    field.ind1 = '9'; // eslint-disable-line functional/immutable-data
+    field.ind1 = '9';
     return;
   }
 }
@@ -168,7 +168,7 @@ function normalize245Indicator1(field, record) {
     return;
   }
   const field1XX = record.get('^1..$');
-  field.ind1 = field1XX.length === 0 ? '0' : '1'; // eslint-disable-line functional/immutable-data
+  field.ind1 = field1XX.length === 0 ? '0' : '1';
 }
 
 function normalize776Indicator2(field) {
@@ -177,7 +177,7 @@ function normalize776Indicator2(field) {
   }
   // If subfield $i exists, ind2 must me '8'
   if (field.subfields.some(sf => sf.code === 'i')) {
-    field.ind2 = '8'; // eslint-disable-line functional/immutable-data
+    field.ind2 = '8';
     return;
   }
 }
@@ -193,13 +193,13 @@ function recordNormalize490(record) {
   if (fields490.length <= fields8XX.length) {
     // Trace found for each field 490:
     fields490.forEach(f => {
-      f.ind1 = '1'; // eslint-disable-line functional/immutable-data
+      f.ind1 = '1';
     });
     return;
   }
   if (fields8XX.length === 0) { // Fields 490 are always untraced (no traces found)
     fields490.forEach(f => {
-      f.ind1 = '0'; // eslint-disable-line functional/immutable-data
+      f.ind1 = '0';
     });
     return;
   }
@@ -235,7 +235,7 @@ export function recordNormalizeIndicators(record) {
   // Language is used to handle non-filing indicators
   const languages = getLanguages(record);
 
-  record.fields.forEach(field => fieldNormalizeIndicators(field, record, languages));
+  record.fields.forEach(field => fieldNormalizeIndicators(field, record, languages)); // eslint-disable-line array-callback-return
 
 }
 

package/src/isbn-issn.js

@@ -173,14 +173,14 @@ export default ({hyphenateISBN = false, handleInvalid = false} = {}) => {
   function fix(record) {
     getRelevantFields(record).forEach(field => {
       if (field.tag === '020') {
-        field.subfields.forEach(subfield => fixField020Subfield(field, subfield));
+        field.subfields.forEach(subfield => fixField020Subfield(field, subfield)); // eslint-disable-line array-callback-return
         return;
       }
       // 022 ISSN:
       const subfield = field.subfields.find(sf => sf.code === 'a' || sf.code === 'l');
-      if (subfield && handleInvalid) { // eslint-disable-line functional/no-conditional-statements
+      if (subfield && handleInvalid) {
         // $a/$l => $y (bit overkill to add $z and remove $a/$l instead of just renaming)
-        field.subfields.push({code: 'y', value: subfield.value}); // eslint-disable-line functional/immutable-data
+        field.subfields.push({code: 'y', value: subfield.value});
         record.removeSubfield(subfield, field);
       }
     });
@@ -198,8 +198,8 @@ export default ({hyphenateISBN = false, handleInvalid = false} = {}) => {
         }
         // ISBN is valid but is missing hyphens
         const normalizedValue = normalizeIsbnValue(subfield.value);
-        if (normalizedValue !== undefined) { // eslint-disable-line functional/no-conditional-statements
-          subfield.value = normalizedValue; // eslint-disable-line functional/immutable-data
+        if (normalizedValue !== undefined) {
+          subfield.value = normalizedValue;
         }
       }
 
@@ -212,7 +212,7 @@ export default ({hyphenateISBN = false, handleInvalid = false} = {}) => {
           return;
         }
         // $a => $z (bit overkill to add $z and remove $a instead of just renaming, but too lazy to fix/test thorougly)
-        field.subfields.push({code: 'z', value: subfield.value}); // eslint-disable-line functional/immutable-data
+        field.subfields.push({code: 'z', value: subfield.value});
         record.removeSubfield(subfield, field);
       }
 
@@ -231,8 +231,8 @@ export default ({hyphenateISBN = false, handleInvalid = false} = {}) => {
           return;
         }
         const tail = value.substring(position + 1);
-        subfield.value = head; // eslint-disable-line functional/immutable-data
-        field.subfields.push({code: 'q', value: tail}); // eslint-disable-line functional/immutable-data
+        subfield.value = head;
+        field.subfields.push({code: 'q', value: tail});
       }
 
       function normalizeIsbnValue(value) {
@@ -244,10 +244,10 @@ export default ({hyphenateISBN = false, handleInvalid = false} = {}) => {
         }
         const numbersOnly = trimmedValue.replace(/[^0-9Xx]+/ug, '');
         const parsedIsbn = ISBN.parse(trimmedValue);
-        if (hyphenateISBN) { // eslint-disable-line functional/no-conditional-statements
-          return numbersOnly.length === 10 ? parsedIsbn.isbn10h : parsedIsbn.isbn13h; // eslint-disable-line functional/immutable-data
+        if (hyphenateISBN) {
+          return numbersOnly.length === 10 ? parsedIsbn.isbn10h : parsedIsbn.isbn13h;
         }
-        return numbersOnly.length === 10 ? parsedIsbn.isbn10 : parsedIsbn.isbn13; // eslint-disable-line functional/immutable-data
+        return numbersOnly.length === 10 ? parsedIsbn.isbn10 : parsedIsbn.isbn13;
       }
     }
   }

package/src/item-language.js

@@ -49,24 +49,24 @@ export default async function (tagPattern, treshold = 0.9) {
     }
 
     if (results.detected && results.detected !== results.currentCode) {
-      const f008 = record.get(/^008$/u).shift(); // eslint-disable-line functional/immutable-data
+      const f008 = record.get(/^008$/u).shift();
 
-      if (f008) { // eslint-disable-line functional/no-conditional-statements
+      if (f008) {
         const start = f008.value.slice(0, 35);
         const end = f008.value.slice(38);
-        f008.value = `${start}${results.detected}${end}`; // eslint-disable-line functional/immutable-data
+        f008.value = `${start}${results.detected}${end}`;
       }
 
-      const f041 = record.get(/^041$/u).shift(); // eslint-disable-line functional/immutable-data
+      const f041 = record.get(/^041$/u).shift();
 
       if (f041) {
         const subfield = f041.subfields.find(sf => sf.code === 'a');
 
-        if (subfield) { // eslint-disable-line functional/no-conditional-statements
-          subfield.value = results.detected; // eslint-disable-line functional/immutable-data
-        } else { // eslint-disable-line functional/no-conditional-statements
-          f041.subfields.push({code: 'a', value: results.detected}); // eslint-disable-line functional/immutable-data
-          f041.subfields.sort((a, b) => { // eslint-disable-line functional/immutable-data
+        if (subfield) {
+          subfield.value = results.detected;
+        } else {
+          f041.subfields.push({code: 'a', value: results.detected});
+          f041.subfields.sort((a, b) => {
             if (a.code < b.code) {
               return -1;
             }
@@ -78,7 +78,7 @@ export default async function (tagPattern, treshold = 0.9) {
             return 0;
           });
         }
-      } else { // eslint-disable-line functional/no-conditional-statements
+      } else {
         record.insertField({tag: '041', ind1: ' ', ind2: ' ', subfields: [
           {
             code: 'a',

package/src/merge-fields/counterpartField.js

@@ -49,8 +49,8 @@ function differentPublisherSubfields(field1, field2) {
 
 export function splitToNameAndQualifier(value) {
   if (value.match(/^.* \([^()]+\)$/u)) {
-    const name = value.replace(/^(.*) \([^()]+\)$/u, '$1'); // eslint-disable-line prefer-named-capture-group
-    const qualifier = value.replace(/^.* (\([^()]+\))$/u, '$1'); // eslint-disable-line prefer-named-capture-group
+    const name = value.replace(/^(.*) \([^()]+\)$/u, '$1');
+    const qualifier = value.replace(/^.* (\([^()]+\))$/u, '$1');
     return [name, qualifier];
   }
   return [value, undefined];
@@ -189,7 +189,7 @@ function pairableValue(tag, subfieldCode, value1, value2) {
 
 
 function counterpartExtraNormalize(tag, subfieldCode, value) {
-  /* eslint-disable prefer-named-capture-group, no-param-reassign */
+
   // Remove trailing punctuation:
   value = value.replace(/(\S)(?:,|\.|\?|!|\. -| *:| *;| =| \/)$/u, '$1');
   // Remove brackets:
@@ -540,7 +540,7 @@ function fieldToNamePart(field) {
   const subsetField = {'tag': field.tag, 'ind1': field.ind1, 'ind2': field.ind2, subfields: relevantSubfields};
 
   /*
-  if (index > -1) { // eslint-disable-line functional/no-conditional-statements
+  if (index > -1) {
     debugDev(`Name subset: ${fieldToString(subsetField)}`);
   }
   */
@@ -624,7 +624,7 @@ function mergablePairWithAltName(normCandField, normalizedField, altName, config
   if (!a) {
     return false;
   }
-  a.value = altName; // eslint-disable-line functional/immutable-data
+  a.value = altName;
 
   return mergablePair(normCandField, normalizedField, config);
 }
@@ -683,9 +683,9 @@ function getCounterpartCandidates(field, record) {
 }
 
 export function baseIsSource(base, source) {
-  base.localTest = true; // eslint-disable-line functional/immutable-data
+  base.localTest = true;
   const result = source.localTest;
-  delete base.localTest; // eslint-disable-line functional/immutable-data
+  delete base.localTest;
   return result;
 }
 

package/src/merge-fields/index.js

@@ -29,7 +29,7 @@ export default function () {
     //const candFields = record.fields.toReversed(); // Node 20+ only! Filter via config?
     const fields = config && config.tagPattern ? record.get(config.tagPattern) : record.get(/^[1678](?:00|10|11|30)$/u);
 
-    fields.reverse(); // eslint-disable-line functional/immutable-data
+    fields.reverse();
     const mergedField = fields.find(f => mergeField(record, record, f, config));
     if (!mergedField) {
       return;
@@ -41,9 +41,9 @@ export default function () {
 
   function fix(record, config = undefined) {
     const config2 = config || defaultConfig;
-    record.internalMerge = true; // eslint-disable-line functional/immutable-data
+    record.internalMerge = true;
     mergeFieldsWithinRecord(record, config2);
-    delete record.internalMerge; // eslint-disable-line functional/immutable-data
+    delete record.internalMerge;
     // Remove deleted fields and field.merged marks:
     postprocessRecords(record, record);
 

package/src/merge-fields/mergeConstraints.js

@@ -303,7 +303,7 @@ export function getMergeConstraintsForTag(tag, constraintName) {
     return null;
   }
   // NB! should we support multiple contains for a field? Eg. 505$a vs 505($tg)+
-  if (tagsConstraintsArray.length > 1) { // eslint-disable-line functional/no-conditional-statements
+  if (tagsConstraintsArray.length > 1) {
     debugDev(`WARNING\tMultiple values for '${constraintName}' (N=${tagsConstraintsArray.length}) found in ${tag}. Using first values.`);
   }
   return constraintToValue(tagsConstraintsArray[0], constraintName);