Skip to content

docs(en): polish English prose and fix stale references#418

Merged
haksungjang merged 1 commit into
mainfrom
docs/en-polish
Jul 16, 2026
Merged

docs(en): polish English prose and fix stale references#418
haksungjang merged 1 commit into
mainfrom
docs/en-polish

Conversation

@haksungjang

Copy link
Copy Markdown
Member

Why

A native-quality audit of the 24 English site pages plus the README (through an OWASP-reviewer lens) rated the docs strong overall, with a short list of high-impact findings: two real defects and a set of literal collocations, run-ons, and consistency drifts.

What

Real defects

  • by-input.md: two firmware-guide links were tagged "(Korean)" but point at the English guide — misleading to a reviewer. Tags removed (the ko mirror was verified correct as-is).
  • architecture.md: tool table pinned ScanCode 32.3.0 and Trivy v0.70.0; the Dockerfile ARGs are 32.5.0 / v0.72.0. Pins refreshed.

Prose polish (meaning preserved)

  • Literal renderings replaced: "do not easily abort", "decisively better", "gathered as", "hand … back and forth", "it is a later addition", "identifies components two ways".
  • The two heaviest run-on passages split (ui.md advanced options; reports-explained deps.dev bullet); a paragraph break added in the dense Overview section; the public-repo caveat in identify-vendored.md now starts its own paragraph.
  • Consistency: "quality criteria" throughout supplier-sbom.md (incl. frontmatter), "Quick start" heading, spelled-out first-/third-party in prose, link text aligned with the target page's H1.

Verification

  • tests/check-docs-drift.sh, scripts/check-doc-coverage.sh — pass
  • mkdocs build --strict — exit 0 (renamed heading has zero inbound anchor links, verified repo-wide)

A native-quality audit of the English pages (OWASP-reviewer lens) found
the prose generally strong, with a short list of high-impact issues.
Fix them:

- Real defects: the by-input firmware links were tagged "(Korean)" but
  point at the English guide; architecture.md pinned ScanCode 32.3.0
  and Trivy v0.70.0 while the Dockerfile ARGs say 32.5.0 / v0.72.0.
- Literal or off collocations: "do not easily abort", "decisively
  better", "gathered as", "hand outputs back and forth", "it is a
  later addition", "identifies components two ways".
- Readability: split the two heaviest run-on passages (ui.md advanced
  options, reports-explained deps.dev bullet), new paragraph for the
  public-repo caveat in identify-vendored.md, drop a doubled
  "immediately", fix a heterogeneous "such as" list.
- Consistency: "quality criteria" throughout supplier-sbom.md,
  "Quick start" heading spelling, spelled-out first-/third-party in
  prose, link text aligned with the target page's title.

Korean mirrors are unaffected (EN-only prose changes; the ko firmware
links were verified correct as-is).
@haksungjang
haksungjang merged commit 7481d73 into main Jul 16, 2026
34 checks passed
@haksungjang
haksungjang deleted the docs/en-polish branch July 16, 2026 09:29
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.

1 participant