Skip to content

test(evals): update the pinned Prismic skill ref - #292

Merged
angeloashmore merged 1 commit into
mainfrom
claude/pin-prismic-skill-tw2l2a
Sep 10, 2026
Merged

test(evals): update the pinned Prismic skill ref#292
angeloashmore merged 1 commit into
mainfrom
claude/pin-prismic-skill-tw2l2a

Conversation

@angeloashmore

@angeloashmore angeloashmore commented Sep 8, 2026

Copy link
Copy Markdown
Member

Description

Updates the Prismic skill ref for evals.

Checklist

  • If my changes require tests, I added them.
  • If my changes affect backward compatibility, it has been discussed.
  • If my changes require an update to the CONTRIBUTING.md guide, I updated it.

Preview

How to QA 1

🤖 Generated with Claude Code

https://claude.ai/code/session_01EDG5bb3gLsNgvJZR3bANY3


Note

Low Risk
Single constant change in isolated eval setup; no auth, data, or shipping code paths.

Overview
Updates the git commit pin used by integration evals when downloading the Prismic agent skill from prismicio/skills.

The constant PRISMIC_SKILL_REF in evals/it.ts changes from 5028eeae… to b458cc6f…, so fetchSkill() pulls the newer SKILL.md for Claude/Codex trial runs. No production CLI or runtime behavior is affected—only eval harness instructions.

Reviewed by Cursor Bugbot for commit bb5038c. Bugbot is set up for automated code reviews on this repo. Configure here.

Footnotes

  1. Please use these labels when submitting a review:
    ❓ #ask: Ask a question.
    💡 #idea: Suggest an idea.
    ⚠️ #issue: Strongly suggest a change.
    🎉 #nice: Share a compliment.

@angeloashmore
angeloashmore force-pushed the claude/pin-prismic-skill-tw2l2a branch from 9240f13 to e1ae8f5 Compare September 8, 2026 13:52
@angeloashmore angeloashmore changed the title test(evals): pin the Prismic skill that authorizes commands test(evals): update the pinned Prismic skill ref Sep 8, 2026
The eval harness reads the Prismic skill from a commit pinned in
evals/it.ts. The pin was behind the skill repository, so runs did not
see the latest rules.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01EDG5bb3gLsNgvJZR3bANY3
@angeloashmore
angeloashmore force-pushed the claude/pin-prismic-skill-tw2l2a branch from e1ae8f5 to bb5038c Compare September 10, 2026 14:21
@angeloashmore
angeloashmore marked this pull request as ready for review September 10, 2026 14:26
@angeloashmore
angeloashmore merged commit 5a97d43 into main Sep 10, 2026
14 of 15 checks passed
@angeloashmore
angeloashmore deleted the claude/pin-prismic-skill-tw2l2a branch September 10, 2026 14:26
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.

3 participants