[codex] add monthly performance audit playbook#3021
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 7cc4d535f8
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| 在报告最后,生成一段完整、可直接粘贴给 Codex 的 `/goal` 风格执行指令,内容包含: | ||
|
|
||
| - 本月建议 PR 的严格顺序和依赖 | ||
| - 建分支 → 实现 → 本地测试 → scope validation → commit → push → PR → poll checks → scoped fix → merge → sync main → cleanup → post-merge revalidate |
There was a problem hiding this comment.
Require manifest authorization before execution
For monthly scans that propose a new performance PR id not already present in docs/codex/pr-train.yaml, the root AGENTS.md planning rule requires the report to include exact manifest/state entries and a follow-up execution prompt that explicitly asks for manifest/state authorization first. This generated /goal goes straight into branch → implement → PR → merge, so pasting it for a newly proposed train item can bypass the required manifest/state authorization step.
Useful? React with 👍 / 👎.
What changed
/goalprompt without changing production.Why
The July performance train changed several independent layers across fap-web and fap-api. Future monthly reviews need a stable checklist so cache, JavaScript, degradation, payload, fanout, and monitoring regressions are evaluated with the same evidence boundaries instead of rediscovering the original scope.
Validation
git diff --check/goaloutput, privacy exclusions, and no-warmup/no-deploy boundaries.Intentionally deferred
Repository rule impact
Documentation only. Backend/CMS authority, SEO enumeration, publishing policy, and deployment ownership are unchanged.