Commit Graph

6 Commits

  • .NET: DevUI: quarantine flaky discovery integration test (#5845) (#5846)
    TestServerWithDevUI_ResolvesMixedAgentsAndWorkflows_AllRegistrationsAsync fails intermittently in the merge_group with NRE on the discovery response, blocking PRs unrelated to DevUI from merging. Skip via Fact(Skip=...) referencing #5845 while the underlying race is investigated.
  • .NET: DevUI: add configurable access controls for the DevUI HTTP surface (#5739)
    * .NET: DevUI: add configurable access controls for the DevUI HTTP surface
    
    * .NET: DevUI: address review and fix dotnet format
    
    - Restore parameterless AddDevUI overloads for binary compatibility on
      IServiceCollection and IHostApplicationBuilder.
    - Keep /meta outside the auth-filtered group so the frontend can discover
      whether a bearer token is required before prompting for one. Surface the
      actual requirement via MetaResponse.auth_required.
    - Invoke DevUIOptions.ConfigureEndpoints before mapping protected endpoints
      so RouteGroupBuilder conventions (RequireAuthorization, rate limiting)
      reliably apply.
    - Treat a null RemoteIpAddress as non-loopback in DevUIAuthFilter; tests
      now set IPAddress.Loopback explicitly when exercising the loopback path.
    - Add a DEVUI_AUTH_TOKEN env-var fallback test and a /meta-public test.
    - Fix dotnet format: add UTF-8 BOM to new files, simplify a cref in
      DevUIOptions, and drop an unused using in the new test.
    
    * .NET: DevUI: add missing authRequired param XML tag
    
    * .NET: DevUI tests: set loopback/AllowRemoteAccess for null-RemoteIp default
    
    DevUIIntegrationTests use the default TestServer which leaves RemoteIpAddress
    null. With the new conservative loopback default those tests now hit 403; set
    AllowRemoteAccess on the option since those tests are not exercising access
    control. Also add the missing SimulateRemoteIp call in the wrong-bearer test.
    
    * .NET: DevUI tests: capture DEVUI_AUTH_TOKEN before parallel tests can see it
    
    The env-var test was leaking DEVUI_AUTH_TOKEN into parallel DevUIIntegrationTests,
    intermittently causing their requests to be rejected as 401. Eagerly resolve the
    singleton DevUIAuthFilter so its constructor captures the token, then restore the
    env var before any HTTP requests run.
  • .NET: Trim src references and add utility to enforce (#4693)
    * Trim src references and add utility to enforce
    
    * Potential fix for pull request finding
    
    Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
    
    ---------
    
    Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
  • .NET: [BREAKING] Implement Polymorphic Routing (#3792)
    * feat: Implement Polymorphic Routing
    
    * feat: Add support for Send/Yield annotations with basic Executor
    
    * Adds annotations to Declarative workflow executors
    
    * fix: Address PR Comments
    
    * Implicit filter in collection loops
    * Remove debug / usused / superfluous code
    * Fix ProtocolBuilder implicit output registrations
    * Fix logic error in ExecuteRouteGeneratorTests.ClassWithManualConfigureProtocol_DoesNotGenerate
    
    * fix: Solidify type checks and send/yield type registrations
    
    * fix: Suppress generation of TurnTokens out of AggregateTurnMessagesExecutor
    
    * Fixes an issue where ConcurrentEndExecutor is not expecting TurnTokens.
    
    * fix: Add ProtocolBuilder support for chained-delegation
    
    * Updates Declarative pacakge to rely on chained-delegation Send/Yield registration
    * Renames DeclarativeActionExectuor's new ExecuteAsync to ExecuteActionAsync to avoid colliding with Executor.ExecutoeAsync
    
    * fix: Address PR Comments
    
    * Fixes type mapping in FanInEdgeRunner
    * Fixes and expalins send/yield type registration in FunctionExecutor
    
    * fixup: build-break
    
    * fix: Add missing SendsMesage declaration to InvokeAzureAgentExecutor
  • .NET: Upgrade to .NET 10 (#2128)
    * Upgrade to .NET 10
    
    - Require .NET 10 SDK
    - Include net10.0 assets in all assemblies
    - Move net9.0-only targets to net10.0
    - Update LangVersion to latest
    - Remove complicated distinctions between debug target TFMs and release target TFMs
    - Remove unnecessary package dependencies when built into netcoreapp
    - Clean up some ifdefs
    - Clean up some analyzer warnings
    
    * Fix CI
  • .NET: Improve AIAgent and Workflow registrations for DevUI integration (#2227)
    * wip
    
    * resolve non-agent workflows as well!
    
    * add tests for devui registrations and resolving
    
    * fixes
    
    * devui for net8 as well!
    
    * simplify TFM
    
    * update tfm...
    
    * tfm rules....
    
    * wip
    
    * roll
    
    * verify entities are registered with a devui call
    
    * tests
    
    * add a proper support for non-keyed workflows
    
    * resolve default aiagent registration
    
    * sort usings :)
    
    * cleanup tests