Skip to content

Add a configurable web_fetch ability with policy enforcement and action logging#36

Merged
bradvin merged 2 commits into
developfrom
feature/web-fetch-ability
Mar 29, 2026
Merged

Add a configurable web_fetch ability with policy enforcement and action logging#36
bradvin merged 2 commits into
developfrom
feature/web-fetch-ability

Conversation

@bradvin
Copy link
Copy Markdown
Owner

@bradvin bradvin commented Mar 29, 2026

Summary

  • add a new web_fetch ability with a default wp fetcher built on WordPress HTTP APIs
  • validate request URLs with wp_http_validate_url() and keep v1 read-only with GET/HEAD
  • enforce allow_network for network-capable abilities and log all web_fetch attempts through the action log helper
  • add the fetcher registry/helper and unit coverage for success, validation, policy, truncation, and failure cases
  • update docs, README, and generated assets to reflect the new ability and related runtime changes

Testing

  • composer test
  • npm run lint:php

Copy link
Copy Markdown

@greptile-apps greptile-apps Bot left a comment

Choose a reason for hiding this comment

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

Your free trial has ended. If you'd like to continue receiving code reviews, you can add a payment method here.

@bradvin bradvin changed the base branch from main to develop March 29, 2026 13:32
@bradvin bradvin self-assigned this Mar 29, 2026
@bradvin bradvin merged commit 62b50cb into develop Mar 29, 2026
2 of 3 checks passed
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.

1 participant