Skip to content

Conversation

@davidgomes
Copy link
Contributor

@davidgomes davidgomes commented May 1, 2025

I tried to make a test for this, but unfortunately it's hard to verify that this is actually working as a fix. Most of the time, Claude is smart and it will pass the search parameter which means that the limit is not actually a problem.

So, we'll have to vibe-trust this PR unfortunately.

CC @Shridhad

@davidgomes davidgomes requested a review from pffigueiredo May 1, 2025 16:33
@davidgomes davidgomes force-pushed the project-limit-hint branch from e81342f to 3dd7e2d Compare May 1, 2025 21:37
@Shridhad Shridhad merged commit 4694081 into main May 2, 2025
2 checks passed
danieltprice added a commit that referenced this pull request Jun 18, 2025
# Update README with New Tools and Features

This PR updates the README.md to include information about the new tools
and features added in recent PRs:

- Added Query Performance Optimization section with:
  - `explain_sql_statement` tool
  - `prepare_query_tuning` tool
  - `complete_query_tuning` tool
  - `list_slow_queries` tool

- Added Compute Management section with:
  - `list_branch_computes` tool

- Updated `list_projects` tool description to mention the default limit
of 10 projects

- Improved clarity and consistency throughout the README

- Updated CHANGELOG.md with an UNRELEASED section for these
documentation changes

## PRs Reviewed
- #52 (list_branch_computes tool)
- #50 (list_projects tool hint)
- #45 (list_slow_queries tool)
- #31 (query tuning tools)

Link to Devin run:
https://app.devin.ai/sessions/f7aaa9a37900460484db98970f21cef7
Requested by: Daniel Price ([email protected])

---------

Co-authored-by: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
Co-authored-by: Daniel Price <[email protected]>
Co-authored-by: Daniel <[email protected]>
@Shridhad Shridhad deleted the project-limit-hint branch September 1, 2025 10:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants