Skip to content

fix(iot): match scaffolded Node typings to the MES runtime - #874

Open
joaoopereira wants to merge 1 commit into
developmentfrom
codex/595-iot-node-typings
Open

joaoopereira wants to merge 1 commit into
developmentfrom
codex/595-iot-node-typings

Conversation

@joaoopereira

Copy link
Copy Markdown
Member

What was done

IoT task libraries, drivers, and business scenarios hardcode Node 18 typings even for projects targeting newer runtimes. Generate @types/node as ^<major>.0.0 using the existing MES-to-Node mapping, and pass the template parameter through each affected generator. Existing projects are unchanged.

Closes #595.

Validation checklist

  • Release solution build.
  • Documentation builds with MkDocs strict validation.
  • Twelve actual generator regressions use in-memory files and scripted prompts to validate supported MES 10/11/12 cases without accessing package registries.
  • PR test suite: 793 passed, 2 skipped, 0 failed.
  • Additional compiler smoke check: TypeScript 4.9.5 with current Node 24 typings, using the templates' skipLibCheck setting.
  • git diff --check and NuGet minimum-age check.
  • Integration suite passes: the local hook fails in 50 tests with Invalid repository type "Portal", expected one of: CIFS, NPM. The identical 50 failing test names reproduce on unchanged development.
  • Main work item linked: IoT Scaffolding #595.
  • Target branch is development (6.0.1-0).

The local pre-commit hook was bypassed after reproducing its integration failures on the baseline. Commitlint was run separately. No integration-test setup or unrelated code was changed.

@joaoopereira
joaoopereira marked this pull request as ready for review October 23, 2026 08:59
@joaoopereira
joaoopereira requested a review from a team October 23, 2026 08:59
@joaoopereira
joaoopereira force-pushed the codex/595-iot-node-typings branch from 19a6900 to 7c812a1 Compare October 28, 2026 23:47
@github-actions

Copy link
Copy Markdown
Contributor

.NET Test Results

View full test report

@github-actions

Copy link
Copy Markdown
Contributor

Code Coverage

Package Line Rate Branch Rate Health
cmf 52% 49% ➖
core 77% 60% ✔
Summary 62% (6731 / 10840) 55% (3425 / 6280) ➖

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants