fix: homebrew filter improvements, filter out other text

Author: kevinwang5658

src/resources/homebrew/casks-parameter.ts

@@ -33,7 +33,11 @@ export class CasksParameter extends StatefulParameter<HomebrewConfig, string[]>
     if (caskQuery.status === SpawnStatus.SUCCESS && caskQuery.data !== null && caskQuery.data !== undefined) {
       const installedCasks = caskQuery.data
         .split('\n')
+        .map((line) => line.trim())
         .filter(Boolean)
+        // Some taps emit Ruby deprecation warnings to stderr, which the PTY interleaves
+        // into this output. Real cask names never contain whitespace.
+        .filter((line) => !line.includes(' '))
 
       const notInstalledCasks = desired?.filter((c) => !installedCasks.includes(c));
       if (!notInstalledCasks || notInstalledCasks.length === 0) {

src/resources/homebrew/formulae-parameter.ts

@@ -31,7 +31,12 @@ export class FormulaeParameter extends StatefulParameter<HomebrewConfig, string[
     if (formulaeQuery.status === SpawnStatus.SUCCESS && formulaeQuery.data !== null && formulaeQuery.data !== undefined) {
       return formulaeQuery.data
         .split('\n')
-        .filter(Boolean);
+        .map((line) => line.trim())
+        .filter(Boolean)
+        // Some taps emit Ruby deprecation warnings (e.g. `depends_on :macos`) to stderr,
+        // which the PTY interleaves into this output. Real formula names never contain
+        // whitespace, so any line with a space is noise, not a formula.
+        .filter((line) => !line.includes(' '));
     }
 
     return null;

src/resources/homebrew/tap-parameter.ts

@@ -17,8 +17,12 @@ export class TapsParameter extends StatefulParameter<HomebrewConfig, string[]> {
     if (tapsQuery.status === SpawnStatus.SUCCESS && tapsQuery.data !== null && tapsQuery.data !== undefined) {
       return tapsQuery.data
         .split('\n')
+        .map((line) => line.trim())
         .filter((t) => t !== 'homebrew/bundle' && t !== 'homebrew/services')
         .filter(Boolean)
+        // Some taps emit Ruby deprecation warnings to stderr, which the PTY interleaves
+        // into this output. Real tap names are always `owner/repo`, with no whitespace.
+        .filter((t) => !t.includes(' '))
     }
 
     return null;