Update by xxnuo · Pull Request #9 · LinguaSpark/server

xxnuo · 2025-09-30T01:42:50Z

@Aalivexy

Aalivexy

如果你要PR的话请一个功能发一个PR，并且保证编译通过

Aalivexy · 2025-10-05T05:40:39Z

Cargo.toml

你换成相对路径这样别人编译的时候是找不到的

Aalivexy · 2025-10-05T05:44:07Z

outlook给我邮件全干垃圾箱了，啧

This reverts commit a3be385.

Aalivexy · 2025-10-09T06:48:43Z

src/endpoint.rs

    text: String,
    from: String,
    to: String,
+    result: String,


建议你先review一下代码，不要vibe完能跑就行，加这玩意真是不明所以
如果你希望有个啥新功能可以开个issue说一下为什么想要，我来做就行，这些commit真是污染git历史......

这些改动是为了兼容 mtranserver 的接口，commit message 里写了。
并非 “不要 vibe 完能跑就行，加这玩意真是不明所以”

Aalivexy · 2025-10-09T06:50:20Z

src/main.rs

 const ENV_API_KEY: &str = "API_KEY";
 const ENV_LOG_LEVEL: &str = "RUST_LOG";

+// Environment variable aliases


为什么要加这些环境变量？我没看懂要兼容什么？

Aalivexy · 2025-10-09T06:51:03Z

src/main.rs

    Ok(models)
 }

+async fn shutdown_signal() {


不同的功能不要在同一个PR实现，没法review

rust 相比原版缺少了关闭服务器的响应事件

Aalivexy · 2025-10-09T06:52:16Z

src/main.rs

    let server_ip = std::env::var(ENV_SERVER_IP).unwrap_or_else(|_| "127.0.0.1".to_string());
-    let server_port = std::env::var(ENV_SERVER_PORT)
-        .ok()
+    let server_port = get_env_with_alias(ENV_SERVER_PORT, ENV_SERVER_PORT_ALIAS)


这环境变量alias真是污染代码啊

Aalivexy · 2025-10-09T06:53:38Z

src/translation.rs

    let from_code = get_iso_code(&source_lang)?;
    let to_code = get_iso_code(&target_lang)?;

+    // If source and target languages are the same, return the original text


不同功能请不要在同一个PR实现

同语言直接返回

Aalivexy · 2025-10-09T06:55:34Z

tests/real_translation_test.rs

这测试我觉得是真没什么用，要说覆盖率它基本没啥覆盖，要说跑通它也没有全流程跑通

调试用的，可以直接删掉，或者你继续增加全覆盖的测试

Aalivexy · 2025-10-09T06:57:30Z

你要不然和我说你想要啥吧，这PR没法合并，不然commit历史会一团糟

xxnuo · 2025-10-12T05:35:36Z

兼容 mtranserver 的接口和环境变量，还有一些都写在注释里了。不想要的你可以直接编辑 PR

xxnuo · 2025-10-12T05:39:25Z

如果你有更好的兼容方式，可以加上，环境变量用别名的方式确实不太优雅。

Aalivexy · 2025-10-12T05:52:28Z

如果你有更好的兼容方式，可以加上，环境变量用别名的方式确实不太优雅。

问题是我没看懂要兼容谁

兼容 mtranserver 的接口和环境变量，还有一些都写在注释里了。不想要的你可以直接编辑 PR

这俩本来也不能直接替换，模型不一样啊，linguaspark默认是不捆绑模型的，如果你希望容器级别的默认模型+兼容，那应该改的是dockerfile/compose文件，而不是代码

xxnuo · 2025-10-12T05:54:18Z

行

xxnuo added 10 commits August 29, 2025 15:40

update: compatibility interface

cd52182

fix: 响应停止事件

84376b0

add: 兼容性环境变量别名

203f3b6

fix: cors allow_credentials

e60477e

Update main.rs

658dc3c

Update main.rs

bcb3019

fix: 目标语言与源语言相同时直接返回

4ce3014

chore: 调试模式使用固定模型文件夹

a147e58

add: 翻译测试

c4e7554

debug: core

a3be385

Aalivexy requested changes Oct 5, 2025

View reviewed changes

Cargo.toml

Copy link
Copy Markdown

Collaborator

Aalivexy Oct 5, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

你换成相对路径这样别人编译的时候是找不到的

xxnuo closed this Oct 9, 2025

Revert "debug: core"

b1e191c

This reverts commit a3be385.

xxnuo reopened this Oct 9, 2025

Aalivexy reviewed Oct 9, 2025

View reviewed changes

Aalivexy closed this Oct 9, 2025

Conversation

xxnuo commented Sep 30, 2025

Uh oh!

Aalivexy left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Aalivexy commented Oct 5, 2025

Uh oh!

Choose a reason for hiding this comment

Uh oh!

xxnuo Oct 12, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Aalivexy commented Oct 9, 2025

Uh oh!

xxnuo commented Oct 12, 2025

Uh oh!

xxnuo commented Oct 12, 2025

Uh oh!

Aalivexy commented Oct 12, 2025

Uh oh!

xxnuo commented Oct 12, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

xxnuo Oct 12, 2025 •

edited

Loading