[issue #660] feat: Update generalised error message for all features by SohamPatel46 · Pull Request #759 · WordPress/ai

SohamPatel46 · 2026-06-22T17:52:48Z

What?

Closes #660

Implements a centralized, context-aware error message mapping in the WordPress REST API when AI requests (e.g. "Generate Title", "Generate Excerpt") are blocked by the Connector Approvals feature.

Why?

Previously, when the Connector Approvals feature blocked the caller (wp-ai), the user was shown a generic error notice: "Title generation failed. Please ensure you have a connected provider that supports text generation." This led users to believe that their API keys or connected providers were broken when the feature was actually just awaiting approval in the WordPress admin panel.

Using a centralized server-side approach via a REST dispatch filter ensures that all features display clean, translated, and context-aware instructions explaining how to authorize the connector under Tools > Connector Approvals, without duplicating approval-checking code across individual feature controllers or frontend JS files.

How?

REST Error Customizer:
- Registered the rest_post_dispatch filter inside the Connector_Approval experiment class.
- If a REST request fails with the wpai_connector_not_approved code, it intercepts the error, parses the Ability ID from the route path, and maps it to a context-aware prefix (e.g. "Title generation failed.", "Excerpt generation failed.").
- Appends: "The AI connector is currently pending authorization. Please approve the request under Tools > Connector Approvals."
Capability Check Fallback:
- Updated ensure_text_generation_supported() and ensure_image_generation_supported() in Abstract_Ability.
- If the remote model fetch requests get blocked by the approvals check during capability check, it now falls back to return wpai_connector_not_approved instead of the generic unsupported_model error.

Use of AI Tools

AI assistance: Yes
Tool(s) used: Antigravity IDE.
Model(s) used: Gemini 3.5 Flash
Used for: Suggestion for designing the centralized server-side solution, code refactoring, test case additions. Implementation was reviewed, modified and tested by me.

Testing Instructions

Install the AI plugin and configure an AI provider (e.g., Google Connector), but do not approve the caller under Tools > Connector Approvals yet.
Open a page/post and click Generate Title.
Verify that the notice displays: "Title generation failed. The AI connector is currently pending authorization. Please approve the request under Tools > Connector Approvals."
Click Generate Excerpt (if excerpt generation experiment is active) and verify it displays: "Excerpt generation failed. The AI connector is currently pending authorization. Please approve the request under Tools > Connector Approvals."
Go to Tools > Connector Approvals and approve the connector.
Return to the post editor and verify that title and excerpt generation now succeed without the error.

Screenshots or screencast

Before	After

Changelog Entry

Fixed - Display a clear, context-aware error notice during Ability execution when the configured AI connector is pending administrator approval.

Added - New feature.
Changed - Existing functionality.
Deprecated - Soon-to-be removed feature.
Removed - Feature.
Fixed - Bug fix.
Security - Vulnerability.
Developer - Development related updates.

github-actions · 2026-06-22T17:52:59Z

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: SohamPatel46 <sohampate1@git.wordpress.org>
Co-authored-by: jeffpaul <jeffpaul@git.wordpress.org>
Co-authored-by: dkotter <dkotter@git.wordpress.org>
Co-authored-by: riccardodicurti <riccardodicurti@git.wordpress.org>

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

codecov · 2026-06-22T17:59:08Z

Codecov Report

❌ Patch coverage is 97.61905% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 75.79%. Comparing base (9db55ee) to head (3cd5918).

Files with missing lines	Patch %	Lines
includes/Abstracts/Abstract_Ability.php	92.30%	2 Missing ⚠️

Additional details and impacted files

@@              Coverage Diff              @@
##             develop     #759      +/-   ##
=============================================
+ Coverage      75.50%   75.79%   +0.29%     
- Complexity      2086     2117      +31     
=============================================
  Files             99       99              
  Lines           8626     8710      +84     
=============================================
+ Hits            6513     6602      +89     
+ Misses          2113     2108       -5

Flag	Coverage Δ
unit	`75.79% <97.61%> (+0.29%)`	⬆️

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 · 2026-07-09T17:16:36Z

+	/**
+	 * Filters the REST response to customize the error message when a request is blocked by Connector Approval.
+	 *
+	 * @since 1.1.0


These still haven't been updated. I'm not going to flag each one but all of them need updated

dkotter · 2026-07-09T17:18:16Z

+	 * @param \WP_REST_Request  $request  The REST request.
+	 * @return mixed The modified REST response.
+	 */
+	public function customize_rest_error( $response, $server, $request ) {


So I don't necessarily have a better approach off the top of my head but not sure I love us hooking into rest_post_dispatch and then running all of this functionality. Seems this is likely to impact things we don't want to impact (slowing down other, non-AI requests as an example). Wondering if there's a better approach to handle this?

dkotter · 2026-07-09T17:18:54Z

+	 */
+	private function get_context_aware_error_message( string $ability_id ): string {
+		switch ( $ability_id ) {
+			case 'ai/title-generation':


Ideally we aren't hardcoding all of these here as we'll need to remember to update this anytime a new feature is added. It would be great if this could be more dynamic

dkotter · 2026-07-09T17:19:22Z

In addition to these integration tests, we should add some E2E tests that verify the error messages show as expected

dkotter · 2026-07-09T17:19:47Z

+	 * @param string $message The fallback error message.
+	 * @return \WP_Error|null A WP_Error if an unapproved connector is found, null otherwise.
+	 */
+	private function maybe_get_unapproved_connector_error( string $message ) {


I'm wondering if this belongs on the Abstract_Ability class? Or if this should be in the connector approval code?

jeffpaul · 2026-08-10T17:51:24Z

@SohamPatel46 I'm seeing lots of test failures here, mind reviewing those to see if any are impacted by the changes in this PR?

feat: Update generalised error message for all features

2412e4c

SohamPatel46 mentioned this pull request Jun 22, 2026

UX: Ambiguous error message in editor when a provider is blocked by Connector Approvals #660

Open

SohamPatel46 added 2 commits June 22, 2026 23:37

Merge branch 'develop' into feature/issue-660

6b974de

fix: Update php unit tests coverage

c15abad

dkotter assigned SohamPatel46 Jun 23, 2026

dkotter requested changes Jun 26, 2026

View reviewed changes

Comment thread includes/Abstracts/Abstract_Ability.php Outdated

jeffpaul added this to the 1.2.0 milestone Jun 26, 2026

SohamPatel46 added 2 commits June 30, 2026 16:50

docs: update since statement to follow standard practices

b7f63b4

Merge branch 'develop' into feature/issue-660

adb853e

SohamPatel46 requested a review from dkotter June 30, 2026 11:20

Merge branch 'WordPress:develop' into feature/issue-660

3cd5918

SohamPatel46 requested a review from a team July 2, 2026 13:21

dkotter requested changes Jul 9, 2026

View reviewed changes

SohamPatel46 added 3 commits July 10, 2026 22:10

Address PR feedbacks

a838775

Merge branch 'develop' into feature/issue-660

7f8d75e

Fix: e2e tests and unit tests

0f9def1

jeffpaul modified the milestones: 1.2.0, 1.3.0 Jul 13, 2026

Merge branch 'develop' into feature/issue-660

6b48734

dkotter mentioned this pull request Jul 15, 2026

[issue #660] Add context-aware error message in Block Editor #686

Closed

jeffpaul mentioned this pull request Aug 10, 2026

Release version 1.3.0 #924

Closed

48 tasks

dkotter modified the milestones: 1.3.0, 1.4.0 Aug 12, 2026

Navigation Menu

Uh oh!

[issue #660] feat: Update generalised error message for all features - #759

[issue #660] feat: Update generalised error message for all features#759
SohamPatel46 wants to merge 10 commits into
WordPress:developfrom
SohamPatel46:feature/issue-660

SohamPatel46 commented Jun 22, 2026 •

edited by github-actions Bot

Loading

Uh oh!

github-actions Bot commented Jun 22, 2026 •

edited

Loading

Uh oh!

codecov Bot commented Jun 22, 2026 •

edited

Loading

Uh oh!

Uh oh!

dkotter Jul 9, 2026

Uh oh!

dkotter Jul 9, 2026

Uh oh!

dkotter Jul 9, 2026

Uh oh!

dkotter Jul 9, 2026

Uh oh!

dkotter Jul 9, 2026

Uh oh!

jeffpaul commented Aug 10, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Uh oh!

Conversation

SohamPatel46 commented Jun 22, 2026 • edited by github-actions Bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

What?

Why?

How?

Use of AI Tools

Testing Instructions

Screenshots or screencast

Changelog Entry

Uh oh!

github-actions Bot commented Jun 22, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

codecov Bot commented Jun 22, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Codecov Report

Uh oh!

Uh oh!

dkotter Jul 9, 2026

Choose a reason for hiding this comment

Uh oh!

dkotter Jul 9, 2026

Choose a reason for hiding this comment

Uh oh!

dkotter Jul 9, 2026

Choose a reason for hiding this comment

Uh oh!

dkotter Jul 9, 2026

Choose a reason for hiding this comment

Uh oh!

dkotter Jul 9, 2026

Choose a reason for hiding this comment

Uh oh!

jeffpaul commented Aug 10, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

SohamPatel46 commented Jun 22, 2026 •

edited by github-actions Bot

Loading

github-actions Bot commented Jun 22, 2026 •

edited

Loading

codecov Bot commented Jun 22, 2026 •

edited

Loading