Skip to content

feat(docs): move the docs under current/ with URLs unchanged - #383

Merged
scttbnsn merged 7 commits into
dev/v0.9from
feat/docs-current-dir
Oct 9, 2026
Merged

scttbnsn merged 7 commits into
dev/v0.9from
feat/docs-current-dir

Conversation

@scttbnsn

@scttbnsn scttbnsn commented Oct 9, 2026

Copy link
Copy Markdown
Contributor

First step of the versioned docs archive for v1.0. The docs content moves under docs/content/docs/current/ so an archived release line can sit beside it later as vX.Y/. No public URL changes.

What changed

  • The 17 pages and meta.json move to docs/content/docs/current/. That commit is a pure rename.
  • docs/src/lib/docs-versions.ts gives the Fumadocs loader a slugs function that drops the current segment and takes the rest from Fumadocs' own getSlugs. A future vX.Y/ directory keeps its segment and serves at /docs/vX.Y/....
  • The sidebar version tab is hidden while there's only one version. It comes back by itself when a second root exists.
  • The three contract scripts that pin doc paths point at current/, and so do six prose and comment references.
  • ROADMAP.md now says what v1.0 does: archive the v0.9 docs from the final v0.9.x tag.

The docs site is a static export, so there are no redirects to lean on. The unversioned URLs are also printed by the binary and the package metadata. That's why current stays at /docs/<page> instead of moving to a versioned path the way drydock does.

New checks

  • package-release-config-test.sh requires current/meta.json's title to equal the release's v<major>.<minor>, with a mutation in the meta-test.
  • scripts/docs-versions.test.mjs: no page outside current/ or vX.Y/, the sitemap and the analytics allowlist each equal the current page set, and every root-relative link resolves inside its own version directory.
  • scripts/docs-export-contract.mjs runs in check:web after the build and requires the exported HTML pages to equal the pages in current/, in both directions, with an index page.

Verification

  • The list of exported docs/out/**/*.html is identical before and after the move (19 files), and the visible text of index.html and authentication.html is unchanged.
  • The export contract fails on a planted docs/out/current/x.html and on a missing index.html.
  • Web tests, lint, knip, the docs type check, the page-weight budget and every scripts/*-test.sh pass. Full pre-push hook passed.

One behaviour differs from the Fumadocs default and is noted in the code: a page next to a same-named folder index fails the build as a duplicate slug. Nothing in the tree is laid out that way.

@coderabbitai

coderabbitai Bot commented Oct 9, 2026

Copy link
Copy Markdown

Warning

Review limit reached

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Next included review available in 43 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

Learn how review limits work.

Review configuration:

⚙️ Run configuration
  • Configuration used: Repository: CodesWhat/portwing/.coderabbit.yaml
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 94281a66-f4da-4a53-94a4-ad9cf1f77a74

📥 Commits

Reviewing files that changed from the base of the PR and between 0c13446 and 3475c7c.


📒 Files selected for processing (34)
  • .github/workflows/quality-integration-engines.yml
  • COMPATIBILITY.md
  • ROADMAP.md
  • SECURITY-ASSURANCE.md
  • cmd/portwing/main.go
  • docs/content/docs/current/api-reference.mdx
  • docs/content/docs/current/audit-logging.mdx
  • docs/content/docs/current/authentication.mdx
  • docs/content/docs/current/competitive-landscape.mdx
  • docs/content/docs/current/configuration.mdx
  • docs/content/docs/current/connection-modes.mdx
  • docs/content/docs/current/drydock-integration.mdx
  • docs/content/docs/current/getting-started.mdx
  • docs/content/docs/current/index.mdx
  • docs/content/docs/current/installation.mdx
  • docs/content/docs/current/mcp-server.mdx
  • docs/content/docs/current/meta.json
  • docs/content/docs/current/migrating-from-watchtower.mdx
  • docs/content/docs/current/observability.mdx
  • docs/content/docs/current/security-model.mdx
  • docs/content/docs/current/stability-policy.mdx
  • docs/content/docs/current/standalone-mode.mdx
  • docs/content/docs/current/verification.mdx
  • docs/content/docs/meta.json
  • docs/src/app/layout.tsx
  • docs/src/lib/docs-versions.ts
  • docs/src/lib/source.ts
  • package.json
  • scripts/docs-export-contract.mjs
  • scripts/docs-versions.test.mjs
  • scripts/package-release-config-test-test.sh
  • scripts/package-release-config-test.sh
  • scripts/standard-mode-bind-config-test.sh
  • website/src/app/sitemap.ts


  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@codecov

codecov Bot commented Oct 9, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@biggest-littlest biggest-littlest left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Docs content moved under current/ with URLs unchanged. 14 required checks and Web Contract green on 3475c7c. Codex reviewed the diff: no blockers, two should-fix findings, both fixed in 582fa6d2 and rechecked. CodeRabbit was out of free reviews.

@ALARGECOMPANY ALARGECOMPANY left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Docs content moved under current/ with URLs unchanged. 14 required checks and Web Contract green on 3475c7c. Codex reviewed the diff: no blockers, two should-fix findings, both fixed in 582fa6d2 and rechecked. CodeRabbit was out of free reviews.

@scttbnsn
scttbnsn merged commit 868ab70 into dev/v0.9 Oct 9, 2026
42 checks passed
@scttbnsn
scttbnsn deleted the feat/docs-current-dir branch October 9, 2026 17:46
@scttbnsn scttbnsn mentioned this pull request Oct 9, 2026
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.

3 participants