style: require type-only imports

zazuko · Jun 28, 2024 · 3d2fc29 · 3d2fc29
1 parent 6ce7d6b
commit 3d2fc29
Show file tree

Hide file tree

Showing 14 changed files with 39 additions and 35 deletions.
diff --git a/packages/core/index.ts b/packages/core/index.ts
@@ -5,7 +5,8 @@ import defaultLoaderRegistry from './lib/defaultLoaderRegistry.js'
 import defaultLogger from './lib/defaultLogger.js'
 import createPipeline from './lib/factory/pipeline.js'
 import run from './lib/run.js'
-import Pipeline, { PipelineOptions } from './lib/Pipeline.js'
+import type { PipelineOptions } from './lib/Pipeline.js'
+import type Pipeline from './lib/Pipeline.js'
 
 export type { default as Step } from './lib/Step.js'
 export type { default as Pipeline, PipelineOptions } from './lib/Pipeline.js'

diff --git a/packages/core/lib/Pipeline.ts b/packages/core/lib/Pipeline.ts
@@ -1,10 +1,12 @@
 import * as otel from '@opentelemetry/api'
 import once from 'onetime'
-import streams, { Stream } from 'readable-stream'
+import type { Stream } from 'readable-stream'
+import streams from 'readable-stream'
 import createStream, { assertWritable } from './factory/stream.js'
 import { isReadable } from './isStream.js'
 import nextLoop from './nextLoop.js'
-import StreamObject, { Options as BaseOptions } from './StreamObject.js'
+import type { Options as BaseOptions } from './StreamObject.js'
+import StreamObject from './StreamObject.js'
 import tracer from './tracer.js'
 
 const { finished } = streams

diff --git a/packages/core/lib/Step.ts b/packages/core/lib/Step.ts
@@ -1,6 +1,7 @@
 import once from 'onetime'
-import { Stream } from 'readable-stream'
-import StreamObject, { Options as BaseOptions } from './StreamObject.js'
+import type { Stream } from 'readable-stream'
+import type { Options as BaseOptions } from './StreamObject.js'
+import StreamObject from './StreamObject.js'
 import tracer from './tracer.js'
 import type { Operation } from './factory/operation.js'
 

diff --git a/packages/core/lib/StreamObject.ts b/packages/core/lib/StreamObject.ts
@@ -1,9 +1,9 @@
-import { Stream } from 'readable-stream'
-import { LoaderRegistry } from 'rdf-loaders-registry'
-import { Logger } from 'winston'
+import type { Stream } from 'readable-stream'
+import type { LoaderRegistry } from 'rdf-loaders-registry'
+import type { Logger } from 'winston'
 import type { GraphPointer } from 'clownface'
