Skip to content

feat(op-program): add vm-compat-triage skill#19444

Merged
joshklop merged 3 commits intodevelopfrom
aj/vm-compat-triage-skill
Mar 9, 2026
Merged

feat(op-program): add vm-compat-triage skill#19444
joshklop merged 3 commits intodevelopfrom
aj/vm-compat-triage-skill

Conversation

@ajsutton
Copy link
Copy Markdown
Contributor

@ajsutton ajsutton commented Mar 9, 2026

Summary

  • Adds a Claude Code skill for triaging analyze-op-program-client CI failures
  • Guides interactive review of new syscall/opcode findings from vm-compat
  • Walks through each finding one at a time with full call stacks and source file paths
  • Cascades unreachable decisions to auto-resolve related findings
  • Regenerates baseline from scratch to keep it clean (removes stale entries)

The skill lives at .claude/skills/vm-compat-triage/SKILL.md and is invokable as /vm-compat-triage.

Test plan

  • Used the skill to triage the geth v1.17.1 upgrade (all: upgrade op-geth #19433) — 196 findings resolved, baseline regenerated and verified

🤖 Generated with Claude Code

@ajsutton ajsutton requested a review from a team as a code owner March 9, 2026 01:36
@ajsutton ajsutton force-pushed the aj/vm-compat-triage-skill branch from ea686c6 to 4cf49e3 Compare March 9, 2026 01:47
Add a skill that guides triage of analyze-op-program-client CI failures.
The skill walks through each new vm-compat finding interactively, letting
the engineer mark call stacks as unreachable or acceptable, then
regenerates the baseline from scratch.

Key features:
- Prefers running vm-compat locally (CI logs truncate large JSON output)
- Shows full call stacks with source file paths from the PR branch
- Cascades unreachable decisions to auto-resolve related findings
- Regenerates baseline from scratch (not merge) to remove stale entries
- Includes MIPS64 syscall reference table

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@ajsutton ajsutton force-pushed the aj/vm-compat-triage-skill branch from 4cf49e3 to 292f57d Compare March 9, 2026 01:52
ajsutton and others added 2 commits March 9, 2026 01:54
CircleCI truncates large log output, silently dropping findings from
the beginning of the JSON array. Triage based on incomplete data gives
false confidence. Only CI artifacts and local execution are supported.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
The triage report was an intermediate artifact that didn't add value.
Decisions are captured in the triage state file and the baseline itself.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Copy link
Copy Markdown
Contributor

@joshklop joshklop left a comment

Choose a reason for hiding this comment

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

🚀

@joshklop joshklop added this pull request to the merge queue Mar 9, 2026
Merged via the queue into develop with commit c694b04 Mar 9, 2026
79 checks passed
@joshklop joshklop deleted the aj/vm-compat-triage-skill branch March 9, 2026 17:43
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.

2 participants