Skip to content

feat: add assignment_origin to ResolvedFlag#472

Open
nicklasl wants to merge 10 commits into
mainfrom
nicklasl/assignment-origin
Open

feat: add assignment_origin to ResolvedFlag#472
nicklasl wants to merge 10 commits into
mainfrom
nicklasl/assignment-origin

Conversation

@nicklasl

@nicklasl nicklasl commented Jul 3, 2026

Copy link
Copy Markdown
Member

Summary

  • Adds assignment_origin field to ResolvedFlag proto across all proto directories
  • Populates it with the matched rule name in the Rust resolver
  • Threads assignmentOrigin through the JS provider's publishFlagEvaluation to window.__confidence.flags
  • Regenerates Go proto bindings
  • Java picks up the field at build time (proto source updated, no checked-in generated code)

Test plan

  • Rust resolver tests pass (370/370)
  • WASM guest compiles
  • New assignment_origin_is_matched_rule_name spec test
  • JS test fixtures updated for required proto field
  • flag-evaluation-global tests updated for assignmentOrigin

🤖 Generated with Claude Code

@nicklasl nicklasl marked this pull request as ready for review July 9, 2026 08:14
nicklasl and others added 10 commits July 9, 2026 10:15
Surfaces the matched rule name on resolved flags.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@nicklasl nicklasl force-pushed the nicklasl/assignment-origin branch from ec0a870 to fd0ba99 Compare July 9, 2026 08:19
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.

1 participant