feat: merge PR #259 (LLM API) and PR #262 (Xpay) into dev#276
Merged
Conversation
Copilot created this pull request from a session on behalf of
canisminor1990
March 26, 2026 08:28
View session
Member
|
❤️ Great PR @Copilot ❤️ |
This was referenced Mar 26, 2026
Closed
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 17a4f608b9
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
| export interface CombineProps extends Omit<RNIconCombineProps, 'Icon' | 'Text'> { | ||
| type?: 'color' | 'mono'; | ||
| } | ||
| const Combine = memo<CombineProps>(({ type = 'color', extraStyle, ...rest }) => { |
There was a problem hiding this comment.
Default LlmApi RN combine variant to mono
<LlmApi.Combine /> in React Native currently defaults type to 'color', which makes the default rendering use the avatar-style logo instead of the mono mark. This is inconsistent with the web implementation (src/LlmApi/components/Combine.tsx, default 'mono') and with the existing RN Combine components, so consumers that rely on the default variant will get an unexpected branded badge layout and different sizing/spacing behavior without opting in.
Useful? React with 👍 / 👎.
canisminor1990
added a commit
that referenced
this pull request
Mar 26, 2026
* ✨ feat(icons): add Qoder brand icon (#270) * ✨ feat(icons): add Qoder brand icon - Add Qoder icon components (Avatar, Color, Mono, Text) - Add Qoder icon documentation - Export Qoder icon in icons.ts - Add Qoder entry in table of contents new file: src/Qoder/components/Avatar.tsx new file: src/Qoder/components/Color.tsx new file: src/Qoder/components/Mono.tsx new file: src/Qoder/components/Text.tsx new file: src/Qoder/index.md new file: src/Qoder/index.ts new file: src/Qoder/style.ts modified: src/icons.ts modified: src/toc.ts * ✨ feat: add Qoder icon - Add Qoder icon with Avatar, Color, Mono, and Text variants - Export Qoder icon from react-native package * ✨ feat: add Coding Plan provider icons support (#272) * ✨ feat: add Coding Plan provider support * ✨ feat: add Coding Plan provider support for React Native * feat: add Apertis icon (#269) * feat: add Apertis icon Add Apertis (apertis.ai) — a unified API platform for 500+ AI models across 30+ providers. Includes Mono, Text, Avatar, and Combine components following the standard icon package structure. Primary color: #0d9488 (Apertis Teal) * feat: export Apertis icon and add react-native support Add Apertis to src/icons.ts barrel export and create the full react-native icon set (Mono, Text, Avatar, Combine) under packages/react-native/src/icons/Apertis. * Update Together AI logo (#268) * wip * wip * Revert "wip" This reverts commit 76dfa33. * wip * feat: add AskVerdict AI brand icons (#265) Add AskVerdict AI (askverdict.ai) — multi-agent AI debate engine. Components: Mono, Color (gold gradient), Text, Combine, Avatar Shared SVG paths extracted to paths.ts to prevent divergence. Brand color: #E8A317 (gold) Website: https://askverdict.ai * ✨ feat: add AgentVoice icon (#257) Co-authored-by: RRyanHoward <ryan@websitellm.com> Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com> * feat: merge PR #259 (LLM API) and PR #262 (Xpay) into dev (#276) * ✨ feat: Add LLM API logo * ✨ feat: add xpay icon * fix: set hasBrandColor to true in toc entry --------- Co-authored-by: Markos Basilio <markosbasilio@MacBook-Pro-Markos.local> Co-authored-by: xPay <xpaysh@gmail.com> Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> * 💄 style: Fix some icon style * ✨ feat: Update rn --------- Co-authored-by: zhi <yantze@126.com> Co-authored-by: Hardy <yimingmail@yandex.com> Co-authored-by: Quert <a97041304@gmail.com> Co-authored-by: Hassan El Mghari <hassan4709@gmail.com> Co-authored-by: GDS K S <39922405+thegdsks@users.noreply.github.com> Co-authored-by: Ryan Howard <ryan@agentvoice.com> Co-authored-by: RRyanHoward <ryan@websitellm.com> Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com> Co-authored-by: Copilot <198982749+Copilot@users.noreply.github.com> Co-authored-by: Markos Basilio <markosbasilio@MacBook-Pro-Markos.local> Co-authored-by: xPay <xpaysh@gmail.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Merges two open PRs adding new provider icons into
dev, resolving conflicts caused bysrc/toc.tsbeing reformatted to JSON-style syntax after both PRs were opened.Changes
https://llmapi.ai, color#3F35FF): Mono, Text, Combine, Avatar variants; includes React Native package supporthttps://xpay.sh, primary#0F1C4D): Mono, Color, Text, Combine, Avatar variantsConflict resolution
Both PRs conflicted on
src/toc.ts— dev had migrated keys to quoted JSON format while both PRs still used unquoted JS object notation. Resolution in both cases:toc.tsLlmApi→ betweenLlamaIndexandLmStudioXpay→ betweenXinferenceandXuanyuansrc/icons.ts,providerEnum.ts,providerConfig.tsx,packages/react-native/src/index.ts) auto-merged cleanly