Skip to content

Commit 0d977f1

Browse files
committed
fix(pipeline): honor language filter for non-JS language runs
When a non-JS language filter is set (e.g., python), the previous logic incorrectly allowed JS files through. Now the special case for 'js' detection only applies when the language filter IS javascript or typescript. Addresses review feedback from Codex.
1 parent 7106ea9 commit 0d977f1

1 file changed

Lines changed: 3 additions & 1 deletion

File tree

lib/patterns/pipeline.js

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -152,7 +152,9 @@ function runPhase1(repoPath, targetFiles, language) {
152152
// Skip if language filter doesn't match file extension
153153
if (language) {
154154
const fileLanguage = analyzers.detectLanguage(file);
155-
if (fileLanguage !== language && fileLanguage !== 'js') continue;
155+
// For JS/TS language filter, accept both 'javascript' and 'js' detection results
156+
const isJsFamily = (language === 'javascript' || language === 'typescript') && fileLanguage === 'js';
157+
if (fileLanguage !== language && !isJsFamily) continue;
156158
}
157159

158160
const filePath = path.isAbsolute(file) ? file : path.join(repoPath, file);

0 commit comments

Comments
 (0)