chore(enterprise): bump submodule pin to 08cd4dd — the ent#545 funnel read never mints - #2835
Merged
Merged
Conversation
… read never mints `origin/dev` pinned `fd25b47b`, one commit behind `enterprise-main`. The missing commit is abilityai/trinity-enterprise#545/#570: `get_activation_funnel` read through the minting `get_or_create_installation_id`, so the first admin open of Settings → Activation persisted the durable identity row from a GET. That is the private counterpart of #2669, which IS in this payload — the OSS guard pinning the caller set of the identity-minting accessors. Shipping the pointer as-is ships the guard without the fix it guards. Ordering prerequisite verified before bumping: the enterprise commit imports the OSS non-minting twin `get_installation_id`, which is present on public dev at `services/operator_intake_service.py:148` (its docstring already cites ent#545), and `tests/unit/test_2669_minting_accessor_callers.py` is in the tree. The bump is a clean fast-forward of exactly one commit. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
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
2 participants
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.
Summary
origin/devpins the enterprise submodule atfd25b47b, one commit behindenterprise-main. Found by/release-planduring the 0.9.5 pre-freeze coherence review.The missing commit is abilityai/trinity-enterprise#545/#570:
get_activation_funnelread through the mintingget_or_create_installation_id, so the first admin open of Settings → Activation persisted the durable identity row from a GET.That is the private counterpart of #2669, which is in this release payload — the OSS guard that pins the caller set of the identity-minting accessors. Shipping the pointer as-is ships the guard without the fix it guards.
Ordering prerequisite — verified before bumping
The enterprise commit imports the OSS non-minting twin
get_installation_id, and its own commit message states that symbol "must be on public dev first — this suite imports the name."services/operator_intake_service.py:148definesget_installation_idonorigin/dev(its docstring already cites ent#545)tests/unit/test_2669_minting_accessor_callers.pyis in the treefd25b47b..08cd4ddis a clean fast-forward of exactly one commit (git merge-base --is-ancestor→ true)Test Plan
fd25b47b9→08cd4dd64devbefore the bumpbuild-without-submodulemust stay green (OSS clones never check this out)deploy-devregistration boot log — the enterprise runner must register without a version-line degradationRelease context
MUST-LAND item 1 of 7 in the 0.9.5 freeze work order.
🤖 Generated with Claude Code