Update dependency NCalcSync to 6.3.3

.gitea/workflows/pr-push-build-and-test.yaml

@@ -15,7 +15,7 @@ jobs:
 
     steps:
       - name: Checkout repository
-        uses: actions/checkout@v7
+        uses: actions/checkout@v6
 
       - name: Setup .NET
         uses: actions/setup-dotnet@v5

MeetingAssistant.Tests/LiteLlmResponsesChatClientTests.cs

@@ -1,253 +1,103 @@
 using MeetingAssistant.Summary;
 using Microsoft.Extensions.AI;
 using System.Net;
-using System.Text;
-using System.Text.Json;
 
 namespace MeetingAssistant.Tests;
 
 public sealed class LiteLlmResponsesChatClientTests
 {
     [Fact]
-    public async Task ClientAssemblesStreamedTextResponseWithMetadataAndUsage()
+    public void ParserIgnoresReasoningItemsWithNullStatusAndReadsText()
     {
-        var handler = new SequencedHttpMessageHandler(
-            new HttpResponseMessage(HttpStatusCode.OK)
+        const string json = """
             {
-                Content = new StringContent(
-                    """
-                    data: {"type":"response.created","response":{"id":"resp_stream","created_at":1779147100,"model":"gpt-5.5","object":"response","output":[],"parallel_tool_calls":true,"status":"in_progress","store":false},"sequence_number":0}
+              "id": "resp_test",
+              "created_at": 1779147100,
+              "model": "gpt-5.5-2026-04-23",
+              "output": [
+                {
+                  "type": "reasoning",
+                  "summary": [],
+                  "status": null
+                },
+                {
+                  "type": "message",
+                  "role": "assistant",
+                  "status": "completed",
+                  "content": [
+                    {
+                      "type": "output_text",
+                      "text": "OK"
+                    }
+                  ]
+                }
+              ]
+            }
+            """;
 
-                    data: {"type":"response.output_item.added","output_index":0,"item":{"id":"msg_stream","type":"message","status":"in_progress","content":[],"role":"assistant"},"sequence_number":1}
+        var response = LiteLlmResponsesChatClient.ParseResponseJson(json);
 
-                    data: {"type":"response.content_part.added","item_id":"msg_stream","output_index":0,"content_index":0,"part":{"type":"output_text","annotations":[],"text":""},"sequence_number":2}
-
-                    data: {"type":"response.output_text.delta","item_id":"msg_stream","output_index":0,"content_index":0,"delta":"Streamed OK","sequence_number":3}
-
-                    data: {"type":"response.output_text.done","item_id":"msg_stream","output_index":0,"content_index":0,"text":"Streamed OK","sequence_number":4}
-
-                    data: {"type":"response.content_part.done","item_id":"msg_stream","output_index":0,"content_index":0,"part":{"type":"output_text","annotations":[],"text":"Streamed OK"},"sequence_number":5}
-
-                    data: {"type":"response.output_item.done","output_index":0,"item":{"id":"msg_stream","type":"message","status":"completed","content":[{"type":"output_text","annotations":[],"text":"Streamed OK"}],"role":"assistant"},"sequence_number":6}
-
-                    data: {"type":"response.completed","response":{"id":"resp_stream","created_at":1779147100,"model":"gpt-5.5","object":"response","output":[],"parallel_tool_calls":true,"status":"completed","store":false,"usage":{"input_tokens":12,"output_tokens":3,"total_tokens":15}},"sequence_number":7}
-
-                    data: [DONE]
-
-                    """,
-                    Encoding.UTF8,
-                    "text/event-stream")
-            });
-        using var client = CreateClient(handler, reconnectionAttempts: 0);
-
-        var response = await client.GetResponseAsync([new ChatMessage(ChatRole.User, "reply")]);
-
-        Assert.Equal("Streamed OK", response.Text);
-        Assert.Equal("resp_stream", response.ResponseId);
-        Assert.Equal("gpt-5.5", response.ModelId);
-        Assert.Equal(DateTimeOffset.FromUnixTimeSeconds(1779147100), response.CreatedAt);
-        Assert.NotNull(response.Usage);
-        Assert.Equal(12, response.Usage.InputTokenCount);
-        Assert.Equal(3, response.Usage.OutputTokenCount);
-        Assert.Equal(15, response.Usage.TotalTokenCount);
+        Assert.Equal("OK", response.Text);
+        Assert.Equal("resp_test", response.ResponseId);
+        Assert.Equal("gpt-5.5-2026-04-23", response.ModelId);
     }
 
     [Fact]
-    public async Task ClientAssemblesStreamedFunctionCall()
+    public void ParserReadsFunctionCalls()
     {
-        var handler = new SequencedHttpMessageHandler(
-            new HttpResponseMessage(HttpStatusCode.OK)
+        const string json = """
             {
-                Content = new StringContent(
-                    """
-                    data: {"type":"response.output_item.done","output_index":0,"item":{"type":"function_call","call_id":"call_stream","name":"write_summary","arguments":"{\"markdown\":\"# Summary\\nDone\"}","status":"completed"}}
+              "output": [
+                {
+                  "type": "function_call",
+                  "call_id": "call_1",
+                  "name": "write_summary",
+                  "arguments": "{\"markdown\":\"# Summary\\nDone\"}",
+                  "status": "completed"
+                }
+              ]
+            }
+            """;
 
-                    data: {"type":"response.completed","response":{"id":"resp_stream","model":"gpt-5.5","output":[]}}
-
-                    data: [DONE]
-
-                    """,
-                    Encoding.UTF8,
-                    "text/event-stream")
-            });
-        using var client = CreateClient(handler, reconnectionAttempts: 0);
-
-        var response = await client.GetResponseAsync([new ChatMessage(ChatRole.User, "write summary")]);
+        var response = LiteLlmResponsesChatClient.ParseResponseJson(json);
         var call = Assert.IsType<FunctionCallContent>(Assert.Single(response.Messages[0].Contents));
 
-        Assert.Equal("call_stream", call.CallId);
+        Assert.Equal("call_1", call.CallId);
         Assert.Equal("write_summary", call.Name);
         Assert.NotNull(call.Arguments);
-        Assert.Equal("# Summary\nDone", call.Arguments["markdown"]?.ToString());
+        Assert.Equal("# Summary\nDone", call.Arguments["markdown"]);
     }
 
     [Fact]
-    public async Task ClientUsesNonStreamingResponsesWhenConfigured()
+    public void ParserReadsUsage()
     {
-        var handler = new RecordingHttpMessageHandler(_ => new HttpResponseMessage(HttpStatusCode.OK)
-        {
-            Content = new StringContent(
-                """
+        const string json = """
+            {
+              "usage": {
+                "input_tokens": 123,
+                "output_tokens": 45,
+                "total_tokens": 168
+              },
+              "output": [
                 {
-                  "id": "resp_nonstream",
-                  "created_at": 1779147100,
-                  "model": "gpt-5.5",
-                  "object": "response",
-                  "output": [
+                  "type": "message",
+                  "content": [
                     {
-                      "id": "msg_nonstream",
-                      "type": "message",
-                      "status": "completed",
-                      "content": [
-                        {
-                          "type": "output_text",
-                          "annotations": [],
-                          "text": "Non-streamed OK"
-                        }
-                      ],
-                      "role": "assistant"
+                      "type": "output_text",
+                      "text": "OK"
                     }
-                  ],
-                  "parallel_tool_calls": true,
-                  "status": "completed",
-                  "store": false,
-                  "usage": {
-                    "input_tokens": 12,
-                    "output_tokens": 4,
-                    "total_tokens": 16
-                  }
+                  ]
                 }
-                """,
-                Encoding.UTF8,
-                "application/json")
-        });
-        using var client = CreateClient(handler, reconnectionAttempts: 0, useStreaming: false);
+              ]
+            }
+            """;
 
-        var response = await client.GetResponseAsync([new ChatMessage(ChatRole.User, "reply")]);
+        var response = LiteLlmResponsesChatClient.ParseResponseJson(json);
 
-        Assert.Equal("Non-streamed OK", response.Text);
-        Assert.Contains("\"stream\":false", Assert.Single(handler.RequestBodies));
-    }
-
-    [Fact]
-    public async Task ClientSendsChatMessagesAsResponsesMessageItems()
-    {
-        var handler = new RecordingHttpMessageHandler(_ => CreateStreamedTextResponse("Done."));
-        using var client = CreateClient(handler, reconnectionAttempts: 0);
-
-        await client.GetResponseAsync(
-        [
-            new ChatMessage(ChatRole.User, "write summary"),
-            new ChatMessage(ChatRole.Assistant, "I will inspect the meeting.")
-        ]);
-
-        using var request = JsonDocument.Parse(Assert.Single(handler.RequestBodies));
-        var inputItems = request.RootElement.GetProperty("input").EnumerateArray().ToArray();
-        Assert.Equal(2, inputItems.Length);
-        Assert.All(inputItems, item => Assert.Equal("message", item.GetProperty("type").GetString()));
-        Assert.Equal("user", inputItems[0].GetProperty("role").GetString());
-        Assert.Equal(
-            "input_text",
-            Assert.Single(inputItems[0].GetProperty("content").EnumerateArray())
-                .GetProperty("type")
-                .GetString());
-        Assert.Equal("assistant", inputItems[1].GetProperty("role").GetString());
-        Assert.Equal(
-            "output_text",
-            Assert.Single(inputItems[1].GetProperty("content").EnumerateArray())
-                .GetProperty("type")
-                .GetString());
-        Assert.DoesNotContain("\"type\":\"unknown\"", handler.RequestBodies[0]);
-    }
-
-    [Fact]
-    public async Task AgentLoopReturnsMalformedFunctionArgumentsWithoutInvokingTool()
-    {
-        var responses = new Queue<HttpResponseMessage>(
-        [
-            new HttpResponseMessage(HttpStatusCode.OK)
-            {
-                Content = new StringContent(
-                    """
-                    data: {"type":"response.output_item.done","output_index":0,"item":{"type":"function_call","call_id":"call_invalid","name":"write_summary","arguments":"{not-json","status":"completed"}}
-
-                    data: {"type":"response.completed","response":{"id":"resp_invalid","model":"gpt-5.5","output":[]}}
-
-                    data: [DONE]
-
-                    """,
-                    Encoding.UTF8,
-                    "text/event-stream")
-            },
-            CreateStreamedTextResponse("Recovered.")
-        ]);
-        var handler = new RecordingHttpMessageHandler(_ => responses.Dequeue());
-        var toolInvoked = false;
-        var tool = AIFunctionFactory.Create(
-            (string markdown) =>
-            {
-                toolInvoked = true;
-                return markdown;
-            },
-            "write_summary",
-            "Writes a summary.");
-        using var innerClient = CreateClient(handler, reconnectionAttempts: 0);
-        using var functionClient = innerClient
-            .AsBuilder()
-            .UseFunctionInvocation(
-                loggerFactory: null,
-                client => client.FunctionInvoker = FunctionInvocationGuard.InvokeAsync)
-            .Build();
-
-        var response = await functionClient.GetResponseAsync(
-            [new ChatMessage(ChatRole.User, "write summary")],
-            new ChatOptions { Tools = [tool] });
-
-        Assert.False(toolInvoked);
-        Assert.Equal("Recovered.", response.Text);
-        Assert.Equal(2, handler.RequestBodies.Count);
-        Assert.Contains("invalid_tool_arguments", handler.RequestBodies[1]);
-        Assert.Contains("call_invalid", handler.RequestBodies[1]);
-    }
-
-    [Fact]
-    public async Task ClientReportsVisibleReasoningSummariesSeparatelyFromResponseText()
-    {
-        var handler = new SequencedHttpMessageHandler(
-            CreateStreamedTextResponse(
-                "Done.",
-                "Checked the configured rules file.",
-                "Prepared a targeted update."));
-        var reasoningSummaries = new List<string>();
-        using var client = CreateClient(
-            handler,
-            reconnectionAttempts: 0,
-            reasoningSummaryChanged: reasoningSummaries.Add);
-
-        var response = await client.GetResponseAsync([new ChatMessage(ChatRole.User, "update rules")]);
-
-        Assert.Equal("Done.", response.Text);
-        Assert.Equal(
-            [
-                "Checked the configured rules file.",
-                "Prepared a targeted update."
-            ],
-            reasoningSummaries);
-    }
-
-    [Fact]
-    public async Task ClientDoesNotFailResponseWhenReasoningSummaryCallbackFails()
-    {
-        var handler = new SequencedHttpMessageHandler(
-            CreateStreamedTextResponse("Done.", "Checked the configured rules file."));
-        using var client = CreateClient(
-            handler,
-            reconnectionAttempts: 0,
-            reasoningSummaryChanged: _ => throw new InvalidOperationException("UI failed"));
-
-        var response = await client.GetResponseAsync([new ChatMessage(ChatRole.User, "update rules")]);
-
-        Assert.Equal("Done.", response.Text);
+        Assert.NotNull(response.Usage);
+        Assert.Equal(123, response.Usage.InputTokenCount);
+        Assert.Equal(45, response.Usage.OutputTokenCount);
+        Assert.Equal(168, response.Usage.TotalTokenCount);
     }
 
     [Fact]
@@ -258,7 +108,24 @@ public sealed class LiteLlmResponsesChatClientTests
             {
                 Content = new StringContent("Internal Server Error")
             },
-            CreateStreamedTextResponse("Done."));
+            new HttpResponseMessage(HttpStatusCode.OK)
+            {
+                Content = new StringContent("""
+                    {
+                      "output": [
+                        {
+                          "type": "message",
+                          "content": [
+                            {
+                              "type": "output_text",
+                              "text": "Done."
+                            }
+                          ]
+                        }
+                      ]
+                    }
+                    """)
+            });
         var retryCount = 0;
         using var client = CreateClient(handler, reconnectionAttempts: 1, retrying: () => retryCount++);
 
@@ -289,7 +156,24 @@ public sealed class LiteLlmResponsesChatClientTests
     [Fact]
     public async Task ClientSendsUserInitiatorOnceThenAgentInitiator()
     {
-        var handler = new RecordingHttpMessageHandler(_ => CreateStreamedTextResponse("Done."));
+        var handler = new RecordingHttpMessageHandler(_ => new HttpResponseMessage(HttpStatusCode.OK)
+        {
+            Content = new StringContent("""
+                {
+                  "output": [
+                    {
+                      "type": "message",
+                      "content": [
+                        {
+                          "type": "output_text",
+                          "text": "Done."
+                        }
+                      ]
+                    }
+                  ]
+                }
+                """)
+        });
         using var client = CreateClient(handler, reconnectionAttempts: 0);
 
         await client.GetResponseAsync([new ChatMessage(ChatRole.User, "write summary")]);
@@ -321,7 +205,24 @@ public sealed class LiteLlmResponsesChatClientTests
                 };
             }
 
-            return CreateStreamedTextResponse("Done.");
+            return new HttpResponseMessage(HttpStatusCode.OK)
+            {
+                Content = new StringContent("""
+                    {
+                      "output": [
+                        {
+                          "type": "message",
+                          "content": [
+                            {
+                              "type": "output_text",
+                              "text": "Done."
+                            }
+                          ]
+                        }
+                      ]
+                    }
+                    """)
+            };
         });
         using var client = CreateClient(
             handler,
@@ -356,7 +257,24 @@ public sealed class LiteLlmResponsesChatClientTests
                 };
             }
 
-            return CreateStreamedTextResponse("Done.");
+            return new HttpResponseMessage(HttpStatusCode.OK)
+            {
+                Content = new StringContent("""
+                    {
+                      "output": [
+                        {
+                          "type": "message",
+                          "content": [
+                            {
+                              "type": "output_text",
+                              "text": "Done."
+                            }
+                          ]
+                        }
+                      ]
+                    }
+                    """)
+            };
         });
         using var client = CreateClient(
             handler,
@@ -388,9 +306,7 @@ public sealed class LiteLlmResponsesChatClientTests
         HttpMessageHandler handler,
         int reconnectionAttempts,
         LiteLlmResponsesCompactionOptions? compactionOptions = null,
-        Action? retrying = null,
-        Action<string>? reasoningSummaryChanged = null,
-        bool useStreaming = true)
+        Action? retrying = null)
     {
         return new LiteLlmResponsesChatClient(
             new HttpClient(handler)
@@ -404,50 +320,7 @@ public sealed class LiteLlmResponsesChatClientTests
             reconnectionAttempts,
             TimeSpan.Zero,
             compactionOptions,
-            retrying: retrying,
-            reasoningSummaryChanged: reasoningSummaryChanged,
-            useStreaming: useStreaming);
-    }
-
-    private static HttpResponseMessage CreateStreamedTextResponse(
-        string text,
-        params string[] reasoningSummaries)
-    {
-        var events = new List<string>();
-        for (var index = 0; index < reasoningSummaries.Length; index++)
-        {
-            events.Add("data: " + JsonSerializer.Serialize(new
-            {
-                type = "response.reasoning_summary_text.delta",
-                item_id = "reasoning_stream",
-                output_index = 0,
-                summary_index = index,
-                delta = reasoningSummaries[index]
-            }));
-        }
-
-        var outputIndex = reasoningSummaries.Length > 0 ? 1 : 0;
-        events.Add("data: " + JsonSerializer.Serialize(new
-        {
-            type = "response.output_text.delta",
-            item_id = "msg_stream",
-            output_index = outputIndex,
-            content_index = 0,
-            delta = text
-        }));
-        events.Add(
-            """data: {"type":"response.completed","response":{"id":"resp_stream","created_at":1779147100,"model":"gpt-5.5","object":"response","output":[],"parallel_tool_calls":true,"status":"completed","store":false,"usage":{"input_tokens":12,"output_tokens":3,"total_tokens":15}}}""");
-        events.Add("data: [DONE]");
-
-        return new HttpResponseMessage(HttpStatusCode.OK)
-        {
-            Content = new StringContent(
-                string.Join($"{Environment.NewLine}{Environment.NewLine}", events)
-                    + Environment.NewLine
-                    + Environment.NewLine,
-                Encoding.UTF8,
-                "text/event-stream")
-        };
+            retrying: retrying);
     }
 
     private sealed class SequencedHttpMessageHandler : HttpMessageHandler

MeetingAssistant.Tests/MeetingAssistant.Tests.csproj

@@ -9,8 +9,8 @@
 
   <ItemGroup>
     <PackageReference Include="coverlet.collector" Version="10.0.1" />
-    <PackageReference Include="Microsoft.AspNetCore.Mvc.Testing" Version="10.0.10" />
-    <PackageReference Include="Microsoft.NET.Test.Sdk" Version="18.8.1" />
+    <PackageReference Include="Microsoft.AspNetCore.Mvc.Testing" Version="10.0.9" />
+    <PackageReference Include="Microsoft.NET.Test.Sdk" Version="18.7.0" />
     <PackageReference Include="xunit" Version="2.9.3" />
     <PackageReference Include="xunit.runner.visualstudio" Version="3.1.5" />
   </ItemGroup>

MeetingAssistant.Tests/MeetingNoteStoreTests.cs

@@ -135,72 +135,6 @@ public sealed class MeetingNoteStoreTests
         Assert.Equal("User notes.", loaded.UserNotes);
     }
 
-    [Fact]
-    public async Task StoreEscapesApostrophePrefixedAttendeesBeforeWritingFrontmatter()
-    {
-        var (store, saved) = await SaveNoteAsync(
-            title: "Escaped Attendees",
-            attendees: ["'Ada Lovelace"],
-            projects: [],
-            userNotes: "Discuss attendee import.");
-        var content = await File.ReadAllTextAsync(saved.Path);
-        var loaded = await store.ReadAsync(saved.Path, CancellationToken.None);
-
-        Assert.Contains("- \"'Ada Lovelace\"", content);
-        Assert.Equal(["'Ada Lovelace"], loaded.Frontmatter.Attendees);
-        Assert.Equal("Discuss attendee import.", loaded.UserNotes);
-    }
-
-    [Theory]
-    [InlineData("Ada # platform lead")]
-    [InlineData("- Ada Lovelace")]
-    [InlineData("? Ada Lovelace")]
-    [InlineData("{Ada: Platform}")]
-    [InlineData("*Ada")]
-    [InlineData("&Ada")]
-    [InlineData("!Ada")]
-    [InlineData("| Ada")]
-    [InlineData("> Ada")]
-    [InlineData("@Ada")]
-    [InlineData("`Ada")]
-    [InlineData("true")]
-    [InlineData("null")]
-    [InlineData("2026-07-08")]
-    [InlineData("Ada \"The Architect\" Lovelace")]
-    [InlineData("C:\\People\\Ada")]
-    public async Task StoreEscapesYamlSensitiveAttendeesBeforeWritingFrontmatter(string attendee)
-    {
-        var (store, saved) = await SaveNoteAsync(
-            title: "Escaped Attendees",
-            attendees: [attendee],
-            projects: [],
-            userNotes: "Discuss attendee import.");
-        var loaded = await store.ReadAsync(saved.Path, CancellationToken.None);
-
-        Assert.Equal([attendee], loaded.Frontmatter.Attendees);
-        Assert.Equal("Discuss attendee import.", loaded.UserNotes);
-    }
-
-    [Theory]
-    [InlineData("Planning # Q3")]
-    [InlineData("- Planning")]
-    [InlineData("{Planning: Q3}")]
-    [InlineData("true")]
-    [InlineData("2026-07-08")]
-    public async Task StoreEscapesYamlSensitiveScalarAndProjectFrontmatterValues(string value)
-    {
-        var (store, saved) = await SaveNoteAsync(
-            title: value,
-            attendees: [],
-            projects: [value],
-            userNotes: "Discuss YAML escaping.");
-        var loaded = await store.ReadAsync(saved.Path, CancellationToken.None);
-
-        Assert.Equal(value, loaded.Frontmatter.Title);
-        Assert.Equal([value], loaded.Frontmatter.Projects);
-        Assert.Equal("Discuss YAML escaping.", loaded.UserNotes);
-    }
-
     [Fact]
     public void ActionLinkEscapesSummaryFileName()
     {
@@ -213,26 +147,6 @@ public sealed class MeetingNoteStoreTests
             link);
     }
 
-    private static async Task<(MarkdownMeetingNoteStore Store, MeetingNote Saved)> SaveNoteAsync(
-        string title,
-        IReadOnlyList<string> attendees,
-        IReadOnlyList<string> projects,
-        string userNotes)
-    {
-        var (vaultRoot, store) = CreateStore();
-        var note = MeetingNoteTemplate.Create(
-            title: title,
-            attendees: attendees,
-            projects: projects,
-            transcriptPath: Path.Combine(vaultRoot, "Meetings", "Transcripts", "20260519-transcript.md"),
-            assistantContextPath: Path.Combine(vaultRoot, "Meetings", "Assistant Context", "20260519-context.md"),
-            summaryPath: Path.Combine(vaultRoot, "Meetings", "Summaries", "20260519-summary.md"),
-            userNotes: userNotes);
-
-        var saved = await store.SaveAsync(note, CancellationToken.None);
-        return (store, saved);
-    }
-
     private static (string VaultRoot, MarkdownMeetingNoteStore Store) CreateStore()
     {
         var vaultRoot = Path.Combine(Path.GetTempPath(), "meeting-assistant-tests", Guid.NewGuid().ToString("N"));

MeetingAssistant.Tests/MeetingScreenshotServiceTests.cs

@@ -1,7 +1,6 @@
 using MeetingAssistant.MeetingNotes;
 using MeetingAssistant.Screenshots;
 using MeetingAssistant.Speakers;
-using MeetingAssistant.Workflow;
 using Microsoft.Extensions.Logging.Abstractions;
 using Microsoft.Extensions.Options;
 using System.Drawing;
@@ -168,35 +167,6 @@ public sealed class MeetingScreenshotServiceTests
             request.SequenceEqual(["Ada Lovelace", "Ada L.", "Grace Hopper", "Ada Lovelace"]));
     }
 
