Skip to content

Ensure the Type Ahead experiment supports provider/model overrides and Guidelines#776

Merged
dkotter merged 5 commits into
WordPress:developfrom
dkotter:fix/type-ahead-missing-integrations
Jun 26, 2026
Merged

Ensure the Type Ahead experiment supports provider/model overrides and Guidelines#776
dkotter merged 5 commits into
WordPress:developfrom
dkotter:fix/type-ahead-missing-integrations

Conversation

@dkotter

@dkotter dkotter commented Jun 25, 2026

Copy link
Copy Markdown
Contributor

What?

Ensure the Type Ahead experiment supports the ability to override the Provider and Model used from the developer settings and ensure any related Guidelines are pulled into the system instructions.

Why?

Recently merged in #151 but realized after the fact that there are a few new things we've implemented in other experiments that didn't exist when that PR was worked on, namely the ability to set the Provider and Model from our developer settings and integration with Guidelines. This PR updates both.

How?

  • Use the set_provider_model_preference method to ensure our Provider and Model can be filtered by our developer settings
  • Use the guideline_categories to ensure Guidelines are pulled in to the system instructions

Use of AI Tools

None

Testing Instructions

Same testing instructions from #151 for the most part, just ensuring the Type Ahead experiment still works

Changelog Entry

n/a, will be pulled from #151

Open WordPress Playground Preview
@dkotter dkotter added this to the 1.1.0 milestone Jun 25, 2026
@dkotter dkotter self-assigned this Jun 25, 2026
@github-actions

Copy link
Copy Markdown

The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the props-bot label.

If you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message.

Co-authored-by: dkotter <dkotter@git.wordpress.org>

To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook.

@codecov

codecov Bot commented Jun 25, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 22.22222% with 7 lines in your changes missing coverage. Please review.
✅ Project coverage is 76.48%. Comparing base (56cef56) to head (20af03f).
⚠️ Report is 1 commits behind head on develop.

Files with missing lines Patch % Lines
includes/Abilities/Type_Ahead/Type_Ahead.php 22.22% 7 Missing ⚠️
Additional details and impacted files
@@              Coverage Diff              @@
##             develop     #776      +/-   ##
=============================================
- Coverage      76.50%   76.48%   -0.03%     
- Complexity      1868     1869       +1     
=============================================
  Files             89       89              
  Lines           7985     7990       +5     
=============================================
+ Hits            6109     6111       +2     
- Misses          1876     1879       +3     
Flag Coverage Δ
unit 76.48% <22.22%> (-0.03%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.
@dkotter dkotter requested a review from jeffpaul June 25, 2026 19:25
@jeffpaul jeffpaul moved this from Triage to Needs review in WordPress AI Roadmap Jun 26, 2026

@jeffpaul jeffpaul left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Provider/model selection appeared as expected, and adding in copy guidelines adjusted the type ahead text as expected

@dkotter dkotter merged commit 5316623 into WordPress:develop Jun 26, 2026
22 of 24 checks passed
@github-project-automation github-project-automation Bot moved this from Needs review to Done in WordPress AI Roadmap Jun 26, 2026
@dkotter dkotter deleted the fix/type-ahead-missing-integrations branch June 26, 2026 17:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants