infra(unicorn): prefer-at (#2654)

.eslintrc.cjs

@@ -44,8 +44,6 @@ module.exports = defineConfig({
     'unicorn/number-literal-case': 'off', // incompatible with prettier
     'unicorn/prefer-ternary': 'off', // ternaries aren't always better
 
-    // TODO @Shinigami92 2023-09-23: prefer-at should be turned on when we drop support for Node 14.
-    'unicorn/prefer-at': 'off',
     // TODO @ST-DDT 2023-10-28: The following rule should be turned on when we switch to esm.
     'unicorn/prefer-top-level-await': 'off',
 

docs/guide/upgrading_v9/2654.md

@@ -0,0 +1,7 @@
+### Examples of code-upgrades that are now used
+
+_This upgrade is an extension to_ [#2121](./2121.md)
+
+Used Node >= v16.6 feature [`at`](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Array/at).
+
+We could hint users to use polyfill or transpile-down steps.

scripts/apidoc/parameter-defaults.ts

@@ -33,7 +33,7 @@ export const parameterDefaultReader: EventCallback = (
     reflection.kindOf(reflectionKindFunctionOrMethod) &&
     symbol.declarations?.length
   ) {
-    const lastDeclaration = symbol.declarations[symbol.declarations.length - 1];
+    const lastDeclaration = symbol.declarations.at(-1);
     if (TypeScript.isFunctionLike(lastDeclaration)) {
       (reflection as ParameterDefaultsAware).implementationDefaultParameters =
         lastDeclaration.parameters.map((param) =>

scripts/apidoc/typedoc.ts

@@ -123,7 +123,8 @@ export function selectApiSignature(
     throw new Error(`Method ${method.name} has no signature.`);
   }
 
-  return signatures[signatures.length - 1];
+  // eslint-disable-next-line @typescript-eslint/no-non-null-assertion
+  return signatures.at(-1)!;
 }
 
 /**
@@ -313,7 +314,8 @@ export function extractSummaryDefault(
 
   if (eraseDefault) {
     summary.splice(-2, 2);
-    const lastSummaryPart = summary[summary.length - 1];
+    // eslint-disable-next-line @typescript-eslint/no-non-null-assertion
+    const lastSummaryPart = summary.at(-1)!;
     lastSummaryPart.text = lastSummaryPart.text.replace(
       /[ \n]Defaults to $/,
       ''

scripts/generate-locales.ts

@@ -128,11 +128,11 @@ async function generateLocaleFile(locale: string): Promise<void> {
   }
 
   // TODO @Shinigami92 2023-03-07: Remove 'en' fallback in a separate PR
-  if (locales[locales.length - 1] !== 'en' && locale !== 'base') {
+  if (locales.at(-1) !== 'en' && locale !== 'base') {
     locales.push('en');
   }
 
-  if (locales[locales.length - 1] !== 'base') {
+  if (locales.at(-1) !== 'base') {
     locales.push('base');
   }
 

src/modules/helpers/index.ts

@@ -989,7 +989,8 @@ export class SimpleHelpersModule extends SimpleModuleBase {
     }
 
     // In case of rounding errors, return the last element
-    return array[array.length - 1].value;
+    // eslint-disable-next-line @typescript-eslint/no-non-null-assertion
+    return array.at(-1)!.value;
   }
 
   /**

test/modules/color.spec.ts

@@ -122,8 +122,8 @@ describe('color', () => {
             format: 'decimal',
             includeAlpha: true,
           });
-          expect(color[color.length - 1]).toBeGreaterThanOrEqual(0);
-          expect(color[color.length - 1]).toBeLessThanOrEqual(1);
+          expect(color.at(-1)).toBeGreaterThanOrEqual(0);
+          expect(color.at(-1)).toBeLessThanOrEqual(1);
           for (const value of color.slice(0, 4)) {
             expect(value).toBeGreaterThanOrEqual(0);
             expect(value).toBeLessThanOrEqual(255);

test/modules/date.spec.ts

@@ -362,7 +362,7 @@ describe('date', () => {
               expect(dates[i]).greaterThan(dates[i - 1]);
             }
 
-            expect(dates[dates.length - 1]).lessThan(to);
+            expect(dates.at(-1)).lessThan(to);
           }
         );
       });

test/modules/lorem.spec.ts

@@ -117,7 +117,7 @@ describe('lorem', () => {
 
           expect(actual).toBeTruthy();
           expect(actual).toBeTypeOf('string');
-          expect(actual[actual.length - 1]).toBe('.');
+          expect(actual.at(-1)).toBe('.');
         });
 
         it.each(times(25))(
@@ -127,7 +127,7 @@ describe('lorem', () => {
 
             expect(actual).toBeTruthy();
             expect(actual).toBeTypeOf('string');
-            expect(actual[actual.length - 1]).toBe('.');
+            expect(actual.at(-1)).toBe('.');
 
             const words = actual.split(' ');
 
@@ -182,15 +182,15 @@ describe('lorem', () => {
 
           expect(actual).toBeTruthy();
           expect(actual).toBeTypeOf('string');
-          expect(actual[actual.length - 1]).toBe('.');
+          expect(actual.at(-1)).toBe('.');
         });
 
         it.each(times(10))('should return %i sentences', (sentenceCount) => {
           const actual = faker.lorem.sentences(sentenceCount);
 
           expect(actual).toBeTruthy();
           expect(actual).toBeTypeOf('string');
-          expect(actual[actual.length - 1]).toBe('.');
+          expect(actual.at(-1)).toBe('.');
 
           const sentences = actual.split('. ');
 
@@ -205,14 +205,14 @@ describe('lorem', () => {
 
             expect(actual).toBeTruthy();
             expect(actual).toBeTypeOf('string');
-            expect(actual[actual.length - 1]).toBe('.');
+            expect(actual.at(-1)).toBe('.');
 
             const sentences = actual.split(separator);
 
             expect(sentences).toHaveLength(sentenceCount);
 
             for (const sentence of sentences) {
-              expect(sentence[sentence.length - 1]).toBe('.');
+              expect(sentence.at(-1)).toBe('.');
             }
           }
         );
@@ -236,7 +236,7 @@ describe('lorem', () => {
 
           expect(actual).toBeTruthy();
           expect(actual).toBeTypeOf('string');
-          expect(actual[actual.length - 1]).toBe('.');
+          expect(actual.at(-1)).toBe('.');
         });
 
         it.each(times(10))(
@@ -246,7 +246,7 @@ describe('lorem', () => {
 
             expect(actual).toBeTruthy();
             expect(actual).toBeTypeOf('string');
-            expect(actual[actual.length - 1]).toBe('.');
+            expect(actual.at(-1)).toBe('.');
 
             const sentences = actual.split('. ');
 
@@ -274,15 +274,15 @@ describe('lorem', () => {
 
           expect(actual).toBeTruthy();
           expect(actual).toBeTypeOf('string');
-          expect(actual[actual.length - 1]).toBe('.');
+          expect(actual.at(-1)).toBe('.');
         });
 
         it.each(times(5))('should return %i paragraphs', (paragraphCount) => {
           const actual = faker.lorem.paragraphs(paragraphCount);
 
           expect(actual).toBeTruthy();
           expect(actual).toBeTypeOf('string');
-          expect(actual[actual.length - 1]).toBe('.');
+          expect(actual.at(-1)).toBe('.');
 
           const paragraphs = actual.split('\n');
 
@@ -297,7 +297,7 @@ describe('lorem', () => {
 
             expect(actual).toBeTruthy();
             expect(actual).toBeTypeOf('string');
-            expect(actual[actual.length - 1]).toBe('.');
+            expect(actual.at(-1)).toBe('.');
 
             const paragraphs = actual.split(separator);
 

test/modules/system.spec.ts

@@ -271,7 +271,7 @@ describe('system', () => {
             'generated filePath should start with /'
           ).toBeTruthy();
           expect(
-            parts[parts.length - 1],
+            parts.at(-1),
             'generated filePath should have a file extension'
           ).toMatch(/^\w+\.\w+$/);
         });