dotansimha/graphql-code-generator-community

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

Open

#100 geöffnet am 17. Dez. 2020

Auf GitHub ansehen
 (3 Kommentare) (2 Reaktionen) (0 zugewiesene Personen)TypeScript (195 Forks)github user discovery
help wanted

Repository-Metriken

Stars
 (136 Stars)
PR-Merge-Metriken
 (PR-Metriken ausstehend)

Beschreibung

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

Contributor Guide

Research-Richtung
Untersuche die Importgenerierungslogik im typescript react apollo Plugin, insbesondere den Teil, der die Konfigurationsoptionen 'withHOC' und 'useTypeImports' behandelt. Der Fehler tritt auf, wenn beide true sind, was zu doppelten Importanweisungen führt. Suche die Stelle, an der Importe generiert werden, und stelle sicher, dass bei 'useTypeImports' true 'import type' verwendet wird, aber nicht, wenn bereits ein '*' Import vorhanden ist.
Tech Stack
typescriptnodejs
Domain
apideveloper experience
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
TypeScriptGraphQLGit
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70