Skip to content

Conversation

@ntindle
Copy link
Member

@ntindle ntindle commented Dec 14, 2025

Replaces user/password Reddit credentials with OAuth2, adds RedditOAuthHandler, and updates Reddit blocks to support OAuth2 authentication. Introduces new blocks for creating posts, fetching post details, searching, editing posts, and retrieving subreddit info. Updates test credentials and input handling to use OAuth2 tokens.

Uploading Screen Recording 2025-12-16 at 1.21.23 PM.mov…

reddit super test_v4.json

Changes 🏗️

Rebuild the reddit blocks to support oauth2 rather than requiring users to provide their password and username.
This is done via a swap from script based to web based authentication on the reddit side faciliatated by the approval of an oauth app by reddit on the account ntindle

Checklist 📋

For code changes:

  • I have clearly listed my changes in the PR description
  • I have made a test plan
  • I have tested my changes according to the test plan:
    • Build a super agent
    • Upload the super agent and a video of it working

Replaces user/password Reddit credentials with OAuth2, adds RedditOAuthHandler, and updates Reddit blocks to support OAuth2 authentication. Introduces new blocks for creating posts, fetching post details, searching, editing posts, and retrieving subreddit info. Updates test credentials and input handling to use OAuth2 tokens.
@coderabbitai
Copy link

coderabbitai bot commented Dec 14, 2025

Important

Review skipped

Auto reviews are disabled on this repository.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch ntindle/reddit-rewrite

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.

@github-actions github-actions bot added platform/frontend AutoGPT Platform - Front end platform/backend AutoGPT Platform - Back end platform/blocks labels Dec 14, 2025
@deepsource-io
Copy link

deepsource-io bot commented Dec 14, 2025

Here's the code health analysis summary for commits e8d37ab..fa6148b. View details on DeepSource ↗.

Analysis Summary

AnalyzerStatusSummaryLink
DeepSource JavaScript LogoJavaScript✅ SuccessView Check ↗
DeepSource Python LogoPython✅ SuccessView Check ↗

💡 If you’re a repository administrator, you can configure the quality gates from the settings.

@netlify
Copy link

netlify bot commented Dec 14, 2025

Deploy Preview for auto-gpt-docs-dev canceled.

Name Link
🔨 Latest commit fa6148b
🔍 Latest deploy log https://app.netlify.com/projects/auto-gpt-docs-dev/deploys/694115bd3efc300008344b9b

@netlify
Copy link

netlify bot commented Dec 14, 2025

Deploy Preview for auto-gpt-docs canceled.

Name Link
🔨 Latest commit fa6148b
🔍 Latest deploy log https://app.netlify.com/projects/auto-gpt-docs/deploys/694115bd9bb4170008bd8ee0

@AutoGPT-Agent
Copy link

Thank you for your PR adding OAuth2 support and advanced blocks for Reddit integration. The code changes look good, but there are some issues that need to be addressed before this can be merged:

  1. PR Description Incomplete: You've provided a brief overview, but the "Changes" section is empty. Please fill this section with a concise description of all changes made, including:

    • The switch from username/password to OAuth2
    • The new RedditOAuthHandler implementation
    • Details about each new Reddit block added
    • Any configuration changes needed
  2. Checklist Not Completed: None of the checklist items have been checked off. Please complete the checklist by:

    • Checking all applicable items
    • Providing a test plan that explains how you've verified these changes work correctly

The code itself looks well-structured, with proper implementation of the OAuth2 flow and comprehensive new Reddit blocks for various actions. Once you update the description and complete the checklist, this PR should be ready for another review.

@AutoGPT-Agent
Copy link

Thank you for your PR implementing Reddit OAuth2 integration and advanced Reddit blocks! This is a substantial addition that will improve the platform's Reddit capabilities.

However, before we can proceed with merging:

Required Changes

  • PR Checklist: Please complete the checklist in the PR description. All items need to be checked off for the PR to be accepted, including the test plan section. Since this introduces new functionality, thorough testing is especially important.

Additional Suggestions

  1. Consider expanding the PR description to provide more details about:

    • What specific new Reddit blocks are being added
    • How the OAuth2 flow works compared to the previous authentication method
    • Any configuration changes needed by users
  2. It would be helpful to add information about how users can get the required Reddit OAuth2 credentials (client ID, client secret) for testing.

