[Sprint 19] Restore markdown editor compile path by mpaulosky · Pull Request #329 · mpaulosky/MyBlog

mpaulosky · 2026-05-13T06:33:28Z

Summary

Closes #323

Restores the compile path for the markdown editor by adding the two missing NuGet package references and defining the previously-undefined IFileStorage infrastructure layer that TextEditor.razor depends on.

Changes

Package references (Web.csproj)

Added PSC.Blazor.Components.MarkdownEditor (centrally pinned at 10.0.7)
Added Markdig (centrally pinned at 1.2.0)

New infrastructure layer (`Infrastructure/FileStorage/`)

IFileStorage — internal interface with AddFileAsync(FileData)
FileStorageModels — FileData and FileMetaData records
LocalDiskFileStorage — baseline impl saving to wwwroot/uploads; validates extension whitelist and 10 MB size cap
FileStorageExtensions — AddFileStorage() DI extension following project conventions

Wiring

GlobalUsings.cs — global using MyBlog.Web.Infrastructure.FileStorage;
Program.cs — builder.Services.AddFileStorage();
TextEditor.razor — added @using for new namespace; renamed AddFile → AddFileAsync

JS helpers (from #321 investigation)

wwwroot/js/markdown-editor-init.js
wwwroot/js/text-editor-helpers.js

Verification

Release build: 0 errors, 0 warnings
Tests: Architecture 16 / Domain 42 / Web.Tests 158 / bUnit 92 — all pass
All 6 pre-push gates passed (including integration tests)

Notes

Full Create/Edit page migration is deferred to [Sprint 19] Create flow markdown authoring slice #324 and [Sprint 19] Edit flow markdown authoring slice #325, now unblocked by this fix.
Working as Sam (Implementation)

github-actions · 2026-05-13T06:33:36Z

🏗️ 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

Pull request overview

Restores a baseline markdown editor integration path in the Web project by adding required package references, introducing a minimal file-storage abstraction for image uploads, and adding supporting JS interop helpers.

Changes:

Added Web package references for Markdig and PSC.Blazor.Components.MarkdownEditor.
Introduced Infrastructure/FileStorage with IFileStorage + LocalDiskFileStorage and DI registration.
Added a shared TextEditor component and JS helper scripts intended to support editor initialization and selection handling.

Reviewed changes

Copilot reviewed 10 out of 11 changed files in this pull request and generated 9 comments.

Show a summary per file

File	Description
src/Web/wwwroot/js/text-editor-helpers.js	Adds a JS helper (`getSelectedText`) intended for toolbar actions.
src/Web/wwwroot/js/markdown-editor-init.js	Adds a custom EasyMDE initialization/disposal wrapper for interop.
src/Web/Web.csproj	Adds new PackageReferences for markdown/editor dependencies.
src/Web/Program.cs	Registers file storage in DI (`AddFileStorage`).
src/Web/Infrastructure/FileStorage/LocalDiskFileStorage.cs	Implements local-disk storage to `wwwroot/uploads` with basic validation and logging.
src/Web/Infrastructure/FileStorage/IFileStorage.cs	Defines the storage abstraction used by the editor component.
src/Web/Infrastructure/FileStorage/FileStorageModels.cs	Adds `FileData` / `FileMetaData` records for storage operations.
src/Web/Infrastructure/FileStorage/FileStorageExtensions.cs	Adds DI extension method to register the storage implementation.
src/Web/GlobalUsings.cs	Adds a global using for the new file storage namespace.
src/Web/Components/Shared/TextEditor.razor	Adds MarkdownEditor-based component with image upload + custom button handler.
src/AppHost/MongoDbResourceBuilderExtensions.cs	Minor formatting adjustment in seed document construction.

+    <PackageReference Include="Markdig" />
    <PackageReference Include="MediatR" />
    <PackageReference Include="Microsoft.Bcl.AsyncInterfaces" />
    <PackageReference Include="MongoDB.EntityFrameworkCore" />
+    <PackageReference Include="PSC.Blazor.Components.MarkdownEditor" />


+
+		// Persist the file and build a retrieval URL
+		var fileName = await FileStorage.AddFileAsync(new FileData(stream, new FileMetaData(file.Name, string.Empty, file.LastModified)));
+		file.UploadUrl = new Uri(new Uri(NavigationManager.BaseUri), $"/api/files/{fileName}").ToString();


+		// Get the selected text using JavaScript
+		var selectedText = await JsRuntime.InvokeAsync<string>("getSelectedText", ".EasyMDEContainer textarea");
+


+			var wrappedText = $"<div style='text-align:{args.Value}'>{selectedText}</div>";
+
+			// Replace the selected text in the editor
+			MyContent = MyContent.Replace(selectedText, wrappedText);
+


+	public async Task<FileEntry> HandleImageUpload(MarkdownEditor editor, FileEntry file)
+	{
+		Console.WriteLine($"Image: {file.Name} Size: {file.Size}");
+


+		// Convert the base64 string to a byte array and load it into a memory stream
+		var bytes = Convert.FromBase64String(file.ContentBase64);
+		var stream = new MemoryStream(bytes);
+
+		// Persist the file and build a retrieval URL
+		var fileName = await FileStorage.AddFileAsync(new FileData(stream, new FileMetaData(file.Name, string.Empty, file.LastModified)));
+		file.UploadUrl = new Uri(new Uri(NavigationManager.BaseUri), $"/api/files/{fileName}").ToString();


+{
+	/// <summary>
+	/// Persists <paramref name="file"/> and returns the stored filename that callers
+	/// can use to build a retrieval URL (e.g. <c>/api/files/{filename}</c>).


+		var uniqueName = $"{Guid.NewGuid()}{extension}";
+		var filePath = Path.Combine(uploadsPath, uniqueName);
+
+		await using var fs = new FileStream(filePath, FileMode.Create, FileAccess.Write);


+
+// Ensure the interop is available globally
+console.log('Markdown Editor Interop Loaded');


github-actions · 2026-05-13T15:20:45Z

Test Results Summary

378 tests ±0 377 ✅ ±0 20s ⏱️ ±0s
6 suites ±0 1 💤 ±0
6 files ±0 0 ❌ ±0

Results for commit 13b7269. ± Comparison against base commit 6d9ffd0.

♻️ This comment has been updated with latest results.

codecov · 2026-05-13T15:21:15Z

Codecov Report

❌ Patch coverage is 0% with 55 lines in your changes missing coverage. Please review.
✅ Project coverage is 80.64%. Comparing base (6d9ffd0) to head (13b7269).

Files with missing lines	Patch %	Lines
src/Web/Components/Shared/TextEditor.razor	0.00%	26 Missing and 2 partials ⚠️
...Infrastructure/FileStorage/LocalDiskFileStorage.cs	0.00%	20 Missing and 3 partials ⚠️
...nfrastructure/FileStorage/FileStorageExtensions.cs	0.00%	2 Missing ⚠️
...eb/Infrastructure/FileStorage/FileStorageModels.cs	0.00%	2 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##              dev     #329      +/-   ##
==========================================
- Coverage   84.01%   80.64%   -3.38%     
==========================================
  Files          47       51       +4     
  Lines        1314     1369      +55     
  Branches      161      166       +5     
==========================================
  Hits         1104     1104              
- Misses        148      198      +50     
- Partials       62       67       +5

Files with missing lines	Coverage Δ
...nfrastructure/FileStorage/FileStorageExtensions.cs	`0.00% <0.00%> (ø)`
...eb/Infrastructure/FileStorage/FileStorageModels.cs	`0.00% <0.00%> (ø)`
...Infrastructure/FileStorage/LocalDiskFileStorage.cs	`0.00% <0.00%> (ø)`
src/Web/Components/Shared/TextEditor.razor	`0.00% <0.00%> (ø)`

🚀 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 · 2026-05-13T16:55:19Z

🤖 Aragorn — Rebase complete

Branch squad/323-restore-markdown-editor-compile-path rebased onto origin/dev. Resolved Web.csproj conflict (kept PSC/Markdig, dropped RTBlazorfied). Added Directory.Packages.props version pins for Markdig v0.44.0 + PSC.Blazor.Components.MarkdownEditor v10.0.7. All pre-push gates passed ✅

@using

- Add PSC.Blazor.Components.MarkdownEditor and Markdig package references to Web.csproj (both centrally pinned in Directory.Packages.props) - Add IFileStorage infrastructure (interface, FileData/FileMetaData models, LocalDiskFileStorage impl, DI extension) under Infrastructure/FileStorage/ - Register IFileStorage via AddFileStorage() in Program.cs - Fix TextEditor.razor: add @using for Infrastructure.FileStorage namespace, rename AddFile -> AddFileAsync to match the new interface - Add GlobalUsings entry for MyBlog.Web.Infrastructure.FileStorage - Include JS helpers (markdown-editor-init.js, text-editor-helpers.js) Build: Release 0 errors Tests: Architecture 16 / Domain 42 / Web.Tests 158 / bUnit 92 - all pass Closes #323 Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

mpaulosky · 2026-05-13T17:24:29Z

🔁 Aragorn restack — rebased onto dev (6d9ffd0, post-#328). No conflicts. Force-pushed with --force-with-lease. Branch now at 13b7269.

## Summary Closes #323 Restores the compile path for the markdown editor by adding the two missing NuGet package references and defining the previously-undefined `IFileStorage` infrastructure layer that `TextEditor.razor` depends on. ## Changes ### Package references (Web.csproj) - Added `PSC.Blazor.Components.MarkdownEditor` (centrally pinned at 10.0.7) - Added `Markdig` (centrally pinned at 1.2.0) ### New infrastructure layer (`Infrastructure/FileStorage/`) - `IFileStorage` — internal interface with `AddFileAsync(FileData)` - `FileStorageModels` — `FileData` and `FileMetaData` records - `LocalDiskFileStorage` — baseline impl saving to `wwwroot/uploads`; validates extension whitelist and 10 MB size cap - `FileStorageExtensions` — `AddFileStorage()` DI extension following project conventions ### Wiring - `GlobalUsings.cs` — `global using MyBlog.Web.Infrastructure.FileStorage;` - `Program.cs` — `builder.Services.AddFileStorage();` - `TextEditor.razor` — added `@using` for new namespace; renamed `AddFile` → `AddFileAsync` ### JS helpers (from #321 investigation) - `wwwroot/js/markdown-editor-init.js` - `wwwroot/js/text-editor-helpers.js` ## Verification - Release build: **0 errors, 0 warnings** - Tests: Architecture 16 / Domain 42 / Web.Tests 158 / bUnit 92 — **all pass** - All 6 pre-push gates passed (including integration tests) ## Notes - Full Create/Edit page migration is deferred to #324 and #325, now unblocked by this fix. - Working as **Sam** (Implementation) Co-authored-by: Boromir <boromir@squad.dev> Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot AI review requested due to automatic review settings May 13, 2026 06:33

github-actions Bot added the squad Squad triage inbox — Lead will assign to a member label May 13, 2026

mpaulosky mentioned this pull request May 13, 2026

[Sprint 19] Restore markdown editor compile path end-to-end #323

Closed

3 tasks

Copilot started reviewing on behalf of mpaulosky May 13, 2026 06:34 View session

Copilot AI reviewed May 13, 2026

View reviewed changes

mpaulosky force-pushed the squad/323-restore-markdown-editor-compile-path branch from e6d474c to 8886140 Compare May 13, 2026 15:14

mpaulosky force-pushed the squad/323-restore-markdown-editor-compile-path branch from 8886140 to 13b7269 Compare May 13, 2026 17:24

mpaulosky mentioned this pull request May 13, 2026

feat(web): Edit flow markdown authoring slice (#325) #331

Merged

mpaulosky enabled auto-merge May 13, 2026 17:25

mpaulosky merged commit a269e2a into dev May 13, 2026
18 checks passed

mpaulosky deleted the squad/323-restore-markdown-editor-compile-path branch May 13, 2026 17:42

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[Sprint 19] Restore markdown editor compile path - #329

[Sprint 19] Restore markdown editor compile path#329
mpaulosky merged 1 commit into
devfrom
squad/323-restore-markdown-editor-compile-path

mpaulosky commented May 13, 2026

Uh oh!

github-actions Bot commented May 13, 2026

Uh oh!

Copilot AI left a comment

Uh oh!

github-actions Bot commented May 13, 2026 •

edited

Loading

Uh oh!

codecov Bot commented May 13, 2026 •

edited

Loading

Uh oh!

mpaulosky commented May 13, 2026

Uh oh!

mpaulosky commented May 13, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

		// Get the selected text using JavaScript
		var selectedText = await JsRuntime.InvokeAsync<string>("getSelectedText", ".EasyMDEContainer textarea");


		// Ensure the interop is available globally
		console.log('Markdown Editor Interop Loaded');

Conversation

mpaulosky commented May 13, 2026

Summary

Changes

Package references (Web.csproj)

New infrastructure layer (Infrastructure/FileStorage/)

Wiring

JS helpers (from #321 investigation)

Verification

Notes

Uh oh!

github-actions Bot commented May 13, 2026

🏗️ PR Added to Squad Triage Queue

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

github-actions Bot commented May 13, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Test Results Summary

Uh oh!

codecov Bot commented May 13, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Codecov Report

Uh oh!

mpaulosky commented May 13, 2026

Uh oh!

mpaulosky commented May 13, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

New infrastructure layer (`Infrastructure/FileStorage/`)

github-actions Bot commented May 13, 2026 •

edited

Loading

codecov Bot commented May 13, 2026 •

edited

Loading