Hi everyone! 👋

I was exploring the codebase and came across what looks like a small bug in:

.agents/skills/bmad-party-mode/scripts/resolve_party.py

While building the collective, the code assumes every entry in the agents config is a dictionary. If a non-dict entry (for example, a string under the description key) is encountered, build_collective crashes because it later calls .get() on that value.

The change I made is simply a type guard:

for code, info in agents.items():
    if not isinstance(info, dict):
        continue

    register(code, {
        "code": code,
        "name": info.get("name", code),
        "icon": info.get("icon", ""),
        "title": info.get("title", ""),
        "description": info.get("description", ""),
        "module": info.get("module", ""),
        "team": info.get("team", ""),
        "source": "installed",
    })
    installed_codes.append(code)

So my question is about the project's preferred contribution workflow rather than the fix itself.

For a small, straightforward bug like this:

• Should I open a GitHub Issue first and wait for feedback?
• Or is it preferred to submit a Pull Request directly with the fix?
• If this behavior is actually intentional, I'd also appreciate understanding the reasoning before I submit anything.

I'd like to follow the project's preferred process rather than creating unnecessary issues or PRs.

Thanks!