chore(multi-turn): remove checkTokenProgression from multi-turn LLM test and update README#123
Conversation
…est and update README
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
Bugbot Autofix prepared a fix for the issue found in the latest run.
- ✅ Fixed: Unused imports left after removing checkTokenProgression
- I removed the now-unused imports from multi-turn.ts and confirmed the project still builds successfully.
Or push these changes by commenting:
@cursor push ea4e951dca
Preview (ea4e951dca)
diff --git a/src/test-cases/llm/multi-turn.ts b/src/test-cases/llm/multi-turn.ts
--- a/src/test-cases/llm/multi-turn.ts
+++ b/src/test-cases/llm/multi-turn.ts
@@ -5,7 +5,7 @@
* Validates that Sentry captures multiple gen_ai spans correctly.
*/
-import { TestDefinition, Check, ErrorLocation } from "../../types.js";
+import { TestDefinition } from "../../types.js";
import {
checkAISpanCount,
checkChatSpanAttributes,
@@ -15,10 +15,7 @@
checkInputMessagesSchema,
checkResponseModel,
} from "../checks.js";
-import { extractGenAISpans, skipIf } from "../utils.js";
-import { CheckError } from "../../validator.js";
-
export const multiTurnLLMTest: TestDefinition = {
name: "Multi-Turn LLM Test",
description: "Multi-turn conversation with back-and-forth exchanges",
@@ -61,15 +58,9 @@
},
],
- criticalChecks: [
- checkAISpanCount(3),
- checkChatSpanAttributes,
- ],
+ criticalChecks: [checkAISpanCount(3), checkChatSpanAttributes],
- checks: [
- checkValidTokenUsage,
- checkInputMessagesSchema,
- ],
+ checks: [checkValidTokenUsage, checkInputMessagesSchema],
warningChecks: [
checkResponseModel,This Bugbot Autofix run was free. To enable autofix for future PRs, go to the Cursor dashboard.
| import { extractGenAISpans, skipIf } from "../utils.js"; | ||
| import { CheckError } from "../../validator.js"; | ||
|
|
||
| /** |
There was a problem hiding this comment.
Unused imports left after removing checkTokenProgression
Low Severity
After removing checkTokenProgression, the imports Check, ErrorLocation (from ../../types.js), extractGenAISpans, skipIf (from ../utils.js), and CheckError (from ../../validator.js) are no longer used anywhere in the file. These are leftover artifacts of the deleted function.
Additional Locations (2)
🔴 AI SDK Integration Test ResultsStatus: 2 regressions detected Summary
🔴 RegressionsThese tests were passing on main but are now failing: browser/openai :: Multi-Turn LLM Test (blocking)Error: Browser test timed out (60s) cloudflare/google-genai :: Vision LLM Test (blocking)Error: 3 check(s) failed: ✅ FixedThese tests were failing on main but are now passing:
Test MatrixAgent Tests
Embedding Tests
LLM Tests
MCP Tests
Legend: ✅ Pass | ❌ Fail | ✅🔧 Fixed | ❌📉 Regressed | ✅🆕 New (pass) | ❌🆕 New (fail) | 🗑️ Removed | str=streaming blk=blocking a=async s=sync io=stdio sse=sse hi=highlevel lo=lowlevel Generated by AI SDK Integration Tests |



Closes #101