Skip to content

Ground flag-release plans in matched release policies (EMSR-1930) - #151

Draft
bradsimantel wants to merge 1 commit into
mainfrom
bsimantel/EMSR-1930/flag-release-policy-grounding
Draft

Ground flag-release plans in matched release policies (EMSR-1930)#151
bradsimantel wants to merge 1 commit into
mainfrom
bsimantel/EMSR-1930/flag-release-policy-grounding

Conversation

@bradsimantel

Copy link
Copy Markdown
Contributor

Summary

Closes the gaps between the flag-release skill and match-release-policies so a release plan is always grounded in the environment's configured release policy — never an invented rollout — and a missing/incomplete policy stops for clarification instead of a fabricated one.

What changed

Skills

  • flag-release (0.1.0 → 0.2.0): preview policy by flagKey when the flag exists or by proposed flagTags before it does; take methods/stages/metrics only from the tool; on a missing_policy / incomplete_policy warning, surface it and stop for clarification rather than fabricating a rollout or assuming a default; re-run matching after a refinement changes the flag, tags, or environment set; state the plan is a preview (policy re-resolves at merge).
  • flag-and-release-change (0.1.0 → 0.2.0): plan-phase policy preview uses flagTags (the flag doesn't exist yet) and surfaces warnings.
  • Updated references/auto-release.md + both READMEs; regenerated skills.json; bumped marketplace.json versions.

Evals (evals/)

  • flag-release: added missing-policy (warn-and-stop, no unbacked policy rollout) and incomplete-policy (no clean-guarded-rollout claim) cases.
  • flag-and-release-change: added a pre-creation flagTags preview asserting match-release-policies is called per planned environment via the flagTags path, and a refinement case asserting a changed environment set re-runs matching for the added environment.
  • Added a match-release-policies mock hook (_mock.js) driving missing/incomplete/progressive scenarios by project key, warnings: [] on the default mock, and the warnings contract in the tool definition.

Testing

  • scripts/validate_skills.py ✅ (50 skills), scripts/generate_catalog.py
  • Both promptfoo configs parse; new assertions inspect the tool-call trajectory (tool actually called, correct flagKey/flagTags path, per-environment coverage). Behavioral evals run in CI.

EMSR-1930

via LD Research 🤖

…d stop on gaps

Close the gaps between the flag-release skill and match-release-policies so
Factory (and any harness composing this skill) presents a release plan grounded
in the configured release policies instead of an invented rollout:

- flag-release SKILL: preview by flagKey when the flag exists OR by proposed
  flagTags before it does; take methods/stages/metrics only from the tool; on a
  missing_policy / incomplete_policy warning, surface it and STOP for
  clarification rather than fabricating a rollout or assuming a default; re-run
  matching after a refinement changes the flag, tags, or environment set. State
  the plan is a preview (policy re-resolves at merge).
- flag-and-release-change: preview policy by flagTags in the plan phase (before
  the flag exists); surface warnings.
- auto-release reference + READMEs updated; both skills bumped to 0.2.0;
  regenerated skills.json.
- evals: assert match-release-policies is called per planned environment and via
  the flagTags path pre-creation; missing/incomplete policies produce a warning
  and no clean-release claim; a changed environment set re-runs matching. Added a
  match-release-policies mock hook (missing/incomplete/progressive scenarios) and
  the warnings contract to the tool definition.

EMSR-1930
@github-actions

Copy link
Copy Markdown

Skill eval results

Skill Before After Δ
agentcontrol/configs-create 100/100 (4/4) 100/100 (4/4) no change
agentcontrol/configs-update 80/100 (4/5) 100/100 (5/5) +20
agentcontrol/configs-variations 80/100 (4/5) 80/100 (4/5) no change
agentcontrol/tools 75/100 (3/4) 75/100 (3/4) no change
feature-flags/flag-and-release-change - 100/100 (6/6) new
feature-flags/flag-release - 100/100 (7/7) new
feature-flags/launchdarkly-flag-command - 100/100 (3/3) new
feature-flags/launchdarkly-flag-create 100/100 (3/3) 100/100 (4/4) no change
feature-flags/launchdarkly-flag-drift - 100/100 (4/4) new
feature-flags/should-flag-change - 100/100 (17/17) new

Only suites whose source actually changed since their last recorded score were re-run. Soft-failing while we stabilise the baseline.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants