Skip to content

S2: Owen.CSharp.Rewriter — deterministic acquire rewrite (postimage + report) - #288

Merged
PhysShell merged 6 commits into
mainfrom
claude/s2-rewriter
Jul 17, 2026
Merged

S2: Owen.CSharp.Rewriter — deterministic acquire rewrite (postimage + report)#288
PhysShell merged 6 commits into
mainfrom
claude/s2-rewriter

fix(S2): platform-aware paths, git-safe patch names, claimed staging,…

6e74a26
Select commit
Loading
Failed to load commit list.
GitHub Advanced Security / Owen completed Jul 17, 2026 in 3s

1 configuration not found

Warning: Code scanning may not have found all the alerts introduced by this pull request, because 1 configuration present on refs/heads/main was not found:

Actions workflow (action-marketplace-readiness.yml)

  • ❓  action-marketplace-consumer-demo

New alerts in code changed by this pull request

  • 5 warnings

See annotations below for details.

View all branch alerts.

Annotations

Check warning on line 23 in frontend/roslyn/samples/FixRewriteSample.cs

See this annotation in the file changed.

Code scanning / Owen

owned resource not released on all paths (possible leak) Warning

event 'pub.PropertyChanged' is subscribed (handler 'new PropertyChangedEventHandler(OnChanged)') but never unsubscribed; its source is an injected dependency whose lifetime is unknown, so it may outlive and keep 'ExplicitDelegate' alive (possible leak) [resource: subscription token]

Check warning on line 34 in frontend/roslyn/samples/FixRewriteSample.cs

See this annotation in the file changed.

Code scanning / Owen

owned resource not released on all paths (possible leak) Warning

event 'pub.PropertyChanged' is subscribed (handler 'this.OnChanged') but never unsubscribed; its source is an injected dependency whose lifetime is unknown, so it may outlive and keep 'ThisQualified' alive (possible leak) [resource: subscription token]

Check warning on line 45 in frontend/roslyn/samples/FixRewriteSample.cs

See this annotation in the file changed.

Code scanning / Owen

owned resource not released on all paths (possible leak) Warning

event 'pub.PropertyChanged' is subscribed (handler 'OnChanged') but never unsubscribed; its source is an injected dependency whose lifetime is unknown, so it may outlive and keep 'GenericHolder' alive (possible leak) [resource: subscription token]

Check warning on line 58 in frontend/roslyn/samples/FixRewriteSample.cs

See this annotation in the file changed.

Code scanning / Owen

owned resource not released on all paths (possible leak) Warning

event 'pub.PropertyChanged' is subscribed (handler 'OnChanged') but never unsubscribed; its source is an injected dependency whose lifetime is unknown, so it may outlive and keep 'Outer' alive (possible leak) [resource: subscription token]

Check warning on line 58 in frontend/roslyn/samples/FixRewriteSample.cs

See this annotation in the file changed.

Code scanning / Owen

owned resource not released on all paths (possible leak) Warning

event 'pub.PropertyChanged' is subscribed (handler 'OnChanged') but never unsubscribed; its source is an injected dependency whose lifetime is unknown, so it may outlive and keep 'Inner' alive (possible leak) [resource: subscription token]