feat/add-oidc-for-codecov-bundle-analysis - Branch - kamiazya/web-csv-toolbox

feat: add OIDC authentication for Codecov bundle analysis

#606Merged

Comparing

feat/add-oidc-for-codecov-bundle-analysis

(

be32a11

) with

main

(

ac1c1a7

)

-59%

Improvements: 2

Regressions: 3

Untouched: 60

Skipped: 4

Benchmarks

Skipped (4)

Failed

Field length: 10 chars (1000 rows)

benchmark/main.ts

Regression

-59%

11.5 ms28.2 ms

parseString engine:fast (1000 rows)

benchmark/main.ts

Regression

-55%

13.8 ms30.5 ms

parseString engine:stable (50 rows)

benchmark/main.ts

Regression

-58%

1.1 ms2.5 ms

Improved

Engine comparison: mainThread (500 rows)

benchmark/main.ts

+73%

30.5 ms17.7 ms

Engine comparison: wasm (5000 rows)

benchmark/main.ts

+63%

184.2 ms113.3 ms

Passed

Concurrent: Parallel mainThread

benchmark/main.ts

+44%

51 ms35.4 ms

parseStringStream: medium (1000 rows)

benchmark/main.ts

+41%

245.8 ms174.2 ms

Stream approach: parseUint8ArrayStream (10KB)

benchmark/main.ts

+35%

48.1 ms35.7 ms

parseString.toArraySync(10000 cols, 10 rows, unquoted)

benchmark/main.ts

+16%

448.5 ms385.9 ms

Engine comparison: mainThread (5000 rows)

benchmark/main.ts

+15%

176.1 ms152.9 ms

Quote ratio: 50% (1000 rows)

benchmark/main.ts

+9%

13.2 ms12.1 ms

Stream approach: parseUint8ArrayStream (100KB)

benchmark/main.ts

+9%

340.9 ms314.1 ms

Worker perf: tiny (10 rows) - mainThread

benchmark/main.ts

+8%

379.5 µs351.2 µs

Scaling: 5000 rows

benchmark/main.ts

+8%

175 ms162.1 ms

parseString.toArraySync(10000 cols, 10 rows, quoted)

benchmark/main.ts

+7%

446.9 ms416.5 ms

parseBinary.toStream(50 rows)

benchmark/main.ts

+7%

2.1 ms2 ms

Queuing: medium (1000 rows) - default HWM

benchmark/main.ts

+6%

244.8 ms230.2 ms

Field length: 100 chars (1000 rows)

benchmark/main.ts

+6%

13.5 ms12.8 ms

Quote ratio: 25% (1000 rows)

benchmark/main.ts

+3%

12.1 ms11.8 ms

Queuing: small (100 rows) - default HWM

benchmark/main.ts

+2%

23.9 ms23.4 ms

Low-level: CSVLexer + CSVRecordAssembler (1000 rows)

benchmark/main.ts

+1%

28 ms27.6 ms

Binary approach: parseBinary (1MB)

benchmark/main.ts

+1%

377.9 ms373.2 ms

Binary approach: parseBinary (100KB)

benchmark/main.ts

+1%

36.1 ms35.7 ms

Queuing: large (10000 rows) - default HWM

benchmark/main.ts

2.3 s2.3 s

Line ending: LF (1000 rows)

benchmark/main.ts

8.9 ms8.8 ms

parseStringStream: small (100 rows)

benchmark/main.ts

19 ms18.9 ms

Custom delimiter: PSV (100 rows)

benchmark/main.ts

1.1 ms1.1 ms

Binary approach: parseBinary (10KB)

benchmark/main.ts

3.7 ms3.7 ms

Custom delimiter: TSV (100 rows)

benchmark/main.ts

1.1 ms1.1 ms

Binary approach: parseBinary (1KB)

benchmark/main.ts

463.1 µs462.2 µs

Worker perf: large (10000 rows) - mainThread

benchmark/main.ts

195.6 ms195.3 ms

Quote ratio: 0% (1000 rows)

benchmark/main.ts

11.6 ms11.5 ms

parseString.toArraySync(50 cols, 50 rows, quoted)

benchmark/main.ts

18.6 ms18.6 ms

Line ending: CRLF (1000 rows)

benchmark/main.ts

17.9 ms17.8 ms

Field length: 10KB (100 rows)

benchmark/main.ts

16.7 ms16.7 ms

parseString.toArraySync(10 cols, 50 rows, quoted)

benchmark/main.ts

3.7 ms3.7 ms

parseString engine:fast (50 rows)

benchmark/main.ts

1.3 ms1.3 ms

parseStringToArraySyncWASM(50 rows)

benchmark/main.ts

1.5 ms1.5 ms

Worker perf: medium (1000 rows) - mainThread

benchmark/main.ts

19.7 ms19.7 ms

Memory: toIterableIterator (streaming)

benchmark/main.ts

27.7 ms27.8 ms

parseString.toIterableIterator(50 rows)

benchmark/main.ts

1.4 ms1.4 ms

Data transformation: no conversion (100 rows)

benchmark/main.ts

2.6 ms2.6 ms

Data transformation: type conversion (100 rows)

benchmark/main.ts

2.6 ms2.6 ms

parseBinary.toIterableIterator(50 rows)

benchmark/main.ts

1.1 ms1.1 ms

parseBinary.toArraySync(50 rows)

benchmark/main.ts

1.1 ms1.1 ms

parseString.toStream(50 rows)

benchmark/main.ts

-1%

2.3 ms2.3 ms

Engine comparison: wasm (500 rows)

benchmark/main.ts

-1%

9.1 ms9.1 ms

Concurrent: Sequential mainThread

benchmark/main.ts

-1%

35.2 ms35.7 ms

Memory: toArraySync (allocate all at once)

benchmark/main.ts

-1%

27.5 ms27.9 ms

Worker perf: small (100 rows) - mainThread

benchmark/main.ts

-1%

2.1 ms2.1 ms

Stream approach: parseUint8ArrayStream (1MB)

benchmark/main.ts

-2%

3.1 s3.1 s

Field length: 1KB (1000 rows)

benchmark/main.ts

-2%

26.7 ms27.3 ms

Low-level: CSVLexer only (1000 rows)

benchmark/main.ts

-2%

17.6 ms18 ms

Stream approach: parseUint8ArrayStream (1KB)

benchmark/main.ts

-2%

4.4 ms4.5 ms

Quote ratio: 100% (1000 rows)

benchmark/main.ts

-7%

12.6 ms13.5 ms

Scaling: 50 rows

benchmark/main.ts

-12%

1.5 ms1.7 ms

parseString engine:stable (1000 rows)

benchmark/main.ts

-12%

18 ms20.4 ms

Scaling: 100 rows

benchmark/main.ts

-12%

2.9 ms3.4 ms

parseString.toArraySync(100 cols, 50 rows, quoted)

benchmark/main.ts

-13%

37.3 ms42.9 ms

Scaling: 1000 rows

benchmark/main.ts

-14%

29.5 ms34.4 ms

Scaling: 500 rows

benchmark/main.ts

-15%

14.9 ms17.4 ms

parseString.toArraySync(100 cols, 50 rows, unquoted)

benchmark/main.ts

-17%

16.7 ms20.1 ms

parseString.toArraySync(10 cols, 50 rows, unquoted)

benchmark/main.ts

-17%

1.8 ms2.1 ms

parseString.toArraySync(50 cols, 50 rows, unquoted)

benchmark/main.ts

-17%

8.3 ms10.1 ms

parseString.toArraySync(50 rows)

benchmark/main.ts

-20%

2.1 ms2.7 ms

Commits

Click on a commit to change the comparison range

Base

main

ac1c1a7

×2.2

fix: add id-token write permission to reusable workflow calls Add explicit id-token: write permission to build, dynamic_tests, and examples jobs in main.yaml to allow nested jobs to use OIDC authentication. This fixes the error: "The nested job 'build_typescript' is requesting 'id-token: write', but is only allowed 'id-token: read'." 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>

1bcff35

10 hours ago

by kamiazya

fix: restore CODECOV_TOKEN support with OIDC fallback Restore CODECOV_TOKEN as a fallback authentication method while keeping OIDC support. This hybrid approach ensures bundle analysis works regardless of the authentication method configured: - Token authentication: Uses uploadToken when CODECOV_TOKEN is available - OIDC authentication: Uses GitHub OIDC when token is not available - Coverage uploads: Automatically chooses between token and OIDC Configuration logic: - enableBundleAnalysis: process.env.CI === 'true' (always enabled in CI) - uploadToken: Set only if CODECOV_TOKEN exists - useGitHubOIDC: true only when CODECOV_TOKEN is not available This fixes the issue where bundle reports were not being uploaded after removing CODECOV_TOKEN entirely. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>

be32a11

10 hours ago

by kamiazya

Home Terms Privacy Docs