From 455ceb7d69c70cbc3639e3e4e4f3ecf40eae6ff1 Mon Sep 17 00:00:00 2001 From: AJ ONeal Date: Sat, 28 Feb 2026 23:54:54 -0700 Subject: [PATCH] fix(monorel): restore cross-module binary discovery in recursive walk The go.mod boundary check added to findMainPackages was stopping the --recursive walk from descending into child module directories, so modules like auth/csvauth/ were never found. The check was redundant: groupByModule already uses findModuleRoot to attribute each binary to its nearest go.mod ancestor, and initModuleGroup already skips modules with uncommitted changes (an untracked go.mod counts as an uncommitted change). Removing the stop restores the pre-regression behaviour. --- tools/monorel/main.go | 5 ----- 1 file changed, 5 deletions(-) diff --git a/tools/monorel/main.go b/tools/monorel/main.go index 69771b3..8c5a6b7 100644 --- a/tools/monorel/main.go +++ b/tools/monorel/main.go @@ -572,11 +572,6 @@ func findMainPackages(root string, all bool) ([]string, error) { continue } child := filepath.Join(dir, name) - // Stop at directories with their own go.mod — they are independent - // module roots and should not be included in this module's walk. - if _, err := os.Stat(filepath.Join(child, "go.mod")); err == nil { - continue - } // Skip directories that contain no git-tracked files. if trackedDirs != nil && !trackedDirs[child] { continue