Skip to content

PM-45227 - Add a Seeder page to the server getting-started section - #896

Merged
theMickster merged 5 commits into
mainfrom
docs-seeder-02
Oct 22, 2026
Merged

theMickster merged 5 commits into
mainfrom
docs-seeder-02

Conversation

@theMickster

@theMickster theMickster commented Oct 21, 2026 •

Copy link
Copy Markdown
Contributor

🎟️ Tracking

PM-45227

📔 Objective

Crafted a new Seeder page to aid in finding the Seeder without searching the server repo. Reference content such as CLI flags and the preset list stays in the server repo per rule 2 of the documentation standard.

🧪 Testing

Expand for lint, spellcheck, build, render, and link checks

Step 1: Lint, spellcheck, build

Command Result
npm run lint All matched files use Prettier code style
npm run spellcheck CSpell: Files checked: 192, Issues found: 0 in 0 files
npm run build [SUCCESS] Generated static files in "build"

Step 2: Render the page

Served the site locally and opened /getting-started/server/seeder with Playwright. The page sits
in the Server sidebar after Manual Setup, the Mermaid flowchart renders with no syntax error, the
perspective caption sits under it, and the inbound links from the Setup Guide and Manual Setup
resolve.

Step 3: Check the outbound links

Link HTTP status
util/SeederUtility/README.md 200
util/SeederApi/README.md 200
util/Seeder/Seeds/docs/presets.md 200
util/Seeder/Seeds/docs/scenarios/README.md 200

A new local database has no accounts, and the contributing site pointed
to the Seeder only from a note inside "Set up Aspire". Manual Setup did
not mention it.

Add Getting started > Server > Seeder: what the Seeder is, a flowchart
for choosing between the Seeder CLI and the Seeder API, the steps to
seed a database, and links to the reference docs in the server repo.
Link to the page from the Setup Guide and Manual Setup after the Api
alive check, and repoint the Aspire note to it.

Reference content such as flags and presets stays in the server repo
per rule 2 of the documentation standard.

Refs: PM-45227
@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Oct 21, 2026 •

Copy link
Copy Markdown

Deploying contributing-docs with  Cloudflare Pages  Cloudflare Pages

Latest commit: f467a12
Status: ✅  Deploy successful!
Preview URL: https://7b2fed18.contributing-docs.pages.dev
Branch Preview URL: https://docs-seeder-02.contributing-docs.pages.dev

View logs

@theMickster
theMickster marked this pull request as ready for review October 21, 2026 08:27
@theMickster
theMickster requested a review from a team as a code owner October 21, 2026 08:27
@theMickster theMickster added ai-review Request a Claude code review documentation Improvements or additions to documentation labels Oct 21, 2026
@github-actions

github-actions Bot commented Oct 21, 2026 •

Copy link
Copy Markdown
Contributor

🤖 Bitwarden Claude Code Review

Overall Assessment: APPROVE

This PR adds docs/getting-started/server/seeder.md, which explains the Seeder CLI and Seeder API entry points and walks through a first seeding run. It also links the new page from the Setup Guide (in the Aspire note and after Start the server) and from Manual Setup. Every in-repo relative link and heading anchor resolves (#azurite, #start-the-server, #configure-user-secrets, #client-connection, manual-setup.md#create-database, database/ef/index.mdx), and sidebar_position: 1.7 places the page between Manual Setup (1.5) and Advanced Setup (2). The latest commit uses "AI coding agent" and drops "Bitwarden developers", which covers the two open nitpick threads, but the links into the bitwarden/server repository, including the seeding-bitwarden-data skill, could not be checked from this review environment.

withinfocus
withinfocus previously approved these changes Oct 21, 2026

@withinfocus withinfocus left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Few ideas for future enhancements but this is a great addition to the setup workflow.

Comment thread docs/getting-started/server/seeder.md Outdated
Comment thread docs/getting-started/server/seeder.md Outdated
Comment thread docs/getting-started/server/seeder.md
Comment thread docs/getting-started/server/seeder.md Outdated
@withinfocus

Copy link
Copy Markdown
Contributor

Still good for merge as I see it.

- Bullet the three Seeder CLI commands, and mark organization and
  individual for use only when no preset fits
- Replace the list, choose, and run steps with one step: ask Claude,
  which uses the seeding-bitwarden-data skill, or find a command in
  the scenarios index
Comment thread docs/getting-started/server/seeder.md Outdated
@theMickster

Copy link
Copy Markdown
Contributor Author

With the merging of server/pull/8678, I think we are ready for a final review here @withinfocus 😁

withinfocus
withinfocus previously approved these changes Oct 22, 2026

@withinfocus withinfocus left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Super hot, but some ⛏️s.

Comment thread docs/getting-started/server/seeder.md Outdated
Comment thread docs/getting-started/server/seeder.md Outdated
Copilot also reads skills from .claude/skills, so the seeding step no longer names Claude. Drop the "Bitwarden developers" sentence from the same step.
@theMickster
theMickster merged commit c11ede6 into main Oct 22, 2026
27 checks passed
@theMickster
theMickster deleted the docs-seeder-02 branch October 22, 2026 19:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ai-review Request a Claude code review documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants