Skip to content

refactor: remove unused Comment functionality#13

Open
xavierandueza wants to merge 1 commit into
masterfrom
feature/remove-unnecessary-endpoints
Open

refactor: remove unused Comment functionality#13
xavierandueza wants to merge 1 commit into
masterfrom
feature/remove-unnecessary-endpoints

Conversation

@xavierandueza
Copy link
Copy Markdown
Owner

@xavierandueza xavierandueza commented Feb 12, 2026

Summary

Removes functionality that are no longer needed:

  • Comments endpoint and associated comment meta fields
  • Application passwords management
  • Remote URL metadata fetcher for the block editor
  • Site health diagnostic test endpoints

The underlying systems remain intact — the comment system, application password management (admin UI + authentication layer), and site health checks (via direct callbacks and existing AJAX handlers) all continue to function. Only the REST API surface is removed.

…ds, URL details, and site health

Remove four REST API endpoint controllers and their associated route registrations,
require statements, and dependent code references. The underlying functionality
(comment system, application password management, site health checks) remains
available through the admin UI and existing AJAX fallback handlers.
@xavierandueza xavierandueza changed the title refactor: remove unused REST API endpoints refactor: remove unused Comment functionality Feb 16, 2026
@xavierandueza xavierandueza marked this pull request as ready for review February 16, 2026 22:50
@xavierandueza xavierandueza reopened this Feb 16, 2026
Repository owner deleted a comment from readme-pr-app Bot Feb 16, 2026
Repository owner deleted a comment from readme-pr-app Bot Feb 16, 2026
@xavierandueza xavierandueza reopened this Feb 17, 2026
@readme-pr-app
Copy link
Copy Markdown

readme-pr-app Bot commented Feb 17, 2026

📚 Documentation update workflow has been queued for reopened PR (Job ID: 6994ec12a0f3185b35c76760). The documentation bot will propose changes (if relevant) shortly.

@readme-pr-app
Copy link
Copy Markdown

readme-pr-app Bot commented Feb 20, 2026

📚 Analyzing PR for documentation updates...

@readme-pr-app
Copy link
Copy Markdown

readme-pr-app Bot commented Feb 20, 2026

📚 Documentation Changes Applied

🔗 View changes in ReadMe

@readme-pr-app
Copy link
Copy Markdown

readme-pr-app Bot commented Feb 27, 2026

📚 Documentation Changes Applied

🔗 View changes in ReadMe

@readme-pr-app
Copy link
Copy Markdown

readme-pr-app Bot commented Feb 28, 2026

📚 Documentation Changes Applied

🔗 View changes in ReadMe

@readme-pr-app
Copy link
Copy Markdown

readme-pr-app Bot commented Mar 2, 2026

📚 Documentation Changes Applied

🔗 View changes in ReadMe

@readme-pr-app
Copy link
Copy Markdown

readme-pr-app Bot commented Mar 2, 2026

📚 Documentation Changes Applied

🔗 View changes in ReadMe

@xavierandueza xavierandueza reopened this Mar 24, 2026
@readme-agent-staging
Copy link
Copy Markdown

readme-agent-staging Bot commented Mar 24, 2026

⚠️ The documentation bot was unable to apply documentation updates (Job ID: 69c21ba26df0bab99c6085dc).

You can re-trigger by reopening this PR.

@readme-pr-app
Copy link
Copy Markdown

readme-pr-app Bot commented Mar 24, 2026

Documentation Changes Added

Page Section Action Preview
listcommentsAPI Reference🔴 DeletedPreview
createcommentAPI Reference🔴 DeletedPreview
getcommentAPI Reference🔴 DeletedPreview
updatecommentAPI Reference🔴 DeletedPreview
deletecommentAPI Reference🔴 DeletedPreview
commentsAPI Reference🔴 DeletedPreview
listapplicationpasswordsAPI Reference🔴 DeletedPreview
createapplicationpasswordAPI Reference🔴 DeletedPreview
deleteallapplicationpasswordsAPI Reference🔴 DeletedPreview
introspectapplicationpasswordAPI Reference🔴 DeletedPreview
getapplicationpasswordAPI Reference🔴 DeletedPreview
updateapplicationpasswordAPI Reference🔴 DeletedPreview
deleteapplicationpasswordAPI Reference🔴 DeletedPreview
application-passwordsAPI Reference🔴 DeletedPreview
sitehealthbackgroundupdatesAPI Reference🔴 DeletedPreview
sitehealthloopbackrequestsAPI Reference🔴 DeletedPreview
sitehealthhttpsstatusAPI Reference🔴 DeletedPreview
sitehealthwordpressorgcommunicationAPI Reference🔴 DeletedPreview
sitehealthauthorizationheaderAPI Reference🔴 DeletedPreview
site-healthAPI Reference🔴 DeletedPreview

