Skip to content

Commit ba2c2ae

Browse files
LiangEeasonLiangWorldedtech
authored andcommitted
u917 mark safeWriteText as an internal direct-test seam
Knip 5.88 counts the export as unused production API because no production file calls it yet. The repository's own quality-checks document, docs/development-quality-checks.md line 36, tells an author in this position to declare the seam with @internal and its reason next to the export rather than in configuration, and scripts/knip-policy.test.mjs asserts that case. Six comment lines, the export line untouched, no change to any configuration file and no suppression added.
1 parent e77527f commit ba2c2ae

1 file changed

Lines changed: 6 additions & 0 deletions

File tree

‎src/services/file-safety/safeWriteText.ts‎

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -263,6 +263,12 @@ export async function resolveLockKey(absoluteFilePath: string): Promise<string>
263263
}
264264
}
265265

266+
/**
267+
* @internal Intentional direct-test seam: no production file calls this write path
268+
* yet, and this unit's own specs exercise it directly, which is the case
269+
* `docs/development-quality-checks.md` line 36 tells authors to declare here rather
270+
* than in configuration. Knip still follows the test imports in default mode.
271+
*/
266272
export async function safeWriteText(
267273
filePath: string,
268274
content: string | Uint8Array,

0 commit comments

Comments
 (0)