Skip to content

Vendor opik-verify: bump CANON_REF to opik-mcp main after #200 - #33

Merged
aswynz merged 1 commit into
mainfrom
aswynz/OPIK-7470/vendor-opik-verify
Sep 25, 2026
Merged

aswynz merged 1 commit into
mainfrom
aswynz/OPIK-7470/vendor-opik-verify

Conversation

@aswynz

@aswynz aswynz commented Sep 21, 2026

Copy link
Copy Markdown
Contributor

Details

Ten skills now, not nine. opik-verify (the ship/hold verdict, opik-mcp#193) landed along with the eval fixtures and the corrected skill text (opik-mcp#200), so the pin moves 0baa5ae → 8fb9c4b and opik-verify joins SHARED.

Re-vendoring at the new pin also picks up the review-driven text fixes that shipped with those PRs:

  • generate_report=False in opik-compare and opik-evaluate (run_tests otherwise writes opik_test_suite_reports/ into the user's cwd).
  • the graded-metric note in opik-optimize.
  • compatibility / allowed-tools / a pinned source_commit on opik.

README skills table gains the verify row; plugin.json 0.4.0 → 0.5.0.

Change checklist

  • User facing
  • Documentation updated (README skills table)
  • Tests added/updated (if needed) — the drift job is the test
  • Breaking changes documented (if any) — none

Testing

bash scripts/sync-shared-skills.sh at the new pin is a no-op after this commit, which is what the drift job asserts. Same bytes as the published opik-skills pack at 8fb9c4b (its index.json names the same commit and the same ten skills).

Ten skills now, not nine. opik-verify (the ship/hold verdict, opik-mcp#193)
landed along with the eval fixtures and the corrected skill text (#200), so
the pin moves from 0baa5ae to 8fb9c4b and `opik-verify` joins SHARED.

Re-vendoring also picks up the review-driven text fixes that shipped with
those PRs: `generate_report=False` in opik-compare and opik-evaluate, the
graded-metric note in opik-optimize, and compatibility / allowed-tools /
a pinned source_commit on opik.

README skills table gains the verify row; plugin.json 0.4.0 -> 0.5.0.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
@aswynz
aswynz requested a review from awkoy September 21, 2026 21:18
@aswynz
aswynz merged commit d35fdb4 into main Sep 25, 2026
1 check passed
@aswynz
aswynz deleted the aswynz/OPIK-7470/vendor-opik-verify branch September 25, 2026 19:06
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