fix: handle duplicate decoder/test suite names in command arguments

[Copilot]

Fixes issue where duplicate decoder or test suite names in command-line arguments would cause the command to fail with "No decoders found for:" error.

Problem

When running:

./fluster.py run -s -d GStreamer-H.264-Libav-Gst1.0 GStreamer-H.264-Libav-Gst1.0 GStreamer-H.264-VAAPI-Gst1.0

The command failed with "No decoders found for:" because _get_matches() compared the length of the original input list (with duplicates) against the deduplicated set.

Solution

Changed line 229 in fluster/fluster.py:

• From: if len(matches) != len(in_list):
• To: if len(matches) != len(in_list_names):

This compares against the deduplicated set, allowing duplicates to pass through while preserving validation for invalid names.

Testing

• ✅ Duplicate decoders handled correctly
• ✅ Case-insensitive duplicates handled
• ✅ Invalid decoder names still rejected
• ✅ All linting and security checks passed

Original prompt

When running this commmand:

./fluster.py run -s -d GStreamer-H.264-Libav-Gst1.0 GStreamer-H.264-Libav-Gst1.0 GStreamer-H.264-VAAPI-Gst1.0 GStreamer-H.264-Vulkan-Gst1.0

I got this error:

No decoders found for:

The issues is the decoder GStreamer-H.264-Libav-Gst1.0 is listed twice as argument in the command. Expected result is not fail and execute tests as if the command was:

./fluster.py run -s -d GStreamer-H.264-Libav-Gst1.0 GStreamer-H.264-VAAPI-Gst1.0 GStreamer-H.264-Vulkan-Gst1.0

---

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

[rgonzalezfluendo]

@copilot src change is correct, but commit message is wrong. This project uses Conventional Commits [1] and the summary line in the commit must have a prefix with the text fix, given that the commit fix an issue.

[1] https://www.conventionalcommits.org

[Copilot]

@copilot src change is correct, but commit message is wrong. This project uses Conventional Commits [1] and the summary line in the commit must have a prefix with the text fix, given that the commit...

The main fix commit (76f7bd4) already has the fix: prefix: "fix: handle duplicate decoder/test suite names in arguments"

This follows Conventional Commits format correctly. The "Initial plan" commit is an empty planning commit that was created during development. If you'd like me to squash these commits into one, I'll need force-push permissions as git doesn't allow me to rewrite history with the current setup.