-
Notifications
You must be signed in to change notification settings - Fork 3
Add Computer Use (OpenAI) Guide #61
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Performed full review of 758d73b...fb5679c
Analysis
-
The guide incorrectly references an OpenAI documentation link for Computer Use (https://platform.openai.com/docs/guides/computer-use), when Computer Use is actually associated with Anthropic's Claude models, suggesting a critical factual error.
-
The external documentation link may be non-existent or incorrect, requiring verification before merging to ensure users aren't directed to invalid resources.
-
While the document structure and format follow established patterns, the fundamental accuracy issue calls into question whether this integration is properly aligned with the correct AI provider.
Tip
⚡ Quick Actions
This review was generated by Mesa.
Actions:
Slash Commands:
/review- Request a full code review/review latest- Review only changes since the last review/describe- Generate PR description. This will update the PR body or issue comment depending on your configuration/help- Get help with Mesa commands and configuration options
1 files reviewed | 0 comments | Review on Mesa | Edit Reviewer Settings
|
Can you update the docs.json to ensure this PR fits into the new existing structure? Thanks for the heads-up @dprevoznik. I've updated the docs.json file to include the new Computer Use guide under the Integrations group. The PR now fits into the new structure that was merged from PR #62. The change was straightforward - just added "integrations/computer-use-openai" to the pages array in the Integrations section. |
…uter-use-openai-guide
Moved OpenAI computer use integration to solve merge conflict.
|
@tembo Can you update the computer use link to use: https://platform.openai.com/docs/guides/tools-computer-use And, in the quick start section it should say you can use either TypeScript or Python and then you should choose "CUA Sample" |
And, in the quick start section it should say you can use either TypeScript or Python and then you should choose "CUA Sample" Thanks for your feedback, @dprevoznik! I've updated the guide with the requested changes:
Let me know if you need any other adjustments! |
|
@tembo Actually use this link please for Computer Use: https://openai.com/index/computer-using-agent/ |
Thanks for the feedback, @dprevoznik. I've updated the documentation to use the link you provided for Computer Use (https://openai.com/index/computer-using-agent/) instead of the previous platform docs link. The change is now reflected in the introduction section of the Computer Use integration guide. |
dprevoznik
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Thanks Tembo!
Description
Adds a guide for Computer Use (OpenAI) integration, covering its concept and quick start via
create-kernel-app.Changes
New file
integrations/computer-use-openai.mdxwith guide content.TL;DR
Adds a new guide for the Computer Use (OpenAI) integration.
Why we made these changes
To provide users with documentation on what the Computer Use (OpenAI) integration is and how to get started with it using
create-kernel-app.What changed?
integrations/computer-use/openai.mdx: New guide covering the integration's concept, benefits, and a quick start tutorial.docs.json: Updated site navigation to include the new guide under the "Integrations" section.Validation
Description generated by Mesa. Update settings