chore(designer): Remove code interpreter experimentation flags

[Elaina-Lee]

Commit Type

• chore - Maintenance/tooling

Risk Level

• Low - Minor changes, limited scope

What & Why

Remove the code interpreter experimentation flags (enableCodeInterpreterConsumption, enableCodeInterpreterStandard) and their associated hooks. Code interpreter is now GA and should always be enabled for both consumption and standard workflows, so the feature gating is no longer needed.

Impact of Change

• Users: Code interpreter builtin tools will now always appear in the agent loop manifest for both consumption and standard SKUs, without requiring an experimentation flag.
• Developers: enableCodeInterpreterConsumption, enableCodeInterpreterStandard, useShouldEnableCodeInterpreterConsumption, and useShouldEnableCodeInterpreterStandard are removed. Any code referencing these will need to be updated (none found in the codebase).
• System: Removes async experimentation service calls from the getOperationManifest path for agent operations, simplifying the flow.

Test Plan

• Unit tests added/updated
• E2E tests added/updated
• Manual testing completed
• Removed corresponding test cases from experimentationFlags.spec.ts
• Existing tests for enableAPIMGatewayConnection remain unchanged

Contributors

Screenshots/Videos

N/A - no visual changes

🤖 Generated with Claude Code

[github-actions]

🤖 AI PR Validation Report

PR Review Results

Thank you for your submission! Here's detailed feedback on your PR title and body compliance:

✅ PR Title

• Current: chore(designer): Remove code interpreter experimentation flags
• Issue: None — title is concise and follows the conventional commit style. It clearly communicates intent and scope.
• Recommendation: (Optional) If you want to be even more explicit you can include the main package(s) changed, e.g. chore(designer, shared): Remove code interpreter experimentation flags — not required.

---

✅ Commit Type

• Properly selected (chore) and only one checked.
• Note: This matches the nature of the changes (maintenance/tooling).

---

✅ Risk Level

• The PR marks Low and the repo label risk:low is present. Based on the diff (removal of experimentation flags & related gating, small number of files changed, tests updated), I agree with Low risk.

---

✅ What & Why

• Current: "Remove the code interpreter experimentation flags (enableCodeInterpreterConsumption, enableCodeInterpreterStandard) and their associated hooks..."
• Issue: None — this is clear and concise.
• Recommendation: (Optional) You may add a short list of the specific files/areas touched (e.g. experimentationFlags, operationmanifest services, related tests) to help reviewers quickly see the surface area.

---

✅ Impact of Change

• Impact section is filled and appropriately scoped.
• Recommendation: Keep the note that "none found in the codebase" for references up-to-date in case follow-up refactors expose references elsewhere. Consider adding a line to the Impact section confirming whether any runtime configuration or release notes need updates.
  • Users: Code interpreter builtin tools will now always appear in the agent loop manifest for both consumption and standard SKUs
  • Developers: Removed functions/hooks and experimentation checks; update any references if discovered
  • System: Removes async experimentation service calls from getOperationManifest path

---

✅ Test Plan

• Test plan claims unit tests were added/updated and the diff shows test updates/removals in experiments and operation manifest tests. This matches the change and is acceptable.
• E2E tests not added — acceptable given the nature of the change, but ensure any integration/end-to-end expectations for manifests are covered elsewhere.

---

⚠️ Contributors

• Current: empty
• Assessment: Optional but recommended to acknowledge contributors (PM, designers, reviewers) if applicable. If none, leaving blank is fine.

---

✅ Screenshots/Videos

• N/A — no visual changes. Correctly marked as N/A.

---

Summary Table

Section	Status	Recommendation
Title	✅	No change needed; optionally scope more packages
Commit Type	✅	Correctly set to chore
Risk Level	✅	risk:low is reasonable based on the diff
What & Why	✅	Clear; optionally list touched files
Impact of Change	✅	Good; consider noting if release notes needed
Test Plan	✅	Unit tests updated in diff — matches claim
Contributors	⚠️	Optional: add credits if applicable
Screenshots/Videos	✅	N/A is appropriate

---

Final message
Please update the PR only if you want to add optional clarifications mentioned above (files touched in What & Why, contributor acknowledgements, or a brief release-note statement). Otherwise this PR body and title are compliant with the template and the risk assessment of Low is appropriate.

Quick checklist before merge:

• Run the full test suite (CI) to ensure no regressions.
• Search for any remaining references to the removed flags/hooks across the repo (the diff appears to have removed the references).
• If this behavior change should be communicated (customers or internal docs), add a short release note/changelog entry.

Thank you for the clear PR — it passes the title/body template checks.

---

Last updated: Mon, 13 Apr 2026 23:20:18 GMT

[github-actions]

📊 Coverage Check

The following changed files need attention:

⚠️ libs/logic-apps-shared/src/designer-client-services/lib/standard/operationmanifest.ts - 16% covered (needs improvement)

Please add tests for the uncovered files before merging.

[Copilot]

Pull request overview

Removes the code interpreter experimentation gating now that code interpreter is GA, simplifying operation manifest generation for both Consumption and Standard SKUs.

Changes:

• Removed enableCodeInterpreterConsumption / enableCodeInterpreterStandard experimentation flags and their unit tests.
• Removed Consumption/Standard manifest patching that previously stripped builtinTools when the flag was disabled.
• Removed Designer V2 React hooks that wrapped the deleted experimentation checks.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated no comments.

Show a summary per file

File	Description
libs/logic-apps-shared/src/designer-client-services/lib/standard/operationmanifest.ts	Removes standard agent manifest feature-gating logic so built-in tools are always included.
libs/logic-apps-shared/src/designer-client-services/lib/consumption/operationmanifest.ts	Removes consumption agent manifest feature-gating logic so built-in tools are always included.
libs/logic-apps-shared/src/designer-client-services/lib/experimentationFlags.ts	Deletes code interpreter flag constants and helper functions, keeping only APIM gateway flag.
libs/logic-apps-shared/src/designer-client-services/lib/test/experimentationFlags.spec.ts	Removes tests for deleted code interpreter flag helpers; retains APIM gateway flag coverage.
libs/designer-v2/src/lib/core/utils/experimentation.ts	Removes deleted code interpreter enablement hooks and unused imports.