Add regression test for campfire message ordering

Verifies that chat messages sends sort=created_at&direction=desc to the
API, respects --limit for truncation, and renders the response in
chronological order (oldest to newest).

Author: jeremy

internal/commands/chat_test.go

@@ -1231,3 +1231,69 @@ func TestChatRoomShorthandFlag(t *testing.T) {
 	require.Len(t, envelope.Data, 1)
 	assert.Equal(t, "Engineering", envelope.Data[0]["title"])
 }
+
+// mockChatMessagesTransport captures the lines request and returns messages
+// in desc order (as the API would with sort=created_at&direction=desc).
+type mockChatMessagesTransport struct {
+	capturedQuery map[string]string
+}
+
+func (t *mockChatMessagesTransport) RoundTrip(req *http.Request) (*http.Response, error) {
+	header := make(http.Header)
+	header.Set("Content-Type", "application/json")
+
+	var body string
+	switch {
+	case strings.Contains(req.URL.Path, "/projects.json"):
+		body = `[{"id": 123, "name": "Test Project"}]`
+	case strings.Contains(req.URL.Path, "/projects/123"):
+		body = `{"id": 123, "dock": [{"name": "chat", "id": 789, "enabled": true}]}`
+	case strings.Contains(req.URL.Path, "/lines.json"):
+		t.capturedQuery = map[string]string{
+			"sort":      req.URL.Query().Get("sort"),
+			"direction": req.URL.Query().Get("direction"),
+		}
+		// Return 3 messages in desc order (newest first), as the API does
+		body = `[
+			{"id": 3, "content": "Third", "created_at": "2026-03-24T12:00:00Z", "status": "active", "visible_to_clients": false, "inherits_status": true, "type": "Chat::Lines::Text", "url": "https://example.com", "app_url": "https://example.com"},
+			{"id": 2, "content": "Second", "created_at": "2026-03-24T11:00:00Z", "status": "active", "visible_to_clients": false, "inherits_status": true, "type": "Chat::Lines::Text", "url": "https://example.com", "app_url": "https://example.com"},
+			{"id": 1, "content": "First", "created_at": "2026-03-24T10:00:00Z", "status": "active", "visible_to_clients": false, "inherits_status": true, "type": "Chat::Lines::Text", "url": "https://example.com", "app_url": "https://example.com"}
+		]`
+	default:
+		body = `{}`
+	}
+
+	return &http.Response{
+		StatusCode: 200,
+		Body:       io.NopCloser(strings.NewReader(body)),
+		Header:     header,
+	}, nil
+}
+
+// TestChatMessagesReturnsNewestInChronologicalOrder verifies that
+// `chat messages` requests newest-first from the API (sort/direction params),
+// respects --limit to truncate the result, and renders in chronological order.
+func TestChatMessagesReturnsNewestInChronologicalOrder(t *testing.T) {
+	transport := &mockChatMessagesTransport{}
+	app, buf := newTestAppWithTransport(t, transport)
+
+	cmd := NewChatCmd()
+	// --limit 2: the mock returns 3 messages; only the 2 newest should appear
+	err := executeChatCommand(cmd, app, "messages", "--room", "789", "--in", "123", "--limit", "2")
+	require.NoError(t, err)
+
+	// Verify sort params were sent to the API
+	assert.Equal(t, "created_at", transport.capturedQuery["sort"])
+	assert.Equal(t, "desc", transport.capturedQuery["direction"])
+
+	// Verify limit is respected (2 of 3 returned) and output is chronological
+	var envelope struct {
+		Data []struct {
+			ID int64 `json:"id"`
+		} `json:"data"`
+	}
+	require.NoError(t, json.Unmarshal(buf.Bytes(), &envelope))
+	require.Len(t, envelope.Data, 2, "should return only 2 messages per --limit")
+	assert.Equal(t, int64(2), envelope.Data[0].ID, "first displayed should be older of the two")
+	assert.Equal(t, int64(3), envelope.Data[1].ID, "last displayed should be newest")
+}