sindresorhus/eslint-plugin-unicorn

`require-module-attributes`: Support `TSImportType`

Open

#2 726 ouverte le 1 sept. 2025

Voir sur GitHub
 (1 commentaire) (0 réactions) (0 assignés)JavaScript (5 022 stars) (468 forks)user submission
change requestgood for beginnerhelp wanted

Description

Description

It seems typescript doesn't allow

type A = import("foo", {})

But allows

type A = import("foo", {with: {}})

I didn't add support for it in #2725, if someone need it, it can reuse the same logic as ImportExpression

Examples

// ❌
type A = import("foo", {with: {}})

// ✅
type A = import("foo")

Additional Info

No response

Guide contributeur

Stack technique
typescriptjavascript
Domaine
toolingdeveloper experience
Type d'issue
feature
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
half day
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
fresh
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
clear
Prérequis
TypeScript AST knowledgeESLint rule developmentReading PR #2725
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
45
Direction de recherche
Review the existing implementation of `require module attributes` rule and the changes made in PR #2725 for `ImportExpression`. The logic for handling `TSImportType` should mirror that of `ImportExpression` but adapted for type only imports. Focus on files like `rules/require module attributes.js` and look for how `ImportExpression` is handled. The issue provides clear examples of what should be flagged and what should be allowed.