Remove the obsolete Poe web integration - #1032
Conversation
Remove the reverse-engineered Poe web client, its GraphQL queries, and the associated manifest, model, session, and configuration entries. The project already routes OpenAI-compatible providers through the shared API implementation, so retaining a separate Poe protocol path only adds stale maintenance and dependencies.
|
Caution The consumer version of Gemini Code Assist on GitHub has been sunset. All code review activity has officially ceased. |
📝 WalkthroughWalkthroughThe change removes Poe Web support from provider configuration, session state, runtime services, GraphQL operations, extension permissions, build processing, dependencies, and tests. ChangesPoe Web provider removal
Estimated code review effort: 2 (Simple) | ~10 minutes 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
PR Summary by QodoRemove obsolete Poe web integration
AI Description
Diagram
High-Level Assessment
Files changed (10)
|
Code Review by Qodo🐞 Bugs (0) 📘 Rule violations (0) 📎 Requirement gaps (0)
Great, no issues found!Qodo reviewed your code and found no material issues that require reviewTo customize comments, go to the Qodo configuration screen, or learn more in the docs. |
There was a problem hiding this comment.
Pull request overview
This PR removes the obsolete Poe (Web) provider integration from the extension, including its client implementation, model/config surface area, permissions, and build/dependency plumbing, while keeping the shared OpenAI-compatible API paths intact.
Changes:
- Removed Poe session/config/model wiring and updated unit tests accordingly.
- Deleted the Poe web client implementation (websocket + GraphQL artifacts) and the Poe Web API adapter.
- Dropped Poe-related host permissions plus GraphQL build/dependency support that is no longer needed.
Reviewed changes
Copilot reviewed 39 out of 40 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| tests/unit/services/init-session.test.mjs | Removes the Poe-specific session-field default assertion. |
| tests/unit/config/config-predicates.test.mjs | Drops the Poe-only isUsingCustomNameOnlyModel coverage. |
| src/services/init-session.mjs | Removes the Poe session field and its JSDoc typing. |
| src/services/clients/poe/websocket.js | Deletes Poe websocket streaming client implementation. |
| src/services/clients/poe/index.mjs | Deletes the reverse-engineered Poe web client wrapper. |
| src/services/clients/poe/graphql/AutoSubscriptionMutation.graphql | Deletes Poe GraphQL artifact. |
| src/services/clients/poe/graphql/AddHumanMessageMutation.graphql | Deletes Poe GraphQL artifact. |
| src/services/clients/poe/graphql/AddMessageBreakMutation.graphql | Deletes Poe GraphQL artifact. |
| src/services/clients/poe/graphql/BioFragment.graphql | Deletes Poe GraphQL artifact. |
| src/services/clients/poe/graphql/ChatAddedSubscription.graphql | Deletes Poe GraphQL artifact. |
| src/services/clients/poe/graphql/ChatFragment.graphql | Deletes Poe GraphQL artifact. |
| src/services/clients/poe/graphql/ChatPaginationQuery.graphql | Deletes Poe GraphQL artifact. |
| src/services/clients/poe/graphql/ChatViewQuery.graphql | Deletes Poe GraphQL artifact. |
| src/services/clients/poe/graphql/DeleteHumanMessagesMutation.graphql | Deletes Poe GraphQL artifact. |
| src/services/clients/poe/graphql/HandleFragment.graphql | Deletes Poe GraphQL artifact. |
| src/services/clients/poe/graphql/LoginWithVerificationCodeMutation.graphql | Deletes Poe GraphQL artifact. |
| src/services/clients/poe/graphql/MessageAddedSubscription.graphql | Deletes Poe GraphQL artifact. |
| src/services/clients/poe/graphql/MessageDeletedSubscription.graphql | Deletes Poe GraphQL artifact. |
| src/services/clients/poe/graphql/MessageFragment.graphql | Deletes Poe GraphQL artifact. |
| src/services/clients/poe/graphql/MessageRemoveVoteMutation.graphql | Deletes Poe GraphQL artifact. |
| src/services/clients/poe/graphql/MessageSetVoteMutation.graphql | Deletes Poe GraphQL artifact. |
| src/services/clients/poe/graphql/SendVerificationCodeForLoginMutation.graphql | Deletes Poe GraphQL artifact. |
| src/services/clients/poe/graphql/ShareMessagesMutation.graphql | Deletes Poe GraphQL artifact. |
| src/services/clients/poe/graphql/SignupWithVerificationCodeMutation.graphql | Deletes Poe GraphQL artifact. |
| src/services/clients/poe/graphql/StaleChatUpdateMutation.graphql | Deletes Poe GraphQL artifact. |
| src/services/clients/poe/graphql/SummarizePlainPostQuery.graphql | Deletes Poe GraphQL artifact. |
| src/services/clients/poe/graphql/SummarizeQuotePostQuery.graphql | Deletes Poe GraphQL artifact. |
| src/services/clients/poe/graphql/SummarizeSharePostQuery.graphql | Deletes Poe GraphQL artifact. |
| src/services/clients/poe/graphql/UserSnippetFragment.graphql | Deletes Poe GraphQL artifact. |
| src/services/clients/poe/graphql/ViewerInfoQuery.graphql | Deletes Poe GraphQL artifact. |
| src/services/clients/poe/graphql/ViewerStateFragment.graphql | Deletes Poe GraphQL artifact. |
| src/services/clients/poe/graphql/ViewerStateUpdatedSubscription.graphql | Deletes Poe GraphQL artifact. |
| src/services/apis/poe-web.mjs | Removes Poe Web API entrypoint. |
| src/manifest.json | Removes Poe host permission from MV3 manifest. |
| src/manifest.v2.json | Removes Poe host permission from MV2 manifest. |
| src/config/index.mjs | Removes Poe model keys, model entries, and deprecated predicate/config fields. |
| build.mjs | Removes GraphQL loader plumbing that was only used by Poe artifacts. |
| package.json | Removes Poe-only dependencies (diff, graphql, md5, graphql-tag). |
| package-lock.json | Removes locked Poe-only dependencies and transitive packages. |
| AGENTS.md | Updates provider list to no longer mention Poe (Web). |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Summary
Validation
npm cinpm run prettynpm run lintnpm test(948 passed)npm run buildSummary by CodeRabbit