Skip to content

Add seed script for local development#7490

Merged
Hinton merged 3 commits intomainfrom
seed-local-env
Apr 21, 2026
Merged

Add seed script for local development#7490
Hinton merged 3 commits intomainfrom
seed-local-env

Conversation

@Hinton
Copy link
Copy Markdown
Member

@Hinton Hinton commented Apr 17, 2026

🎟️ Tracking

📔 Objective

Introduces a repeatable, declarative way to seed a local dev environment via the existing SeederUtility.

  • dev/seed.ps1 — new script that reads dev/seeds.json (and optional, gitignored dev/seeds.local.json for per-developer overrides), converts each entry's args from camelCase to kebab-case CLI flags,
    and invokes dotnet run --project util/SeederUtility once per seed. Supports -DryRun to preview the commands without executing. Fails fast on non-zero exit codes.
  • dev/seeds.json — default seed set (two individual users + the qa.enterprise-basic preset) so new contributors get a consistent baseline dataset.
  • dev/.gitignore — ignores seeds.local.json so per-developer overrides don't leak into the repo.
  • IndividualUserOptions / IndividualArgs / RecipeOrchestrator — adds an optional --email flag to the individual command, falling back to the existing firstName.lastName@individual.example
    pattern when omitted. This lets seeds.json pin deterministic emails (e.g. jane@bitwarden.pw) instead of relying on the generated default.

📸 Screenshots

@Hinton Hinton requested a review from theMickster April 17, 2026 13:29
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 17, 2026

Logo
Checkmarx One – Scan Summary & Details2807b73b-6ae9-41a6-aad2-41cfb07a8b69


New Issues (1) Checkmarx found the following issues in this Pull Request
# Severity Issue Source File / Package Checkmarx Insight
1 MEDIUM CSRF src/Api/Auth/Controllers/AccountsController.cs: 452
detailsMethod at line 452 of /src/Api/Auth/Controllers/AccountsController.cs gets a parameter from a user request from model. This parameter value flow...
Attack Vector

Copy link
Copy Markdown
Contributor

@theMickster theMickster left a comment

Choose a reason for hiding this comment

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

The domain name in the seeds.json is the only thing that I think must be fixed.

Comment thread dev/seeds.json
Comment thread dev/seed.ps1
Comment thread dev/.gitignore
@Hinton Hinton requested a review from theMickster April 20, 2026 14:40
@theMickster
Copy link
Copy Markdown
Contributor

@Hinton Will you please take a peek at the workflow failures?
#7490

@sonarqubecloud
Copy link
Copy Markdown

@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 20, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 59.10%. Comparing base (94f2bc3) to head (3ab916a).
⚠️ Report is 6 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff             @@
##             main    #7490       +/-   ##
===========================================
+ Coverage   14.39%   59.10%   +44.71%     
===========================================
  Files        1278     2078      +800     
  Lines       55352    91705    +36353     
  Branches     4270     8152     +3882     
===========================================
+ Hits         7966    54201    +46235     
+ Misses      47246    35572    -11674     
- Partials      140     1932     +1792     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@Hinton Hinton merged commit 076804f into main Apr 21, 2026
43 checks passed
@Hinton Hinton deleted the seed-local-env branch April 21, 2026 10:27
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