fix(community): possible race condition on FileSystemChatMessageHistory

[sinedied]

Fixes a small possible race condition on FileSystemChatMessageHistory, and removes unneeded extra check

[changeset-bot]

🦋 Changeset detected

Latest commit: 411df8f

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package

Name	Type
@langchain/community	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[christian-bromann]

Fixes a small possible race condition on FileSystemChatMessageHistory, and removes unneeded extra check

I can see the removes unneeded extra check part but not the fix of a race condition 🤔

[christian-bromann]

Seems like the PR is not complete?

[sinedied]

@christian-bromann It's surprising but complete :) I started initially by refactoring the code to use file descriptors, but it turned out that the first fs.access() check that causes the race condition wasn't even needed in the first place, so the fix was even simpler 😅

[sinedied]

Bumping this, @christian-bromann / @hntrl could you review this one? Just confirming that removing this single line is the fix 🙂

[christian-bromann]

LGTM 👍

Good catch!