Skip to content
cccskills
FreePublished on GitHub

alignment-review

Review code changes for bugs and alignment with OpenEnv principles and RFCs. Use when reviewing PRs, checking code before commit, or when asked to review changes. Implements two-tier review model.

The English summary is being prepared. Showing the original description.

See how to install

Files included (1)

  • SKILL.md3.3 KB

SKILL.md (original)

Review the instructions the agent will receive before you install.

Alignment Review

Review code changes for alignment with OpenEnv principles using a two-tier model.

Instructions

  1. Run automated checks first:

    • Execute bash .claude/hooks/lint.sh - capture lint issues
    • Execute bash .claude/hooks/check-debug.sh - capture debug code
  2. Read alignment documents:

    • .claude/docs/PRINCIPLES.md - design principles
    • .claude/docs/INVARIANTS.md - system invariants
  3. Read open RFCs:

    • Scan rfcs/ directory for all RFC files
    • Note the status of each RFC (Draft, In Review, Accepted, Implemented)
    • Pay special attention to Draft and In Review RFCs - these represent active design discussions
  4. Analyze changes (use git diff or provided diff):

    • Identify mechanical issues (Tier 1)
    • Flag alignment concerns (Tier 2)
    • Flag conflicts with open RFCs (Tier 2)

Tier 1: Uncontentious Issues (Fix Immediately)

These are issues to fix without human input:

  • Lint failures from hook output
  • Debug code from hook output (print statements, breakpoints)
  • Uninitialized variables, type errors
  • Missing imports, syntax errors
  • Security issues (credential exposure, injection vulnerabilities)

Tier 2: Alignment Discussion Points

For each potential alignment concern, format as:

**ALIGNMENT FLAG**: [Brief description]
- **Principle/RFC at stake**: [Which principle from PRINCIPLES.md or RFC number]
- **The concern**: [What seems misaligned or in conflict]
- **Suggested reviewer**: @darktex [pull actual reviewers based on authors of the specific line of PRINCIPLES.md and INVARIANTS.md using git blame, and/or authors of conflicting RFCs]

Examples of Tier 2 Issues

Principle conflicts:

  • Adding external reward computation (violates "rewards in environment")
  • Client importing server code (violates client-server separation)
  • New API that differs from Gymnasium pattern

RFC conflicts (flag even for Draft/In Review RFCs):

  • Change conflicts with design proposed in an open RFC
  • Change pre-empts a decision being discussed in an RFC
  • Change implements something differently than an RFC proposes
  • Change affects an area covered by an RFC under review

Why flag RFC conflicts? Even if an RFC isn't finalized, flagging conflicts helps focus design discussions. The change might be correct and the RFC might need updating, or vice versa - either way, the team should discuss.

Output Format

## Alignment Review Report

### Automated Checks
- Lint: [PASS/FAIL] - [summary]
- Debug code: [CLEAN/FOUND] - [details]

### Open RFCs Context
[List any RFCs in Draft or In Review status that might be relevant to these changes]

### Tier 1: Fixes Required
- [ ] path/file.py:123 - [issue description]
- [ ] path/file.py:456 - [issue description]

### Tier 2: Alignment Discussion

#### Principle Conflicts
[ALIGNMENT FLAGS for principle violations, or "None identified"]

#### RFC Conflicts
[ALIGNMENT FLAGS for RFC conflicts, or "None identified"]

### Summary
- X mechanical issues to fix
- Y alignment points for human review
- Z RFC conflicts to discuss

Reviews

No reviews yet. Tell others what you thought.

Comments

No login needed. Share how it worked for you, ask a question, or note where it did not run.

No comments yet. Be the first to write one.

Comments are published right away. Do not include personal information such as email addresses or phone numbers. Up to 2 URLs.

0 / 1,000 characters

This form is protected by reCAPTCHA, and the Google Privacy Policy and Terms of Service apply.

More skills from this repository

Deploy an OpenEnv environment to Hugging Face Spaces. Use when asked to deploy, push to Hugging Face, or update a space.

The English summary is being prepared. Showing the original description.

huggingface/OpenEnv2.7KUpdated Oct 16, 2026

Generate OpenEnv environments from a concrete use case (for example, "generate an env for the library textarena"). Use when asked to design or implement a new environment under envs/ by researching a target library/API, selecting matching OpenEnv examples, asking key implementation questions, and building models/client/server/openenv.yaml. Do not use for model training or evaluation tasks.

The English summary is being prepared. Showing the original description.

huggingface/OpenEnv2.7KUpdated Oct 16, 2026

hf-cli

Free

Hugging Face Hub CLI (`hf`) for downloading, uploading, and managing repositories, models, datasets, and Spaces on the Hugging Face Hub. Replaces now deprecated `huggingface-cli` command.

The English summary is being prepared. Showing the original description.

huggingface/OpenEnv2.7KUpdated Oct 16, 2026

Diagnose and recover failing or stuck Hugging Face Space deployments for OpenEnv environments. Use when deploying envs from `envs/` to the Hub (`openenv` namespace with version suffixes), when Spaces are in `BUILDING`/`APP_STARTING`/`RUNTIME_ERROR`, or when release collections need to be reconciled after targeted redeploys.

The English summary is being prepared. Showing the original description.

huggingface/OpenEnv2.7KUpdated Oct 16, 2026

Make tests pass. Invoke after /write-tests produces failing tests.

The English summary is being prepared. Showing the original description.

huggingface/OpenEnv2.7KUpdated Oct 16, 2026

OpenEnv CLI (`openenv`) for scaffolding, validating, building, and pushing OpenEnv environments.

The English summary is being prepared. Showing the original description.

huggingface/OpenEnv2.7KUpdated Oct 16, 2026

See all skills by huggingface

Report a problem with this skill