fix(pullAt): fix incorrect element removal with negative indices

src/array/pullAt.spec.ts

@@ -50,6 +50,30 @@ describe('pullAt', () => {
     expect(actual).toEqual([2, 4, 12, 8, 6, 10]);
   });
 
+  it('should work with a single negative index', () => {
+    const array = [10, 20, 30, 40, 50];
+    const result = pullAt(array, [-1]);
+
+    expect(array).toEqual([10, 20, 30, 40]);
+    expect(result).toEqual([50]);
+  });
+
+  it('should work with multiple negative indices', () => {
+    const array = [10, 20, 30, 40, 50];
+    const result = pullAt(array, [-1, -2]);
+
+    expect(array).toEqual([10, 20, 30]);
+    expect(result).toEqual([50, 40]);
+  });
+
+  it('should work with mixed positive and negative indices', () => {
+    const array = [10, 20, 30, 40, 50];
+    const result = pullAt(array, [0, -1]);
+
+    expect(array).toEqual([20, 30, 40]);
+    expect(result).toEqual([10, 50]);
+  });
+
   it('should work with objects', () => {
     const foo = { foo: 1 };
     const bar = { foo: 2 };

src/array/pullAt.ts

@@ -18,7 +18,7 @@ import { at } from './at.ts';
  */
 export function pullAt<T>(arr: T[], indicesToRemove: readonly number[]): T[] {
   const removed = at(arr, indicesToRemove);
-  const indices = new Set(indicesToRemove.slice().sort((x, y) => y - x));
+  const indices = new Set(indicesToRemove.map(index => (index < 0 ? index + arr.length : index)).sort((x, y) => y - x));
 
   for (const index of indices) {
     arr.splice(index, 1);