tools: wire-app-sources finds --dir among the top-level groups

[teamleaderleo]

Summary

Follow-up to #15057 from its review. wire-app-sources.py looked up the --dir group by the first group whose path matched, so a nested group with the same path (say a Settings group inside another folder) could be picked and every file resolved against the wrong directory. It now takes the group from the project's main group's children, and rejects a nested (Sources/Foo) or missing --dir with a clear error. No directory the guard uses today was affected.

Testing

tests/test_wire_app_sources.py (13 cases) adds a fixture with a nested group also named Sources, listed before the real one; the old lookup picks the nested group and fails this case. It also covers wiring a file into another top-level directory, and the nested and missing --dir errors. --check passes for cmux/Sources, cmuxUITests/cmuxUITests and cmuxTests/cmuxCLITestSupport.

Changelog

none

🤖 Generated with Claude Code

---

^{Need help on this PR? Tag @codesmith-bot with what you need. Autofix is disabled.}

---

Summary by cubic

Fixes wire-app-sources.py resolving --dir against a nested group with the same path, so files get wired into the correct top-level directory.

• Looks up --dir among the project's main group's children.
• Rejects nested or missing --dir values with a clear error.
• Adds tests covering a nested namesake group, another top-level directory, and the new errors.

^{Written for commit b4647be. Summary will update on new commits.}

Summary by CodeRabbit

• Bug Fixes
  • Source files are now wired to the matching top-level project group, avoiding confusion with nested groups that share a name.
  • Requests to wire files using a nested or nonexistent directory now return a clear error instead of matching an unrelated group.

[github-actions]

All contributors have signed the CLA ✍️ ✅
_{Posted by the CLA Assistant Lite bot.}

[coderabbitai]

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

Note

Currently processing new changes in this PR. This may take a few minutes, please wait...

⚙️ Run configuration

Configuration used: Repository: manaflow-ai/cmux/.coderabbit.yaml

Review profile: ASSERTIVE

Plan: Advanced

Run ID: 979ec00d-3de9-4023-9e5c-2e8cc3c4b5be

📥 Commits

Reviewing files that changed from the base of the PR and between 2207bc8 and b4647be.

📒 Files selected for processing (2)

• scripts/wire-app-sources.py
• tests/test_wire_app_sources.py

 _________________________________________________________
< Fun fact: 'idempotent' doesn't mean 'it usually works'. >
 ---------------------------------------------------------
  \
   \   \
        \ /\
        ( )
      .( o ).

✨ Finishing Touches

📝 Generate docstrings

• Commit to this branch
• Create a new PR

🧪 Generate unit tests (beta)

• Commit to this branch
• Create a new PR

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[github-actions]

Merge receipt for b4647be4b1: every check was green at merge (15 verified; 16 skipped by policy). Full suite runs on main after merge.