Skip to content

fix: lint multiline MDX tags - #1837

Open
ayaanoncrypto wants to merge 1 commit into
base:masterfrom
ayaanoncrypto:fix/mdx-lint-multiline-tags
Open

fix: lint multiline MDX tags#1837
ayaanoncrypto wants to merge 1 commit into
base:masterfrom
ayaanoncrypto:fix/mdx-lint-multiline-tags

Conversation

@ayaanoncrypto

Copy link
Copy Markdown

Addresses issues #1806, #1795, and #1792.

What changed:

  • Exclude the asset-only static directory from generated documentation indexes.
    • Update the MDX linter to inspect component attributes and image alt attributes when tags span multiple lines.
    • Regenerate docs/AGENTS.md after the index change.
      Why:
      The previous checks missed invalid metadata when a component tag or image attribute crossed a line boundary. The generated index also included a directory without documentation pages.

Testing:

  • Node syntax checks passed.
    • Targeted multiline MDX fixture detected the missing Card title and missing image alt while accepting valid multiline tags.
    • git diff --check passed.
      The full existing repository lint baseline reports 1,447 errors and 74 warnings, so validation focused on the changed logic and a targeted regression fixture.

@cb-heimdall

Copy link
Copy Markdown
Collaborator

🟡 Heimdall Review Status

Requirement Status More Info
Reviews 🟡 0/2
Emergency override progress: 0/5
These approvals do not satisfy normal or CODEOWNER requirements because the commit identity is unverified.
Denominator calculation
Show calculation
1 if user is bot 0
1 if user is external 0
2 if repo is sensitive 0
From .codeflow.yml 1
Additional review requirements
Show calculation
Max 0
0
From CODEOWNERS 0
Global minimum 0
Max 1
1
1 if commit is unverified 1
Sum 2

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.

2 participants