move recheck to post-all-actions rather than per-action

[scpubgit/DX.git] / lib / DX / Hypothesis.pm

diff --git a/lib/DX/Hypothesis.pm b/lib/DX/Hypothesis.pm
index 61993c5..0386cd0 100644 (file)
--- a/lib/DX/Hypothesis.pm
+++ b/lib/DX/Hypothesis.pm
@@ -1,26 +1,37 @@
 package DX::Hypothesis;
 
+use DX::ActionPolicy::LockScope;
+use Types::Standard qw(ArrayRef);
+use DX::Utils qw(deparse);
 use DX::Class;
 
-has scope => (is => 'ro', required => 1);
+has scope => (is => 'ro', isa => Scope, required => 1);
 
-has resolved_propositions => (is => 'ro', required => 1);
+has resolved_propositions => (
+  is => 'ro', isa => ResolvedPropositionSet, required => 1
+);
 
-has outstanding_propositions => (is => 'ro', required => 1);
+has actions => (
+  is => 'ro', isa => ArrayRef[Action], required => 1
+);
 
-has actions => (is => 'ro', required => 1);
+has action_applications => (
+  is => 'ro', isa => ArrayRef[Action], required => 1
+);
 
-sub head_proposition { shift->outstanding_propositions->[0] }
+has action_policy => (is => 'ro', isa => ActionPolicy, required => 1);
 
 sub with_actions {
   my ($self, @actions) = @_;
   my $hyp = $self;
+  my @events;
   foreach my $act (@actions) {
-    ($hyp, my @events) = $act->dry_run($hyp);
-    return undef unless $hyp;
-    $hyp = $hyp->but_recheck_for(@events);
+    return undef unless $self->action_policy->allows($act);
+    ($hyp, my @these_events) = $act->dry_run($hyp);
     return undef unless $hyp;
+    push @events, @these_events;
   }
+  $hyp = $hyp->but_recheck_for(@events);
   return $hyp;
 }
 
@@ -28,21 +39,62 @@ sub but_recheck_for {
   my ($self, @events) = @_;
   my ($still_resolved, @recheck) = $self->resolved_propositions
                                         ->but_expire_for(@events);
-  my $hyp = $self->but(resolved_propositions => $still_resolved);
-  $hyp = $_->but_recheck_against($hyp) or return undef for @recheck;
-  return $hyp;
+  return $self unless @recheck;
+
+  my $ap = DX::ActionPolicy::LockScope->new(
+    lock_to_depth => $self->scope->depth,
+    next_policy => $self->action_policy,
+  );
+
+  # we should probably be doing something about pruning the scope
+  # but that's completely pointless until we have rules
+
+  my $hyp = ref($self)->new(
+    scope => $self->scope,
+    resolved_propositions => DX::ResolvedPropositionSet->new_empty,
+    actions => [],
+    action_applications => [],
+    action_policy => $ap,
+  );
+
+  my $pseq = DX::PropositionSequence->new(
+    members => \@recheck,
+    external_names => {},
+    internal_names => {},
+  );
+
+  trace 'step.recheck.hyp' => $hyp;
+
+  my $ss = DX::SearchProcess->new_for($hyp, $pseq);
+
+  my $sol_ss = $ss->find_solution;
+
+  unless ($sol_ss) {
+    trace 'step.recheck.fail' => 'argh';
+    return undef;
+  }
+
+  my $sol_rps = $sol_ss->current_hypothesis->resolved_propositions;
+
+  my $rps = $still_resolved;
+
+  $rps = $rps->with_updated_dependencies_for(
+    $_, $sol_rps->dependencies_for($_)
+  ) for @recheck;
+
+  trace 'step.recheck.done' => 'yay';
+
+  return $self->but(resolved_propositions => $rps);
 }
 
-sub resolve_head_dependent_on {
-  my ($self, $depends) = @_;
-  my ($first, @rest) = @{$self->outstanding_propositions};
+sub with_resolution {
+  my ($self, $prop, $depends) = @_;
   $self->but(
     resolved_propositions => $self->resolved_propositions
                                   ->with_resolution_for(
-                                      $first,
+                                      $prop,
                                       $depends,
                                     ),
-    outstanding_propositions => \@rest,
   );
 }