[PR #624] [MERGED] fix(checkpoint-validation): cli fixes #896

Closed
opened 2026-02-15 19:16:19 -05:00 by yindo · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/langchain-ai/langgraphjs/pull/624
Author: @benjamincburns
Created: 10/23/2024
Status: Merged
Merged: 10/24/2024
Merged by: @nfcampos

Base: mainHead: fix-ts-jest-import


📝 Commits (2)

  • 3fe8587 fix(checkpoint-validation): cli fixes
  • 8b7d249 Include note about test framework compatibility in README

📊 Changes

9 files changed (+74 additions, -319 deletions)

View changed files

📝 libs/checkpoint-validation/README.md (+3 -2)
📝 libs/checkpoint-validation/bin/cli.js (+1 -4)
📝 libs/checkpoint-validation/package.json (+1 -3)
📝 libs/checkpoint-validation/src/cli.ts (+32 -13)
📝 libs/checkpoint-validation/src/import_utils.ts (+6 -0)
📝 libs/checkpoint-validation/src/parse_args.ts (+28 -28)
📝 libs/checkpoint-validation/src/runner.ts (+2 -7)
libs/checkpoint-validation/src/runtime_jest_config.ts (+0 -28)
📝 yarn.lock (+1 -234)

📄 Description

  • move ts-jest to dependencies from devDependencies
  • get rid of @swc-node/register in favor of @swc/jest
  • handle relative path to initializer import without leading dot
  • configure CLI to run with ESM support in jest, enabling top-level await (gets rid of the ugly global arg passing)
  • pass jest config directly to runCLI, no need for extra jest config file

Tested on mac as global install by publishing to a local verdaccio registry


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/langchain-ai/langgraphjs/pull/624 **Author:** [@benjamincburns](https://github.com/benjamincburns) **Created:** 10/23/2024 **Status:** ✅ Merged **Merged:** 10/24/2024 **Merged by:** [@nfcampos](https://github.com/nfcampos) **Base:** `main` ← **Head:** `fix-ts-jest-import` --- ### 📝 Commits (2) - [`3fe8587`](https://github.com/langchain-ai/langgraphjs/commit/3fe85872483d48625282566001df1acb7ebbd7bf) fix(checkpoint-validation): cli fixes - [`8b7d249`](https://github.com/langchain-ai/langgraphjs/commit/8b7d24906afa159cdc884a6ca2189b1764e46dc0) Include note about test framework compatibility in README ### 📊 Changes **9 files changed** (+74 additions, -319 deletions) <details> <summary>View changed files</summary> 📝 `libs/checkpoint-validation/README.md` (+3 -2) 📝 `libs/checkpoint-validation/bin/cli.js` (+1 -4) 📝 `libs/checkpoint-validation/package.json` (+1 -3) 📝 `libs/checkpoint-validation/src/cli.ts` (+32 -13) 📝 `libs/checkpoint-validation/src/import_utils.ts` (+6 -0) 📝 `libs/checkpoint-validation/src/parse_args.ts` (+28 -28) 📝 `libs/checkpoint-validation/src/runner.ts` (+2 -7) ➖ `libs/checkpoint-validation/src/runtime_jest_config.ts` (+0 -28) 📝 `yarn.lock` (+1 -234) </details> ### 📄 Description - move ts-jest to `dependencies` from `devDependencies` - get rid of `@swc-node/register` in favor of `@swc/jest` - handle relative path to initializer import without leading dot - configure CLI to run with ESM support in jest, enabling top-level await (gets rid of the ugly global arg passing) - pass jest config directly to runCLI, no need for extra jest config file Tested on mac as global install by publishing to a local verdaccio registry --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
yindo added the pull-request label 2026-02-15 19:16:19 -05:00
yindo closed this issue 2026-02-15 19:16:19 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: langchain-ai/langgraphjs#896