scriptoriumadmin/litellm
1
0
Fork 0
mirror of https://github.com/BerriAI/litellm.git synced 2026-09-12 23:01:41 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 88

fix(skills): keep review routes within the type discipline budget

Browse source 
Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
This commit is contained in:
yassin 2026-08-12 03:42:48 +00:00
parent 7652824325
commit 4cccfe6408
1 changed files with 12 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

19
litellm/proxy/anthropic_endpoints/claude_code_endpoints/claude_code_marketplace.py
View file

@ -27,6 +27,7 @@ import json
import re
from collections.abc import Mapping, Sequence
from datetime import datetime, timezone
from types import MappingProxyType
from typing import Annotated, Final, Protocol, TypedDict
from fastapi import APIRouter, Depends, HTTPException
@ -92,6 +93,12 @@ def published_skill_filter() -> dict[str, object]: # mutable-ok: prisma query a
return {"enabled": True, "approval_status": SKILL_ACTIVE} # mutable-ok: prisma query arguments are dicts
def _submitter_edit_resets_review(*, is_admin: bool) -> Mapping[str, object]:
if is_admin:
return MappingProxyType({})
return MappingProxyType({"approval_status": SKILL_PENDING_REVIEW, "enabled": False})
def _as_approval_status(raw: str | None) -> SkillApprovalStatus:
"""Rows written before approval existed, and any hand-edited row, read back as active."""
match raw:
@ -630,9 +637,7 @@ async def update_plugin(
"manifest_json": json.dumps(manifest),
"files_json": "{}",
"updated_at": datetime.now(timezone.utc),
**(
{} if is_admin else {"approval_status": SKILL_PENDING_REVIEW, "enabled": False}
), # mutable-ok: prisma query arguments must be plain dicts
**_submitter_edit_resets_review(is_admin=is_admin),
},
)
@ -709,8 +714,8 @@ async def _record_review(
@router.post(
"/claude-code/plugins/{plugin_name}/approve",
tags=["Claude Code Marketplace"],
dependencies=[Depends(user_api_key_auth)],
tags=["Claude Code Marketplace"], # mutable-ok: FastAPI route decorators take lists
dependencies=[Depends(user_api_key_auth)], # mutable-ok: FastAPI route decorators take lists
response_model=ReviewPluginResponse,
)
async def approve_plugin(
@ -739,8 +744,8 @@ async def approve_plugin(
@router.post(
"/claude-code/plugins/{plugin_name}/reject",
tags=["Claude Code Marketplace"],
dependencies=[Depends(user_api_key_auth)],
tags=["Claude Code Marketplace"], # mutable-ok: FastAPI route decorators take lists
dependencies=[Depends(user_api_key_auth)], # mutable-ok: FastAPI route decorators take lists
response_model=ReviewPluginResponse,
)
async def reject_plugin(