Skip to content

Add custom LLM copy action - #5364

Merged
chrarnoldus merged 3 commits into
mainfrom
feature/custom-llm-copy
Aug 19, 2026
Merged

Add custom LLM copy action#5364
chrarnoldus merged 3 commits into
mainfrom
feature/custom-llm-copy

Conversation

@chrarnoldus

Copy link
Copy Markdown
Contributor

Summary

  • add a copy action to each custom LLM row
  • prompt for a new public ID and display name
  • copy the full definition and encrypted credentials without exposing secrets
  • reject copies that would overwrite an existing custom LLM

Testing

  • local tests not run; CI will handle verification as requested
  • formatting and whitespace checks passed

Co-authored-by: kiloconnect[bot] <240665456+kiloconnect[bot]@users.noreply.github.com>
@chrarnoldus chrarnoldus self-assigned this Aug 19, 2026
@kilo-code-bot

kilo-code-bot Bot commented Aug 19, 2026

Copy link
Copy Markdown
Contributor

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Executive Summary

Reviewed the copy-dialog prefill of source public ID and display name; no new issues on the incremental change.

Files Reviewed (1 files)
  • apps/web/src/app/admin/custom-llms/CustomLlmsContent.tsx
Previous Review Summary (commit 90c4154)

Current summary above is authoritative. Previous snapshots are kept for context only.

Previous review (commit 90c4154)

Status: No Issues Found | Recommendation: Merge

Executive Summary

Reviewed the custom LLM copy mutation, admin UI, hook, and tests; the copy path is admin-gated, avoids overwriting existing rows, and does not expose encrypted credentials.

Files Reviewed (4 files)
  • apps/web/src/app/admin/api/custom-llms/hooks.ts
  • apps/web/src/app/admin/custom-llms/CustomLlmsContent.tsx
  • apps/web/src/routers/admin/custom-llm-router.test.ts
  • apps/web/src/routers/admin/custom-llm-router.ts

Reviewed by grok-4.6 · Input: 88.1K · Output: 4.3K · Cached: 107.3K

Review guidance: REVIEW.md from base branch main

chrarnoldus and others added 2 commits August 19, 2026 10:24
Co-authored-by: kiloconnect[bot] <240665456+kiloconnect[bot]@users.noreply.github.com>
@chrarnoldus
chrarnoldus merged commit 7fa4eb5 into main Aug 19, 2026
18 checks passed
@chrarnoldus
chrarnoldus deleted the feature/custom-llm-copy branch August 19, 2026 09:01
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.

2 participants