feat: Add polyfill for Array.prototype.toSpliced method (#30031)

Co-authored-by: CodingOnStar <hanxujiang@dify.ai>
This commit is contained in:
Coding On Star 2025-12-23 17:48:20 +08:00 committed by GitHub
parent f2842da397
commit 72ca3607a3
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
2 changed files with 92 additions and 0 deletions

View File

@ -0,0 +1,78 @@
/**
* Tests for Array.prototype.toSpliced polyfill
*/
describe('toSpliced polyfill', () => {
let originalToSpliced: typeof Array.prototype.toSpliced
beforeEach(() => {
// Save original method
originalToSpliced = Array.prototype.toSpliced
})
afterEach(() => {
// Restore original method
// eslint-disable-next-line no-extend-native
Array.prototype.toSpliced = originalToSpliced
})
const applyPolyfill = () => {
// @ts-expect-error - intentionally deleting for test
delete Array.prototype.toSpliced
if (!Array.prototype.toSpliced) {
// eslint-disable-next-line no-extend-native
Array.prototype.toSpliced = function <T>(this: T[], start: number, deleteCount?: number, ...items: T[]): T[] {
const copy = this.slice()
copy.splice(start, deleteCount ?? copy.length - start, ...items)
return copy
}
}
}
it('should add toSpliced method when not available', () => {
applyPolyfill()
expect(typeof Array.prototype.toSpliced).toBe('function')
})
it('should return a new array without modifying the original', () => {
applyPolyfill()
const arr = [1, 2, 3, 4, 5]
const result = arr.toSpliced(1, 2)
expect(result).toEqual([1, 4, 5])
expect(arr).toEqual([1, 2, 3, 4, 5]) // original unchanged
})
it('should insert items at the specified position', () => {
applyPolyfill()
const arr: (number | string)[] = [1, 2, 3]
const result = arr.toSpliced(1, 0, 'a', 'b')
expect(result).toEqual([1, 'a', 'b', 2, 3])
})
it('should replace items at the specified position', () => {
applyPolyfill()
const arr: (number | string)[] = [1, 2, 3, 4, 5]
const result = arr.toSpliced(1, 2, 'a', 'b')
expect(result).toEqual([1, 'a', 'b', 4, 5])
})
it('should handle negative start index', () => {
applyPolyfill()
const arr = [1, 2, 3, 4, 5]
const result = arr.toSpliced(-2, 1)
expect(result).toEqual([1, 2, 3, 5])
})
it('should delete to end when deleteCount is omitted', () => {
applyPolyfill()
const arr = [1, 2, 3, 4, 5]
const result = arr.toSpliced(2)
expect(result).toEqual([1, 2])
})
})

View File

@ -1,5 +1,19 @@
'use client'
// Polyfill for Array.prototype.toSpliced (ES2023, Chrome 110+)
if (!Array.prototype.toSpliced) {
// eslint-disable-next-line no-extend-native
Array.prototype.toSpliced = function <T>(this: T[], start: number, deleteCount?: number, ...items: T[]): T[] {
const copy = this.slice()
// When deleteCount is undefined (omitted), delete to end; otherwise let splice handle coercion
if (deleteCount === undefined)
copy.splice(start, copy.length - start, ...items)
else
copy.splice(start, deleteCount, ...items)
return copy
}
}
class StorageMock {
data: Record<string, string>