Skip to content

feat: add rule and tags support to fme_experiment - #1048

Merged
puthrayaharness merged 1 commit into
harness:mainfrom
MatiasCasaliSplit:fme-experiment-rule-tags-mcp
Oct 3, 2026
Merged

puthrayaharness merged 1 commit into
harness:mainfrom
MatiasCasaliSplit:fme-experiment-rule-tags-mcp

Conversation

@MatiasCasaliSplit

@MatiasCasaliSplit MatiasCasaliSplit commented Oct 2, 2026 •

Copy link
Copy Markdown
Contributor

What

Adds rule and tags support to the fme_experiment resource:

  • create: optional rule (defaults to "default" when omitted or null) and tags ([{name}]).
  • update (merge-patch): rule (null clears) and tags (replacement list; null or [] clears). Omitting either leaves it unchanged.
  • list: new tags array filter (experiments having any of the given tag names); list items now keep tags and rule in the compacted output.
  • Descriptions, body schemas and README updated.

Why

The v4 experiments API now exposes rule and tags on read, create and update, and a tags list filter. Without this, agents cannot scope an experiment to a targeting rule, manage its tags, or filter by tag.

Testing

  • pnpm vitest run tests/registry/feature-flags.test.ts: 212 passed.
  • pnpm build: passed.
  • pnpm standards:check: 79 passed.
  • pnpm docs:generate then pnpm docs:check: README up to date.
  • Full pnpm test not run.
  • Live QA over stdio against the QA backend (org anotherorg, project projectA), on an existing test experiment, restored afterwards:
    • list with tags filter: single tag and any-of ([nonexistent, existing]) return the tagged experiment; a nonexistent tag returns none. Items include tags.
    • update rule: "default": accepted and returned as "default".
    • update tags replacement, tags: [], and rule: null all behave as documented.
    • create with rule/tags was not exercised live (unit-tested only).

🤖 Generated with Claude Code

Create and update accept rule and tags; create defaults rule to "default" when omitted.
List accepts a tags filter, and list items keep tags and rule.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@puthrayaharness
puthrayaharness merged commit 87e1c1f into harness:main Oct 3, 2026
8 checks passed
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