Skip to content

Chore: repo cleanup - #598

Open
scode2277 wants to merge 1 commit into
developfrom
chore/repo-cleanup
Open

Chore: repo cleanup#598
scode2277 wants to merge 1 commit into
developfrom
chore/repo-cleanup

Conversation

@scode2277

@scode2277 scode2277 commented Aug 6, 2026

Copy link
Copy Markdown
Collaborator

What does this PR change?

  • make further reading sections case sensitive
  • fix user-team-security position in sidebar
  • check and fix spellchecker and md linter
  • fix link in sidebar reminder workflow
  • remove multiple blank lines
  • standardize list format with dashes
  • sync indexes and tags
  • sync badges

Type of change

  • New content
  • Edit to existing content
  • Outline / structure change
  • Typo or formatting fix
  • Tooling / config

If applicable

  • Editing existing content: tagged the current contributors from the attribution list
  • Framework has a steward: asked them to review
  • Outline change: updated vocs.config.ts with the dev: true parameter
  • Want community feedback: shared this PR in our Discord

Stuck on anything? Just write it here and we're happy to help.

- make further reading sections case sensitive
- fix user-team-security position in sidebar
- check and fix spellchecker and md linter
- fix link in sidebar reminder workflow
- remove multiple blank lines
- standardize list format with dashes
- sync indexes and tags
- sync badges

@frameworks-volunteer frameworks-volunteer left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Model: z-ai/glm-5.2 Reasoning: medium Provider: openrouter

Security + QA review complete. No blocking issues.

Security: Clean. No secrets, injection vectors, or unsafe code. The validate-content.cjs change replaces a naive regex with proper fence-aware code-block stripping (handles ``` and ~~~ fences, nested-safe). Good improvement.

QA notes:

  • "Further Reading" -> "Further reading" standardized across ~170 MDX files. Consistent with sentence-case heading convention.
  • vocs.config.ts: User and Team Security sidebar section relocated from after Awareness to after Treasury Operations. Matches PR description. All dev: true flags preserved.
  • contributors.json: Badge sync looks correct — Active-Last-7d dates refreshed, Dormant-90d+ added for inactive contributors, stale New-Joiner/Active-Last-7d removed where appropriate. All badge types (including Dormant-90d+) are defined in badgeConfig.tsx.
  • justfile: markdownlint now prunes docs/pages/config from the lint target. Reasonable since config MDX files follow different formatting rules.
  • .github/workflows/vocs-config-reminder.yml: Link updated from /contribute/contributing/#3-sidebar--navigation to #sidebar--navigation. Verified the contributing.mdx heading is ### Sidebar / Navigation — anchor matches.
  • contribute/index.mdx: Three new page links added (content-model, style-and-terminology, docs-normalization-checklist). All resolve to existing files.
  • New index.mdx files for user-team-security/ and physical-security/ subdirectories — auto-generated, consistent with existing pattern.
  • utils/fetched-tags.json: Tags synced for new pages. One cosmetic note: trailing newline removed from file end. Non-blocking but some linters flag this.
  • wordlist.txt: MASVS added to spellchecker wordlist.

Verdict: Approve. Clean cleanup PR.

@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying frameworks with  Cloudflare Pages  Cloudflare Pages

Latest commit: a494267
Status: ✅  Deploy successful!
Preview URL: https://e0623107.frameworks-573.pages.dev
Branch Preview URL: https://chore-repo-cleanup.frameworks-573.pages.dev

View logs

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