From de1071d0b13bc88805c83bc9e59bad5e10c39eec Mon Sep 17 00:00:00 2001 From: Gergo Magyar Date: Mon, 15 Jun 2026 18:17:09 +0000 Subject: [PATCH] refactor(lbug): drop the unused WriteStreamFactory re-export (#2215 review) The type was re-exported from lbug-adapter 'to preserve this module's surface,' but no external code imports it by name from here (the only test reference is a comment). Keep the import from rel-pair-routing.ts (its canonical home, still used by splitRelCsvByLabelPair's signature) and drop the dead re-export. Co-Authored-By: Claude Opus 4.8 (1M context) --- gitnexus/src/core/lbug/lbug-adapter.ts | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/gitnexus/src/core/lbug/lbug-adapter.ts b/gitnexus/src/core/lbug/lbug-adapter.ts index 6b5d9c8fa..457c49175 100644 --- a/gitnexus/src/core/lbug/lbug-adapter.ts +++ b/gitnexus/src/core/lbug/lbug-adapter.ts @@ -49,11 +49,9 @@ import { logger } from '../logger.js'; // --------------------------------------------------------------------------- // Relationship CSV splitting — extracted for testability (PR #818) // --------------------------------------------------------------------------- - -/** Factory for creating WriteStreams — injectable for testing. Canonical - * definition lives in rel-pair-routing.ts (imported above for local use by - * splitRelCsvByLabelPair); re-exported here to preserve this module's surface. */ -export type { WriteStreamFactory }; +// WriteStreamFactory is imported above from rel-pair-routing.ts (its canonical +// home) for splitRelCsvByLabelPair's signature; no external code imports it from +// here, so it is not re-exported. /** Result of splitting the relationship CSV into per-label-pair files. */ export interface RelCsvSplitResult {