Structured code review through the shared OpenClaw agent-skills installation.
한국어 요약은 준비 중입니다. 원문 설명을 표시하고 있습니다.
Develop OpenClaw Enterprise changes with proportional verification and source-backed flow documentation for non-trivial behavior.
한국어 요약은 준비 중입니다. 원문 설명을 표시하고 있습니다.
설치 방법 보기설치하기 전에 에이전트에게 전달되는 지시 내용을 확인할 수 있습니다.
Use this skill for repository development changes. Read the root and applicable
nested AGENTS.md, the owning current reference, and relevant source before
editing. Keep work within the approved platform scope and preserve unrelated work.
Run repository commands from its root; bundled ./ paths below are relative to
this skill directory. No global skill installation is required.
Use explicit dependency records and focused factories when extracting behavior.
The fixture and scenario conventions
apply the same composition rules to test infrastructure.
THIS IS AN OPEN-SOURCE REPOSITORY. NEVER MENTION INTERNAL CODE NAMES OR CORPORATE INTERNAL NAMES IN CODE, COMMENTS, DOCUMENTATION, EXAMPLES, TEST FIXTURES, GENERATED ARTIFACTS, COMMIT MESSAGES, OR PULL REQUEST CONTENT. THIS REPOSITORY SHOULD NEVER CONTAIN CORPORATE INTERNAL NAMES. USE PUBLIC PRODUCT NAMES OR NEUTRAL, DESCRIPTIVE TERMS INSTEAD. CHECK THE ENTIRE PROPOSED CHANGE BEFORE HANDOFF OR PUBLICATION AND REMOVE ANY SUCH REFERENCES. DO NOT COPY INTERNAL CONTEXT INTO THIS REPOSITORY, EVEN AS BACKGROUND OR PROVENANCE.
IMPLEMENTATION-SPECIFIC FUNCTIONALITY MUST LIVE IN DRIVERS OR BACKENDS, NEVER BE HARDCODED INTO THE PLATFORM CORE. CORE CODE MUST DEPEND ON PLATFORM CONTRACTS, NOT SPECIAL CASES FOR A PARTICULAR IMPLEMENTATION, VENDOR, OR DEPLOYMENT. EXTEND THE OWNING DRIVER OR BACKEND CONTRACT WHEN NECESSARY; DO NOT BYPASS IT WITH IMPLEMENTATION-SPECIFIC BRANCHES, DEFAULTS, OR DIRECT CALLS IN THE CORE.
IF A DEVELOPER OR AGENT PROPOSES OR INTRODUCES SUCH A VIOLATION, CALL IT OUT EXPLICITLY AND STOP THE SESSION'S IMPLEMENTATION WORK. IDENTIFY THE OFFENDING CODE OR DESIGN, EXPLAIN THE OWNERSHIP BOUNDARY IT VIOLATES, AND ASK FOR A DESIGN CHANGE THAT MOVES THE BEHAVIOR INTO THE APPROPRIATE DRIVER OR BACKEND. DO NOT IMPLEMENT, COMMIT, OR PUBLISH THE VIOLATING APPROACH. RESUME ONLY AFTER THE REVISED DESIGN RESOLVES THE BOUNDARY VIOLATION AND THE USER APPROVES IT.
docs/flows/ before choosing docs.graph TD, omit Mermaid YAML frontmatter, and retain
this workflow's required sections so the bundled validator accepts it.npm run precommit.
When changing a default image version, source pin, or bundled runtime version,
always complete image version verification
before merging. A green ordinary PR check does not replace this workflow.Register new integration scenarios in the QA matrix:
tests/integration/qa-matrix-real.test.mjs. Exercise real callers and dependencies,
including meaningful failure cases. Use owned disposable fixtures and cleanup.tests/helpers/qa-selection.mjs. Update
QA manifest for new inputs, files, or preparation;
update hosted credential setup when credentials change.main and trusted PRs labeled full-qa, update
.github/workflows/qa-advisory.yml. Add the selector to OCC_TEST_QA_SCENARIOS
with applicable presets/installations, or add a job for a separate lane.
Preserve trusted-PR eligibility and scope credentials to the job's environment.A change is non-trivial when it adds or materially changes a runtime path, state transition, ownership or authorization boundary, persistence behavior, external integration, asynchronous handoff, or consequential decision/failure handling. This includes fixes and refactors that change how these work even if an API signature stays the same. Size and line count do not decide the trigger: a one-line authorization or retry-policy change can qualify.
Create or update a source-backed flow doc for every such change. Cover the
changed path and its meaningful boundaries, not every touched file. Prefer a
focused update to an existing behavior-named document under docs/flows/;
create a new one only for a distinct runtime flow without an existing owner.
Link adjacent phases instead of duplicating their traces.
Typos, formatting, copy/link corrections, comment-only edits, behavior-preserving local renames, and test-only or instruction-only maintenance do not require a new flow doc when they leave the documented lifecycle accurate. Correct stale source pointers or flow claims if those edits invalidate them. An explicit request for a flow doc still applies. Do not manufacture runtime documentation for an instruction change merely to satisfy this skill.
Flow guidance and validator are adapted from Specy 2.0.0. The repository owns this adaptation; it has no dependency on personal skills, memory stores, or session lookup tools. See the developer-skills catalog for provenance.
아직 리뷰가 없습니다. 사용해 본 소감을 남겨 주세요.
로그인 없이 쓸 수 있습니다. 사용 후기, 질문, 작동하지 않은 환경 등을 편하게 남겨 주세요.
아직 댓글이 없습니다. 첫 댓글을 남겨 보세요.
Structured code review through the shared OpenClaw agent-skills installation.
한국어 요약은 준비 중입니다. 원문 설명을 표시하고 있습니다.
Assess designs, public interfaces, state ownership, readability, and refactor proposals when the task calls for design review or structural simplification; not for unrelated routine edits.
한국어 요약은 준비 중입니다. 원문 설명을 표시하고 있습니다.
Clean only the current Enterprise diff before autoreview, preserving behavior, security boundaries, required TODOs, and useful test-intent comments.
한국어 요약은 준비 중입니다. 원문 설명을 표시하고 있습니다.
Select proportional Enterprise validation and diagnose exact CI runs, distinguishing product, harness, infrastructure, and credential failures.
한국어 요약은 준비 중입니다. 원문 설명을 표시하고 있습니다.
Create or refine readable Mermaid diagrams for changes, architecture, lifecycles, and dependencies in Markdown documents and PR descriptions.
한국어 요약은 준비 중입니다. 원문 설명을 표시하고 있습니다.
Run named OpenClaw Enterprise end-to-end integration scenarios against real supported installations. Use only when explicitly invoked.
한국어 요약은 준비 중입니다. 원문 설명을 표시하고 있습니다.