fix(fish): clean up the fish script options

[arcuru]

Closes #1368

I reviewed and updated the options we use in the fish script, fixing several issues:

1. Removing the --groups option as reported in atuin reports an unknown option "-qg" for string match #1368, it's not needed if we're only testing whether the match exists.
2. Setting the correct scoping for variables. Setting all local variables to be local, -l, and removing the -x (export) setting for ATUIN_HISTORY_ID as it's unnecessary. The non-local variables (s, h, hist) were clobbering any user-defined variables with the same name.
3. Renaming h and hist to have an ATUIN_ prefix, as those variables are accessible from user-defined code. Without this, using enter_accept with the history item echo $h would have given the user the script value of $h instead of their own.

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
atuin-docs	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Nov 2, 2023 2:46pm

[ellie]

Thank you! 🙏