Repository navigation
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## develop #1053 +/- ##
=============================================
+ Coverage 80.72% 80.87% +0.15%
Complexity 2998 2998
=============================================
Files 124 124
Lines 11978 11987 +9
=============================================
+ Hits 9669 9695 +26
+ Misses 2309 2292 -17
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
|
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 If you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message. To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook. |
yogeshbhutkar
left a comment
There was a problem hiding this comment.
Adding a few minor nits:
…n into shared utility
|
Hi @dkotter and @yogeshbhutkar, I've addressed all the feedback. Can you please have a look at your convenience. Thank you! |
…ging in editorial notes
…ent block is missing
yogeshbhutkar
left a comment
There was a problem hiding this comment.
Tests well. Looping @dkotter for another look.
|
Just an FYI, I have added a test that is: |
What?
Closes: #1052
Ensures Editorial Notes analyzes the actual post and page content blocks instead of the surrounding template blocks when the "Show template" option is enabled in the WordPress block editor.
Why?
When "Show template" is turned on in WordPress, the editor wraps the post inside template elements (such as the Header, Post Title, Featured Image, and Footer). Editorial Notes was fetching blocks from the root of the template rather than the post content itself. As a result, it only reviewed the template blocks (e.g., 4 blocks) instead of the actual post blocks (e.g., 14 blocks).
Use of AI Tools
AI assistance: Yes
Tool(s): Claude Code
Model(s): Claude Sonnet 4.6
Used for: Validating bug, suggesting a fix.
Testing Instructions
Screenshots or screencast
Changelog Entry