Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: update prompts and invalidate cache #493

Merged
merged 25 commits into from
Jan 28, 2025
Merged
Show file tree
Hide file tree
Changes from 7 commits
Commits
Show all changes
25 commits
Select commit Hold shift + click to select a range
78dbaff
feat: update prompts and invalidate cache
maxdeichmann Jan 20, 2025
3a35096
feat: update prompts and invalidate cache
maxdeichmann Jan 20, 2025
cd3fb2b
feat: update prompts and invalidate cache
maxdeichmann Jan 20, 2025
e03ee35
feat: update prompts and invalidate cache
maxdeichmann Jan 20, 2025
3faa622
feat: update prompts and invalidate cache
maxdeichmann Jan 20, 2025
5622d7c
feat: update prompts and invalidate cache
maxdeichmann Jan 20, 2025
509d547
feat: update prompts and invalidate cache
maxdeichmann Jan 20, 2025
512a35e
feat: update prompts and invalidate cache
maxdeichmann Jan 20, 2025
272766b
feat: update prompts and invalidate cache
maxdeichmann Jan 20, 2025
bc7162f
feat: update prompts and invalidate cache
maxdeichmann Jan 20, 2025
4cf0625
feat: update prompts and invalidate cache
maxdeichmann Jan 20, 2025
611d816
Update integration-test/langfuse-integration-node.spec.ts
hassiebp Jan 21, 2025
b983d5f
Update langfuse-core/src/openapi/server.ts
hassiebp Jan 21, 2025
d5fd31f
Merge branch 'main' into max/lfe-3587-update-prompt-versions
maxdeichmann Jan 24, 2025
3c820da
Merge branch 'main' into max/lfe-3587-update-prompt-versions
maxdeichmann Jan 24, 2025
a1d9a9a
fix
maxdeichmann Jan 27, 2025
e4c6d90
push
maxdeichmann Jan 27, 2025
bfe3dd0
Merge branch 'main' into max/lfe-3587-update-prompt-versions
maxdeichmann Jan 27, 2025
f2a171a
fix
maxdeichmann Jan 27, 2025
0cf97b9
Merge branch 'max/lfe-3587-update-prompt-versions' of github.com:lang…
maxdeichmann Jan 27, 2025
70e52db
fix
maxdeichmann Jan 27, 2025
d9acb0f
fix
maxdeichmann Jan 27, 2025
0168fec
push
maxdeichmann Jan 27, 2025
27bf6ab
fix
maxdeichmann Jan 27, 2025
6b8d300
push
maxdeichmann Jan 27, 2025
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
20 changes: 20 additions & 0 deletions integration-test/langfuse-integration-node.spec.ts
Original file line number Diff line number Diff line change
Expand Up @@ -432,6 +432,26 @@ describe("Langfuse Node.js", () => {
version: expect.any(Number),
});
});

it.only("update a prompt", async () => {
hassiebp marked this conversation as resolved.
Show resolved Hide resolved
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove it.only to ensure all tests run.

const promptName = "test-prompt" + Math.random().toString(36);
await langfuse.createPrompt({
name: promptName,
prompt: "This is a prompt with a {{variable}}",
isActive: true,
labels: ["john"],
});

const updatedPrompt = await langfuse.updatePrompt({
name: promptName,
version: 1,
newLabels: ["john", "doe"],
});
maxdeichmann marked this conversation as resolved.
Show resolved Hide resolved

const prompt = await langfuse.getPrompt(promptName);
expect(prompt.labels).toEqual(expect.arrayContaining(["john", "doe"]));
expect(updatedPrompt.labels).toEqual(expect.arrayContaining(["john", "doe"]));
});
});

it("link prompt to generation", async () => {
Expand Down
Loading
Loading