-import * as otel from '@opentelemetry/api'
-import { Context, VariableMap } from '../index.js'
+import type * as otel from '@opentelemetry/api'
+import type { Context, VariableMap } from '../index.js'
 import { VariableMap as VariableMapImpl } from './VariableMap.js'
 
 export interface Options {

diff --git a/packages/core/lib/factory/arguments.ts b/packages/core/lib/factory/arguments.ts
@@ -1,9 +1,9 @@
 import parseArguments from 'rdf-loader-code/arguments.js'
 import type { GraphPointer } from 'clownface'
-import { Logger } from 'winston'
-import { LoaderRegistry } from 'rdf-loaders-registry'
+import type { Logger } from 'winston'
+import type { LoaderRegistry } from 'rdf-loaders-registry'
 import { unknownVariable } from '../loader/variable.js'
-import { Context, VariableMap } from '../../index.js'
+import type { Context, VariableMap } from '../../index.js'
 
 async function createArguments(ptr: GraphPointer, { basePath, context, loaderRegistry, variables }: { basePath: string; context: Pick<Context, 'env'>; loaderRegistry: LoaderRegistry; logger: Logger; variables: VariableMap }) {
  // eslint-disable-next-line @typescript-eslint/no-explicit-any

diff --git a/packages/core/lib/factory/operation.ts b/packages/core/lib/factory/operation.ts
@@ -1,9 +1,9 @@
 import type { MultiPointer } from 'clownface'
-import { Logger } from 'winston'
-import { LoaderRegistry } from 'rdf-loaders-registry'
+import type { Logger } from 'winston'
+import type { LoaderRegistry } from 'rdf-loaders-registry'
 import { isGraphPointer } from 'is-graph-pointer'
-import { Stream } from 'readable-stream'
-import { Context, VariableMap } from '../../index.js'
+import type { Stream } from 'readable-stream'
+import type { Context, VariableMap } from '../../index.js'
 
 export type Operation = (this: Context, ...args: unknown[]) => Promise<Stream | (() => AsyncGenerator)> | Stream | (() => AsyncGenerator)
 

diff --git a/packages/core/lib/factory/pipeline.ts b/packages/core/lib/factory/pipeline.ts
@@ -1,14 +1,14 @@
 import type { DatasetCore, Term } from '@rdfjs/types'
 import type { GraphPointer, MultiPointer } from 'clownface'
-import { Logger } from 'winston'
-import { LoaderRegistry } from 'rdf-loaders-registry'
+import type { Logger } from 'winston'
+import type { LoaderRegistry } from 'rdf-loaders-registry'
 import type { Environment } from 'barnard59-env'
 import defaultLoaderRegistry from '../defaultLoaderRegistry.js'
 import defaultLogger from '../defaultLogger.js'
 import metadata from '../metadata.js'
 import Pipeline from '../Pipeline.js'
 import { VariableMap as VariableMapImpl } from '../VariableMap.js'
-import { Context, VariableMap } from '../../index.js'
+import type { Context, VariableMap } from '../../index.js'
 import createStep from './step.js'
 import createVariables from './variables.js'
 

diff --git a/packages/core/lib/factory/step.ts b/packages/core/lib/factory/step.ts
@@ -1,14 +1,14 @@
 import { Duplex } from 'node:stream'
-import { Stream } from 'readable-stream'
+import type { Stream } from 'readable-stream'
 import { SpanStatusCode } from '@opentelemetry/api'
 import type { GraphPointer } from 'clownface'
-import { Logger } from 'winston'
-import { LoaderRegistry } from 'rdf-loaders-registry'
+import type { Logger } from 'winston'
+import type { LoaderRegistry } from 'rdf-loaders-registry'
 import { isStream } from '../isStream.js'
 import PipelineError from '../PipelineError.js'
 import Step from '../Step.js'
 import tracer from '../tracer.js'
-import { Context, VariableMap } from '../../index.js'
+import type { Context, VariableMap } from '../../index.js'
 import createArguments from './arguments.js'
 import createOperation from './operation.js'
 

diff --git a/packages/core/lib/factory/stream.ts b/packages/core/lib/factory/stream.ts
@@ -1,6 +1,6 @@
 import { Readable, Writable, Duplex } from 'readable-stream'
-import Pipeline from '../Pipeline.js'
-import StreamObject from '../StreamObject.js'
+import type Pipeline from '../Pipeline.js'
+import type StreamObject from '../StreamObject.js'
 import { isWritable as streamIsWritable } from '../isStream.js'
 
 export function isWritable(arg: StreamObject): arg is StreamObject<Writable> {

diff --git a/packages/core/lib/factory/variables.ts b/packages/core/lib/factory/variables.ts
@@ -1,9 +1,9 @@
 import type { MultiPointer } from 'clownface'
-import { Logger } from 'winston'
-import { LoaderRegistry } from 'rdf-loaders-registry'
+import type { Logger } from 'winston'
+import type { LoaderRegistry } from 'rdf-loaders-registry'
 import { isGraphPointer } from 'is-graph-pointer'
 import { VariableMap } from '../VariableMap.js'
-import { Context } from '../../index.js'
+import type { Context } from '../../index.js'
 
 async function createVariables(ptr: MultiPointer, { basePath, context, loaderRegistry, logger }: { basePath: string; context: Pick<Context, 'env'>; loaderRegistry: LoaderRegistry; logger: Logger }) {
  const variables = new VariableMap()

diff --git a/packages/core/lib/isStream.ts b/packages/core/lib/isStream.ts
@@ -3,7 +3,7 @@ import {
  isWritableStream as isWritable,
  isDuplexStream as isDuplex,
 } from 'is-stream'
-import { Stream, Writable } from 'readable-stream'
+import type { Stream, Writable } from 'readable-stream'
 
 const isReadableObjectMode = (stream: Stream) => isReadable(stream) && stream._readableState.objectMode
 const isWritableObjectMode = (stream: Stream) => isWritable(stream) && (stream as Writable)._writableState.objectMode

diff --git a/packages/core/lib/loader/file.ts b/packages/core/lib/loader/file.ts
@@ -3,9 +3,9 @@ import * as fs from 'fs'
 import { isLiteral } from 'is-graph-pointer'
 import untildify from 'untildify'
 import type { GraphPointer } from 'clownface'
-import LoaderRegistry from 'rdf-loaders-registry'
+import type LoaderRegistry from 'rdf-loaders-registry'
 import type { Environment } from 'barnard59-env'
-import { VariableMap } from '../VariableMap.js'
+import type { VariableMap } from '../VariableMap.js'
 
 export default function loader(ptr: GraphPointer, { basePath, variables }: { basePath: string; variables: VariableMap }) {
  if (isLiteral(ptr)) {

diff --git a/packages/core/lib/loader/pipeline.ts b/packages/core/lib/loader/pipeline.ts
@@ -1,8 +1,8 @@
-import { LoaderRegistry } from 'rdf-loaders-registry'
+import type { LoaderRegistry } from 'rdf-loaders-registry'
 import type { Environment } from 'barnard59-env'
 import type { GraphPointer } from 'clownface'
 import createPipeline from '../factory/pipeline.js'
-import { Context, VariableMap } from '../../index.js'
+import type { Context, VariableMap } from '../../index.js'
 
 interface Options {
  basePath: string

diff --git a/packages/core/lib/loader/variable.ts b/packages/core/lib/loader/variable.ts
@@ -1,7 +1,7 @@
 import type { Term } from '@rdfjs/types'
 import type { Environment } from 'barnard59-env'
 import type { GraphPointer } from 'clownface'
-import LoaderRegistry from 'rdf-loaders-registry'
+import type LoaderRegistry from 'rdf-loaders-registry'
 import cloneTerm from '../cloneTerm.js'
 import { VariableMap } from '../VariableMap.js'