Skip to content

fix: stale hardcoded version in health check, deprecated API, typos - #908

Open
fix2015 wants to merge 1 commit into
czlonkowski:mainfrom
fix2015:fix/stale-version-typos-and-deprecated-api
Open

fix: stale hardcoded version in health check, deprecated API, typos#908
fix2015 wants to merge 1 commit into
czlonkowski:mainfrom
fix2015:fix/stale-version-typos-and-deprecated-api

Conversation

@fix2015

@fix2015 fix2015 commented Jul 3, 2026

Copy link
Copy Markdown

A few things I spotted:

  • `healthCheck()` was returning a hardcoded version `'2.24.1'` in both the healthy and unhealthy branches, but package.json is at 2.62.0. Imported `PROJECT_VERSION` from `utils/version` instead.
  • `res.finished` was deprecated in Node 13.4.0 — changed to `res.writableEnded`
  • fixed "paralel" and "Concieved" typos in CLAUDE.md
  • replaced the placeholder `issues/XXX` link in shared-database.ts with an inline description
  • removed an unnecessary `(this as any)` cast in workflow-validator — `currentWorkflow` is already typed as a private field on the class

- healthCheck() was returning hardcoded version '2.24.1' instead of the
  actual package version. imported PROJECT_VERSION and used that instead.
- res.finished was deprecated in Node 13, changed to res.writableEnded
- fixed "paralel" → "parallel" and "Concieved" → "Conceived" in CLAUDE.md
- replaced dead issue link (issues/XXX) with inline description
- removed unnecessary (this as any) cast in workflow-validator since
  currentWorkflow is already properly typed on the class
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