chore: remove CI-enforced checks from Flue style review#31287
Conversation
|
This pull request requires reviews from CODEOWNERS as it changes files that match the following patterns:
|
Review⏸️ Automatic reviews for this PR are paused. This PR has already received 2 automatic reviews. To run another review, a codeowner can comment
✅ No style-guide issues found in commit CommandsOnly codeowners can run commands. Post a comment with the command to trigger it.
|
|
One minor suggestion posted.
|
|
PR #31287 reviewed. Labels applied: Summary: One issue found:
An inline suggestion has been posted on the PR to address this. |
|
/bonk review |
|
Review posted on PR #31287. I reviewed the current diff (commit |
|
/review |
|
Preview URL: https://d9a5f83c.preview.developers.cloudflare.com |
Summary
Removes Flue style-guide review rules that duplicate issues already caught by failing CI/build steps. This keeps agent review focused on style and conventions that CI does not reliably enforce.
Removed checks include MDX syntax parse failures, missing component imports or import placement failures, invalid
DashButtonroutes, schema/build-validated required component props, and schema-validated frontmatter fields such as requiredtitleand numericsidebar.order.Kept checks include style-only link wording/path conventions, image parent-directory guidance,
pcx_content_type/description expectations not currently schema-enforced, and component usage preferences such asTypeScriptExample,PackageManagers, andDashButtonfor dashboard navigation.Also adds
.flue/AGENTS.mdwith the policy for future review rules: do not add agent findings for issues already reliably caught by CI.