Skip to content

Conversation

taiichiro-nagase
Copy link
Contributor

@taiichiro-nagase taiichiro-nagase commented Sep 6, 2025

This PR adds a test case that reveals a circumstance in which Mustache nested objects do not work correctly.
Since I don't have a detailed enough understanding to make code changes, I'll start by adding test cases.
This test case passes on @langchain/[email protected] which I'm using for now.

This issue may be caused by the following PR: #8580.

This PR fixes an mustache template issue that nested objects won't work correctly.

Copy link

changeset-bot bot commented Sep 6, 2025

🦋 Changeset detected

Latest commit: cf35293

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

This PR includes changesets to release 2 packages
Name Type
@langchain/core Patch
@langchain/standard-tests 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

Copy link

vercel bot commented Sep 6, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
langchainjs-docs Ready Ready Preview Sep 9, 2025 0:05am
1 Skipped Deployment
Project Deployment Preview Comments Updated (UTC)
langchainjs-api-refs Ignored Ignored Sep 9, 2025 0:05am

@taiichiro-nagase taiichiro-nagase changed the title test(core): add test case for mustache prompt with nested object fix(core): make mustache prompt with nested object working correctly Sep 6, 2025
Copy link
Member

@christian-bromann christian-bromann left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

Thanks for the contributions!

@hntrl hntrl self-assigned this Sep 8, 2025
@hntrl hntrl merged commit 58e9522 into langchain-ai:main Sep 9, 2025
42 of 43 checks passed
@taiichiro-nagase
Copy link
Contributor Author

@hntrl @christian-bromann thanks for the rapid review & merging!

hntrl added a commit that referenced this pull request Oct 14, 2025
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.

3 participants