agentic-devtools 0.2.125__tar.gz → 0.2.127__tar.gz

{agentic_devtools-0.2.125 → agentic_devtools-0.2.127}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: agentic-devtools
-Version: 0.2.125
+Version: 0.2.127
 Summary: Agentic devtools integrate Jira, DevOps & more
 Author: ayaiayorg
 License-Expression: MIT

{agentic_devtools-0.2.125 → agentic_devtools-0.2.127}/agentic_devtools/_version.py

@@ -18,7 +18,7 @@ version_tuple: tuple[int | str, ...]
 commit_id: str | None
 __commit_id__: str | None
 
-__version__ = version = '0.2.125'
-__version_tuple__ = version_tuple = (0, 2, 125)
+__version__ = version = '0.2.127'
+__version_tuple__ = version_tuple = (0, 2, 127)
 
 __commit_id__ = commit_id = None

{agentic_devtools-0.2.125 → agentic_devtools-0.2.127}/agentic_devtools/cli/azure_devops/review_commands.py

@@ -104,12 +104,43 @@ def _get_linked_pull_request_from_jira(issue_key: str) -> int | None:
     return None
 
 
+def _try_force_push_after_rebase(dry_run: bool) -> bool | None:
+    """Attempt force push after a successful rebase.
+
+    Wraps ``force_push()`` so that a push failure is non-blocking: the rebase
+    already succeeded locally and the caller can continue.
+
+    Args:
+        dry_run: If True, delegates to ``force_push(dry_run=True)`` which only
+            prints what would happen.
+
+    Returns:
+        ``True`` if the push succeeded, ``False`` if it failed, or ``None``
+        when running in dry-run mode.
+    """
+    from ..git.operations import force_push
+
+    if dry_run:
+        force_push(dry_run=True)
+        return None
+
+    try:
+        force_push(dry_run=False)
+        return True
+    except SystemExit:
+        print(
+            "Warning: Rebase succeeded but push failed. "
+            "You can manually push with: git push --force-with-lease"
+        )
+        return False
+
+
 def checkout_and_sync_branch(
     source_branch: str,
     pull_request_id: int | None = None,
     save_files_on_branch: bool = False,
     dry_run: bool = False,
-) -> tuple[bool, str | None, set[str], bool]:
+) -> tuple[bool, str | None, set[str], bool, bool | None]:
     """
     Checkout the PR source branch, sync it with origin, and rebase onto main.
 
@@ -119,6 +150,7 @@ def checkout_and_sync_branch(
        so the local copy reflects the author's latest pushed commits
     3. Fetching the latest from origin/main
     4. Rebasing onto main (continues even if conflicts, with warning)
+    5. If rebase rewrote history, force-pushing to update the remote
 
     Args:
         source_branch: The PR source branch name (without refs/heads/)
@@ -129,11 +161,13 @@ def checkout_and_sync_branch(
             based on the current HEAD.
 
     Returns:
-        Tuple of (success, error_message, files_on_branch, had_rebase_conflicts)
+        Tuple of (success, error_message, files_on_branch, had_rebase_conflicts, push_succeeded)
         - success: True if checkout succeeded and we can proceed
         - error_message: If success is False, the message to show the user
         - files_on_branch: Set of file paths changed on this branch vs main
         - had_rebase_conflicts: True if rebase conflicts were detected
+        - push_succeeded: True if auto-push succeeded, False if it failed,
+          None if no push was attempted (no rebase, or dry-run)
     """
     from ..git.operations import (
         checkout_branch,
@@ -161,12 +195,14 @@ def checkout_and_sync_branch(
                 f"{'=' * 60}",
                 set(),
                 False,
+                None,
             )
         return (  # pragma: no cover
             False,
             f"Error checking out branch: {checkout_result.message}",
             set(),
             False,
+            None,
         )
 
     # Step 1b: Fetch source branch from origin to get latest changes
@@ -176,6 +212,7 @@ def checkout_and_sync_branch(
             f"Failed to fetch origin/{source_branch}. Cannot proceed with review on potentially stale code.",
             set(),
             False,
+            None,
         )
 
     # Step 1c: Reset local branch to match origin
@@ -186,10 +223,12 @@ def checkout_and_sync_branch(
             "See the messages above for details and resolution steps.",
             set(),
             False,
+            None,
         )
 
     # Step 2: Fetch latest from main
     had_rebase_conflicts = False
+    push_succeeded: bool | None = None
     fetch_success = fetch_main(dry_run=dry_run)
     if not fetch_success:
         print("Warning: Could not fetch from origin/main, continuing without rebase...")
@@ -200,6 +239,9 @@ def checkout_and_sync_branch(
 
         if rebase_result.is_success:
             print("Branch is synced with main.")
+            # Step 3b: Auto-push if rebase rewrote history
+            if rebase_result.was_rebased:
+                push_succeeded = _try_force_push_after_rebase(dry_run)
         elif rebase_result.needs_manual_resolution:
             had_rebase_conflicts = True
             # Continue with review but warn about conflicts
@@ -232,7 +274,7 @@ def checkout_and_sync_branch(
             json.dump({"files": list(files_set)}, f, indent=2)
         print(f"Saved files on branch to: {files_on_branch_path}")
 
-    return True, None, files_set, had_rebase_conflicts
+    return True, None, files_set, had_rebase_conflicts, push_succeeded
 
 
 def _normalize_path_for_comparison(path: str) -> str:
@@ -869,7 +911,9 @@ def setup_pull_request_review() -> None:
     had_rebase_conflicts = False
     if source_branch:
         print(f"\nChecking out source branch '{source_branch}' and syncing with main...")
-        checkout_success, checkout_error, files_on_branch, had_rebase_conflicts = checkout_and_sync_branch(
+        (
+            checkout_success, checkout_error, files_on_branch, had_rebase_conflicts, _push_succeeded,
+        ) = checkout_and_sync_branch(
             source_branch, pull_request_id, save_files_on_branch=True, dry_run=is_dry_run()
         )
 

{agentic_devtools-0.2.125 → agentic_devtools-0.2.127}/agentic_devtools/cli/ci/ado_provider.py

@@ -144,7 +144,7 @@ class AzureDevOpsProvider(CIPlatformProvider):
         """Not implemented for ADO stub."""
         raise NotImplementedError("AzureDevOpsProvider.approve_pr() not yet implemented")
 
-    def merge_pr(self, pr_number: int, head_sha: str, method: str) -> None:
+    def merge_pr(self, pr_number: int, head_sha: str, method: str, *, commit_title: str | None = None) -> None:
         """Not implemented for ADO stub."""
         raise NotImplementedError("AzureDevOpsProvider.merge_pr() not yet implemented")
 

{agentic_devtools-0.2.125 → agentic_devtools-0.2.127}/agentic_devtools/cli/ci/github_provider.py

@@ -663,12 +663,19 @@ class GitHubActionsProvider(CIPlatformProvider):
             raise
 
     @retry_with_backoff()
-    def merge_pr(self, pr_number: int, head_sha: str, method: str) -> None:
-        """Merge a pull request."""
+    def merge_pr(self, pr_number: int, head_sha: str, method: str, *, commit_title: str | None = None) -> None:
+        """Merge a pull request.
+
+        ``commit_title`` maps to GitHub's ``commit_title`` field, so it only
+        controls the squash commit subject line.
+        """
+        body: dict[str, str] = {"sha": head_sha, "merge_method": method}
+        if commit_title and method == "squash":
+            body["commit_title"] = commit_title
         _gh_api(
             self._repo_api(f"/pulls/{pr_number}/merge"),
             method="PUT",
-            body={"sha": head_sha, "merge_method": method},
+            body=body,
         )
 
     @retry_with_backoff()

{agentic_devtools-0.2.125 → agentic_devtools-0.2.127}/agentic_devtools/cli/ci/orchestrator.py

@@ -484,8 +484,22 @@ def _normalize_review_body(body: str) -> str:
 def _get_copilot_review_request_skip_reason(
     pr_meta: PRMetadata,
     copilot_review: ReviewInfo | None,
+    *,
+    repair_dispatched: bool = False,
+    copilot_review_comment_count: int = 0,
 ) -> str | None:
-    """Return a reason to skip requesting Copilot review, if any."""
+    """Return a reason to skip requesting Copilot review, if any.
+
+    Args:
+        copilot_review_comment_count: Number of inline comments on the current
+            Copilot review (not a full unresolved-thread count). This guards
+            against requesting a new review when an actionable Copilot review
+            with inline comments is already present on HEAD.
+    """
+    if repair_dispatched:
+        return "repair_dispatched"
+    if copilot_review_comment_count > 0:
+        return "unresolved_comments"
     requested_reviewers = {reviewer.casefold() for reviewer in pr_meta.requested_reviewers}
     if any(login.casefold() in requested_reviewers for login in COPILOT_LOGINS):
         return "copilot_already_requested"
@@ -506,9 +520,17 @@ def _request_copilot_review_if_needed(
     copilot_review: ReviewInfo | None,
     *,
     failure_context: str,
+    repair_dispatched: bool = False,
+    copilot_review_comment_count: int = 0,
 ) -> str | None:
     """Request Copilot review unless it is already pending or already exhausted."""
-    skip_reason = _get_copilot_review_request_skip_reason(pr_meta, copilot_review)
+
+    skip_reason = _get_copilot_review_request_skip_reason(
+        pr_meta,
+        copilot_review,
+        repair_dispatched=repair_dispatched,
+        copilot_review_comment_count=copilot_review_comment_count,
+    )
     if skip_reason:
         logger.info(
             "PR #%d skipping Copilot review request (reason=%s)",
@@ -1387,6 +1409,8 @@ def run_ai_pr_loop(
             pr_meta,
             _copilot_review,
             failure_context="PR",
+            repair_dispatched=False,
+            copilot_review_comment_count=len(copilot_review_comments),
         )
         summary["decision"] = "awaiting_copilot_review_after_ci"
         if request_skip_reason is not None:
@@ -1532,6 +1556,8 @@ def run_ai_pr_loop(
             pr_meta,
             _copilot_review,
             failure_context="draft",
+            repair_dispatched=False,
+            copilot_review_comment_count=len(copilot_review_comments),
         )
         summary["decision"] = "published_awaiting_review"
         if request_skip_reason is not None:
@@ -1557,6 +1583,8 @@ def run_ai_pr_loop(
             pr_meta,
             _copilot_review,
             failure_context="PR",
+            repair_dispatched=False,
+            copilot_review_comment_count=len(copilot_review_comments),
         )
         summary["decision"] = "awaiting_copilot_review"
         if request_skip_reason is not None:

{agentic_devtools-0.2.125 → agentic_devtools-0.2.127}/agentic_devtools/cli/ci/pipeline/actions/dispatch_repair.py

@@ -195,6 +195,8 @@ class DispatchRepairAction:
             comment_id,
         )
 
+        derived.set("repair_dispatched", True)
+
         return ActionResult(
             name=self.name,
             decision=ActionDecision.EXECUTE,

{agentic_devtools-0.2.125 → agentic_devtools-0.2.127}/agentic_devtools/cli/ci/pipeline/actions/merge.py

@@ -16,6 +16,16 @@ from agentic_devtools.cli.ci.provider import CIPlatformProvider
 logger = logging.getLogger(__name__)
 
 
+def _build_squash_commit_title(snapshot: PRStateSnapshot) -> str:
+    """Build a descriptive commit title for squash merges.
+
+    Uses the PR title as the commit subject line with the PR number appended.
+    """
+    if snapshot.title:
+        return f"{snapshot.title} (#{snapshot.pr_number})"
+    return f"PR #{snapshot.pr_number}"
+
+
 def _is_review_clean(snapshot: PRStateSnapshot) -> bool:
     """Return True if the Copilot review on HEAD is clean (not actionable).
 
@@ -179,8 +189,20 @@ class MergeAction:
         derived: DerivedState,
     ) -> ActionResult:
         """Execute the merge."""
+        # Use squash merge for multi-commit PRs to maintain clean history
+        commit_count = getattr(derived, "commit_count", snapshot.commit_count)
+        if commit_count > 1:
+            method = "squash"
+            commit_title = _build_squash_commit_title(snapshot)
+        else:
+            method = "rebase"
+            commit_title = None
+
         try:
-            provider.merge_pr(snapshot.pr_number, snapshot.head_sha, "rebase")
+            if method == "squash" and commit_title:
+                provider.merge_pr(snapshot.pr_number, snapshot.head_sha, method, commit_title=commit_title)
+            else:
+                provider.merge_pr(snapshot.pr_number, snapshot.head_sha, method)
         except Exception as exc:
             logger.error("PR #%d: Merge failed: %s", snapshot.pr_number, exc)
             return ActionResult(
@@ -190,9 +212,9 @@ class MergeAction:
                 details="merge_pr call failed",
             )
 
-        logger.info("PR #%d: Merged successfully", snapshot.pr_number)
+        logger.info("PR #%d: Merged successfully (method=%s)", snapshot.pr_number, method)
         return ActionResult(
             name=self.name,
             decision=ActionDecision.EXECUTE,
-            details="PR merged via rebase",
+            details=f"PR merged via {method}",
         )

{agentic_devtools-0.2.125 → agentic_devtools-0.2.127}/agentic_devtools/cli/ci/pipeline/actions/request_review.py

@@ -19,6 +19,10 @@ class RequestReviewAction:
 
     Preconditions:
     - PR is not draft (uses DerivedState)
+    - Repair was not dispatched in this pipeline run
+    - No active Copilot coding session
+    - No unresolved review threads
+    - CI is passing
     - No effective Copilot review on HEAD
     - Copilot not already requested as reviewer
 
@@ -44,6 +48,38 @@ class RequestReviewAction:
                 details="PR is a draft",
             )
 
+        # Guard: never request review when repair was just dispatched
+        repair_dispatched = getattr(derived, "repair_dispatched", False)
+        preconditions["no_repair_dispatched"] = not repair_dispatched
+        if repair_dispatched:
+            return ActionResult(
+                name=self.name,
+                decision=ActionDecision.SKIP,
+                preconditions=preconditions,
+                details="Repair dispatched — deferring review request",
+            )
+
+        # Guard: never request review when active coding session is in progress
+        preconditions["no_active_session"] = not snapshot.active_session
+        if snapshot.active_session:
+            return ActionResult(
+                name=self.name,
+                decision=ActionDecision.SKIP,
+                preconditions=preconditions,
+                details="Copilot session active — deferring review request",
+            )
+
+        # Guard: block review request when unresolved review threads exist
+        unresolved = derived.unresolved_threads
+        preconditions["no_unresolved_threads"] = unresolved == 0
+        if unresolved > 0:
+            return ActionResult(
+                name=self.name,
+                decision=ActionDecision.SKIP,
+                preconditions=preconditions,
+                details=f"{unresolved} unresolved thread(s) — deferring review request",
+            )
+
         # CI must be passing before requesting review
         ci_passing = snapshot.ci_status == "passing"
         preconditions["ci_passing"] = ci_passing

{agentic_devtools-0.2.125 → agentic_devtools-0.2.127}/agentic_devtools/cli/ci/pipeline/actions/squash.py

@@ -16,7 +16,8 @@ class SquashAction:
 
     Preconditions:
     - Commits above merge-base > 1
-    - No active Copilot session (coding or review)
+    - No repair dispatched in this run (keeps HEAD stable for the repair cycle)
+    - No active Copilot coding session (pending review does NOT block squash)
     - CI passing
 
     Idempotency: Already 1 commit → skip.
@@ -40,25 +41,25 @@ class SquashAction:
                 details=f"Only {snapshot.commit_count} commit(s) — nothing to squash",
             )
 
-        # No active Copilot session
-        preconditions["no_active_session"] = not snapshot.active_session
-        if snapshot.active_session:
+        # Repair dispatch in this run should keep HEAD stable for the repair cycle.
+        repair_dispatched = getattr(derived, "repair_dispatched", False)
+        preconditions["no_repair_dispatched"] = not repair_dispatched
+        if repair_dispatched:
             return ActionResult(
                 name=self.name,
                 decision=ActionDecision.SKIP,
                 preconditions=preconditions,
-                details="Copilot session active — deferring squash",
+                details="Repair dispatched — deferring squash",
             )
 
-        # No pending Copilot review (review in progress = session-like)
-        pending_review = derived.copilot_review_pending
-        preconditions["no_pending_review"] = not pending_review
-        if pending_review:
+        # No active Copilot session (coding/repair only — pending review does NOT block squash)
+        preconditions["no_active_session"] = not snapshot.active_session
+        if snapshot.active_session:
             return ActionResult(
                 name=self.name,
                 decision=ActionDecision.SKIP,
                 preconditions=preconditions,
-                details="Copilot review pending — deferring squash",
+                details="Copilot session active — deferring squash",
             )
 
         # CI must be passing

{agentic_devtools-0.2.125 → agentic_devtools-0.2.127}/agentic_devtools/cli/ci/pipeline/command.py

@@ -45,6 +45,12 @@ def run_ai_pr_loop_v2(
     of 8 action evaluators. Every run evaluates all actions regardless of
     trigger type.
 
+    Pipeline ordering:
+        Guards → Publish → DispatchRepair → Squash → ResolveThreads → RequestReview → Approve → Merge
+
+    ResolveThreads runs before RequestReview so that resolved threads are reflected in
+    derived state before the review-request guard evaluates unresolved_threads.
+
     Args:
         provider: CI platform provider for API interactions.
         event_payload: Normalized event payload from the trigger.
@@ -87,10 +93,10 @@ def run_ai_pr_loop_v2(
         actions: list[Action] = [
             GuardsAction(),
             PublishAction(),
-            RequestReviewAction(),
-            ResolveThreadsAction(),
             DispatchRepairAction(),
             SquashAction(),
+            ResolveThreadsAction(),
+            RequestReviewAction(),
             ApproveAction(),
             MergeAction(),
         ]

{agentic_devtools-0.2.125 → agentic_devtools-0.2.127}/agentic_devtools/cli/ci/provider.py

@@ -123,13 +123,14 @@ class CIPlatformProvider(ABC):
         """
 
     @abstractmethod
-    def merge_pr(self, pr_number: int, head_sha: str, method: str) -> None:
+    def merge_pr(self, pr_number: int, head_sha: str, method: str, *, commit_title: str | None = None) -> None:
         """Merge a pull request.
 
         Args:
             pr_number: Pull request number.
             head_sha: Expected HEAD SHA (for safety check).
             method: Merge method (e.g., "squash", "merge", "rebase").
+            commit_title: Optional squash commit title (subject line only).
         """
 
     @abstractmethod

{agentic_devtools-0.2.125 → agentic_devtools-0.2.127}/agentic_devtools/cli/git/commands.py

@@ -269,6 +269,12 @@ def commit_cmd() -> None:
             force_push(dry_run=False)
         else:
             publish_branch(dry_run=False)
+    else:
+        # Dry-run: report the push that would occur
+        if needs_force_push:
+            force_push(dry_run=True)
+        else:
+            publish_branch(dry_run=True)
 
     # Mark checklist items if specified
     if completed_items and not dry_run:

agentic_devtools-0.2.127/specs/1587-feature-webhook-event-driven/analysis-report.md

@@ -0,0 +1,79 @@
+# Cross-Artifact Consistency & Quality Analysis Report
+
+## Findings Table
+
+| ID | Category | Severity | Location(s) | Summary | Recommendation |
+|----|----------|----------|-------------|---------|----------------|
+| F-01 | E | MEDIUM | NFR-002 | NFR-002 (15% Actions minutes cap) has no dedicated task or validation mechanism in tasks.md | Add a task to establish a baseline measurement and define how the 15% threshold will be monitored post-deployment |
+| F-02 | F | MEDIUM | tasks.md T008-T011 vs Plan Phase 4 | Tasks T008-T011 are labeled "Dry-run test" but Plan Phase 4 states "no Python unit tests needed (pure bash implementation)" — the tasks reference dry-run validation which requires T007 (DRY_RUN support) to be implemented first, yet no explicit test script or fixture is defined for T008-T011 beyond the fixture in T018 | Clarify whether T008-T011 are manual validations or scripted checks; if scripted, define the test harness |
+| F-03 | F | MEDIUM | E.2 Test Coverage Data vs tasks.md | The pre-validated coverage data maps FR-004→T024, FR-001→T030, FR-003→T031, FR-005→T032, FR-007→T033, but the FR Traceability Matrix in tasks.md maps differently (e.g., FR-001→T013,T016,T008; FR-004→T014,T012,T029). The E.2 data references task IDs that don't align with the traceability matrix | Reconcile E.2 coverage data task ID references with the actual FR Traceability Matrix; the E.2 data appears to reference a different task numbering scheme |
+| F-04 | C | MEDIUM | NFR-001 | "Prioritize recently-active PRs" — no explicit mechanism defined for how deferred PRs are tracked or guaranteed to be processed in subsequent cycles | Document that deferred PRs are naturally re-processed on the next cycle via the same `gh pr list` sorted query (no explicit tracking needed) |
+| F-05 | B | LOW | NFR-002 | "Under 30 seconds per run for typical repositories with fewer than 20 open PRs" — "typical" is subjective; the 20-PR threshold is specific but "typical" weakens it | Reword to "repositories with fewer than 20 open PRs" without "typical" qualifier |
+| F-06 | G | CRITICAL | T008, T009, T010, T011 | Four dry-run validation tasks all target the same file and dispatch logic section with highly overlapping scope — each validates a single field of the same `gh workflow run` command output | Consider consolidating into a single dry-run validation task that checks all dispatch command fields (pr_number, trigger_reason, both event types) in one pass |
+| F-07 | G | CRITICAL | T025, T013 | T025 implements structured log output in the scan step; T013 creates the scan step skeleton — both target the same bash step in the same file with overlapping code sections | Ensure T025 is clearly scoped as an enhancement to the step created in T013, not a rewrite; add explicit dependency |
+| F-08 | F | LOW | Spec "Key Entities" vs Plan | Spec defines "Event Dispatch Record" as containing "a JSON array of integer event IDs" but Plan Phase 1 task 3 says "prune the array to the last N IDs (e.g., 500)" — the spec doesn't mention pruning or size limits | Add pruning/size-limit language to the spec's Key Entities section or NFR-001 |
+| F-09 | D | LOW | Spec | No explicit "Out of Scope" section documenting what this feature intentionally does NOT cover (e.g., webhook-based real-time triggers, changes to orchestrator.py) | Add an "Out of Scope" section to the spec for clarity |
+| F-10 | A | LOW | FR-001, FR-003, FR-005 | FR-001, FR-003, and FR-005 all specify aspects of the same `gh workflow run` dispatch command — FR-001 specifies the full command, FR-003 specifies `pr_number` parameter, FR-005 specifies `trigger_reason` parameter | Acceptable decomposition for traceability but note that FR-003 and FR-005 are subsumed by FR-001's full command specification |
+
+### Category G Structured Findings
+
+[
+  {
+    "id": "F-06",
+    "overlap_type": "overlapping",
+    "severity": "CRITICAL",
+    "task_ids": ["T008", "T009", "T010", "T011"],
+    "dimensions": ["file_path", "code_section"],
+    "rationale": "All four tasks validate different fields of the same dispatch command in the same file and code section. Distinct validation outcomes make this overlapping rather than duplicate."
+  },
+  {
+    "id": "F-07",
+    "overlap_type": "overlapping",
+    "severity": "CRITICAL",
+    "task_ids": ["T025", "T013"],
+    "dimensions": ["file_path", "code_section"],
+    "rationale": "T013 creates the scan step skeleton and T025 adds structured log output within it. Both target the same file and bash step, creating overlap risk during implementation."
+  }
+]
+
+## Coverage Summary Table
+
+| Requirement Key | Has Task? | Task IDs | Notes |
+|----------------|-----------|----------|-------|
+| FR-001 | Yes | T008, T013, T016 | Core dispatch mechanism |
+| FR-002 | Yes | T006, T018, T019, T020, T021, T022 | Deduplication via cache |
+| FR-003 | Yes | T009, T016 | PR number field in dispatch |
+| FR-004 | Yes | T012, T014, T029 | Guard checks (fork, label, state) |
+| FR-005 | Yes | T010, T016 | Trigger reason field |
+| FR-006 | Yes | T023, T024, T030 | Coexistence validation |
+| FR-007 | Yes | T011, T015 | Both terminal event types |
+| NFR-001 | Yes | T004, T017 | 2-minute budget + scan cap |
+| NFR-002 | No | — | No measurement/validation task |
+| NFR-003 | Yes | T027, T028 | Per-PR error isolation |
+| NFR-004 | Yes | T025, T026 | Structured logging |
+
+## Metrics
+
+| Metric | Value |
+|--------|-------|
+| Total Requirements | 11 (7 FR + 4 NFR) |
+| Total Tasks | 34 |
+| Coverage % | 91% (10/11 requirements have tasks) |
+| Ambiguity Count | 1 |
+| Requirement Duplication Count (Category A) | 1 (partial subsumption) |
+| Critical Issues Count | 2 |
+| Task Deduplication Finding Count | 2 |
+| Task Deduplication by Type | duplicate: 0 / overlapping: 2 / conflicting: 0 |
+| Multi-Task Group Count | 1 (F-06 involves 4 tasks) |
+
+## Next Actions
+
+1. **Consolidate dry-run validation tasks (F-06)**: Merge T008–T011 into a single task that validates all dispatch command fields in one pass, reducing overlap risk.
+2. **Clarify T025/T013 dependency (F-07)**: Add an explicit dependency from T025 → T013 and scope T025 as an additive enhancement (not a rewrite) of the scan step created in T013.
+3. **Add NFR-002 validation task (F-01)**: Create a task to measure Actions minutes usage against the 15% cap defined in NFR-002.
+4. **Reconcile E.2 coverage data (F-03)**: Align E.2 test coverage task IDs with the FR Traceability Matrix in tasks.md.
+
+> Would you like me to suggest concrete remediation edits to the spec artifacts?
+
+---
+*Generated by Copilot SDK (claude-opus-4.6)*

agentic_devtools-0.2.127/specs/1587-feature-webhook-event-driven/fr-coverage.json

@@ -0,0 +1 @@
+{"covered": ["FR-001", "FR-002", "FR-003", "FR-004", "FR-005", "FR-006", "FR-007"], "uncovered": [], "total": 7, "max_retries": 2}