Skip to content

[codex] add monthly performance audit playbook#3021

Merged
fermatmind merged 1 commit into
mainfrom
codex/performance-monthly-audit-docs
Jul 12, 2026
Merged

[codex] add monthly performance audit playbook#3021
fermatmind merged 1 commit into
mainfrom
codex/performance-monthly-audit-docs

Conversation

@fermatmind

Copy link
Copy Markdown
Owner

What changed

  • Added a durable map of the 12 merged July 2026 performance PRs, the product/runtime areas they protect, their original failure modes, and monthly regression checks.
  • Added a reusable monthly full-stack performance audit prompt covering Next.js/RSC/SSR/hydration, Laravel/API/query/cache/serialization, browser/network evidence, cache privacy boundaries, and PR-train output.
  • Added a short monthly invocation that instructs Codex to audit first and generate a detailed report, scoped PR queue, and executable /goal prompt 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
  • Verified the worktree contains exactly the two new documentation files.
  • Verified all 12 historical PR IDs are represented.
  • Verified the reusable template contains the required /goal output, privacy exclusions, and no-warmup/no-deploy boundaries.

Intentionally deferred

  • No runtime, CMS, database, cache, workflow, PR-train ledger, or deployment changes.
  • No production scan or warmup was executed.

Repository rule impact

Documentation only. Backend/CMS authority, SEO enumeration, publishing policy, and deployment ownership are unchanged.

@fermatmind fermatmind marked this pull request as ready for review July 12, 2026 10:28

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 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

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge 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 👍 / 👎.

@fermatmind fermatmind merged commit 127f3e4 into main Jul 12, 2026
9 checks passed
@fermatmind fermatmind deleted the codex/performance-monthly-audit-docs branch July 12, 2026 10:33
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