feat: add proxy #97

ayangweb · 2023-03-25T18:36:46Z

No description provided.

ayangweb · 2023-03-25T18:37:11Z

.env

这段代码的变更只有两处，第一处是删除了 VITE_OPEN_AI_URL 和 VITE_OPEN_AI_API_KEY 的定义，第二处是保留了 VITE_APP_NAME 的定义并且去掉了文件结尾的换行符。

从这段代码来看，并没有直接的错误风险。但如果你打算在之后使用 OpenAI API，那么删除 VITE_OPEN_AI_URL 和 VITE_OPEN_AI_API_KEY 可能会导致问题。此外，建议将 VITE_CONFIG_PATH 设置为一个更具描述性的名称，以提高可读性。最后，确保文件结尾有一个换行符可能更好，因为这是一些工具和惯例所要求的。

ayangweb · 2023-03-25T18:37:19Z

.eslintrc-auto-import.json

@@ -9,8 +9,6 @@
 "HOST_URL": true,
 "InjectionKey": true,
 "Message": true,
- "OPENAI_CHAT_URL": true,
- "OPENAI_CREDIT_URL": true,
 "OPEN_AI_MODEL": true,
 "PropType": true,
 "Ref": true,


这段代码补丁中删除了 "OPENAI_CHAT_URL" 和 "OPENAI_CREDIT_URL" 两个变量的定义。如果这两个变量被其他代码使用，那么这个更改可能会导致错误。如果这两个变量不再需要，请忽略此警告。

建议对代码进行更详细的检查，以确保删除这些变量不会影响其他部分的功能，并且可以考虑添加注释来说明更改的原因和目的。

ayangweb · 2023-03-25T18:37:32Z

src/api/github.ts

+ },
+ host: 'GITHUB'
+ })
+}


这段代码看起来已经完成了对请求函数的修改，同时也将异步操作加入到了函数中。

不过还有一些需要改进的地方：

在请求头中设置HostUrl时使用了一个常量HOST_URL.GITHUB，但是在函数定义的上下文中没有找到该常量的定义。应该检查一下是否有必要将其引入或者定义。

在异步函数中使用return await语句是多余的，可以直接使用return语句。

请求头中使用host属性时，应该将其名称改为headers。

如果这个request函数不是内部实现的，建议添加try-catch语句以处理可能的异常情况。

如果这个函数被频繁调用，可以考虑将常用的URL和请求头信息单独提取出来，作为参数传入该函数，以便复用。

ayangweb · 2023-03-25T18:37:46Z

src/api/openAi.ts

