From e363350d93752cd99101be0e4e18afe943172353 Mon Sep 17 00:00:00 2001 From: Amaad Martin Date: Mon, 3 Aug 2026 15:33:51 -0700 Subject: [PATCH] chore(scripts): drop redundant "browser" prune from check_license.sh find evaluates -prune against each directory's basename as it descends, so dev/dist is pruned before dev/dist/browser is ever reached. The dist prune alone already excludes the vendored ADK Web assets that the browser clause was meant to skip, making the clause dead for every directory the repository contains or its build produces. Verified on a fully built tree (node_modules installed, dev/dist/browser populated with 101 unlicensed vendor .js files): the four-prune and three-prune file lists are byte-identical at 545 paths, and the script's stdout and exit status are unchanged. Removing the clause also closes a gap: a first-party source directory named browser outside a pruned subtree would previously have been exempted from the license check, which is the opposite of the check's purpose. --- scripts/check_license.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/scripts/check_license.sh b/scripts/check_license.sh index 2fd67f76f..f34e5d185 100644 --- a/scripts/check_license.sh +++ b/scripts/check_license.sh @@ -4,7 +4,7 @@ echo "🔍 Checking for license headers..." MISSING_FILES="" # Find files and check them using Perl (more robust for multi-line regex) -FILES=$(find . -type d \( -name "node_modules" -o -name "dist" -o -name ".git" -o -name "browser" \) -prune -o \ +FILES=$(find . -type d \( -name "node_modules" -o -name "dist" -o -name ".git" \) -prune -o \ -type f \( -name "*.js" -o -name "*.ts" \) -print) for FILE in $FILES; do