[codex] Retarget org plugin contributions to platform repos#2
Conversation
📝 WalkthroughWalkthroughThe profile README is updated to redirect users from a generic "Plugins" catalog link to a dedicated "Add-ons" catalog, and introduces a new contribution section establishing clear rules for plugin contributions and strict compliance requirements for third-party provider integrations. ChangesProfile README Ecosystem Documentation
Estimated code review effort🎯 2 (Simple) | ⏱️ ~8 minutes Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
There was a problem hiding this comment.
🧹 Nitpick comments (1)
profile/README.md (1)
56-57: ⚡ Quick winConsider breaking the compliance requirements into bullet points for better readability.
The sentence covering compliance requirements is grammatically correct but quite dense with multiple prohibitions. Breaking it into a bulleted list would make it easier for plugin developers to understand and follow each requirement.
📋 Proposed refactor for improved clarity
-Plugins that connect to external providers must use access paths the provider authorizes for third-party integrations. They must not impersonate a provider's first-party client, use consumer subscription credentials as API access unless the provider explicitly supports that third-party path, or rely on unofficial OAuth clients, copied client IDs, hidden endpoints, or token refresh flows intended for another first-party product. +Plugins that connect to external providers must use access paths the provider authorizes for third-party integrations. They must not: + +- Impersonate a provider's first-party client +- Use consumer subscription credentials as API access unless the provider explicitly supports that third-party path +- Rely on unofficial OAuth clients +- Use copied client IDs +- Use hidden endpoints +- Use token refresh flows intended for another first-party product🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@profile/README.md` around lines 56 - 57, Split the long compliance sentence in profile/README.md into a short introductory sentence followed by a bulleted list where each prohibition is its own item (e.g., "Must not impersonate a provider's first-party client", "Must not use consumer subscription credentials as API access unless explicitly supported", "Must not rely on unofficial OAuth clients or copied client IDs", "Must not use hidden endpoints or token refresh flows intended for another first-party product"); update the surrounding text to introduce the list and keep original wording for each rule to avoid changing intent.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Nitpick comments:
In `@profile/README.md`:
- Around line 56-57: Split the long compliance sentence in profile/README.md
into a short introductory sentence followed by a bulleted list where each
prohibition is its own item (e.g., "Must not impersonate a provider's
first-party client", "Must not use consumer subscription credentials as API
access unless explicitly supported", "Must not rely on unofficial OAuth clients
or copied client IDs", "Must not use hidden endpoints or token refresh flows
intended for another first-party product"); update the surrounding text to
introduce the list and keep original wording for each rule to avoid changing
intent.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro Plus
Run ID: 36bc9cbb-60db-4493-b9d1-94de65a3e96a
📒 Files selected for processing (1)
profile/README.md
Summary
typewhisper-pluginsrepository row.TypeWhisper/typewhisper-macand Windows plugin contributions toTypeWhisper/typewhisper-win.Companion
Validation
git diff --checkSummary by CodeRabbit