Skip to content

feat: add tools for common operations#3

Open
ojasinha wants to merge 5 commits intoSnehil-Shah:mainfrom
ojasinha:main
Open

feat: add tools for common operations#3
ojasinha wants to merge 5 commits intoSnehil-Shah:mainfrom
ojasinha:main

Conversation

@ojasinha
Copy link

@ojasinha ojasinha commented Aug 6, 2025

What's this PR doing?

  • Adds new tools for various common operations in Libresprite:
    • create_sprite
    • create_layer
    • fill_layer
    • draw_circle
    • draw_rectangle
    • draw_line
    • draw_ellipse
    • put_pixel
    • flood_fill
    • get_sprite_info
    • delete_layer
    • replace_color
  • Expands documentation and examples to demonstrate usage of these new tools and combined workflows. The prompt now advises using run_script only for advanced custom cases.

@ojasinha
Copy link
Author

ojasinha commented Aug 11, 2025

Just a heads up though, a lot of the times the non-painting tools — create_sprite, create_layer — don't actually work as intended even though they are being called with the correct arguments. For example, if I ask the model to create a 64x64 pixel sprite it does correctly call the create_sprite tool with the correct arguments for width and height. But once the sprite is actually created the dimensions are always 32x32 pixel. Although sometimes it does work for some reason. 🤷‍♂️

@sksarvesh007
Copy link

LGTM!

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.

2 participants