Skip to content

Unify reviewed-safe shell path evaluation - #1961

Merged
Aaronontheweb merged 1 commit into
devfrom
refactor/simplify-shell-path-projection-3
Aug 14, 2026
Merged

Unify reviewed-safe shell path evaluation#1961
Aaronontheweb merged 1 commit into
devfrom
refactor/simplify-shell-path-projection-3

Conversation

@Aaronontheweb

Copy link
Copy Markdown
Collaborator

Summary

  • route the public compatibility path through the same typed candidate path facts used by the coordinator
  • remove the obsolete aggregate reviewed-safe implementation and duplicate candidate/path containers
  • preserve the declared-shell typed route and the bounded host-style compatibility behavior
  • record the cumulative reduction evidence while leaving the complete reduction gate open

Reduction

  • 57 fewer production lines
  • 4 fewer control-flow lines
  • no public or durable-contract change

Validation

  • dotnet test src/Netclaw.Actors.Tests/Netclaw.Actors.Tests.csproj -c Release --no-restore: 3,439 passed, 1 expected Windows-only skip
  • strict OpenSpec validation for simplify-shell-policy-evaluator and structure-shell-approval-policy
  • copyright headers
  • git diff --check
  • changed-file Slopwatch: 0 issues
  • stable patch identity preserved across the Close shell policy coverage states #1960 squash rebase

@Aaronontheweb
Aaronontheweb enabled auto-merge (squash) August 14, 2026 23:13
@Aaronontheweb
Aaronontheweb merged commit 0f1347d into dev Aug 14, 2026
23 checks passed
@Aaronontheweb
Aaronontheweb deleted the refactor/simplify-shell-path-projection-3 branch August 14, 2026 23:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant