dotansimha/graphql-code-generator-community

typescript-react-apollo: Invalid TypeScript output when both withHOC and useTypeImports are true

Open

#100 aberto em 17 de dez. de 2020

Ver no GitHub
 (3 comments) (2 reactions) (0 assignees)TypeScript (195 forks)github user discovery
help wanted

Métricas do repositório

Stars
 (136 stars)
Métricas de merge de PR
 (Métricas PR pendentes)

Description

Describe the bug

When using typescript-react-apollo plugin, and enabling both withHOC and useTypeImports output is not a valid TypeScript file:

import * as ApolloReactHoc from '@apollo/client/react/hoc';
import type * as ApolloReactHoc from '@apollo/client/react/hoc';

To Reproduce Steps to reproduce the behavior:

  1. Open live-demo
  2. Choose the Operations types template
  3. Change the config to one below
  1. My GraphQL schema:
# Default from demo
  1. My GraphQL operations:
# Default from demo
  1. My codegen.yml config file:
generates:
  types-and-hooks.tsx:
    config:
      withHOC: true
      useTypeImports: true
    plugins:
      - typescript
      - typescript-react-apollo

Expected behavior

Only this line should be in the output:

import * as ApolloReactHoc from '@apollo/client/react/hoc';

Actual behavoir Both lines are in the output:

import * as ApolloReactHoc from '@apollo/client/react/hoc';
import type * as ApolloReactHoc from '@apollo/client/react/hoc';

Environment: Using the live-demo reproduces it

  • OS:
  • @graphql-codegen/...:
  • NodeJS:

Additional context

Guia do colaborador

Direção de pesquisa
Investigue a lógica de geração de importação no plugin typescript react apollo, especificamente a parte que lida com as opções de configuração 'withHOC' e 'useTypeImports'. O bug ocorre quando ambas são verdadeiras, causando declarações de importação duplicadas. Procure onde as importações são geradas e garanta que quando 'useTypeImports' for verdadeiro, use 'import type' mas não quando já houver uma importação com '*'.
Pilha de tecnologia
typescriptnodejs
Domain
apideveloper experience
Tipo Issue
Bug
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.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
TypeScriptGraphQLGit
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
70