bugfix/3447: Address quality-debt in tests/test-dual-cli-e2e.sh

[marcusquinn]

Summary

• Addresses all 8 medium priority quality-debt findings from PR t1164: End-to-end verification of dual-CLI architecture #2105 review feedback
• Removes blanket 2>/dev/null suppression from worktree remove so errors remain visible for debugging
• Replaces 2>/dev/null with -q flag on worktree prune to suppress stdout while keeping stderr visible
• Adds explicit return 0 to cleanup function
• Adds explicit return $? to test_db, get_status, get_field, and run_in_supervisor_env functions to propagate exit codes
• Separates local declarations from assignments in create_log for exit code safety, adds return 0
• Adds named local variables (task_id, field) to get_status and get_field instead of direct $1/$2 use
• Removes 2>/dev/null from run_in_supervisor_env so sourcing errors and syntax issues remain visible

ShellCheck: zero violations (was zero before, remains zero after).

Closes #3447

[gemini-code-assist]

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!

[coderabbitai]

Warning

Rate limit exceeded

@marcusquinn has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 3 minutes and 52 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

ℹ️ Review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 67c9505b-1887-4374-ab89-1ad94e9aa41d

📥 Commits

Reviewing files that changed from the base of the PR and between eccad3c and ef01f44.

📒 Files selected for processing (1)

• tests/test-dual-cli-e2e.sh

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch bugfix/issue-3447-quality-debt

📝 Coding Plan

• Generate coding plan for human review comments

---

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 and usage tips.}

[github-actions]

🔍 Code Quality Report

�[0;35m[MONITOR]�[0m Code Review Monitoring Report

�[0;34m[INFO]�[0m Latest Quality Status:
SonarCloud: 0 bugs, 0 vulnerabilities, 414 code smells

�[0;34m[INFO]�[0m Recent monitoring activity:
Sat Mar 14 01:23:59 UTC 2026: Code review monitoring started
Sat Mar 14 01:23:59 UTC 2026: SonarCloud - Bugs: 0, Vulnerabilities: 0, Code Smells: 414

📈 Current Quality Metrics

• BUGS: 0
• CODE SMELLS: 414
• VULNERABILITIES: 0

Generated on: Sat Mar 14 01:24:02 UTC 2026

---

Generated by AI DevOps Framework Code Review Monitoring

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud