Skip to content

Commit 1fd3127

Browse files
fix(eslint): remove stale suppressions from merged attachments code
ESLint --fix removes no-op suppression comments for rules not enabled in our config. These were introduced by the attachments PR (#176) merge and are cleaned up by the new ESLint 10 flat config. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent 961c035 commit 1fd3127

2 files changed

Lines changed: 2 additions & 2 deletions

File tree

cdk/src/handlers/shared/attachment-screening.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -238,7 +238,7 @@ const PDF_EXTRACT_TIMEOUT_MS = 15_000;
238238

239239
async function extractPdfText(content: Buffer, filename: string): Promise<string> {
240240
// Dynamic import — pdf-parse is only used for PDF attachments.
241-
// eslint-disable-next-line @typescript-eslint/no-explicit-any
241+
242242
let pdfParseFn: (data: Buffer, options?: { max?: number }) => Promise<{ text: string }>;
243243
try {
244244
// pdf-parse uses a default export; handle both CJS and ESM module shapes.

cdk/src/handlers/shared/validation.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -266,7 +266,7 @@ export const MAX_TOTAL_ATTACHMENT_SIZE_BYTES = 50 * 1024 * 1024;
266266
/** Compile-time exhaustiveness check for AttachmentType. */
267267
const ATTACHMENT_TYPE_LIST = ['image', 'file', 'url'] as const satisfies readonly AttachmentType[];
268268
type _AssertAttachmentExhaustive = Exclude<AttachmentType, (typeof ATTACHMENT_TYPE_LIST)[number]> extends never ? true : never;
269-
const _attachmentExhaustiveCheck: _AssertAttachmentExhaustive = true; // eslint-disable-line @typescript-eslint/no-unused-vars
269+
const _attachmentExhaustiveCheck: _AssertAttachmentExhaustive = true;
270270
const VALID_ATTACHMENT_TYPES = new Set<string>(ATTACHMENT_TYPE_LIST);
271271

272272
/** Allowed image MIME types (PNG and JPEG only — passed directly to Bedrock). */

0 commit comments

Comments
 (0)