Fix UserCommands services so that they return the actual status the executed command (backport #2999)

[mergify]

🦟 Bug fix

Summary

Previously, the services immediately returned true after adding
the command to the pending commands queue which meant that
the return statuses did not reflect the success or failure of
the subsequent execution of the commands.

• Builds on top of Refactor Transport service handlers UserCommands #2996
• Needs Fix deadlock in service requests when running local callbacks gz-transport#671

Needed for gazebosim/ros_gz#790

Checklist

• Signed all commits for DCO
• Added tests
• Updated documentation (as needed)
• Updated migration guide (as needed)
• Consider updating Python bindings (if the library has them)
• codecheck passed (See contributing)
• All tests passed (See test coverage)
• While waiting for a review on your PR, please help review another open pull request to support the maintainers
• Was GenAI used to generate this PR? If so, make sure to add "Generated-by" to your commits. (See this policy for more info.)

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining Signed-off-by and Generated-by messages.

---

This is an automatic backport of pull request #2999 done by [Mergify](https://mergify.com).

[azeey]

@osrf-jenkins run tests please

[azeey]

The windows failure doesn't seem to be related to this since it's also failing on the stable branch: https://build.osrfoundation.org/view/gz-harmonic/job/gz_sim-8-clowin/test_results_analyzer/

There were some failures on jammy:

• https://build.osrfoundation.org/job/gz_sim-ci-pr_any-jammy-amd64/904/testReport/
• https://build.osrfoundation.org/job/gz_sim-ci-pr_any-jammy-amd64/903/testReport/
  But the latest build passed, so I think it's a flakey test.