-
Notifications
You must be signed in to change notification settings - Fork 16
Add optional moeflow-companion service #38
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes from 20 commits
a4bd396
4fcf8a4
e399d85
f6f82ff
8cdce06
0a36d0c
507fa21
31930fe
d4c89d9
492b055
74e4002
d0e7ab9
4517128
d1f8f83
cfa7415
f6285fa
219c384
c8279e3
b673879
88b2938
6744e82
d6fa56a
0ddc876
a8c6700
5eb7668
4538da3
83ac745
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -28,3 +28,4 @@ yarn-error.log* | |
| .vscode/* | ||
| .idea | ||
| stats.html | ||
| /public/moeflow-runtime-config.json | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,14 @@ | ||
| locale-json: src/locales/en.json src/locales/zh-cn.json | ||
|
|
||
| locale-json-watch: | ||
| watch make locale-json | ||
|
|
||
| format: | ||
| npm run format:fix | ||
|
|
||
| src/locales/en.json: src/locales/messages.yaml | ||
| node_modules/.bin/tsx scripts/generate-locale-json.ts | ||
|
|
||
| src/locales/zh-cn.json: src/locales/messages.yaml | ||
| node_modules/.bin/tsx scripts/generate-locale-json.ts | ||
|
|
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,7 @@ | ||
| { | ||
| "comment": "Runtime configuration overrides. See src/configs.tsx", | ||
| "moeflowCompanion": { | ||
| "gradioUrl": "http://localhost:7860", | ||
| "defaultMultimodalModel": "gemini-2.0-flash-lite" | ||
| } | ||
| } |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -8,7 +8,7 @@ import { | |
| Tag, | ||
| message, | ||
| } from 'antd'; | ||
| import { Button as CustomButton } from './Button'; | ||
| import { Button as CustomButton } from './shared/Button'; | ||
| import classNames from 'classnames'; | ||
| import produce from 'immer'; | ||
| import qs from 'qs'; | ||
|
|
@@ -324,7 +324,7 @@ export const TeamInsightProjectList: FC<TeamInsightProjectListProps> = ({ | |
| }} | ||
| /> | ||
| <Column | ||
| title={formatMessage({ id: 'site.output' })} | ||
| title={formatMessage({ id: 'project.export' })} | ||
| key="project" | ||
| width="30%" | ||
|
Comment on lines
+327
to
329
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 💡 Verification agent 🧩 Analysis chainHeader i18n key updated to project.export — LGTM Ensure the new key exists in locales to avoid runtime fallback. 🏁 Script executed: #!/bin/bash
set -euo pipefail
for f in src/locales/en.json src/locales/zh-cn.json; do
jq -e '.project.export' "$f" >/dev/null || { echo "Missing project.export in $f"; exit 1; }
done
echo "OK: project.export present"Length of output: 227 Add missing i18n key 🤖 Prompt for AI Agents |
||
| render={(text, record: APIInsightProjectWithPage, index) => { | ||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -15,7 +15,7 @@ import { createTeam } from '../store/team/slice'; | |
| import { toLowerCamelCase } from '../utils'; | ||
| import { FC, UserTeam } from '../interfaces'; | ||
| import { Team } from '../interfaces'; | ||
| import { LIST_ITEM_DEFAULT_HEIGHT } from './ListItem'; | ||
| import { LIST_ITEM_DEFAULT_HEIGHT } from '@/components/shared/ListItem'; | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 💡 Verification agent 🧩 Analysis chainSwitch to shared ListItem constant is aligned with the alias move. Looks good. Verify alias setup for "@/..." across TS, bundler, and tests, and that the constant remains a named export. 🏁 Script executed: #!/bin/bash
# 1) TS path alias
rg -nP --glob 'tsconfig*.json' '"paths"\s*:\s*{[^}]*"@/\*"' -S
# 2) Bundler alias (vite/webpack/craco)
rg -nP '(alias|resolve\.alias).*@' -S
# 3) Jest/Vitest mapping
rg -nP 'moduleNameMapper|alias.*@' -S
# 4) Ensure named export exists
rg -nP "export\s+const\s+LIST_ITEM_DEFAULT_HEIGHT\b" src/components/shared/ListItem.tsx
# 5) (Optional) locate deprecated antd internal types to migrate off if using antd v5
rg -nP "antd/(lib|es)/select" -SLength of output: 647 Alias configuration for import { LIST_ITEM_DEFAULT_HEIGHT } from '@/components/shared/ListItem';will fail. Add the 🤖 Prompt for AI Agents |
||
|
|
||
| /** 搜索团队的属性接口 */ | ||
| interface TeamSearchListProps { | ||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
💡 Verification agent
🧩 Analysis chain
Spin and ListItem constant imports updated to shared modules.
All good. If migrating fully to antd v5 (Modal/Drawer use
open), plan to replaceSelectValuewithSelectProps['value']to avoid internal imports.🏁 Script executed:
Length of output: 564
Replace internal
SelectValueimports withSelectProps['value']across componentsRemove
import { SelectValue } from 'antd/lib/select'and replace withThen update all
onChange(and other) type annotations fromSelectValuetoSelectProps['value']in:• src/components/RoleSelect.tsx (line 3 import, line 46 annotation)
• src/components/InvitationList.tsx (line 4 import, line 294 annotation)
• src/components/InviteUser.tsx (line 4 import, line 197 annotation)
🤖 Prompt for AI Agents