[MLOB-5361][nodejs] re-enable flaky anthropic tests #6155
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Motivation
Re-enabling anthropic tests so they run in
dd-trace-jsCI successfully.The issue and reason for skipping these tests is two-fold:
@anthropic-ai/sdkstarts requiringzodin hot paths (ie, just requiring@anthropic-ai/sdk) without making it a required dependency. Hence, trying to just install the package and requiring it does not work, even in a local reproduction.FRAMEWORK_VERSIONas a build arg for this app's Dockerfile, we were instead using latest, hitting the error above. Making sure we actually use the pinned (and working) version is good for now.I didn't include a "fix" for correctly installing
zodsince it's not needed for now. In the future, maybe we should have safeguards for this. At the same time, this is an odd scenario.Changes
Includes a forgotten
FRAMEWORK_VERSIONbuild arg (these are in Python, but I missed them for Node.js).Workflow
🚀 Once your PR is reviewed and the CI green, you can merge it!
🛟 #apm-shared-testing 🛟
Reviewer checklist
tests/ormanifests/is modified ? I have the approval from R&P teambuild-XXX-imagelabel is present