test(provin): pin Provin at its auth-baseline bump (develop abb14d9) - #30
Merged
Merged
Conversation
provin-line/auth#23 moved every Provin package to ^0.8.1 / ^0.12.0 — the same revisions this Makefile pins as VERIFIER_REV / PROVIDER_REV — and removed the 0.3.x compat shim. provin-compatibility now builds a consumer whose declared ranges agree with the candidate tarballs it is overridden with; before this, the override was doing the agreeing for it. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
There was a problem hiding this comment.
🟢 Approval recommended
The change is a straightforward revision pin update and is consistently referenced in the existing Make targets.
Pull request overview
Pins the Provin E2E dependency in this repo’s test harness to the provin-line/auth merge commit referenced in the PR description, keeping the integration test setup aligned with the Provin auth-baseline bump.
Changes:
- Bump
PROVIN_REVin the Makefile toabb14d9ff971e88261447dd255863301f22c3a7c.
File summaries
| File | Description |
|---|---|
| Makefile | Updates the pinned Provin repo revision used by setup-provin / test-provin. |
Review details
- Files reviewed: 1/1 changed files
- Comments generated: 0
- Review effort level: Lite
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Bumps
PROVIN_REVto provin-line/auth#23's merge commitabb14d9: every Provin package now requires^0.8.1/^0.12.0— the same revisions this Makefile pins asVERIFIER_REV/PROVIDER_REV— and the 0.3.x compat shim (collectors/context.mts, thebuiltinKeyResolversModulenamespace probe, theJWT_MODE_MIGRATION_MESSAGEskipIf) is gone.What
provin-compatibilityproves on this run that it could not before: the consumer's declared ranges agree with the candidate tarballs it is overridden with. Previously the override was doing the agreeing on Provin's behalf while itspackage.jsonstill said^0.3.1/^0.5.3.PROVIDER_REV,VERIFIER_REV,PROXY_REVunchanged (v0.12.0 / v0.8.1 / v0.5.1).🤖 Generated with Claude Code