Skip to content

Conversation

@mondoreale
Copy link
Contributor

@mondoreale mondoreale commented Dec 8, 2025

This pull request simplifies and standardizes the TypeScript configuration across several packages, particularly for autocertifier-client and related dependencies. It removes redundant or unnecessary configuration files, updates script commands, and streamlines references to improve maintainability and consistency.

Changes

TypeScript configuration simplification and cleanup:

  • Removed tsconfig.node.json and tsconfig.jest.json from autocertifier-client, consolidating configuration into a single tsconfig.json that now extends the root config and includes only necessary files and references.
  • Updated TypeScript references in other packages (cdn-location, dht) to point directly to the autocertifier-client directory instead of its removed config files.

Script and build process updates:

  • Updated build and check scripts in autocertifier-client and autocertifier-server to use the new TypeScript config structure and to improve type checking (e.g., using tsc --noEmit).
  • Cleaned up the clean script in autocertifier-client to remove unnecessary Jest cache clearing, aligning it with the new configuration.

Reference and include adjustments:

  • Updated references in autocertifier-server and its test configs to match the new structure and ensure correct dependency tracking.

@mondoreale mondoreale requested review from juslesan and teogeb December 8, 2025 14:51
@linear
Copy link

linear bot commented Dec 8, 2025

@github-actions github-actions bot added the dht Related to DHT package label Dec 8, 2025
@mondoreale mondoreale marked this pull request as ready for review December 8, 2025 15:04
@mondoreale mondoreale force-pushed the net-1643-clean-up-autocertifier-tsconfigs branch from 3ebf813 to a81466e Compare December 12, 2025 00:24
@mondoreale mondoreale merged commit b92286b into main Dec 12, 2025
42 of 43 checks passed
@mondoreale mondoreale deleted the net-1643-clean-up-autocertifier-tsconfigs branch December 12, 2025 00:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dht Related to DHT package

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants