sindresorhus/eslint-plugin-unicorn

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

Open

#2.726 aberto em 1 de set. de 2025

Ver no GitHub
 (1 comment) (0 reactions) (0 assignees)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

Guia do colaborador

Pilha de tecnologia
typescriptjavascript
Domain
toolingdeveloper experience
Tipo Issue
feature
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
half day
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
fresh
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
clear
Prerequisites
TypeScript AST knowledgeESLint rule developmentReading PR #2725
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
45
Direção de pesquisa
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.