Add migration from ArangoDB #687
Conversation
|
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
There was a problem hiding this comment.
This is the final PR Bugbot will review for you during this billing cycle
Your free Bugbot reviews will reset on November 12
Details
Your team is on the Bugbot Free tier. On this plan, Bugbot will review limited PRs each billing cycle for each member of your team.
To receive Bugbot reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.
| if len(words) > 1 or any( | ||
| keyword.lower() in collection_or_query.lower() | ||
| for keyword in ["FOR", "RETURN", "FILTER", "LET", "COLLECT", "SORT", "LIMIT"] | ||
| ): |
There was a problem hiding this comment.
Bug: AQL Keyword Matching Fails on Substrings
The _formulate_aql_query function uses substring matching to detect AQL keywords, which is incorrect. It checks keyword.lower() in collection_or_query.lower(), but this will incorrectly match keywords that are part of a word (e.g., "FOR" would match "FOREVER"). This causes collection names like "FOREVER" to be treated as AQL queries, which will then fail at execution time. The keyword matching should use word boundary checking or proper tokenization instead of simple substring matching.
Description
Migration from ArangoDB
Pull request type
Related issues
Delete if this PR doesn't resolve any issues. Link the issue if it does.
######################################
Reviewer checklist (the reviewer checks this part)
Module/Algorithm
Documentation checklist