🔗 View all changes in ReadMe


Actions

  • Merge doc branch with PR merge
  • Delete doc branch with PR close

@readme-agent-staging
Copy link
Copy Markdown

readme-agent-staging Bot commented Mar 24, 2026

⚠️ The documentation bot was unable to apply documentation updates (Job ID: 69c221e86df0bab99c608643).

You can re-trigger by reopening this PR.

@readme-pr-app
Copy link
Copy Markdown

readme-pr-app Bot commented Mar 24, 2026

Documentation Changes Added

Page Section Action Preview
listapplicationpasswordsAPI Reference🔴 DeletedPreview
createapplicationpasswordAPI Reference🔴 DeletedPreview
deleteallapplicationpasswordsAPI Reference🔴 DeletedPreview
updateapplicationpasswordAPI Reference🔴 DeletedPreview
getapplicationpasswordAPI Reference🔴 DeletedPreview
createcommentAPI Reference🔴 DeletedPreview
listcommentsAPI Reference🔴 DeletedPreview
updatecommentAPI Reference🔴 DeletedPreview
sitehealthhttpsstatusAPI Reference🔴 DeletedPreview
sitehealthbackgroundupdatesAPI Reference🔴 DeletedPreview

🔗 View all changes in ReadMe


Actions

  • Merge doc branch with PR merge
  • Delete doc branch with PR close

@readme-agent-staging
Copy link
Copy Markdown

readme-agent-staging Bot commented Mar 24, 2026

⚠️ The documentation bot was unable to apply documentation updates (Job ID: 69c224c46df0bab99c608685).

You can re-trigger by reopening this PR.

@readme-pr-app
Copy link
Copy Markdown

readme-pr-app Bot commented Mar 24, 2026

Documentation Changes Added

Page Section Action Preview
listapplicationpasswordsAPI Reference🔴 DeletedPreview
createapplicationpasswordAPI Reference🔴 DeletedPreview
deleteallapplicationpasswordsAPI Reference🔴 DeletedPreview
updateapplicationpasswordAPI Reference🔴 DeletedPreview
listcommentsAPI Reference🔴 DeletedPreview
createcommentAPI Reference🔴 DeletedPreview
getcommentAPI Reference🔴 DeletedPreview
sitehealthhttpsstatusAPI Reference🔴 DeletedPreview
sitehealthbackgroundupdatesAPI Reference🔴 DeletedPreview
sitehealthauthorizationheaderAPI Reference🔴 DeletedPreview

🔗 View all changes in ReadMe


Actions

  • Merge doc branch with PR merge
  • Delete doc branch with PR close

@readme-agent-staging
Copy link
Copy Markdown

readme-agent-staging Bot commented Mar 26, 2026

📚 No documentation updates were needed for this PR.

@readme-pr-app
Copy link
Copy Markdown

readme-pr-app Bot commented Mar 26, 2026

Documentation Changes Added

Page Section Action Summary
listapplicationpasswordsAPI Reference🗑️ DeletedThe Application Passwords REST API controller has been deleted and its routes unregistered in this PR, so this endpoint no longer exists.
createapplicationpasswordAPI Reference🗑️ DeletedThe Application Passwords REST API controller has been deleted and its routes unregistered in this PR, so this endpoint no longer exists.
deleteallapplicationpasswordsAPI Reference🗑️ DeletedThe Application Passwords REST API controller has been deleted and its routes unregistered in this PR, so this endpoint no longer exists.
getapplicationpasswordAPI Reference🗑️ DeletedThe Application Passwords REST API controller has been deleted and its routes unregistered in this PR, so this endpoint no longer exists.
introspectapplicationpasswordAPI Reference🗑️ DeletedThe Application Passwords REST API controller has been deleted and its routes unregistered in this PR, so this endpoint no longer exists.
updateapplicationpasswordAPI Reference🗑️ DeletedThe Application Passwords REST API controller has been deleted and its routes unregistered in this PR, so this endpoint no longer exists.
deleteapplicationpasswordAPI Reference🗑️ DeletedThe Application Passwords REST API controller has been deleted and its routes unregistered in this PR, so this endpoint no longer exists.
sitehealthhttpsstatusAPI Reference🗑️ DeletedThe Site Health REST API controller has been deleted and its routes unregistered in this PR, so this endpoint no longer exists.
sitehealthbackgroundupdatesAPI Reference🗑️ DeletedThe Site Health REST API controller has been deleted and its routes unregistered in this PR, so this endpoint no longer exists.
sitehealthloopbackrequestsAPI Reference🗑️ DeletedThe Site Health REST API controller has been deleted and its routes unregistered in this PR, so this endpoint no longer exists.
sitehealthauthorizationheaderAPI Reference🗑️ DeletedThe Site Health REST API controller has been deleted and its routes unregistered in this PR, so this endpoint no longer exists.
sitehealthwordpressorgcommunicationAPI Reference🗑️ DeletedThe Site Health REST API controller has been deleted and its routes unregistered in this PR, so this endpoint no longer exists.
listcommentsAPI Reference🗑️ DeletedThe Comments REST API controller has been deleted and its routes unregistered in this PR, so this endpoint no longer exists.
createcommentAPI Reference🗑️ DeletedThe Comments REST API controller has been deleted and its routes unregistered in this PR, so this endpoint no longer exists.
getcommentAPI Reference🗑️ DeletedThe Comments REST API controller has been deleted and its routes unregistered in this PR, so this endpoint no longer exists.
updatecommentAPI Reference🗑️ DeletedThe Comments REST API controller has been deleted and its routes unregistered in this PR, so this endpoint no longer exists.
deletecommentAPI Reference🗑️ DeletedThe Comments REST API controller has been deleted and its routes unregistered in this PR, so this endpoint no longer exists.
application-passwordsAPI Reference🗑️ DeletedAll Application Passwords REST API endpoints have been removed in this PR, so the parent category page is no longer needed.
site-healthAPI Reference🗑️ DeletedAll Site Health REST API endpoints have been removed in this PR, so the parent category page is no longer needed.
commentsAPI Reference🗑️ DeletedAll Comments REST API endpoints have been removed in this PR, so the parent category page is no longer needed.

🔗 View all changes in ReadMe


Actions

  • Merge documentation branch with PR merge
  • Delete documentation branch with PR close

@readme-pr-app
Copy link
Copy Markdown

readme-pr-app Bot commented Apr 2, 2026

Documentation Changes Added

