Skip to content

TS, Next, and other updates#116

Open
elsaperelli wants to merge 2 commits into
mainfrom
ts-next-update
Open

TS, Next, and other updates#116
elsaperelli wants to merge 2 commits into
mainfrom
ts-next-update

Conversation

@elsaperelli

@elsaperelli elsaperelli commented May 8, 2026

Copy link
Copy Markdown
Contributor

Summary

This PR does a number of upgrades and associated changes including:

  • next
  • TypeScript
  • tsconfig.json

New Behavior

There should be no new behavior in the app.

Code Changes

  • .tool-versions - node version 24 now that we are using asdf
  • Removing catch (e) and catch(error) to just be catch when e/error is not used
  • Importing correct icon name from @tabler/icons-react
  • Updating any types when easy, when more complicated I just added a eslint-disable line for now
  • .github/workflows updates - package-manager detection step is now deprecated so removes that, runs npm ci directory, updated actions versions, use .tool-versions file for node version
  • .npmrc - added to force registry.npmjs.org
  • immer imports updated when produce could still be used as is
  • createDraft / finishDraft from 'immer' rather than produce in some scenarios
  • imports updated for fs and path
  • empty.ts - this is a little odd, but seems necessary for cql-exec-fhir (see the Upgrading Next Version 16 docs)
  • next version and associated updates in next-env.d.ts, next.config.js, package-lock.json (including scripts)
  • tsconfig.json - udpates made
  • tsconfig.scripts.json - config file for scripts/ files to use path and fs

Testing Guidance

  • npm install
  • npm run check
  • npm run dev
  • Make sure the app functions as normal

@elsaperelli elsaperelli marked this pull request as ready for review May 8, 2026 16:40
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.

1 participant