Add cross-agent policy federation#16
Conversation
nagasatish007
left a comment
There was a problem hiding this comment.
Hey @lleonardo-franco — this has merge conflicts with main (from the hot-reload PR #15 that just merged). Could you rebase your branch on main and resolve the conflicts? Both features touch TealEngine.ts in the same areas.
Key merge point: updatePolicies() should use your federation merge + the new applyPolicyReload() from hot-reload. Something like:
public updatePolicies(policies: TealPolicy): void {
const nextPolicies = this.federationConstraints
? PolicyFederation.mergePolicies(policies, this.federationConstraints)
: policies;
const result = this.applyPolicyReload(nextPolicies);
if (!result.success) {
throw new Error(TealEngine: Invalid policy configuration: ${result.error});
}
this.localPolicies = policies;
}
Once rebased and tests pass, this is approved to merge. Thanks!
366032c to
652a36f
Compare
|
Rebased this branch onto Validated with:
Could you please take another look? |
Resolves agentguard-ai/tealtiger#123.
Summary:
Validation:
Note: