echo \"hi\";\n";
+
+ $html = 'Heading
' + . 'Hello world link.
' + . '- One
- Two
echo \"hi\";\nHeading
' @@ -32,7 +48,7 @@ public function test_basic_conversion() { . '
H1
H2
H3
H4
H5
H6
'; + + $markdown = $this->converter->convert( $html ); + + $this->assertStringContainsString( '# H1', $markdown ); + $this->assertStringContainsString( '## H2', $markdown ); + $this->assertStringContainsString( '### H3', $markdown ); + $this->assertStringContainsString( '#### H4', $markdown ); + $this->assertStringContainsString( '##### H5', $markdown ); + $this->assertStringContainsString( '###### H6', $markdown ); + } + + /** + * Test conversion of bold text using both strong and b tags. + * + * @since x.x.x + */ + public function test_converts_bold_text(): void { + $html = 'This is bold and also bold.
'; + $markdown = $this->converter->convert( $html ); + + $this->assertStringContainsString( '**bold**', $markdown ); + $this->assertStringContainsString( '**also bold**', $markdown ); + } + + /** + * Test conversion of italic text using both em and i tags. + * + * @since x.x.x + */ + public function test_converts_italic_text(): void { + $html = 'This is italic and also italic.
'; + $markdown = $this->converter->convert( $html ); + + $this->assertStringContainsString( '*italic*', $markdown ); + $this->assertStringContainsString( '*also italic*', $markdown ); + } + + /** + * Test conversion of links with href attribute. + * + * @since x.x.x + */ + public function test_converts_links(): void { + $html = 'Visit Example Site for more info.
'; + $markdown = $this->converter->convert( $html ); + + $this->assertStringContainsString( '[Example Site](https://example.com)', $markdown ); + } + + /** + * Test conversion of images with alt text. + * + * @since x.x.x + */ + public function test_converts_images_with_alt(): void { + $html = '
';
+ $markdown = $this->converter->convert( $html );
+
+ $this->assertStringContainsString( '', $markdown );
+ }
+
+ /**
+ * Test conversion of unordered lists.
+ *
+ * @since x.x.x
+ */
+ public function test_converts_unordered_lists(): void {
+ $html = '- Apple
- Banana
- Cherry
- First
- Second
- Third
This is a quoted text.'; + $markdown = $this->converter->convert( $html ); + + $this->assertStringContainsString( '> This is a quoted text.', $markdown ); + } + + /** + * Test conversion of inline code. + * + * @since x.x.x + */ + public function test_converts_inline_code(): void { + $html = '
Use the console.log() function for debugging.
const x = 42;Above the line.
Below the line.
'; + $markdown = $this->converter->convert( $html ); + + $this->assertStringContainsString( '---', $markdown ); + } + + /** + * Test conversion of line breaks. + * + * @since x.x.x + */ + public function test_converts_line_breaks(): void { + $html = 'Line one
Line two
| Name | Age |
|---|---|
| Alice | 30 |
| Bob | 25 |
| Product | Price |
|---|---|
| Widget | $10 |
Safe content
More safe content
'; + $markdown = $this->converter->convert( $html ); + + $this->assertStringContainsString( 'Safe content', $markdown ); + $this->assertStringContainsString( 'More safe content', $markdown ); + $this->assertStringNotContainsString( 'malicious', $markdown ); + $this->assertStringNotContainsString( 'script', $markdown ); + } + + /** + * Test that style tags are completely removed. + * + * @since x.x.x + */ + public function test_removes_style_tags(): void { + $html = 'Content
'; + $markdown = $this->converter->convert( $html ); + + $this->assertStringContainsString( 'Content', $markdown ); + $this->assertStringNotContainsString( 'color', $markdown ); + $this->assertStringNotContainsString( 'style', $markdown ); + } + + /** + * Test that empty input returns empty output. + * + * @since x.x.x + */ + public function test_empty_input(): void { + $markdown = $this->converter->convert( '' ); + + $this->assertEmpty( $markdown ); + } + + /** + * Test that whitespace-only input is handled gracefully. + * + * @since x.x.x + */ + public function test_whitespace_only_input(): void { + $markdown = $this->converter->convert( ' ' ); + + $this->assertEmpty( trim( $markdown ) ); + } + + /** + * Test nested formatting (bold within italic, etc.). + * + * @since x.x.x + */ + public function test_nested_formatting(): void { + $html = 'This is bold italic text.
'; + $markdown = $this->converter->convert( $html ); + + $this->assertStringContainsString( '***bold italic***', $markdown ); + } + + /** + * Test complex nested structure with blockquote containing formatted text. + * + * @since x.x.x + */ + public function test_complex_blockquote(): void { + $html = '+'; + + $markdown = $this->converter->convert( $html ); + + $this->assertStringContainsString( '>', $markdown ); + $this->assertStringContainsString( '**bold**', $markdown ); + $this->assertStringContainsString( '*italic*', $markdown ); + } + + /** + * Test that excessive whitespace is normalized. + * + * @since x.x.x + */ + public function test_normalizes_whitespace(): void { + $html = 'A quote with bold and italic text.
+
Multiple spaces here
'; + $markdown = $this->converter->convert( $html ); + + // Should not contain multiple consecutive spaces. + $this->assertStringNotContainsString( ' ', $markdown ); + $this->assertStringContainsString( 'Multiple spaces here', $markdown ); + } } diff --git a/tests/Integration/Includes/Experiments/Markdown_Feeds/Markdown_FeedsTest.php b/tests/Integration/Includes/Experiments/Markdown_Feeds/Markdown_FeedsTest.php index 3eef1bb5..4bcf3f47 100644 --- a/tests/Integration/Includes/Experiments/Markdown_Feeds/Markdown_FeedsTest.php +++ b/tests/Integration/Includes/Experiments/Markdown_Feeds/Markdown_FeedsTest.php @@ -18,6 +18,14 @@ * @since x.x.x */ class Markdown_FeedsTest extends WP_UnitTestCase { + + /** + * The experiment instance. + * + * @var \WordPress\AI\Experiments\Markdown_Feeds\Markdown_Feeds + */ + private Markdown_Feeds $experiment; + /** * Set up test case. * @@ -42,6 +50,9 @@ public function setUp(): void { $experiment = $registry->get_experiment( 'markdown-feeds' ); $this->assertInstanceOf( Markdown_Feeds::class, $experiment, 'Markdown feeds experiment should be registered in the registry.' ); + + $this->experiment = new Markdown_Feeds(); + $this->experiment->register(); } /** @@ -62,11 +73,132 @@ public function tearDown(): void { * * @since x.x.x */ - public function test_experiment_registration() { + public function test_experiment_registration(): void { $experiment = new Markdown_Feeds(); $this->assertEquals( 'markdown-feeds', $experiment->get_id() ); - $this->assertEquals( 'Markdown', $experiment->get_label() ); + $this->assertEquals( 'Markdown Feeds', $experiment->get_label() ); $this->assertTrue( $experiment->is_enabled() ); } + + /** + * Test that the Markdown feed is registered. + * + * @since x.x.x + */ + public function test_markdown_feed_is_registered(): void { + global $wp_rewrite; + + $this->experiment->register_feed(); + + // The feed should be added to the registered feeds. + $this->assertContains( 'markdown', $wp_rewrite->feeds ); + } + + /** + * Test that get_markdown_feed_link returns the correct URL. + * + * @since x.x.x + */ + public function test_get_markdown_feed_link(): void { + $feed_link = $this->experiment->get_markdown_feed_link(); + + $this->assertStringContainsString( 'feed', $feed_link ); + $this->assertStringContainsString( 'markdown', $feed_link ); + } + + /** + * Test that get_markdown_permalink returns the correct URL for a post. + * + * @since x.x.x + */ + public function test_get_markdown_permalink(): void { + $post_id = self::factory()->post->create( + array( + 'post_title' => 'Test Post', + 'post_name' => 'test-post', + 'post_status' => 'publish', + ) + ); + + $post = get_post( $post_id ); + $md_permalink = $this->experiment->get_markdown_permalink( $post ); + + $this->assertStringEndsWith( '.md', $md_permalink ); + $this->assertStringContainsString( 'test-post', $md_permalink ); + } + + /** + * Test that request filter strips .md from post name. + * + * @since x.x.x + */ + public function test_filter_request_strips_md_from_name(): void { + $_SERVER['REQUEST_METHOD'] = 'GET'; + + $query_vars = array( 'name' => 'test-post.md' ); + $result = $this->experiment->filter_request_for_markdown_extension( $query_vars ); + + $this->assertEquals( 'test-post', $result['name'] ); + } + + /** + * Test that request filter strips .md from pagename. + * + * @since x.x.x + */ + public function test_filter_request_strips_md_from_pagename(): void { + $_SERVER['REQUEST_METHOD'] = 'GET'; + + $query_vars = array( 'pagename' => 'sample-page.md' ); + $result = $this->experiment->filter_request_for_markdown_extension( $query_vars ); + + $this->assertEquals( 'sample-page', $result['pagename'] ); + } + + /** + * Test that request filter ignores non-.md extensions. + * + * @since x.x.x + */ + public function test_filter_request_ignores_non_md_extensions(): void { + $_SERVER['REQUEST_METHOD'] = 'GET'; + + $query_vars = array( 'name' => 'test-post.html' ); + $result = $this->experiment->filter_request_for_markdown_extension( $query_vars ); + + $this->assertEquals( 'test-post.html', $result['name'] ); + } + + /** + * Test that request filter ignores POST requests. + * + * @since x.x.x + */ + public function test_filter_request_ignores_post_requests(): void { + $_SERVER['REQUEST_METHOD'] = 'POST'; + + $query_vars = array( 'name' => 'test-post.md' ); + $result = $this->experiment->filter_request_for_markdown_extension( $query_vars ); + + $this->assertEquals( 'test-post.md', $result['name'] ); + } + + /** + * Test canonical redirect is prevented for .md requests. + * + * @since x.x.x + */ + public function test_filter_redirect_canonical_prevents_redirect_for_md(): void { + $_SERVER['REQUEST_METHOD'] = 'GET'; + + // First, trigger the markdown extension detection. + $query_vars = array( 'name' => 'test-post.md' ); + $this->experiment->filter_request_for_markdown_extension( $query_vars ); + + // Now test that canonical redirect is prevented. + $result = $this->experiment->filter_redirect_canonical( 'http://example.com/test-post/', 'http://example.com/test-post.md' ); + + $this->assertFalse( $result ); + } } From c79a617eda15d712bf458a956bfa3f79a9a2b2a0 Mon Sep 17 00:00:00 2001 From: James LePage <36246732+Jameswlepage@users.noreply.github.com> Date: Mon, 19 Jan 2026 14:03:25 -0500 Subject: [PATCH 3/9] fix: handle markdown permalinks without pretty URLs --- docs/experiments/markdown-feeds.md | 1 + .../Markdown_Feeds/Markdown_Feeds.php | 10 ++- .../Markdown_Feeds/Markdown_FeedsTest.php | 62 +++++++++++++++---- 3 files changed, 59 insertions(+), 14 deletions(-) diff --git a/docs/experiments/markdown-feeds.md b/docs/experiments/markdown-feeds.md index 60e75dab..18293dfa 100644 --- a/docs/experiments/markdown-feeds.md +++ b/docs/experiments/markdown-feeds.md @@ -40,3 +40,4 @@ The output is intended to be a lightweight, text-first format that is easier for - The HTML-to-Markdown conversion is intentionally conservative and based on WordPress core’s HTML API (`WP_HTML_Processor`) rather than a bundled third-party parser. - This experiment currently targets singular post content (title + metadata + content). It does not attempt to convert full theme templates or archive views. +- `.md` permalinks require pretty permalinks (a non-empty permalink structure). diff --git a/includes/Experiments/Markdown_Feeds/Markdown_Feeds.php b/includes/Experiments/Markdown_Feeds/Markdown_Feeds.php index 1542eb70..c7c032d0 100644 --- a/includes/Experiments/Markdown_Feeds/Markdown_Feeds.php +++ b/includes/Experiments/Markdown_Feeds/Markdown_Feeds.php @@ -140,6 +140,9 @@ public function add_feed_autodiscovery_links(): void { } $md_url = $this->get_markdown_permalink( $post ); + if ( '' === $md_url ) { + return; + } /* translators: %s: Post title. */ $md_title = sprintf( __( '%s (Markdown)', 'ai' ), get_the_title( $post ) ); @@ -176,8 +179,13 @@ public function get_markdown_feed_link( string $context = '' ): string { * @return string Markdown permalink. */ public function get_markdown_permalink( \WP_Post $post ): string { + $permalink_structure = (string) get_option( 'permalink_structure' ); + if ( '' === $permalink_structure ) { + return ''; + } + $permalink = get_permalink( $post ); - if ( ! $permalink ) { + if ( ! $permalink || false !== strpos( $permalink, '?' ) ) { return ''; } diff --git a/tests/Integration/Includes/Experiments/Markdown_Feeds/Markdown_FeedsTest.php b/tests/Integration/Includes/Experiments/Markdown_Feeds/Markdown_FeedsTest.php index 4bcf3f47..6a4d21ee 100644 --- a/tests/Integration/Includes/Experiments/Markdown_Feeds/Markdown_FeedsTest.php +++ b/tests/Integration/Includes/Experiments/Markdown_Feeds/Markdown_FeedsTest.php @@ -113,19 +113,55 @@ public function test_get_markdown_feed_link(): void { * @since x.x.x */ public function test_get_markdown_permalink(): void { - $post_id = self::factory()->post->create( - array( - 'post_title' => 'Test Post', - 'post_name' => 'test-post', - 'post_status' => 'publish', - ) - ); - - $post = get_post( $post_id ); - $md_permalink = $this->experiment->get_markdown_permalink( $post ); - - $this->assertStringEndsWith( '.md', $md_permalink ); - $this->assertStringContainsString( 'test-post', $md_permalink ); + global $wp_rewrite; + + $original_structure = (string) get_option( 'permalink_structure' ); + $wp_rewrite->set_permalink_structure( '/%postname%/' ); + + try { + $post_id = self::factory()->post->create( + array( + 'post_title' => 'Test Post', + 'post_name' => 'test-post', + 'post_status' => 'publish', + ) + ); + + $post = get_post( $post_id ); + $md_permalink = $this->experiment->get_markdown_permalink( $post ); + + $this->assertStringEndsWith( '.md', $md_permalink ); + $this->assertStringContainsString( 'test-post', $md_permalink ); + } finally { + $wp_rewrite->set_permalink_structure( $original_structure ); + } + } + + /** + * Test that get_markdown_permalink returns empty without pretty permalinks. + * + * @since x.x.x + */ + public function test_get_markdown_permalink_returns_empty_with_plain_permalinks(): void { + global $wp_rewrite; + + $original_structure = (string) get_option( 'permalink_structure' ); + $wp_rewrite->set_permalink_structure( '' ); + + try { + $post_id = self::factory()->post->create( + array( + 'post_title' => 'Plain Post', + 'post_name' => 'plain-post', + 'post_status' => 'publish', + ) + ); + + $post = get_post( $post_id ); + $this->assertSame( '', $this->experiment->get_markdown_permalink( $post ) ); + } finally { + $wp_rewrite->set_permalink_structure( $original_structure ); + } } /** From 461b4297c76cf5b03de5283504d4294911b8d16e Mon Sep 17 00:00:00 2001 From: James LePage <36246732+Jameswlepage@users.noreply.github.com> Date: Mon, 19 Jan 2026 14:13:51 -0500 Subject: [PATCH 4/9] feat: add filters for markdown entry sections --- docs/experiments/markdown-feeds.md | 2 + .../Markdown_Feeds/Markdown_Feed_Renderer.php | 40 ++++++++++++++--- .../Markdown_Singular_Renderer.php | 45 ++++++++++++++----- 3 files changed, 68 insertions(+), 19 deletions(-) diff --git a/docs/experiments/markdown-feeds.md b/docs/experiments/markdown-feeds.md index 18293dfa..510fb054 100644 --- a/docs/experiments/markdown-feeds.md +++ b/docs/experiments/markdown-feeds.md @@ -19,8 +19,10 @@ The output is intended to be a lightweight, text-first format that is easier for - Filters: - `ai_experiments_markdown_feed_html` -> adjust HTML before conversion. - `ai_experiments_markdown_feed_markdown` -> adjust Markdown after conversion. + - `ai_experiments_markdown_feed_post_sections` -> reorder or inject sections in each feed entry. - `ai_experiments_markdown_singular_html` -> adjust HTML before conversion (singular). - `ai_experiments_markdown_singular_markdown` -> adjust Markdown after conversion (singular). + - `ai_experiments_markdown_singular_post_sections` -> reorder or inject sections in each singular response. ## Assets & Data Flow diff --git a/includes/Experiments/Markdown_Feeds/Markdown_Feed_Renderer.php b/includes/Experiments/Markdown_Feeds/Markdown_Feed_Renderer.php index 8107238b..098c1807 100644 --- a/includes/Experiments/Markdown_Feeds/Markdown_Feed_Renderer.php +++ b/includes/Experiments/Markdown_Feeds/Markdown_Feed_Renderer.php @@ -204,21 +204,47 @@ private function send_post( WP_Post $post ): void { $permalink = esc_url_raw( get_permalink( $post ) ); $date_r = (string) get_post_time( 'r', true, $post ); - echo '## ' . esc_html( $title ) . "\n\n"; - echo esc_html__( 'URL:', 'ai' ) . ' <' . esc_url( $permalink ) . ">\n"; - echo esc_html__( 'Published:', 'ai' ) . ' ' . esc_html( $date_r ) . "\n\n"; - $content = (string) get_post_field( 'post_content', $post ); $html = (string) apply_filters( 'the_content', $content ); $markdown = $this->convert_html_to_markdown( $html ); $markdown = trim( $markdown ); + + $meta_lines = array( + esc_html__( 'URL:', 'ai' ) . ' <' . esc_url( $permalink ) . '>', + esc_html__( 'Published:', 'ai' ) . ' ' . esc_html( $date_r ), + ); + + $sections = array( + 'header' => '## ' . esc_html( $title ), + 'meta' => implode( "\n", $meta_lines ), + ); + if ( '' !== $markdown ) { - // phpcs:ignore WordPress.Security.EscapeOutput.OutputNotEscaped -- Markdown response. - echo $markdown . "\n\n"; + $sections['content'] = $markdown; + } + + $sections['footer'] = '---'; + + /** + * Filters the Markdown feed entry sections. + * + * Allows reordering or inserting custom sections before output is emitted. + * + * @since x.x.x + * + * @param array%s', 'ai' ),
- esc_html( '/?feed=markdown' )
+ '/?feed=markdown'
),
array( 'code' => array() )
);
@@ -285,7 +260,7 @@ public function render_settings_fields(): void {
sprintf(
/* translators: %s: File extension used for Markdown permalinks. */
__( 'Enable %s permalinks for singular content', 'ai' ),
- esc_html( '.md' )
+ '.md'
),
array( 'code' => array() )
);
@@ -307,7 +282,7 @@ public function render_settings_fields(): void {
sprintf(
/* translators: %s: HTTP Accept header value used for Markdown negotiation. */
__( 'Enable %s negotiation for singular content', 'ai' ),
- esc_html( 'Accept: text/markdown' )
+ 'Accept: text/markdown'
),
array( 'code' => array() )
);
@@ -508,8 +483,7 @@ private function is_post_accessible( \WP_Post $post ): bool {
return false;
}
- $cap = $post_type_obj->cap->read_private_posts ?? 'read_private_posts';
- return current_user_can( $cap );
+ return current_user_can( $post_type_obj->cap->read_private_posts ?? 'read_private_posts' );
}
// Draft, pending, future, trash, etc. are not accessible.
diff --git a/includes/Experiments/Markdown_Feeds/Markdown_Singular_Renderer.php b/includes/Experiments/Markdown_Feeds/Markdown_Singular_Renderer.php
index 5e3f0560..a0be5a3c 100644
--- a/includes/Experiments/Markdown_Feeds/Markdown_Singular_Renderer.php
+++ b/includes/Experiments/Markdown_Feeds/Markdown_Singular_Renderer.php
@@ -11,21 +11,6 @@
use WP_Post;
-use function apply_filters;
-use function esc_html;
-use function esc_html__;
-use function esc_url;
-use function get_option;
-use function get_permalink;
-use function get_post_field;
-use function get_post_modified_time;
-use function get_post_time;
-use function get_the_title;
-use function sanitize_key;
-use function status_header;
-use function wp_strip_all_tags;
-use function wp_unslash;
-
/**
* Outputs a Markdown representation of a singular post.
*
From 498a3f9dadb00fdd0316b409a9abdd18e3849224 Mon Sep 17 00:00:00 2001
From: Darin Kotter
+