chore: update skott dependencies (#118)

* chore: move devDeps into appropriate section * chore: update devDeps * chore: update major prod deps * add changeset * chore: set typescript module to nodenext * chore: set workspace TypeScript to v5 * docs: changesets * refactor: process changes following verbatimModuleSyntax
antoine-coulon · Oct 3, 2023 · 9dcb23f · 9dcb23f
1 parent 65a4ced
commit 9dcb23f
Show file tree

Hide file tree

Showing 24 changed files with 665 additions and 322 deletions.
diff --git a/.changeset/quiet-panthers-compare.md b/.changeset/quiet-panthers-compare.md
@@ -0,0 +1,6 @@
+---
+"remote-tarball-fetcher": patch
+"fs-tree-structure": patch
+---
+
+Bump TypeScript to v5
diff --git a/.changeset/shaggy-hotels-sip.md b/.changeset/shaggy-hotels-sip.md
@@ -0,0 +1,5 @@
+---
+"skott": patch
+---
+
+Bump most dependencies to their latest available version. Notably bumps `TypeScript` to v5 and `@typescript-eslint/typescript-estree` to v6. This version also lightens `skott` node_modules size from ~144MB to ~120MB.
diff --git a/apps/web/package.json b/apps/web/package.json
@@ -19,7 +19,8 @@
     "dev": "vite & node server.js",
     "prepublishOnly": "npm run build",
     "preview": "vite preview",
-    "test": "vitest"
+    "test": "vitest run",
+    "test:watch": "vitest"
   },
   "devDependencies": {
     "@effect/data": "^0.4.1",

diff --git a/packages/config/tsconfig.json b/packages/config/tsconfig.json
@@ -1,14 +1,15 @@
 {
   "compilerOptions": {
-    "target": "es2020",
-    "lib": ["es2020"],
-    "module": "es2020",
-    "moduleResolution": "nodenext",
+    "target": "ESNext",
+    "lib": ["ESNext"],
+    "module": "NodeNext",
+    "moduleResolution": "NodeNext",
     "esModuleInterop": true,
     "forceConsistentCasingInFileNames": true,
     "strict": true,
     "skipLibCheck": true,
-    "allowSyntheticDefaultImports": true
+    "allowSyntheticDefaultImports": true,
+    "verbatimModuleSyntax": true
   },
   "include": ["src", "bin", "lib", "index.ts"],
   "exclude": ["node_modules", "dist"]

diff --git a/packages/fs-tree-structure/package.json b/packages/fs-tree-structure/package.json
@@ -38,7 +38,7 @@
     "prettier": "^2.8.8",
     "rimraf": "^3.0.2",
     "ts-node": "^10.9.1",
-    "typescript": "^4.9.5"
+    "typescript": "^5.2.2"
   },
   "files": [
     "dist",

diff --git a/packages/remote-tarball-fetcher/package.json b/packages/remote-tarball-fetcher/package.json
@@ -40,7 +40,7 @@
     "prettier": "^2.8.8",
     "rimraf": "^3.0.2",
     "ts-node": "^10.9.1",
-    "typescript": "^4.9.5"
+    "typescript": "^5.2.2"
   },
   "files": [
     "dist",

diff --git a/packages/remote-tarball-fetcher/src/fetcher/npm.ts b/packages/remote-tarball-fetcher/src/fetcher/npm.ts
@@ -3,7 +3,7 @@ import { Readable } from "node:stream";
 import fetch from "node-fetch";
 import semver from "semver";
 
-import { Fetcher } from "./common.js";
+import type { Fetcher } from "./common.js";
 
 function isSemverValid(version: string): boolean {
   return semver.valid(version) !== null;

diff --git a/packages/remote-tarball-fetcher/src/tarball-manager.spec.ts b/packages/remote-tarball-fetcher/src/tarball-manager.spec.ts
@@ -4,7 +4,7 @@ import * as path from "node:path";
 
 import { expect } from "chai";
 
-import { Fetcher, PackageInformation } from "./fetcher/index.js";
+import type { Fetcher, PackageInformation } from "./fetcher/index.js";
 import { TarballManager } from "./tarball-manager.js";
 
 const kTemporaryDirFixture = path.join(process.cwd(), "fixture");

diff --git a/packages/remote-tarball-fetcher/src/tarball-manager.ts b/packages/remote-tarball-fetcher/src/tarball-manager.ts
@@ -6,7 +6,7 @@ import { pipeline } from "node:stream/promises";
 import semver from "semver";
 import tar from "tar";
 
-import { Fetcher } from "./fetcher/index.js";
+import type { Fetcher } from "./fetcher/index.js";
 
 const kSkottStore = "skott_store";
 

diff --git a/packages/skott/.eslintrc b/packages/skott/.eslintrc
@@ -32,6 +32,7 @@
     "max-classes-per-file": "off",
     "no-useless-constructor": "off",
     "no-empty-function": "off",
+    "no-duplicate-imports": "off",
     "brace-style": "off",
     "id-length": "off",
     "func-style": "off",

diff --git a/packages/skott/bin/ui/console.ts b/packages/skott/bin/ui/console.ts
@@ -1,7 +1,7 @@
 import path from "node:path";
 import { performance } from "node:perf_hooks";
 
-import { TreeStructure } from "fs-tree-structure";
+import type { TreeStructure } from "fs-tree-structure";
 import kleur from "kleur";
 
 import type { SkottNode, SkottNodeBody } from "../../src/graph/node.js";

diff --git a/packages/skott/bin/ui/webapp.ts b/packages/skott/bin/ui/webapp.ts
@@ -6,7 +6,7 @@ import polka from "polka";
 import sirv from "sirv";
 import resolveWebAppStaticPath from "skott-webapp";
 
-import { SkottInstance, SkottStructure } from "../../src/skott.js";
+import type { SkottInstance, SkottStructure } from "../../src/skott.js";
 
 import { open } from "./open-url.js";
 

diff --git a/packages/skott/index.ts b/packages/skott/index.ts
@@ -5,7 +5,8 @@ import { FileSystemReader } from "./src/filesystem/file-reader.js";
 import { FileSystemWriter } from "./src/filesystem/file-writer.js";
 import { FakeLogger, Logger } from "./src/logger.js";
 import { ModuleWalkerSelector } from "./src/modules/walkers/common.js";
-import { Skott, SkottConfig, SkottInstance } from "./src/skott.js";
+import { Skott } from "./src/skott.js";
+import type { SkottConfig, SkottInstance } from "./src/skott.js";
 
 interface InputConfig<T> extends Partial<SkottConfig<T>> {
   cwd?: string;

diff --git a/packages/skott/package.json b/packages/skott/package.json
@@ -43,9 +43,7 @@
   "dependencies": {
     "@effect/data": "^0.4.1",
     "@effect/io": "^0.8.0",
-    "@typescript-eslint/eslint-plugin": "^6.7.0",
-    "@typescript-eslint/parser": "^6.7.0",
-    "@typescript-eslint/typescript-estree": "^5.60.0",
+    "@typescript-eslint/typescript-estree": "^6.7.4",
     "commander": "^11.0.0",
     "compression": "^1.7.4",
     "depcheck": "^1.4.3",
@@ -60,30 +58,32 @@
     "kleur": "^4.1.5",
     "lodash.difference": "^4.5.0",
     "meriyah": "^4.3.7",
-    "minimatch": "^9.0.2",
+    "minimatch": "^9.0.3",
     "ora": "^6.3.1",
     "polka": "^0.5.2",
     "sirv": "^2.0.3",
     "skott-webapp": "workspace:^",
-    "typescript": "^4.9.5"
+    "typescript": "^5.2.2"
   },
   "devDependencies": {
     "@nodesecure/eslint-config": "^1.7.0",
     "@skottorg/config": "workspace:*",
     "@types/compression": "^1.7.2",
     "@types/ignore-walk": "^4.0.0",
     "@types/lodash.difference": "^4.5.7",
-    "@types/node": "^16.18.36",
+    "@types/node": "^20.8.2",
     "@types/polka": "^0.5.4",
-    "eslint": "^8.49.0",
+    "@typescript-eslint/eslint-plugin": "^6.7.4",
+    "@typescript-eslint/parser": "^6.7.4",
     "eslint-config-prettier": "^8.8.0",
     "eslint-import-resolver-typescript": "^2.7.1",
     "eslint-plugin-import": "^2.27.5",
     "eslint-plugin-prettier": "^4.2.1",
+    "eslint": "^8.49.0",
     "memfs": "^3.5.3",
-    "prettier": "^2.8.8",
-    "rimraf": "^3.0.2",
-    "vitest": "^0.28.5"
+    "prettier": "^3.0.3",
+    "rimraf": "^5.0.5",
+    "vitest": "^0.34.6"
   },
   "files": [
     "dist",

diff --git a/packages/skott/src/cache/index.ts b/packages/skott/src/cache/index.ts
@@ -1,6 +1,3 @@
 export { createNodeHash, isFileAffected } from "./affected.js";
-export {
-  createInitialSkottNodeValue,
-  SkottCache,
-  SkottCacheHandler
-} from "./handler.js";
+export { createInitialSkottNodeValue, SkottCacheHandler } from "./handler.js";
+export type { SkottCache } from "./handler.js";
diff --git a/packages/skott/src/config.ts b/packages/skott/src/config.ts
@@ -4,7 +4,7 @@ import * as E from "fp-ts/lib/Either.js";
 import * as D from "io-ts/lib/Decoder.js";
 
 import { dependencyResolverDecoder } from "./modules/resolvers/base-resolver.js";
-import { defaultConfig, SkottConfig } from "./skott.js";
+import { defaultConfig, type SkottConfig } from "./skott.js";
 
 function withDefaultValue<T>(defaultValue: T) {
   return (decoder: D.Decoder<unknown, T>): D.Decoder<unknown, T> => {

diff --git a/packages/skott/src/filesystem/fake/file-reader.ts b/packages/skott/src/filesystem/fake/file-reader.ts
@@ -8,7 +8,7 @@ import {
   isFileSupportedByDefault,
   isManifestFile
 } from "../../modules/resolvers/base-resolver.js";
-import { FileReader } from "../file-reader.js";
+import type { FileReader } from "../file-reader.js";
 
 interface FileSystemConfig {
   cwd: string;

diff --git a/packages/skott/src/graph/node.ts b/packages/skott/src/graph/node.ts
@@ -1,4 +1,4 @@
-import { VertexDefinition } from "digraph-js";
+import type { VertexDefinition } from "digraph-js";
 
 export type SkottNodeBody = {
   size: number;

diff --git a/packages/skott/src/graph/traversal.ts b/packages/skott/src/graph/traversal.ts
@@ -1,6 +1,6 @@
 import type { DiGraph } from "digraph-js";
 
-import { SkottConfig } from "../skott.js";
+import type { SkottConfig } from "../skott.js";
 
 import type { SkottNode } from "./node.js";
 

diff --git a/packages/skott/src/modules/resolvers/base-resolver.ts b/packages/skott/src/modules/resolvers/base-resolver.ts
@@ -6,7 +6,10 @@ import * as Effect from "@effect/io/Effect";
 import { DiGraph } from "digraph-js";
 import * as D from "io-ts/lib/Decoder.js";
 
-import { FileReader, FileReaderTag } from "../../filesystem/file-reader.js";
+import {
+  FileReaderTag,
+  type FileReader
+} from "../../filesystem/file-reader.js";
 import type { SkottNode } from "../../graph/node.js";
 import { Logger, LoggerTag, highlight } from "../../logger.js";
 import type { SkottConfig, WorkspaceConfiguration } from "../../skott.js";

diff --git a/packages/skott/src/modules/resolvers/ecmascript/resolver.ts b/packages/skott/src/modules/resolvers/ecmascript/resolver.ts
@@ -15,10 +15,10 @@ import {
 } from "../../walkers/ecmascript/typescript/path-alias.js";
 import {
   continueResolution,
-  DependencyResolver,
-  DependencyResolverControlFlow,
-  DependencyResolverOptions,
-  kExpectedModuleExtensions
+  kExpectedModuleExtensions,
+  type DependencyResolver,
+  type DependencyResolverControlFlow,
+  type DependencyResolverOptions
 } from "../base-resolver.js";
 
 const NODE_PROTOCOL = "node:";

diff --git a/packages/skott/src/skott.ts b/packages/skott/src/skott.ts
@@ -9,24 +9,24 @@ import difference from "lodash.difference";
 
 import {
   isFileAffected,
-  SkottCache,
+  type SkottCache,
   SkottCacheHandler
 } from "./cache/index.js";
-import { FileReader, FileReaderTag } from "./filesystem/file-reader.js";
-import { FileWriter } from "./filesystem/file-writer.js";
+import { type FileReader, FileReaderTag } from "./filesystem/file-reader.js";
+import type { FileWriter } from "./filesystem/file-writer.js";
 import type { SkottNode } from "./graph/node.js";
-import { makeTraversalApi, TraversalApi } from "./graph/traversal.js";
+import { type TraversalApi, makeTraversalApi } from "./graph/traversal.js";
 import {
   highlight,
   logFailureM,
   LoggerTag,
   logSuccessM,
   lowlight,
-  SkottLogger
+  type SkottLogger
 } from "./logger.js";
 import {
-  DependencyResolver,
-  FollowModuleDeclarationOptions,
+  type DependencyResolver,
+  type FollowModuleDeclarationOptions,
   isManifestFile,
   kExpectedModuleExtensions,
   resolveImportedModulePath
@@ -38,7 +38,7 @@ import {
 import { ModuleWalkerSelector } from "./modules/walkers/common.js";
 import {
   buildPathAliases,
-  TSConfig
+  type TSConfig
 } from "./modules/walkers/ecmascript/typescript/path-alias.js";
 import {
   extractInformationFromManifest,

diff --git a/packages/skott/src/workspace/index.ts b/packages/skott/src/workspace/index.ts
@@ -5,8 +5,8 @@ import { pipe } from "@effect/data/Function";
 import * as Effect from "@effect/io/Effect";
 import depcheck from "depcheck";
 
-import { FileReader, FileReaderTag } from "../filesystem/file-reader.js";
-import { SkottLogger } from "../logger.js";
+import { type FileReader, FileReaderTag } from "../filesystem/file-reader.js";
+import type { SkottLogger } from "../logger.js";
 
 export async function findWorkspaceEntrypointModule(): Promise<string> {
   // look for package.json