Skip to content

DOC-6741 Codex-only skill to review Claude's work#3484

Merged
andy-stark-redis merged 2 commits into
mainfrom
DOC-6741-codex-skill-to-review-claudes-work
Jun 11, 2026
Merged

DOC-6741 Codex-only skill to review Claude's work#3484
andy-stark-redis merged 2 commits into
mainfrom
DOC-6741-codex-skill-to-review-claudes-work

Conversation

@andy-stark-redis

@andy-stark-redis andy-stark-redis commented Jun 11, 2026

Copy link
Copy Markdown
Contributor

Note: we can add a corresponding Claude skill to review Codex's work if there is demand for it :-)


Note

Low Risk
Documentation-only agent skill files under .codex/skills/; no product code, build, or publishing paths change.

Overview
Adds a Codex-only agent skill (DOC-6741) so reviewers can independently audit Claude-authored docs and code in this repo without bias from prior Claude mistakes.

SKILL.md defines when to use the skill, a five-step workflow (target → source of truth → pattern checklist → findings-first output → no-findings handling), independence rules, and optional “skill memory” suggestions after reviews.

references/review-style.md standardizes review output (Findings / Open Questions / Notes), severity labels (blocker through nit), and discipline (no speculation, cite paths/lines).

references/claude-review-patterns.md is a living checklist of Redis-docs–specific heuristics (stale snapshots, cross-client drift, generated docs drift, async/timeouts, bogus config, RediSearch TAG escaping, shared demo UI copy), each with status, what to check, pass criteria, and false-positive guards.

Reviewed by Cursor Bugbot for commit f0ac03a. Bugbot is set up for automated code reviews on this repo. Configure here.

@andy-stark-redis andy-stark-redis requested a review from a team June 11, 2026 15:13
@andy-stark-redis andy-stark-redis self-assigned this Jun 11, 2026
@github-actions

github-actions Bot commented Jun 11, 2026

Copy link
Copy Markdown
Contributor

DOC-6741

@jit-ci

jit-ci Bot commented Jun 11, 2026

Copy link
Copy Markdown

🛡️ Jit Security Scan Results

CRITICAL HIGH MEDIUM

✅ No security findings were detected in this PR


Security scan by Jit

@cursor cursor 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.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 0c40fb7. Configure here.

Comment thread .codex/skills/claude-review/references/claude-review-patterns.md Outdated

@dwdougherty dwdougherty left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

👍🏻

@andy-stark-redis

Copy link
Copy Markdown
Contributor Author

Thanks @dwdougherty !

@andy-stark-redis andy-stark-redis merged commit 3296c1f into main Jun 11, 2026
73 checks passed
@andy-stark-redis andy-stark-redis deleted the DOC-6741-codex-skill-to-review-claudes-work branch June 11, 2026 15:37
EliShteinman added a commit to EliShteinman/docs that referenced this pull request Jun 11, 2026
Adds a Codex-only skill for reviewing Claude's work under .codex/skills/.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants