[andreivince]

The help for --exclude-titles says "Can specify multiple times", but argparse kept only the last occurrence. With --exclude-titles bank --exclude-titles password, titles matching bank were still logged. --exclude-apps, described in #156 as taking multiple patterns, had the same problem: only the last app was excluded.

Both flags now use action="extend", so every occurrence counts, and several patterns after one flag still work. The config values are applied after parsing, because with extend argparse would add the command line patterns to the config list instead of replacing it (and fail on an invalid config value). Patterns given on the command line still replace the ones from the config file, as before.

Added a test for both flags in tests/test_config.py. It fails on master (only the last flag's patterns are kept) and passes with the fix. I also ran the real main() with fake windows against a testing-mode aw-server-rust: before, a "Bank ..." title and a KeePassXC window were recorded; after, the title is excluded and the app isn't logged. The command for the macOS Swift helper now gets all the patterns too.

make test passes on macOS with Python 3.9. Haven't tested Linux or Windows.

[Copilot]

Copilot was unable to review this pull request because the user who requested the review has insufficient remaining quota.

[greptile-apps]

Confidence Score: 5/5

[Medium impact] The PR appears safe to merge.

Summary

Repeated --exclude-titles and --exclude-apps flags now keep every supplied pattern.

• Repeated exclusion flags keep every title and app pattern in the filter list.

No actionable issues found. Tests were inspected but not run.

_{Reviews (1) · Last reviewed commit: "fix(config): let --exclude-titles and --..." · Reviewed by Greptile}