Skip to content

Comments

Alex/first-version#1

Merged
KrastyTC merged 13 commits intomainfrom
alex/all-original-files
Nov 20, 2025
Merged

Alex/first-version#1
KrastyTC merged 13 commits intomainfrom
alex/all-original-files

Conversation

@KrastyTC
Copy link
Collaborator

No description provided.

Initial implementation of the PromptProvider library with full Langfuse API integration and local prompt fallback. Includes service interfaces, models, options, dependency injection, and README. Supports prompt creation, retrieval, batch operations, and label updates via Langfuse, with local defaults as fallback.
Moved source files out of PromptProvider subfolder to project root and updated namespaces. Added IDefaultPromptsProvider interface and ConfigurationDefaultPromptsProvider implementation for configurable default prompts. Updated dependency injection to register the default prompts provider. Refactored PromptService to use IDefaultPromptsProvider instead of direct options access. Added CI and NuGet publish GitHub workflows, new README, and initial unit tests. Removed unused files and consolidated project files.
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR introduces the initial version of PromptProvider, a .NET 9.0 library for managing prompts with Langfuse integration and local fallback support.

Key Changes:

  • Implements core services (PromptService, LangfuseService) for prompt management with Langfuse API integration
  • Provides dependency injection setup and configuration-based default prompts provider
  • Adds complete model definitions for prompts, requests, and responses with JSON serialization support

Reviewed Changes

Copilot reviewed 27 out of 27 changed files in this pull request and generated 11 comments.

Show a summary per file
File Description
Services/PromptService.cs Main service orchestrating prompt operations with Langfuse and local fallback
Services/LangfuseService.cs HTTP client wrapper for Langfuse API interactions with authentication
Services/ConfigurationDefaultPromptsProvider.cs Simple provider for retrieving default prompts from configuration
Models/*.cs Data models for prompts, requests, responses, and Langfuse API structures
Options/*.cs Configuration option classes including LangfuseOptions and PromptsOptions
Interfaces/*.cs Service and provider interfaces defining the public API contracts
DependencyInjection.cs Extension method for registering PromptProvider services
README.md Library documentation with usage examples and configuration guidance
PromptProvider.csproj Project configuration targeting .NET 9.0 with required dependencies
PromptProvider.sln Visual Studio solution file
LICENSE Simplified MIT license text (incomplete)
.github/workflows/*.yml CI/CD workflows for building, testing, and publishing to NuGet

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

KrastyTC and others added 8 commits November 19, 2025 16:03
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copy link
Contributor

@otapiero otapiero left a comment

Choose a reason for hiding this comment

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

Langfuse also has a different type of prompt, a chat type
When in the request to get or create a prompt, the type is chat and not text, the schema of the prompt changes a lot, and for now, our code will not support them
We should consider adding functions to get and create chat prompts

@KrastyTC KrastyTC merged commit 9c487ab into main Nov 20, 2025
otapiero added a commit that referenced this pull request Feb 16, 2026
…ration

Auto-apply configured Label from PromptConfiguration during prompt fetching
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants