README.md (1)

526-529: Changelog entry OK; add a short privacy note.

Clarify that v1 wrapper no longer auto-logs user prompts; apps must call trace.input() explicitly if they want inputs captured.

 ### v6.19.0
 - fix: Removes trace input setting for AI SDK v1
 - adds test cases for AI SDK v2 wrapper
+ - Note: AI SDK v1 models no longer auto-populate trace input. If you want to capture user prompts, call `trace.input(...)` explicitly.

src/lib/tests/vercel/agent_testing.ts (2)

15-33: Prefer raiseExceptions during tests.

Enable raiseExceptions for faster failure in CI.

- const maxim = new Maxim({ baseUrl, apiKey, debug: true });
+ const maxim = new Maxim({ baseUrl, apiKey, debug: true, raiseExceptions: true });

---

103-115: Also log the final output to the trace.

Optional: set trace.output(formattedOutput) before ending the trace to complete the I/O pair.

- const formattedOutput = await formatOutput(model, structuredData, trace);
+ const formattedOutput = await formatOutput(model, structuredData, trace);
+ trace.output(formattedOutput);

src/lib/logger/vercel/vercelLoggerV2.test.ts (1)

23-30: Env var message consistency.

Use the same name everywhere: MAXIM_LOG_REPO_ID.

- throw new Error("MAXIM_BASE_URL, MAXIM_API_KEY & LOG_REPO_ID environment variables are required");
+ throw new Error("MAXIM_BASE_URL, MAXIM_API_KEY & MAXIM_LOG_REPO_ID environment variables are required");

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

src/lib/logger/vercel/vercelLoggerV2.test.ts (1)

685-701: Wrap case-local declarations in 'median' to satisfy noSwitchDeclarations.

Biome flags const declarations in switch without a block; scope them with braces.

- case "median":
- const sorted = [...data].sort((a, b) => a - b);
- const mid = Math.floor(sorted.length / 2);
- return { 
- result: sorted.length % 2 === 0 ? (sorted[mid - 1] + sorted[mid]) / 2 : sorted[mid],
- type: "median",
- dataPoints: data.length 
- };
+ case "median": {
+ const sorted = [...data].sort((a, b) => a - b);
+ const mid = Math.floor(sorted.length / 2);
+ return {
+ result: sorted.length % 2 === 0 ? (sorted[mid - 1] + sorted[mid]) / 2 : sorted[mid],
+ type: "median",
+ dataPoints: data.length,
+ };
+ }

src/lib/tests/vercel/agent_testing.ts (1)

95-123: Also cleanup the Maxim instance; destructure it from initializeMaxim.

Prevents agent/socket leaks and ensures flush. This was flagged earlier as well.

- const { logger } = await initializeMaxim();
+ const { maxim, logger } = await initializeMaxim();
 try {
 ...
 } catch (error) {
 console.error("Error in city prediction demo:", error);
 } finally {
 await logger.cleanup();
+ await maxim.cleanup();
 }

src/lib/logger/vercel/vercelLoggerV2.test.ts (2)

682-711: Enum has "mode" but switch lacks a handler. Add a mode case.

Prevents default/error path for a declared option.

 switch (analysisType) {
 case "mean":
 return { 
 result: data.reduce((a, b) => a + b, 0) / data.length,
 type: "mean",
 dataPoints: data.length 
 };
- case "median": {
+ case "median": {
 const sorted = [...data].sort((a, b) => a - b);
 const mid = Math.floor(sorted.length / 2);
 return { 
 result: sorted.length % 2 === 0 ? (sorted[mid - 1] + sorted[mid]) / 2 : sorted[mid],
 type: "median",
 dataPoints: data.length 
 };
 }
+ case "mode": {
+ const counts = new Map<number, number>();
+ for (const n of data) counts.set(n, (counts.get(n) ?? 0) + 1);
+ let mode = data[0];
+ let freq = 0;
+ for (const [k, v] of counts) {
+ if (v > freq) {
+ mode = k;
+ freq = v;
+ }
+ }
+ return { result: mode, type: "mode", dataPoints: data.length };
+ }
 case "range":
 return { 
 result: Math.max(...data) - Math.min(...data),
 type: "range",
 min: Math.min(...data),
 max: Math.max(...data),
 dataPoints: data.length 
 };

---

492-516: Prefer rejects assertion over try/catch + fail in error test.

Cleaner and avoids false positives.

- try {
- // Intentionally cause an error with invalid parameters
- await generateText({
- model: model,
- maxOutputTokens: -1, // Invalid token count
- prompt: "This should cause an error.",
- providerOptions: {
- maxim: {
- traceName: "V2 Error Handling Test",
- generationName: "Invalid Parameters",
- generationTags: {
- test_type: "error_handling",
- specification: "v2",
- },
- } as MaximVercelProviderMetadata,
- },
- });
- 
- // If we reach here, the test should fail
- fail("Expected an error to be thrown");
- } catch (error) {
- console.log("Expected error caught in V2 error handling test:", error);
- expect(error).toBeDefined();
- }
+ await expect(
+ generateText({
+ model: model,
+ maxOutputTokens: -1,
+ prompt: "This should cause an error.",
+ providerOptions: {
+ maxim: {
+ traceName: "V2 Error Handling Test",
+ generationName: "Invalid Parameters",
+ generationTags: { test_type: "error_handling", specification: "v2" },
+ } as MaximVercelProviderMetadata,
+ },
+ })
+ ).rejects.toThrow();

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro