chore: enforce types imports (#237)

.eslintrc

@@ -4,7 +4,7 @@
     "ecmaVersion": 12,
     "sourceType": "module"
   },
-  "plugins": ["@typescript-eslint"],
+  "plugins": ["@typescript-eslint", "import"],
   "extends": [
     "eslint:recommended",
     "plugin:@typescript-eslint/recommended",
@@ -14,7 +14,9 @@
   ],
   "rules": {
     "@typescript-eslint/no-var-requires": "off",
-    "@typescript-eslint/no-explicit-any": "off"
+    "@typescript-eslint/no-explicit-any": "off",
+    "@typescript-eslint/consistent-type-imports": ["error", { "fixStyle": "separate-type-imports" }],
+    "import/consistent-type-specifier-style": ["error", "prefer-top-level"]
   },
   "env": {
     "browser": true,

examples/next-app/app/[locale]/client/layout.tsx

@@ -1,4 +1,4 @@
-import { ReactNode } from 'react';
+import type { ReactNode } from 'react';
 import { Provider } from '../provider';
 
 export default function Layout({ params: { locale }, children }: { params: { locale: string }; children: ReactNode }) {

examples/next-app/app/[locale]/layout.tsx

@@ -1,4 +1,4 @@
-import { ReactElement } from 'react';
+import type { ReactElement } from 'react';
 import './globals.css';
 import { Switch } from './switch';
 import Link from 'next/link';

examples/next-app/app/[locale]/provider.tsx

@@ -1,6 +1,6 @@
 'use client';
 
-import { ReactNode } from 'react';
+import type { ReactNode } from 'react';
 import { I18nProviderClient } from '../../locales/client';
 
 type ProviderProps = {

examples/next-app/middleware.ts

@@ -1,5 +1,5 @@
 import { createI18nMiddleware } from 'next-international/middleware';
-import { NextRequest } from 'next/server';
+import type { NextRequest } from 'next/server';
 
 const I18nMiddleware = createI18nMiddleware({
   locales: ['en', 'fr'],

examples/next-pages/pages/_app.tsx

@@ -1,5 +1,5 @@
 import React from 'react';
-import { AppProps } from 'next/app';
+import type { AppProps } from 'next/app';
 import { I18nProvider } from '../locales';
 import en from '../locales/en';
 

examples/next-pages/pages/ssr-ssg.tsx

@@ -1,4 +1,4 @@
-import { GetServerSideProps } from 'next';
+import type { GetServerSideProps } from 'next';
 import { getLocaleProps, useChangeLocale, useCurrentLocale, useI18n, useScopedI18n } from '../locales';
 
 export const getServerSideProps: GetServerSideProps = getLocaleProps();

package.json

@@ -29,6 +29,7 @@
     "@vitest/coverage-v8": "^0.32.2",
     "eslint": "^8.47.0",
     "eslint-config-prettier": "^9.0.0",
+    "eslint-plugin-import": "^2.28.1",
     "eslint-plugin-prettier": "^5.0.0",
     "eslint-plugin-react": "^7.30.1",
     "eslint-plugin-react-hooks": "5.0.0-canary-7118f5dd7-20230705",

packages/next-international/src/app/client/create-i18n-provider-client.tsx

@@ -1,8 +1,8 @@
-import React, { Context, ReactNode, Suspense, use, useMemo } from 'react';
 import type { BaseLocale, ImportedLocales } from 'international-types';
-
-import type { LocaleContext } from '../../types';
+import type { Context, ReactNode } from 'react';
+import React, { Suspense, use, useMemo } from 'react';
 import { flattenLocale } from '../../common/flatten-locale';
+import type { LocaleContext } from '../../types';
 
 type I18nProviderProps = Omit<I18nProviderWrapperProps, 'fallback'>;
 

packages/next-international/src/app/client/create-use-current-locale.ts

@@ -1,7 +1,7 @@
 import { notFound, useParams } from 'next/navigation';
 import { useMemo } from 'react';
 import { DEFAULT_SEGMENT_NAME } from '../../common/constants';
-import { I18nClientConfig } from '../../types';
+import type { I18nClientConfig } from '../../types';
 import { error } from '../../helpers/log';
 
 export function createUseCurrentLocale<LocalesKeys>(locales: LocalesKeys[], config: I18nClientConfig) {

packages/next-international/src/app/middleware/index.ts

@@ -1,4 +1,5 @@
-import { NextRequest, NextResponse } from 'next/server';
+import type { NextRequest } from 'next/server';
+import { NextResponse } from 'next/server';
 
 import { LOCALE_COOKIE, LOCALE_HEADER } from '../../common/constants';
 import type { I18nMiddlewareConfig } from '../../types';

packages/next-international/src/app/server/create-get-i18n.ts

@@ -1,8 +1,8 @@
 import type { BaseLocale, ImportedLocales } from 'international-types';
 import { createT } from '../../common/create-t';
-import { I18nServerConfig, LocaleContext } from '../../types';
-import { getLocaleCache } from './get-locale-cache';
 import { flattenLocale } from '../../common/flatten-locale';
+import type { I18nServerConfig, LocaleContext } from '../../types';
+import { getLocaleCache } from './get-locale-cache';
 
 export function createGetI18n<Locales extends ImportedLocales, Locale extends BaseLocale>(
   locales: Locales,

packages/next-international/src/app/server/create-get-scoped-i18n.ts

@@ -1,6 +1,6 @@
 import type { BaseLocale, ImportedLocales, Scopes } from 'international-types';
 import { createT } from '../../common/create-t';
-import { I18nServerConfig, LocaleContext } from '../../types';
+import type { I18nServerConfig, LocaleContext } from '../../types';
 import { getLocaleCache } from './get-locale-cache';
 import { flattenLocale } from '../../common/flatten-locale';
 

packages/next-international/src/app/server/create-get-static-params.ts

@@ -1,5 +1,5 @@
 import type { ImportedLocales } from 'international-types';
-import { I18nServerConfig } from '../../types';
+import type { I18nServerConfig } from '../../types';
 import { DEFAULT_SEGMENT_NAME } from '../../common/constants';
 
 export function createGetStaticParams<Locales extends ImportedLocales>(locales: Locales, config: I18nServerConfig) {

packages/next-international/src/app/server/index.ts

@@ -1,11 +1,11 @@
 import 'server-only';
 
 import type { ExplicitLocales, FlattenLocale, GetLocaleType, ImportedLocales } from 'international-types';
+import type { I18nServerConfig } from '../../types';
+import { createGetCurrentLocale } from './create-get-current-locale';
 import { createGetI18n } from './create-get-i18n';
 import { createGetScopedI18n } from './create-get-scoped-i18n';
-import { createGetCurrentLocale } from './create-get-current-locale';
 import { createGetStaticParams } from './create-get-static-params';
-import { I18nServerConfig } from '../../types';
 
 export { setStaticParamsLocale } from './get-locale-cache';
 

packages/next-international/src/common/create-t.ts

@@ -1,4 +1,3 @@
-import { isValidElement, cloneElement, ReactNode } from 'react';
 import type {
   BaseLocale,
   CreateParams,
@@ -8,7 +7,9 @@ import type {
   ScopedValue,
   Scopes,
 } from 'international-types';
-import type { ReactParamsObject, LocaleContext, LocaleMap } from '../types';
+import type { ReactNode } from 'react';
+import { cloneElement, isValidElement } from 'react';
+import type { LocaleContext, LocaleMap, ReactParamsObject } from '../types';
 
 export function createT<Locale extends BaseLocale, Scope extends Scopes<Locale> | undefined>(
   context: LocaleContext<Locale>,

packages/next-international/src/common/create-use-i18n.ts

@@ -1,4 +1,5 @@
-import { useContext, Context } from 'react';
+import type { Context } from 'react';
+import { useContext } from 'react';
 import type { BaseLocale } from 'international-types';
 import type { LocaleContext } from '../types';
 import { createT } from './create-t';

packages/next-international/src/common/create-use-scoped-i18n.ts

@@ -1,4 +1,5 @@
-import { useContext, Context } from 'react';
+import type { Context } from 'react';
+import { useContext } from 'react';
 import type { BaseLocale, Scopes } from 'international-types';
 import type { LocaleContext } from '../types';
 import { createT } from '../common/create-t';

packages/next-international/src/common/flatten-locale.ts

@@ -1,4 +1,4 @@
-import { BaseLocale } from 'international-types';
+import type { BaseLocale } from 'international-types';
 
 export const flattenLocale = <Locale extends BaseLocale>(locale: Record<string, unknown>, prefix = ''): Locale =>
   Object.entries(locale).reduce(

packages/next-international/src/pages/create-i18n-provider.tsx

@@ -1,9 +1,10 @@
-import React, { Context, ReactElement, ReactNode, useCallback, useEffect, useMemo, useRef, useState } from 'react';
-import type { LocaleContext } from '../types';
 import type { BaseLocale, ImportedLocales } from 'international-types';
 import { useRouter } from 'next/router';
-import { error, warn } from '../helpers/log';
+import type { Context, ReactElement, ReactNode } from 'react';
+import React, { useCallback, useEffect, useMemo, useRef, useState } from 'react';
 import { flattenLocale } from '../common/flatten-locale';
+import { error, warn } from '../helpers/log';
+import type { LocaleContext } from '../types';
 
 type I18nProviderProps<Locale extends BaseLocale> = {
   locale: Locale;

packages/next-international/src/pages/index.ts

@@ -1,5 +1,5 @@
 import { createContext } from 'react';
-import { ImportedLocales, ExplicitLocales, GetLocaleType, FlattenLocale } from 'international-types';
+import type { ImportedLocales, ExplicitLocales, GetLocaleType, FlattenLocale } from 'international-types';
 import type { LocaleContext } from '../types';
 import { createDefineLocale } from '../common/create-define-locale';
 import { createGetLocaleProps } from './create-get-locale-props';