-    [Fact]
-    public async Task CaptureTransformsOcrAttendeesBeforeWritingMeetingNote()
-    {
-        var fixture = await ScreenshotFixture.CreateAsync(
-            options =>
-            {
-                options.Screenshots.Ocr.Enabled = true;
-            });
-        var workflowEngine = new TransformingAttendeeWorkflowEngine("Ada Lovelace (Contoso)", "Ada Lovelace");
-        var service = fixture.CreateService(
-            new FixedScreenshotCapture([1, 2, 3]),
-            new CapturingScreenshotOcrClient(
-                "Visible participant tile: Ada.",
-                attendees: ["Ada Lovelace (Contoso)"]),
-            meetingWorkflowEngine: workflowEngine);
-
-        await service.CaptureAsync(
-            fixture.Artifacts,
-            DateTimeOffset.Parse("2026-05-26T10:00:00+02:00"),
-            DateTimeOffset.Parse("2026-05-26T10:00:10+02:00"),
-            fixture.Options,
-            CancellationToken.None);
-        await service.WaitForPendingOcrAsync(fixture.Artifacts, TimeSpan.FromSeconds(5), CancellationToken.None);
-
-        var meeting = await fixture.NoteStore.ReadAsync(fixture.Artifacts.MeetingNotePath, CancellationToken.None);
-        Assert.Equal(["Ada Lovelace"], meeting.Frontmatter.Attendees);
-        Assert.Equal(["Ada Lovelace (Contoso)"], workflowEngine.AttendeeRequests);
-    }
-
     [Fact]
     public async Task CaptureWritesRetryLinkWhenOcrFails()
     {
@@ -453,8 +423,7 @@ public sealed class MeetingScreenshotServiceTests
         public MeetingScreenshotService CreateService(
             IActiveWindowScreenshotCapture capture,
             IScreenshotOcrClient ocrClient,
-            ISpeakerIdentityAttendeeCanonicalizer? attendeeCanonicalizer = null,
-            IMeetingWorkflowEngine? meetingWorkflowEngine = null)
+            ISpeakerIdentityAttendeeCanonicalizer? attendeeCanonicalizer = null)
         {
             return new MeetingScreenshotService(
                 capture,
@@ -462,8 +431,7 @@ public sealed class MeetingScreenshotServiceTests
                 NoteStore,
                 attendeeCanonicalizer ?? PassthroughSpeakerIdentityAttendeeCanonicalizer.Instance,
                 ocrClient,
-                NullLogger<MeetingScreenshotService>.Instance,
-                meetingWorkflowEngine);
+                NullLogger<MeetingScreenshotService>.Instance);
         }
     }
 

MeetingAssistant.Tests/MeetingSummaryInstructionBuilderTests.cs

@@ -24,7 +24,6 @@ public sealed class MeetingSummaryInstructionBuilderTests
         Assert.Contains("You are the Meeting Assistant summary agent.", instructions);
         Assert.Contains("include only the most relevant cropped screenshots", instructions);
         Assert.Contains("Do not include every cropped screenshot", instructions);
-        Assert.Contains("encode spaces in image-link targets as `%20`", instructions);
         Assert.Contains("Use add_attendee and remove_attendee", instructions);
         Assert.Contains("partial screenshot", instructions);
         Assert.Contains("override_speaker", instructions);
@@ -40,27 +39,6 @@ public sealed class MeetingSummaryInstructionBuilderTests
         Assert.Contains("title parameter", instructions);
     }
 
-    [Fact]
-    public async Task DefaultPromptTreatsAssistantContextAsMeetingMemoryForUncertainty()
-    {
-        var root = Path.Combine(Path.GetTempPath(), "meeting-assistant-tests", Guid.NewGuid().ToString("N"));
-        var artifacts = CreateArtifacts(root);
-        await WriteMeetingNoteAsync(artifacts.MeetingNotePath, []);
-        var builder = new MeetingSummaryInstructionBuilder(Options.Create(new MeetingAssistantOptions
-        {
-            Agent = new AgentOptions { InitialPrompt = " " },
-            Vault = new VaultOptions { ProjectsFolder = Path.Combine(root, "Projects") }
-        }));
-
-        var instructions = await builder.BuildAsync(artifacts, CancellationToken.None);
-
-        Assert.Contains("meeting-specific memory", instructions);
-        Assert.Contains("unexpected problems", instructions);
-        Assert.Contains("missing information", instructions);
-        Assert.Contains("assumptions", instructions);
-        Assert.Contains("write_context", instructions);
-    }
-
     [Fact]
     public async Task BuilderUsesConfiguredPrompt()
     {

MeetingAssistant.Tests/MeetingSummaryToolTests.cs

@@ -293,44 +293,6 @@ public sealed class MeetingSummaryToolTests
         Assert.Contains("transcript: \"[[../Transcripts/transcript|Transcript]]\"", meetingNote);
     }
 
-    [Fact]
-    public async Task ToolsTransformAddedAttendeeBeforeWritingMeetingNote()
-    {
-        var root = Path.Combine(Path.GetTempPath(), "meeting-assistant-tests", Guid.NewGuid().ToString("N"));
-        var artifacts = new MeetingSessionArtifacts(
-            MeetingNotePath: Path.Combine(root, "Meetings", "Notes", "meeting.md"),
-            TranscriptPath: Path.Combine(root, "Meetings", "Transcripts", "transcript.md"),
-            AssistantContextPath: Path.Combine(root, "Meetings", "Assistant Context", "context.md"),
-            SummaryPath: Path.Combine(root, "Meetings", "Summaries", "summary.md"));
-        Directory.CreateDirectory(Path.GetDirectoryName(artifacts.MeetingNotePath)!);
-        await File.WriteAllTextAsync(
-            artifacts.MeetingNotePath,
-            """
-            ---
-            title: Meeting
-            attendees: []
-            projects: []
-            transcript: "[[../Transcripts/transcript|Transcript]]"
-            assistant_context: "[[../Assistant Context/context|Assistant Context]]"
-            summary: "[[../Summaries/summary|Summary]]"
-            ---
-
-            User note line.
-            """);
-        var workflowEngine = new TransformingAttendeeWorkflowEngine("Ada Lovelace (Contoso)", "Ada Lovelace");
-        var tools = new MeetingSummaryTools(
-            artifacts,
-            new MeetingAssistantOptions(),
-            meetingWorkflowEngine: workflowEngine);
-
-        Assert.Equal("Added attendee Ada Lovelace.", await tools.AddAttendee("Ada Lovelace (Contoso)"));
-
-        var meetingNote = await File.ReadAllTextAsync(artifacts.MeetingNotePath);
-        Assert.Contains("- Ada Lovelace", meetingNote);
-        Assert.DoesNotContain("Ada Lovelace (Contoso)", meetingNote);
-        Assert.Equal(["Ada Lovelace (Contoso)"], workflowEngine.AttendeeRequests);
-    }
-
     [Fact]
     public async Task ToolsOverrideSpeakerInTranscriptAndRecordOverride()
     {
@@ -837,5 +799,4 @@ public sealed class MeetingSummaryToolTests
         Assert.False(tools.SummaryWasWritten);
         Assert.False(File.Exists(artifacts.SummaryPath));
     }
-
 }

MeetingAssistant.Tests/MeetingWorkflowEngineTests.cs

@@ -310,10 +310,6 @@ public sealed class MeetingWorkflowEngineTests
     [InlineData("speaker_identified:\n        name: ADA", "speaker", null, null, "ada", true)]
     [InlineData("speaker_identified:\n        name: Ada", "speaker", null, null, "Grace", false)]
     [InlineData("speaker_identified: {}", "speaker", null, null, "Grace", true)]
-    [InlineData("attendee_added:\n        equals: Ada Lovelace", "attendee", null, null, "ada lovelace", true)]
-    [InlineData("attendee_added:\n        contains: contoso", "attendee", null, null, "Ada (Contoso)", true)]
-    [InlineData("attendee_added:\n        regex: '^Ada .+Contoso\\)$'", "attendee", null, null, "Ada (Contoso)", true)]
-    [InlineData("attendee_added:\n        regex: '^Grace'", "attendee", null, null, "Ada (Contoso)", false)]
     public async Task TriggerMatchingScenarios(
         string triggerYaml,
         string eventKind,
@@ -322,16 +318,6 @@ public sealed class MeetingWorkflowEngineTests
         string? speaker,
         bool shouldRun)
     {
-        var stepYaml = eventKind == "attendee"
-            ? """
-                - uses: set_property
-                  property: attendee.name
-                  value: Triggered
-            """
-            : """
-                - uses: add_project
-                  value: Triggered
-            """;
         var fixture = await WorkflowFixture.CreateAsync(
             $$"""
             rules:
@@ -339,21 +325,14 @@ public sealed class MeetingWorkflowEngineTests
               on:
                 - {{triggerYaml}}
               steps:
-            {{stepYaml}}
+                - uses: add_project
+                  value: Triggered
             """);
 
-        var workflowEvent = CreateEvent(eventKind, fixture.Artifacts, from, to, speaker);
-        if (eventKind == "attendee")
-        {
-            var transformed = await fixture.Engine.TransformAttendeeAsync(
-                workflowEvent,
-                fixture.Options,
-                CancellationToken.None);
-            Assert.Equal(shouldRun ? "Triggered" : speaker, transformed);
-            return;
-        }
-
-        await fixture.Engine.RunAsync(workflowEvent, fixture.Options, CancellationToken.None);
+        await fixture.Engine.RunAsync(
+            CreateEvent(eventKind, fixture.Artifacts, from, to, speaker),
+            fixture.Options,
+            CancellationToken.None);
 
         var meeting = await fixture.ReadMeetingAsync();
         Assert.Equal(shouldRun, meeting.Frontmatter.Projects.Contains("Triggered"));
@@ -624,34 +603,6 @@ public sealed class MeetingWorkflowEngineTests
         Assert.Equal(["Ada Lovelace"], meeting.Frontmatter.Attendees);
     }
 
-    [Fact]
-    public async Task AttendeeAddedRuleCanTransformWorkflowAddedAttendee()
-    {
-        var fixture = await WorkflowFixture.CreateAsync(
-            """
-            rules:
-            - name: add-raw-attendee
-              on:
-                - created: {}
-              steps:
-                - uses: add_attendee
-                  value: 'Ada Lovelace (Contoso)'
-            - name: clean-contoso-attendee
-              on:
-                - attendee_added:
-                    contains: 'Contoso'
-              steps:
-                - uses: set_property
-                  property: attendee.name
-                  value: '@Model.Attendee.Name.Replace(" (Contoso)", "")'
-            """);
-
-        await RunCreatedAsync(fixture);
-
-        var meeting = await fixture.ReadMeetingAsync();
-        Assert.Equal(["Ada Lovelace"], meeting.Frontmatter.Attendees);
-    }
-
     [Fact]
     public async Task AddAttendeeDoesNotDuplicateExistingDisplayNameFromEmailAddress()
     {
@@ -841,7 +792,6 @@ public sealed class MeetingWorkflowEngineTests
         {
             "created" => MeetingWorkflowEvent.Created(artifacts),
             "speaker" => MeetingWorkflowEvent.SpeakerIdentified(artifacts, speaker ?? "Ada"),
-            "attendee" => MeetingWorkflowEvent.AttendeeAdded(artifacts, speaker ?? "Ada"),
             "state" => MeetingWorkflowEvent.StateTransition(
                 artifacts,
                 ParseState(from ?? "collecting metadata"),

MeetingAssistant.Tests/RecordingCoordinatorTests.cs

@@ -117,7 +117,7 @@ public sealed class RecordingCoordinatorTests
     }
 
     [Fact]
-    public async Task TranscriptLineWorkflowRuleTransformsLiveTranscriptAfterDurableAppend()
+    public async Task TranscriptLineWorkflowRuleTransformsLiveTranscriptBeforePersistingMarkdown()
     {
         var root = Path.Combine(Path.GetTempPath(), "meeting-assistant-tests", Guid.NewGuid().ToString("N"));
         var rulesPath = Path.Combine(root, "rules.yaml");
@@ -900,39 +900,6 @@ public sealed class RecordingCoordinatorTests
         await coordinator.StopAsync(CancellationToken.None);
     }
 
-    [Fact]
-    public async Task StartTransformsMetadataAttendeesBeforeWritingNote()
-    {
-        var audioSource = new ControlledAudioSource();
-        var noteStore = new InMemoryMeetingNoteStore("C:\\Vault\\Meetings\\Notes\\metadata-meeting.md");
-        var workflowEngine = new TransformingAttendeeWorkflowEngine("Ada Lovelace (Contoso)", "Ada Lovelace");
-        var coordinator = new MeetingRecordingCoordinator(
-            audioSource,
-            new TestSpeechRecognitionPipelineFactory(new EchoStreamingTranscriptionProvider()),
-            new InMemoryTranscriptStore(),
-            noteStore,
-            new CapturingMeetingNoteOpener(),
-            new InMemoryMeetingArtifactStore(),
-            new InMemoryRecordedAudioStore(),
-            new CapturingMeetingSummaryPipeline(),
-            Options.Create(new MeetingAssistantOptions()),
-            NullLogger<MeetingRecordingCoordinator>.Instance,
-            meetingMetadataProvider: new FixedMeetingMetadataProvider(new MeetingMetadata(
-                "Architecture Sync",
-                ["Ada Lovelace (Contoso)"],
-                "",
-                null)),
-            meetingWorkflowEngine: workflowEngine);
-
-        await coordinator.StartAsync(CancellationToken.None);
-        await WaitUntilAsync(() => noteStore.SavedNote?.Frontmatter.Attendees.Count > 0);
-
-        Assert.Equal(["Ada Lovelace"], noteStore.SavedNote?.Frontmatter.Attendees);
-        Assert.Equal(["Ada Lovelace (Contoso)"], workflowEngine.AttendeeRequests);
-
-        await coordinator.StopAsync(CancellationToken.None);
-    }
-
     [Fact]
     public async Task StartSkipsOutlookMeetingAttendeesAboveConfiguredImportLimit()
     {

MeetingAssistant.Tests/TaskbarIconTests.cs

@@ -18,7 +18,7 @@ public sealed class TaskbarIconTests
         Assert.Equal(RecordingProcessState.Idle, menu.State);
         Assert.Contains(menu.Items, item =>
             item.Action == MeetingTaskbarAction.EditRules &&
-            item.Text == "Open agent");
+            item.Text == "Settings and logs");
         Assert.Contains(menu.Items, item =>
             item.Action == MeetingTaskbarAction.StartRecording &&
             item.ProfileName == "default" &&

MeetingAssistant.Tests/TransformingAttendeeWorkflowEngine.cs

@@ -1,44 +0,0 @@
-using MeetingAssistant.Workflow;
-
-namespace MeetingAssistant.Tests;
-
-internal sealed class TransformingAttendeeWorkflowEngine : IMeetingWorkflowEngine
-{
-    private readonly string source;
-    private readonly string replacement;
-
-    public TransformingAttendeeWorkflowEngine(string source, string replacement)
-    {
-        this.source = source;
-        this.replacement = replacement;
-    }
-
-    public List<string> AttendeeRequests { get; } = [];
-
-    public Task RunAsync(
-        MeetingWorkflowEvent workflowEvent,
-        MeetingAssistantOptions options,
-        CancellationToken cancellationToken)
-    {
-        return Task.CompletedTask;
-    }
-
-    public Task<string> TransformTranscriptLineAsync(
-        MeetingWorkflowEvent workflowEvent,
-        MeetingAssistantOptions options,
-        CancellationToken cancellationToken)
-    {
-        return Task.FromResult(workflowEvent.TranscriptLineText ?? "");
-    }
-
-    public Task<string> TransformAttendeeAsync(
-        MeetingWorkflowEvent workflowEvent,
-        MeetingAssistantOptions options,
-        CancellationToken cancellationToken)
-    {
-        AttendeeRequests.Add(workflowEvent.AttendeeName ?? "");
-        return Task.FromResult(string.Equals(workflowEvent.AttendeeName, source, StringComparison.OrdinalIgnoreCase)
-            ? replacement
-            : workflowEvent.AttendeeName ?? "");
-    }
-}

MeetingAssistant.Tests/WorkflowRulesEditorTests.cs

@@ -20,7 +20,6 @@ public sealed class WorkflowRulesEditorTests
             Key = "summary-key",
             KeyEnv = "SUMMARY_KEY",
             Model = "summary-model",
-            UseStreaming = true,
             EnableThinking = true,
             ReasoningEffort = ReasoningEffortOption.High,
             ReconnectionAttempts = 7,
@@ -34,7 +33,6 @@ public sealed class WorkflowRulesEditorTests
         var editor = new WorkflowRulesEditorOptions
         {
             Model = "editor-model",
-            UseStreaming = false,
             EnableThinking = false,
             MaxOutputTokens = 50
         };
@@ -45,7 +43,6 @@ public sealed class WorkflowRulesEditorTests
         Assert.Equal("summary-key", effective.Key);
         Assert.Equal("SUMMARY_KEY", effective.KeyEnv);
         Assert.Equal("editor-model", effective.Model);
-        Assert.False(effective.UseStreaming);
         Assert.False(effective.EnableThinking);
         Assert.Equal(ReasoningEffortOption.High, effective.ReasoningEffort);
         Assert.Equal(7, effective.ReconnectionAttempts);
@@ -640,28 +637,6 @@ public sealed class WorkflowRulesEditorTests
         Assert.Equal("rules: []", await File.ReadAllTextAsync(fixture.RulesPath));
     }
 
-    [Fact]
-    public async Task RulesEditorToolsRefuseSideEffectingAttendeeAddedRules()
-    {
-        var fixture = await CreateRulesEditorFixtureAsync();
-
-        var result = await fixture.Tools.WriteRules("""
-            rules:
-            - name: attendee-side-effect
-              on:
-                - attendee_added:
-                    contains: Contoso
-              steps:
-                - uses: add_context
-                  value: '@Model.Attendee.Name'
-            """, replace_file: true);
-
-        Assert.StartsWith("Refused: workflow rules are invalid.", result);
-        Assert.Contains("attendee-side-effect", result);
-        Assert.Contains("attendee.name", result);
-        Assert.Equal("rules: []", await File.ReadAllTextAsync(fixture.RulesPath));
-    }
-
     [Fact]
     public async Task RulesEditorToolsAcceptAndParseMaskedProfanityRedactionRule()
     {
@@ -763,29 +738,6 @@ public sealed class WorkflowRulesEditorTests
         Assert.Contains("matching correction", instructions);
     }
 
-    [Fact]
-    public async Task InstructionBuilderTreatsAssistantContextAsMeetingMemoryDuringRepairs()
-    {
-        var builder = new WorkflowRulesEditorInstructionBuilder(
-            NullLogger<WorkflowRulesEditorInstructionBuilder>.Instance);
-        var options = new MeetingAssistantOptions
-        {
-            WorkflowRulesEditor = new WorkflowRulesEditorOptions
-            {
-                InitialPrompt = "Custom interactive agent instructions."
-            }
-        };
-
-        var instructions = await builder.BuildAsync(options, CancellationToken.None);
-
-        Assert.Contains("Custom interactive agent instructions.", instructions);
-        Assert.Contains("meeting-specific memory", instructions);
-        Assert.Contains("fix or investigate a meeting or summary", instructions);
-        Assert.Contains("read the matching assistant context", instructions);
-        Assert.Contains("problems, missing information, assumptions, prior fixes, and conclusions", instructions);
-        Assert.Contains("append a concise record of your fixes and conclusions", instructions);
-    }
-
     [Fact]
     public async Task RulesEditorToolsCrudAndSearchSpeakerIdentities()
     {
@@ -905,13 +857,15 @@ public sealed class WorkflowRulesEditorTests
         Assert.Equal("", viewModel.Draft);
         Assert.Equal(
             [new WorkflowRulesEditorChatMessage(WorkflowRulesEditorChatRole.User, "Rename ABS Daily")],
-            viewModel.Messages.Select(item => item.Message).OfType<WorkflowRulesEditorChatMessage>().ToArray());
+            viewModel.Messages.ToArray());
 
         pipeline.Release.SetResult();
         await sendTask.WaitAsync(TimeSpan.FromSeconds(2));
 
         Assert.False(viewModel.IsThinking);
-        AssertCompletedActivity(viewModel, ["Thinking..."], "Updated rules.");
+        Assert.Equal(2, viewModel.Messages.Count);
+        Assert.Equal(WorkflowRulesEditorChatRole.Agent, viewModel.Messages[1].Role);
+        Assert.Equal("Updated rules.", viewModel.Messages[1].Content);
     }
 
     [Fact]
@@ -927,19 +881,20 @@ public sealed class WorkflowRulesEditorTests
 
         Assert.False(viewModel.IsThinking);
         Assert.Equal("", viewModel.Draft);
-        Assert.Equal(new WorkflowRulesEditorChatMessage(WorkflowRulesEditorChatRole.User, "check logs"), viewModel.Messages[0].Message);
-        AssertCompletedActivity(
-            viewModel,
-            ["Thinking..."],
-            "Meeting Summary Agent failed: The request timed out.");
+        Assert.Equal(
+            [
+                new WorkflowRulesEditorChatMessage(WorkflowRulesEditorChatRole.User, "check logs"),
+                new WorkflowRulesEditorChatMessage(
+                    WorkflowRulesEditorChatRole.Agent,
+                    "Settings and logs failed: The request timed out.")
+            ],
+            viewModel.Messages.ToArray());
     }
 
     [Fact]
     public async Task ViewModelShowsReconnectStatusReportedByPipeline()
     {
-        var pipeline = new ReportingRulesEditorPipeline(
-            "Updated rules.",
-            WorkflowRulesEditorActivityUpdate.Status("Reconnecting..."));
+        var pipeline = new StatusReportingRulesEditorPipeline("Updated rules.");
         var viewModel = new WorkflowRulesEditorChatViewModel(pipeline)
         {
             Draft = "check logs"
@@ -956,146 +911,7 @@ public sealed class WorkflowRulesEditorTests
 
         Assert.False(viewModel.IsThinking);
         Assert.Equal("Thinking...", viewModel.ActivityMessage);
-        AssertCompletedActivity(viewModel, ["Thinking...", "Reconnecting..."], "Updated rules.");
-    }
-
-    [Fact]
-    public async Task ViewModelShowsToolCallsAbovePersistentThinkingLine()
-    {
-        var pipeline = new ReportingRulesEditorPipeline(
-            "Updated rules.",
-            WorkflowRulesEditorActivityUpdate.ToolCall("read_logs"));
-        var viewModel = new WorkflowRulesEditorChatViewModel(pipeline)
-        {
-            Draft = "check logs"
-        };
-
-        var sendTask = viewModel.SendAsync();
-        await pipeline.Reported.Task.WaitAsync(TimeSpan.FromSeconds(2));
-
-        Assert.True(viewModel.IsThinking);
-        Assert.Equal("Thinking...", viewModel.ActivityMessage);
-        Assert.Equal(["Called tool: read_logs"], viewModel.ActivityMessages.ToArray());
-
-        pipeline.Release.SetResult();
-        await sendTask.WaitAsync(TimeSpan.FromSeconds(2));
-
-        Assert.False(viewModel.IsThinking);
-        Assert.Empty(viewModel.ActivityMessages);
-        AssertCompletedActivity(viewModel, ["Thinking...", "Called tool: read_logs"], "Updated rules.");
-    }
-
-    [Fact]
-    public async Task ViewModelShowsVisibleThinkingOutputInActivityExpander()
-    {
-        var pipeline = new ReportingRulesEditorPipeline(
-            "Updated rules.",
-            WorkflowRulesEditorActivityUpdate.Thinking("Checked the recent logs."),
-            WorkflowRulesEditorActivityUpdate.ToolCall("read_logs"),
-            WorkflowRulesEditorActivityUpdate.Thinking("Matched the rule to the user request."));
-        var viewModel = new WorkflowRulesEditorChatViewModel(pipeline)
-        {
-            Draft = "check logs"
-        };
-
-        var sendTask = viewModel.SendAsync();
-        await pipeline.Reported.Task.WaitAsync(TimeSpan.FromSeconds(2));
-
-        Assert.True(viewModel.IsThinking);
-        Assert.Equal("Thinking...", viewModel.ActivityMessage);
-        Assert.Equal(
-            [
-                "Checked the recent logs.",
-                "Called tool: read_logs",
-                "Matched the rule to the user request."
-            ],
-            viewModel.ActivityMessages.ToArray());
-
-        pipeline.Release.SetResult();
-        await sendTask.WaitAsync(TimeSpan.FromSeconds(2));
-
-        Assert.False(viewModel.IsThinking);
-        Assert.Empty(viewModel.ActivityMessages);
-        AssertCompletedActivity(
-            viewModel,
-            [
-                "Checked the recent logs.",
-                "Called tool: read_logs",
-                "Matched the rule to the user request."
-            ],
-            "Updated rules.");
-    }
-
-    [Fact]
-    public async Task ViewModelMarshalsBackgroundActivityUpdatesToCapturedContext()
-    {
-        var previousContext = SynchronizationContext.Current;
-        var context = new RecordingSynchronizationContext();
-        var pipeline = new BackgroundReportingRulesEditorPipeline(
-            "Updated rules.",
-            WorkflowRulesEditorActivityUpdate.Thinking("Checked the recent logs."));
-        var viewModel = new WorkflowRulesEditorChatViewModel(pipeline)
-        {
-            Draft = "check logs"
-        };
-
-        SynchronizationContext.SetSynchronizationContext(context);
-        var sendTask = viewModel.SendAsync();
-        SynchronizationContext.SetSynchronizationContext(previousContext);
-        await pipeline.Reported.Task.WaitAsync(TimeSpan.FromSeconds(2));
-
-        Assert.True(viewModel.IsThinking);
-        Assert.Equal(["Checked the recent logs."], viewModel.ActivityMessages.ToArray());
-        Assert.Equal(1, context.SendCount);
-
-        pipeline.Release.SetResult();
-        await sendTask.WaitAsync(TimeSpan.FromSeconds(2));
-
-        AssertCompletedActivity(viewModel, ["Checked the recent logs."], "Updated rules.");
-    }
-
-    [Fact]
-    public async Task ViewModelKeepsActivityItemsOutOfModelConversation()
-    {
-        var pipeline = new BlockingRulesEditorPipeline("Updated rules.");
-        var viewModel = new WorkflowRulesEditorChatViewModel(pipeline)
-        {
-            Draft = "first"
-        };
-
-        var firstSend = viewModel.SendAsync();
-        await pipeline.Started.Task.WaitAsync(TimeSpan.FromSeconds(2));
-        pipeline.Release.SetResult();
-        await firstSend.WaitAsync(TimeSpan.FromSeconds(2));
-
-        pipeline.Reset("Second response.");
-        viewModel.Draft = "second";
-        var secondSend = viewModel.SendAsync();
-        await pipeline.Started.Task.WaitAsync(TimeSpan.FromSeconds(2));
-
-        Assert.Equal(
-            [
-                new WorkflowRulesEditorChatMessage(WorkflowRulesEditorChatRole.User, "first"),
-                new WorkflowRulesEditorChatMessage(WorkflowRulesEditorChatRole.Agent, "Updated rules.")
-            ],
-            pipeline.LastConversation);
-
-        pipeline.Release.SetResult();
-        await secondSend.WaitAsync(TimeSpan.FromSeconds(2));
-    }
-
-    private static void AssertCompletedActivity(
-        WorkflowRulesEditorChatViewModel viewModel,
-        IReadOnlyList<string> expectedActivityLines,
-        string expectedAgentMessage)
-    {
-        Assert.Equal(3, viewModel.Messages.Count);
-        Assert.Equal(WorkflowRulesEditorConversationItemKind.Activity, viewModel.Messages[1].Kind);
-        Assert.StartsWith("Worked for ", viewModel.Messages[1].Content);
-        Assert.Equal(expectedActivityLines, viewModel.Messages[1].ActivityLines);
-        Assert.Equal(WorkflowRulesEditorConversationItemKind.Message, viewModel.Messages[2].Kind);
-        Assert.Equal(WorkflowRulesEditorChatRole.Agent, viewModel.Messages[2].Message?.Role);
-        Assert.Equal(expectedAgentMessage, viewModel.Messages[2].Message?.Content);
+        Assert.Equal("Updated rules.", viewModel.Messages[1].Content);
     }
 
     [Theory]
@@ -1380,36 +1196,31 @@ public sealed class WorkflowRulesEditorTests
 
     private sealed class BlockingRulesEditorPipeline : IWorkflowRulesEditorChatPipeline
     {
-        private string response;
+        private readonly string response;
 
         public BlockingRulesEditorPipeline(string response)
         {
             this.response = response;
         }
 
-        public TaskCompletionSource Started { get; private set; } = new(TaskCreationOptions.RunContinuationsAsynchronously);
+        public TaskCompletionSource Started { get; } = new(TaskCreationOptions.RunContinuationsAsynchronously);
 
-        public TaskCompletionSource Release { get; private set; } = new(TaskCreationOptions.RunContinuationsAsynchronously);
-
-        public IReadOnlyList<WorkflowRulesEditorChatMessage> LastConversation { get; private set; } = [];
-
-        public void Reset(string nextResponse)
-        {
-            response = nextResponse;
-            Started = new TaskCompletionSource(TaskCreationOptions.RunContinuationsAsynchronously);
-            Release = new TaskCompletionSource(TaskCreationOptions.RunContinuationsAsynchronously);
-        }
+        public TaskCompletionSource Release { get; } = new(TaskCreationOptions.RunContinuationsAsynchronously);
 
         public async Task<WorkflowRulesEditorChatResult> SendAsync(
             IReadOnlyList<WorkflowRulesEditorChatMessage> conversation,
             string userMessage,
             CancellationToken cancellationToken,
-            Action<WorkflowRulesEditorActivityUpdate>? activityChanged = null)
+            Action<string>? statusChanged = null)
         {
-            LastConversation = conversation;
             Started.SetResult();
             await Release.Task.WaitAsync(cancellationToken);
-            return new WorkflowRulesEditorChatResult(response);
+            return new WorkflowRulesEditorChatResult(
+                response,
+                conversation
+                    .Append(new WorkflowRulesEditorChatMessage(WorkflowRulesEditorChatRole.User, userMessage))
+                    .Append(new WorkflowRulesEditorChatMessage(WorkflowRulesEditorChatRole.Agent, response))
+                    .ToList());
         }
     }
 
