ref: Replace "js" platform naming with "node"#37
Merged
priscilawebdev merged 2 commits intomainfrom Feb 11, 2026
Merged
Conversation
Rename platform identifier from "js" to "node" throughout the codebase to more accurately reflect that these are Node.js SDKs, not generic JavaScript. This includes: - Type definitions (platform: "js" | "py" -> "node" | "py") - Template directory structure (templates/*/js/* -> templates/*/node/*) - Base template file (base.js.njk -> base.node.njk) - Validation logic and error messages - Documentation and comments No functional changes, only naming conventions updated. Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
🟡 AI SDK Integration Test ResultsStatus: 141 tests failing (no regressions) Summary
🆕 New TestsPassing (8):
Failing (41): ❌ node/mastra :: Tool Call Agent TestError: 2 check(s) failed: ❌ node/mastra :: Tool Error Agent TestError: 1 check(s) failed: ❌ node/vercel :: Basic Agent TestError: 5 check(s) failed: ❌ node/vercel :: Tool Call Agent TestError: 10 check(s) failed: ❌ node/vercel :: Tool Error Agent TestError: 8 check(s) failed: ❌ node/vercel :: Vision Agent TestError: 6 check(s) failed: ❌ node/vercel :: Long Input Agent TestError: 6 check(s) failed: ❌ node/anthropic :: Basic LLM Test (streaming)Error: 2 check(s) failed: ❌ node/anthropic :: Basic LLM Test (blocking)Error: 2 check(s) failed: ❌ node/anthropic :: Multi-Turn LLM Test (streaming)Error: 2 check(s) failed: ❌ node/anthropic :: Multi-Turn LLM Test (blocking)Error: 2 check(s) failed: ❌ node/anthropic :: Vision LLM Test (streaming)Error: 3 check(s) failed: ❌ node/anthropic :: Vision LLM Test (blocking)Error: 3 check(s) failed: ❌ node/anthropic :: Long Input LLM Test (streaming)Error: 4 check(s) failed: ❌ node/anthropic :: Long Input LLM Test (blocking)Error: 4 check(s) failed: ❌ node/google-genai :: Basic LLM Test (streaming)Error: 3 check(s) failed: ❌ node/google-genai :: Basic LLM Test (blocking)Error: 3 check(s) failed: ❌ node/google-genai :: Multi-Turn LLM Test (streaming)Error: 3 check(s) failed: ❌ node/google-genai :: Multi-Turn LLM Test (blocking)Error: 3 check(s) failed: ❌ node/google-genai :: Basic Error LLM Test (streaming)Error: 2 check(s) failed: ❌ node/google-genai :: Basic Error LLM Test (blocking)Error: 2 check(s) failed: ❌ node/google-genai :: Vision LLM Test (streaming)Error: 3 check(s) failed: ❌ node/google-genai :: Vision LLM Test (blocking)Error: 3 check(s) failed: ❌ node/google-genai :: Long Input LLM Test (streaming)Error: 2 check(s) failed: ❌ node/google-genai :: Long Input LLM Test (blocking)Error: 2 check(s) failed: ❌ node/langchain :: Basic LLM Test (streaming)Error: 1 check(s) failed: ❌ node/langchain :: Basic LLM Test (blocking)Error: 1 check(s) failed: ❌ node/langchain :: Multi-Turn LLM Test (streaming)Error: 1 check(s) failed: ❌ node/langchain :: Multi-Turn LLM Test (blocking)Error: 1 check(s) failed: ❌ node/langchain :: Vision LLM Test (streaming)Error: 2 check(s) failed: ❌ node/langchain :: Vision LLM Test (blocking)Error: 2 check(s) failed: ❌ node/langchain :: Long Input LLM Test (streaming)Error: 2 check(s) failed: ❌ node/langchain :: Long Input LLM Test (blocking)Error: 2 check(s) failed: ❌ node/openai :: Basic LLM Test (streaming)Error: 2 check(s) failed: ❌ node/openai :: Basic LLM Test (blocking)Error: 1 check(s) failed: ❌ node/openai :: Multi-Turn LLM Test (streaming)Error: 3 check(s) failed: ❌ node/openai :: Multi-Turn LLM Test (blocking)Error: 1 check(s) failed: ❌ node/openai :: Vision LLM Test (streaming)Error: 3 check(s) failed: ❌ node/openai :: Vision LLM Test (blocking)Error: 2 check(s) failed: ❌ node/openai :: Long Input LLM Test (streaming)Error: 3 check(s) failed: ❌ node/openai :: Long Input LLM Test (blocking)Error: 3 check(s) failed: 🗑️ Removed TestsThese tests existed on main but are not in the PR:
Test Matrix
Legend: ✅ Pass | ❌ Fail | ✅🔧 Fixed | ❌📉 Regressed | ✅🆕 New (pass) | ❌🆕 New (fail) | 🗑️ Removed Generated by AI SDK Integration Tests |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
Rename platform identifier from "js" to "node" throughout the codebase to more accurately reflect that these are Node.js SDKs, not generic JavaScript.
The following will be handled in a follow-up:
Add a browser directory to test Sentry helper functions for OpenAI, Anthropic, LangChain, LangGraph and Google Gen AI.
Add a dedicated Next.js directory with tests for Vercel AI SDK integration and for Sentry helper–based integrations (OpenAI, Anthropic, LangChain, LangGraph, Google Gen AI, etc). The use of helper functions is required due to a current bubbling issue in the default Node.js runtime for meta frameworks.
contributes to https://linear.app/getsentry/issue/TET-1872/ai-testing-framework-split-js-into-browser-and-node