fix: Fix dependsOn requiring a full id (type.name). Instead modify it to only require type and match multiple resources. A full qualified name (type.name) will match only one resource.

Author: kevinwang5658

src/entities/project.ts

@@ -248,7 +248,7 @@ ${JSON.stringify(projectConfigs, null, 2)}`);
 
     for (const r of this.resourceConfigs) {
       // User specified dependencies are hard dependencies. They must be present.
-      r.addDependenciesFromDependsOn((id) => resourceMap.has(id));
+      r.addDependenciesFromDependsOn((idOrType) => this.getMatchingResourceIds(resourceMap, idOrType));
       r.addDependenciesBasedOnParameters((id) => resourceMap.has(id));
 
       // Plugin dependencies are soft dependencies. They only activate if the dependent resource is present.
@@ -299,4 +299,28 @@ ${JSON.stringify(projectConfigs, null, 2)}`);
       }
     }
   }
+
+  /**
+   * This function supports both full (type.name) and partial IDs (type) when matching. It's meant
+   * for the dependsOn field to simplify dependency resolution for. users.
+   * @param resourceMap
+   * @param idOrType
+   * @private
+   */
+  private getMatchingResourceIds(
+    resourceMap: Map<string, ResourceConfig>,
+    idOrType: string
+  ): string[] {
+    const hasName = idOrType.includes('.');
+
+    if (hasName) {
+      // Full ID (type.name): return exact match or empty array
+      return resourceMap.has(idOrType) ? [idOrType] : [];
+    } else {
+      // Partial ID (type only): return all resources with this type
+      return [...resourceMap.values()]
+        .filter((resource) => resource.type === idOrType)
+        .map((resource) => resource.id);
+    }
+  }
 }

src/entities/resource-config.ts

@@ -100,13 +100,15 @@ export class ResourceConfig implements ConfigBlock {
     this.raw[name] = value;
   }
 
-  addDependenciesFromDependsOn(resourceExists: (id: string) => boolean) {
-    for (const id of this.dependsOn) {
-      if (!resourceExists(id)) {
-        throw new Error(`Reference ${id} is not a valid resource`);
+  addDependenciesFromDependsOn(getMatchingResourceIds: (idOrType: string) => string[]) {
+    for (const idOrType of this.dependsOn) {
+      const matchingIds = getMatchingResourceIds(idOrType);
+
+      if (matchingIds.length === 0) {
+        throw new Error(`Reference ${idOrType} is not a valid resource`);
       }
 
-      this.dependencyIds.push(id);
+      this.dependencyIds.push(...matchingIds);
     }
   }