[dotnet-code] Cover service-managed compaction skip

[microsoft-agent-framework-for-go]

Summary

Adds a focused test for the existing compaction.NewContextProvider contract that skips compaction when the session is service-managed. This protects the Go behavior that avoids locally rewriting history owned by the provider service, using the remote-session branch of the pinned .NET compaction coverage as evidence. Leaving this uncovered makes future compaction ports more likely to conflate local/no-session compaction with service-owned history handling.

.NET Reference

• Upstream pinned commit: 301a43c862be8dbeb1754f60a0c538b1b6dc92cb
• Relevant upstream change: 5343347312795ef589c8d8047326eb1b6ef3aa3c / .NET: Allow compaction for per-service-call local chat history agent-framework#8845, .NET: Allow compaction for per-service-call local chat history
• Upstream source: https://github.com/microsoft/agent-framework/blob/301a43c862be8dbeb1754f60a0c538b1b6dc92cb/dotnet/src/Microsoft.Agents.AI/Compaction/CompactionProvider.cs
• Upstream test: https://github.com/microsoft/agent-framework/blob/301a43c862be8dbeb1754f60a0c538b1b6dc92cb/dotnet/tests/Microsoft.Agents.AI.UnitTests/Compaction/CompactionProviderTests.cs
• Upstream scenario inspected: InvokingAsyncCompactsOnlyLocallyManagedHistoryAsync verifies remote conversation IDs are not compacted while locally managed/no-ID sessions are eligible.
• Go implementation inspected: agent/compaction/provider.go (NewContextProvider / contextProvider.Invoking) already skips non-empty Session.ServiceID(); agent/history.go and agent/agent.go define the per-service-call local-history lifecycle.
• Existing coverage inspected: agent/compaction/compaction_test.go, agent/compaction/historyprovider_test.go, and related merged compaction work in [dotnet-port-api] Add compaction-backed history provider #1092 and Keep generated compaction summaries out of chat history #1185.

Public API and Behavior

No public Go API changed. No intentional behavior change was made. The production code is unchanged; the new test asserts the existing service-managed-session skip behavior.

Tests

• Added TestNewProvider_SkipsServiceManagedSession, which sets a remote service ID on a session, invokes the compaction context provider, and verifies the strategy is not called and the original messages are returned.
• go test ./agent/compaction passed.
• git diff --check passed.

Notes

History checked included recent [dotnet-code], [dotnet-port-fixes], and [dotnet-port-api] PRs, with related compaction work in #1092 and #1185. Skipped candidates included the OpenAI in-memory Responses input-item range change because no existing Go counterpart was found, and HTTP header delimiter validation because any applicable change would be a behavior/security fix for the port-fixes workflow, not this no-behavior-change audit.

Warning

Firewall blocked 3 domains

The following domains were blocked by the firewall during workflow execution:

• proxy.golang.org
• storage.googleapis.com
• sum.golang.org

To allow these domains, add them to the network.allowed list in your workflow frontmatter:

network:
  allowed:
    - defaults
    - "proxy.golang.org"
    - "storage.googleapis.com"
    - "sum.golang.org"

See Network Configuration for more information.

Generated by .NET-to-Go Weekly Portability Audit · copilot · gpt55 · 131.8 AIC · ⌖ 14.6 AIC · ⊞ 23.6K · ◷

[Copilot]

Copilot review overview

🟢 Approval recommended

The focused test accurately covers the existing remote-session behavior.

Review effort: Balanced
Findings: None

What changed in this PR

Adds regression coverage ensuring compaction is bypassed for service-managed sessions.

Changes:

• Verifies the compaction strategy is not invoked.
• Confirms original messages are returned unchanged.

File	Description
agent/​compaction/​compaction_test.go	Adds service-managed session skip coverage.

---

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