@@ -1426,23 +1237,19 @@ public sealed class WorkflowRulesEditorTests
             IReadOnlyList<WorkflowRulesEditorChatMessage> conversation,
             string userMessage,
             CancellationToken cancellationToken,
-            Action<WorkflowRulesEditorActivityUpdate>? activityChanged = null)
+            Action<string>? statusChanged = null)
         {
             throw exception;
         }
     }
 
-    private sealed class ReportingRulesEditorPipeline : IWorkflowRulesEditorChatPipeline
+    private sealed class StatusReportingRulesEditorPipeline : IWorkflowRulesEditorChatPipeline
     {
         private readonly string response;
-        private readonly IReadOnlyList<WorkflowRulesEditorActivityUpdate> updates;
 
-        public ReportingRulesEditorPipeline(
-            string response,
-            params WorkflowRulesEditorActivityUpdate[] updates)
+        public StatusReportingRulesEditorPipeline(string response)
         {
             this.response = response;
-            this.updates = updates;
         }
 
         public TaskCompletionSource Reported { get; } = new(TaskCreationOptions.RunContinuationsAsynchronously);
@@ -1453,57 +1260,17 @@ public sealed class WorkflowRulesEditorTests
             IReadOnlyList<WorkflowRulesEditorChatMessage> conversation,
             string userMessage,
             CancellationToken cancellationToken,
-            Action<WorkflowRulesEditorActivityUpdate>? activityChanged = null)
+            Action<string>? statusChanged = null)
         {
-            foreach (var update in updates)
-            {
-                activityChanged?.Invoke(update);
-            }
-
+            statusChanged?.Invoke("Reconnecting...");
             Reported.SetResult();
             await Release.Task.WaitAsync(cancellationToken);
-            return new WorkflowRulesEditorChatResult(response);
-        }
-    }
-
-    private sealed class BackgroundReportingRulesEditorPipeline : IWorkflowRulesEditorChatPipeline
-    {
-        private readonly string response;
-        private readonly WorkflowRulesEditorActivityUpdate update;
-
-        public BackgroundReportingRulesEditorPipeline(
-            string response,
-            WorkflowRulesEditorActivityUpdate update)
-        {
-            this.response = response;
-            this.update = update;
-        }
-
-        public TaskCompletionSource Reported { get; } = new(TaskCreationOptions.RunContinuationsAsynchronously);
-
-        public TaskCompletionSource Release { get; } = new(TaskCreationOptions.RunContinuationsAsynchronously);
-
-        public async Task<WorkflowRulesEditorChatResult> SendAsync(
-            IReadOnlyList<WorkflowRulesEditorChatMessage> conversation,
-            string userMessage,
-            CancellationToken cancellationToken,
-            Action<WorkflowRulesEditorActivityUpdate>? activityChanged = null)
-        {
-            await Task.Run(() => activityChanged?.Invoke(update), cancellationToken);
-            Reported.SetResult();
-            await Release.Task.WaitAsync(cancellationToken);
-            return new WorkflowRulesEditorChatResult(response);
-        }
-    }
-
-    private sealed class RecordingSynchronizationContext : SynchronizationContext
-    {
-        public int SendCount { get; private set; }
-
-        public override void Send(SendOrPostCallback callback, object? state)
-        {
-            SendCount++;
-            callback(state);
+            return new WorkflowRulesEditorChatResult(
+                response,
+                conversation
+                    .Append(new WorkflowRulesEditorChatMessage(WorkflowRulesEditorChatRole.User, userMessage))
+                    .Append(new WorkflowRulesEditorChatMessage(WorkflowRulesEditorChatRole.Agent, response))
+                    .ToList());
         }
     }
 

MeetingAssistant/MeetingAssistant.csproj

@@ -20,19 +20,19 @@
   </ItemGroup>
 
   <ItemGroup>
-    <PackageReference Include="Microsoft.Agents.AI.OpenAI" Version="1.16.0" />
+    <PackageReference Include="Microsoft.Agents.AI.OpenAI" Version="1.11.1" />
     <PackageReference Include="DiffPlex" Version="1.9.0" />
     <PackageReference Include="Microsoft.CognitiveServices.Speech" Version="$(MicrosoftSpeechVersion)" />
     <PackageReference Include="Microsoft.CognitiveServices.Speech.Extension.MAS" Version="$(MicrosoftSpeechVersion)" ExcludeAssets="build" />
-    <PackageReference Include="Microsoft.EntityFrameworkCore.Sqlite" Version="10.0.10" />
+    <PackageReference Include="Microsoft.EntityFrameworkCore.Sqlite" Version="10.0.9" />
     <PackageReference Include="NAudio" Version="2.3.0" />
-    <PackageReference Include="NCalcSync" Version="6.4.0" />
+    <PackageReference Include="NCalcSync" Version="6.3.3" />
     <PackageReference Include="RazorLight" Version="2.3.1" />
-    <PackageReference Include="SQLitePCLRaw.bundle_e_sqlite3" Version="3.0.5" />
-    <PackageReference Include="System.Drawing.Common" Version="10.0.10" />
+    <PackageReference Include="SQLitePCLRaw.bundle_e_sqlite3" Version="3.0.3" />
+    <PackageReference Include="System.Drawing.Common" Version="10.0.9" />
     <PackageReference Include="Whisper.net" Version="1.9.1" />
     <PackageReference Include="Whisper.net.Runtime" Version="1.9.1" />
-    <PackageReference Include="YamlDotNet" Version="18.1.0" />
+    <PackageReference Include="YamlDotNet" Version="18.0.0" />
   </ItemGroup>
 
   <ItemGroup Condition="$([MSBuild]::GetTargetPlatformIdentifier('$(TargetFramework)')) == 'windows'">

MeetingAssistant/MeetingAssistantOptions.cs

@@ -383,8 +383,6 @@ public sealed class AgentOptions
 
     public string Model { get; set; } = "chatgpt/gpt-5.5";
 
-    public bool UseStreaming { get; set; } = true;
-
     public bool EnableThinking { get; set; } = true;
 
     public ReasoningEffortOption ReasoningEffort { get; set; } = ReasoningEffortOption.Medium;
@@ -416,8 +414,6 @@ public sealed class WorkflowRulesEditorOptions
 
     public string? Model { get; set; }
 
-    public bool? UseStreaming { get; set; }
-
     public bool? EnableThinking { get; set; }
 
     public ReasoningEffortOption? ReasoningEffort { get; set; }
@@ -446,7 +442,6 @@ public sealed class WorkflowRulesEditorOptions
             Key = string.IsNullOrWhiteSpace(Key) ? defaults.Key : Key,
             KeyEnv = string.IsNullOrWhiteSpace(KeyEnv) ? defaults.KeyEnv : KeyEnv!,
             Model = string.IsNullOrWhiteSpace(Model) ? defaults.Model : Model!,
-            UseStreaming = UseStreaming ?? defaults.UseStreaming,
             EnableThinking = EnableThinking ?? defaults.EnableThinking,
             ReasoningEffort = ReasoningEffort ?? defaults.ReasoningEffort,
             ReconnectionAttempts = ReconnectionAttempts ?? defaults.ReconnectionAttempts,

MeetingAssistant/MeetingNotes/MarkdownMeetingNoteStore.cs

@@ -1,6 +1,4 @@
 using Microsoft.Extensions.Options;
-using System.Globalization;
-using System.Text;
 using YamlDotNet.Core;
 using YamlDotNet.Serialization;
 
@@ -115,24 +113,7 @@ public sealed class MarkdownMeetingNoteStore : IMeetingNoteStore
 
     private static string EscapeQuoted(string value)
     {
-        var escaped = new StringBuilder(value.Length + 2);
-        escaped.Append('"');
-        foreach (var character in value)
-        {
-            escaped.Append(character switch
-            {
-                '\\' => "\\\\",
-                '"' => "\\\"",
-                '\r' => "\\r",
-                '\n' => "\\n",
-                '\t' => "\\t",
-                < ' ' => "\\x" + ((int)character).ToString("X2", CultureInfo.InvariantCulture),
-                _ => character.ToString()
-            });
-        }
-
-        escaped.Append('"');
-        return escaped.ToString();
+        return $"\"{value.Replace("\"", "\\\"", StringComparison.Ordinal)}\"";
     }
 
     private static string EscapeNullableDateTime(DateTimeOffset? value)
@@ -154,7 +135,7 @@ public sealed class MarkdownMeetingNoteStore : IMeetingNoteStore
             return "\"\"";
         }
 
-        if (RequiresQuotedScalar(value))
+        if (value.Contains(':', StringComparison.Ordinal) || value.StartsWith("[", StringComparison.Ordinal))
         {
             return EscapeQuoted(value);
         }
@@ -162,49 +143,6 @@ public sealed class MarkdownMeetingNoteStore : IMeetingNoteStore
         return value;
     }
 
-    private static bool RequiresQuotedScalar(string value)
-    {
-        if (char.IsWhiteSpace(value[0]) || char.IsWhiteSpace(value[^1]))
-        {
-            return true;
-        }
-
-        if (value.Contains(':', StringComparison.Ordinal) ||
-            value.Contains('\'', StringComparison.Ordinal) ||
-            value.Contains(" #", StringComparison.Ordinal) ||
-            value.Any(char.IsControl))
-        {
-            return true;
-        }
-
-        if (IsYamlIndicator(value[0]))
-        {
-            return true;
-        }
-
-        if (IsYamlCoreSchemaKeyword(value))
-        {
-            return true;
-        }
-
-        return DateTimeOffset.TryParse(value, out _) ||
-            double.TryParse(value, CultureInfo.InvariantCulture, out _);
-    }
-
-    private static bool IsYamlIndicator(char character)
-    {
-        return character is '-' or '?' or ':' or ',' or '[' or ']' or '{' or '}' or
-            '#' or '&' or '*' or '!' or '|' or '>' or '\'' or '"' or '%' or '@' or '`';
-    }
-
-    private static bool IsYamlCoreSchemaKeyword(string value)
-    {
-        return string.Equals(value, "true", StringComparison.OrdinalIgnoreCase) ||
-            string.Equals(value, "false", StringComparison.OrdinalIgnoreCase) ||
-            string.Equals(value, "null", StringComparison.OrdinalIgnoreCase) ||
-            value == "~";
-    }
-
     private sealed class MeetingNoteYaml
     {
         [YamlMember(Alias = "title")]

MeetingAssistant/Recording/MeetingRecordingCoordinator.cs

@@ -213,12 +213,7 @@ public sealed class MeetingRecordingCoordinator
             currentMeetingNote = await meetingNoteStore.ReadAsync(currentMeetingNote.Path, cancellationToken);
             if (suppliedMetadata is not null)
             {
-                await ApplyMeetingMetadataAsync(
-                    currentArtifacts,
-                    currentMeetingNote,
-                    suppliedMetadata,
-                    runOptions,
-                    cancellationToken);
+                await ApplyMeetingMetadataAsync(currentMeetingNote, suppliedMetadata, runOptions, cancellationToken);
                 currentMeetingNote = await meetingNoteStore.SaveAsync(
                     currentMeetingNote,
                     runOptions,
@@ -1035,7 +1030,7 @@ public sealed class MeetingRecordingCoordinator
                 }
 
                 var meetingNote = await meetingNoteStore.ReadAsync(run.MeetingNotePath, CancellationToken.None);
-                await ApplyMeetingMetadataAsync(run.Artifacts, meetingNote, metadata, run.Options, CancellationToken.None);
+                await ApplyMeetingMetadataAsync(meetingNote, metadata, run.Options, CancellationToken.None);
                 meetingNote = await meetingNoteStore.SaveAsync(meetingNote, run.Options, CancellationToken.None);
                 if (currentMeetingNote?.Path == meetingNote.Path)
                 {
@@ -1087,7 +1082,6 @@ public sealed class MeetingRecordingCoordinator
     }
 
     private async Task ApplyMeetingMetadataAsync(
-        MeetingSessionArtifacts artifacts,
         MeetingNote meetingNote,
         MeetingMetadata metadata,
         MeetingAssistantOptions options,
@@ -1110,39 +1104,10 @@ public sealed class MeetingRecordingCoordinator
                 return;
             }
 
-            var canonicalized = await CanonicalizeAttendeesAsync(metadata.Attendees, cancellationToken);
-            meetingNote.Frontmatter.Attendees = await TransformAttendeesAsync(
-                artifacts,
-                canonicalized,
-                options,
-                cancellationToken);
+            meetingNote.Frontmatter.Attendees = await CanonicalizeAttendeesAsync(metadata.Attendees, cancellationToken);
         }
     }
 
-    private async Task<List<string>> TransformAttendeesAsync(
-        MeetingSessionArtifacts artifacts,
-        IReadOnlyList<string> attendees,
-        MeetingAssistantOptions options,
-        CancellationToken cancellationToken)
-    {
-        var transformed = new List<string>();
-        foreach (var attendee in attendees)
-        {
-            var value = await TransformAttendeeAsync(artifacts, attendee, options, cancellationToken);
-            var storageValue = string.Equals(value, attendee, StringComparison.Ordinal)
-                ? attendee.Trim()
-                : NormalizeAttendeeName(value);
-            var normalized = NormalizeAttendeeName(storageValue);
-            if (!string.IsNullOrWhiteSpace(normalized) &&
-                !transformed.Contains(normalized, StringComparer.OrdinalIgnoreCase))
-            {
-                transformed.Add(storageValue);
-            }
-        }
-
-        return transformed;
-    }
-
     private async Task<List<string>> CanonicalizeAttendeesAsync(
         IReadOnlyList<string> attendees,
         CancellationToken cancellationToken)
@@ -1457,17 +1422,8 @@ public sealed class MeetingRecordingCoordinator
                 continue;
             }
 
-            var transformedDisplayName = await TransformAttendeeAsync(
-                artifacts,
-                match.DisplayName.Trim(),
-                runOptions,
-                cancellationToken);
-            var displayName = string.Equals(transformedDisplayName, match.DisplayName.Trim(), StringComparison.Ordinal)
-                ? match.DisplayName.Trim()
-                : NormalizeAttendeeName(transformedDisplayName);
             var acceptedNames = match.AcceptedNames
                 .Append(match.DisplayName)
-                .Append(displayName)
                 .Select(NormalizeAttendeeName)
                 .Where(name => !string.IsNullOrWhiteSpace(name))
                 .Distinct(StringComparer.OrdinalIgnoreCase)
@@ -1486,6 +1442,7 @@ public sealed class MeetingRecordingCoordinator
                 continue;
             }
 
+            var displayName = match.DisplayName.Trim();
             meetingNote.Frontmatter.Attendees.Add(displayName);
             existingNames.Add(NormalizeAttendeeName(displayName));
             changed = true;
@@ -1545,18 +1502,6 @@ public sealed class MeetingRecordingCoordinator
         return MeetingAttendeeNames.NormalizeDisplayName(attendee);
     }
 
-    private async Task<string> TransformAttendeeAsync(
-        MeetingSessionArtifacts artifacts,
-        string attendee,
-        MeetingAssistantOptions options,
-        CancellationToken cancellationToken)
-    {
-        return await meetingWorkflowEngine.TransformAttendeeAsync(
-            MeetingWorkflowEvent.AttendeeAdded(artifacts, attendee),
-            options,
-            cancellationToken);
-    }
-
     private static bool RemoveDuplicateAcceptedAliases(
         List<string> attendees,
         string displayName,

MeetingAssistant/Screenshots/MeetingScreenshotService.cs

@@ -5,7 +5,6 @@ using System.Runtime.InteropServices;
 using System.Text.RegularExpressions;
 using MeetingAssistant.MeetingNotes;
 using MeetingAssistant.Speakers;
-using MeetingAssistant.Workflow;
 
 namespace MeetingAssistant.Screenshots;
 
@@ -150,7 +149,6 @@ public sealed partial class MeetingScreenshotService :
     private readonly IMeetingArtifactStore artifactStore;
     private readonly IMeetingNoteStore meetingNoteStore;
     private readonly ISpeakerIdentityAttendeeCanonicalizer attendeeCanonicalizer;
-    private readonly IMeetingWorkflowEngine meetingWorkflowEngine;
     private readonly IScreenshotOcrClient ocrClient;
     private readonly ILogger<MeetingScreenshotService> logger;
     private readonly ConcurrentDictionary<string, List<PendingOcrTask>> pendingOcrByContext = new(StringComparer.OrdinalIgnoreCase);
@@ -162,14 +160,12 @@ public sealed partial class MeetingScreenshotService :
         IMeetingNoteStore meetingNoteStore,
         ISpeakerIdentityAttendeeCanonicalizer attendeeCanonicalizer,
         IScreenshotOcrClient ocrClient,
-        ILogger<MeetingScreenshotService> logger,
-        IMeetingWorkflowEngine? meetingWorkflowEngine = null)
+        ILogger<MeetingScreenshotService> logger)
     {
         this.screenshotCapture = screenshotCapture;
         this.artifactStore = artifactStore;
         this.meetingNoteStore = meetingNoteStore;
         this.attendeeCanonicalizer = attendeeCanonicalizer;
-        this.meetingWorkflowEngine = meetingWorkflowEngine ?? NoopMeetingWorkflowEngine.Instance;
         this.ocrClient = ocrClient;
         this.logger = logger;
     }
@@ -438,7 +434,6 @@ public sealed partial class MeetingScreenshotService :
                 await AddOcrAttendeesAsync(
                     artifacts,
                     result.Attendees,
-                    options,
                     CancellationToken.None);
             }
         }
@@ -482,7 +477,6 @@ public sealed partial class MeetingScreenshotService :
     private async Task AddOcrAttendeesAsync(
         MeetingSessionArtifacts artifacts,
         IReadOnlyList<string> attendees,
-        MeetingAssistantOptions options,
         CancellationToken cancellationToken)
     {
         var additions = MeetingAttendeeNames.NormalizeDistinct(attendees);
@@ -494,13 +488,8 @@ public sealed partial class MeetingScreenshotService :
         try
         {
             var meetingNote = await meetingNoteStore.ReadAsync(artifacts.MeetingNotePath, cancellationToken);
-            var transformedAdditions = await TransformAttendeesAsync(
-                artifacts,
-                additions,
-                options,
-                cancellationToken);
             var canonicalized = await attendeeCanonicalizer.CanonicalizeAsync(
-                meetingNote.Frontmatter.Attendees.Concat(transformedAdditions).ToList(),
+                meetingNote.Frontmatter.Attendees.Concat(additions).ToList(),
                 cancellationToken);
             if (meetingNote.Frontmatter.Attendees.SequenceEqual(canonicalized, StringComparer.Ordinal))
             {
@@ -523,30 +512,6 @@ public sealed partial class MeetingScreenshotService :
         }
     }
 
-    private async Task<IReadOnlyList<string>> TransformAttendeesAsync(
-        MeetingSessionArtifacts artifacts,
-        IReadOnlyList<string> attendees,
-        MeetingAssistantOptions options,
-        CancellationToken cancellationToken)
-    {
-        var transformed = new List<string>();
-        foreach (var attendee in attendees)
-        {
-            var value = await meetingWorkflowEngine.TransformAttendeeAsync(
-                MeetingWorkflowEvent.AttendeeAdded(artifacts, attendee),
-                options,
-                cancellationToken);
-            var normalized = MeetingAttendeeNames.NormalizeDisplayName(value);
-            if (!string.IsNullOrWhiteSpace(normalized) &&
-                !transformed.Contains(normalized, StringComparer.OrdinalIgnoreCase))
-            {
-                transformed.Add(normalized);
-            }
-        }
-
-        return transformed;
-    }
-
     private async Task<bool> ReplaceOcrPlaceholderAsync(
         string assistantContextPath,
         string screenshotId,

MeetingAssistant/Summary/FunctionInvocationGuard.cs

@@ -1,26 +0,0 @@
-using System.Text.Json.Nodes;
-using Microsoft.Extensions.AI;
-
-namespace MeetingAssistant.Summary;
-
-internal static class FunctionInvocationGuard
-{
-    public static ValueTask<object?> InvokeAsync(
-        FunctionInvocationContext context,
-        CancellationToken cancellationToken)
-    {
-        if (context.CallContent.Exception is not null)
-        {
-            return ValueTask.FromResult<object?>(new JsonObject
-            {
-                ["error"] = new JsonObject
-                {
-                    ["code"] = "invalid_tool_arguments",
-                    ["message"] = "Tool arguments must be a valid JSON object."
-                }
-            });
-        }
-
-        return context.Function.InvokeAsync(context.Arguments, cancellationToken);
-    }
-}

MeetingAssistant/Summary/LiteLlmResponsesChatClient.cs

@@ -1,28 +1,20 @@
 using System.Net.Http.Headers;
-using System.ClientModel;
-using System.ClientModel.Primitives;
 using System.Text;
 using System.Text.Json;
 using System.Text.Json.Nodes;
 using Microsoft.Agents.AI.Compaction;
 using Microsoft.Extensions.AI;
 using Microsoft.Extensions.Logging;
-using OpenAI;
-using OpenAI.Responses;
 
 namespace MeetingAssistant.Summary;
 
 #pragma warning disable MAAI001
-#pragma warning disable OPENAI001
 public sealed class LiteLlmResponsesChatClient : IChatClient
 {
     private static readonly JsonSerializerOptions JsonOptions = new(JsonSerializerDefaults.Web);
 
     private readonly HttpClient httpClient;
-    private readonly ResponsesClient responsesClient;
-    private readonly AsyncLocal<string?> requestInitiator = new();
     private readonly string model;
-    private readonly bool useStreaming;
     private readonly bool enableThinking;
     private readonly string reasoningEffort;
     private readonly int reconnectionAttempts;
@@ -30,7 +22,6 @@ public sealed class LiteLlmResponsesChatClient : IChatClient
     private readonly LiteLlmResponsesCompactionOptions? compactionOptions;
     private readonly ILogger? logger;
     private readonly Action? retrying;
-    private readonly Action<string>? reasoningSummaryChanged;
     private int responsesRequestCount;
 
     public LiteLlmResponsesChatClient(
@@ -44,9 +35,7 @@ public sealed class LiteLlmResponsesChatClient : IChatClient
         LiteLlmResponsesCompactionOptions? compactionOptions = null,
         ILogger? logger = null,
         bool firstRequestIsUser = true,
-        Action? retrying = null,
-        Action<string>? reasoningSummaryChanged = null,
-        bool useStreaming = true)
+        Action? retrying = null)
         : this(
             new HttpClient { BaseAddress = NormalizeEndpoint(endpoint) },
             apiKey,
@@ -58,9 +47,7 @@ public sealed class LiteLlmResponsesChatClient : IChatClient
             compactionOptions,
             logger,
             firstRequestIsUser,
-            retrying,
-            reasoningSummaryChanged,
-            useStreaming)
+            retrying)
     {
     }
 
@@ -75,15 +62,11 @@ public sealed class LiteLlmResponsesChatClient : IChatClient
         LiteLlmResponsesCompactionOptions? compactionOptions = null,
         ILogger? logger = null,
         bool firstRequestIsUser = true,
-        Action? retrying = null,
-        Action<string>? reasoningSummaryChanged = null,
-        bool useStreaming = true)
+        Action? retrying = null)
     {
         this.httpClient = httpClient;
         this.httpClient.DefaultRequestHeaders.Authorization = new AuthenticationHeaderValue("Bearer", apiKey);
-        responsesClient = CreateResponsesClient(httpClient, apiKey, requestInitiator);
         this.model = model;
-        this.useStreaming = useStreaming;
         this.enableThinking = enableThinking;
         this.reasoningEffort = reasoningEffort;
         this.reconnectionAttempts = Math.Max(0, reconnectionAttempts);
@@ -91,7 +74,6 @@ public sealed class LiteLlmResponsesChatClient : IChatClient
         this.compactionOptions = compactionOptions;
         this.logger = logger;
         this.retrying = retrying;
-        this.reasoningSummaryChanged = reasoningSummaryChanged;
         responsesRequestCount = firstRequestIsUser ? 0 : 1;
     }
 
@@ -100,50 +82,19 @@ public sealed class LiteLlmResponsesChatClient : IChatClient
         httpClient.Dispose();
     }
 
-    internal static LiteLlmResponsesChatClient Create(
-        AgentOptions options,
-        string apiKey,
-        LiteLlmResponsesCompactionOptions? compactionOptions,
-        ILogger? logger,
-        bool firstRequestIsUser,
-        Action? retrying = null,
-        Action<string>? reasoningSummaryChanged = null)
-    {
-        return new LiteLlmResponsesChatClient(
-            new Uri(options.Endpoint),
-            apiKey,
-            options.Model,
-            options.EnableThinking,
-            options.ReasoningEffort switch
-            {
-                ReasoningEffortOption.None => "none",
-                ReasoningEffortOption.Low => "low",
-                ReasoningEffortOption.High => "high",
-                ReasoningEffortOption.ExtraHigh => "xhigh",
-                _ => "medium"
-            },
-            options.ReconnectionAttempts,
-            options.ReconnectionDelay,
-            compactionOptions,
-            logger,
-            firstRequestIsUser,
-            retrying,
-            reasoningSummaryChanged,
-            options.UseStreaming);
-    }
-
     public async Task<ChatResponse> GetResponseAsync(
         IEnumerable<ChatMessage> messages,
         ChatOptions? options = null,
         CancellationToken cancellationToken = default)
     {
         var payload = await CreateCompactedPayloadAsync(messages.ToList(), options, cancellationToken).ConfigureAwait(false);
-        var result = await PostWithRetryAsync(payload, cancellationToken).ConfigureAwait(false);
-        ReportVisibleReasoningSummaries(result.ReasoningSummaries);
-        LogResponseDiagnostics(result.Response);
-        ThrowIfResponseHasNoContent(result.Response);
-        LogResponseUsage(result.Response.Usage);
-        return result.Response;
+        var responseJson = await PostWithRetryAsync(payload, cancellationToken).ConfigureAwait(false);
+
+        var response = ParseResponseJson(responseJson);
+        LogResponseDiagnostics(responseJson, response);
+        ThrowIfResponseHasNoContent(responseJson, response);
+        LogResponseUsage(response.Usage);
+        return response;
     }
 
     public async IAsyncEnumerable<ChatResponseUpdate> GetStreamingResponseAsync(
@@ -170,36 +121,45 @@ public sealed class LiteLlmResponsesChatClient : IChatClient
             : null;
     }
 
-    private static IReadOnlyList<string> ExtractVisibleReasoningSummaries(ChatResponse response)
+    public static ChatResponse ParseResponseJson(string responseJson)
     {
-        return response.Messages
-            .SelectMany(message => message.Contents)
-            .OfType<TextReasoningContent>()
-            .Select(content => content.Text)
-            .Where(text => !string.IsNullOrWhiteSpace(text))
-            .ToArray();
-    }
+        using var document = JsonDocument.Parse(responseJson);
+        var root = document.RootElement;
+        var contents = new List<AIContent>();
 
-    private void ReportVisibleReasoningSummaries(IReadOnlyList<string> summaries)
-    {
-        if (reasoningSummaryChanged is null)
+        if (root.TryGetProperty("output", out var output) && output.ValueKind == JsonValueKind.Array)
         {
-            return;
-        }
-
-        foreach (var summary in summaries)
-        {
-            try
+            foreach (var item in output.EnumerateArray())
             {
-                reasoningSummaryChanged(summary);
-            }
-            catch (Exception exception)
-            {
-                logger?.LogWarning(
-                    exception,
-                    "Reasoning summary callback failed; continuing with the LiteLLM response.");
+                var type = GetString(item, "type");
+                if (type == "message")
+                {
+                    AddMessageContent(contents, item);
+                }
+                else if (type == "function_call")
+                {
+                    contents.Add(new FunctionCallContent(
+                        GetRequiredString(item, "call_id"),
+                        GetRequiredString(item, "name"),
+                        ParseArguments(GetString(item, "arguments"))));
+                }
             }
         }
+
+        var message = new ChatMessage
+        {
+            Role = ChatRole.Assistant,
+            Contents = contents
+        };
+
+        return new ChatResponse(message)
+        {
+            ResponseId = GetString(root, "id"),
+            ModelId = GetString(root, "model"),
+            CreatedAt = GetUnixTimestamp(root, "created_at"),
+            Usage = ParseUsage(root),
+            RawRepresentation = responseJson
+        };
     }
 
     private async Task<JsonObject> CreateCompactedPayloadAsync(
@@ -385,9 +345,7 @@ public sealed class LiteLlmResponsesChatClient : IChatClient
         return payload;
     }
 
-    private async Task<ResponsesChatResult> PostWithRetryAsync(
-        JsonObject payload,
-        CancellationToken cancellationToken)
+    private async Task<string> PostWithRetryAsync(JsonObject payload, CancellationToken cancellationToken)
     {
         var payloadJson = payload.ToJsonString(JsonOptions);
         Exception? lastException = null;
@@ -398,36 +356,26 @@ public sealed class LiteLlmResponsesChatClient : IChatClient
             {
                 var initiator = NextInitiator();
                 LogRequestDiagnostics(payload, initiator, attempt);
+                using var request = CreateJsonRequest(
+                    "responses",
+                    payloadJson,
+                    initiator);
+                using var response = await httpClient.SendAsync(request, cancellationToken).ConfigureAwait(false);
+                var responseJson = await response.Content.ReadAsStringAsync(cancellationToken).ConfigureAwait(false);
 
-                var createOptions = ModelReaderWriter.Read<CreateResponseOptions>(
-                        BinaryData.FromString(payloadJson),
-                        ModelReaderWriterOptions.Json)
-                    ?? throw new InvalidOperationException("Unable to create OpenAI Responses request options.");
-                createOptions.StreamingEnabled = useStreaming;
+                if (response.IsSuccessStatusCode)
+                {
+                    return responseJson;
+                }
 
-                var previousInitiator = requestInitiator.Value;
-                requestInitiator.Value = initiator;
-                try
+                var exception = new InvalidOperationException(
+                    $"LiteLLM Responses request failed with {(int)response.StatusCode} {response.ReasonPhrase}: {responseJson}");
+                if (!IsRetryableStatusCode((int)response.StatusCode) || attempt == reconnectionAttempts)
                 {
-                    return useStreaming
-                        ? await GetStreamingResponseAsync(createOptions, cancellationToken).ConfigureAwait(false)
-                        : CreateNonStreamingResult((await responsesClient
-                                .CreateResponseAsync(createOptions, cancellationToken)
-                                .ConfigureAwait(false))
-                            .Value
-                            .AsChatResponse(createOptions));
+                    throw exception;
                 }
-                finally
-                {
-                    requestInitiator.Value = previousInitiator;
-                }
-            }
-            catch (ClientResultException exception)
-                when (!IsRetryableStatusCode(exception.Status) || attempt == reconnectionAttempts)
-            {
-                throw new InvalidOperationException(
-                    $"LiteLLM Responses request failed with {exception.Status}: {exception.Message}",
-                    exception);
+
+                lastException = exception;
             }
             catch (Exception exception) when (IsRetryableException(exception) && attempt < reconnectionAttempts)
             {
@@ -441,48 +389,6 @@ public sealed class LiteLlmResponsesChatClient : IChatClient
         throw lastException ?? new InvalidOperationException("LiteLLM Responses request failed.");
     }
 
-    private async Task<ResponsesChatResult> GetStreamingResponseAsync(
-        CreateResponseOptions createOptions,
-        CancellationToken cancellationToken)
-    {
-        var updates = new List<ChatResponseUpdate>();
-        var reasoningSummaries = new Dictionary<(string ItemId, int SummaryIndex), StringBuilder>();
-        await foreach (var update in responsesClient
-                           .CreateResponseStreamingAsync(createOptions, cancellationToken)
-                           .AsChatResponseUpdatesAsync(createOptions, cancellationToken)
-                           .ConfigureAwait(false))
-        {
-            updates.Add(update);
-            var reasoningUpdate = update.RawRepresentation
-                as StreamingResponseReasoningSummaryTextDeltaUpdate;
-            if (reasoningUpdate is null)
-            {
-                continue;
-            }
-
-            var key = (reasoningUpdate.ItemId, reasoningUpdate.SummaryIndex);
-            if (!reasoningSummaries.TryGetValue(key, out var summary))
-            {
-                summary = new StringBuilder();
-                reasoningSummaries[key] = summary;
-            }
-
-            summary.Append(reasoningUpdate.Delta);
-        }
-
-        return new ResponsesChatResult(
-            updates.ToChatResponse(),
-            reasoningSummaries.Values
-                .Select(summary => summary.ToString())
-                .Where(summary => !string.IsNullOrWhiteSpace(summary))
-                .ToArray());
-    }
-
-    private static ResponsesChatResult CreateNonStreamingResult(ChatResponse response)
-    {
-        return new ResponsesChatResult(response, ExtractVisibleReasoningSummaries(response));
-    }
-
     private HttpRequestMessage CreateJsonRequest(
         string requestUri,
         string payloadJson,
@@ -521,9 +427,7 @@ public sealed class LiteLlmResponsesChatClient : IChatClient
 
     private static bool IsRetryableException(Exception exception)
     {
-        return exception is ClientResultException clientException
-                && IsRetryableStatusCode(clientException.Status)
-            || exception is HttpRequestException or TaskCanceledException;
+        return exception is HttpRequestException or TaskCanceledException;
     }
 
     private void LogContextWindow(int estimatedTokens, bool compacted, string source)
@@ -595,18 +499,18 @@ public sealed class LiteLlmResponsesChatClient : IChatClient
             instructionsPreview);
     }
 
-    private void LogResponseDiagnostics(ChatResponse response)
+    private void LogResponseDiagnostics(string responseJson, ChatResponse response)
     {
         if (logger is null)
         {
             return;
         }
 
+        var outputTypes = GetOutputItemTypes(responseJson);
         var parsedTypes = response.Messages
             .SelectMany(message => message.Contents)
             .Select(content => content.GetType().Name)
             .ToArray();
-        var outputTypes = GetOutputItemTypes(response);
         var functionCalls = response.Messages
             .SelectMany(message => message.Contents)
             .OfType<FunctionCallContent>()
@@ -624,29 +528,40 @@ public sealed class LiteLlmResponsesChatClient : IChatClient
         if (parsedTypes.Length == 0)
         {
             logger.LogWarning(
-                "LiteLLM Responses response contained no parseable message text or function calls.");
+                "LiteLLM Responses response contained no parseable message text or function calls. Raw response preview: {ResponsePreview}",
+                Truncate(responseJson, maxLength: 6000));
         }
     }
 
-    private static void ThrowIfResponseHasNoContent(ChatResponse response)
+    private static void ThrowIfResponseHasNoContent(string responseJson, ChatResponse response)
     {
         if (response.Messages.SelectMany(message => message.Contents).Any())
         {
             return;
         }
 
-        var outputTypes = GetOutputItemTypes(response);
+        var outputTypes = GetOutputItemTypes(responseJson);
         throw new InvalidOperationException(
             "LiteLLM Responses returned no parseable message text or function calls. " +
             $"ResponseId={response.ResponseId ?? "<none>"}, outputTypes=[{string.Join(", ", outputTypes)}].");
     }
 
-    private static string[] GetOutputItemTypes(ChatResponse response)
+    private static string[] GetOutputItemTypes(string responseJson)
     {
-        return response.Messages
-            .SelectMany(message => message.Contents)
-            .Select(content => content.RawRepresentation?.GetType().Name ?? content.GetType().Name)
-            .ToArray();
+        try
+        {
+            using var document = JsonDocument.Parse(responseJson);
+            return document.RootElement.TryGetProperty("output", out var output) && output.ValueKind == JsonValueKind.Array
+                ? output
+                    .EnumerateArray()
+                    .Select(item => GetString(item, "type") ?? item.ValueKind.ToString())
+                    .ToArray()
+                : [];
+        }
+        catch (JsonException)
+        {
+            return ["<invalid-json>"];
+        }
     }
 
     private static string Truncate(string value, int maxLength)
@@ -672,19 +587,10 @@ public sealed class LiteLlmResponsesChatClient : IChatClient
 
         if (!string.IsNullOrWhiteSpace(text))
         {
-            var isAssistant = role == ChatRole.Assistant.Value;
             input.Add(new JsonObject
             {
-                ["type"] = "message",
-                ["role"] = isAssistant ? "assistant" : "user",
-                ["content"] = new JsonArray
-                {
-                    new JsonObject
-                    {
-                        ["type"] = isAssistant ? "output_text" : "input_text",
-                        ["text"] = text
-                    }
-                }
+                ["role"] = role == ChatRole.Assistant.Value ? "assistant" : "user",
+                ["content"] = text
             });
         }
 
@@ -697,7 +603,7 @@ public sealed class LiteLlmResponsesChatClient : IChatClient
                     ["type"] = "function_call",
                     ["call_id"] = call.CallId,
                     ["name"] = call.Name,
-                    ["arguments"] = SerializeFunctionArguments(call)
+                    ["arguments"] = JsonSerializer.Serialize(call.Arguments, JsonOptions)
                 });
             }
             else if (content is FunctionResultContent result)
@@ -734,15 +640,50 @@ public sealed class LiteLlmResponsesChatClient : IChatClient
         return result;
     }
 
-    private static string SerializeFunctionArguments(FunctionCallContent call)
+    private static void AddMessageContent(List<AIContent> contents, JsonElement item)
     {
-        if (call.RawRepresentation is FunctionCallResponseItem responseItem
-            && responseItem.FunctionArguments is not null)
+        if (!item.TryGetProperty("content", out var messageContent) || messageContent.ValueKind != JsonValueKind.Array)
         {
-            return responseItem.FunctionArguments.ToString();
+            return;
         }
 
-        return JsonSerializer.Serialize(call.Arguments, JsonOptions);
+        foreach (var content in messageContent.EnumerateArray())
+        {
+            if (GetString(content, "type") == "output_text")
+            {
+                var text = GetString(content, "text");
+                if (!string.IsNullOrEmpty(text))
+                {
+                    contents.Add(new TextContent(text));
+                }
+            }
+        }
+    }
+
+    private static Dictionary<string, object?> ParseArguments(string? arguments)
+    {
+        if (string.IsNullOrWhiteSpace(arguments))
+        {
+            return [];
+        }
+
+        using var document = JsonDocument.Parse(arguments);
+        return document.RootElement.EnumerateObject()
+            .ToDictionary(property => property.Name, property => ConvertJsonValue(property.Value));
+    }
+
+    private static object? ConvertJsonValue(JsonElement element)
+    {
+        return element.ValueKind switch
+        {
+            JsonValueKind.String => element.GetString(),
+            JsonValueKind.Number when element.TryGetInt64(out var integer) => integer,
+            JsonValueKind.Number when element.TryGetDouble(out var number) => number,
+            JsonValueKind.True => true,
+            JsonValueKind.False => false,
+            JsonValueKind.Null => null,
+            _ => JsonSerializer.Deserialize<object>(element.GetRawText(), JsonOptions)
+        };
     }
 
     private static string ResultToString(object? result)
@@ -755,12 +696,57 @@ public sealed class LiteLlmResponsesChatClient : IChatClient
         };
     }
 
+    private static UsageDetails? ParseUsage(JsonElement root)
+    {
+        if (!root.TryGetProperty("usage", out var usage) || usage.ValueKind != JsonValueKind.Object)
+        {
+            return null;
+        }
+
+        var inputTokens = GetInt(usage, "input_tokens") ?? GetInt(usage, "prompt_tokens");
+        var outputTokens = GetInt(usage, "output_tokens") ?? GetInt(usage, "completion_tokens");
+        var totalTokens = GetInt(usage, "total_tokens");
+        return new UsageDetails
+        {
+            InputTokenCount = inputTokens,
+            OutputTokenCount = outputTokens,
+            TotalTokenCount = totalTokens
+        };
+    }
+
     private static int EstimateTokens(JsonObject payload)
     {
         var json = payload.ToJsonString(JsonOptions);
         return Math.Max(1, (int)Math.Ceiling(json.Length / 4.0));
     }
 
+    private static int? GetInt(JsonElement element, string propertyName)
+    {
+        return element.TryGetProperty(propertyName, out var property) && property.TryGetInt32(out var value)
+            ? value
+            : null;
+    }
+
+    private static string GetRequiredString(JsonElement element, string propertyName)
+    {
+        return GetString(element, propertyName)
+            ?? throw new InvalidOperationException($"LiteLLM response item did not include '{propertyName}'.");
+    }
+
+    private static string? GetString(JsonElement element, string propertyName)
+    {
+        return element.TryGetProperty(propertyName, out var property) && property.ValueKind == JsonValueKind.String
+            ? property.GetString()
+            : null;
+    }
+
+    private static DateTimeOffset? GetUnixTimestamp(JsonElement element, string propertyName)
+    {
+        return element.TryGetProperty(propertyName, out var property) && property.TryGetInt64(out var value)
+            ? DateTimeOffset.FromUnixTimeSeconds(value)
+            : null;
+    }
+
     private static Uri NormalizeEndpoint(Uri endpoint)
     {
         var value = endpoint.ToString().TrimEnd('/');
@@ -776,60 +762,5 @@ public sealed class LiteLlmResponsesChatClient : IChatClient
     {
         return value.TrimStart('/');
     }
-
-    private static ResponsesClient CreateResponsesClient(
-        HttpClient httpClient,
-        string apiKey,
-        AsyncLocal<string?> requestInitiator)
-    {
-        var options = new OpenAIClientOptions
-        {
-            Endpoint = httpClient.BaseAddress
-                ?? throw new InvalidOperationException("LiteLLM HTTP client requires a base address."),
-            Transport = new HttpClientPipelineTransport(httpClient),
-            RetryPolicy = new ClientRetryPolicy(0)
-        };
-        options.AddPolicy(
-            new InitiatorHeaderPolicy(requestInitiator),
-            PipelinePosition.PerCall);
-
-        return new ResponsesClient(
-            new ApiKeyCredential(apiKey),
-            options);
-    }
-
-    private sealed record ResponsesChatResult(
-        ChatResponse Response,
-        IReadOnlyList<string> ReasoningSummaries);
-
-    private sealed class InitiatorHeaderPolicy(AsyncLocal<string?> requestInitiator) : PipelinePolicy
-    {
-        public override void Process(
-            PipelineMessage message,
-            IReadOnlyList<PipelinePolicy> pipeline,
-            int currentIndex)
-        {
-            SetHeader(message);
-            ProcessNext(message, pipeline, currentIndex);
-        }
-
-        public override ValueTask ProcessAsync(
-            PipelineMessage message,
-            IReadOnlyList<PipelinePolicy> pipeline,
-            int currentIndex)
-        {
-            SetHeader(message);
-            return ProcessNextAsync(message, pipeline, currentIndex);
-        }
-
-        private void SetHeader(PipelineMessage message)
-        {
-            if (!string.IsNullOrWhiteSpace(requestInitiator.Value))
-            {
-                message.Request.Headers.Set("X-Initiator", requestInitiator.Value);
-            }
-        }
-    }
 }
-#pragma warning restore OPENAI001
 #pragma warning restore MAAI001

MeetingAssistant/Summary/MeetingSummaryInstructionBuilder.cs

@@ -14,7 +14,7 @@ public sealed class MeetingSummaryInstructionBuilder : IMeetingSummaryInstructio
         write_summary is only for the current meeting's summary file. Past project meeting summaries are read-only historical context; use list_past_project_meetings and read_past_project_meeting_summary to inspect them, and never try to mutate them.
         If the meeting note has no title, or still has a generated default title like `Meeting yyyy-MM-dd HH:mm`, provide a concise title parameter to write_summary when the purpose of the meeting is clear from transcript, user notes, or assistant context. If the purpose is not clear, omit the title parameter.
         Use read_meetingnote to inspect frontmatter such as title, attendees, projects, start_time, and end_time.
-        Treat the assistant context as persistent meeting-specific memory and use read_context and write_context as your own meeting notebook. Its frontmatter may include agenda from the calendar appointment. Whenever you encounter unexpected problems, discover missing information, or make assumptions while summarizing, use write_context to append a concise note so later agents working on this meeting can understand them. Also record useful internal notes, requests for future tools, suggested improvements, and relevant context discovered from other sources. write_context appends by default; use replace_file=true only when you are intentionally replacing the whole assistant context body. Keep user-facing summary content in the summary note.
+        Use read_context and write_context as your own meeting notebook. Its frontmatter may include agenda from the calendar appointment. Record useful internal notes, missing context, requests for future tools, suggested improvements, and relevant context discovered from other sources. write_context appends by default; use replace_file=true only when you are intentionally replacing the whole assistant context body. Keep user-facing summary content in the summary note.
         Use add_dictation_word when project context, user notes, or transcript evidence show that a domain term, acronym, name, or unusual word is likely to be repeatedly mistranscribed. Add only the canonical spelling, one term at a time.
         Use add_attendee and remove_attendee to sharpen the meeting attendees list from clear transcript evidence and screenshot OCR participant evidence. Treat OCR that says visible people are a partial screenshot result as incomplete evidence; do not remove attendees solely because they are absent from a partial screenshot.
         Use override_speaker only when you are very certain that a transcript speaker label belongs to a named person, for example from user notes, OCR evidence with a matching timestamp, or very clear context cues. Provide the exact speaker label from the transcript and the replacement speaker name. If the replacement speaker already exists in the transcript, use merge=true only when you are certain both speaker labels are the same identity; otherwise do not merge them.
