Skip to content

Consolidate shell policy state - #1962

Merged
Aaronontheweb merged 1 commit into
devfrom
refactor/simplify-shell-state-4
Aug 15, 2026
Merged

Consolidate shell policy state#1962
Aaronontheweb merged 1 commit into
devfrom
refactor/simplify-shell-state-4

Conversation

@Aaronontheweb

Copy link
Copy Markdown
Collaborator

Summary

  • Remove duplicate candidate identity and causal scope from path facts.
  • Reuse one empty actor result and preserve immutable evidence elements.
  • Keep the real candidate scope separate from the occurrence resolution base.
  • Remove 35 production lines with no control-flow increase.

Validation

  • Netclaw.Actors.Tests: 3,439 passed; one expected Windows-only test skipped.
  • Both active shell-policy OpenSpec changes pass strict validation.
  • Copyright headers and git diff --check pass.
  • The stable patch identity stayed unchanged after PR Unify reviewed-safe shell path evaluation #1961 merged.

@Aaronontheweb
Aaronontheweb enabled auto-merge (squash) August 14, 2026 23:45

Assert.Equal("/work/sub", facts.RealScope.Path?.Value);
Assert.Equal("/work", facts.Real?.ResolutionBase.Path?.Value);
Assert.Equal("/work", facts.Real.ResolutionBase.Path?.Value);

Assert.Equal("/tmp", facts.IntentScope?.Path?.Value);
Assert.Contains(facts.FallbackScopes, scope => scope.Path?.Value == "/work");
Assert.Equal("/tmp", facts.Intent?.ResolutionBase.Path?.Value);
Assert.Equal("/tmp", facts.IntentScope?.Path?.Value);
Assert.Contains(facts.FallbackScopes, scope => scope.Path?.Value == "/work");
Assert.Equal("/tmp", facts.Intent?.ResolutionBase.Path?.Value);
Assert.Contains(facts.Fallbacks, view => view.ResolutionBase.Path?.Value == "/work");
@Aaronontheweb
Aaronontheweb merged commit db5a6df into dev Aug 15, 2026
23 checks passed
@Aaronontheweb
Aaronontheweb deleted the refactor/simplify-shell-state-4 branch August 15, 2026 00:13
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