fix: `from-folder` discarding files as "extension not included"

[niieani]

Fixes #1121

Summary

• Default immich-go upload from-folder runs were discarding files as "extension not included" because SetIncludeTypeExtensions() always seeded the include list with only sidecar extensions. That regression arrived when the Cobra PreRun guard was removed in a recent refactor.
• The same refactor stopped wiring the parsed filesystem slice and journal recorder into ImportFolderCmd, so browsing never saw any assets and logging nil-panicked.
• Fixes the bugs, add regression tests, and confirm the CLI can scan fixtures again.

Key Changes

1. Extension guard & tests
   • Skip SetIncludeTypeExtensions() work unless --include-type is set.
   • Added unit tests in internal/cliFlags/extensionList_test.go for both default and VIDEO modes.
2. Folder runner wiring
   • Persist the fshelper.ParsePath result in ImportFolderCmd.fsyss before spinning workers.
   • Ensure the command attaches a fileevent.Recorder when app.Jnl() is nil, preventing nil-pointer panics while logging.

Verification

• ran go test ./...
• ran go run . upload from-folder test_folder --no-ui ... now correctly discovers files and no longer drops it as "extension not included" and uploads the files

[jeffreyskipl]

hey so we need to wait untill merged and new version availale to see it working? (sorry I don't get Git yet)

[Gustry]

Related to #1122 as well

[harryfine]

I am not that technical a person, I've just been uploading today into a new immich installation and getting a lot of discrded. So I read your explanation above, but I'm not understanding if that actually discarded all these pictures, or it just said it did. Do I have to re-do all my uploadsd? And can I, will there be hash issues? Using immich-go version 0.28.0.

[niieani]

@simulot rebased on develop

[simulot]

Hi, have you checked if the problem isn't already solved on the develop branch?