Agent frontend by CHELSEADOPAMIN · Pull Request #283 · wiperi/qrent

CHELSEADOPAMIN · 2026-01-15T06:20:48Z

实现了 agent的前端可以生成 cover letter 和 parent letter

vercel · 2026-01-15T06:20:53Z

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Review	Updated (UTC)
qrent-frontend	Ready	Preview, Comment	Jan 25, 2026 4:03am

Copilot

Pull request overview

This pull request implements an agent-powered frontend for the Qrent platform that enables users to generate rental cover letters and parent guarantee letters using AI assistance.

Changes:

Adds a comprehensive AI assistant frontend using the @assistant-ui/react library with streaming chat capabilities
Implements a backend agent service using LangGraph with RAG (Retrieval-Augmented Generation) for rental knowledge queries and automated cover letter generation
Integrates the assistant UI as a resizable sidebar panel alongside existing application content

Reviewed changes

Copilot reviewed 51 out of 68 changed files in this pull request and generated 11 comments.

Show a summary per file

File	Description
pnpm-lock.yaml	Added dependencies for assistant-ui React components, Radix UI components, and typography plugin
packages/frontend/package.json	Updated with new frontend dependencies for the AI assistant interface
packages/frontend/src/lib/trpc.ts	Improved error handling type safety in retry logic
packages/frontend/src/lib/templates.ts	Added cover letter data types and prompt generation logic
packages/frontend/src/components/ui/*	Added new UI components (tooltip, resizable, dialog, avatar) to support the assistant interface
packages/frontend/src/components/assistant-ui/*	Implemented complete assistant UI with thread management, markdown rendering, and file attachments
packages/frontend/src/components/CoverLetterForm.tsx	Created comprehensive form for collecting user information to generate cover letters
packages/frontend/src/components/AIChatBox.tsx	Enhanced existing chat component with markdown support and cover letter form integration
packages/frontend/src/app/layout.tsx	Removed duplicate AIChatBox components from root layout
packages/frontend/src/app/[locale]/layout.tsx	Wrapped application in MyAssistant provider for AI functionality
packages/frontend/src/app/api/assistant/chat/route.ts	Created API route to proxy requests to the backend agent service
packages/agent/*	Complete restructure of agent package with LangGraph-based architecture, RAG tools, and cover letter generation

Files not reviewed (1)

pnpm-lock.yaml: Language not supported

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

Copilot · 2026-01-15T06:25:03Z

packages/frontend/src/app/api/assistant/chat/route.ts

@@ -0,0 +1,100 @@
+import { NextRequest } from 'next/server';
+
+const BACKEND_URL = 'http://45.32.212.237:8000/stream';


The hardcoded backend URL http://45.32.212.237:8000/stream exposes infrastructure details and creates a security risk. This should be moved to an environment variable (e.g., NEXT_PUBLIC_ASSISTANT_API_URL) for better security and configurability across environments.

Copilot · 2026-01-15T06:25:03Z

packages/agent/config/__init__.py

@@ -0,0 +1 @@
+""


The empty file config/__init__.py contains only "" instead of being truly empty or having a proper docstring. This should either be removed entirely or contain a proper empty string/docstring like """Configuration module."""

Suggested change

""

"""Configuration module."""

Copilot · 2026-01-15T06:25:04Z

packages/agent/src/agent/graph.py

+llm = ChatOpenAI(model="gpt-4o-mini", temperature=0.2).with_config({"async": True})
+TOOLS = ALL_TOOLS
+TOOLS_BY_NAME = TOOLS_BY_NAME
+CONSULTANT_PROMPT = CONSULTANT_PROMPT = """


The variable CONSULTANT_PROMPT is defined redundantly with a double assignment operator (CONSULTANT_PROMPT = CONSULTANT_PROMPT = ...). This is likely a typo and should be a single assignment.

Copilot · 2026-01-15T06:25:04Z

packages/agent/tools/coverletter_tool.py

+    # 工作人员版本
+    elif data.user_type == "worker":
+        # 中文工作者身份
+        employer_name = data.employer_name if data.employer_name else "[请在此处补充公司名称]"


The tool accesses data.employer_name on line 109, but this field is not defined in the FullApplicationInput model. This will cause an AttributeError when user_type is "worker". The employer_name field should be added to the model or this logic should be refactored to use existing fields.

Copilot · 2026-01-15T06:25:04Z

packages/agent/.gitignore

-__pycache__/
-Qrent_knowledge_base/
-coverletter.ts
+<<<<<<< HEAD


The .gitignore file has a Git merge conflict marker (<<<<<<< HEAD) at line 1 that was not resolved. This needs to be cleaned up before merging.

Suggested change

<<<<<<< HEAD

Copilot · 2026-01-15T06:25:05Z

packages/agent/config/config.py

@@ -0,0 +1,11 @@
+import os


Import of 'os' is not used.

Suggested change

import os

Copilot · 2026-01-15T06:25:05Z

packages/agent/src/agent/graph.py

+from langgraph.runtime import Runtime
+from typing_extensions import TypedDict
+
+import json


Import of 'json' is not used.

Copilot · 2026-01-15T06:25:06Z

packages/agent/src/agent/graph.py

+from typing_extensions import TypedDict
+
+import json
+import asyncio


Import of 'asyncio' is not used.

Copilot · 2026-01-15T06:25:06Z

packages/agent/src/agent/graph.py

+from langchain_core.messages import (
+    BaseMessage, HumanMessage, SystemMessage, ToolMessage, AIMessage
+)


Import of 'HumanMessage' is not used.
Import of 'AIMessage' is not used.

Copilot · 2026-01-15T06:25:06Z

packages/agent/src/agent/graph.py

+from langchain_core.messages import (
+    BaseMessage, HumanMessage, SystemMessage, ToolMessage, AIMessage
+)
+from langchain_core.runnables import RunnableConfig


Import of 'RunnableConfig' is not used.

wiperi · 2026-01-18T05:39:38Z

Would it better to put the code change of the agent in a separate PR?

wiperi · 2026-01-18T05:42:41Z

Please reference current ai-chatbot design i.e. fold/unfold button, floating chat box. Sliding side bar looks nice on PC but it might bring trouble to mobile users. We want to put mobile user experience on the first place.

wiperi

Is it better let AI put some description text on the beginning of each file / components. This might help other human / AI better understand the code structure.

wiperi · 2026-01-25T05:18:08Z

may be we can put these 2 button together. On the bottom of the page.

wiperi · 2026-01-25T05:20:06Z

I think we may need to adjust the size and location of chat box for mobile size. Remove the gap between chat box and switch button.

and avoid overflowing on the right side, make it at the center of screen.

wiperi · 2026-01-25T05:21:18Z

may be wen can enhance the style of button. Add some enhancement based on current button style.

dylan added 5 commits January 5, 2026 22:09

optimize structure

5fb00ed

chore: bump qrent gitlink

be09038

fix: vercel issue

56373dd

chore: sync pnpm lockfile

88cc45f

update parents letter function

e8563af

CHELSEADOPAMIN requested review from dulala12 and wiperi as code owners January 15, 2026 06:20

CHELSEADOPAMIN requested a review from Copilot January 15, 2026 06:21

Copilot started reviewing on behalf of CHELSEADOPAMIN January 15, 2026 06:22 View session

Copilot AI reviewed Jan 15, 2026

View reviewed changes

wiperi requested changes Jan 18, 2026

View reviewed changes

CHELSEADOPAMIN added 3 commits January 19, 2026 11:48

feat: add assistant chatbox and cover letter form

d696730

fix: eslint errors

8a53408

Feat: parent letter button in assistant ui

5de7234

CHELSEADOPAMIN force-pushed the agent-frontend branch from fad09a5 to 5de7234 Compare January 19, 2026 00:49

vercel bot deployed to Preview January 19, 2026 00:51 View deployment

feat: add floating assistant UI components

34bb163

vercel bot had a problem deploying to Preview January 25, 2026 03:30 Failure

feat: add floating assistant UI components

fc53a5c

vercel bot deployed to Preview January 25, 2026 03:34 View deployment

Merge remote-tracking branch 'origin/agent' into agent-frontend

ef7124d

vercel bot deployed to Preview January 25, 2026 04:03 View deployment

		@@ -0,0 +1,100 @@
		import { NextRequest } from 'next/server';

		const BACKEND_URL = 'http://45.32.212.237:8000/stream';

		@@ -0,0 +1 @@
		"" No newline at end of file

Conversation

CHELSEADOPAMIN commented Jan 15, 2026

Uh oh!

vercel bot commented Jan 15, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Copilot AI Jan 15, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 15, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 15, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 15, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 15, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 15, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 15, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 15, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 15, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 15, 2026

Choose a reason for hiding this comment

Uh oh!

wiperi commented Jan 18, 2026

Uh oh!

wiperi commented Jan 18, 2026

Uh oh!

wiperi left a comment

Choose a reason for hiding this comment

Uh oh!

wiperi commented Jan 25, 2026

Uh oh!

wiperi commented Jan 25, 2026

Uh oh!

wiperi commented Jan 25, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

vercel bot commented Jan 15, 2026 •

edited

Loading