Skip to content

Comments

[Fundamentals] Add Configuration Rules instructions for Markdown for Agents#28491

Merged
celso merged 4 commits intoproductionfrom
dina/markdown-agents-config-rules
Feb 23, 2026
Merged

[Fundamentals] Add Configuration Rules instructions for Markdown for Agents#28491
celso merged 4 commits intoproductionfrom
dina/markdown-agents-config-rules

Conversation

@dinasaur404
Copy link
Contributor

Summary

  • Adds instructions for enabling Markdown for Agents on specific subdomains or paths using Configuration Rules
  • Updates both Dashboard and API tabs in the partial with new subsections

Users can now granularly control where Markdown for Agents is enabled rather than only at the zone level.

@dinasaur404 dinasaur404 requested a review from a team as a code owner February 20, 2026 21:54
@github-actions
Copy link
Contributor

This PR requires additional review attention because it affects the following areas:

Partials

This PR updates partial files, which are pieces of content used across multiple files in our Render component.

@github-actions
Copy link
Contributor

This pull request requires reviews from CODEOWNERS as it changes files that match the following patterns:

Pattern Owners
* @cloudflare/pcx-technical-writing

@dinasaur404
Copy link
Contributor Author

/bonk review and suggest fixes

Copy link
Contributor

@ask-bonk ask-bonk bot left a comment

Choose a reason for hiding this comment

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

Overall this looks good. The content is accurate, headings are correctly nested (H3 under the parent page's H2), links are valid, and the API example is well-structured. A few minor suggestions below.

@ask-bonk
Copy link
Contributor

ask-bonk bot commented Feb 20, 2026

@pedrosousa Bonk workflow was cancelled.

View workflow run · To retry, trigger Bonk again.

@ask-bonk
Copy link
Contributor

ask-bonk bot commented Feb 20, 2026

Review posted successfully. Here's a summary:

Review Summary for PR #28491

Overall: The PR is well-written and ready to merge with minor suggestions. No build breakers or content accuracy issues found.

What looks good:

  • Heading hierarchy is correct (H3 under the parent page's H2 ## How to enable)
  • All internal links are valid (/rules/configuration-rules/, /rules/configuration-rules/create-api/, /ruleset-engine/rules-language/)
  • API example is well-structured with correct endpoint and payload
  • Placeholder style ({zone_id}, {api_token}) is consistent with the existing Custom Hostnames tab
  • MDX syntax is clean — no unescaped special characters in prose
  • "select" used correctly for UI actions, UI elements are bolded

3 suggestions posted (all LOW severity):

  1. Line 20: "Log into" should be "Log in to" per style guide (pre-existing pattern in file, but worth fixing in new content)
  2. Line 23: Minor rewording of the step for selecting Create rule > Configuration Rule for clarity
  3. Line 63: The starts_with(...) expression is already in backticks — this one was a false positive on my part since the expression was already properly formatted in the inline code

Pre-existing issue noted (not flagged on PR): The original API tab uses {zone_tag} while the rest of the file and the broader docs convention use {zone_id}. The new code correctly uses {zone_id} — no action needed in this PR.

github run

@github-actions
Copy link
Contributor

github-actions bot commented Feb 20, 2026

dinasaur404 and others added 2 commits February 20, 2026 17:31
Co-authored-by: ask-bonk[bot] <249159057+ask-bonk[bot]@users.noreply.github.com>
Co-authored-by: ask-bonk[bot] <249159057+ask-bonk[bot]@users.noreply.github.com>
@celso celso merged commit 445c4d7 into production Feb 23, 2026
9 checks passed
@celso celso deleted the dina/markdown-agents-config-rules branch February 23, 2026 15:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants