allow .github and .gitlab directories in WalkFiles traversal

[maxcleme]

When working on CI it is super annoying to be not able to use @ with anything inside .github/ folder.

[docker-agent]

Review Summary

Found one issue in the changed code: the modification to the hidden file/directory filtering logic now causes hidden files (not just directories) to be included in WalkFiles results when they should be skipped.

The original code would skip all hidden items (both files and directories) by returning from the condition early. The new code only applies the skip logic to directories, allowing hidden files like .gitignore, .env, .editorconfig, etc. to pass through and be collected at line 207.

[krissetto]

maybe we should guard against .env as a precaution

[maxcleme]

This .env isn't a dir, it will return nil, right? (and so not be processed?)

[aheritier]

I don't think in claude code & friends they block it?
What they advice is to configure the permission to not read some files ( ex: https://code.claude.com/docs/en/settings#excluding-sensitive-files ) and they have a setting 'respectGitignore' to block by default the file-picker (personally I disable it)

[krissetto]

@maxcleme my bad, i only gave a very quick look at this earlier and made a bad assumption

[krissetto]

@aheritier yeah true, and we did have similar things in place in cagent as well :)

[maxcleme]

@krissetto Fwiw I do not have write permission here 😁