Skip to content

refactor [NET-1646]: Clean-up tsconfigs for the cli-tools package#3270

Merged
mondoreale merged 8 commits into
mainfrom
net-1646-clean-up-cli-tools-package-tsconfigs
Dec 12, 2025
Merged

refactor [NET-1646]: Clean-up tsconfigs for the cli-tools package#3270
mondoreale merged 8 commits into
mainfrom
net-1646-clean-up-cli-tools-package-tsconfigs

Conversation

@mondoreale

@mondoreale mondoreale commented Dec 8, 2025

Copy link
Copy Markdown
Contributor

This pull request updates TypeScript configuration for the cli-tools package to improve project references and build reliability. The changes primarily enhance how the package manages its dependencies and build process, ensuring better integration with related packages and more robust project setup.

Changes

TypeScript project configuration improvements:

  • Added additional project references in tsconfig.node.json and tsconfig.jest.json for dht, trackerless-network, and test-utils to ensure proper dependency tracking and type checking.
  • Included tsconfig.node.json as a reference in the main tsconfig.json for cli-tools, clarifying the build order and dependencies.
  • Added package.json to the include array in tsconfig.jest.json to ensure it is considered in type checking and builds.

Build process and compiler options:

  • Simplified the build script in package.json to use the default TypeScript project references build (tsc -b), making the build process more consistent.
  • Enabled the disableSourceOfProjectReferenceRedirect compiler option in the root tsconfig.node.json to prevent issues with source file redirection in project references.

Other adjustments:

  • Removed the composite option from cli-tools/tsconfig.node.json, as it is now managed at a higher level.

@mondoreale mondoreale requested a review from teogeb December 8, 2025 15:30
@linear

linear Bot commented Dec 8, 2025

Copy link
Copy Markdown

@github-actions github-actions Bot added the cli-tools Related to CLI Tools Package label Dec 8, 2025
@mondoreale mondoreale force-pushed the net-1646-clean-up-cli-tools-package-tsconfigs branch from 67fe16b to 3a31694 Compare December 12, 2025 00:56
@mondoreale mondoreale requested a review from juslesan December 12, 2025 01:05
@mondoreale mondoreale force-pushed the net-1646-clean-up-cli-tools-package-tsconfigs branch from 93cff2c to a30133f Compare December 12, 2025 01:18
vsc uses source code, `tsc -b` uses .d.ts – the new flag consolidates the 2 (we use .d.ts now).
It's ok this way now.
@mondoreale mondoreale marked this pull request as draft December 12, 2025 14:22
@mondoreale mondoreale marked this pull request as ready for review December 12, 2025 14:22
@mondoreale mondoreale merged commit 1b27c8f into main Dec 12, 2025
23 checks passed
@mondoreale mondoreale deleted the net-1646-clean-up-cli-tools-package-tsconfigs branch December 12, 2025 15:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cli-tools Related to CLI Tools Package

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants