feature: providerlog context is no longer a json

We think it's better if context is just a string with the full conversation excluding the response.
latitude-dev · Sep 29, 2024 · 82e97dd · 82e97dd
1 parent cdf58dc
commit 82e97dd
Show file tree

Hide file tree

Showing 6 changed files with 463 additions and 9 deletions.
diff --git a/...mentUuid]/evaluations/dashboard/connect/_components/ConnectEvaluationModal/List/index.tsx b/...mentUuid]/evaluations/dashboard/connect/_components/ConnectEvaluationModal/List/index.tsx
@@ -23,7 +23,7 @@ export default function EvaluationList({
   onSearchChange,
 }: EvaluationListProps) {
   return (
-    <div className='w-1/2'>
+    <div className='w-1/2 max-h-[520px] overflow-y-auto'>
       <Input
         type='text'
         placeholder='Search evaluations and templates...'

diff --git a/packages/core/src/helpers.ts b/packages/core/src/helpers.ts
@@ -4,7 +4,7 @@ import { ProviderApiKey } from './browser'
 
 export function objectToString(object: any) {
   try {
-    return JSON.stringify(object)
+    return JSON.stringify(object, null, 2)
   } catch (error) {
     return 'Error: Provider returned an object that could not be stringified'
   }

diff --git a/packages/core/src/repositories/documentVersionsRepository/index.test.ts b/packages/core/src/repositories/documentVersionsRepository/index.test.ts
@@ -47,7 +47,10 @@ describe('DocumentVersionsRepository', () => {
 
       const documents = result.unwrap()
       expect(documents).toHaveLength(2)
-      expect(documents.map((d) => d.path)).toEqual(['foo', 'bar'])
+      expect(documents.map((d) => d.path).sort()).toEqual(['bar', 'foo'])
+      expect(new Set(documents.map((d) => d.path))).toEqual(
+        new Set(['foo', 'bar']),
+      )
       expect(documents.every((d) => d.commitId === commit1Id)).toBe(true)
     })
 

diff --git a/packages/core/src/services/chains/run.test.ts b/packages/core/src/services/chains/run.test.ts
@@ -2,7 +2,7 @@ import { Chain, ContentType, MessageRole } from '@latitude-data/compiler'
 import { v4 as uuid } from 'uuid'
 import { beforeEach, describe, expect, it, vi } from 'vitest'
 
-import { Workspace } from '../../browser'
+import { objectToString, Workspace } from '../../browser'
 import { LogSources, Providers } from '../../constants'
 import * as factories from '../../tests/factories'
 import * as aiModule from '../ai'
@@ -159,7 +159,7 @@ describe('runChain', () => {
       expect.objectContaining({
         documentLogUuid: expect.any(String),
         object: { name: 'John', age: 30 },
-        text: '{"name":"John","age":30}',
+        text: objectToString({ name: 'John', age: 30 }),
         usage: { totalTokens: 15 },
       }),
     )
@@ -377,7 +377,7 @@ describe('runChain', () => {
       expect.objectContaining({
         documentLogUuid: expect.any(String),
         object: { name: 'John', age: 30 },
-        text: '{"name":"John","age":30}',
+        text: objectToString({ name: 'John', age: 30 }),
         usage: { totalTokens: 15 },
       }),
     )
@@ -463,7 +463,10 @@ describe('runChain', () => {
           { name: 'John', age: 30 },
           { name: 'Jane', age: 25 },
         ],
-        text: '[{"name":"John","age":30},{"name":"Jane","age":25}]',
+        text: objectToString([
+          { name: 'John', age: 30 },
+          { name: 'Jane', age: 25 },
+        ]),
         usage: { totalTokens: 20 },
       }),
     )