Skip to content

docs: remove Builder from Get started sidebar - #252

Merged
intech merged 2 commits into
mainfrom
docs/remove-get-started-builder
Nov 4, 2026
Merged

intech merged 2 commits into
mainfrom
docs/remove-get-started-builder

Conversation

@intech

@intech intech commented Nov 4, 2026 •

Copy link
Copy Markdown
Contributor

Summary

Remove Service builder from the shared Get started sidebar section. Keep the outlined Builder link in the header and the Builder page unchanged.

Add source and browser regressions for the sidebar removal; retain the existing header Builder checks.

Validation

  • Guides: 93 tests passed; strict navigation types and scoped Biome checks passed.
  • VitePress build, built discovery validation and fragment checks passed.
  • Navigation browser suite passed in both themes, including desktop/mobile header Builder access, compatibility routes and no-JavaScript pages.

No package, API or documentation URL is removed. The existing package README link audit remains applicable; repeat it against the deployed site.

Summary by CodeRabbit

  • Navigation
    • Removed the “Service builder” link from the guide’s “Get started” section. It no longer appears in the desktop sidebar or the guide’s nested navigation.
  • Tests
    • Updated navigation checks to verify that the link is absent.

@intech intech added the type:docs Documentation: guides, README, JSDoc label Nov 4, 2026
@coderabbitai

coderabbitai Bot commented Nov 4, 2026 •

Copy link
Copy Markdown

Explain Change →

No actionable comments were generated in the recent review.

Walkthrough

The Get started guide navigation no longer includes the Service builder link to /en/builder. The browser check and guide navigation test now assert that the link is absent; the test also confirms the get-started section exists.

Changes: Service builder navigation

Layer / File(s) Summary
Remove and verify the builder link
.vitepress/guide/navigation.ts, tests/guide/navigation.test.mjs, scripts/check-navigation-browser.mjs
The Get started section no longer includes the /en/builder link. The guide test and browser check assert its absence. The guide test also confirms that the get-started section exists.

Estimated code review effort: 1 (Trivial) | ~5 minutes

Change: Other

Possibly related issues

  • Connectum-Framework/docs issue 178 — The change removes the Service builder link from the Get started sidebar, but does not implement the issue’s broader onboarding content proposal.
Merge Risk: Minimal · up to 83861

The navigation change is narrowly scoped and covered by source and browser checks.

Pre-merge checks: Passed 5
Check name Status Explanation
Description Check Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check Passed The title clearly and concisely describes the main change: removing Builder from the Get started sidebar while remaining consistent with the navigation updates.
Docstring Coverage Passed No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentat…
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.
Full details: Docstring Coverage

Explanation

No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentation-only edits and deletions within surviving functions are included; unchanged diff context and fully deleted functions are excluded. Analyzed 0 functions across 2 files.

Finishing Touches
Generate docstrings
  • Commit to this branch
  • Create a new PR
Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • 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
Recent review info
Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 048e4968-2d51-43c6-b521-5961a96c85d1
Commits

Reviewing files that changed from the base of the PR and between 484cd17 and 83861ca.

Files selected for processing: 3
  • .vitepress/guide/navigation.ts
  • scripts/check-navigation-browser.mjs
  • tests/guide/navigation.test.mjs
Files with no reviewable changes: 1
  • .vitepress/guide/navigation.ts

Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.

@intech
intech merged commit 94bca45 into main Nov 4, 2026
6 checks passed
@intech
intech deleted the docs/remove-get-started-builder branch November 4, 2026 20:11
@intech

intech commented Nov 4, 2026

Copy link
Copy Markdown
Contributor Author

Publication confirmed by the successful deployment of main at 211b4b3: https://github.com/Connectum-Framework/docs/actions/runs/41909779571. The public-site navigation suite passed: Service builder is absent from Get started, while the header Builder remains available. The preceding Pages lock is resolved.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type:docs Documentation: guides, README, JSDoc

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant