[PR #618] [MERGED] fix(checkpoint-validation): fix broken initial release #890

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

📋 Pull Request Information

Original PR: https://github.com/langchain-ai/langgraphjs/pull/618
Author: @benjamincburns
Created: 10/21/2024
Status: Merged
Merged: 10/22/2024
Merged by: @jacoblee93

Base: mainHead: fix-validator-release


📝 Commits (1)

  • d7f3421 fix(checkpoint-validation): fix broken initial release

📊 Changes

5 files changed (+18 additions, -11 deletions)

View changed files

📝 libs/checkpoint-validation/bin/cli.js (+3 -1)
📝 libs/checkpoint-validation/src/cli.ts (+1 -3)
📝 libs/checkpoint-validation/src/runtime_jest_config.ts (+11 -5)
📝 libs/checkpoint-validation/tsconfig.cjs.json (+2 -1)
📝 libs/checkpoint-validation/tsconfig.json (+1 -1)

📄 Description

Unfortunately the initial release of @langchain/langgraph-checkpoint-validator is broken for two reasons.

  1. A bug in @swc-node/register attempts to resolve SWC packages from the CWD rather than the installed package's node_modules directory.
  2. The jest config used by the CLI wasn't included in the release because I forgot to add it to the files array in package.json.

I fixed the first issue by applying the workaround mentioned here: https://github.com/swc-project/swc-node/pull/805#issuecomment-2257146304.

I fixed the second issue by moving the runtime jest config into the src dir and making it a TS file.

I tested this locally using npm link and it seems to fix both issues.

Sorry for the hiccup!


🔄 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/618 **Author:** [@benjamincburns](https://github.com/benjamincburns) **Created:** 10/21/2024 **Status:** ✅ Merged **Merged:** 10/22/2024 **Merged by:** [@jacoblee93](https://github.com/jacoblee93) **Base:** `main` ← **Head:** `fix-validator-release` --- ### 📝 Commits (1) - [`d7f3421`](https://github.com/langchain-ai/langgraphjs/commit/d7f34210d5534c8c964f007e47d4feee56226ac0) fix(checkpoint-validation): fix broken initial release ### 📊 Changes **5 files changed** (+18 additions, -11 deletions) <details> <summary>View changed files</summary> 📝 `libs/checkpoint-validation/bin/cli.js` (+3 -1) 📝 `libs/checkpoint-validation/src/cli.ts` (+1 -3) 📝 `libs/checkpoint-validation/src/runtime_jest_config.ts` (+11 -5) 📝 `libs/checkpoint-validation/tsconfig.cjs.json` (+2 -1) 📝 `libs/checkpoint-validation/tsconfig.json` (+1 -1) </details> ### 📄 Description Unfortunately the initial release of `@langchain/langgraph-checkpoint-validator` is broken for two reasons. 1. A bug in `@swc-node/register` attempts to resolve SWC packages from the CWD rather than the installed package's `node_modules` directory. 2. The jest config used by the CLI wasn't included in the release because I forgot to add it to the `files` array in `package.json`. I fixed the first issue by applying the workaround mentioned here: https://github.com/swc-project/swc-node/pull/805#issuecomment-2257146304. I fixed the second issue by moving the runtime jest config into the `src` dir and making it a TS file. I tested this locally using `npm link` and it seems to fix both issues. Sorry for the hiccup! --- <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:17 -05:00
yindo closed this issue 2026-02-15 19:16:18 -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#890