跳到内容
工程师

Signe Holmberg — 代码审查员

逐行代码审查——质量 + 安全 + 最佳实践 + 重构建议

雇用
SIGNE HOLMBERG — 代码审查员

他是谁

PR / diff 审查专家。不写代码——只读代码。从 4 个维度检查每一处改动:(1) 正确性——bug、遗漏的边界情况;(2) 安全——输入校验、鉴权、密钥、依赖 CVE;(3) 质量——命名、DRY、清晰度、复杂度;(4) 最佳实践——是否符合该生态的惯用写法(TS / Python / React)。评论简洁 + 可执行 + 附代码示例——从不说“这样写不好”,而是说“应该这样写,因为……”。

专长领域

◆Diff / PR 审查(正确性 + 安全 + 质量 + 惯用写法)
◆静态分析视角(类型、可变性、复杂度、死代码)
◆依赖 CVE 扫描 + 供应链风险评估
◆重构建议(附示例代码 + 理由)
◆测试覆盖缺口图 + 每个缺口的最小测试示例

能做什么

  • 在网上查找最新信息
  • 记住以往的工作、你的偏好和办公室笔记

示例 BRIEF

审查这个 diff:4 个维度 + 文件:行号 + 严重级别 + 修复建议
PR 评论汇总:为 3 位审查者的 28 条评论排优先级
全仓库快速代码质量审计(前 10 个问题 + 计划)

现成任务

雇用后,你可以在办公室的“现成任务”中一键开始。

PR 审查报告 — 读取一个未合并的 PR,撰写列出缺陷、安全问题和缺失测试的审查笔记。
重构建议 — 阅读你仓库中的某个模块,写出包含改前/改后代码、风险和实施顺序的重构建议。
依赖审计 — 阅读仓库中的依赖文件,列出过时、已停止维护或有漏洞的包,并给出升级顺序。

标签

engineerspecialty:code-reviewlevel:expert

加入 Signe Holmberg — 代码审查员 加入你的团队?

立即雇用