审查 Remix v2 代码中的 v1 形态 meta 导出(v2 中为破坏性变更),cookie 安全漏洞(httpOnly、secure、密钥轮换),认证门位于错误层级,以及其他
Remix v2 Meta, 会话, Auth, 和 CSRF 代码审查.是一项面向实际任务的技能,主要用于Remix v2 meta/ SEO, 会话, 认证门, 和 CSRF 代码路径.;由 diff 触碰任何 o 时的 magazine Review-remix- v2 编辑器。
实际使用前应先确认任务范围、数据来源、运行环境、必要权限和关键参数,再依据技能说明逐步执行;若输入条件不完整,应先补齐信息或采用保守配置,避免因错误假设导致结果偏离需求。执行过程中需要关注工具调用是否成功、接口或依赖是否可用、输出格式是否符合预期,并对异常提示、缺失字段和边界情况进行处理;
涉及批量任务时,还应保存进度,避免中断后重复操作。该技能适合用于一次性任务,也可以接入自动化工作流,与其他技能或上层代理配合完成更完整的业务链路;在组合使用时,应明确每一步的输入输出关系,并避免不同步骤之间出现参数冲突。
审查 Remix v2 的 meta/SEO、session、auth-gate 和 CSRF 相关代码路径。当 diff 涉及以下任一内容时,由统一评审器 review-remix-v2 加载执行本检查:meta/links 导出项、root.tsx、*.server.ts 中的 session/cookie 模块、读写 session 的 loader/action,或使用 /useFetcher 发起变更操作的代码。
参见 remix-v2-meta-sessions 获取权威实践模式。
| 问题类型 | 参考文档 |
|---|---|
meta 返回 v1 对象结构(BREAKING)、OG 简写语法、document.title 生效、缺失 /、父级合并逻辑 |
references/meta-v2-shape.md |
缺失 httpOnly/secure 属性、硬编码密钥、单字符串 secrets、替换式而非前置式密钥轮换 |
references/cookie-security.md |
在组件中执行鉴权检查、在 loader 中执行登出、缺失 commitSession 调用、未提交即调用 flash |
references/auth-gates.md |
手动 fetch POST 绕过 CSRF 防护、CSRF token 存于 session cookie 中、无 CSRF 防护机制、共享密钥使用不当 |
references/csrf.md |
最高风险检测项 —— 应优先指出: 在 v2 项目中仍使用 v1 的 meta 对象结构(即 return { title, description })。该写法可通过 TypeScript 类型检查,但运行时会被忽略,导致页面渲染时无标题且无任何 meta 标签。请对所有 export const meta 进行 grep 检查,并确认其返回值以 [ 开头,而非 {。
meta 返回 MetaDescriptor[](数组以 [ 开头),而非 v1 对象结构{ property, content } 格式,而非 v1 简写格式 { "og:title": "..." }document.title = "..." 或 useEffect(() => { document.title = ... }) — meta 信息应通过 meta 导出声明设置root.tsx 在 内包含 和 meta 值,则使用 matches.flatMap((m) => m.meta ?? [])meta 对 data 做空值防护(loader 可能未执行 / 在 404 时返回 undefined)httpOnly: true,且 secure: process.env.NODE_ENV === "production"secrets 从 process.env 读取(无硬编码字符串,.env.example 中无已提交的敏感值)secrets 是支持轮换的数组(新密钥前置,旧密钥保留),而非单一字符串值session.set/session.unset/session.flash 调用后,响应中均包含 "Set-Cookie": await commitSession(session)loader(或 action)中,通过 requireUserId(request) 实现 —— 不在组件内做重定向action(POST),而非 loader(GET)csrf.validate(request)createCookie("csrf", ...) 创建,而非复用 session cookie / useFetcher,以便 AuthenticityTokenInput 自动注入 token(禁止手动 fetch POST)以下为正确用法,不应作为问题上报:
sameSite: "lax" —— 可接受的默认值。并非所有应用都需要 "strict";仅当威胁模型要求更强防护(例如:无其他 CSRF 防护措施)时才需标记。meta 返回 [] —— 若路由明确不需输出任何 meta 标签(例如继承 root 标签或依赖兄弟路由),此行为合法。links 返回 [] —— 若路由无专属样式表或预加载资源,此行为合法。session.flash(...) 后紧跟 commitSession(session) —— 此为标准的两行 flash 模式。分离写法是正确的,不应标记为“缺失 commit”。action(非 loader)中 —— 对仅支持 POST 的路由(如登出、删除)属正确做法。loader 用于保护 GET 请求,action 用于保护变更操作。charset 和 viewport 作为普通 JSX 元素直接写入 root.tsx 的 中 —— 推荐此方式,可避免 v2 的“不自动合并”行为引发重复标签警告。secrets: [process.env.X!, process.env.X_OLD!] —— 当上方存在 fail-fast 守卫(如 if (!process.env.X) throw)时,! 非空断言可接受。throw redirect(...) 出现在 loader/action 内 —— Remix 标准模式;抛出响应是预期行为。commitSession 在 loader 中调用(不仅限于 action) —— 当 loader 需读取 flash 消息并清除它时,此调用为必需。仅当满足特定上下文条件时,才标记以下问题:
| 问题 | 仅当满足以下条件时标记 |
|---|---|
| action 中缺失 CSRF 校验 | 应用声明使用 remix-utils/csrf 作为防护机制,或该 action 面向公网(非内网/VPN 受限)且未实施 Origin 校验 |
sameSite: "lax" |
应用未使用基于库的 CSRF 防护,且无 Origin 校验 —— 此时 "lax" 成为唯一防线,但防护不足 |
缺失 secure 标志 |
Cookie 配置对象对应生产环境的 session/CSRF cookie(非测试桩或注释示例) |
meta 返回 [] |
该路由明确文档化要求设置路由专属标签(例如公开落地页)—— 空数组通常表示有意继承,不应默认标记 |
鉴权检查位于 action 而非 loader |
该路由支持 GET 渲染(即定义了 loader)—— 对纯 POST 路由,action 是正确鉴权位置 |
登出在 action 中实现且使用 |
永不标记 —— 此为标准模式 |
手动 fetch POST |
目标为内部 Remix action,且请求头中未携带 CSRF token |
secrets: [singleValue] |
应用已上线生产环境,或部署时间足够长、确有密钥轮换需求 —— 此类情况建议标记为改进建议,而非 CRITICAL 级别 |
请按顺序执行以下检查。除非当前批次所有门禁均已通过,否则不得起草面向用户的发现报告。
位置证据 —— 通过标准: 每个问题需注明仓库路径,并附带具体行号范围或所查阅文件中的简短原文摘录。仅依据 diff 或记忆得出的结论不视为通过。对于 meta/links/session 类问题,引用文件必须是 .ts/.tsx 路由模块、root.tsx 或 *.server.ts —— 不得为通用配置文件。
豁免核查 —— 通过标准: 对每个问题,需用一句话说明其为何不属于“有效模式(请勿标记)”所涵盖的情形。尤其需明确排除:sameSite: "lax"、空 meta/links 数组、以及标准的 flash + commitSession 两行模式。
Meta 结构核查 —— 通过标准: 在标记任何关于 meta 的问题前,必须阅读实际函数体并确认其返回值。TypeScript 类型可能掩盖真实结构(v1 对象可满足弱类型的 MetaFunction 别名)。核查要点:返回表达式以 [ 开头,且每个元素均为 descriptor 对象。若以 { 开头,则为 v1 结构 —— 标记为 CRITICAL;若为 [],则属合法(不标记)。
流程合规 —— 通过标准: 已完成 review-verification-protocol 中规定的《提交前验证检查清单》。
export const meta 或 export const links,或 root.tsx → 查阅 meta-v2-shape.mdcreateCookieSessionStorage、createCookie,或任何配置 cookie 的 *.server.ts → 查阅 cookie-security.mdsession 的 loader/action,或任意 auth 辅助函数 → 查阅 auth-gates.mdmeta 导出是否均返回数组?所有 OG/Twitter 标签是否均采用 { property, content } 格式?root.tsx 是否在 内包含 和 ?httpOnly + secure: NODE_ENV === 'production'?secrets 是否来自环境变量且为数组(支持轮换)?Set-Cookie: await commitSession(session) 头?action(POST)?变更类 action 是否校验 CSRF(或已明确记录威胁模型)?meta 对象结构(BREAKING)、OG 简写、document.title 反模式、root 模板搭建、父级合并逻辑httpOnly/secure/sameSite 设置、硬编码密钥、密钥轮换规范remix-utils/csrf 集成方式、手动 fetch 绕过风险、专用 cookie 使用、共享密钥管理规范请先完成全部硬性准入门禁(尤其是第 3 项 —— Meta 结构核查),再仅报告那些仍通过 review-verification-protocol 提交前验证检查的问题。