fix: build task once per execution, not once per file (#148)

closes #147

Author: Greg Bowler

src/Build.php

@@ -45,12 +45,12 @@ public function check(array &$errors = null):int {
 	 */
 	public function build(array &$errors = null):array {
 		$updatedTasks = [];
-		foreach($this->taskList as $pathMatch => $task) {
+		foreach($this->taskList as $task) {
 			if($task->build($errors)) {
 				$updatedTasks []= $task;
 			}
 		}
 
 		return $updatedTasks;
 	}
-}
+}

src/Task.php

@@ -57,20 +57,23 @@ public function check(array &$errors = null):void {
 	}
 
 	public function build(array &$errors = null):bool {
-		$changes = false;
+		$hashMiss = false;
 
 		foreach(Glob::glob($this->absolutePath) as $matchedPath) {
 			$hash = filemtime($matchedPath);
 			$existingHash = $this->fileHashList[$matchedPath] ?? null;
 
 			if($hash !== $existingHash) {
-				$changes = $this->execute($errors);
+				$hashMiss = true;
 			}
 
 			$this->fileHashList[$matchedPath] = $hash;
 		}
 
-		return $changes;
+		if($hashMiss) {
+			return $this->execute($errors);
+		}
+		return false;
 	}
 
 	public function requirementFromRequireBlockItem(
@@ -160,4 +163,4 @@ protected function isAbsolutePath(string $path):bool {
 			// Windows:
 			|| preg_match('~\A[A-Z]:(?![^/\\\\])~i',$path) > 0;
 	}
-}
+}