Return updated comment on PUT /cards/:n/comments/:id

[robzolkos]

Summary

The JSON response for PUT /:account_slug/cards/:card_number/comments/:comment_id was 204 No Content, which conflicted with both the API docs ("Returns the updated comment") and the Smithy contract the SDKs are generated from, which declares UpdateComment returns a Comment.

Rendering the show template on the JSON path makes the response match what the docs and spec already advertise. The Turbo Stream path is unchanged.

Discovered during a fizzy-cli QA sweep — same class of issue as #2848, #2849, #2851.

Changes

• app/controllers/cards/comments_controller.rb — format.json { head :no_content } → format.json { render :show }
• test/controllers/cards/comments_controller_test.rb — both "update as JSON" tests now assert the returned body (id, body plain_text, creator)
• No docs change: docs/api/sections/comments.md already reads "Returns the updated comment."

Mobile App check

• Neither mobile app has native client code calling PUT /cards/:n/comments/:id
• Neither mobile app has code paths that depend on this endpoint returning 204 No Content
• Neither mobile app is affected by this response change to 200 OK with the updated comment

[claude]

Claude Code Review

This pull request is from a fork — automated review is disabled. A repository maintainer can comment @claude review to run a one-time review.

[Copilot]

Pull request overview

This PR aligns PUT /:account_slug/cards/:card_number/comments/:comment_id JSON behavior with the API docs and Smithy contract by returning the updated comment payload (instead of 204 No Content) after a successful update.

Changes:

• Update comments JSON update response to render the show representation (200 OK with body).
• Expand controller tests for JSON updates to assert key fields in the returned response body.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File	Description
app/controllers/cards/comments_controller.rb	Switches JSON update response from 204 to rendering :show so clients receive the updated comment.
test/controllers/cards/comments_controller_test.rb	Updates JSON update tests to validate the returned comment JSON fields (id/body/creator).

Tip

If you aren't ready for review, convert to a draft PR.
Click "Convert to draft" or run gh pr ready --undo.
Click "Ready for review" or run gh pr ready to reengage.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.