feat(registry): resolve + load registry assets into tasks (#246) - #4
Draft
Kalindi-Dev wants to merge 1 commit into
Draft
feat(registry): resolve + load registry assets into tasks (#246)#4Kalindi-Dev wants to merge 1 commit into
Kalindi-Dev wants to merge 1 commit into
Conversation
…s#246) Builds on the catalog PR to actually consume registry assets at task time: - Orchestrator resolve-step (`resolveRegistryAssets`): resolves a blueprint's `registry://` mcp_server / cedar_policy_module / skill refs at task start, fail-closed; stamps the `{kind,id,version}` triples on the TaskRecord for audit, merges resolved cedar_text into `cedar_policies`, and threads the runtime bundle into the agent payload. - Blueprint asset props + onUpdate fix: `assets.{mcpServers,cedarPolicyModules, skills}` with `RegistryRefValidation`; the three onUpdate helpers now write the asset-ref columns so redeploying an onboarded repo no longer drops them. - Agent loaders (registry.loader): mcp_server merges into `.mcp.json`; cedar_policy_module flows through PolicyEngine's unannotated `extra_policies`; skill prompt fragments append to the system prompt (build_skill_prompt_fragment). - TaskOrchestrator IAM: read-only bedrock-agentcore registry access so the orchestrator can resolve refs. Depends on the catalog PR (feat/246-registry-catalog): imports the RegistryClient port, ref grammar, and resolver from that branch.
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
1 participant
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
Builds on the catalog PR to consume registry assets at task time.
resolveRegistryAssets): resolves a blueprint'sregistry://mcp_server/cedar_policy_module/skill refs at task start, fail-closed; stamps{kind,id,version}triples on the TaskRecord, merges resolved cedar_text intocedar_policies, threads the runtime bundle into the agent payload.assets.{mcpServers,cedarPolicyModules,skills}withRegistryRefValidation; the three onUpdate helpers now write the asset-ref columns so redeploying an onboarded repo no longer drops them.registry.loader): mcp_server →.mcp.json; cedar_policy_module → PolicyEngine unannotatedextra_policies; skill prompt fragments → system prompt.Test plan
mise run buildgreen (2595 tests total)--context forkBlueprintRepo=owner/repo; submit a task that pins all three asset kinds01KYJ5FT0RDZJ72J7R5BKCDM38)