Include lifecycle candidates in active cleanup evidence (#669)

* Add stale active worktree cleanup policy

* Revert "Add stale active worktree cleanup policy"

This reverts commit 09b682a.

* Include lifecycle candidates in active cleanup evidence

Author: chubes4

inc/Workspace/WorkspaceActiveNoSignalCleanup.php

@@ -47,7 +47,7 @@ public function worktree_active_no_signal_report( array $opts = array() ): array
 		$active = array_values(
 			array_filter(
 				(array) ( $inventory['skipped'] ?? array() ),
-				fn( $row ) => is_array($row) && in_array( (string) ( $row['reason_code'] ?? '' ), array( 'active_no_signal', 'no_inventory_cleanup_signal' ), true)
+				fn( $row ) => is_array($row) && in_array( (string) ( $row['reason_code'] ?? '' ), array( 'active_no_signal', 'no_inventory_cleanup_signal', 'lifecycle_reconciliation_candidate' ), true)
 			)
 		);
 		$total  = count($active);
@@ -140,7 +140,7 @@ public function worktree_active_no_signal_report( array $opts = array() ): array
 			'summary'      => array_merge($summary, array( 'slow_rows' => $this->summarize_slow_worktree_rows($rows) )),
 			'pagination'   => $pagination,
 			'evidence'     => array(
-				'scope'       => 'review-only active_no_signal worktree lifecycle evidence',
+				'scope'       => 'review-only active/no-signal and lifecycle reconciliation worktree evidence',
 				'safety'      => 'No worktrees or remote branches are deleted. Dirty and unpushed probes are evidence only.',
 				'budget'      => null === $budget_context ? null : $this->summarize_worktree_loop_budget_context($budget_context, $budget_stopped),
 				'probe_cache' => $probe_cache['stats'],
@@ -1121,6 +1121,7 @@ private function build_active_no_signal_evidence_row( array $row, array &$github
 			'branch_identity'         => $branch_identity,
 			'path'                    => $path,
 			'created_at'              => $row['created_at'] ?? null,
+			'inventory_reason_code'   => $row['reason_code'] ?? null,
 			'lifecycle_state'         => $metadata['lifecycle_state'] ?? null,
 			'metadata'                => $metadata,
 			'last_seen_at'            => $metadata['last_seen_at'] ?? ( $metadata['observed_at'] ?? null ),

tests/smoke-worktree-metadata-reconcile.php

@@ -558,6 +558,8 @@ function () use ( $tmp ) {
         $active_rows[ $row['handle'] ?? '' ] = $row;
     }
     $assert('finalized_pr_reconcile', $active_rows['demo@head-merged']['suggested_action'] ?? '', 'active/no-signal report finds merged PRs by branch head');
+    $assert('finalized_pr_reconcile', $active_rows['demo@pr-merged']['suggested_action'] ?? '', 'active/no-signal report includes lifecycle reconciliation candidates with merged PR evidence');
+    $assert('lifecycle_reconciliation_candidate', $active_rows['demo@pr-merged']['inventory_reason_code'] ?? '', 'active/no-signal report preserves lifecycle reconciliation source reason');
     $assert('active_open_pr', $active_rows['demo@already-current']['suggested_action'] ?? '', 'active/no-signal report preserves open PRs as active');
     $assert(106, (int) ( $active_rows['demo@head-merged']['pr']['number'] ?? 0 ), 'active/no-signal report includes PR evidence');
     $assert('unsafe_dirty_or_unpushed', $active_rows['demo@dirty-active']['suggested_action'] ?? '', 'active/no-signal report keeps dirty rows unsafe');
@@ -702,7 +704,7 @@ public function worktree_list( ?string $repo = null, ?string $state = null, arra
     $run(sprintf('git remote set-url origin %s', escapeshellarg($remote)), $primary);
     $assert(true, ! is_wp_error($finalized_dry_run) && ( $finalized_dry_run['success'] ?? false ), 'finalized active/no-signal dry-run succeeds');
     $assert(true, (bool) ( $finalized_dry_run['dry_run'] ?? false ), 'finalized active/no-signal dry-run does not write');
-    $assert(1, (int) ( $finalized_dry_run['summary']['planned'] ?? 0 ), 'finalized active/no-signal dry-run plans merged PR rows only');
+    $assert(2, (int) ( $finalized_dry_run['summary']['planned'] ?? 0 ), 'finalized active/no-signal dry-run plans merged PR rows only');
     $assert('', \DataMachineCode\Workspace\WorktreeContextInjector::get_metadata('demo@head-merged')['cleanup_eligible_at'] ?? '', 'finalized active/no-signal dry-run leaves metadata unchanged');
     $run('git remote set-url origin https://github.com/acme/demo.git', $primary);
     $budgeted_finalized_dry_run = $ws->worktree_active_no_signal_finalized_apply(array( 'dry_run' => true, 'limit' => 20, 'offset' => 0, 'internal_budget_label' => '1s', 'internal_budget_seconds' => 1, 'internal_budget_started' => microtime(true) - 1 ));