Merge pull request #71 from nextcloud/fix/tmp-file-doc-conversion

fix: use ITempManager instead of manually handling temp files

lib/Service/AssistantService.php

@@ -19,6 +19,7 @@
 use OCP\Files\IRootFolder;
 use OCP\Files\NotPermittedException;
 use OCP\IL10N;
+use OCP\ITempManager;
 use OCP\Lock\LockedException;
 use OCP\PreConditionNotMetException;
 use OCP\SpeechToText\ISpeechToTextManager;
@@ -52,6 +53,7 @@ public function __construct(
 		private IJobList $jobList,
 		private IL10N $l10n,
 		private ContainerInterface $container,
+		private ITempManager $tempManager,
 	) {
 	}
 
@@ -437,19 +439,10 @@ public function parseTextFromFile(string $filePath, string $userId): string {
 			case 'application/msword':
 			case 'application/vnd.oasis.opendocument.text':
 				{
-					// Store the file in a temp dir and provide a path for the doc parser to use
-					$tempFilePath = sys_get_temp_dir() . '/assistant_app/' . uniqid() . '.tmp';
-					// Make sure the temp dir exists
-					if (!file_exists(dirname($tempFilePath))) {
-						mkdir(dirname($tempFilePath), 0700, true);
-					}
+					$tempFilePath = $this->tempManager->getTemporaryFile();
 					file_put_contents($tempFilePath, $fileContent);
-
 					$text = $this->parseDocument($tempFilePath, $mimeType);
-
-					// Remove the hardlink to the file (delete it):
-					unlink($tempFilePath);
-
+					$this->tempManager->clean();
 					break;
 				}
 		}