Skip to content

fix: 修正 week4 导读翻译错误 + 添加 starter app 复用设计说明#17

Open
AlexAnys wants to merge 1 commit intomasterfrom
fix/review-verified-improvements
Open

fix: 修正 week4 导读翻译错误 + 添加 starter app 复用设计说明#17
AlexAnys wants to merge 1 commit intomasterfrom
fix/review-verified-improvements

Conversation

@AlexAnys
Copy link
Copy Markdown
Owner

@AlexAnys AlexAnys commented Apr 4, 2026

变更内容

1. 修正 week4/assignment-zh.md 翻译错误

中文导读中写的是「改进 Week 2 的应用」,但原始 assignment.md 说的是 the starter application found in week4/。修正为「改进 week4/ 中的 Starter Application」。

2. 添加 Week 4-7 共用 Starter App 的设计说明

在 agentic-coding/README.md 中添加一段说明,解释 Week 4-7 为什么使用同一个 Notes/Action Items 应用。

验证方式:对比上游 mihail911/modern-software-dev-assignments 的代码:

  • Week 4 vs Week 5:代码只有 main.py 的 title 一行不同(Week 4Week 5),上游完全一致
  • Week 5 vs Week 6:增加了 CORS middleware、requirements.txt、tests/init.py 等(为安全扫描准备漏洞)
  • Week 6 vs Week 7:少量差异(移除 CORS、调整部分文件)

这是原课程的刻意设计:同一个 playground,每周用不同的工具/方法论(Claude Code → Warp → Semgrep → Graphite)来操作。

附:审查验证摘要

原始发现 验证结果
Week 4-7 代码几乎完全相同 假问题 — 上游完全一致,刻意设计
Week 6/7 main.py 标题互换 ✅ 真 bug,但是上游 bug(week6 写了 Week 7,week7 写了 Week 6)。本仓库忠实镜像了上游,不建议单独修改
Week 9/10 实质上是空的 夸大了 — Week 9 有 60 行(讲义链接、嘉宾、6 篇阅读、自学建议),Week 10 有 45 行(结业指引)。上游甚至没有 week9/10 文件夹,这些是本仓库新增内容,已明确标注「阅读 + 自学」
Getting Started 空壳 ⚠️ 真但已标注 — 根 README 和模块 README 都标注了「规划中」
两模块间路径不清晰 ⚠️ 主观建议 — App Dev README 的指引其实是合理的(有/无开发背景走不同路),可以未来迭代
App Dev 缺少动手练习 ⚠️ 主观建议 — 模块状态是「框架已搭建」,还在迭代中
缺少学完后出口 ⚠️ 主观建议 — Week 10 已有自学实践建议,可以未来扩展
术语表缺双向链接 ⚠️ 主观建议 — 有价值但优先级低
缺可视化学习地图 ⚠️ 主观建议 — 未来可加

结论:上次报告中的主要「问题」大部分是假问题或夸大了。本 PR 修正了唯一确认的翻译错误,并添加说明预防未来的困惑。

1. week4/assignment-zh.md: 修正「改进 Week 2 的应用」→「改进 week4/ 中的 Starter Application」
   原始 assignment.md 说的是 week4/ 的 starter app,中文导读误写为 Week 2

2. agentic-coding/README.md: 添加 Week 4-7 共用同一个 starter app 的设计说明
   经上游 mihail911/modern-software-dev-assignments 验证,这是原课程的刻意设计
   添加说明避免学习者困惑
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.

1 participant