ReviewOS

also looking at this

stacks/buddy

fix: read composer.json from the project path, and drop groups the strategy filter emptied

#1444
Merged glennmichael123 wants to merge fix/read-where-you-checked into main
3 files +13 -12
src/buddy.tsmodified+9-8
Changes to src/buddy.ts
@@ -2187,15 +2187,16 @@ export class Buddy {
21872187 }
21882188 }
21892189
2190 if (groupUpdates.length > 0) {
2191 // Apply group-specific strategy if defined
2192 let filteredUpdates = groupUpdates
2193 if (groupConfig.strategy) {
2194 filteredUpdates = this.filterUpdatesByStrategy(groupUpdates, groupConfig.strategy)
2195 }
2190 // Apply group-specific strategy if defined
2191 let filteredUpdates = groupUpdates
2192 if (groupConfig.strategy) {
2193 filteredUpdates = this.filterUpdatesByStrategy(groupUpdates, groupConfig.strategy)
2194 }
21962195
2197 // Note: Minimum release age filtering is already applied globally before grouping,
2198 // so we don't need to apply it again here
2196 // Note: minimum release age filtering is already applied globally before
2197 // grouping. Push only when something survives the group's strategy filter —
2198 // an empty group would otherwise reach the PR loop as a no-op entry.
2199 if (filteredUpdates.length > 0) {
21992200
22002201 groups.push({
22012202 name: groupConfig.name,