Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Fixed immutability of tuples #2973

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 2 commits into from
Sep 6, 2025
Merged

Fixed immutability of tuples #2973

merged 2 commits into from
Sep 6, 2025

Conversation

gregpr07
Copy link
Member

@gregpr07 gregpr07 commented Sep 6, 2025 •
edited by cubic-dev-ai bot
Loading

Summary by cubic

Fix tuple handling in recursive URL replacement. We now return new tuples and assign processed containers so replacements apply inside nested structures.

  • Bug Fixes
    • _recursive_process_list_or_tuple now returns a processed container: lists are edited in place; tuples are rebuilt and returned.
    • Call sites updated to set the processed value on models and dicts.
    • Ensures nested BaseModel, dict, list, and tuple items are fully processed without mutating tuples.

@gregpr07 gregpr07 enabled auto-merge September 6, 2025 00:53
@gregpr07 gregpr07 merged commit 3b6205e into main Sep 6, 2025
53 checks passed
@gregpr07 gregpr07 deleted the feature/url-shortener branch September 6, 2025 00:55
@github-actions GitHub Actions
Copy link

github-actions bot commented Sep 6, 2025 •
edited
Loading

Agent Task Evaluation Results: 2/3 (67%)

View detailed results
Task Result Reason
amazon_laptop ✅ Pass The agent successfully navigated to amazon.com, performed a search for 'laptop', and returned detailed information about the first laptop result including the title, rating, number of reviews, price, and additional buying options. This meets all the criteria specified for the task.
browser_use_pip ✅ Pass The agent correctly identified and provided the pip installation command 'pip install browser-use' as requested. The output includes the exact command, meeting the success criteria.
captcha_cloudflare ❌ Fail The agent was unable to solve the Cloudflare Turnstile captcha on the specified page, resulting in no success message or dictionary being displayed. Consequently, it could not extract the 'hostname' value as required. Since the captcha was not solved and the expected hostname 'example.com' was not retrieved, the task was not completed successfully.

Check the evaluate-tasks job for detailed task execution logs.

cubic-dev-ai[bot]
cubic-dev-ai bot reviewed Sep 6, 2025
View reviewed changes
Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 1 file

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

@cubic-dev-ai cubic-dev-ai[bot] cubic-dev-ai[bot] left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@gregpr07