wip

Author: Julusian

packages/job-worker/src/blueprints/context/watchedPackages.ts

@@ -146,7 +146,8 @@ export class WatchedPackagesHelper {
 
 	/**
 	 * Create a new helper with a subset of the data in the current helper.
-	 * This is useful so that all the data for a rundown can be loaded at the start of an ingest operation, and then subsets can be taken for particular blueprint methods without needing to do more db operations.
+	 * This is useful so that all the data for a rundown can be loaded at the start of an ingest operation,
+	 * and then subsets can be taken for particular blueprint methods without needing to do more db operations.
 	 * @param func A filter to check if each package should be included
 	 */
 	filter(
@@ -171,9 +172,11 @@ export class WatchedPackagesHelper {
 	}
 
 	getPackageInfo(blueprintPackageId: string): Readonly<Array<PackageInfo.Any>> {
-		// nocommit - can this/should this do more scoping by source?
+		// Perhaps this should do some scoped source checks, but this should not be necessary.
+		// The caller should be ensuring that this helper has been filtered to only contain relevant packages
 		for (const packages of this.packages.values()) {
 			for (const pkg of packages) {
+				// Note: This finds the first package with the same blueprintPackageId. There could be multiple if the blueprints don't respect the uniqueness rules.
 				if (pkg.source.blueprintPackageId === blueprintPackageId) {
 					const info = this.packageInfos.filter((p) => p.packageId === pkg.packageId)
 					return unprotectObjectArray(info)