Skip to content

fix(issue-161): refactor(test): use within() scoping instead of DOM-indexed copyButtons[1] - #167

Open
luandro wants to merge 2 commits into
mainfrom
agent/comapeo-cloud-app/issue-161
Open

fix(issue-161): refactor(test): use within() scoping instead of DOM-indexed copyButtons[1]#167
luandro wants to merge 2 commits into
mainfrom
agent/comapeo-cloud-app/issue-161

Conversation

@luandro

@luandro luandro commented Jul 29, 2026

Copy link
Copy Markdown
Contributor

Changes

Implements #161 via autonomous Pool pipeline.

Verification

  • tsc --noEmit, vitest (unit), eslint src/ gate passed
  • Review: PASSED

Closes #161

github-actions Bot added 2 commits July 29, 2026 12:46
Replace fragile DOM-indexed copyButtons[0]! selection with within()-scoped
queries that target the invite-URL and invite-code rows by their label.
Adds a new test covering the invite-code Copy button.
@luandro
luandro marked this pull request as ready for review July 29, 2026 16:00
@luandro luandro added agent:pr-readiness-in-progress PR lifecycle label agent:ready-for-human-review Agent review is complete and PR is ready for human review. and removed agent:pr-readiness-in-progress PR lifecycle label labels Jul 30, 2026
@luandro

luandro commented Jul 31, 2026

Copy link
Copy Markdown
Contributor Author

PR Readiness Orchestrator — Status Summary

Gate or reviewer Result Reviewed commit Details
Base synchronization ✅ Passed a99a2b917a Rebased onto main (a99a2b9→a99a2b917a)
Required CI ✅ Passed a99a2b917a passed=8 failing=0 pending=0 skipped=13
PR feedback ✅ Addressed a99a2b917a No actionable unresolved threads
Qwen 3.8 ✅ Mergeable a99a2b917a Prompt 3 · confidence 0.85
Final readiness gate ⏳ In progress a99a2b917a Pending

Last updated: 2026-07-31T07:22:16Z

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

agent:ready-for-human-review Agent review is complete and PR is ready for human review.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

refactor(test): use within() scoping instead of DOM-indexed copyButtons[1]

1 participant