Fix incorrect str.join() usage in children_content_hash

[hobostay]

Summary

• Fix children_content_hash method in computer-server/computer_server/handlers/macos.py where str.join() was used instead of string concatenation
• content_hash.join(content_structure_hash) treats content_structure_hash as an iterable, joining its characters with content_hash as separator — producing nonsensical hashes
• Changed to use + operator for correct concatenation

Test plan

• Verify that children_content_hash now produces deterministic, correct hashes
• Existing accessibility tree tests should pass (hash values will change since they were previously incorrect)

🤖 Generated with Claude Code

Summary by CodeRabbit

Release Notes

• Bug Fixes
  • Improved hash computation for UI element identification to ensure more accurate and consistent tracking of user interface components.

[vercel]

Someone is attempting to deploy a commit to the Cua Team on Vercel.

A member of the Team first needs to authorize it.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: cf05e61e-2091-45b6-8661-9deac770d4e6

📥 Commits

Reviewing files that changed from the base of the PR and between e5098af and c2208af.

📒 Files selected for processing (1)

• libs/python/computer-server/computer_server/handlers/macos.py

---

📝 Walkthrough

Walkthrough

The UIElement.children_content_hash() method is corrected to concatenate hashes using the + operator instead of calling str.join(), which was misapplied to the hash strings.

Changes

Hash Computation Fix

Layer / File(s)	Summary
Core Logic libs/python/computer-server/computer_server/handlers/macos.py	UIElement.children_content_hash() now concatenates content_hash + content_structure_hash directly instead of misusing str.join() on the hashes.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

A rabbit hops through the code so deep,
Found a .join() that should not keep,
With + now swift, the hashes align,
Content and structure combined just fine! 🐇✨

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The pull request title 'Fix incorrect str.join() usage in children_content_hash' accurately describes the main change: correcting the misuse of str.join() in the children_content_hash method.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

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

_{Review rate limit: 7/8 reviews remaining, refill in 7 minutes and 30 seconds.}

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