goose icon indicating copy to clipboard operation
goose copied to clipboard

feat: check file type with validation

Open nahiyankhan opened this issue 11 months ago • 6 comments

This feature adds a check to make sure the selected file is of type text and not binary. If the user selects a filetype thats not allowed, a system modal shows an error.

Screenshot 2025-02-26 at 8 53 20 PM

nahiyankhan avatar Feb 27 '25 01:02 nahiyankhan

@alexhancock @lily-de - I wonder if this is restrictive in the sense that the wrong file sent to goose does give back some feedback on why it might not work. I noticed that for .docx or so - it suggests using a text extractor. Thoughts on whether goose should make a reasonable attempt or it's up to the user?

nahiyankhan avatar Feb 27 '25 10:02 nahiyankhan

@nahiyankhan i think if goose can reliably do it for docx it can try! sounds cool. does it start like programming something lol? i think that will freak non-devs out a bit 👻

lily-de avatar Feb 27 '25 16:02 lily-de

@nahiyankhan i think if goose can reliably do it for docx it can try! sounds cool. does it start like programming something lol? i think that will freak non-devs out a bit 👻

It doesn't attempt by default - but does suggest options and whether I want to try them! I'll file it under future consideration for now

nahiyankhan avatar Feb 27 '25 17:02 nahiyankhan

@nahiyankhan it can now do PDF and soon some others

michaelneale avatar Mar 07 '25 03:03 michaelneale

Also, just added in image processing as part of the developer extension so image files should generally work now too.

zakiali avatar Mar 07 '25 06:03 zakiali

@michaelneale @zakiali - awesome 🤙🏽 let me adjust this and / or gauge if even needed

nahiyankhan avatar Mar 07 '25 14:03 nahiyankhan