Not really blocking - the format string parameters 'policy_id' and 'trigger' are a little confusing, since I was expecting that to actually be the policy id and trigger.  Would it make sense to use the same convention we do for all the other strings, and have the format string be ":{name}policyId" and ":{name}trigger" or something instead?