artifacts

KSemenenko · KSemenenko · commit a60bb03ba7bb · 2026-04-07T11:44:34.000+02:00
diff --git a/.codex/skills/dotnet-cloc/references/examples.md b/.codex/skills/dotnet-cloc/references/examples.md
@@ -109,7 +109,7 @@ Goal: Track code size over time in CI.
     echo "Total lines: $(jq '.SUM.code' cloc-report.json)"
 
 - name: Upload metrics artifact
-  uses: actions/upload-artifact@v4
+  uses: actions/upload-artifact@v7
   with:
     name: code-metrics
     path: cloc-report.json
diff --git a/.codex/skills/dotnet-codeql/references/workflow.md b/.codex/skills/dotnet-codeql/references/workflow.md
@@ -242,7 +242,7 @@ Automatic with `github/codeql-action/analyze`:
 
 ```yaml
 - name: Upload SARIF as artifact
-  uses: actions/upload-artifact@v4
+  uses: actions/upload-artifact@v7
   with:
     name: sarif-results
     path: sarif-results
diff --git a/.codex/skills/dotnet-quality-ci/references/workflows.md b/.codex/skills/dotnet-quality-ci/references/workflows.md
@@ -88,7 +88,7 @@ jobs:
             -reporttypes:"HtmlSummary;Cobertura;MarkdownSummaryGithub"
 
       - name: Upload coverage report
-        uses: actions/upload-artifact@v4
+        uses: actions/upload-artifact@v7
         with:
           name: coverage-report
           path: coverage/report
@@ -228,7 +228,7 @@ jobs:
             -reporttypes:"HtmlSummary;Cobertura"
 
       - name: Upload coverage
-        uses: actions/upload-artifact@v4
+        uses: actions/upload-artifact@v7
         with:
           name: coverage
           path: coverage/report
@@ -346,7 +346,7 @@ jobs:
         working-directory: ./tests/MyProject.Tests
 
       - name: Upload mutation report
-        uses: actions/upload-artifact@v4
+        uses: actions/upload-artifact@v7
         with:
           name: stryker-report
           path: '**/StrykerOutput/**'
@@ -424,7 +424,7 @@ jobs:
             --severity=WARNING
 
       - name: Upload inspection report
-        uses: actions/upload-artifact@v4
+        uses: actions/upload-artifact@v7
         with:
           name: inspectcode-report
           path: inspectcode-report.xml
diff --git a/.codex/skills/dotnet-stryker/references/commands.md b/.codex/skills/dotnet-stryker/references/commands.md
@@ -209,7 +209,7 @@ dotnet stryker --since --since-target main
   continue-on-error: true
 
 - name: Upload Mutation Report
-  uses: actions/upload-artifact@v4
+  uses: actions/upload-artifact@v7
   with:
     name: mutation-report
     path: StrykerOutput/
diff --git a/.github/workflows/deploy-github-pages.yml b/.github/workflows/deploy-github-pages.yml
@@ -124,7 +124,7 @@ jobs:
 
       - name: Upload Playwright artifacts
         if: always()
-        uses: actions/upload-artifact@v4
+        uses: actions/upload-artifact@v7
         with:
           name: playwright-artifacts-release-pipeline
           path: output/playwright/
diff --git a/.github/workflows/pr-validation.yml b/.github/workflows/pr-validation.yml
@@ -110,7 +110,7 @@ jobs:
 
       - name: Upload Playwright artifacts
         if: always()
-        uses: actions/upload-artifact@v4
+        uses: actions/upload-artifact@v7
         with:
           name: playwright-artifacts-pr-validation
           path: output/playwright/
diff --git a/src/PrompterOne.Shared/Editor/Components/EditorMetadataRail.razor b/src/PrompterOne.Shared/Editor/Components/EditorMetadataRail.razor
@@ -50,43 +50,8 @@
 
         @if (SplitFeedback is not null)
         {
-            <div class="ed-sidebar-note ed-sidebar-note-split ed-meta-split-feedback" data-testid="@UiTestIds.Editor.SplitStatus">
-                <div class="ed-sidebar-note-head">
-                    <div>
-                        <div class="ed-sidebar-note-kicker" data-testid="@UiTestIds.Editor.SplitResultTitle">@SplitFeedback.Title</div>
-                        <div class="ed-sidebar-note-summary" data-testid="@UiTestIds.Editor.SplitResultSummary">@SplitFeedback.Summary</div>
-                    </div>
-                    <span class="ed-sidebar-note-badge" data-testid="@UiTestIds.Editor.SplitResultBadge">@SplitFeedback.HeadingBadge</span>
-                </div>
-
-                <p class="ed-sidebar-note-copy" data-testid="@UiTestIds.Editor.SplitResultLibrary">@SplitFeedback.DestinationNote</p>
-                <p class="ed-sidebar-note-copy ed-sidebar-note-copy-muted" data-testid="@UiTestIds.Editor.SplitResultCurrentDraft">@SplitFeedback.DraftNote</p>
-
-                @if (SplitFeedback.CreatedTitles.Count > 0)
-                {
-                    <div class="ed-sidebar-note-list" data-testid="@UiTestIds.Editor.SplitResultList">
-                        @for (var index = 0; index < SplitFeedback.CreatedTitles.Count; index++)
-                        {
-                            <div class="ed-sidebar-note-item" data-testid="@UiTestIds.Editor.SplitResultItem(index)">
-                                <span class="ed-sidebar-note-item-index">@($"{index + 1:00}")</span>
-                                <span class="ed-sidebar-note-item-title">@SplitFeedback.CreatedTitles[index]</span>
-                            </div>
-                        }
-                    </div>
-                }
-
-                @if (SplitFeedback.AdditionalCount > 0)
-                {
-                    <div class="ed-sidebar-note-more" data-testid="@UiTestIds.Editor.SplitResultMore">@Format(UiTextKey.EditorSplitMoreInLibraryFormat, SplitFeedback.AdditionalCount)</div>
-                }
-
-                <button class="ed-sidebar-note-link"
-                        data-testid="@UiTestIds.Editor.SplitResultOpenLibrary"
-                        type="button"
-                        @onclick="() => OpenLibraryRequested.InvokeAsync()">
-                    @SplitFeedback.OpenLibraryLabel
-                </button>
-            </div>
+            <EditorSplitFeedbackCard Feedback="@SplitFeedback"
+                                     OpenLibraryRequested="OpenLibraryRequested" />
         }
 
         <div class="ed-meta-field">
diff --git a/src/PrompterOne.Shared/Editor/Components/EditorSplitFeedbackCard.razor b/src/PrompterOne.Shared/Editor/Components/EditorSplitFeedbackCard.razor
@@ -0,0 +1,51 @@
+@namespace PrompterOne.Shared.Components.Editor
+@using Microsoft.Extensions.Localization
+@using PrompterOne.Shared.Contracts
+@using PrompterOne.Shared.Localization
+@inject IStringLocalizer<SharedResource> Localizer
+
+<div class="ed-sidebar-note ed-sidebar-note-split ed-meta-split-feedback" data-testid="@UiTestIds.Editor.SplitStatus">
+    <div class="ed-sidebar-note-head">
+        <div>
+            <div class="ed-sidebar-note-kicker" data-testid="@UiTestIds.Editor.SplitResultTitle">@Feedback.Title</div>
+            <div class="ed-sidebar-note-summary" data-testid="@UiTestIds.Editor.SplitResultSummary">@Feedback.Summary</div>
+        </div>
+        <span class="ed-sidebar-note-badge" data-testid="@UiTestIds.Editor.SplitResultBadge">@Feedback.HeadingBadge</span>
+    </div>
+
+    <p class="ed-sidebar-note-copy" data-testid="@UiTestIds.Editor.SplitResultLibrary">@Feedback.DestinationNote</p>
+    <p class="ed-sidebar-note-copy ed-sidebar-note-copy-muted" data-testid="@UiTestIds.Editor.SplitResultCurrentDraft">@Feedback.DraftNote</p>
+
+    @if (Feedback.CreatedTitles.Count > 0)
+    {
+        <div class="ed-sidebar-note-list" data-testid="@UiTestIds.Editor.SplitResultList">
+            @for (var index = 0; index < Feedback.CreatedTitles.Count; index++)
+            {
+                <div class="ed-sidebar-note-item" data-testid="@UiTestIds.Editor.SplitResultItem(index)">
+                    <span class="ed-sidebar-note-item-index">@($"{index + 1:00}")</span>
+                    <span class="ed-sidebar-note-item-title">@Feedback.CreatedTitles[index]</span>
+                </div>
+            }
+        </div>
+    }
+
+    @if (Feedback.AdditionalCount > 0)
+    {
+        <div class="ed-sidebar-note-more" data-testid="@UiTestIds.Editor.SplitResultMore">@Format(UiTextKey.EditorSplitMoreInLibraryFormat, Feedback.AdditionalCount)</div>
+    }
+
+    <button class="ed-sidebar-note-link"
+            data-testid="@UiTestIds.Editor.SplitResultOpenLibrary"
+            type="button"
+            @onclick="() => OpenLibraryRequested.InvokeAsync()">
+        @Feedback.OpenLibraryLabel
+    </button>
+</div>
+
+@code {
+    [Parameter, EditorRequired] public EditorSplitFeedbackViewModel Feedback { get; set; } = null!;
+
+    [Parameter] public EventCallback OpenLibraryRequested { get; set; }
+
+    private string Format(UiTextKey key, params object[] args) => Localizer[key.ToString(), args];
+}
diff --git a/tests/PrompterOne.Testing/Editor/EditorSplitFeedbackTestData.cs b/tests/PrompterOne.Testing/Editor/EditorSplitFeedbackTestData.cs
@@ -0,0 +1,45 @@
+namespace PrompterOne.Testing.Editor;
+
+public static class EditorSplitFeedbackTestData
+{
+    public const int MetadataRailAdditionalCount = 2;
+    public const int PostAutosaveObservationDelayMs = 1_700;
+    public const string EpisodeOneCardId = "untitled-script-split-01-episode-1-how-to-think-about-systems";
+    public const string EpisodeOneTitle = "Episode 1 - How to Think About Systems";
+    public const string EpisodeThreeTitle = "Episode 3 - Event Sourcing and CQRS";
+    public const string EpisodeTwoCardId = "untitled-script-split-02-episode-2-how-systems-talk-to-each-other";
+    public const string EpisodeTwoTitle = "Episode 2 - How Systems Talk to Each Other";
+    public const string SplitActionLabel = "Open in Library";
+    public const string SplitFeedbackBadge = "From ##";
+    public const string SplitFeedbackDestination = "New scripts were saved to the library.";
+    public const string SplitFeedbackDraftNote = "The current draft stayed open.";
+    public const string SplitFeedbackSummary = "2 new scripts created.";
+    public const string SplitFeedbackTitle = "Split complete";
+    public const string SplitSegmentActionLabel = "New scripts from ## headings";
+    public const string SplitSource =
+        """
+        ## [Episode 1 - How to Think About Systems|140WPM|Professional]
+        Before you write code, / you need to think about the system. //
+
+        ## [Episode 2 - How Systems Talk to Each Other|140WPM|Professional]
+        APIs, events, and retries matter. //
+        """;
+
+    public const string EditedSplitSource =
+        """
+        ## [Episode 1 - How to Think About Systems|140WPM|Professional]
+        Before you write code, / you need to think about the system. //
+
+        ## [Episode 2 - How Systems Talk to Each Other|140WPM|Professional]
+        APIs, events, and retries matter. //
+
+        Notes: keep the current draft open while reviewing the split.
+        """;
+
+    public static IReadOnlyList<string> MetadataRailCreatedTitles { get; } =
+    [
+        EpisodeOneTitle,
+        EpisodeTwoTitle,
+        EpisodeThreeTitle
+    ];
+}
diff --git a/tests/PrompterOne.Web.Tests/Editor/EditorMetadataRailTests.cs b/tests/PrompterOne.Web.Tests/Editor/EditorMetadataRailTests.cs
@@ -3,24 +3,15 @@
 using PrompterOne.Shared.Components.Editor;
 using PrompterOne.Shared.Contracts;
 using PrompterOne.Shared.Tests;
+using PrompterOne.Testing.Editor;
 
 namespace PrompterOne.Web.Tests;
 
 public sealed class EditorMetadataRailTests : BunitContext
 {
-    private const string SplitActionLabel = "Open in Library";
-    private const string SplitFeedbackBadge = "From ##";
-    private const string SplitFeedbackDestination = "New scripts were saved to the library.";
-    private const string SplitFeedbackDraftNote = "The current draft stayed open.";
-    private const string SplitFeedbackSummary = "2 new scripts created.";
-    private const string SplitFeedbackTitle = "Split complete";
     private const string SplitHint = "Create separate scripts from headings in the current draft.";
     private const string SplitSection = "Split Into New Scripts";
     private const string SplitTopLevelLabel = "New scripts from # headings";
-    private const string SplitSegmentLabel = "New scripts from ## headings";
-    private const string CreatedEpisodeOne = "Episode 1 - How to Think About Systems";
-    private const string CreatedEpisodeTwo = "Episode 2 - How Systems Talk to Each Other";
-    private const string CreatedEpisodeThree = "Episode 3 - Event Sourcing and CQRS";
 
     public EditorMetadataRailTests()
     {
@@ -44,7 +35,7 @@ public void EditorMetadataRail_SplitActionsInvokeExpectedModes_AndRenderSplitFee
         Assert.Contains(SplitSection, cut.Markup, StringComparison.Ordinal);
         Assert.Equal(SplitHint, cut.Find($".ed-meta-action-note").TextContent.Trim());
         Assert.Equal(SplitTopLevelLabel, cut.FindByTestId(UiTestIds.Editor.SplitTopLevel).TextContent.Trim());
-        Assert.Equal(SplitSegmentLabel, cut.FindByTestId(UiTestIds.Editor.SplitSegment).TextContent.Trim());
+        Assert.Equal(EditorSplitFeedbackTestData.SplitSegmentActionLabel, cut.FindByTestId(UiTestIds.Editor.SplitSegment).TextContent.Trim());
 
         cut.FindByTestId(UiTestIds.Editor.SplitTopLevel).Click();
         cut.FindByTestId(UiTestIds.Editor.SplitSegment).Click();
@@ -54,31 +45,26 @@ public void EditorMetadataRail_SplitActionsInvokeExpectedModes_AndRenderSplitFee
             [TpsDocumentSplitMode.TopLevelHeading, TpsDocumentSplitMode.SegmentHeading],
             requestedModes);
         Assert.Equal(1, openLibraryRequests);
-        Assert.Equal(SplitFeedbackTitle, cut.FindByTestId(UiTestIds.Editor.SplitResultTitle).TextContent.Trim());
-        Assert.Equal(SplitFeedbackSummary, cut.FindByTestId(UiTestIds.Editor.SplitResultSummary).TextContent.Trim());
-        Assert.Equal(SplitFeedbackBadge, cut.FindByTestId(UiTestIds.Editor.SplitResultBadge).TextContent.Trim());
-        Assert.Equal(SplitFeedbackDestination, cut.FindByTestId(UiTestIds.Editor.SplitResultLibrary).TextContent.Trim());
-        Assert.Equal(SplitFeedbackDraftNote, cut.FindByTestId(UiTestIds.Editor.SplitResultCurrentDraft).TextContent.Trim());
-        Assert.Equal(CreatedEpisodeOne, cut.FindByTestId(UiTestIds.Editor.SplitResultItem(0)).TextContent.Trim().Replace("01", string.Empty).Trim());
-        Assert.Equal(CreatedEpisodeTwo, cut.FindByTestId(UiTestIds.Editor.SplitResultItem(1)).TextContent.Trim().Replace("02", string.Empty).Trim());
-        Assert.Equal(CreatedEpisodeThree, cut.FindByTestId(UiTestIds.Editor.SplitResultItem(2)).TextContent.Trim().Replace("03", string.Empty).Trim());
+        Assert.Equal(EditorSplitFeedbackTestData.SplitFeedbackTitle, cut.FindByTestId(UiTestIds.Editor.SplitResultTitle).TextContent.Trim());
+        Assert.Equal(EditorSplitFeedbackTestData.SplitFeedbackSummary, cut.FindByTestId(UiTestIds.Editor.SplitResultSummary).TextContent.Trim());
+        Assert.Equal(EditorSplitFeedbackTestData.SplitFeedbackBadge, cut.FindByTestId(UiTestIds.Editor.SplitResultBadge).TextContent.Trim());
+        Assert.Equal(EditorSplitFeedbackTestData.SplitFeedbackDestination, cut.FindByTestId(UiTestIds.Editor.SplitResultLibrary).TextContent.Trim());
+        Assert.Equal(EditorSplitFeedbackTestData.SplitFeedbackDraftNote, cut.FindByTestId(UiTestIds.Editor.SplitResultCurrentDraft).TextContent.Trim());
+        Assert.Equal(EditorSplitFeedbackTestData.EpisodeOneTitle, cut.FindByTestId(UiTestIds.Editor.SplitResultItem(0)).TextContent.Trim().Replace("01", string.Empty).Trim());
+        Assert.Equal(EditorSplitFeedbackTestData.EpisodeTwoTitle, cut.FindByTestId(UiTestIds.Editor.SplitResultItem(1)).TextContent.Trim().Replace("02", string.Empty).Trim());
+        Assert.Equal(EditorSplitFeedbackTestData.EpisodeThreeTitle, cut.FindByTestId(UiTestIds.Editor.SplitResultItem(2)).TextContent.Trim().Replace("03", string.Empty).Trim());
         Assert.Equal("+2 more in Library", cut.FindByTestId(UiTestIds.Editor.SplitResultMore).TextContent.Trim());
-        Assert.Equal(SplitActionLabel, cut.FindByTestId(UiTestIds.Editor.SplitResultOpenLibrary).TextContent.Trim());
+        Assert.Equal(EditorSplitFeedbackTestData.SplitActionLabel, cut.FindByTestId(UiTestIds.Editor.SplitResultOpenLibrary).TextContent.Trim());
     }
 
     private static EditorSplitFeedbackViewModel BuildSplitFeedback() =>
         new(
-            Title: SplitFeedbackTitle,
-            Summary: SplitFeedbackSummary,
-            HeadingBadge: SplitFeedbackBadge,
-            DestinationNote: SplitFeedbackDestination,
-            DraftNote: SplitFeedbackDraftNote,
-            OpenLibraryLabel: SplitActionLabel,
-            CreatedTitles:
-            [
-                CreatedEpisodeOne,
-                CreatedEpisodeTwo,
-                CreatedEpisodeThree
-            ],
-            AdditionalCount: 2);
+            Title: EditorSplitFeedbackTestData.SplitFeedbackTitle,
+            Summary: EditorSplitFeedbackTestData.SplitFeedbackSummary,
+            HeadingBadge: EditorSplitFeedbackTestData.SplitFeedbackBadge,
+            DestinationNote: EditorSplitFeedbackTestData.SplitFeedbackDestination,
+            DraftNote: EditorSplitFeedbackTestData.SplitFeedbackDraftNote,
+            OpenLibraryLabel: EditorSplitFeedbackTestData.SplitActionLabel,
+            CreatedTitles: EditorSplitFeedbackTestData.MetadataRailCreatedTitles,
+            AdditionalCount: EditorSplitFeedbackTestData.MetadataRailAdditionalCount);
 }
diff --git a/tests/PrompterOne.Web.Tests/Editor/EditorSplitFeedbackInteractionTests.cs b/tests/PrompterOne.Web.Tests/Editor/EditorSplitFeedbackInteractionTests.cs
diff --git a/tests/PrompterOne.Web.UITests/Editor/EditorDocumentSplitFlowTests.cs b/tests/PrompterOne.Web.UITests/Editor/EditorDocumentSplitFlowTests.cs