@@ -22,7 +22,7 @@ public sealed class MeetingSummaryInstructionBuilder : IMeetingSummaryInstructio
         After writing the summary, update existing project files when the meeting produced durable project knowledge, decisions, next steps, or context.
         Use list_projects first to see which projects are bound to this meeting. Use search, list_past_project_meetings, read_past_project_meeting_summary, and read_projectfile before changing existing project files. search includes both project files and past meeting summaries for the requested current-meeting project scope.
         The summary note should contain concise sections for summary, decisions, open questions, and next steps.
-        If the assistant context contains cropped screenshot markdown links, include only the most relevant cropped screenshots in the summary by markdown-linking them near the related summary text. When embedding them, encode spaces in image-link targets as `%20`; never leave literal spaces in the link target. Do not include every cropped screenshot by default, and do not link uncropped screenshots unless no cropped version exists and the image is important.
+        If the assistant context contains cropped screenshot markdown links, include only the most relevant cropped screenshots in the summary by markdown-linking them near the related summary text. Do not include every cropped screenshot by default, and do not link uncropped screenshots unless no cropped version exists and the image is important.
         Keep the output grounded in the source material and explicitly say when a section has no known items.
         """;
 

MeetingAssistant/Summary/MeetingSummaryTools.cs

@@ -1,7 +1,6 @@
 using MeetingAssistant.MeetingNotes;
 using MeetingAssistant.Speakers;
 using MeetingAssistant.Transcription;
-using MeetingAssistant.Workflow;
 using YamlDotNet.Serialization;
 
 namespace MeetingAssistant.Summary;
@@ -14,7 +13,6 @@ public sealed class MeetingSummaryTools
     private readonly MeetingAssistantOptions options;
     private readonly IDictationWordStore? dictationWordStore;
     private readonly SummaryAgentWriteAudit? writeAudit;
-    private readonly IMeetingWorkflowEngine meetingWorkflowEngine;
     private readonly BoundMeetingProjectResolver projectResolver;
 
     public MeetingSummaryTools(MeetingSessionArtifacts artifacts)
@@ -26,14 +24,12 @@ public sealed class MeetingSummaryTools
         MeetingSessionArtifacts artifacts,
         MeetingAssistantOptions options,
         IDictationWordStore? dictationWordStore = null,
-        SummaryAgentWriteAudit? writeAudit = null,
-        IMeetingWorkflowEngine? meetingWorkflowEngine = null)
+        SummaryAgentWriteAudit? writeAudit = null)
     {
         this.artifacts = artifacts;
         this.options = options;
         this.dictationWordStore = dictationWordStore;
         this.writeAudit = writeAudit;
-        this.meetingWorkflowEngine = meetingWorkflowEngine ?? NoopMeetingWorkflowEngine.Instance;
         projectResolver = new BoundMeetingProjectResolver(options);
     }
 
@@ -102,11 +98,7 @@ public sealed class MeetingSummaryTools
             return "Refused: attendee must not be empty.";
         }
 
-        var displayName = await meetingWorkflowEngine.TransformAttendeeAsync(
-            MeetingWorkflowEvent.AttendeeAdded(artifacts, attendee.Trim()),
-            options,
-            CancellationToken.None);
-        displayName = displayName.Trim();
+        var displayName = attendee.Trim();
         var normalized = MeetingAttendeeNames.NormalizeDisplayName(displayName);
         if (string.IsNullOrWhiteSpace(normalized))
         {

MeetingAssistant/Summary/OpenAiMeetingSummaryAgentPipeline.cs

@@ -1,6 +1,5 @@
 using MeetingAssistant.MeetingNotes;
 using MeetingAssistant.Transcription;
-using MeetingAssistant.Workflow;
 using Microsoft.Agents.AI;
 using Microsoft.Agents.AI.Compaction;
 using Microsoft.Extensions.AI;
@@ -18,7 +17,6 @@ public sealed class OpenAiMeetingSummaryAgentPipeline : IMeetingSummaryPipeline
     private readonly IMeetingSummaryFailureWriter failureWriter;
     private readonly IMeetingSummaryInstructionBuilder instructionBuilder;
     private readonly IDictationWordStore dictationWordStore;
-    private readonly IMeetingWorkflowEngine meetingWorkflowEngine;
 
     public OpenAiMeetingSummaryAgentPipeline(
         IOptions<MeetingAssistantOptions> options,
@@ -27,8 +25,7 @@ public sealed class OpenAiMeetingSummaryAgentPipeline : IMeetingSummaryPipeline
         IServiceProvider services,
         IMeetingSummaryFailureWriter failureWriter,
         IMeetingSummaryInstructionBuilder instructionBuilder,
-        IDictationWordStore dictationWordStore,
-        IMeetingWorkflowEngine meetingWorkflowEngine)
+        IDictationWordStore dictationWordStore)
     {
         this.options = options.Value;
         this.loggerFactory = loggerFactory;
@@ -37,7 +34,6 @@ public sealed class OpenAiMeetingSummaryAgentPipeline : IMeetingSummaryPipeline
         this.failureWriter = failureWriter;
         this.instructionBuilder = instructionBuilder;
         this.dictationWordStore = dictationWordStore;
-        this.meetingWorkflowEngine = meetingWorkflowEngine;
     }
 
     public async Task<MeetingSummaryRunResult> RunAsync(
@@ -55,32 +51,35 @@ public sealed class OpenAiMeetingSummaryAgentPipeline : IMeetingSummaryPipeline
         var agentOptions = options.Agent;
         var key = ResolveApiKey(agentOptions);
         var writeAudit = new SummaryAgentWriteAudit(artifacts);
-        var meetingTools = new MeetingSummaryTools(
-            artifacts,
-            options,
-            dictationWordStore,
-            writeAudit,
-            meetingWorkflowEngine);
+        var meetingTools = new MeetingSummaryTools(artifacts, options, dictationWordStore, writeAudit);
         var tools = CreateTools(meetingTools);
         var instructions = await instructionBuilder.BuildAsync(artifacts, options, cancellationToken);
-        using var compactionSummaryClient = LiteLlmResponsesChatClient.Create(
-            agentOptions,
+        using var compactionSummaryClient = new LiteLlmResponsesChatClient(
+            new Uri(agentOptions.Endpoint),
             key,
+            agentOptions.Model,
+            agentOptions.EnableThinking,
+            ToReasoningEffortValue(agentOptions.ReasoningEffort),
+            agentOptions.ReconnectionAttempts,
+            agentOptions.ReconnectionDelay,
             compactionOptions: null,
             logger,
             firstRequestIsUser: false);
         var compactionOptions = CreateCompactionOptions(agentOptions, compactionSummaryClient);
-        using var chatClient = LiteLlmResponsesChatClient.Create(
-            agentOptions,
+        using var chatClient = new LiteLlmResponsesChatClient(
+            new Uri(agentOptions.Endpoint),
             key,
+            agentOptions.Model,
+            agentOptions.EnableThinking,
+            ToReasoningEffortValue(agentOptions.ReasoningEffort),
+            agentOptions.ReconnectionAttempts,
+            agentOptions.ReconnectionDelay,
             compactionOptions,
             logger,
             firstRequestIsUser: false);
         var agent = chatClient
             .AsBuilder()
-            .UseFunctionInvocation(
-                loggerFactory,
-                client => client.FunctionInvoker = FunctionInvocationGuard.InvokeAsync)
+            .UseFunctionInvocation()
             .Build()
             .AsAIAgent(
                 instructions,
@@ -304,6 +303,18 @@ public sealed class OpenAiMeetingSummaryAgentPipeline : IMeetingSummaryPipeline
         return new PipelineCompactionStrategy(strategies);
     }
 
+    private static string ToReasoningEffortValue(ReasoningEffortOption effort)
+    {
+        return effort switch
+        {
+            ReasoningEffortOption.None => "none",
+            ReasoningEffortOption.Low => "low",
+            ReasoningEffortOption.High => "high",
+            ReasoningEffortOption.ExtraHigh => "xhigh",
+            _ => "medium"
+        };
+    }
+
     private static ReasoningEffort ToReasoningEffort(ReasoningEffortOption effort)
     {
         return effort switch

MeetingAssistant/Taskbar/MeetingTaskbarMenu.cs

@@ -38,7 +38,7 @@ public static class MeetingTaskbarMenuBuilder
     {
         var items = new List<MeetingTaskbarMenuItem>
         {
-            new("Open agent", MeetingTaskbarAction.EditRules)
+            new("Settings and logs", MeetingTaskbarAction.EditRules)
         };
 
         if (microphones is { Count: > 0 })

MeetingAssistant/Workflow/MeetingWorkflowEngine.cs

@@ -17,14 +17,6 @@ public interface IMeetingWorkflowEngine
         MeetingWorkflowEvent workflowEvent,
         MeetingAssistantOptions options,
         CancellationToken cancellationToken);
-
-    Task<string> TransformAttendeeAsync(
-        MeetingWorkflowEvent workflowEvent,
-        MeetingAssistantOptions options,
-        CancellationToken cancellationToken)
-    {
-        return Task.FromResult(workflowEvent.AttendeeName ?? "");
-    }
 }
 
 public sealed class NoopMeetingWorkflowEngine : IMeetingWorkflowEngine
@@ -46,14 +38,6 @@ public sealed class NoopMeetingWorkflowEngine : IMeetingWorkflowEngine
     {
         return Task.FromResult(workflowEvent.TranscriptLineText ?? "");
     }
-
-    public Task<string> TransformAttendeeAsync(
-        MeetingWorkflowEvent workflowEvent,
-        MeetingAssistantOptions options,
-        CancellationToken cancellationToken)
-    {
-        return Task.FromResult(workflowEvent.AttendeeName ?? "");
-    }
 }
 
 public sealed class MeetingWorkflowEngine : IMeetingWorkflowEngine
@@ -70,8 +54,7 @@ public sealed class MeetingWorkflowEngine : IMeetingWorkflowEngine
         "state.to",
         "speaker.name",
         MeetingWorkflowRuleSchema.PropertyTranscriptLine,
-        MeetingWorkflowRuleSchema.ParameterTranscriptSpeaker,
-        MeetingWorkflowRuleSchema.PropertyAttendeeName
+        MeetingWorkflowRuleSchema.ParameterTranscriptSpeaker
     ];
 
     private readonly IMeetingWorkflowRulesProvider rulesProvider;
@@ -115,21 +98,6 @@ public sealed class MeetingWorkflowEngine : IMeetingWorkflowEngine
             ?? "";
     }
 
-    public async Task<string> TransformAttendeeAsync(
-        MeetingWorkflowEvent workflowEvent,
-        MeetingAssistantOptions options,
-        CancellationToken cancellationToken)
-    {
-        if (workflowEvent.Type != MeetingWorkflowEventType.AttendeeAdded)
-        {
-            throw new ArgumentException("Workflow event must be an attendee added event.", nameof(workflowEvent));
-        }
-
-        return await RunCoreAsync(workflowEvent, options, cancellationToken)
-            ?? workflowEvent.AttendeeName
-            ?? "";
-    }
-
     private async Task<string?> RunCoreAsync(
         MeetingWorkflowEvent workflowEvent,
         MeetingAssistantOptions options,
@@ -139,7 +107,7 @@ public sealed class MeetingWorkflowEngine : IMeetingWorkflowEngine
         var context = new MeetingWorkflowExecutionContext(workflowEvent);
         if (rules.Count == 0)
         {
-            return context.ResultValue;
+            return context.TranscriptLine;
         }
 
         var meeting = await meetingNoteStore.ReadAsync(
@@ -166,18 +134,10 @@ public sealed class MeetingWorkflowEngine : IMeetingWorkflowEngine
                 var model = CreateTemplateModel(meeting, context);
                 foreach (var step in rule.Steps)
                 {
-                    if (workflowEvent.Type == MeetingWorkflowEventType.AttendeeAdded &&
-                        !IsAttendeeTransformStep(step))
-                    {
-                        throw new InvalidOperationException(
-                            "attendee_added workflow rules can only transform attendee.name with set_property.");
-                    }
-
                     var stepChanged = await ApplyStepAsync(
                         step,
                         meeting,
                         context,
-                        options,
                         model,
                         cancellationToken);
                     ruleNoteChanged |= stepChanged;
@@ -216,7 +176,7 @@ public sealed class MeetingWorkflowEngine : IMeetingWorkflowEngine
                 cancellationToken);
         }
 
-        return context.ResultValue;
+        return context.TranscriptLine;
     }
 
     private static bool MatchesTrigger(
@@ -269,41 +229,9 @@ public sealed class MeetingWorkflowEngine : IMeetingWorkflowEngine
                         StringComparison.OrdinalIgnoreCase));
         }
 
-        if (trigger.AttendeeAdded is not null)
-        {
-            return workflowEvent.Type == MeetingWorkflowEventType.AttendeeAdded &&
-                MatchesAttendeeAddedTrigger(trigger.AttendeeAdded, workflowEvent.AttendeeName);
-        }
-
         return false;
     }
 
-    private static bool MatchesAttendeeAddedTrigger(
-        MeetingWorkflowAttendeeAddedTrigger trigger,
-        string? attendeeName)
-    {
-        var attendee = attendeeName ?? "";
-        if (!string.IsNullOrWhiteSpace(trigger.EqualsValue) &&
-            !string.Equals(attendee, trigger.EqualsValue.Trim(), StringComparison.OrdinalIgnoreCase))
-        {
-            return false;
-        }
-
-        if (!string.IsNullOrWhiteSpace(trigger.Contains) &&
-            !attendee.Contains(trigger.Contains.Trim(), StringComparison.OrdinalIgnoreCase))
-        {
-            return false;
-        }
-
-        if (!string.IsNullOrWhiteSpace(trigger.Regex) &&
-            !Regex.IsMatch(attendee, trigger.Regex, RegexOptions.IgnoreCase))
-        {
-            return false;
-        }
-
-        return true;
-    }
-
     private bool EvaluateConditions(
         IReadOnlyList<MeetingWorkflowCondition> conditions,
         MeetingNote meeting,
@@ -370,7 +298,6 @@ public sealed class MeetingWorkflowEngine : IMeetingWorkflowEngine
         MeetingWorkflowStep step,
         MeetingNote meeting,
         MeetingWorkflowExecutionContext context,
-        MeetingAssistantOptions options,
         MeetingWorkflowTemplateModel model,
         CancellationToken cancellationToken)
     {
@@ -378,11 +305,7 @@ public sealed class MeetingWorkflowEngine : IMeetingWorkflowEngine
         switch (step.Uses.Trim().ToLowerInvariant())
         {
             case MeetingWorkflowRuleSchema.StepAddAttendee:
-                var attendee = await TransformAttendeeAsync(
-                    MeetingWorkflowEvent.AttendeeAdded(context.Event.Artifacts, value),
-                    options,
-                    cancellationToken);
-                return AddUnique(meeting.Frontmatter.Attendees, attendee, MeetingAttendeeNames.NormalizeDisplayName);
+                return AddUnique(meeting.Frontmatter.Attendees, value, MeetingAttendeeNames.NormalizeDisplayName);
             case MeetingWorkflowRuleSchema.StepRemoveAttendee:
                 return RemoveValue(meeting.Frontmatter.Attendees, value);
             case MeetingWorkflowRuleSchema.StepAddProject:
@@ -396,9 +319,6 @@ public sealed class MeetingWorkflowEngine : IMeetingWorkflowEngine
             case MeetingWorkflowRuleSchema.StepSetProperty when IsTranscriptLineProperty(step.Property ?? step.Name):
                 SetTranscriptLine(context, value);
                 return false;
-            case MeetingWorkflowRuleSchema.StepSetProperty when IsAttendeeNameProperty(step.Property ?? step.Name):
-                SetAttendeeName(context, value);
-                return false;
             case MeetingWorkflowRuleSchema.StepSetProperty:
                 return SetProperty(meeting, step.Property ?? step.Name, value);
             default:
@@ -450,39 +370,11 @@ public sealed class MeetingWorkflowEngine : IMeetingWorkflowEngine
         context.TranscriptLine = value;
     }
 
-    private static void SetAttendeeName(
-        MeetingWorkflowExecutionContext context,
-        string value)
-    {
-        if (context.Event.Type != MeetingWorkflowEventType.AttendeeAdded)
-        {
-            throw new InvalidOperationException("The attendee.name property can only be set during attendee_added events.");
-        }
-
-        if (string.Equals(context.AttendeeName, value, StringComparison.Ordinal))
-        {
-            return;
-        }
-
-        context.AttendeeName = value;
-    }
-
     private static bool IsTranscriptLineProperty(string? property)
     {
         return MeetingWorkflowRuleSchema.IsProperty(property, MeetingWorkflowRuleSchema.PropertyTranscriptLine);
     }
 
-    private static bool IsAttendeeNameProperty(string? property)
-    {
-        return MeetingWorkflowRuleSchema.IsProperty(property, MeetingWorkflowRuleSchema.PropertyAttendeeName);
-    }
-
-    private static bool IsAttendeeTransformStep(MeetingWorkflowStep step)
-    {
-        return MeetingWorkflowRuleSchema.IsStep(step.Uses, MeetingWorkflowRuleSchema.StepSetProperty) &&
-            IsAttendeeNameProperty(step.Property ?? step.Name);
-    }
-
     private static bool AddUnique(
         List<string> values,
         string value,
@@ -564,8 +456,7 @@ public sealed class MeetingWorkflowEngine : IMeetingWorkflowEngine
             ["state.to"] = workflowEvent.ToState is { } to ? MeetingWorkflowStateNames.ToRuleName(to) : "",
             ["speaker.name"] = workflowEvent.SpeakerName,
             [MeetingWorkflowRuleSchema.PropertyTranscriptLine] = context.TranscriptLine ?? "",
-            [MeetingWorkflowRuleSchema.ParameterTranscriptSpeaker] = workflowEvent.SpeakerName ?? "",
-            [MeetingWorkflowRuleSchema.PropertyAttendeeName] = context.AttendeeName ?? ""
+            [MeetingWorkflowRuleSchema.ParameterTranscriptSpeaker] = workflowEvent.SpeakerName ?? ""
         };
     }
 
@@ -589,10 +480,7 @@ public sealed class MeetingWorkflowEngine : IMeetingWorkflowEngine
                 : new MeetingWorkflowSpeakerModel(workflowEvent.SpeakerName),
             new MeetingWorkflowTranscriptModel(
                 context.TranscriptLine ?? "",
-                workflowEvent.SpeakerName ?? ""),
-            string.IsNullOrWhiteSpace(context.AttendeeName)
-                ? null
-                : new MeetingWorkflowAttendeeModel(context.AttendeeName));
+                workflowEvent.SpeakerName ?? ""));
     }
 
     private sealed class MeetingWorkflowExecutionContext
@@ -601,23 +489,10 @@ public sealed class MeetingWorkflowEngine : IMeetingWorkflowEngine
         {
             Event = workflowEvent;
             TranscriptLine = workflowEvent.TranscriptLineText;
-            AttendeeName = workflowEvent.AttendeeName;
         }
 
         public MeetingWorkflowEvent Event { get; }
 
         public string? TranscriptLine { get; set; }
-
-        public string? AttendeeName { get; set; }
-
-        public string? ResultValue
-        {
-            get
-            {
-                return Event.Type == MeetingWorkflowEventType.AttendeeAdded
-                    ? AttendeeName
-                    : TranscriptLine;
-            }
-        }
     }
 }

MeetingAssistant/Workflow/MeetingWorkflowEvent.cs

@@ -7,8 +7,7 @@ public enum MeetingWorkflowEventType
     Created,
     StateTransition,
     SpeakerIdentified,
-    TranscriptLine,
-    AttendeeAdded
+    TranscriptLine
 }
 
 public sealed record MeetingWorkflowEvent(
@@ -17,8 +16,7 @@ public sealed record MeetingWorkflowEvent(
     AssistantContextState? FromState = null,
     AssistantContextState? ToState = null,
     string? SpeakerName = null,
-    string? TranscriptLineText = null,
-    string? AttendeeName = null)
+    string? TranscriptLineText = null)
 {
     public static MeetingWorkflowEvent Created(MeetingSessionArtifacts artifacts)
     {
@@ -51,14 +49,4 @@ public sealed record MeetingWorkflowEvent(
             SpeakerName: speakerName,
             TranscriptLineText: line);
     }
-
-    public static MeetingWorkflowEvent AttendeeAdded(
-        MeetingSessionArtifacts artifacts,
-        string attendeeName)
-    {
-        return new MeetingWorkflowEvent(
-            MeetingWorkflowEventType.AttendeeAdded,
-            artifacts,
-            AttendeeName: attendeeName);
-    }
 }

MeetingAssistant/Workflow/MeetingWorkflowModels.cs

@@ -37,9 +37,6 @@ public sealed class MeetingWorkflowTrigger
 
     [YamlMember(Alias = "transcript_line")]
     public MeetingWorkflowTranscriptLineTrigger? TranscriptLine { get; set; }
-
-    [YamlMember(Alias = "attendee_added")]
-    public MeetingWorkflowAttendeeAddedTrigger? AttendeeAdded { get; set; }
 }
 
 public sealed class MeetingWorkflowStateTransitionTrigger
@@ -63,18 +60,6 @@ public sealed class MeetingWorkflowTranscriptLineTrigger
     public string? Speaker { get; set; }
 }
 
-public sealed class MeetingWorkflowAttendeeAddedTrigger
-{
-    [YamlMember(Alias = "equals")]
-    public string? EqualsValue { get; set; }
-
-    [YamlMember(Alias = "contains")]
-    public string? Contains { get; set; }
-
-    [YamlMember(Alias = "regex")]
-    public string? Regex { get; set; }
-}
-
 public sealed class MeetingWorkflowCondition
 {
     [YamlMember(Alias = "condition")]
@@ -109,8 +94,7 @@ public sealed record MeetingWorkflowTemplateModel(
     MeetingWorkflowMeetingModel Meeting,
     MeetingWorkflowEventModel Event,
     MeetingWorkflowSpeakerModel? Speaker,
-    MeetingWorkflowTranscriptModel? Transcript,
-    MeetingWorkflowAttendeeModel? Attendee);
+    MeetingWorkflowTranscriptModel? Transcript);
 
 public sealed record MeetingWorkflowMeetingModel(
     string Title,
@@ -127,8 +111,6 @@ public sealed record MeetingWorkflowSpeakerModel(string Name);
 
 public sealed record MeetingWorkflowTranscriptModel(string Line, string Speaker);
 
-public sealed record MeetingWorkflowAttendeeModel(string Name);
-
 internal static class MeetingWorkflowStateNames
 {
     public static string ToRuleName(AssistantContextState state)

MeetingAssistant/Workflow/MeetingWorkflowRuleSchema.cs

@@ -12,15 +12,13 @@ internal static class MeetingWorkflowRuleSchema
     public const string PropertyMeetingTitle = "meeting.title";
     public const string PropertyTranscriptLine = "transcript.line";
     public const string ParameterTranscriptSpeaker = "transcript.speaker";
-    public const string PropertyAttendeeName = "attendee.name";
 
     public static readonly string[] SupportedTriggerKeys =
     [
         "created",
         "state_transition",
         "speaker_identified",
-        "transcript_line",
-        "attendee_added"
+        "transcript_line"
     ];
 
     public static readonly string[] SupportedConditionKeys =
@@ -44,8 +42,7 @@ internal static class MeetingWorkflowRuleSchema
     [
         PropertyTitle,
         PropertyMeetingTitle,
-        PropertyTranscriptLine,
-        PropertyAttendeeName
+        PropertyTranscriptLine
     ];
 
     public static bool IsStep(string? value, string step)

MeetingAssistant/Workflow/MeetingWorkflowRulesValidator.cs

@@ -59,8 +59,7 @@ internal static class MeetingWorkflowRulesValidator
             trigger.Created is not null,
             trigger.StateTransition is not null,
             trigger.SpeakerIdentified is not null,
-            trigger.TranscriptLine is not null,
-            trigger.AttendeeAdded is not null);
+            trigger.TranscriptLine is not null);
         if (configuredCount == 0)
         {
             errors.Add($"{ruleName} on[{triggerIndex + 1}] must contain one supported trigger: {JoinAllowed(MeetingWorkflowRuleSchema.SupportedTriggerKeys)}.");
@@ -138,12 +137,6 @@ internal static class MeetingWorkflowRulesValidator
             return;
         }
 
-        if (HasAttendeeAddedTrigger(rule) &&
-            !IsAttendeeNameSetPropertyStep(step))
-        {
-            errors.Add($"{stepPath} on attendee_added rules can only transform the attendee with 'set_property' property 'attendee.name'.");
-        }
-
         if (MeetingWorkflowRuleSchema.IsStep(uses, MeetingWorkflowRuleSchema.StepSetProperty))
         {
             var property = step.Property ?? step.Name;
@@ -160,11 +153,6 @@ internal static class MeetingWorkflowRulesValidator
             {
                 errors.Add($"{stepPath} can set 'transcript.line' only on rules with a transcript_line trigger.");
             }
-            else if (MeetingWorkflowRuleSchema.IsProperty(property, MeetingWorkflowRuleSchema.PropertyAttendeeName) &&
-                !HasAttendeeAddedTrigger(rule))
-            {
-                errors.Add($"{stepPath} can set 'attendee.name' only on rules with an attendee_added trigger.");
-            }
         }
 
         if (step.Value is { } value &&
@@ -193,17 +181,6 @@ internal static class MeetingWorkflowRulesValidator
         return rule.On.Any(static trigger => trigger.TranscriptLine is not null);
     }
 
-    private static bool HasAttendeeAddedTrigger(MeetingWorkflowRule rule)
-    {
-        return rule.On.Any(static trigger => trigger.AttendeeAdded is not null);
-    }
-
-    private static bool IsAttendeeNameSetPropertyStep(MeetingWorkflowStep step)
-    {
-        return MeetingWorkflowRuleSchema.IsStep(step.Uses, MeetingWorkflowRuleSchema.StepSetProperty) &&
-            MeetingWorkflowRuleSchema.IsProperty(step.Property ?? step.Name, MeetingWorkflowRuleSchema.PropertyAttendeeName);
-    }
-
     private static int CountConfigured(params bool[] values)
     {
         return values.Count(static value => value);
@@ -229,8 +206,7 @@ internal static class MeetingWorkflowRulesValidator
             new MeetingWorkflowSpeakerModel("Ada Lovelace"),
             new MeetingWorkflowTranscriptModel(
                 "[00:00:01] Ada Lovelace: Validation line.",
-                "Ada Lovelace"),
-            new MeetingWorkflowAttendeeModel("Ada Lovelace"));
+                "Ada Lovelace"));
     }
 
     private static string FormatRazorValidationMessage(Exception exception)

MeetingAssistant/Workflow/WorkflowRulesEditorChatModels.cs

@@ -10,66 +10,9 @@ public sealed record WorkflowRulesEditorChatMessage(
     WorkflowRulesEditorChatRole Role,
     string Content);
 
-public enum WorkflowRulesEditorConversationItemKind
-{
-    Message,
-    Activity
-}
-
-public sealed record WorkflowRulesEditorConversationItem(
-    WorkflowRulesEditorConversationItemKind Kind,
-    WorkflowRulesEditorChatMessage? Message,
-    string Content,
-    IReadOnlyList<string>? ActivityLines = null)
-{
-    public bool IsExpanded { get; set; }
-
-    public static WorkflowRulesEditorConversationItem ChatMessage(WorkflowRulesEditorChatMessage message)
-    {
-        return new WorkflowRulesEditorConversationItem(
-            WorkflowRulesEditorConversationItemKind.Message,
-            message,
-            message.Content);
-    }
-
-    public static WorkflowRulesEditorConversationItem Activity(string content, IReadOnlyList<string> activityLines)
-    {
-        return new WorkflowRulesEditorConversationItem(
-            WorkflowRulesEditorConversationItemKind.Activity,
-            null,
-            content,
-            activityLines);
-    }
-}
-
-public sealed record WorkflowRulesEditorChatResult(string Response);
-
-public enum WorkflowRulesEditorActivityKind
-{
-    Status,
-    ToolCall,
-    Thinking
-}
-
-public sealed record WorkflowRulesEditorActivityUpdate(
-    WorkflowRulesEditorActivityKind Kind,
-    string Text)
-{
-    public static WorkflowRulesEditorActivityUpdate Status(string text)
-    {
-        return new WorkflowRulesEditorActivityUpdate(WorkflowRulesEditorActivityKind.Status, text);
-    }
-
-    public static WorkflowRulesEditorActivityUpdate ToolCall(string toolName)
-    {
-        return new WorkflowRulesEditorActivityUpdate(WorkflowRulesEditorActivityKind.ToolCall, toolName);
-    }
-
-    public static WorkflowRulesEditorActivityUpdate Thinking(string text)
-    {
-        return new WorkflowRulesEditorActivityUpdate(WorkflowRulesEditorActivityKind.Thinking, text);
-    }
-}
+public sealed record WorkflowRulesEditorChatResult(
+    string Response,
+    IReadOnlyList<WorkflowRulesEditorChatMessage> Conversation);
 
 public interface IWorkflowRulesEditorChatPipeline
 {
@@ -77,5 +20,5 @@ public interface IWorkflowRulesEditorChatPipeline
         IReadOnlyList<WorkflowRulesEditorChatMessage> conversation,
         string userMessage,
         CancellationToken cancellationToken,
-        Action<WorkflowRulesEditorActivityUpdate>? activityChanged = null);
+        Action<string>? statusChanged = null);
 }

MeetingAssistant/Workflow/WorkflowRulesEditorChatPipeline.cs

@@ -59,11 +59,11 @@ public sealed class WorkflowRulesEditorChatPipeline : IWorkflowRulesEditorChatPi
         IReadOnlyList<WorkflowRulesEditorChatMessage> conversation,
         string userMessage,
         CancellationToken cancellationToken,
-        Action<WorkflowRulesEditorActivityUpdate>? activityChanged = null)
+        Action<string>? statusChanged = null)
     {
         if (string.IsNullOrWhiteSpace(userMessage))
         {
-            return new WorkflowRulesEditorChatResult("");
+            return new WorkflowRulesEditorChatResult("", conversation);
         }
 
         var agentOptions = options.WorkflowRulesEditor.ToEffectiveAgentOptions(options.Agent);
@@ -85,35 +85,33 @@ public sealed class WorkflowRulesEditorChatPipeline : IWorkflowRulesEditorChatPi
             .ToList();
         var instructions = await instructionBuilder.BuildAsync(options, cancellationToken);
 
-        using var compactionSummaryClient = LiteLlmResponsesChatClient.Create(
-            agentOptions,
+        using var compactionSummaryClient = new LiteLlmResponsesChatClient(
+            new Uri(agentOptions.Endpoint),
             key,
+            agentOptions.Model,
+            agentOptions.EnableThinking,
+            ToReasoningEffortValue(agentOptions.ReasoningEffort),
+            agentOptions.ReconnectionAttempts,
+            agentOptions.ReconnectionDelay,
             compactionOptions: null,
             logger,
             firstRequestIsUser: false);
         var compactionOptions = CreateCompactionOptions(agentOptions, compactionSummaryClient);
-        using var chatClient = LiteLlmResponsesChatClient.Create(
-            agentOptions,
+        using var chatClient = new LiteLlmResponsesChatClient(
+            new Uri(agentOptions.Endpoint),
             key,
+            agentOptions.Model,
+            agentOptions.EnableThinking,
+            ToReasoningEffortValue(agentOptions.ReasoningEffort),
+            agentOptions.ReconnectionAttempts,
+            agentOptions.ReconnectionDelay,
             compactionOptions,
             logger,
             firstRequestIsUser: true,
-            retrying: () => activityChanged?.Invoke(WorkflowRulesEditorActivityUpdate.Status("Reconnecting...")),
-            reasoningSummaryChanged: text => activityChanged?.Invoke(WorkflowRulesEditorActivityUpdate.Thinking(text)));
+            retrying: () => statusChanged?.Invoke("Reconnecting..."));
         var functionClient = chatClient
             .AsBuilder()
-            .UseFunctionInvocation(loggerFactory, client =>
-            {
-                client.FunctionInvoker = async (context, token) =>
-                {
-                    if (context.CallContent.Exception is null)
-                    {
-                        activityChanged?.Invoke(WorkflowRulesEditorActivityUpdate.ToolCall(context.Function.Name));
-                    }
-
-                    return await FunctionInvocationGuard.InvokeAsync(context, token);
-                };
-            })
+            .UseFunctionInvocation(loggerFactory)
             .Build();
 
         var response = await functionClient.GetResponseAsync(
@@ -123,7 +121,11 @@ public sealed class WorkflowRulesEditorChatPipeline : IWorkflowRulesEditorChatPi
         var responseText = string.IsNullOrWhiteSpace(response.Text)
             ? "(No response text returned.)"
             : response.Text.Trim();
-        return new WorkflowRulesEditorChatResult(responseText);
+        var nextConversation = conversation
+            .Append(new WorkflowRulesEditorChatMessage(WorkflowRulesEditorChatRole.User, userMessage.Trim()))
+            .Append(new WorkflowRulesEditorChatMessage(WorkflowRulesEditorChatRole.Agent, responseText))
+            .ToList();
+        return new WorkflowRulesEditorChatResult(responseText, nextConversation);
     }
 
     private static ChatMessage ToChatMessage(WorkflowRulesEditorChatMessage message)
@@ -390,6 +392,18 @@ public sealed class WorkflowRulesEditorChatPipeline : IWorkflowRulesEditorChatPi
             $"No {agentName} API key configured. Set MeetingAssistant:WorkflowRulesEditor:Key, MeetingAssistant:Agent:Key, or environment variable '{options.KeyEnv}'.");
     }
 
+    private static string ToReasoningEffortValue(ReasoningEffortOption effort)
+    {
+        return effort switch
+        {
+            ReasoningEffortOption.None => "none",
+            ReasoningEffortOption.Low => "low",
+            ReasoningEffortOption.High => "high",
+            ReasoningEffortOption.ExtraHigh => "xhigh",
+            _ => "medium"
+        };
+    }
+
     private static ReasoningEffort ToReasoningEffort(ReasoningEffortOption effort)
     {
         return effort switch

MeetingAssistant/Workflow/WorkflowRulesEditorChatViewModel.cs

@@ -1,11 +1,9 @@
 using System.Collections.ObjectModel;
-using System.Diagnostics;
 
 namespace MeetingAssistant.Workflow;
 
 public sealed class WorkflowRulesEditorChatViewModel
 {
-    private const string ThinkingPlaceholder = "Thinking...";
     private readonly IWorkflowRulesEditorChatPipeline pipeline;
 
     public WorkflowRulesEditorChatViewModel(IWorkflowRulesEditorChatPipeline pipeline)
@@ -13,15 +11,13 @@ public sealed class WorkflowRulesEditorChatViewModel
         this.pipeline = pipeline;
     }
 
-    public ObservableCollection<WorkflowRulesEditorConversationItem> Messages { get; } = [];
-
-    public ObservableCollection<string> ActivityMessages { get; } = [];
+    public ObservableCollection<WorkflowRulesEditorChatMessage> Messages { get; } = [];
 
     public string Draft { get; set; } = "";
 
     public bool IsThinking { get; private set; }
 
-    public string ActivityMessage { get; private set; } = ThinkingPlaceholder;
+    public string ActivityMessage { get; private set; } = "Thinking...";
 
     public event EventHandler? Changed;
 
@@ -34,19 +30,10 @@ public sealed class WorkflowRulesEditorChatViewModel
         }
 
         Draft = "";
-        var startedAt = Stopwatch.GetTimestamp();
-        var activityLines = new List<string>();
-        var hasVisibleThinking = false;
-        var activityContext = SynchronizationContext.Current;
-        var priorConversation = Messages
-            .Select(item => item.Message)
-            .OfType<WorkflowRulesEditorChatMessage>()
-            .ToList();
-        Messages.Add(WorkflowRulesEditorConversationItem.ChatMessage(
-            new WorkflowRulesEditorChatMessage(WorkflowRulesEditorChatRole.User, prompt)));
-        ActivityMessages.Clear();
+        var priorConversation = Messages.ToList();
+        Messages.Add(new WorkflowRulesEditorChatMessage(WorkflowRulesEditorChatRole.User, prompt));
         IsThinking = true;
-        ActivityMessage = ThinkingPlaceholder;
+        ActivityMessage = "Thinking...";
         OnChanged();
 
         try
@@ -55,121 +42,36 @@ public sealed class WorkflowRulesEditorChatViewModel
                 priorConversation,
                 prompt,
                 cancellationToken,
-                update => hasVisibleThinking |= ApplyActivityUpdate(activityContext, update, activityLines));
-            AddCompletedActivity(startedAt, activityLines, hasVisibleThinking);
-            Messages.Add(WorkflowRulesEditorConversationItem.ChatMessage(
-                new WorkflowRulesEditorChatMessage(
-                    WorkflowRulesEditorChatRole.Agent,
-                    string.IsNullOrWhiteSpace(result.Response)
-                        ? "(No response text returned.)"
-                        : result.Response.Trim())));
+                SetActivityMessage);
+            Messages.Clear();
+            foreach (var message in result.Conversation)
+            {
+                Messages.Add(message);
+            }
         }
         catch (Exception exception) when (exception is not OperationCanceledException || !cancellationToken.IsCancellationRequested)
         {
-            AddCompletedActivity(startedAt, activityLines, hasVisibleThinking);
-            Messages.Add(WorkflowRulesEditorConversationItem.ChatMessage(
-                new WorkflowRulesEditorChatMessage(
-                    WorkflowRulesEditorChatRole.Agent,
-                    $"Meeting Summary Agent failed: {exception.Message}")));
+            Messages.Add(new WorkflowRulesEditorChatMessage(
+                WorkflowRulesEditorChatRole.Agent,
+                $"Settings and logs failed: {exception.Message}"));
         }
         finally
         {
             IsThinking = false;
-            ActivityMessages.Clear();
-            ActivityMessage = ThinkingPlaceholder;
+            ActivityMessage = "Thinking...";
             OnChanged();
         }
     }
 
-    private bool ApplyActivityUpdate(
-        SynchronizationContext? context,
-        WorkflowRulesEditorActivityUpdate update,
-        List<string> activityLines)
+    private void SetActivityMessage(string message)
     {
-        if (context is null || SynchronizationContext.Current == context)
+        if (!IsThinking || string.IsNullOrWhiteSpace(message))
         {
-            return ApplyActivityUpdate(update, activityLines);
-        }
-
-        var hasVisibleThinking = false;
-        Exception? exception = null;
-        context.Send(
-            _ =>
-            {
-                try
-                {
-                    hasVisibleThinking = ApplyActivityUpdate(update, activityLines);
-                }
-                catch (Exception caught)
-                {
-                    exception = caught;
-                }
-            },
-            null);
-
-        if (exception is not null)
-        {
-            throw exception;
-        }
-
-        return hasVisibleThinking;
-    }
-
-    private bool ApplyActivityUpdate(
-        WorkflowRulesEditorActivityUpdate update,
-        List<string> activityLines)
-    {
-        if (!IsThinking || string.IsNullOrWhiteSpace(update.Text))
-        {
-            return false;
-        }
-
-        var hasVisibleThinking = false;
-        if (update.Kind == WorkflowRulesEditorActivityKind.ToolCall)
-        {
-            var line = $"Called tool: {update.Text.Trim()}";
-            ActivityMessages.Add(line);
-            activityLines.Add(line);
-        }
-        else if (update.Kind == WorkflowRulesEditorActivityKind.Thinking)
-        {
-            var line = update.Text.Trim();
-            ActivityMessages.Add(line);
-            activityLines.Add(line);
-            hasVisibleThinking = true;
-        }
-        else
-        {
-            var line = update.Text.Trim();
-            ActivityMessage = line;
-            activityLines.Add(line);
+            return;
         }
 
+        ActivityMessage = message;
         OnChanged();
-        return hasVisibleThinking;
-    }
-
-    private void AddCompletedActivity(
-        long startedAt,
-        IReadOnlyList<string> activityLines,
-        bool hasVisibleThinking)
-    {
-        var completedActivityLines = hasVisibleThinking
-            ? activityLines.ToArray()
-            : new[] { ThinkingPlaceholder }.Concat(activityLines).ToArray();
-        Messages.Add(WorkflowRulesEditorConversationItem.Activity(
-            $"Worked for {FormatDuration(Stopwatch.GetElapsedTime(startedAt))}",
-            completedActivityLines));
-    }
-
-    private static string FormatDuration(TimeSpan duration)
-    {
-        if (duration.TotalMinutes >= 1)
-        {
-            return $"{(int)duration.TotalMinutes}m {duration.Seconds}s";
-        }
-
-        return $"{Math.Max(0, (int)Math.Round(duration.TotalSeconds))}s";
     }
 
     private void OnChanged()

MeetingAssistant/Workflow/WorkflowRulesEditorInstructionBuilder.cs

@@ -60,7 +60,7 @@ public sealed class WorkflowRulesEditorInstructionBuilder : IWorkflowRulesEditor
             "Log tools can read and search the current application-owned log file and four rotated older files under the temp log folder." + Environment.NewLine + Environment.NewLine +
             "Spec tools can search and read copied OpenSpec markdown files from openspec/specs." + Environment.NewLine + Environment.NewLine +
             "Project tools can create project folders and read, write, list, and search files in configured projects." + Environment.NewLine + Environment.NewLine +
-            "Meeting artifact tools can list recent summaries and read/search/write summaries, transcripts, meeting notes, and assistant context files for note post-processing and repair. Prefer frontmatter-specific write tools for metadata-only fixes. Treat each assistant context file as meeting-specific memory. When asked to fix or investigate a meeting or summary, read the matching assistant context for clues about problems, missing information, assumptions, prior fixes, and conclusions. After completing repairs, use write_context to append a concise record of your fixes and conclusions to that assistant context for future work on the meeting." + Environment.NewLine + Environment.NewLine +
+            "Meeting artifact tools can list recent summaries and read/search/write summaries, transcripts, meeting notes, and assistant context files for note post-processing and repair. Prefer frontmatter-specific write tools for metadata-only fixes." + Environment.NewLine + Environment.NewLine +
             "Diagnostic tools mirror the local health, recording status, Outlook metadata, speaker identity merge, workflow reload, and ASR diagnostic endpoints without requiring HTTP." + Environment.NewLine + Environment.NewLine +
             "Speaker identity tools can search/list/read/update/delete/merge identities, refuse sampleless identity creation, list/read/delete identity samples, and queue a sample for local playback. Do not delete the last sample from an identity; delete the identity instead." + Environment.NewLine + Environment.NewLine +
             "Workflow rules reference documentation:" + Environment.NewLine +

MeetingAssistant/Workflow/WpfWorkflowRulesEditorWindowService.Windows.cs

@@ -11,7 +11,7 @@ namespace MeetingAssistant.Workflow;
 
 internal sealed class WpfWorkflowRulesEditorWindowService : IWorkflowRulesEditorWindowService
 {
-    internal const string WindowTitle = "Meeting Summary Agent";
+    internal const string WindowTitle = "Settings and logs";
 
     private readonly IServiceProvider services;
     private readonly WorkflowRulesEditorMarkdownLinkResolver linkResolver;
@@ -288,21 +288,19 @@ internal sealed class WorkflowRulesEditorWpfWindow : Window
             conversationPanel.ActualHeight);
 
         conversationPanel.Children.Clear();
-        foreach (var item in viewModel.Messages)
+        foreach (var message in viewModel.Messages)
         {
-            conversationPanel.Children.Add(item.Kind == WorkflowRulesEditorConversationItemKind.Activity
-                ? CreateActivityExpander(item)
-                : CreateMessageCard(item.Message!));
+            conversationPanel.Children.Add(CreateMessageCard(message));
         }
 
         if (viewModel.IsThinking)
         {
-            foreach (var activity in viewModel.ActivityMessages)
+            conversationPanel.Children.Add(new TextBlock
             {
-                conversationPanel.Children.Add(CreateActivityLine(activity));
-            }
-
-            conversationPanel.Children.Add(CreateActivityLine(viewModel.ActivityMessage));
+                Text = viewModel.ActivityMessage,
+                Foreground = MutedText,
+                Margin = new Thickness(4, 2, 4, 2)
+            });
         }
 
         sendButton.IsEnabled = !viewModel.IsThinking;
@@ -340,43 +338,6 @@ internal sealed class WorkflowRulesEditorWpfWindow : Window
         return card;
     }
 
-    private static Expander CreateActivityExpander(WorkflowRulesEditorConversationItem item)
-    {
-        var details = new StackPanel
-        {
-            Orientation = Orientation.Vertical,
-            Margin = new Thickness(16, 2, 4, 6)
-        };
-
-        foreach (var line in item.ActivityLines ?? [])
-        {
-            details.Children.Add(CreateActivityLine(line));
-        }
-
-        var expander = new Expander
-        {
-            Header = item.Content,
-            Content = details,
-            IsExpanded = item.IsExpanded,
-            Foreground = MutedText,
-            Background = Brushes.Transparent,
-            Margin = new Thickness(4, 0, 4, 8)
-        };
-        expander.Expanded += (_, _) => item.IsExpanded = true;
-        expander.Collapsed += (_, _) => item.IsExpanded = false;
-        return expander;
-    }
-
-    private static TextBlock CreateActivityLine(string text)
-    {
-        return new TextBlock
-        {
-            Text = text,
-            Foreground = MutedText,
-            Margin = new Thickness(4, 2, 4, 2)
-        };
-    }
-
     private static Style CreateSendButtonStyle()
     {
         var style = new Style(typeof(Button));

MeetingAssistant/appsettings.json

@@ -172,7 +172,6 @@
       "Endpoint": "http://127.0.0.1:4021",
       "KeyEnv": "LITELLM_API_KEY",
       "Model": "chatgpt/gpt-5.5",
-      "UseStreaming": true,
       "EnableThinking": true,
       "ReasoningEffort": "Medium",
       "ReconnectionAttempts": 2,

README.md

@@ -108,7 +108,7 @@ Important settings:
 - `CalendarRecordingPrompts`: enables Outlook Classic Teams-start prompts on Windows.
 - `Screenshots`: controls the capture hotkey, attachment folder, and optional OCR/vision model.
 - `Agent`: configures the OpenAI-compatible summary/project agent endpoint, model, retries, output limits, and compaction.
-- `WorkflowRulesEditor`: optionally overrides the agent settings for the tray-launched assistant.
+- `WorkflowRulesEditor`: optionally overrides the agent settings for the tray-launched settings/logs assistant.
 
 Required or commonly used secrets:
 
@@ -125,14 +125,14 @@ See `docs/meeting-assistant-configuration.md` for the full configuration referen
 - **Azure AI Speech**: default checked-in ASR path, live diarized conversation transcription, and speaker identity matching.
 - **FunASR**: optional WebSocket streaming ASR. When managed backend startup is enabled, the app pulls and runs the configured Docker image as `meeting-assistant-funasr` on port `10095`.
 - **Whisper.NET plus pyannote**: optional local Whisper fallback and Docker-backed final diarization.
-- **LiteLLM/OpenAI-compatible Responses endpoint**: summary generation, screenshot OCR fallback, project tools, the tray-launched assistant, and retry flows.
+- **LiteLLM/OpenAI-compatible Responses endpoint**: summary generation, screenshot OCR fallback, project tools, settings/logs assistant, and retry flows.
 - **Docker Desktop or compatible Docker CLI**: required only for managed FunASR and pyannote paths.
 
 ## Workflow Rules And Agents
 
-Meeting-specific automation lives in a local YAML file, not in committed personal rules. Rules can trigger on meeting creation, assistant-context state transitions, identified speakers, transcript-line writes, and added attendees. They can add/remove attendees, set supported properties, add context, add projects, rewrite a transcript line, and transform an attendee name before it is stored.
+Meeting-specific automation lives in a local YAML file, not in committed personal rules. Rules can trigger on meeting creation, assistant-context state transitions, identified speakers, and transcript-line writes. They can add/remove attendees, set supported properties, add context, add projects, and rewrite a transcript line before persistence.
 
-The tray menu exposes `Open agent`, which opens the `Meeting Summary Agent` window. It can edit workflow rules with validation, inspect logs and health/status, manage speaker identities and samples, run ASR diagnostics, and read/write scoped meeting/project artifacts through explicit tools.
+The tray menu exposes the settings/logs assistant. It can edit workflow rules with validation, inspect logs and health/status, manage speaker identities and samples, run ASR diagnostics, and read/write scoped meeting/project artifacts through explicit tools.
 
 Detailed workflow syntax and extension guidance live in `docs/meeting-workflow-engine.md`.
 

docs/meeting-assistant-configuration.md

@@ -339,7 +339,6 @@ When enabled on Windows, Meeting Assistant periodically syncs today's Outlook Cl
 | `Key` | Optional inline API key. Prefer `KeyEnv`. |
 | `KeyEnv` | Environment variable name for the agent API key. |
 | `Model` | Model id sent to the endpoint. |
-| `UseStreaming` | Uses the Responses SSE transport when `true` (the default); uses the non-streaming Responses transport when `false`. |
 | `EnableThinking` | Enables reasoning options in requests when supported by the model/backend. |
 | `ReasoningEffort` | Reasoning effort: `None`, `Low`, `Medium`, `High`, or `ExtraHigh`. |
 | `ReconnectionAttempts` | Retry count for transient model endpoint failures. |
@@ -353,22 +352,18 @@ When enabled on Windows, Meeting Assistant periodically syncs today's Outlook Cl
 
 After transcription has fully finished, Meeting Assistant automatically runs the summary pipeline for the meeting. The summary agent writes the full markdown summary through `write_summary` and must provide a required `oneliner` value, which is stored in summary frontmatter and must not contain line breaks.
 
-The built-in summary-agent instructions treat assistant context as persistent meeting-specific memory. When the agent encounters an unexpected problem, missing information, or an assumption while summarizing, it appends a concise note through `write_context` so later work on the same meeting can use that history. A configured `Agent:InitialPrompt` completely replaces the built-in instructions, so custom prompts must include equivalent guidance when this behavior is desired.
-
 The summary agent can add and remove meeting-note attendees when transcript or OCR evidence is clear. It can override transcript speaker labels only when the evidence is very certain, and it can delete wrongfully matched identities. Final speaker identity learning and candidate updates run after the summary pipeline finishes so they use the summary-refined attendee list and any recorded speaker identity changes.
 
 `ContextWindowTokens`, `MaxOutputTokens`, `EnableCompaction`, and `CompactionRemainingRatio` configure summary-agent context monitoring. Meeting Assistant estimates the outgoing Responses payload size, logs the estimated token count and remaining context, and compacts the conversation when only the configured remaining ratio is left.
 
 ## Workflow Rules Editor
 
-`WorkflowRulesEditor` configures the tray-launched `Meeting Summary Agent` window. Blank values inherit from `Agent`, so it uses the summarizer endpoint, key, model, Responses transport, reasoning, retry, output, and compaction settings unless explicitly overridden.
+`WorkflowRulesEditor` configures the tray-launched settings/logs assistant. Blank values inherit from `Agent`, so it uses the summarizer endpoint, key, model, reasoning, retry, output, and compaction settings unless explicitly overridden.
 
-The overridable fields are `Endpoint`, `Key`, `KeyEnv`, `Model`, `UseStreaming`, `EnableThinking`, `ReasoningEffort`, `ReconnectionAttempts`, `ReconnectionDelay`, `ContextWindowTokens`, `MaxOutputTokens`, `EnableCompaction`, `CompactionRemainingRatio`, `ResponsesCompactPath`, and `InitialPrompt`.
+The overridable fields are `Endpoint`, `Key`, `KeyEnv`, `Model`, `EnableThinking`, `ReasoningEffort`, `ReconnectionAttempts`, `ReconnectionDelay`, `ContextWindowTokens`, `MaxOutputTokens`, `EnableCompaction`, `CompactionRemainingRatio`, `ResponsesCompactPath`, and `InitialPrompt`.
 
 The assistant can edit workflow rules, manage speaker identities, read and replace the local appsettings file, read this configuration document, search and read copied OpenSpec specs, inspect application logs, create/search/read/write project files, and post-process past meeting artifacts by listing recent summaries and reading, writing, or searching summaries, transcripts, meeting notes, and assistant context files. For artifact metadata repairs, it has frontmatter-specific write tools that preserve the markdown body.
 
-When asked to fix or investigate a meeting or summary, the agent is instructed to treat the matching assistant context as meeting-specific memory and read it for clues about problems, missing information, assumptions, prior fixes, and conclusions. After completing a repair, it appends a concise record of its fixes and conclusions to that context for future work on the meeting. This meeting-memory guidance is appended to the effective interactive-agent prompt even when `WorkflowRulesEditor:InitialPrompt` is configured.
-
 It also has in-process diagnostic tools that mirror the local HTTP diagnostics for health, recording status, current Outlook meeting lookup, recent speaker identity merging, workflow configuration reload, and ASR transcribe/diarize checks.
 
 The `search_spec` and `read_spec_file` tools are scoped to the copied `openspec/specs` markdown tree. Spec files are copied into build and publish output through an MSBuild glob, so newly added folders under `openspec/specs` are included automatically.
@@ -383,7 +378,7 @@ Independently from stdout and stderr redirection, Meeting Assistant writes an ap
 %TEMP%\MeetingAssistant\Logs\meeting-assistant.log
 ```
 
-On startup it rotates the previous current file to `meeting-assistant.log.1` and keeps up to `.4`. The tray-launched assistant reads and searches these current and rotated files. If another app instance or test host still has the file open, rotation is skipped and logging appends to the current file so startup is not blocked.
+On startup it rotates the previous current file to `meeting-assistant.log.1` and keeps up to `.4`. The settings/logs assistant reads and searches these current and rotated files. If another app instance or test host still has the file open, rotation is skipped and logging appends to the current file so startup is not blocked.
 
 ## API
 

docs/meeting-workflow-engine.md

@@ -22,7 +22,7 @@ The rules file path is configured through `MeetingAssistant:Automation:RulesPath
 
 If the configured path is empty, missing, or points to a blank file, the workflow engine does nothing.
 
-The tray menu includes `Open agent`, which opens the `Meeting Summary Agent` chat window for this configured rules file, the local speaker identity database, appsettings configuration, and application logs. The assistant uses the summarizer agent configuration by default and can be overridden through `MeetingAssistant:WorkflowRulesEditor`.
+The tray menu includes `Settings and logs`, which opens a small MewUI chat assistant for this configured rules file, the local speaker identity database, appsettings configuration, and application logs. The assistant uses the summarizer agent configuration by default and can be overridden through `MeetingAssistant:WorkflowRulesEditor`.
 
 ```json
 {
@@ -31,7 +31,6 @@ The tray menu includes `Open agent`, which opens the `Meeting Summary Agent` cha
       "Endpoint": "",
       "KeyEnv": "",
       "Model": "",
-      "UseStreaming": null,
       "EnableThinking": null,
       "ReasoningEffort": null,
       "MaxOutputTokens": null,
@@ -46,8 +45,6 @@ The tray menu includes `Open agent`, which opens the `Meeting Summary Agent` cha
 
 Blank editor values inherit from `MeetingAssistant:Agent`. This keeps the editor on the same LiteLLM Responses endpoint and model as the summarizer unless a value is explicitly configured. Each user-submitted editor turn sends its first model request with `X-Initiator: user`; follow-up model requests in the same turn, such as tool-call continuations, use `X-Initiator: agent`.
 
-While the agent is working, the window shows visible reasoning summaries returned by the Responses endpoint and called tool names as lightweight activity lines above the bottom activity line. After the turn completes, those lines move into a collapsible `Worked for <duration>` expander between the user and assistant messages.
-
 The editor agent prompt includes this document and is restricted to these tools:
 
 - `read_rules`: read the configured rules file, optionally by inclusive 1-based line range.
@@ -72,12 +69,6 @@ The engine runs when `MeetingRecordingCoordinator` emits a meeting workflow even
 - `state_transition`: after the assistant context state moves forward.
 - `speaker_identified`: when live or final speaker identification reports a display name.
 - `transcript_line`: after a formatted live transcript line is durably appended, before any changed line is rewritten in place, and before lines are used in full transcript rewrites.
-- `attendee_added`: before a newly added attendee is stored in the meeting note.
-
-`attendee_added` is emitted only by Meeting Assistant code paths that intentionally add an attendee
-through the workflow engine, notification actions, OCR/screenshot processing, summarizer tools, or
-other attendee-add operations. Direct edits to a meeting note file, including direct artifact/frontmatter
-repair writes by an agent, are left verbatim and do not trigger attendee automation.
 
 When a recording is started by accepting a calendar notification, the cached appointment metadata is applied before the `created` event. Rules still receive the normal `created` event and the normal `collecting metadata` to `transcribing` state transition, but they see the accepted appointment title, attendees, agenda, and scheduled end instead of a generated placeholder or a separate Outlook current-meeting lookup result.
 
@@ -169,22 +160,6 @@ on:
       speaker: Guest-1
 ```
 
-### `attendee_added`
-
-Runs before a newly added attendee is stored. `equals`, `contains`, and `regex` filters are optional; omitted filters match any attendee. `equals` and `contains` are case-insensitive.
-
-```yaml
-on:
-  - attendee_added:
-      contains: Contoso
-```
-
-```yaml
-on:
-  - attendee_added:
-      regex: '@contoso\.com>$'
-```
-
 ## Conditions
 
 Conditions use NCalc expressions. Dotted workflow variables may be written directly; the engine rewrites them into NCalc parameters internally.
@@ -207,7 +182,6 @@ Available condition variables:
 - `speaker.name`
 - `transcript.line`
 - `transcript.speaker`
-- `attendee.name`
 
 Available helper functions:
 
@@ -241,7 +215,6 @@ Step `value` fields can be plain strings or Razor templates. Razor templates rec
 - `Model.Speaker.Name`
 - `Model.Transcript.Line`
 - `Model.Transcript.Speaker`
-- `Model.Attendee.Name`
 
 Example:
 
@@ -284,11 +257,7 @@ steps:
 
 ### `set_property`
 
-Sets a supported property. Supported meeting properties are `title` and `meeting.title`. During `transcript_line` events, `transcript.line` is also supported. During `attendee_added` events, `attendee.name` is supported. For live transcription, changing `transcript.line` rewrites the referenced formatted line after the workflow task completes. For attendee additions, changing `attendee.name` stores the transformed attendee instead of the original added value.
-
-Rules triggered by `attendee_added` are transformation-only: their steps must use `set_property`
-with `property: attendee.name`. They can still use trigger filters, conditions, and Razor templates
-to decide and compute the transformed value, but they cannot perform note/context side effects.
+Sets a supported property. Supported meeting properties are `title` and `meeting.title`. During `transcript_line` events, `transcript.line` is also supported. For live transcription, changing `transcript.line` rewrites the referenced formatted line after the workflow task completes.
 
 ```yaml
 steps:
@@ -304,13 +273,6 @@ steps:
     value: '@Model.Transcript.Line.Replace("*****", "[redacted]")'
 ```
 
-```yaml
-steps:
-  - uses: set_property
-    property: attendee.name
-    value: '@Model.Attendee.Name.Replace(" (Contoso)", "")'
-```
-
 ### `add_context`
 
 Appends rendered text to the assistant context artifact body. This step does not count as a meeting-note mutation and does not cause a meeting-note save by itself.
@@ -406,20 +368,6 @@ rules:
       value: '@Model.Transcript.Line.Replace("*****", "[redacted]")'
 ```
 
-Clean attendee names as they are added:
-
-```yaml
-rules:
-- name: clean-contoso-attendees
-  on:
-    - attendee_added:
-        contains: Contoso
-  steps:
-    - uses: set_property
-      property: attendee.name
-      value: '@Model.Attendee.Name.Replace(" (Contoso)", "")'
-```
-
 ## Implementation Notes
 
 Core files:

openspec/changes/archive/2026-07-01-transform-added-attendees/.openspec.yaml

@@ -1,2 +0,0 @@
-schema: spec-driven
-created: 2026-07-01

openspec/changes/archive/2026-07-01-transform-added-attendees/design.md

@@ -1,30 +0,0 @@
-# Design
-
-## Event Model
-
-Add a workflow event type, `attendee_added`, that carries the attendee string being added. The event is used as a transformation hook before a caller stores the attendee in the meeting note. Rules can match the event with `equals`, `contains`, or `regex` trigger filters over the current attendee value.
-
-## Transformation Contract
-
-The workflow engine exposes `TransformAttendeeAsync`, mirroring `TransformTranscriptLineAsync`. During `attendee_added` events:
-
-- conditions can read `attendee.name`,
-- Razor templates can read `Model.Attendee.Name`,
-- `set_property attendee.name` mutates the attendee value returned to the caller.
-
-Other workflow steps keep their existing semantics, but the intended transformation path is `set_property attendee.name`.
-Rules triggered by `attendee_added` are limited to that transformation path so the value transform does not hide unrelated note or context side effects.
-
-Direct meeting note file edits remain outside the workflow event model. If a user or agent writes a meeting note file/frontmatter directly, Meeting Assistant preserves that content verbatim and does not emit `attendee_added`.
-
-## Call Sites
-
-Callers transform attendee candidates before writing them:
-
-- metadata and accepted prompt metadata after canonicalization,
-- workflow `add_attendee` steps before de-duplication,
-- screenshot OCR candidates before canonicalization and merge,
-- summary-agent `add_attendee` tool calls before duplicate checks and note writes,
-- speaker identity attendee additions before adding matched names.
-
-Unchanged metadata attendees preserve the existing canonicalizer output, including email display strings.

openspec/changes/archive/2026-07-01-transform-added-attendees/proposal.md

@@ -1,17 +0,0 @@
-## Why
-
-Attendees enter Meeting Assistant from several paths: calendar metadata, accepted recording prompts, workflow actions, screenshot OCR, speaker identity updates, and summarizer add-attendee tools. Today those paths can normalize display names, but they cannot apply local transformation rules such as trimming company suffixes, replacing aliases, or cleaning attendee strings consistently.
-
-## What Changes
-
-- Add an `attendee_added` workflow trigger that runs whenever Meeting Assistant adds an attendee to a meeting note.
-- Allow `attendee_added` triggers to filter the added attendee with full equality, substring, or regex matching.
-- Expose `attendee.name` to workflow conditions and Razor templates.
-- Allow `set_property` to update `attendee.name` during `attendee_added` events, mirroring transcript-line transformation.
-- Apply attendee transformations to attendees added by metadata import, prompted-start metadata, workflow `add_attendee`, screenshot OCR, and summary/workflow editor agent tools.
-
-## Impact
-
-- Workflow engine, rules schema validation, and workflow documentation.
-- Attendee write paths in recording metadata, screenshot OCR, summary tools, and workflow actions.
-- Behavior tests for workflow transformation and representative attendee-add sources.

openspec/changes/archive/2026-07-01-transform-added-attendees/specs/meeting-session/spec.md

@@ -1,29 +0,0 @@
-## MODIFIED Requirements
-
-### Requirement: Meeting automation rules support lifecycle triggers
-Meeting Assistant SHALL support rule triggers for `created`, `state_transition`, `speaker_identified`, `transcript_line`, and `attendee_added`.
-
-An `attendee_added` trigger MAY filter by `equals`, `contains`, or `regex` against the attendee value being added. `equals` and `contains` filters SHALL match case-insensitively.
-
-Meeting Assistant SHALL apply attendee-added transformations before storing attendees added from meeting metadata, accepted recording-prompt metadata, workflow `add_attendee` steps, screenshot OCR, speaker identity updates, and summarizer add-attendee tools.
-
-#### Scenario: Attendee added rule filters and transforms a metadata attendee
-- **GIVEN** a configured rule that triggers on added attendees containing `@contoso.com`
-- **AND** the rule sets `attendee.name` to a display name derived from the added attendee
-- **WHEN** Meeting Assistant adds attendee `Ada Lovelace <ada@contoso.com>` from meeting metadata
-- **THEN** the stored meeting attendee is the transformed attendee name
-- **AND** the original attendee string is not stored
-
-### Requirement: Meeting automation rules support conditions and steps
-Meeting Assistant SHALL expose the added attendee name to `attendee_added` rule conditions and Razor step templates as `attendee.name`.
-
-The `set_property` step SHALL support setting `attendee.name` during `attendee_added` events.
-
-Rules with an `attendee_added` trigger SHALL be limited to transforming `attendee.name` with `set_property`.
-
-Direct meeting note file writes SHALL NOT emit `attendee_added` or transform attendee values.
-
-#### Scenario: Attendee added rules can use regex and Razor
-- **GIVEN** a configured `attendee_added` rule with a regex filter over the added attendee
-- **WHEN** Meeting Assistant adds an attendee matching the regex
-- **THEN** the rule can set `attendee.name` with a Razor template using `Model.Attendee.Name`

openspec/changes/archive/2026-07-01-transform-added-attendees/tasks.md

@@ -1,8 +0,0 @@
-# Tasks
-
-- [x] Add OpenSpec scenarios for attendee-added trigger filtering and transformation.
-- [x] Add a failing workflow-engine behavior test for transforming an added attendee through `attendee_added`.
-- [x] Implement attendee-added workflow event, trigger filters, `attendee.name` condition/template value, and `set_property attendee.name`.
-- [x] Apply attendee transformations from metadata/prompted starts, workflow `add_attendee`, screenshot OCR, and agent attendee tools.
-- [x] Update workflow engine documentation.
-- [x] Run focused tests plus `openspec validate transform-added-attendees --strict`.

openspec/changes/archive/2026-07-17-use-assistant-context-as-meeting-memory/.openspec.yaml

@@ -1,2 +0,0 @@
-schema: spec-driven
-created: 2026-07-17

openspec/changes/archive/2026-07-17-use-assistant-context-as-meeting-memory/design.md

@@ -1,45 +0,0 @@
-## Context
-
-Meeting Assistant already creates one assistant context artifact per meeting and gives both the automatic summary agent and the interactive agent window tools that can read and append to it. The current summary prompt calls the file a notebook, while the interactive agent prompt describes only generic artifact repair. Neither prompt establishes assistant context as the durable handoff point for uncertainty, failures, assumptions, repairs, and conclusions about one meeting.
-
-## Goals / Non-Goals
-
-**Goals:**
-
-- Make the assistant context's meeting-specific memory role explicit to both agents.
-- Preserve summarization problems, missing information, and assumptions for later investigation.
-- Make the interactive agent consult that memory before repairing a meeting or summary.
-- Make the interactive agent record completed fixes and conclusions for later agents.
-
-**Non-Goals:**
-
-- Add a new artifact type, schema, section format, or database.
-- Automatically rewrite or summarize existing assistant context files.
-- Change tool permissions or allow agents to access artifacts outside existing configured scopes.
-- Override an explicitly configured custom summary-agent prompt with built-in summary guidance.
-
-## Decisions
-
-### Extend the existing instruction builders
-
-The automatic summary behavior will be added to `MeetingSummaryInstructionBuilder.DefaultInitialPrompt`. This preserves the existing contract that a configured `Agent:InitialPrompt` replaces the built-in summary prompt.
-
-The interactive behavior will be included in the instruction builder's always-appended meeting-artifact guidance. This keeps the meeting-memory reminder available even when the interactive agent has a custom initial prompt, alongside the tool capabilities that Meeting Assistant already appends.
-
-Alternative considered: implement automatic interception or mandatory writes whenever an agent encounters uncertainty. The application cannot reliably infer those semantic events from arbitrary model turns, so explicit instructions are the smallest dependable mechanism.
-
-### Keep memory entries append-oriented and concise
-
-Agents will be told to append problems, missing information, assumptions, fixes, and conclusions to the matching assistant context. Existing `write_context` behavior already appends by default, which preserves earlier observations and avoids introducing a structured migration.
-
-Alternative considered: define mandatory headings or a machine-readable memory schema. That would add formatting and compatibility obligations without being necessary for the requested agent handoff behavior.
-
-### Keep user-facing summary content separate
-
-Assistant context remains internal meeting memory; the generated summary remains the user-facing artifact. Assumptions and missing evidence can inform explicit uncertainty in the summary, but diagnostic notes and repair history belong in assistant context.
-
-## Risks / Trade-offs
-
-- [Repeated agents may append duplicate observations] → Instruct agents to read existing context first and keep entries concise.
-- [Assistant context can grow over time] → Preserve existing ranged and tail reads; no new retention mechanism is introduced.
-- [Configured custom summary prompts omit the new built-in guidance] → Preserve the documented replacement semantics and leave responsibility with the custom prompt author.

openspec/changes/archive/2026-07-17-use-assistant-context-as-meeting-memory/proposal.md

@@ -1,27 +0,0 @@
-## Why
-
-The automatic summarizer and interactive agent window can encounter uncertainty, missing information, and repair conclusions that are useful to later work on the same meeting. Those observations currently depend on the active agent turn instead of being consistently preserved in the meeting's assistant context.
-
-## What Changes
-
-- Treat each assistant context file as meeting-specific memory shared by agents working on that meeting.
-- Guide the automatic summarizer to record unexpected problems, missing information, and assumptions in assistant context.
-- Guide the interactive agent window to consult assistant context when repairing a meeting or summary.
-- Guide the interactive agent window to append its fixes and conclusions to assistant context for future work on that meeting.
-
-## Capabilities
-
-### New Capabilities
-
-None.
-
-### Modified Capabilities
-
-- `meeting-summary`: Define the automatic summarizer's responsibility to persist uncertainty and problems in meeting-specific assistant context.
-- `meeting-session`: Define how the interactive agent window uses and updates assistant context during meeting and summary repairs.
-
-## Impact
-
-- Summary-agent default instructions and their behavior tests.
-- Interactive agent-window instructions and their behavior tests.
-- Agent documentation describing assistant context usage.

openspec/changes/archive/2026-07-17-use-assistant-context-as-meeting-memory/specs/meeting-session/spec.md

@@ -1,14 +0,0 @@
-## ADDED Requirements
-
-### Requirement: Interactive agent uses assistant context as meeting memory
-The interactive agent window instructions SHALL identify each assistant context file as meeting-specific memory.
-
-When the user asks the interactive agent to fix or investigate a meeting or summary, the instructions SHALL direct the agent to read the matching assistant context for clues about problems, missing information, assumptions, prior fixes, and conclusions.
-
-After repairing a meeting or summary, the instructions SHALL direct the agent to append a concise record of its fixes and conclusions to the matching assistant context.
-
-#### Scenario: Interactive agent repairs a meeting artifact
-- **GIVEN** a meeting has an assistant context file
-- **WHEN** the user asks the interactive agent to fix that meeting or its summary
-- **THEN** the agent instructions direct it to inspect the matching assistant context for relevant meeting-specific memory
-- **AND** direct it to append its fixes and conclusions to that assistant context

openspec/changes/archive/2026-07-17-use-assistant-context-as-meeting-memory/specs/meeting-summary/spec.md

@@ -1,12 +0,0 @@
-## ADDED Requirements
-
-### Requirement: Summary agent preserves meeting-specific working memory
-When Meeting Assistant uses the built-in summary-agent instructions, those instructions SHALL identify assistant context as persistent meeting-specific memory.
-
-The built-in instructions SHALL direct the summary agent to append unexpected problems, missing information, and assumptions encountered while summarizing to assistant context.
-
-#### Scenario: Summarizer encounters uncertainty
-- **GIVEN** the built-in summary-agent instructions are in use
-- **WHEN** the summary agent encounters an unexpected problem, cannot find needed information, or must make an assumption
-- **THEN** its instructions direct it to append a concise record to the meeting's assistant context
-- **AND** later agents can discover that record when working on the same meeting

openspec/changes/archive/2026-07-17-use-assistant-context-as-meeting-memory/tasks.md

@@ -1,14 +0,0 @@
-## 1. Automatic Summary Agent
-
-- [x] 1.1 Add a behavior test that requires the built-in summary instructions to treat assistant context as meeting-specific memory and record problems, missing information, and assumptions.
-- [x] 1.2 Update the built-in summary-agent guidance to satisfy the memory behavior.
-
-## 2. Interactive Agent Window
-
-- [x] 2.1 Add a behavior test that requires the interactive instructions to inspect assistant context during meeting or summary repair and record fixes and conclusions.
-- [x] 2.2 Update the interactive agent-window guidance to satisfy the memory behavior.
-
-## 3. Documentation and Validation
-
-- [x] 3.1 Document assistant context as shared meeting-specific agent memory.
-- [x] 3.2 Run focused behavior tests, the full test suite, and strict OpenSpec validation.

openspec/changes/archive/2026-07-27-handle-responses-event-streams/.openspec.yaml

@@ -1,2 +0,0 @@
-schema: spec-driven
-created: 2026-07-27

openspec/changes/archive/2026-07-27-handle-responses-event-streams/design.md

@@ -1,40 +0,0 @@
-## Context
-
-`LiteLlmResponsesChatClient` exposes a non-streaming `IChatClient` interface and currently buffers every successful `/v1/responses` body before parsing it as one JSON document. The deployed LiteLLM `chatgpt/gpt-5.5` route instead returns Responses Server-Sent Events. Microsoft.Extensions.AI.OpenAI already provides a Responses streaming adapter that maps the OpenAI SDK's typed SSE updates into `ChatResponseUpdate` values, including text, reasoning, function calls, response metadata, and usage.
-
-That adapter uses `FunctionCallContent.CreateFromParsedArguments`, which records argument-mapping failures on the function call instead of throwing them from the response parser. The default function invoker does not itself prevent invocation when that property is populated, so Meeting Assistant must turn the recorded parse failure into a matching function result before calling the tool.
-
-## Goals / Non-Goals
-
-**Goals:**
-
-- Preserve the existing non-streaming `IChatClient` contract while selecting the supported Responses streaming or non-streaming path through configuration.
-- Delegate SSE framing, event deserialization, streamed function-call assembly, response metadata, and usage mapping to the OpenAI SDK and Microsoft.Extensions.AI.OpenAI adapter.
-- Preserve the original encoded function arguments when forwarding conversation history.
-- Return a safe, structured invalid-arguments result for malformed argument JSON without invoking the requested tool.
-- Apply the guarded invocation behavior to both summary and workflow-editor agent pipelines that use this Responses client.
-
-**Non-Goals:**
-
-- Expose token-by-token upstream streaming to the UI.
-- Replace Microsoft.Extensions.AI function invocation or implement general JSON Schema validation.
-- Recover from an incomplete or malformed Responses event stream that has no usable completed output.
-
-## Decisions
-
-1. Add `Agent:UseStreaming`, defaulting to `true`, and send summary requests through the matching OpenAI SDK method. The streaming path uses `ResponsesClient.CreateResponseStreamingAsync` and collects the Microsoft.Extensions.AI.OpenAI typed updates into one `ChatResponse`; the non-streaming path uses `ResponsesClient.CreateResponseAsync` and its supported `AsChatResponse` adapter.
-
-2. Keep the existing JSON payload builder because it owns Meeting Assistant-specific compaction input, tool serialization, reasoning settings, retry diagnostics, and initiator behavior. Deserialize that payload into the OpenAI SDK's `CreateResponseOptions`, set its streaming flag from configuration, and let the selected SDK method own the HTTP response protocol.
-
-   The SDK deserializes `ResponseItem` values by their `type` discriminator. Message items must therefore include `"type": "message"` before the JSON payload is converted to `CreateResponseOptions`; otherwise the SDK reserializes them as `"type": "unknown"` and the provider rejects the request.
-
-3. Rely on the framework adapter's `FunctionCallContent.CreateFromParsedArguments` mapping for function-call argument decoding and raw response preservation. A shared function-invocation callback checks the recorded parse exception before invocation. For invalid JSON it returns a structured `invalid_tool_arguments` value that the framework associates with the original call ID; otherwise it delegates to the actual function.
-
-4. Configure both agent pipelines with the shared guarded invoker. This avoids duplicating the safety decision and prevents zero-argument tools from accidentally running when malformed JSON would otherwise map to an empty argument set.
-
-## Risks / Trade-offs
-
-- **[Provider event variants]** A provider could emit events outside the OpenAI Responses schema. → Use the maintained OpenAI SDK parser and fail diagnostically for genuinely incompatible streams instead of maintaining local event variants.
-- **[Buffered result]** The summary pipeline does not expose token-by-token updates to its caller. → Collect the framework updates only at the existing non-streaming boundary; the HTTP response itself remains streamed and incrementally parsed.
-- **[Experimental API]** The OpenAI Responses adapter is marked experimental in the currently referenced package. → Keep it behind `LiteLlmResponsesChatClient`, which isolates future package API changes from the rest of Meeting Assistant.
-- **[Error disclosure]** Raw parser exceptions may contain implementation details. → Return a stable error code and concise validation message rather than exception text or the malformed arguments.

openspec/changes/archive/2026-07-27-handle-responses-event-streams/proposal.md

@@ -1,26 +0,0 @@
-## Why
-
-The configured LiteLLM Responses endpoint can return successful responses as Server-Sent Events even when the request sets `stream: false`. Meeting Assistant currently treats every successful body as one JSON document, and it also lets malformed function-call argument JSON escape as a fatal parse exception, so either condition can abort an otherwise recoverable summary run.
-
-## What Changes
-
-- Consume the configured LiteLLM Responses endpoint through the supported OpenAI Responses streaming transport and Agent Framework adapter.
-- Assemble completed response output items and usage metadata from Responses SSE events before returning them through the existing non-streaming chat-client interface.
-- Add an agent setting that selects streaming or non-streaming Responses transport, with streaming enabled by default.
-- Preserve valid Responses item discriminators when translating agent messages through the OpenAI SDK request model.
-- Treat malformed function-call arguments as invalid tool input that is returned to the agent for correction without invoking the tool or terminating the summary run.
-- Add behavior tests for streamed text, streamed function calls, and invalid function-call arguments.
-
-## Capabilities
-
-### New Capabilities
-
-- None.
-
-### Modified Capabilities
-
-- `meeting-summary`: Make the existing Responses-based summary pipeline interoperable with SSE responses and resilient to malformed tool-call input.
-
-## Impact
-
-The change affects agent configuration, the custom LiteLLM Responses chat client, its summary-agent integration, and focused tests. It does not change the public HTTP API.

openspec/changes/archive/2026-07-27-handle-responses-event-streams/specs/meeting-summary/spec.md

@@ -1,36 +0,0 @@
-## ADDED Requirements
-
-### Requirement: Summary agents tolerate Responses event streams and malformed tool arguments
-Meeting Assistant SHALL consume successful summary-agent Responses results through the supported OpenAI Responses and Agent Framework Server-Sent Events adapter.
-
-When a Responses event stream delivers completed output items separately from the final response metadata, Meeting Assistant SHALL assemble those output items into one agent response while preserving final response metadata and usage.
-
-Meeting Assistant SHALL provide an agent setting that selects streaming or non-streaming Responses transport. Streaming SHALL be enabled by default. When streaming is disabled, Meeting Assistant SHALL use the supported non-streaming OpenAI Responses client and adapter.
-
-When a returned function call contains arguments that are not a valid JSON object, Meeting Assistant SHALL NOT invoke the requested function and SHALL return an invalid-tool-arguments result associated with the original call ID to the agent so it can correct the call.
-
-When Meeting Assistant translates chat messages into a Responses request, every message input item SHALL retain the `message` item discriminator required by the OpenAI SDK and Responses API.
-
-#### Scenario: Streamed text response is assembled
-- **WHEN** the configured Responses endpoint returns completed message output in Server-Sent Events followed by final response metadata
-- **THEN** the summary agent receives the completed message text, response metadata, and usage without a JSON document parse failure
-
-#### Scenario: Streamed function call is assembled
-- **WHEN** the configured Responses endpoint returns a completed function-call output item in Server-Sent Events
-- **THEN** the summary agent receives the function call with its call ID, function name, and parsed arguments
-
-#### Scenario: Streaming transport can be disabled
-- **WHEN** `MeetingAssistant:Agent:UseStreaming` is `false`
-- **THEN** the summary agent requests a non-streaming Responses result
-- **AND** converts the response through the supported OpenAI Responses adapter
-
-#### Scenario: Chat message input remains a Responses message
-- **WHEN** the summary agent sends a user or assistant chat message through the Responses client
-- **THEN** the outbound Responses input item has type `message`
-- **AND** the request does not contain an `unknown` input item type
-
-#### Scenario: Malformed function arguments are returned to the agent
-- **WHEN** the model returns a function call whose arguments are not a valid JSON object
-- **THEN** Meeting Assistant does not invoke the requested function
-- **AND** sends a function result with an invalid-tool-arguments error for the original call ID back to the agent
-- **AND** allows the agent loop to continue

openspec/changes/archive/2026-07-27-handle-responses-event-streams/tasks.md

@@ -1,24 +0,0 @@
-## 1. Responses SSE compatibility
-
-- [x] 1.1 Add a failing client behavior test for a streamed text response with final metadata and usage.
-- [x] 1.2 Add failing configuration and client behavior tests for selecting non-streaming Responses transport.
-- [x] 1.3 Route Responses requests through the Agent Framework/OpenAI SDK streaming or non-streaming adapter according to configuration.
-- [x] 1.4 Add behavior coverage for streamed function-call output.
-
-## 2. Invalid tool-argument recovery
-
-- [x] 2.1 Add a failing agent-loop behavior test proving malformed function-call JSON does not invoke the tool and is returned to the agent.
-- [x] 2.2 Preserve argument parse failures on function-call content and add the shared guarded function invoker.
-- [x] 2.3 Apply guarded function invocation to the summary and workflow-editor agent pipelines.
-
-## 3. Verification
-
-- [x] 3.1 Refactor the touched response and invocation paths for DRYness, SOLID boundaries, and simplicity while preserving behavior.
-- [x] 3.2 Run focused tests, the full solution tests, and strict OpenSpec validation.
-- [x] 3.3 Verify the client behavior against the deployed LiteLLM Responses endpoint and restart Meeting Assistant only after confirming it is idle.
-
-## 4. Responses message-item compatibility
-
-- [x] 4.1 Add a failing client behavior test proving an outbound chat message retains the `message` discriminator through SDK serialization.
-- [x] 4.2 Emit the Responses `message` discriminator for chat message input items.
-- [x] 4.3 Run focused and full tests, validate OpenSpec strictly, restart while idle, and retry the failed summary through the local API.

openspec/changes/resilient-transcript-workflow/tasks.md

@@ -6,4 +6,4 @@
 - [x] Add workflow rule completion/error logging.
 - [x] Update workflow engine documentation.
 - [x] Run focused tests and `openspec validate resilient-transcript-workflow --strict`.
-- [x] Follow up later: investigate anomalous temporary-recording disk-full `IOException` despite small expected WAV size. Closed for now because the anomaly has not recurred.
+- [ ] Follow up later: investigate anomalous temporary-recording disk-full `IOException` despite small expected WAV size.

openspec/specs/meeting-session/spec.md

@@ -44,8 +44,6 @@ The meeting note frontmatter SHALL link to the transcript, assistant context, an
 
 Generated artifact notes SHALL link only to the other notes from the same run and SHALL omit the frontmatter property that would reference themselves.
 
-Meeting Assistant SHALL escape generated meeting-note frontmatter string values after metadata enrichment and workflow rules have been applied, immediately before writing the final markdown file.
-
 #### Scenario: Meeting note links to generated artifacts
 - **WHEN** Meeting Assistant creates a meeting note
 - **THEN** the note frontmatter links to the configured transcript, assistant context, and summary note locations
@@ -61,12 +59,6 @@ Meeting Assistant SHALL escape generated meeting-note frontmatter string values
 - **THEN** the artifact frontmatter links to the other run notes
 - **AND** the artifact frontmatter omits the property for the artifact's own note type
 
-#### Scenario: Generated frontmatter remains parseable after attendee transforms
-- **GIVEN** meeting metadata or workflow rules produce an attendee name that contains a single quote
-- **WHEN** Meeting Assistant writes the final meeting note
-- **THEN** the attendee value is escaped in frontmatter
-- **AND** the meeting note frontmatter remains parseable when read back
-
 ### Requirement: Meeting notes preserve user-authored content
 Meeting Assistant SHALL preserve user-authored meeting notes and include them as input when generating summaries, decisions, and next steps.
 
@@ -229,43 +221,95 @@ Meeting Assistant SHALL expose a diagnostic endpoint that reloads application co
 - **AND** future workflow events use the reloaded workflow automation configuration
 
 ### Requirement: Meeting automation rules support lifecycle triggers
-Meeting Assistant SHALL support rule triggers for `created`, `state_transition`, `speaker_identified`, `transcript_line`, and `attendee_added`.
+Meeting Assistant SHALL support rule triggers for `created`, `state_transition`, `speaker_identified`, and `transcript_line`.
 
-An `attendee_added` trigger MAY filter by `equals`, `contains`, or `regex` against the attendee value being added. `equals` and `contains` filters SHALL match case-insensitively.
+A `state_transition` trigger MAY filter by `from`, `to`, or both state values.
 
-Meeting Assistant SHALL apply attendee-added transformations before storing attendees added from meeting metadata, accepted recording-prompt metadata, workflow `add_attendee` steps, screenshot OCR, speaker identity updates, and summarizer add-attendee tools.
+A `speaker_identified` trigger MAY filter by speaker name.
 
-#### Scenario: Attendee added rule filters and transforms a metadata attendee
-- **GIVEN** a configured rule that triggers on added attendees containing `@contoso.com`
-- **AND** the rule sets `attendee.name` to a display name derived from the added attendee
-- **WHEN** Meeting Assistant adds attendee `Ada Lovelace <ada@contoso.com>` from meeting metadata
-- **THEN** the stored meeting attendee is the transformed attendee name
-- **AND** the original attendee string is not stored
+A `transcript_line` trigger MAY filter by speaker name.
+
+#### Scenario: State transition rule matches from and to
+- **GIVEN** a configured rule that triggers on a state transition from `collecting metadata` to `transcribing`
+- **WHEN** Meeting Assistant transitions that meeting from `collecting metadata` to `transcribing`
+- **THEN** it applies the rule steps
+
+#### Scenario: Speaker identified rule filters by name
+- **GIVEN** a configured rule that triggers when speaker `Ada` is identified
+- **WHEN** Meeting Assistant identifies speaker `Grace`
+- **THEN** it does not apply the rule
+- **WHEN** Meeting Assistant identifies speaker `Ada`
+- **THEN** it applies the rule steps
+
+#### Scenario: Transcript line rule rewrites masked profanity before persistence
+- **GIVEN** a configured rule that triggers on transcript line writes and sets `transcript.line` by replacing `*****` with `[redacted]`
+- **WHEN** Meeting Assistant writes a transcript line for speaker `Guest-1` containing `*****`
+- **THEN** the written transcript line contains `[redacted]`
+- **AND** the written transcript line does not contain `*****`
 
 ### Requirement: Meeting automation rules support conditions and steps
-Meeting Assistant SHALL expose the added attendee name to `attendee_added` rule conditions and Razor step templates as `attendee.name`.
+Meeting Assistant SHALL support rule conditions using an expression engine.
 
-The `set_property` step SHALL support setting `attendee.name` during `attendee_added` events.
+Rules SHALL support nested `and`, `or`, and `not` condition groups.
 
-Rules with an `attendee_added` trigger SHALL be limited to transforming `attendee.name` with `set_property`.
+Step values SHALL support Razor syntax against the current meeting event model.
 
-Direct meeting note file writes SHALL NOT emit `attendee_added` or transform attendee values.
+Step values SHALL treat `@` characters inside valid email address tokens as literal text rather than Razor transitions.
 
-#### Scenario: Attendee added rules can use regex and Razor
-- **GIVEN** a configured `attendee_added` rule with a regex filter over the added attendee
-- **WHEN** Meeting Assistant adds an attendee matching the regex
-- **THEN** the rule can set `attendee.name` with a Razor template using `Model.Attendee.Name`
+Meeting Assistant SHALL expose the formatted transcript line and transcript speaker to `transcript_line` rule conditions and Razor step templates.
+
+Meeting Assistant SHALL support these initial rule steps:
+
+- `add_attendee`
+- `remove_attendee`
+- `set_property`
+- `add_context`
+- `add_project`
+
+The `set_property` step SHALL support setting `transcript.line` during `transcript_line` events.
+
+#### Scenario: Nested conditions choose a matching rule
+- **GIVEN** a configured rule with nested `and`, `or`, and `not` conditions over meeting title, attendees, and event data
+- **WHEN** the condition evaluates to true
+- **THEN** Meeting Assistant applies the rule
+- **WHEN** the condition evaluates to false
+- **THEN** Meeting Assistant skips the rule
+
+#### Scenario: Templated context mentions identified speaker
+- **GIVEN** a configured `speaker_identified` rule with an `add_context` step using Razor syntax
+- **WHEN** Meeting Assistant identifies matching speaker `Ada`
+- **THEN** it appends rendered context text containing `Ada` to the assistant context note
+
+#### Scenario: Email addresses do not trigger Razor templating
+- **GIVEN** a configured rule step value containing `Support@contoso.com`
+- **WHEN** the rule runs
+- **THEN** Meeting Assistant preserves the email address as literal text
+
+#### Scenario: Email addresses can appear beside Razor templating
+- **GIVEN** a configured rule step value containing both `Support@contoso.com` and a Razor expression
+- **WHEN** the rule runs
+- **THEN** Meeting Assistant renders the Razor expression and preserves the email address as literal text
+
+#### Scenario: Rule can clean a meeting title
+- **GIVEN** a configured state-transition rule that matches a title containing a configured marker
+- **WHEN** the rule runs
+- **THEN** Meeting Assistant can update the meeting title through `set_property`
+
+#### Scenario: Transcript line conditions can use the written line and speaker
+- **GIVEN** a configured `transcript_line` rule with conditions over `transcript.line` and `transcript.speaker`
+- **WHEN** Meeting Assistant writes a transcript line that matches both conditions
+- **THEN** it applies the rule steps before the line is persisted
 
 ### Requirement: Workflow rules and speaker identities can be edited through a tray-launched assistant
-Meeting Assistant SHALL expose an `Open agent` item from the tray icon menu.
+Meeting Assistant SHALL expose an `Edit rules and identities` item from the tray icon menu.
 
 The tray icon SHALL be implemented through the Uno notification icon stack.
 
 The tray icon menu SHALL show every configured launch profile when recording can be started and SHALL include each profile's configured toggle hotkey in the corresponding start or switch menu item.
 
-When the user selects `Open agent`, Meeting Assistant SHALL open a basic MewUI chat window for editing the configured workflow rules file and speaker identities.
+When the user selects `Edit rules and identities`, Meeting Assistant SHALL open a basic MewUI chat window for editing the configured workflow rules file and speaker identities.
 
-The chat window SHALL be titled `Meeting Summary Agent`, SHALL display user and assistant messages as visually distinct cards, SHALL display basic markdown emphasis, inline code, fenced code blocks, pipe tables rendered as grid views, and line breaks in agent responses, SHALL display lightweight visible reasoning summary lines and tool-call lines that include only the called tool name while the agent is working, SHALL display a plain `Thinking...` line as the bottom activity line while the agent is working, MAY replace that bottom line with `Reconnecting...` while retrying a transient agent request, SHALL replace per-turn activity after completion with a collapsible `Worked for <duration>` expander between the user and assistant cards, SHALL show visible reasoning summary lines, tool-call lines, and status lines in order inside the expander when expanded, SHALL collapse those details again when the expander is closed, SHALL provide a multiline text input at the bottom with placeholder text for asking to make a rule or list identities, SHALL send on Enter, SHALL insert a newline on Shift+Enter, and SHALL provide an explicit Send button.
+The chat window SHALL be titled `Edit rules and identities`, SHALL display user and assistant messages as visually distinct cards, SHALL display basic markdown emphasis, inline code, fenced code blocks, pipe tables rendered as grid views, and line breaks in agent responses, SHALL display a plain `Thinking...` line while the agent is working, MAY replace that line with `Reconnecting...` while retrying a transient agent request, SHALL provide a multiline text input at the bottom with placeholder text for asking to make a rule or list identities, SHALL send on Enter, SHALL insert a newline on Shift+Enter, and SHALL provide an explicit Send button.
 
 When a new chat message or thinking state is appended, the chat window SHALL scroll to the bottom of the newly rendered content if the conversation was already near the bottom or did not need scrolling before the append.
 
@@ -291,8 +335,8 @@ Meeting Assistant SHALL provide a diagnostic endpoint that opens the workflow ru
 
 #### Scenario: Tray menu opens the editor
 - **WHEN** the user opens the tray icon menu
-- **THEN** the menu includes `Open agent`
-- **WHEN** the user selects `Open agent`
+- **THEN** the menu includes `Edit rules and identities`
+- **WHEN** the user selects `Edit rules and identities`
 - **THEN** Meeting Assistant opens the workflow rules and identities editor chat window
 
 #### Scenario: Tray menu shows configured profile hotkeys
@@ -341,16 +385,9 @@ Meeting Assistant SHALL provide a diagnostic endpoint that opens the workflow ru
 - **GIVEN** the rules editor chat window is open
 - **WHEN** the user types a prompt and presses Enter
 - **THEN** the user message appears in the conversation
-- **AND** a plain `Thinking...` line appears at the bottom while the agent is running
-- **WHEN** the agent emits visible reasoning summaries or calls tools
-- **THEN** lightweight activity lines appear above the bottom activity line with the visible reasoning summary text or only the called tool names
-- **WHEN** the agent turn completes
-- **THEN** the activity is represented between the user and assistant cards as a collapsed `Worked for <duration>` expander
-- **AND** expanding it shows the visible reasoning summary, status, and tool-call lines in order
-- **AND** collapsing it hides those details again
-- **AND** if no visible reasoning summary was emitted, the expander includes the fallback `Thinking...` line
+- **AND** a plain `Thinking...` line appears while the agent is running
 - **AND** the first model request for that turn is marked as user-initiated
-- **AND** the final assistant response replaces the live activity lines
+- **AND** the final assistant response replaces the thinking line
 
 #### Scenario: Rules editor displays agent request failures
 - **GIVEN** the rules editor chat window is open
@@ -363,15 +400,3 @@ Meeting Assistant SHALL provide a diagnostic endpoint that opens the workflow ru
 - **WHEN** a new user or assistant message is appended
 - **THEN** the conversation scrolls to the bottom of the newly rendered message content
 
-### Requirement: Interactive agent uses assistant context as meeting memory
-The interactive agent window instructions SHALL identify each assistant context file as meeting-specific memory.
-
-When the user asks the interactive agent to fix or investigate a meeting or summary, the instructions SHALL direct the agent to read the matching assistant context for clues about problems, missing information, assumptions, prior fixes, and conclusions.
-
-After repairing a meeting or summary, the instructions SHALL direct the agent to append a concise record of its fixes and conclusions to the matching assistant context.
-
-#### Scenario: Interactive agent repairs a meeting artifact
-- **GIVEN** a meeting has an assistant context file
-- **WHEN** the user asks the interactive agent to fix that meeting or its summary
-- **THEN** the agent instructions direct it to inspect the matching assistant context for relevant meeting-specific memory
-- **AND** direct it to append its fixes and conclusions to that assistant context

openspec/specs/meeting-summary/spec.md

@@ -140,49 +140,3 @@ The summary-agent instructions SHALL tell the agent to keep the one-line summary
 - **THEN** Meeting Assistant refuses the write
 - **AND** does not mark the summary as written
 
-### Requirement: Summary agent preserves meeting-specific working memory
-When Meeting Assistant uses the built-in summary-agent instructions, those instructions SHALL identify assistant context as persistent meeting-specific memory.
-
-The built-in instructions SHALL direct the summary agent to append unexpected problems, missing information, and assumptions encountered while summarizing to assistant context.
-
-#### Scenario: Summarizer encounters uncertainty
-- **GIVEN** the built-in summary-agent instructions are in use
-- **WHEN** the summary agent encounters an unexpected problem, cannot find needed information, or must make an assumption
-- **THEN** its instructions direct it to append a concise record to the meeting's assistant context
-- **AND** later agents can discover that record when working on the same meeting
-
-### Requirement: Summary agents tolerate Responses event streams and malformed tool arguments
-Meeting Assistant SHALL consume successful summary-agent Responses results through the supported OpenAI Responses and Agent Framework Server-Sent Events adapter.
-
-When a Responses event stream delivers completed output items separately from the final response metadata, Meeting Assistant SHALL assemble those output items into one agent response while preserving final response metadata and usage.
-
-Meeting Assistant SHALL provide an agent setting that selects streaming or non-streaming Responses transport. Streaming SHALL be enabled by default. When streaming is disabled, Meeting Assistant SHALL use the supported non-streaming OpenAI Responses client and adapter.
-
-When a returned function call contains arguments that are not a valid JSON object, Meeting Assistant SHALL NOT invoke the requested function and SHALL return an invalid-tool-arguments result associated with the original call ID to the agent so it can correct the call.
-
-When Meeting Assistant translates chat messages into a Responses request, every message input item SHALL retain the `message` item discriminator required by the OpenAI SDK and Responses API.
-
-#### Scenario: Streamed text response is assembled
-- **WHEN** the configured Responses endpoint returns completed message output in Server-Sent Events followed by final response metadata
-- **THEN** the summary agent receives the completed message text, response metadata, and usage without a JSON document parse failure
-
-#### Scenario: Streamed function call is assembled
-- **WHEN** the configured Responses endpoint returns a completed function-call output item in Server-Sent Events
-- **THEN** the summary agent receives the function call with its call ID, function name, and parsed arguments
-
-#### Scenario: Streaming transport can be disabled
-- **WHEN** `MeetingAssistant:Agent:UseStreaming` is `false`
-- **THEN** the summary agent requests a non-streaming Responses result
-- **AND** converts the response through the supported OpenAI Responses adapter
-
-#### Scenario: Chat message input remains a Responses message
-- **WHEN** the summary agent sends a user or assistant chat message through the Responses client
-- **THEN** the outbound Responses input item has type `message`
-- **AND** the request does not contain an `unknown` input item type
-
-#### Scenario: Malformed function arguments are returned to the agent
-- **WHEN** the model returns a function call whose arguments are not a valid JSON object
-- **THEN** Meeting Assistant does not invoke the requested function
-- **AND** sends a function result with an invalid-tool-arguments error for the original call ID back to the agent
-- **AND** allows the agent loop to continue
-