Skip to content

COE-370: Implement sessionless LiteLLM key lifecycle commands #45

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.

Sign up for GitHub

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

Open
kumanday wants to merge 7 commits into
base: main
Choose a base branch
from
Open

COE-370: Implement sessionless LiteLLM key lifecycle commands #45

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
93b8352
COE-370: Implement sessionless LiteLLM key lifecycle commands
kumanday Apr 22, 2026
a96c6b5
fix: formatting and type-check errors for failing CI checks\n\n- ruff…
kumanday Apr 22, 2026
bbe8b65
refactor: address AI review feedback on PR #45
kumanday Apr 22, 2026
d5682e4
fix(proxy-key-service): explicit None checks for budget_amount, allow…
kumanday Apr 22, 2026
65db609
fix(cli): accept alias in revoke, add budget/metadata to create
kumanday Apr 22, 2026
2f4a830
fix(proxy-key-service): address PR #45 feedback round 2
kumanday Apr 22, 2026
7d8dbe2
test(proxy): verify TTL rounding with usage policy (ceil division)
kumanday Apr 22, 2026
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
4 changes: 4 additions & 0 deletions src/benchmark_core/config.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -289,6 +289,10 @@ class UsagePolicyProfile(BaseModel):
default=None,
description="Budget limit in currency units",
)
budget_currency: str = Field(
default="USD",
description="Budget currency",
)
ttl_seconds: int | None = Field(
default=None,
description="Key TTL in seconds (time-to-live before expiration)",
Expand Down
19 changes: 14 additions & 5 deletions src/benchmark_core/services/__init__.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,9 +6,19 @@

# Comprehensive SQL-based services (COE-305 implementation)
from benchmark_core.services.benchmark_metadata_service import BenchmarkMetadataService

# Unified credential service (COE-370)
from benchmark_core.services.credential_service import CredentialService
from benchmark_core.services.experiment_service import ExperimentService
from benchmark_core.services.harness_profile_service import HarnessProfileService
from benchmark_core.services.provider_service import ProviderService
Comment thread
kumanday marked this conversation as resolved.

# Sessionless proxy key services
from benchmark_core.services.proxy_key_service import (
LiteLLMAPIError,
ProxyKeyService,
ProxyKeyServiceError,
)
from benchmark_core.services.rendering import (
EnvRenderingService,
EnvSnippet,
Expand All @@ -25,11 +35,6 @@
from benchmark_core.services.task_card_service import TaskCardService
from benchmark_core.services.variant_service import VariantService

# ABC service exports - Note: old services_abc SessionService removed, use services.session_service
from benchmark_core.services_abc import (
CredentialService,
)

__all__ = [
# Core SQL-based services
"SessionService",
Expand All @@ -49,6 +54,10 @@
"RenderingError",
"ProfileValidationError",
"render_env_for_session",
# Sessionless proxy key services
"ProxyKeyService",
"ProxyKeyServiceError",
"LiteLLMAPIError",
# ABC services
"CredentialService",
]
Loading
Loading