build(deps): bump actions/cache from 4 to 6 - #2
Conversation
How to use the Graphite Merge QueueAdd either label to this PR to merge it via the merge queue:
You must have a Graphite account in order to use the merge queue. Sign up using this link. An organization admin has enabled the Graphite Merge Queue in this repository. Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue. |
Gated (private desktop repo) needs multi-key trust-ring lookup and revocation for license verification, but under the Phase 2 submodule model it consumes Trackdub.Licensing read-only and can no longer fork LicenseTokenValidator/LicenseService in place to get it, per docs/plans/trackdub-gated-split-manifest.md (Trackdub-gated PR #2) §1. Adds ILicenseSignatureTrustStore: resolves a PEM public key per token key id, or null to reject an unknown/revoked key. LicenseService takes it as a new optional constructor parameter; omitting it preserves the existing single embedded-key behavior exactly. LicenseTokenClaims gains an optional KeyId parsed from the token's "kid" claim and threaded into LicenseTokenValidator.VerifySignature, which now resolves the key per-store when one is supplied instead of always using the embedded key. Revocation falls out of the same seam: a trust store returning null for a given key id fails verification closed, so no separate revocation plumbing is needed. Everything added is internal to Trackdub.Licensing except the new public interface and the optional constructor parameter, so the project keeps its zero-ProjectReference, BCL-only-crypto invariants (LicensingIsolationTests) and the dependency graph in AGENTS.md is unchanged. Production-policy concerns from the same manifest section -- rejecting dev-unlimited tokens under a production trust ring, and startup validation of production ring configuration -- are deliberately left out of this seam. Both read from already-public LicenseValidationResult fields (UnlimitedActivations, degradation reason) and are naturally implementable as a decorator over ILicenseInitializer/ILicenseTierProvider in the consuming product, without needing any further core change. The manifest's other blocker, headless export-tier enforcement, needs no core change at all: Trackdub.Sdk's TrackdubBuilder.ConfigureServices (-> TrackdubOptions/HeadlessTrackdubOptions.ServiceConfigurator -> HeadlessCompositionRoot.AddHeadlessTrackdub step 7) already lets a consumer register a custom IExportTierGate for Cli/Sdk hosts. Trackdub.Cli goes through this same builder. That seam already exists; using it is Phase 2.3 work in the desktop repo, not something this PR needs to add. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_014Yd883rGkFx5xN1TyT1Lzf
tonythethompson
left a comment
There was a problem hiding this comment.
Push an empty commit to this PR so that CI tests restart
tonythethompson
left a comment
There was a problem hiding this comment.
@dependabot rebase.
Bumps [actions/cache](https://github.com/actions/cache) from 4 to 6. - [Release notes](https://github.com/actions/cache/releases) - [Changelog](https://github.com/actions/cache/blob/main/RELEASES.md) - [Commits](actions/cache@v4...v6) --- updated-dependencies: - dependency-name: actions/cache dependency-version: '6' dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com>
0823b12 to
a1c883c
Compare
|
This is a major version update. Auto-merge is disabled for major bumps. Please review manually. |
Code Review SummaryStatus: No Issues Found | Recommendation: Merge Files Reviewed (1 file)
Reviewed by step-3.7-flash · Input: 80.7K · Output: 3K · Cached: 143.6K |
|
This is a major version update. Auto-merge is disabled for major bumps. Please review manually. |
Bumps actions/cache from 4 to 6.
Release notes
Sourced from actions/cache's releases.
... (truncated)
Changelog
Sourced from actions/cache's changelog.
... (truncated)
Commits
55cc834Merge pull request #1768 from jasongin/readonly-cached8cd72fBump@actions/cacheto v6.1.0 - handle cache write error due to RO token2c8a9bdMerge pull request #1760 from actions/samirat/esm_migration_and_package_updatee9b91fdPrettier fixese4884b8Rebuild dist10baf01Fixed licensese39b386Fix test mock return orderb692820PR feedback6074912Rebuild dist bundles as ESM to match type:module5a912e8Fix lint and jest issues