Skip to content

test: cover release-management dispatch, rollout prompt, and IaCM provider guards - #849

Draft
cursor[bot] wants to merge 1 commit into
mainfrom
cursor/missing-test-coverage-c432
Draft

test: cover release-management dispatch, rollout prompt, and IaCM provider guards#849
cursor[bot] wants to merge 1 commit into
mainfrom
cursor/missing-test-coverage-c432

Conversation

@cursor

@cursor cursor Bot commented Aug 21, 2026

Copy link
Copy Markdown
Contributor

Description

Adds regression tests for recently merged high-risk paths that lacked dispatch-level or prompt-level coverage.

Risky behavior now covered

Test files added/updated

  • tests/prompts/feature-flag-rollout.test.ts (new)
  • tests/registry/toolsets/release-management.test.ts
  • tests/registry/iacm.test.ts

Why these tests materially reduce regression risk

Release Management is default-on with header-scoped routing and body-carried scope — metadata-only tests missed dispatch wiring bugs that would silently hit the wrong gateway or leak scope params. The rollout prompt drives multi-step FME agent workflows; without tests, dual-mode scope migration could ship broken guidance. IaCM provider version writes are medium-risk infra mutations where missing-field errors must fail before network calls.

Small production fix

Append nativeModeCaveat to the feature-flag-rollout prompt text — the variable was defined in #806 but never included in the rendered message.

Type of Change

  • Bug fix (prompt caveat append)
  • New feature
  • Refactor
  • Documentation
  • Other (test coverage)

Checklist

  • Targeted tests pass (144 tests across 3 files)
  • pnpm test passes
  • pnpm typecheck passes
  • pnpm build passes
  • pnpm standards:check passes
  • pnpm docs:check passes (no registry/tool count changes)
Open in Web View Automation 

…vider guards

Add regression tests for recently merged high-risk paths:
- Release Management registry dispatch (RMG gateway, header scoping, scope body)
- RMG path-builder guards and list/activity extractor edge cases
- feature-flag-rollout prompt scope validation and native-mode caveat
- IaCM provider create/update preflight rejection before API calls

Fix feature-flag-rollout prompt to append the nativeModeCaveat that was
defined but never included in the rendered prompt text.

Co-authored-by: Rohan Gupta <thisrohangupta@users.noreply.github.com>
@CLAassistant

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

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