Skip to content

Conversation

@cp89cyber
Copy link
Contributor

@cp89cyber cp89cyber commented Oct 19, 2025

I just added Poe API support

Summary by CodeRabbit

  • New Features
    • Added Poe as a new AI language model provider option (disabled by default)
    • Integrated Poe into provider selection and configuration UI
    • Automatic model discovery for Poe with models grouped/labelled "Poe" in lists
    • Added Poe icon/color for visual identification in model/provider displays

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Oct 19, 2025

Walkthrough

Registers a new Poe LLM provider: adds configuration, a PoeProvider subclass of OpenAICompatibleProvider, wires it into the provider factory, and adds a Poe model icon mapping for the UI.

Changes

Cohort / File(s) Summary
Provider Registry
src/main/presenter/configPresenter/providers.ts
Added a poe provider entry to DEFAULT_PROVIDERS (id: 'poe', name: 'Poe', apiType: 'poe', baseUrl: 'https://api.poe.com/v1', enable: false) and associated websites metadata (official, apiKey, docs, models, defaultBaseUrl).
Provider Factory
src/main/presenter/llmProviderPresenter/index.ts
Imported PoeProvider and added cases to instantiate it: case 'poe' in the provider.id switch and case 'poe' in the provider.apiType fallback switch.
Provider Implementation
src/main/presenter/llmProviderPresenter/providers/poeProvider.ts
Added PoeProvider class extending OpenAICompatibleProvider; overrides fetchOpenAIModels to delegate to the base method then set each model's group to "Poe".
UI Icon Mapping
src/renderer/src/components/icons/ModelIcon.vue
Imported Poe color icon asset and added a poe entry to the icons map; models with "poe" in their modelId now use the Poe color icon.

Sequence Diagram(s)

sequenceDiagram
  participant Config as ConfigRegistry
  participant Factory as LLMProviderFactory
  participant Provider as PoeProvider
  participant UI as ModelListUI

  Note over Config,Factory: App reads DEFAULT_PROVIDERS
  Config->>Factory: provide provider entry (id: "poe", apiType: "poe")
  Factory->>Provider: instantiate PoeProvider(provider, configPresenter)
  Provider->>Provider: fetchOpenAIModels() [calls super]
  Provider-->>Factory: return MODEL_META (group set to "Poe")
  Factory->>UI: expose provider + models
  UI->>UI: ModelIcon lookup (modelId contains "poe") -> use Poe icon
Loading

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

  • feat: add TokenFlux.ai provider support #804 — Adds a new LLM provider via DEFAULT_PROVIDERS, provider factory wiring, provider subclass, and icon registration (same multi-file pattern).
  • Add Groq as Provider #609 — Similar addition of an LLM provider that modifies the same config, presenter factory, provider subclass, and icon mapping.
  • feat: add cherryin provider #993 — Adds an OpenAI-compatible provider with changes across DEFAULT_PROVIDERS, LLMProviderPresenter switches, provider implementation, and UI icon entries.

Poem

🐰 I hopped in with a pocket of code,
Poe stitched to the list, along the road.
Models grouped neat, an icon to show,
Factory shakes paw, and off we go! 🥕

Pre-merge checks and finishing touches

❌ Failed checks (1 inconclusive)
Check name Status Explanation Resolution
Title Check ❓ Inconclusive The title "Add poe" is directly related to the changeset, which adds support for Poe as a new provider across multiple files (configuration, provider instantiation, and UI). However, the title is extremely terse and lacks sufficient context to clearly communicate the change to a teammate scanning the repository history. Without additional context, "poe" alone is ambiguous—a developer might not immediately understand it refers to the Poe API provider integration. While the title avoids being completely vague like "misc updates," it falls short of the clarity standard that would allow a teammate to understand the primary change at a glance. Consider revising the title to be more descriptive and clear, such as "Add Poe provider integration" or "Add Poe API support." This would immediately communicate to reviewers and future developers that a new provider is being added and provide sufficient context without requiring them to read the full PR description.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.
✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 00f3741 and 18894ac.

📒 Files selected for processing (1)
  • src/main/presenter/llmProviderPresenter/providers/poeProvider.ts (1 hunks)
🚧 Files skipped from review as they are similar to previous changes (1)
  • src/main/presenter/llmProviderPresenter/providers/poeProvider.ts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@zerob13
Copy link
Collaborator

zerob13 commented Oct 19, 2025

@codex review

@chatgpt-codex-connector
Copy link

Codex Review: Didn't find any major issues. What shall we delve into next?

ℹ️ About Codex in GitHub

Your team has set up Codex to 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 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@zerob13
Copy link
Collaborator

zerob13 commented Oct 19, 2025

Please run pnpm format, pnpm lint, and pnpm build, and only merge after they all pass successfully. Thank you.

@zerob13 zerob13 merged commit 660bc80 into ThinkInAIXYZ:dev Oct 19, 2025
2 checks passed
@zerob13
Copy link
Collaborator

zerob13 commented Oct 19, 2025

@cp90-pixel Thank you very much for your contribution, the pr has been merged. If you’re interested, you can also contribute to the https://github.com/ThinkInAIXYZ/PublicProviderConf repository by providing default configuration information for the ‘poe’ Provider’s models.
You can refer to the templates in the https://github.com/ThinkInAIXYZ/PublicProviderConf/tree/dev/manual-templates directory, create a new poe.json file, and follow the example of alibaba.json to add the configuration.
Thank you again for your contribution.

@coderabbitai coderabbitai bot mentioned this pull request Oct 19, 2025
zerob13 added a commit that referenced this pull request Oct 22, 2025
* style(settings): format about page link handler (#1016)

* style(ollama): format model config handlers (#1018)

* fix: think content scroll issue (#1023)

* fix: remove shimmer for think content

* chore: update screen shot and fix scroll issue

* chore: update markdown renderer

* fix: import button bug and prevent backup overwriting during import (#1024)

* fix(sync): fix import button bug and prevent backup overwriting during import

* fix(sync): fix import button bug and prevent backup overwriting during import

* fix(sync): fix import button bug and prevent backup overwriting during import

* refactor(messageList): refactor message list ui components (#1026)

* feat: remove new thread button, add clean button.

* refactor(messageList): refactor message list ui components

* feat: add configurable fields for chat settings

- Introduced ConfigFieldHeader component for consistent field headers.
- Added ConfigInputField, ConfigSelectField, ConfigSliderField, and ConfigSwitchField components for various input types.
- Created types for field configurations in types.ts to standardize field definitions.
- Implemented useChatConfigFields composable to manage field configurations dynamically.
- Added useModelCapabilities and useModelTypeDetection composables for handling model-specific capabilities and requirements.
- Developed useSearchConfig and useThinkingBudget composables for managing search and budget configurations.

* feat: implement input history management in prompt input

- Added `useInputHistory` composable for managing input history and navigation.
- Implemented methods for setting, clearing, and confirming history placeholders.
- Integrated arrow key navigation for browsing through input history.

feat: enhance mention data handling in prompt input

- Created `useMentionData` composable to aggregate mention data from selected files and MCP resources.
- Implemented watchers to update mention data based on selected files, MCP resources, tools, and prompts.

feat: manage prompt input configuration with store synchronization

- Developed `usePromptInputConfig` composable for managing model configuration.
- Implemented bidirectional sync between local config and chat store.
- Added debounced watcher to reduce updates and improve performance.

feat: streamline TipTap editor operations in prompt input

- Introduced `usePromptInputEditor` composable for managing TipTap editor lifecycle and content transformation.
- Implemented methods for handling mentions, pasting content, and clearing editor content.

feat: handle file operations in prompt input

- Created `usePromptInputFiles` composable for managing file selection, paste, and drag-drop operations.
- Implemented methods for processing files, handling dropped files, and clearing selected files.

feat: manage rate limit status in prompt input

- Developed `useRateLimitStatus` composable for displaying and polling rate limit status.
- Implemented methods for handling rate limit events and computing status icons, classes, and tooltips.

* refactor(artifacts): migrate component logic to composables and update documentation

- Refactor ArtifactDialog.vue to use composables for view mode, viewport size, code editor, and export functionality
- Simplify HTMLArtifact.vue by removing drag-resize logic and using fixed viewport dimensions
- Clean up MermaidArtifact.vue styling and structure
- Update component refactoring guide to reflect new patterns and best practices
- Adjust prompt input composable to allow delayed editor initialization
- Update internationalization files for new responsive label

* fix(lint): unused variables

* fix(format): format code

* CodeRabbit Generated Unit Tests: Add renderer unit tests for components and composables

* feat: implement input history management in chat input component

- Added `useInputHistory` composable for managing input history and placeholder navigation.
- Implemented methods for setting, clearing, and confirming history placeholders.
- Integrated arrow key navigation for cycling through input history.

feat: enhance mention data handling in chat input

- Created `useMentionData` composable to manage mention data aggregation.
- Implemented watchers for selected files and MCP resources/tools/prompts to update mention data.

feat: manage prompt input configuration and synchronization

- Developed `usePromptInputConfig` composable for managing model configuration.
- Implemented bidirectional sync between local config refs and chat store.
- Added debounced watcher to reduce updates to the store.

feat: manage prompt input editor operations

- Introduced `usePromptInputEditor` composable for handling TipTap editor operations.
- Implemented content transformation, mention insertion, and paste handling.
- Added methods for handling editor updates and restoring focus.

feat: handle prompt input files management

- Created `usePromptInputFiles` composable for managing file operations in prompt input.
- Implemented file selection, paste, drag-drop, and prompt files integration.

feat: implement rate limit status management

- Developed `useRateLimitStatus` composable for managing rate limit status display and polling.
- Added methods for retrieving rate limit status icon, class, tooltip, and wait time formatting.

* feat: enhance chat input component with context length management and settings integration

* feat: update model configuration and enhance error handling in providers

* feat: add MCP tools list component and integrate with chat settings
feat: enhance artifact dialog with improved error handling and localization
fix: update Mermaid artifact rendering error handling and localization
fix: improve input settings error handling and state management
fix: update drag and drop composable to handle drag events correctly
fix: update Vitest configuration for better project structure and alias resolution

* fix(i18n): add unknownError translation

---------

Co-authored-by: deepinsect <deepinsect@github.com>
Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>

* feat: add Poe provider integration and icon support  (#1028)

* feat: add Poe provider integration and icon support

* chore: format and lint

---------

Co-authored-by: zerob13 <zerob13@gmail.com>

* fix: make auto scroll works (#1030)

* fix: allow settings window links to open externally (#1029)

* fix(settings): allow target blank links

* fix: harden settings window link handling

* feat: enhance GitHub Copilot Device Flow with OAuth token management and API token retrieval (#1021)

* feat: enhance GitHub Copilot Device Flow with OAuth token management and API token retrieval

- Fixed request header for managing OAuth tokens and retrieving API tokens.
- Enhanced model definitions and added new models for better compatibility.

* fix: remove privacy related log

* fix: OAuth 2.0 for slow_down response

* fix: handle lint errors

* fix: provider fetched from publicdb

* fix(githubCopilotProvider): update request body logging format for clarity

* fix(githubCopilotProvider): improve error handling and logging in device flow

* feat(theme): fix message paragraph gap and toolcall block (#1031)

Co-authored-by: deepinsect <deepinsect@github.com>

* fix: scroll to bottom (#1034)

* fix: add debounce for renderer

* feat: add max wait for renderer

* chore(deps): upgrade markdown renderer add worker support

* chore: bump markdown version

* fix(build): use es module worker format (#1037)

* feat: remove function deleteOllamaModel (#1036)

* feat: remove function deleteOllamaModel

* fix(build): use es module worker format (#1037)

---------

Co-authored-by: duskzhen <zerob13@gmail.com>

* perf: update dependencies to use stream-monaco and bump vue-renderer-markdown version (#1038)

* feat(theme): add markdown layout style and table style (#1039)

* feat(theme): add markdown layout style and table style

* fix(lint): remove props

---------

Co-authored-by: deepinsect <deepinsect@github.com>

* feat: support effort and verbosity (#1040)

* chore: bump up version

* feat: add jiekou.ai as LLM provider (#1041)

* feat: add jiekou.ai as LLM provider

* fix: change api type to jiekou

---------

Co-authored-by: zerob13 <zerob13@gmail.com>

* chore: update provider db

---------

Co-authored-by: 韦伟 <xweimvp@gmail.com>
Co-authored-by: Happer <ericted8810us@gmail.com>
Co-authored-by: deepinsect <deepinsect@github.com>
Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
Co-authored-by: cp90 <153345481+cp90-pixel@users.noreply.github.com>
Co-authored-by: Cedric <14017092+douyixuan@users.noreply.github.com>
Co-authored-by: Simon He <57086651+Simon-He95@users.noreply.github.com>
Co-authored-by: yyhhyyyyyy <yyhhyyyyyy8@gmail.com>
Co-authored-by: cnJasonZ <gbdzxalbb@qq.com>
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