Skip to content

Conversation

@star-e
Copy link
Contributor

@star-e star-e commented Nov 4, 2025

No description provided.

@wuzhiming
Copy link
Contributor

wuzhiming commented Nov 4, 2025

我感觉是不是可以这样

  1. 这个应该是一个 file 的补充,比如放到 core/filesystem,看着不需要写入子进程,直接在主进程弄一个就行
  2. 我们封装除了需要提供这几个 api,还需要做几个事情
  1. 限制目前支持的格式,避免 ai 乱调用,比如 ts、js、json 等有限的几个
  2. api 对文件操作完成后,我们需要对资源进行操作,比如 refresh 操作,保证操作完后,资源是最新的,
  3. 如果失败或报错要抛出异常

@wuzhiming
Copy link
Contributor

✅ 测试通过

类型 结果
🧪 ✅ 单元测试
🔄 ⏭️ E2E 测试 (跳过)
📈 ⏭️ 覆盖率检查 (跳过)

🎄 圣诞老人说代码写得不错

Run #19061045162

@wuzhiming
Copy link
Contributor

✅ 测试通过

类型 结果
🧪 ✅ 单元测试
🔄 ✅ E2E 测试 · 报告
📈 覆盖率 90.70% (39/43) · 详情

🏆 完美通过!这就是高质量代码!

Run #19100832402

@star-e star-e marked this pull request as ready for review November 5, 2025 16:12
@wuzhiming
Copy link
Contributor

✅ 测试通过

类型 结果
🧪 ✅ 单元测试
🔄 ✅ E2E 测试 · 报告
📈 覆盖率 90.70% (39/43) · 详情

🎉 哎呦,不错哦!

Run #19108474779

@star-e star-e requested a review from wuzhiming November 6, 2025 03:02
@wuzhiming
Copy link
Contributor

✅ 测试通过

类型 结果
🧪 ✅ 单元测试
🔄 ✅ E2E 测试 · 报告
📈 覆盖率 90.70% (39/43) · 详情

💾 内存表示:这代码真省心

Run #19126826403

@wuzhiming
Copy link
Contributor

✅ 测试通过

类型 结果
🧪 ✅ 单元测试
🔄 ✅ E2E 测试 · 报告
📈 覆盖率 92.86% (39/42) · 详情

💾 内存表示:这代码真省心

Run #19127223868

@wuzhiming
Copy link
Contributor

✅ 测试通过

类型 结果
🧪 ✅ 单元测试
🔄 ✅ E2E 测试 · 报告
📈 覆盖率 92.86% (39/42) · 详情

☕ 测试通过,咖啡可以更香了

Run #19127686208

@wuzhiming
Copy link
Contributor

✅ 测试通过

类型 结果
🧪 ✅ 单元测试
🔄 ✅ E2E 测试 · 报告
📈 覆盖率 92.86% (39/42) · 详情

🎄 圣诞老人说代码写得不错

Run #19127875890

@wuzhiming
Copy link
Contributor

✅ 测试通过

类型 结果
🧪 ✅ 单元测试
🔄 ✅ E2E 测试 · 报告
📈 覆盖率 92.86% (39/42) · 详情

💾 内存表示:这代码真省心

Run #19129152790

@wuzhiming
Copy link
Contributor

✅ 测试通过

类型 结果
🧪 ✅ 单元测试
🔄 ✅ E2E 测试 · 报告
📈 覆盖率 90.70% (39/43) · 详情

🎉 哎呦,不错哦!

Run #19129930221

@wuzhiming
Copy link
Contributor

✅ 测试通过

类型 结果
🧪 ✅ 单元测试
🔄 ✅ E2E 测试 · 报告
📈 覆盖率 90.70% (39/43) · 详情

📡 信号满格,代码质量5G速度

Run #19129989911

@wuzhiming
Copy link
Contributor

✅ 测试通过

类型 结果
🧪 ✅ 单元测试
🔄 ✅ E2E 测试 · 报告
📈 覆盖率 90.70% (39/43) · 详情

🎉 哎呦,不错哦!

Run #19130933003

import { insertTextAtLine, eraseLinesInRange, replaceTextInFile, listTextEditFileExtensions } from '../../core/filesystem/file-edit';

export class FileEditorApi {
@tool('file-query-extensions')
Copy link
Contributor

Choose a reason for hiding this comment

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

这个可能我写的不够清楚,我的意思是,咱们在 file-xxx-text 这三个函数中的入参,增加一个 fileType 的参数,这个参数是个 z.enum(['xx', 'xx', 'xx']),让 ai 填写,设置这个值的主要目的其实是让 ai 能看到我们支持的类型,并且他不需要在通过 api 查询了。

@wuzhiming
Copy link
Contributor

✅ 测试通过

类型 结果
🧪 ✅ 单元测试
🔄 ✅ E2E 测试 · 报告
📈 覆盖率 92.86% (39/42) · 详情

💾 内存表示:这代码真省心

Run #19156613323

@wuzhiming wuzhiming merged commit fc628f8 into cocos:main Nov 7, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants