Skip to content

fix: uploadFiles 工具返回结构复杂且无法验证静态托管可访问性#644

Open
binggg wants to merge 3 commits intomainfrom
automation/attribution-issue-moizgwrc-0s6rbv-uploadfiles-moj1byk6
Open

fix: uploadFiles 工具返回结构复杂且无法验证静态托管可访问性#644
binggg wants to merge 3 commits intomainfrom
automation/attribution-issue-moizgwrc-0s6rbv-uploadfiles-moj1byk6

Conversation

@binggg
Copy link
Copy Markdown
Member

@binggg binggg commented Apr 28, 2026

Attribution issue

  • issueId: issue_moizgwrc_0s6rbv
  • category: tool
  • canonicalTitle: uploadFiles 工具返回结构复杂且无法验证静态托管可访问性
  • representativeRun: application-js-vite-hello-world-hosting-deploy/2026-04-28T18-49-43-3bt7rc

Automation summary

root_cause:
AI 在部署 Vite 项目到子目录时,SKILL.md 和工具描述错误地指导使用相对路径 (./) 而非绝对路径 (/vite-test/)。当用户访问 URL 不带尾部斜杠时,相对路径解析错误导致静态资源 404。
具体问题:

  1. SKILL.md 第 131 行:"Prefer relative asset paths for static hosting compatibility" - 错误指导
  2. hosting.ts 工具描述:建议检查 publicPath/base/assetPrefix 是否使用相对路径 - 错误指导
  3. 缺少子目录部署前的强制验证机制
    changes:
  4. config/source/skills/web-development/SKILL.md:
  • 将 "Static hosting defaults" 改为 "Static hosting deployment checklist"
  • 移除错误的相对路径建议
  • 添加子目录部署强制检查清单(3项验证)
  • 明确 cloudPath 格式要求
  1. mcp/src/tools/hosting.ts:
  • 更新 uploadFiles 工具描述,包含子目录部署强制检查要求
  • 更新 cloudPath 参数描述,明确格式和子目录部署要求
  1. config/source/skills/web-development/frameworks.md:
  • 添加 Vite 子目录部署详细指导
  • 说明为何必须使用绝对路径而非相对路径
  • 提供代码示例
    validation:
  • 所有 3 个技能回归测试通过 (10/10 tests)
  • TypeScript 编译检查通过
    follow_up:
    无。修改已完成,可以提交 PR 进行评测验证。

Changed files

  • config/source/skills/web-development/SKILL.md
  • config/source/skills/web-development/frameworks.md
  • mcp/src/tools/hosting.ts

@chatgpt-codex-connector
Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

binggg and others added 2 commits April 29, 2026 03:51
- Add '通常不需要调用此工具' to main description
- Change '云存储对象请使用' to '云存储对象路径请改用' in cloudPath description

This fixes the test failure in storage-hosting-guidance.test.ts
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant