fix: include judgeConfiguration in CompletionConfigDefault.ToLdValue()

[devin-ai-integration]

Requirements

• I have added test coverage for new or changed functionality
• I have followed the repository's pull request submission guidelines
• I have validated my changes against all supported platform versions

Related issues

Builds on PR 3 branch (mmccarthy/AIC-2636/net-sdk-v-1-0-step-3). Fixes a gap in LdAiCompletionConfigDefault.ToLdValue().

Describe the solution you've provided

LdAiCompletionConfigDefault.ToLdValue() was not serializing judgeConfiguration, unlike LdAiAgentConfigDefault.ToLdValue() which conditionally includes it. This means when a user sets SetJudgeConfiguration(...) on a completion default and the flag falls back to the default LdValue via JsonVariation, the judge config is lost on the round-trip through BuildCompletionConfig.

Fix: changed ToLdValue() to use a mutable dictionary and conditionally add judgeConfiguration when non-null — matching the agent default pattern exactly.

Added test CompletionConfigDefaultToLdValueIncludesJudgeConfiguration that mocks JsonVariation to echo back the default LdValue and verifies JudgeConfiguration survives the round-trip.

Describe alternatives you've considered

N/A — this is a direct consistency fix to match the agent default serialization pattern.

Additional context

All 119 tests pass locally on net8.0.

Link to Devin session: https://app.devin.ai/sessions/223e995121584359a2ba14c106f186da

---

Note

Low Risk
Targeted default-value serialization aligned with existing agent behavior; behavior change only affects completion defaults that set judge configuration on flag fallback.

Overview
Fixes LdAiCompletionConfigDefault.ToLdValue() so completion defaults serialize judgeConfiguration when set (same conditional pattern as LdAiAgentConfigDefault). Without this, a completion default with SetJudgeConfiguration(...) loses judges when the flag evaluation returns that default LdValue and BuildCompletionConfig parses it again.

Adds CompletionConfigDefaultToLdValueIncludesJudgeConfiguration, which mocks JsonVariation to echo the default LdValue and asserts judge key and sampling rate survive the round-trip.

^{Reviewed by Cursor Bugbot for commit 39e9f53. Bugbot is set up for automated code reviews on this repo. Configure here.}

[devin-ai-integration]

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

• Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
• Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

• Disable automatic comment, CI, and merge conflict monitoring