Skip to content

docs: fix broken links, restore missing Development page, drop duplicate copies - #2370

Open
Suneha Bose (bosesuneha) wants to merge 2 commits into
Azure:mainfrom
bosesuneha:docs/checks-and-structural-fixes
Open

docs: fix broken links, restore missing Development page, drop duplicate copies#2370
Suneha Bose (bosesuneha) wants to merge 2 commits into
Azure:mainfrom
bosesuneha:docs/checks-and-structural-fixes

Conversation

@bosesuneha

Copy link
Copy Markdown
Member

Summary

Fixes broken links and a blank published page in the docs, and removes three stale duplicate copies of pages that already live under docs/book/src/.

Changes

Missing Development page

SUMMARY.md linked to ./development.md, which did not exist. mdBook's create-missing default silently generated an empty stub, so the published Development section rendered as a bare heading. Adds the page and sets create-missing = false so this fails the build instead.

Duplicate copies removed

Three files under docs/ were outdated forks of their docs/book/src/ counterparts, and docs/README.md linked only to the forks:

Deleted Superseded by Missing from the fork
docs/webview-development.md book/src/development/webview-development.md Custom UI Elements section
docs/package-scripts.md book/src/development/development.md Local VSIX Sharing section
docs/maintenance/README.md book/src/release/releasing.md Pinned third-party versions section; last updated 2021

docs/README.md is rewritten as an index into book/.

.agents/skills/vscode-aks-tools-release-pr/SKILL.md instructed maintainers to mirror every releasing.md edit into docs/maintenance/README.md, which is why that fork stayed alive. Updated to name book/src/ as the single source of truth.

Broken links and stale references

  • ../package.json and ../webview-ui/ resolved outside the book
  • dev-webview / build-webview are not real npm scripts (correct names: dev:webview, build:webview)
  • webpack-ui typo for webview-ui
  • SUMMARY.md nav labels disagreed with the pages' own titles

Other

  • bin/ added to .gitignore. make -C docs/book build writes mdBook binaries to the repo root because the Makefile uses $(PWD) rather than $(CURDIR).

Verification

  • mdbook build exits 0 with create-missing = false
  • lychee --offline --include-fragments over docs/**/*.md: 0 errors
  • Development page renders real content rather than a bare heading

…ate copies

The published Development section rendered as a bare heading because
SUMMARY.md linked to ./development.md, which did not exist. mdBook's
create-missing default silently generated an empty stub instead of failing,
so this went unnoticed. Add the page and set create-missing = false.

Three pages under docs/ were stale forks of their docs/book/src/
counterparts, and docs/README.md linked only to the forks:

  docs/webview-development.md  missing the Custom UI Elements section
  docs/package-scripts.md      missing the Local VSIX Sharing section
  docs/maintenance/README.md   last updated 2021; missing the pinned
                               third-party versions section entirely

Delete them and rewrite docs/README.md as an index into book/.
The release skill instructed maintainers to mirror releasing.md into
docs/maintenance/README.md, which is why that fork stayed alive; update it
to name book/src/ as the single source of truth.

Also fixed:
- ../package.json and ../webview-ui/ links resolved outside the book
- dev-webview / build-webview are not real npm scripts (dev:webview,
  build:webview)
- webpack-ui typo for webview-ui
- SUMMARY.md nav labels disagreed with the pages' own titles
- bin/ ignored; make -C docs/book build writes mdBook binaries to the repo
  root because the Makefile uses $(PWD) rather than $(CURDIR)
Clarify the documentation policy regarding the release process and the changelog.
@bosesuneha
Suneha Bose (bosesuneha) marked this pull request as ready for review August 7, 2026 20:37

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.

lgtm

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