[bug] Overwrite flag not being followed

[YellowRoseCx]

On linux mint 22.1, built from source at this commit: commit 20cd89fd13b262f4a265c2b860f83efd870c11f2 (HEAD -> main, origin/main, origin/HEAD)

I'm using this upload command:
user@pc-ws-linux:~/immich-go$ ./immich-go upload from-folder --concurrent-uploads 10 --into-album "my_album" --server=http://127.0.0.1:2283 --api-key="00api_key00" --overwrite "/path/to/image/files/test"

and then when the software runs, it says:

│2025-10-18 18:49:45 INF server has same asset file=test:2012-10-imagefile_33h383h50ddd.jpg reason=An asset with the same name:"2012-10-imagefile_33h383h50ddd.jpg", date:"2025-10-17 16:23:45" and size:59.0 KB exists on the server. No need to upload.                                                                                            

I tried moving the --overwrite flag into a few different positions but it was never respected.

I was wanting to overwrite the files because I saw Immich-go can use the date from the filename to set the date of the image whereas it doesn't get set if uploading over the immich webui

Even after running the files through "touch" to change the modification times, they were still skipped

[YellowRoseCx]

For some reason, the overwrite flag was not causing the "return ii.adviceForceUpload" to trigger, it was instead returning "adviceSameOnServer" here in app/cmd/upload/advice.go:

			case compareDate == 0 && upCmd.Overwrite:
				return ii.adviceForceUpload(sa), nil
			case compareDate == 0 && compareSize == 0:
				return ii.adviceSameOnServer(sa), nil

I ended up being able to get the functionality I needed by replacing "adviceSameOnServer" with "adviceSmallerOnServer" which successfully replaced my images with their same copies, but now with the correct date attached and no loss of facial data.

I also had to make that modification here on line 260:

	if sa, ok := ii.byChecksum.Load(checksum); ok {
		if ii.isAlreadyProcessed(checksum) {
			return ii.adviceAlreadyProcessed(sa), nil
		}
		return ii.adviceSameOnServer(sa), nil
	}

	if sa, ok := ii.byChecksum.Load(checksum); ok {
		if ii.isAlreadyProcessed(checksum) {
			return ii.adviceAlreadyProcessed(sa), nil
		}
		return ii.adviceSmallerOnServer(sa), nil
	}

I know that's not a fix though

[filipcoja]

++ needed the reprocessing for dates with the filename since i found a source where the filenames are correct - just wanted the date to change

[taufikterdidik]

me too.. I want to force upload (overwrite) because I was accidentally reformat some disk (I use mergefs), so some asset is missing, when I use immich-go --overwrite not working, the immich-go still skipped/not upload.