fix(files): report new files as new in the files:scan summary

The addToCache listener decides between NodeAddedToCache and
FileCacheUpdated by checking `if ($fileId)`, but Cache\Scanner emits
$fileId = -1 for newly inserted entries, which is truthy. As a result
the NodeAddedToCache branch was unreachable and `occ files:scan` has
reported every new file as "Updated" (never "New") since the summary
was introduced in 292c0e5.

Check for the actual -1 sentinel instead, and add a regression test
asserting that a first scan dispatches NodeAddedToCache and a re-scan
of a modified file dispatches FileCacheUpdated.

Assisted-by: ClaudeCode:claude-fable-5
Signed-off-by: Nick Manning <nicholas.manning@gmail.com>

Author: seenickcode

lib/private/Files/Utils/Scanner.php

@@ -208,7 +208,7 @@ public function scan(string $dir = '', $recursive = \OC\Files\Cache\Scanner::SCA
 			});
 			$scanner->listen('\OC\Files\Cache\Scanner', 'addToCache', function ($path, $storageId, $data, $fileId) use ($storage): void {
 				$this->triggerPropagator($storage, $path);
-				if ($fileId) {
+				if ($fileId !== -1) {
 					$this->eventDispatcher->dispatchTyped(new FileCacheUpdated($storage, $path));
 				} else {
 					$this->eventDispatcher->dispatchTyped(new NodeAddedToCache($storage, $path));

tests/lib/Files/Utils/ScannerTest.php

@@ -18,6 +18,8 @@
 use OCP\EventDispatcher\IEventDispatcher;
 use OCP\Files\Config\IMountProvider;
 use OCP\Files\Config\IMountProviderCollection;
+use OCP\Files\Events\FileCacheUpdated;
+use OCP\Files\Events\NodeAddedToCache;
 use OCP\Files\Storage\IStorageFactory;
 use OCP\IDBConnection;
 use OCP\IUser;
@@ -209,6 +211,53 @@ public function testPropagateEtag(): void {
 		$this->assertNotEquals($oldRoot->getEtag(), $newRoot->getEtag());
 	}
 
+	public function testScanDispatchesAddedForNewAndUpdatedForChangedEntries(): void {
+		$storage = new Temporary([]);
+		$mount = new MountPoint($storage, '');
+		Filesystem::getMountManager()->addMount($mount);
+
+		$storage->mkdir('folder');
+		$storage->file_put_contents('folder/bar.txt', 'qwerty');
+		$storage->touch('folder/bar.txt', time() - 200);
+
+		/** @var list<object> $events */
+		$events = [];
+		$dispatcher = $this->createMock(IEventDispatcher::class);
+		$dispatcher->method('dispatchTyped')
+			->willReturnCallback(function (object $event) use (&$events): void {
+				$events[] = $event;
+			});
+
+		$scanner = new TestScanner(
+			Server::get(IUserManager::class)->get(''),
+			Server::get(IDBConnection::class),
+			$dispatcher,
+			Server::get(LoggerInterface::class),
+			Server::get(SetupManager::class),
+		);
+		$scanner->addMount($mount);
+
+		$pathsForEvents = function (string $class) use (&$events): array {
+			return array_values(array_map(
+				static fn (object $event): string => $event->getPath(),
+				array_filter($events, static fn (object $event): bool => $event instanceof $class),
+			));
+		};
+
+		// first scan: everything is new, nothing is updated
+		$scanner->scan('');
+		$this->assertContains('folder/bar.txt', $pathsForEvents(NodeAddedToCache::class));
+		$this->assertContains('folder', $pathsForEvents(NodeAddedToCache::class));
+		$this->assertSame([], $pathsForEvents(FileCacheUpdated::class));
+
+		// re-scan after modifying the file: updated, not new
+		$events = [];
+		$storage->file_put_contents('folder/bar.txt', 'qwerty asdf');
+		$scanner->scan('');
+		$this->assertContains('folder/bar.txt', $pathsForEvents(FileCacheUpdated::class));
+		$this->assertSame([], $pathsForEvents(NodeAddedToCache::class));
+	}
+
 	public function testShallow(): void {
 		$storage = new Temporary([]);
 		$mount = new MountPoint($storage, '');