fix: from-folder discarding files as "extension not included"
#1130
+76
−0
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Develop branch broke the default from-folder scan by always pre-populating InclusionFlags.IncludedExtensions with sidecar-only values. That meant Include() stopped returning true for MP4/ARW files whenever --include-type was left unset, so our fixture uploads discarded every clip. Only expand the include list when the user explicitly asks for IMAGE/VIDEO filtering, which matches the older Cobra PreRun hook. Added regression tests to prove the default path remains empty while VIDEO mode still seeds video+sidecar extensions.
Refactors in develop stopped assigning the parsed filesystem list to ImportFolderCmd and never initialised the file-event journal. Browse() therefore iterated a nil slice (so zero assets were discovered) and any logging attempt crashed with a nil recorder. Copy the fshelper.ParsePath result into ifc.fsyss before kicking off workers and ensure the command boots a Recorder from the application logger when none exists. Fixture upload now finds the clip again and logs without panicking.
when no groupers are registered, simply forward every asset from the adapter as a single-item group instead of closing the channel
|
hey so we need to wait untill merged and new version availale to see it working? (sorry I don't get Git yet) |
niieani
changed the title
from-folder defaults to blank in immich-go developfrom-folder discarding files as "extension not included"
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fixes #1121
Summary
immich-go upload from-folderruns were discarding files as "extension not included" becauseSetIncludeTypeExtensions()always seeded the include list with only sidecar extensions. That regression arrived when the CobraPreRunguard was removed in a recent refactor.ImportFolderCmd, so browsing never saw any assets and logging nil-panicked.Key Changes
SetIncludeTypeExtensions()work unless--include-typeis set.internal/cliFlags/extensionList_test.gofor both default andVIDEOmodes.fshelper.ParsePathresult inImportFolderCmd.fsyssbefore spinning workers.fileevent.Recorderwhenapp.Jnl()is nil, preventing nil-pointer panics while logging.it simply forwards every asset from the adapter as a single-item group instead of closing the
channel.
Verification
go test ./...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