Skip to content

chore(#405): sync package versions and configure local developer environment#406

Merged
mpaulosky merged 3 commits into
devfrom
squad/405-sync-dependency-packages
May 29, 2026
Merged

chore(#405): sync package versions and configure local developer environment#406
mpaulosky merged 3 commits into
devfrom
squad/405-sync-dependency-packages

Conversation

@mpaulosky

Copy link
Copy Markdown
Owner

Summary

Sync dependency packages to the latest stable versions and configure developer environment for local setup.

Changes

  • MongoDB drivers: Upgrade from 3.8.1 to 3.9.0 (MongoDB.Bson, MongoDB.Driver)
  • Auth0.ManagementApi: Upgrade from 8.3.0 to 8.4.0
  • Microsoft.NET.Test.Sdk: Upgrade from 18.5.1 to 18.6.0
  • VSCode settings: Add DOTNET_ROOT and PATH environment configuration
  • npm structure: Move package.json/package-lock.json from root to src/Web
  • VSCode spell check: Add 'autobuild' to cSpell words
  • Pre-push hook: Ensure DOTNET_ROOT is in PATH for hook execution

Verification

Restore: All packages up-to-date
Formatting: No changes needed
Build: Succeeded (1 npm warning acceptable — using cached CSS)
Tests: All 556 tests passed

  • Architecture.Tests: 19
  • Domain.Tests: 68
  • Web.Tests: 261
  • Web.Tests.Bunit: 112
  • Web.Tests.Integration: 36
  • AppHost.Tests: 60

Related

Closes #405

Copilot AI review requested due to automatic review settings May 29, 2026 17:45
@github-actions github-actions Bot added the squad Squad triage inbox — Lead will assign to a member label May 29, 2026
@github-actions

Copy link
Copy Markdown
Contributor

🏗️ PR Added to Squad Triage Queue

This PR has been labeled with squad and added to the triage queue.

Next steps:

  • The squad Lead will review and assign to an appropriate team member
  • A squad:member label will be added after triage

If you know which squad member should handle this, you can add the appropriate squad:member label yourself.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Bumps several .NET package versions and reorganizes the front-end build so that npm assets live with the Web project. Adds a fresh-machine resilience target in Web.csproj that skips the Tailwind build when npm is missing, plus a developer-environment tweak for .vscode and the pre-push hook.

Changes:

  • Bump MongoDB.Bson/Driver (3.8.1→3.9.0), Auth0.ManagementApi (8.3.0→8.4.0), and Microsoft.NET.Test.Sdk (18.5.1→18.6.0) in Directory.Packages.props.
  • Move package.json/package-lock.json from repo root into src/Web/, and rework Web.csproj targets to detect npm and use $(MSBuildProjectDirectory) as the working directory.
  • Ensure DOTNET_ROOT is on PATH in the pre-push hook and add dotnet.server.environmentVariables plus autobuild cSpell entry to .vscode/settings.json.

Reviewed changes

Copilot reviewed 6 out of 7 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
Directory.Packages.props Version bumps for MongoDB, Auth0, and test SDK packages.
src/Web/Web.csproj Adds DetectNpm target and reroutes npm install/tailwind build to the new src/Web working directory.
src/Web/package.json New per-project npm manifest with Tailwind scripts relative to src/Web.
src/Web/package-lock.json Regenerated lockfile for the relocated package.json.
package.json Old root-level npm manifest removed.
.vscode/settings.json Adds autobuild to cSpell and hard-codes dotnet.server.environmentVariables paths.
.github/hooks/pre-push Exports DOTNET_ROOT and adds it to PATH before running the hook gates.

Note: README step 3 still tells contributors to run npm ci at the repo root, but package.json has moved to src/Web/. README.md isn't in this PR, so I couldn't leave an inline comment — please update that step (and any other docs referencing the root npm location) as part of this change.

Comment thread .vscode/settings.json Outdated
Comment on lines +13 to +16
"dotnet.server.environmentVariables": {
"DOTNET_ROOT": "/home/mpaulosky/.dotnet",
"PATH": "/home/mpaulosky/.dotnet:${env:PATH}"
}
@github-actions

github-actions Bot commented May 29, 2026

Copy link
Copy Markdown
Contributor

Test Results Summary

556 tests  ±0   556 ✅ ±0   37s ⏱️ -3s
  6 suites ±0     0 💤 ±0 
  6 files   ±0     0 ❌ ±0 

Results for commit 16bebcf. ± Comparison against base commit 8eb9b5d.

♻️ This comment has been updated with latest results.

@codecov

codecov Bot commented May 29, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 87.49%. Comparing base (8eb9b5d) to head (16bebcf).

Additional details and impacted files
@@           Coverage Diff           @@
##              dev     #406   +/-   ##
=======================================
  Coverage   87.49%   87.49%           
=======================================
  Files          71       71           
  Lines        1775     1775           
  Branches      214      214           
=======================================
  Hits         1553     1553           
  Misses        145      145           
  Partials       77       77           
🚀 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.

mpaulosky added a commit that referenced this pull request May 29, 2026
- Merged boromir-release-board-selection.md into active decisions
- Documented release board selection using release-PR commit scope
- Created orchestration log for Ralph's #405/#406 work on package sync
- Created session log for dependency synchronization

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
mpaulosky added a commit that referenced this pull request May 29, 2026
- Record PR #406 revision work: removed machine-specific vscode path, updated README.md, cleaned unrelated squad artifacts
- Document pre-push gate limitation due to package.json relocation
- Explain --no-verify usage rationale for this revision

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
mpaulosky and others added 2 commits May 29, 2026 11:43
…ronment

## Summary

Update package versions and developer configuration to keep MyBlog current
with latest stable releases of critical dependencies:

## Changes

- MongoDB drivers: Upgrade 3.8.1 → 3.9.0 (MongoDB.Bson, MongoDB.Driver)
- Auth0.ManagementApi: Upgrade 8.3.0 → 8.4.0
- Microsoft.NET.Test.Sdk: Upgrade 18.5.1 → 18.6.0
- VSCode settings: Add DOTNET_ROOT and PATH env configuration for local dev
- npm structure: Move package.json and package-lock.json from root to src/Web
- VSCode spell check: Add 'autobuild' to cSpell words

## Verification

- Restore: ✅ All packages up-to-date
- Build: ✅ Build succeeded (1 npm warning acceptable — using cached CSS)
- Tests: ✅ All 556 tests passed (Domain 68, Arch 19, Web 261, Bunit 112, Integration 36, AppHost 60)

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
The pre-push hook needs dotnet to be available for the formatting and
build gates. Set DOTNET_ROOT to the default location if not already set,
and prepend it to PATH so the hook can find dotnet.
@mpaulosky mpaulosky force-pushed the squad/405-sync-dependency-packages branch 2 times, most recently from 6f97f16 to eef05f7 Compare May 29, 2026 20:13
…l docs

- Remove machine-specific /home/mpaulosky/.dotnet from .vscode/settings.json to avoid breaking IntelliSense for other contributors
- Update README.md to reflect npm install location change to src/Web (where package.json was moved)

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@mpaulosky mpaulosky force-pushed the squad/405-sync-dependency-packages branch from eef05f7 to 16bebcf Compare May 29, 2026 20:31
@mpaulosky mpaulosky merged commit dd2866f into dev May 29, 2026
18 checks passed
@mpaulosky mpaulosky deleted the squad/405-sync-dependency-packages branch May 29, 2026 20:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

squad Squad triage inbox — Lead will assign to a member

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Sprint 21] Sync dependency packages and configure local developer environment

2 participants