tinyhttp / tinyhttp / 21558930052

99%

push

github

refactor(accepts): inline negotiator dependency (#479)

* refactor(accepts): inline negotiator dependency

- Add packages/accepts/src/negotiator.ts with full TypeScript implementation
- Remove negotiator and @types/negotiator dependencies
- Add comprehensive test suite (44 tests) for negotiator functionality
- Reduces external dependencies by 2 packages

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* refactor(accepts): use # for private fields

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* refactor(accepts): remove unused BaseSpec interface

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* refactor(accepts): deduplicate compare functions

Replace 4 identical compareXxxSpecs functions with single compareByQI.
525 → 513 lines.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* refactor(accepts): modernize negotiator.ts with ES2022+ features

- Use flatMap instead of for loops with push for parse functions
- Use find for q-value extraction instead of for loops
- Use reduce for priority calculation functions
- Use template literals for string concatenation
- Use regex match for quoteCount function
- Use ternary expressions where cleaner

Reduces file from 513 to 430 lines (~16% reduction)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* style(accepts): apply Biome formatting

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* chore: add changeset for inlined negotiator

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* fix(accepts): fix split functions and improve test coverage

- Fix splitMediaTypes to truncate array after merging quoted sections
- Fix splitParameters similarly
- Add test for mismatched media type parameters
- Add test for comma inside quoted parameter
- Add test for semicolon inside quoted parameter

Achieves 100% line coverage for negotiator.ts

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* test(accepts): add more tests for full coverage

- Add array h... (continued)

935 of 1016 branches covered (92.03%)

Branch coverage included in aggregate %.

174 of 174 new or added lines in 1 file covered. (100.0%)

1298 of 1311 relevant lines covered (99.01%)

218.88 hits per line