Skip to content

Commit

Permalink
fix(core): update prompts (#6738)
Browse files Browse the repository at this point in the history
Update prompts:

When sending a message to openai, we actually send two messages.
There are two issues here:

1. the role in the first message shall be user or system
2. the content in the first message is not being interpolated at all.

```
[
  {
    role: "assistant",
    createdAt: "2024-04-27T06:30:34.820Z",
    params: {
      language: "French",
    },
    content: "You are a translation expert, please translate the following content into French, and only perform the translation action, keeping the translated content in the same format as the original content:\n\n          \"\"\"\"\n\n          \n\n          \"\"\"\"",
  },
  {
    role: "user",
    content: "Basic things you should know:",
    attachments: undefined,
    params: {
      language: "French",
    },
    createdAt: "2024-04-29T09:42:11.466Z",
  },
]
```

In this PR I removed `content` from prompts and let open ai use the second message (user) as the input.
  • Loading branch information
pengx17 committed Apr 29, 2024
1 parent 1d3c477 commit 7c5a259
Show file tree
Hide file tree
Showing 3 changed files with 100 additions and 157 deletions.
Original file line number Diff line number Diff line change
@@ -0,0 +1,13 @@
import { PrismaClient } from '@prisma/client';

import { refreshPrompts } from './utils/prompts';

export class UpdatePrompts1714386922280 {
// do the migration
static async up(db: PrismaClient) {
await refreshPrompts(db);
}

// revert the migration
static async down(_db: PrismaClient) {}
}
Loading

0 comments on commit 7c5a259

Please sign in to comment.