@@ -96,7 +109,7 @@ export const getOpenAICreditApi = async () => {
 const apiKey = getOpenAIKey()
 if (!apiKey) return

- const result = await request(OPENAI_CREDIT_URL, {
+ const result = await request('/dashboard/billing/credit_grants', {
 method: 'GET',
 headers: {
 Authorization: `Bearer ${apiKey}`,


这段代码主要是使用 OpenAI 的 API 进行聊天和获取授权信息。以下是我的简要代码审查：

在 getOpenAIResultApi 函数中，URL 被更改为 /v1/chat/completions，这是正确的 URL。

在 getOpenAIResultStreamApi 函数中，URL 也被更改为 /v1/chat/completions。这个函数还包括代理设置、温度和最大令牌等参数。在这种情况下，建议添加注释来说明这些参数的作用和默认值。另外，如果使用代理，应该在请求 URL 中添加代理 URL。

在 getOpenAICreditApi 函数中，URL 被更改为 /dashboard/billing/credit_grants，这是正确的 URL。

总体而言，这段代码看起来很好，没有明显的 bug 或问题。如果需要改进，可以考虑添加更多的注释和错误处理代码。

ayangweb · 2023-03-25T18:38:01Z

src/api/request.ts

- } catch ({ message }: any) {
- dialogErrorMessage(`请求出错：${message}`)
+ } catch (error) {
+ dialogErrorMessage(`请求出错：${error}`)
 }
 }


这段代码的作用是进行网络请求。以下是我的代码审核意见：

在 options 参数中增加了一个新的属性 host，并且使用了一个常量 REQUEST_HOST。建议在代码中加入对 REQUEST_HOST 常量的定义。

代码中使用了一个名为 useSettingsStore() 的函数，但是该函数未被定义或导入。请确认是否有正确导入该函数。

当开启了代理时，会将请求地址修改为代理地址。然而，在代理地址和请求地址拼接时没有进行字符处理，可能会导致请求地址出现问题。建议在拼接时加入合适的字符处理方法。

在捕获错误时，使用了 error.message，但是 error 可能是一个字符串，没有 message 属性。建议在捕获错误时，先判断 error 是否为对象，再取 message 属性。

以上是我的审核意见，希望对你有所帮助。

ayangweb · 2023-03-25T18:38:15Z

src/components/Settings/components/About.vue

@@ -39,7 +48,7 @@ onMounted(async () => {
 </div>

 <div class="flex flex-col gap-3" v-if="contributors.length">
- <b>贡献者</b>
+ <span>贡献者</span>
 <div class="max-h-30 flex flex-col gap-3 overflow-auto">
 <a
 v-for="item in contributors"


这是一段 Vue.js 代码，目的是在页面中显示 ChatGPT-Desktop 应用程序的信息。以下是我的建议：

在第五行，添加 alt 属性到 img 标签中，以提高可访问性。

在第六行，使用 div 元素包装 img 元素，以便在图像大小变化时保持元素的纵横比。

在第十二行，为复制电脑信息按钮添加点击事件监听器，并在处理程序中实现该功能。

在第十四行，为 BUG 反馈按钮添加 target="_blank" 属性，以便在新标签页中打开链接。

在第二十二行和第三十行，将 b 元素替换为 span 元素，因为 b 元素不再被视为语义上的强调。

以上是我的建议和注意事项，希望对你有所帮助！

ayangweb · 2023-03-25T18:38:32Z

src/components/Settings/components/General.vue

@@ -58,7 +63,8 @@ const { themeMode, autoStart, isRememberPosition, proxyUrl } =
 li {
 --uno: flex items-center gap-3;
 > i {
- --uno: min-w-50 text-right not-italic text-[var(--color-text-2)];
+ --uno: cursor-default min-w-50 text-right not-italic
+ text-[var(--color-text-2)];
 }
 ::v-deep(.arco-radio-button-content) {
 --uno: flex h-[30px] items-center;


这段代码主要是关于设置页面的一些渲染和交互逻辑，总体上看没有明显的错误风险。

但是可以提出以下改进建议：

在第3行中，toggleTheme并没有被使用到，可以考虑删除。

在第4行中，变量名为proxy，但在第5行中却写成了proxyUrl，这会导致后面代码中出现混淆。建议在这里将变量名修改为proxy，或者在第5行中修改为proxyUrl。

在第6-13行中，对代理地址进行了设置，并使用了v-show来控制其显示与否。建议在这里增加一些验证逻辑，例如：检查输入的代理地址是否合法、是否支持该代理类型等。

第18行中，gap-3可以考虑改为space-x-3以保持风格统一。

最后，建议对代码进行进一步注释，以提高代码可读性和可维护性。

ayangweb · 2023-03-25T18:39:13Z

src/components/Settings/components/Modal.vue

@@ -64,7 +76,8 @@ watch(
 li {
 --uno: flex items-center gap-3;
 > i {
- --uno: min-w-50 text-right not-italic text-[var(--color-text-2)];
+ --uno: cursor-default min-w-50 text-right not-italic
+ text-[var(--color-text-2)];
 }
 }
 }


这段代码主要是关于一个设置页面的代码片段。以下是一些值得注意的地方：

在第2行，使用了 storeToRefs 函数，将 useSettingsStore() 中的多个属性解构为一个对象。这样可以更方便地管理状态，并且可以使代码更加简洁。

在第10-11行，定义了两个响应式引用类型的变量 totalCredit 和 usedCredit，但是没有看到它们在后面的代码中被使用到。这可能是因为这部分代码只是一个片段，不包括完整的代码。

在第14行，使用了 watchEffect 函数，用于监听 modalParams.isMemory 属性的变化并执行相应的回调函数。由于代码片段不完整，无法确定这个函数的实际作用和效果。

在第16-36行，定义了一个列表，其中包含了一些设置项。每个设置项都是一个列表项，包括一个标签和一个控件，例如输入框或滑块。这些控件的属性和事件处理程序在这里没有给出，但是可以通过其他的代码来获得。

在第19-21行，使用了 <a-input-password> 组件来创建一个密码输入框，并且将其与 apiKey 变量双向绑定。这意味着当用户在输入框中输入内容时，apiKey 变量的值也会更新。

在第23-25行，使用了 <a-slider> 组件来创建一个滑块，并且将其与 modalParams.temperature 变量双向绑定。这意味着当用户在滑块上拖动时，modalParams.temperature 变量的值也会更新。

在第27-29行，使用了 <a-slider> 组件来创建另一个滑块，并且将其与 modalParams.max_tokens 变量双向绑定。这意味着当用户在滑块上拖动时，modalParams.max_tokens 变量的值也会更新。

在第31-34行，定义了一个 div 元素，其中包含了两个按钮。这些按钮的作用和事件处理程序在这里没有给出，但是可以通过其他的代码来获得。

在第38-43行，使用了 <a-tooltip> 组件来创建了一个提示框，当用户双击温度或最大长度标签时，会将 modalParams.temperature 或 modalParams.max_tokens 的值重置为默认值。

建议改进：

建议在代码中添加一些注释，以便于其他开发人员更好地理解代码的作用和实现方式。

建议在代码中添加一些异常处理机制，例如在获取 API 密钥时，如果出现错误应该如何处理等。

建议将一些常量和字符串抽象成常量或者枚举类型，以便于统一管理和修改。

ayangweb · 2023-03-25T18:39:24Z

src/components/Settings/index.vue

@@ -17,7 +22,7 @@ onMounted(() => {
 :visible="visible"
 width="70%"
 :mask-style="{
- borderRadius: '0.75rem'
+ borderRadius: platform === 'Darwin' ? '0.75rem' : '0'
 }"
 :footer="false"
 @ok="visible = false"


这段代码主要是一个Vue 3的组件，通过@tauri-apps/api/event和@tauri-apps/api/os库来监听事件和获取当前操作系统类型。以下是我的一些评论：

在onMounted钩子函数中，可以使用async/await异步地获取操作系统类型。

const platform = ref ('Darwin')中的Darwin应该改为操作系统的默认值，以免在其他平台上出现问题。

在mask-style中，建议使用三元运算符代替if语句，因为它更简洁。

至于潜在的错误风险和改进建议，我需要更多的上下文和代码背景才能给出更具体的答案。

ayangweb · 2023-03-25T18:39:43Z

src/constants/shared.ts

- OPENAI = 'https://api.openai.com/',
- GITHUB = 'https://api.github.com/'
+ OPENAI = 'https://api.openai.com',
+ GITHUB = 'https://api.github.com'
 }


这段代码是一个 TypeScript 模块，主要定义了一些常量和枚举类型。以下是我对这段代码的简要代码审查：

第 7 行到第 9 行定义了一个名为 DEFAULT_SHORTCUT_KEY 的常量数组，其中存储了默认的快捷键。

第 11 行定义了一个名为 OPEN_AI_MODEL 的常量字符串，其中存储了 OpenAI 模型的名称。

第 13 行到第 18 行定义了两个名为 OPENAI_CHAT_URL 和 OPENAI_CREDIT_URL 的常量字符串，其中存储了与 OpenAI 相关的 API 地址。

第 20 行到第 26 行定义了一个名为 HOST_URL 的枚举类型，其中存储了不同的 API 主机地址。

从代码中可以看出，可能存在以下几点问题或改进建议：

第 13 行到第 18 行中使用了环境变量 import.meta.env.VITE_OPEN_AI_URL 来构造 API 地址，但没有对其进行检查或验证，可能会导致安全问题。建议在使用前进行检查或限制。

第 24 行和第 26 行中的 API 地址末尾多余了斜杠字符 /，可能会导致请求失败或不必要的重定向。建议将其去掉。

ayangweb · 2023-03-25T18:40:02Z

src/stores/settings.ts

@@ -146,7 +146,7 @@ export const useSettingsStore = defineStore(
 'autoStart',
 'isMemory',
 'isRememberPosition',
- 'proxyUrl',
+ 'proxy',
 'modalParams'
 ]
 }


这段代码主要是一个 Vue.js 的组件定义，定义了一个名为 useSettingsStore 的组件。

对于代码的风险和建议：

在第 4 行，代理地址已被改成了一个包含绕过和 url 的 reactive 对象，这样做可能会增加一些额外的复杂度，但也可以更好地控制代理设置。需要根据具体情况进行评估。

在第 7 行，max_tokens 属性的命名方式与其他属性不同，可能会导致混乱和错误。建议使用统一的命名方式。

在第 10 行，registerKey 函数没有返回任何值，这可能是意外的。需要根据具体情况进行评估。

在第 17 行，toRefs(modalParams) 返回的是 modalParams 的属性引用，而 proxyUrl 只是一个普通的 ref 对象，这可能会导致不一致性。建议使用 reactive 对象来管理所有的属性。

在第 22 行，getters 和 actions 数组中缺少一些属性，可能是由于被省略或者删除了。需要根据具体情况进行评估。

除此之外，代码看起来比较清晰和简洁，没有明显的风险和问题。

ayangweb · 2023-03-25T18:40:14Z

src/types/shared.d.ts

@@ -1 +1,3 @@
 export type THEME_MODE = 'light' | 'dark' | 'system'
+
+export type REQUEST_HOST = 'OPENAI' | 'GITHUB'


这段代码补丁只是添加了两个 TypeScript 类型的定义。

对于第一个类型 THEME_MODE，它只有三种取值：'light'、'dark' 和 'system'，这符合预期。

对于第二个类型 REQUEST_HOST，它只有两种取值：'OPENAI' 和 'GITHUB'，这也是合理的。

因为这段代码只是定义了类型，并没有实际的逻辑操作，所以不存在明显的 bug 风险。但如果在实际使用中发现问题，可以考虑对类型进行调整或增加约束条件。

关于代码改进的建议，可以考虑添加注释来解释这些类型的含义和用法，以便其他开发人员更好地理解和使用这些类型。

ayangweb · 2023-03-25T18:40:26Z

src/utils/openai.ts

 Message.warning('请先填写 OpenAi API Key')
 return false
 }

- return apiKey || import.meta.env.VITE_OPEN_AI_API_KEY
+ return apiKey
 }

 /**


这段代码主要是用于获取OpenAI API Key的函数。以下是我的一些建议和改进点：

在getOpenAIKey函数中，移除了对import.meta.env.VITE_OPEN_AI_API_KEY的判断。这意味着，如果apiKey不存在，函数将返回false，而不是从环境变量中获取API密钥。因此，建议将这个判断重新加回去，以确保API密钥可以从环境变量中获取。

考虑添加参数到getOpenAIKey函数中，以允许在调用该函数时覆盖默认的apiKey值。

在使用Message.warning()函数时，最好提供更具体的错误消息，以帮助用户更好地理解问题所在。

除此之外，代码看起来很不错，没有明显的风险或缺陷。

ayangweb · 2023-03-25T18:40:38Z

src/utils/saveImage.ts

- ? '#fff'
- : '#000'
+ backgroundColor:
+ document.body.getAttribute('arco-theme') === 'light' ? '#fff' : '#000'
 })

 // base64 转 buffer


这段代码的作用是将一个HTML节点转换为canvas图像，并将其保存为base64编码的字符串。以下是我的一些建议和注意事项：

在使用document.body.appendChild()添加元素时，应该始终检查元素是否已经存在于DOM中，以避免出现重复添加的问题。

对于html2canvas函数，应该考虑设置其他选项来优化图像质量和性能。

在处理颜色值时，可以使用ES6中的模板字面量来使代码更加清晰易读。

在将base64字符串转换为buffer时，应该使用Buffer.from()方法而不是手动解码。

除此之外，我没有看到其他潜在的错误或风险。

ayangweb · 2023-03-25T18:40:49Z

src/vite-env.d.ts

@@ -4,8 +4,6 @@
 interface ImportMetaEnv {
 readonly VITE_CONFIG_PATH: string
 readonly VITE_APP_NAME: string
- readonly VITE_OPEN_AI_URL: string
- readonly VITE_OPEN_AI_API_KEY: string
 }

 // eslint-disable-next-line no-unused-vars


这段代码是一个 TypeScript 接口，用于定义环境变量。从代码来看，该应用程序需要使用两个环境变量：VITE_CONFIG_PATH 和 VITE_APP_NAME。在代码中注释了一个 eslint-disable-next-line no-unused-vars，这表示该文件中存在未使用的变量或参数，但因为它们可能在其他地方被使用，所以 ESLint 不会报错。

根据代码片段，没有发现任何明显的错误或风险。如果您想要改进代码，则可以添加有关如何使用 VITE_OPEN_AI_URL 和 VITE_OPEN_AI_API_KEY 环境变量的文档。此外，您可以考虑将接口名称更改为更具描述性的名称，以提高代码的可读性。

feat: add proxy

15129d0

ayangweb merged commit fd0e493 into master Mar 25, 2023

ayangweb commented Mar 25, 2023

View reviewed changes

ayangweb deleted the dev_ayang branch March 25, 2023 19:12

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

feat: add proxy #97

feat: add proxy #97

ayangweb commented Mar 25, 2023

ayangweb Mar 25, 2023

ayangweb Mar 25, 2023

ayangweb Mar 25, 2023

ayangweb Mar 25, 2023

ayangweb Mar 25, 2023

ayangweb Mar 25, 2023

ayangweb Mar 25, 2023

ayangweb Mar 25, 2023

ayangweb Mar 25, 2023

ayangweb Mar 25, 2023

ayangweb Mar 25, 2023

ayangweb Mar 25, 2023

ayangweb Mar 25, 2023

ayangweb Mar 25, 2023

ayangweb Mar 25, 2023

		@@ -1 +1,3 @@
		export type THEME_MODE = 'light' \| 'dark' \| 'system'

		export type REQUEST_HOST = 'OPENAI' \| 'GITHUB'

feat: add proxy #97

feat: add proxy #97

Conversation

ayangweb commented Mar 25, 2023

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment