Typescript 7 and dependency upgrades - #125
Merged
Merged
Conversation
hl7-bot
approved these changes
Aug 20, 2026
Contributor
|
Purely a dependency upgrade PR with no change in function. |
There was a problem hiding this comment.
Pull request overview
This PR modernizes the project’s TypeScript/Node ESM setup and refreshes dependencies to align with newer Node runtimes and TypeScript behavior.
Changes:
- Upgrade TypeScript (to v7) and bump a range of runtime/dev dependencies.
- Move Node built-in imports to
node:specifiers and convert many imports toimport typeto satisfy stricter ESM/TS settings. - Update TS compiler settings for Node ESM (
nodenext) and add an alias forfhir/r4typing.
Reviewed changes
Copilot reviewed 22 out of 25 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| tsconfig.json | Switch to nodenext, enable verbatimModuleSyntax, update target/lib, and add a paths mapping for fhir/r4. |
| package.json | Bump package version, adjust entrypoints, raise Node engine requirement, and upgrade deps/devDeps. |
| package-lock.json | Lockfile updates reflecting the dependency upgrades and Node engine metadata. |
| test/results-validator.test.ts | Use node: imports and import.meta.dirname for ESM-friendly path handling in tests. |
| src/test-results/cql-test-results.ts | Use node: imports and import type for model types under verbatimModuleSyntax. |
| src/shared/server-connectivity.ts | Replace require() usage with node: ESM imports for reading package version. |
| src/shared/run-test-core.ts | Convert model/library imports to import type where appropriate. |
| src/shared/results-shared.ts | Convert imports to import type, including fhir/r4 types. |
| src/shared/library-publisher.ts | Convert fhir/r4 import to import type. |
| src/server/schema-utils.ts | Use node:path and import.meta.dirname for schema path resolution in ESM. |
| src/server/rest-routes.ts | Split express value import vs type-only imports; move fs to node:. |
| src/server/mcp-server-setup.ts | Move fs to node: import. |
| src/models/results-types.ts | Convert TestResult import to import type. |
| src/loaders/test-loader.ts | Move Node built-ins to node: and convert Tests import to import type. |
| src/jobs/job-processor.ts | Update dynamic imports to node: specifiers. |
| src/jobs/job-manager.ts | Move Node built-ins to node: and convert job model imports to import type. |
| src/extractors/value-type-extractors/numeric-interval-extractor.ts | Convert interval type imports to import type. |
| src/cql-engine/cql-engine.ts | Split axios value import vs type-only AxiosResponse import. |
| src/conf/results-validator.ts | Use node: built-ins, import.meta.dirname, and convert ValidationError to type-only import. |
| src/conf/config-validator.ts | Use node: built-ins and import.meta.dirname for schema loading in ESM. |
| src/conf/config-loader.ts | Use node: built-ins and convert config/validation types to import type. |
| src/commands/server-command.ts | Split express value import vs type-only imports; move http Server to node:http type import. |
| src/commands/build-cql-command.ts | Move Node built-ins to node: imports. |
| src/bin/cql-tests.ts | Move Node built-ins to node: and use import.meta.dirname for locating package.json. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Comment on lines
+10
to
12
| "engines": { | ||
| "node": ">=26" | ||
| }, |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.