docs(physical-security): normalize structure and wording - #588
Open
mattaereal wants to merge 6 commits into
Open
docs(physical-security): normalize structure and wording#588mattaereal wants to merge 6 commits into
mattaereal wants to merge 6 commits into
Conversation
Canonical Key Takeaways, description band, Related frameworks on overviews, checklist takeaway, line-length and chrome cleanup across the framework and coercion & duress subtree. Stub overviews preserved.
Deploying frameworks with
|
| Latest commit: |
d18ea61
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://fbf54ddd.frameworks-573.pages.dev |
| Branch Preview URL: | https://docs-normalize-physical-secu.frameworks-573.pages.dev |
Child pages missing a canonical further/related H2 failed the updated #561 validator. Add a minimal Further Reading block linking the framework overview.
Neutralize fence comments that look like headings, canonicalize overview page-map titles, clamp descriptions, demote excess H2s, and mark template pages with content-model exceptions.
frameworks-volunteer
approved these changes
Aug 5, 2026
frameworks-volunteer
left a comment
frameworks-volunteer
left a comment
Collaborator
There was a problem hiding this comment.
Model: z-ai/glm-5.2 Reasoning: medium Provider: openrouter
Reviewed all 14 changed MDX files in docs/pages/physical-security/**.
Security review: No issues. No secrets, no injection vectors, no unsafe code. The "secrets" mentions are security guidance prose, not code.
QA review:
- All 14 files are editorial/structural normalization matching the SEAL content model
- Key Takeaway format unified to
> 🔑 **Key Takeaway**:across all pages - Prose re-wrapped for MD013 (line length); no substantive guidance changed
- "Further Reading" sections added to all sub-pages with valid internal links
- "Related frameworks" cross-links added to root overview — all 5 links verified to existing files
- Stub pages retain
⚠️ Stub/in progressnotices and usecontent-model-exceptioncomments to defer page maps - Stub boundary markers (
<!-- stub-boundary -->) cleanly separate stub content from substantive KT - Description fields tightened; all remain accurate and within length band
- No debug content, TODOs, or placeholder text
- No frontmatter contributors changes (correct — editorial only)
No blocking issues. Approving.
Sidebar Configuration ReminderDocumentation files update: New in this push:
Please ensure that:
See Contributing Guide – Sidebar & Navigation for more details. This is an automated reminder. If this PR doesn't need sidebar changes, you can ignore this message. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Scope
Normalize
docs/pages/physical-security/**(14 hand-authored pages).Why
Match the SEAL content model used by other framework PRs.
Content model
> 🔑 **Key Takeaway**:on every page (added for checklist)Changes
Substantive security
None — editorial/structural only. Control recommendations preserved.
Intentionally unchanged
vocs.config.tsalready complete; WIPdev: truestubs unchanged)Validation
npx markdownlint-cli2 "docs/pages/physical-security/**/*.mdx"— 0 issuespnpm exec cspell "docs/pages/physical-security/**/*.mdx"— 0 issuesDependencies
Depends on (by reference, unmerged): #561 — Content normalization standard.
Reviewer focus