Skip to content

Commit fe5f1ea

Browse files
committed
refactor feedback api
1 parent 658b522 commit fe5f1ea

File tree

4 files changed

+4
-4
lines changed

4 files changed

+4
-4
lines changed

packages/backend/src/routes/api/chat-feedback.ts renamed to packages/backend/src/routes/api/chat/feedback.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ import appConfig from '@/config/app'
55
import { langfuseClient } from '@/helpers/langfuse'
66
import logger from '@/helpers/logger'
77

8-
import { getAuthenticatedContext } from './middleware/authentication'
8+
import { getAuthenticatedContext } from '../middleware/authentication'
99

1010
interface ChatFeedbackRequest {
1111
traceId: string

packages/backend/src/routes/api/chat/index.ts

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -11,6 +11,7 @@ import { getPrompt } from '@/helpers/pair/get-prompt'
1111

1212
import { getAuthenticatedContext } from '../middleware/authentication'
1313

14+
import chatFeedbackRouter from './feedback'
1415
import chatReadinessRouter from './readiness'
1516

1617
interface ChatRequest {
@@ -174,6 +175,7 @@ async function handleChatStream(req: Request, res: Response) {
174175
const router = Router()
175176

176177
router.post('/', handleChatStream)
178+
router.use('/feedback', chatFeedbackRouter)
177179
router.use('/readiness', chatReadinessRouter)
178180

179181
export default router

packages/backend/src/routes/api/index.ts

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,6 @@ import {
55
setCurrentUserContext,
66
} from './middleware/authentication'
77
import chatRouter from './chat'
8-
import chatFeedbackRouter from './chat-feedback'
98

109
const router = Router()
1110

@@ -16,7 +15,6 @@ router.use(requireAuthentication)
1615

1716
// Mount individual API routes
1817
router.use('/chat', chatRouter)
19-
router.use('/chat-feedback', chatFeedbackRouter)
2018

2119
// Future routes can be added here:
2220
// router.use('/users', usersRouter)

packages/frontend/src/pages/AiBuilder/components/ChatMessages/ChatMessageToolbar.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,7 @@ export default function ChatMessageToolbar({
4141
// NOTE: we send feedback to the backend instead of using Langfuse directly
4242
// as there are additional headers required to call Rome/Istanbul endpoints
4343
// that should not be exposed to the frontend
44-
await fetch('/api/chat-feedback', {
44+
await fetch('/api/chat/feedback', {
4545
method: 'POST',
4646
headers: { 'Content-Type': 'application/json' },
4747
body: JSON.stringify({ traceId, feedback: comment }),

0 commit comments

Comments
 (0)