Once you've addressed the checklist requirement, we can continue with the review process. The code changes themselves look well-structured and consistent with our existing patterns.

@AutoGPT-Agent
Copy link

Thank you for the PR implementing Reddit OAuth2 integration and advanced Reddit blocks. This is a substantial addition to our platform's capabilities. However, there are a few issues that need to be addressed before this can be merged:

  1. Incomplete PR Description: The 'Changes' section is empty. Please provide a detailed description of all changes made, including:

    • The OAuth2 integration implementation details
    • Each new Reddit block functionality added
    • Any changes to existing functionality
  2. Incomplete Checklist: The checklist items are not checked off, indicating the PR may not be ready for review. Please ensure:

    • You've completed all items in the checklist
    • The test plan is more comprehensive for a change of this size (the current plan to "Build a super agent" and "Upload the super agent and a video of it working" lacks specific test scenarios for the Reddit functionality)
  3. Testing: For a feature this complex, the test plan should include specific test cases for:

    • OAuth2 authentication flow
    • Each new Reddit block's functionality
    • Error handling scenarios
    • Credential refresh mechanisms

The core implementation looks solid, but we need these documentation and testing aspects to be completed before merging. Please update the PR description with these details.

@AutoGPT-Agent
Copy link

Thank you for this PR adding Reddit OAuth2 integration and advanced Reddit blocks. The changes look good overall, but there's one issue that needs to be addressed before merging:

Checklist Completion
You've indicated that you have a test plan, but haven't checked off that you've completed testing according to that plan. Please either:

  • Complete the testing according to your test plan (building a super agent and uploading with a video of it working)
  • Update the PR description with the results of your testing

Please check the box in the checklist once testing is complete. This is required for all PRs with material code changes.

The code changes themselves look well-structured - I appreciate the comprehensive approach to the Reddit OAuth2 integration and the addition of multiple new block types for different Reddit operations. Once the checklist is complete, this should be ready for review.

@AutoGPT-Agent
Copy link

Thank you for this comprehensive PR adding Reddit OAuth2 integration and advanced Reddit blocks. The changes look well-structured and implement a more secure authentication method for Reddit integration.

However, before this can be merged, there's one issue that needs to be addressed:

Incomplete Checklist

The PR checklist in your description is not fully completed. You've checked off that you've listed the changes and made a test plan, but you haven't completed the testing according to your plan. According to your checklist, you need to:

  • Build a super agent
  • Upload the super agent and a video of it working

Please complete the testing according to your test plan and check off all items in the checklist before this PR can be approved for merging.

Once the testing is completed and the checklist is fully checked off, this PR should be ready for review again.

@AutoGPT-Agent
Copy link

Thank you for this comprehensive Reddit OAuth2 integration PR! The code changes look well-structured and add valuable new functionality.

Before we can approve this PR, please complete the test plan you've outlined:

  • Build a super agent that uses these Reddit blocks
  • Upload the super agent and provide a video showing it working

This testing is particularly important since we're changing the authentication mechanism from username/password to OAuth2, which is a significant architectural change. We need to verify that the new authentication flow works end-to-end.

Once you've completed the testing part of your checklist, please update the PR and we can proceed with the review.

@AutoGPT-Agent
Copy link

Thank you for your PR to add Reddit OAuth2 integration and advanced Reddit blocks! This is a valuable addition that improves security by replacing password-based authentication with OAuth2.

I noticed that your test plan isn't complete yet - you've marked that you've built a super agent, but haven't yet uploaded the super agent with a video showing it working.

Before this PR can be merged, please:

  1. Complete your testing according to your test plan by uploading the super agent and providing a video demonstration of the functionality working
  2. Check off the remaining test plan item once completed

The code changes look good overall - the OAuth2 integration is well-implemented, and the new Reddit blocks provide useful functionality. The alphabetical ordering of the RedditOAuthHandler in the imports is correct as well.

Please let us know when you've completed the testing so we can proceed with the review.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

platform/backend AutoGPT Platform - Back end platform/blocks platform/frontend AutoGPT Platform - Front end size/xl

Projects

Status: 🆕 Needs initial review
Status: No status

Development

Successfully merging this pull request may close these issues.

3 participants