Skip to content

Modernize Beam to .NET 10 and fix dev port - #28

Merged
Dedac merged 2 commits into
mainfrom
modernize-dotnet10-and-fix-dev-port
Sep 16, 2026
Merged

Dedac merged 2 commits into
mainfrom
modernize-dotnet10-and-fix-dev-port

Conversation

@Dedac

@Dedac Dedac commented Sep 16, 2026

Copy link
Copy Markdown
Owner

Summary

  • upgrade Beam projects, packages, SDK tooling, and dev container to .NET 10
  • improve development database connection handling with local certificate trust
  • move the development server and dev-container forwarding from port 5000 to 5080 to avoid the macOS AirPlay conflict
  • update startup configuration and add the devcontainer lock file

Validation

  • dotnet restore Beam.Server/Beam.Server.csproj
  • dotnet build Beam.Server/Beam.Server.csproj --no-restore
  • verified http://localhost:5080/ returns 200 OK and serves the Blazor page

Closes #N/A

Upgrade the solution, tooling, and dev container to .NET 10, improve development database configuration, and move the development server away from macOS AirPlay's port 5000 conflict.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 279b2234-e59e-4a47-8f00-af458d7e565b

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.

Copilot review overview

🟡 Changes recommended

The dev-container image setup is incompatible with its new base, and VS Code debugging still defaults to port 5000.

Get a fresh assessment by requesting another Copilot review.

Review tier: Balanced
Findings: 1 High severity · 2 Medium severity

Open (3)
What changed in this PR

Modernizes Beam to .NET 10, updates development startup behavior, and moves local hosting to port 5080.

Changes:

  • Upgrades frameworks, SDK tooling, and packages to .NET 10.
  • Updates database certificate handling and dev-container initialization.
  • Changes development hosting and forwarding to port 5080.
File Description
global.json Pins the .NET 10 SDK.
Beam.Shared/​Beam.Shared.csproj Targets .NET 10.
Beam.Server/​Properties/​launchSettings.json Moves server profiles to port 5080.
Beam.Server/​Program.cs Adds development certificate trust handling.
Beam.Server/​Beam.Server.csproj Upgrades server dependencies.
Beam.Data/​Beam.Data.csproj Upgrades EF Core and target framework.
Beam.Client/​Beam.Client.csproj Upgrades Blazor client dependencies.
Beam.Animation/​Beam.Animation.csproj Upgrades Razor component dependencies.
.vscode/​launch.json Updates the debugger output path.
.devcontainer/​on-create.sh Hardens container initialization.
.devcontainer/​Dockerfile Uses the .NET 10 SDK image.
.devcontainer/​docker-compose.yml Updates container naming and workspace mounting.
.devcontainer/​devcontainer.json Forwards port 5080.
.devcontainer/​devcontainer-lock.json Locks dev-container features.
.config/​dotnet-tools.json Upgrades the EF CLI tool.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread .devcontainer/Dockerfile
Comment thread .devcontainer/on-create.sh Outdated
Comment thread .vscode/launch.json
Remove obsolete SQL tooling setup, retry migrations while SQL starts, and bind VS Code debugging to port 5080.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 279b2234-e59e-4a47-8f00-af458d7e565b

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.

Copilot review overview

🔵 Needs a closer look

The hard-coded workspace mount conflicts with the dynamic workspace path when the checkout directory has another name.

Review tier: Balanced
Findings: None

Resolved since last review (3)
Previously missed (1)

In code that hasn't changed since last review

Medium severity Keep the mounted and configured workspace paths aligned

.devcontainer/​docker-compose.yml:17

This mount target only matches workspaceFolder while the checkout directory is literally named Beam. If a contributor clones the repository under another directory name, workspaceFolder and ${containerWorkspaceFolder} resolve to /workspaces/<actual-basename>, but the source is mounted at /workspaces/Beam; VS Code then opens an unmounted path and the on-create script cannot restore the repository. Use the same fixed path in both files, or make both paths derive from one shared value.

@Dedac
Dedac merged commit 0f74ca6 into main Sep 16, 2026
1 check passed
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