Page Section Action Summary
listcommentsAPI Reference🗑️ DeletedThe Comments REST API endpoint (/wp/v2/comments) has been removed from WordPress core.
createcommentAPI Reference🗑️ DeletedThe Comments REST API endpoint (/wp/v2/comments) has been removed from WordPress core.
getcommentAPI Reference🗑️ DeletedThe Comments REST API endpoint (/wp/v2/comments) has been removed from WordPress core.
updatecommentAPI Reference🗑️ DeletedThe Comments REST API endpoint (/wp/v2/comments) has been removed from WordPress core.
deletecommentAPI Reference🗑️ DeletedThe Comments REST API endpoint (/wp/v2/comments) has been removed from WordPress core.
commentsAPI Reference🗑️ DeletedThe Comments REST API endpoint (/wp/v2/comments) has been removed from WordPress core.
listapplicationpasswordsAPI Reference🗑️ DeletedThe Application Passwords REST API endpoint has been removed from WordPress core.
createapplicationpasswordAPI Reference🗑️ DeletedThe Application Passwords REST API endpoint has been removed from WordPress core.
deleteallapplicationpasswordsAPI Reference🗑️ DeletedThe Application Passwords REST API endpoint has been removed from WordPress core.
introspectapplicationpasswordAPI Reference🗑️ DeletedThe Application Passwords REST API endpoint has been removed from WordPress core.
getapplicationpasswordAPI Reference🗑️ DeletedThe Application Passwords REST API endpoint has been removed from WordPress core.
updateapplicationpasswordAPI Reference🗑️ DeletedThe Application Passwords REST API endpoint has been removed from WordPress core.
deleteapplicationpasswordAPI Reference🗑️ DeletedThe Application Passwords REST API endpoint has been removed from WordPress core.
application-passwordsAPI Reference🗑️ DeletedThe Application Passwords REST API endpoint has been removed from WordPress core.
sitehealthbackgroundupdatesAPI Reference🗑️ DeletedThe Site Health REST API endpoint has been removed from WordPress core.
sitehealthloopbackrequestsAPI Reference🗑️ DeletedThe Site Health REST API endpoint has been removed from WordPress core.
sitehealthhttpsstatusAPI Reference🗑️ DeletedThe Site Health REST API endpoint has been removed from WordPress core.
sitehealthwordpressorgcommunicationAPI Reference🗑️ DeletedThe Site Health REST API endpoint has been removed from WordPress core.
sitehealthauthorizationheaderAPI Reference🗑️ DeletedThe Site Health REST API endpoint has been removed from WordPress core; the authorization header test was also removed entirely.
site-healthAPI Reference🗑️ DeletedThe Site Health REST API endpoint has been removed from WordPress core.

🔗 View all changes in ReadMe


Actions

  • Merge documentation branch with PR merge
  • Delete documentation branch with PR close

@readme-pr-app
Copy link
Copy Markdown

readme-pr-app Bot commented Apr 8, 2026

Documentation Changes Added

Page Section Action Summary
listcommentsAPI Reference🗑 DeletedThe Comments REST API endpoint (/wp/v2/comments) has been removed; the WP_REST_Comments_Controller class was deleted.
createcommentAPI Reference🗑 DeletedThe Comments REST API endpoint (/wp/v2/comments) has been removed; the WP_REST_Comments_Controller class was deleted.
getcommentAPI Reference🗑 DeletedThe Comments REST API endpoint (/wp/v2/comments/{id}) has been removed; the WP_REST_Comments_Controller class was deleted.
updatecommentAPI Reference🗑 DeletedThe Comments REST API endpoint (/wp/v2/comments/{id}) has been removed; the WP_REST_Comments_Controller class was deleted.
deletecommentAPI Reference🗑 DeletedThe Comments REST API endpoint (/wp/v2/comments/{id}) has been removed; the WP_REST_Comments_Controller class was deleted.
commentsAPI Reference🗑 DeletedThe Comments REST API resource has been removed; the WP_REST_Comments_Controller class was deleted.
listapplicationpasswordsAPI Reference🗑 DeletedThe Application Passwords REST API endpoint has been removed; the WP_REST_Application_Passwords_Controller class was deleted.
createapplicationpasswordAPI Reference🗑 DeletedThe Application Passwords REST API endpoint has been removed; the WP_REST_Application_Passwords_Controller class was deleted.
deleteallapplicationpasswordsAPI Reference🗑 DeletedThe Application Passwords REST API endpoint has been removed; the WP_REST_Application_Passwords_Controller class was deleted.
introspectapplicationpasswordAPI Reference🗑 DeletedThe Application Passwords REST API endpoint has been removed; the WP_REST_Application_Passwords_Controller class was deleted.
getapplicationpasswordAPI Reference🗑 DeletedThe Application Passwords REST API endpoint has been removed; the WP_REST_Application_Passwords_Controller class was deleted.
updateapplicationpasswordAPI Reference🗑 DeletedThe Application Passwords REST API endpoint has been removed; the WP_REST_Application_Passwords_Controller class was deleted.
deleteapplicationpasswordAPI Reference🗑 DeletedThe Application Passwords REST API endpoint has been removed; the WP_REST_Application_Passwords_Controller class was deleted.
application-passwordsAPI Reference🗑 DeletedThe Application Passwords REST API resource has been removed; the WP_REST_Application_Passwords_Controller class was deleted.
sitehealthbackgroundupdatesAPI Reference🗑 DeletedThe Site Health REST API endpoint has been removed; the WP_REST_Site_Health_Controller class was deleted.
sitehealthloopbackrequestsAPI Reference🗑 DeletedThe Site Health REST API endpoint has been removed; the WP_REST_Site_Health_Controller class was deleted.
sitehealthhttpsstatusAPI Reference🗑 DeletedThe Site Health REST API endpoint has been removed; the WP_REST_Site_Health_Controller class was deleted.
sitehealthwordpressorgcommunicationAPI Reference🗑 DeletedThe Site Health REST API endpoint has been removed; the WP_REST_Site_Health_Controller class was deleted.
sitehealthauthorizationheaderAPI Reference🗑 DeletedThe Site Health REST API endpoint has been removed; the WP_REST_Site_Health_Controller class was deleted.
site-healthAPI Reference🗑 DeletedThe Site Health REST API resource has been removed; the WP_REST_Site_Health_Controller class was deleted.

🔗 View all changes in ReadMe


Actions

  • Merge documentation branch with PR merge
  • Delete documentation branch with PR close

@readme-pr-app
Copy link
Copy Markdown

readme-pr-app Bot commented Apr 10, 2026

Documentation Changes Added

Page Section Action Summary
listcommentsAPI Reference🗑 DeletedThe Comments REST API controller (WP_REST_Comments_Controller) has been removed, so the List comments endpoint no longer exists.
createcommentAPI Reference🗑 DeletedThe Comments REST API controller (WP_REST_Comments_Controller) has been removed, so the Create a comment endpoint no longer exists.
getcommentAPI Reference🗑 DeletedThe Comments REST API controller (WP_REST_Comments_Controller) has been removed, so the Retrieve a comment endpoint no longer exists.
updatecommentAPI Reference🗑 DeletedThe Comments REST API controller (WP_REST_Comments_Controller) has been removed, so the Update a comment endpoint no longer exists.
deletecommentAPI Reference🗑 DeletedThe Comments REST API controller (WP_REST_Comments_Controller) has been removed, so the Delete a comment endpoint no longer exists.
commentsAPI Reference🗑 DeletedThe Comments REST API controller (WP_REST_Comments_Controller) has been removed, so the Comments parent reference page is no longer needed.
listapplicationpasswordsAPI Reference🗑 DeletedThe Application Passwords REST API controller (WP_REST_Application_Passwords_Controller) has been removed, so the List application passwords endpoint no longer exists.
createapplicationpasswordAPI Reference🗑 DeletedThe Application Passwords REST API controller has been removed, so the Create an application password endpoint no longer exists.
deleteallapplicationpasswordsAPI Reference🗑 DeletedThe Application Passwords REST API controller has been removed, so the Delete all application passwords endpoint no longer exists.
introspectapplicationpasswordAPI Reference🗑 DeletedThe Application Passwords REST API controller has been removed, so the Introspect application password endpoint no longer exists.
getapplicationpasswordAPI Reference🗑 DeletedThe Application Passwords REST API controller has been removed, so the Retrieve an application password endpoint no longer exists.
updateapplicationpasswordAPI Reference🗑 DeletedThe Application Passwords REST API controller has been removed, so the Update an application password endpoint no longer exists.
deleteapplicationpasswordAPI Reference🗑 DeletedThe Application Passwords REST API controller has been removed, so the Delete an application password endpoint no longer exists.
application-passwordsAPI Reference🗑 DeletedThe Application Passwords REST API controller has been removed, so the Application Passwords parent reference page is no longer needed.
sitehealthbackgroundupdatesAPI Reference🗑 DeletedThe Site Health REST API controller (WP_REST_Site_Health_Controller) has been removed, so the Background Updates health test endpoint no longer exists.
sitehealthloopbackrequestsAPI Reference🗑 DeletedThe Site Health REST API controller has been removed, so the Loopback Requests health test endpoint no longer exists.
sitehealthhttpsstatusAPI Reference🗑 DeletedThe Site Health REST API controller has been removed, so the HTTPS Status health test endpoint no longer exists.
sitehealthwordpressorgcommunicationAPI Reference🗑 DeletedThe Site Health REST API controller has been removed, so the WordPress.org Communication health test endpoint no longer exists.
sitehealthauthorizationheaderAPI Reference🗑 DeletedThe Site Health REST API controller has been removed, so the Authorization Header health test endpoint no longer exists.
site-healthAPI Reference🗑 DeletedThe Site Health REST API controller has been removed, so the Site Health parent reference page is no longer needed.

🔗 View all changes in ReadMe


Actions

  • Merge documentation branch with PR merge
  • Delete documentation branch with PR close

@readme-pr-app
Copy link
Copy Markdown

readme-pr-app Bot commented Apr 24, 2026

Documentation Changes Added

Page Section Action Summary
listcommentsAPI Reference🗑 DeletedThe Comments REST API endpoint (/wp/v2/comments) has been removed; the WP_REST_Comments_Controller class was deleted.
createcommentAPI Reference🗑 DeletedThe Comments REST API endpoint (/wp/v2/comments) has been removed; the WP_REST_Comments_Controller class was deleted.
getcommentAPI Reference🗑 DeletedThe Comments REST API endpoint (/wp/v2/comments/{id}) has been removed; the WP_REST_Comments_Controller class was deleted.
updatecommentAPI Reference🗑 DeletedThe Comments REST API endpoint (/wp/v2/comments/{id}) has been removed; the WP_REST_Comments_Controller class was deleted.
deletecommentAPI Reference🗑 DeletedThe Comments REST API endpoint (/wp/v2/comments/{id}) has been removed; the WP_REST_Comments_Controller class was deleted.
commentsAPI Reference🗑 DeletedThe Comments REST API endpoints have been removed; the WP_REST_Comments_Controller class was deleted.
listapplicationpasswordsAPI Reference🗑 DeletedThe Application Passwords REST API endpoint has been removed; the WP_REST_Application_Passwords_Controller class was deleted.
createapplicationpasswordAPI Reference🗑 DeletedThe Application Passwords REST API endpoint has been removed; the WP_REST_Application_Passwords_Controller class was deleted.
deleteallapplicationpasswordsAPI Reference🗑 DeletedThe Application Passwords REST API endpoint has been removed; the WP_REST_Application_Passwords_Controller class was deleted.
introspectapplicationpasswordAPI Reference🗑 DeletedThe Application Passwords REST API endpoint has been removed; the WP_REST_Application_Passwords_Controller class was deleted.
getapplicationpasswordAPI Reference🗑 DeletedThe Application Passwords REST API endpoint has been removed; the WP_REST_Application_Passwords_Controller class was deleted.
updateapplicationpasswordAPI Reference🗑 DeletedThe Application Passwords REST API endpoint has been removed; the WP_REST_Application_Passwords_Controller class was deleted.
deleteapplicationpasswordAPI Reference🗑 DeletedThe Application Passwords REST API endpoint has been removed; the WP_REST_Application_Passwords_Controller class was deleted.
application-passwordsAPI Reference🗑 DeletedThe Application Passwords REST API endpoints have been removed; the WP_REST_Application_Passwords_Controller class was deleted.
sitehealthbackgroundupdatesAPI Reference🗑 DeletedThe Site Health REST API endpoints have been removed; the WP_REST_Site_Health_Controller class was deleted.
sitehealthloopbackrequestsAPI Reference🗑 DeletedThe Site Health REST API endpoints have been removed; the WP_REST_Site_Health_Controller class was deleted.
sitehealthhttpsstatusAPI Reference🗑 DeletedThe Site Health REST API endpoints have been removed; the WP_REST_Site_Health_Controller class was deleted.
sitehealthauthorizationheaderAPI Reference🗑 DeletedThe Site Health REST API endpoints have been removed; the WP_REST_Site_Health_Controller class was deleted.
sitehealthwordpressorgcommunicationAPI Reference🗑 DeletedThe Site Health REST API endpoints have been removed; the WP_REST_Site_Health_Controller class was deleted.
site-healthAPI Reference🗑 DeletedThe Site Health REST API endpoints have been removed; the WP_REST_Site_Health_Controller class was deleted.

🔗 View all changes in ReadMe


Actions

  • Merge documentation branch with PR merge
  • Delete documentation branch with PR close

If neither actions are selected, on PR close/merge the docs branch in ReadMe will remain open.

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