Skip to content

Fix wp-ai-client timeout semantics#1817

Merged
chubes4 merged 2 commits intomainfrom
fix/ai-timeout-semantics
May 6, 2026
Merged

Fix wp-ai-client timeout semantics#1817
chubes4 merged 2 commits intomainfrom
fix/ai-timeout-semantics

Conversation

@chubes4
Copy link
Copy Markdown
Member

@chubes4 chubes4 commented May 6, 2026

Summary

  • Adds a first-class wp_ai_client_request_timeout setting alongside the existing connect timeout setting.
  • Separates defaults to a short connect timeout (15s) and longer request timeout (300s) while preserving the existing Data Machine filters.
  • Logs and exposes the resolved AI transport profile, including request/connect timeouts, RequestOptions usage, cURL hook state, job, flow step, provider, model, and mode.

Testing

  • php tests/wp-ai-client-request-timeout-smoke.php
  • php -l inc/Engine/AI/RequestBuilder.php
  • php -l inc/Abilities/SettingsAbilities.php
  • php -l inc/Core/PluginSettings.php
  • php -l inc/Engine/AI/RequestInspector.php
  • php -l tests/wp-ai-client-request-timeout-smoke.php
  • php -l tests/Unit/Abilities/SettingsAbilitiesTest.php
  • git diff --check
  • npm run build
  • homeboy test data-machine

Caveats

  • homeboy lint data-machine still reports the existing repository-wide admin React lint baseline (867 findings), so it is not a clean gate for this branch.
  • Existing saved wp_ai_client_connect_timeout values are preserved; operators can now set request timeout separately.

Fixes #1816

AI assistance

  • AI assistance: Yes
  • Tool(s): OpenCode (openai/gpt-5.5)
  • Used for: Separating AI connect/request timeout settings, adding transport observability, and updating tests.
    Chris remains responsible for review and merge.

@homeboy-ci
Copy link
Copy Markdown
Contributor

homeboy-ci Bot commented May 6, 2026

Homeboy Results — data-machine

Lint

lint — passed

ℹ️ Full options: homeboy docs commands/lint
Deep dive: homeboy lint data-machine --changed-since 77589dc

Test

test — passed

  • 962 passed
  • 3 skipped

ℹ️ Auto-fix lint issues: homeboy refactor data-machine --from lint --write
ℹ️ Collect coverage: homeboy test data-machine --coverage
ℹ️ Save test baseline: homeboy test data-machine --baseline
ℹ️ Pass args to test runner: homeboy test -- [args]
ℹ️ Full options: homeboy docs commands/test
Deep dive: homeboy test data-machine --changed-since 77589dc

Audit

audit — passed

  • test_coverage — 57 finding(s)
  • dead_code — 46 finding(s)
  • intra-method-duplication — 14 finding(s)
  • parallel-implementation — 3 finding(s)
  • dead_guard — 2 finding(s)
  • repeated_literal_shape — 2 finding(s)
  • Content — 1 finding(s)
  • Directives — 1 finding(s)
  • Retention — 1 finding(s)
  • Tasks — 1 finding(s)
  • Total: 128 finding(s)

Deep dive: homeboy audit data-machine --changed-since 77589dc

Tooling versions
  • Homeboy CLI: homeboy 0.158.0+a3108c1f
  • Extension: wordpress from https://github.com/Extra-Chill/homeboy-extensions
  • Extension revision: de03692
  • Action: Extra-Chill/homeboy-action@v2

@chubes4 chubes4 merged commit f90ee70 into main May 6, 2026
3 checks passed
@chubes4 chubes4 deleted the fix/ai-timeout-semantics branch May 6, 2026 23:21
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.

Separate AI connect/request timeouts and log resolved transport profile

1 participant