refactor: remove unused code and superfluous test arguments in loader bridge

[Copilot]

Addressed code review feedback on commit 8533ab7 identifying unused code duplication and unnecessary test parameters.

Changes

• Removed unused collectNamedExports function: Duplicated logic from resolveCssModules, never called
• Cleaned up pitch loader test calls: Created type-safe callPitch helper to omit unused previousRequest and data parameters

The pitch function signature requires three parameters per webpack's PitchLoaderDefinitionFunction interface, but the implementation only uses remainingRequest. Tests now use a helper that maintains type safety while omitting the unused parameters:

// Before: passing unused parameters
const result = await pitch.call(
  ctx as LoaderContext<KnightedCssBridgeLoaderOptions>,
  './styles.module.css?knighted-css',
  '',  // previousRequest - unused
  {},  // data - unused
)

// After: type-safe helper omits unused parameters
const result = await callPitch(
  ctx as LoaderContext<KnightedCssBridgeLoaderOptions>,
  './styles.module.css?knighted-css',
)

---

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

[knightedcodemonkey]

closing in favor of #64