Skip to content

Ai 1343 Add Data App tools #236

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 47 commits into from
Aug 22, 2025
Merged

Ai 1343 Add Data App tools #236

merged 47 commits into from
Aug 22, 2025

Conversation

mariankrotil
Copy link
Contributor

Adding Data Apps tools

  • create, update
  • deploy, suspend
  • get details

@mariankrotil
Merge AI-1343-cleint methods to AI-1343-tools
ce67f45
@mariankrotil
Merge branch 'AI-1343-mcp-add-support-for-data-apps' into AI-1343-mcp…
152b68b 
…-add-support-for-data-apps-tool
@mariankrotil
AI-1343 feat: improve secrets encryption in the tool and in the data …
27bf4d8 
…app, add support for data app logs when fetching details
@mariankrotil
AI-1343 test: align test wrt changes
a0b4e73
@mariankrotil
AI-1343 docs: update tools md
c57dfb0
@mariankrotil
Merge branch 'AI-1343-mcp-add-support-for-data-apps' into AI-1343-mcp…
f0d892e 
…-add-support-for-data-apps-tool
@mariankrotil
AI-1343 refactor: update code wrt changes
780082d
@mariankrotil
AI-1343 chore: update vers and add cryptography dep
63c8da1
@mariankrotil
Merge branch 'AI-1343-mcp-add-support-for-data-apps' into AI-1343-mcp…
b369b48 
…-add-support-for-data-apps-tool
@mariankrotil
AI-1343 fix: update pyproject
6845ef9
@mariankrotil
AI-1343 refactor: unify get_data_apps output
a671bc9
@mariankrotil
AI-1343 fix: remove format from source code as causing errors when us…
87ee479 
…ing with bots, improve descriptions
@mariankrotil
AI-1343 feat: add metadata created by mcp to the data apps
ca66b06
@mariankrotil
AI-1343 style: apply flake8 changes
862a631
@mariankrotil
AI-1343 docs: update tools.md documentation wrt recent changes
f17c63a
@mariankrotil
AI-1343 perf(docs): improve docs of create data app wrt agent testing…
4fe4822 
… results
@mariankrotil
AI-1343 docs: update tool.md file wrt new changes
67f83f5
@mariankrotil
AI-1343 style: apply flake8 changes
b41cf5d
radcliffkey
radcliffkey reviewed Aug 19, 2025
View reviewed changes
vita-stejskal
vita-stejskal reviewed Aug 20, 2025
View reviewed changes
vita-stejskal
vita-stejskal reviewed Aug 20, 2025
View reviewed changes
ncapek
ncapek reviewed Aug 20, 2025
View reviewed changes
@radcliffkey
Copy link
Contributor

When adding/removing tools, we should also update the list in README.md (AI can do it very well using TOOLS.md). Or we can agree to drop it and reference TOOLS.md

@mariankrotil
Copy link
Contributor Author

Showing examples how it works in cursor:
CREATION OF EXAMPLE Data App

  • Conversation:
Screenshot 2025-08-20 131856 - **App Result:** Screenshot 2025-08-20 133945

CREATION OF WRONG Data App - code contains wrong sql query does not follow the dialect

  • Conversation
Screenshot 2025-08-20 133552 - **APP RESULT:** Screenshot 2025-08-20 134156

radcliffkey
radcliffkey reviewed Aug 20, 2025
View reviewed changes
vita-stejskal
vita-stejskal reviewed Aug 20, 2025
View reviewed changes
radcliffkey
radcliffkey reviewed Aug 20, 2025
View reviewed changes
radcliffkey
radcliffkey reviewed Aug 20, 2025
View reviewed changes
radcliffkey
radcliffkey reviewed Aug 20, 2025
View reviewed changes
radcliffkey
radcliffkey reviewed Aug 20, 2025
View reviewed changes
vita-stejskal
vita-stejskal reviewed Aug 20, 2025
View reviewed changes
vita-stejskal
vita-stejskal reviewed Aug 20, 2025
View reviewed changes
vita-stejskal
vita-stejskal reviewed Aug 20, 2025
View reviewed changes
vita-stejskal
vita-stejskal approved these changes Aug 20, 2025
View reviewed changes
Copy link
Contributor

@vita-stejskal vita-stejskal left a comment

Choose a reason for hiding this comment

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

I think that this code deserves a bit more polishing and streamlining, but it is generally acceptable in its current form.

@vita-stejskal vita-stejskal merged commit 9d3b87e into main Aug 22, 2025
19 checks passed
@vita-stejskal vita-stejskal deleted the AI-1343-mcp-add-support-for-data-apps-tool branch August 22, 2025 11:41
@mariankrotil mariankrotil mentioned this pull request Aug 29, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@radcliffkey radcliffkey radcliffkey left review comments

@ncapek ncapek ncapek left review comments

@vita-stejskal vita-stejskal vita-stejskal approved these changes

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

Successfully merging this pull request may close these issues.
4 participants
@mariankrotil @radcliffkey @ncapek @vita-stejskal