Skip to content

chore(autoreview): sync canonical skill#13

Merged
vincentkoc merged 1 commit into
mainfrom
chore/autoreview-canonical-4664d27
Jul 12, 2026
Merged

chore(autoreview): sync canonical skill#13
vincentkoc merged 1 commit into
mainfrom
chore/autoreview-canonical-4664d27

Conversation

@vincentkoc

Copy link
Copy Markdown
Member

What Problem This Solves

The vendored autoreview skill had drifted behind the canonical
openclaw/agent-skills implementation, including its model defaults,
credential-isolation hardening, and review-input safety fixes.

Why This Change Was Made

Delete-sync the complete .agents/skills/autoreview directory from canonical
commit 4664d27da471d1cb71bebdd9845dc8a6c56d6bbe (openclaw/agent-skills#82).
The resulting directory exactly matches canonical subtree
0c6893e09512920c2af2ddb71c3a5de308cd47f0, including AGENTS.md and the
CLAUDE.md -> AGENTS.md symlink. No repository-local variants remain.

User Impact

Repository autoreview now uses the canonical gpt-5.6-sol high-effort default
with access-only fallback to gpt-5.6-terra, plus the current isolation and
secret-scanning behavior maintained by the organization skill repository.

Evidence

  • Recursive archive parity and matching SHA-256 hashes for all nine skill entries

  • Canonical subtree: 0c6893e09512920c2af2ddb71c3a5de308cd47f0

  • Focused #82 regression tests: 10 passed

  • Canonical hardening suite: 192 passed, 1 skipped

  • Compatibility suite: 26 passed

  • Autoreview self-tests: passed

  • bun run check: 19 tests, 286 assertions; 5 skills validated; package contents valid

  • git diff --check: passed

  • Canonical #82 Sol/high autoreview proof: clean

  • Tests or contract checks cover behavior changes

  • Affected consumer and compatibility surfaces were considered

  • No credentials, private paths, private hosts, or restricted assets are included

Related: openclaw/agent-skills#82

@vincentkoc
vincentkoc marked this pull request as ready for review July 12, 2026 09:28
@vincentkoc
vincentkoc requested review from a team and Patrick-Erichsen as code owners July 12, 2026 09:28
@vincentkoc
vincentkoc merged commit 9f26ffc into main Jul 12, 2026
4 of 5 checks passed
@vincentkoc
vincentkoc deleted the chore/autoreview-canonical-4664d27 branch July 12, 2026 09:29
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