agentic-devtools 0.2.485__py3-none-any.whl → 0.2.487__py3-none-any.whl

agentic_devtools/_bundled_skills/skills/ai-pr-loop-dispatch/SKILL.md

@@ -54,9 +54,11 @@ Review requests remain outside this workflow.
    Use `--resume` with the same review URL to reuse an existing monitor state file, skip tasks
    whose assigned comments are already complete, and continue only unfinished comments without
    creating duplicate Agent Tasks.
-4. Confirm the helper reports one output path and accepted Agent Task URL per finding. The task payload
-   must use custom agent `agdt.address-copilot-review.evaluate-and-respond`, the PR base ref, the
-   PR head ref, `create_pull_request=false`, and model `gpt-5.6-luna` unless the user overrides it.
+4. Confirm the helper reports one output path and accepted Agent Task URL per unresolved finding.
+   Resolved review threads are excluded; if every recovered finding is already resolved, the helper
+   exits successfully as a no-op without rendering or dispatching tasks. The task payload must use
+   custom agent `agdt.address-copilot-review.evaluate-and-respond`, the PR base ref, the PR head ref,
+   `create_pull_request=false`, and model `gpt-5.6-luna` unless the user overrides it.
 5. Create or reuse one unresolved review-comment thread for each suppressed finding before its
    task is dispatched. Anchor it to the PR head when a line is available, otherwise create a
    file-level review comment. Include the returned discussion URL in that task's rendered prompt.
@@ -102,8 +104,9 @@ Review requests remain outside this workflow.
    against a fresh PR snapshot. Let its normal preconditions decide whether a human re-authored
    push is needed; report a failed takeover rather than claiming completion.
 10. Return one generated `.agdt-temp/ai-pr-loop-dispatch-<PR>-<REVIEW>-comment-<N>.md` link and
-   one accepted Agent Task URL per finding. Mention the per-thread tracking replies, monitoring
-   state file, and takeover result. A reply update failure must not cause task creation to repeat.
+   one accepted Agent Task URL per unresolved finding. If all findings were resolved, report the
+   successful no-op instead. Mention the per-thread tracking replies, monitoring state file, and
+   takeover result. A reply update failure must not cause task creation to repeat.
    Sequential dispatch is the default; parallel dispatch is available only through the explicit
    `--parallel` option.
 

agentic_devtools/_bundled_skills/skills/ai-pr-loop-dispatch/render_dispatch.py

@@ -26,6 +26,7 @@ if str(_REPOSITORY_ROOT) not in sys.path:
     sys.path.insert(0, str(_REPOSITORY_ROOT))
 
 from agentic_devtools.cli.ci.evaluator.diff_heuristic import check_lines_modified  # noqa: E402
+from agentic_devtools.cli.ci.review_thread_state import fetch_review_thread_states  # noqa: E402
 
 _REVIEW_URL_PATTERN = re.compile(
     r"^https://github\.com/(?P<owner>[A-Za-z0-9_.-]+)/(?P<repo>[A-Za-z0-9_.-]+)"
@@ -511,6 +512,26 @@ def _comment_metadata(comment: Any, review_url: str) -> dict[str, Any]:
     }
 
 
+def _filter_unresolved_review_comments(*, provider: Any, pr_number: int, comments: list[Any]) -> list[Any]:
+    """Discard comments whose review thread is already resolved.
+
+    Suppressed findings use synthetic negative IDs until a thread is created, so they
+    remain eligible for dispatch. Providers without thread-state support retain the
+    previous behavior and return the recovered comments unchanged.
+    """
+    result = fetch_review_thread_states(provider, pr_number)
+    if result.degraded:
+        return comments
+    thread_states = result.states
+    return [
+        comment
+        for comment in comments
+        if not isinstance(getattr(comment, "id", None), int)
+        or comment.id < 0
+        or not thread_states.get(comment.id, (False, False))[0]
+    ]
+
+
 def _parse_follow_up_issue(task_text: str) -> dict[str, Any] | None:
     """Parse the exact structured option-four issue block emitted by the repair agent."""
     marker_index = task_text.find(_FOLLOW_UP_MARKER)
@@ -1526,6 +1547,14 @@ def main() -> int:
 
     args = _build_parser().parse_args()
     owner, repo, pr_number, review_id = _parse_review_url(args.review_url)
+    if args.post_task_link and not args.dispatch_task:
+        raise ValueError("--post-task-link requires --dispatch-task")
+    if args.monitor and not args.post_task_link:
+        raise ValueError("--monitor requires --post-task-link")
+    if args.resume and not args.post_task_link:
+        raise ValueError("--resume requires --post-task-link")
+    if args.poll_interval_seconds < 0:
+        raise ValueError("--poll-interval-seconds must be non-negative")
     review_body = _load_review_body(
         content_file=args.content_file,
         owner=owner,
@@ -1535,11 +1564,22 @@ def main() -> int:
     )
     suppressed_comments = _parse_suppressed_from_review_body(review_body, source_review_id=review_id)
     provider = GitHubActionsProvider(f"{owner}/{repo}")
+
+    def _fresh_provider() -> Any:
+        return GitHubActionsProvider(f"{owner}/{repo}")
+
     review_comments = provider.list_review_comments(pr_number, review_id)
     review_comments = _deduplicate_review_comments(review_comments, suppressed_comments)
     review_comments.sort(key=lambda comment: not (comment.is_suppressed and not comment.html_url))
     if not review_comments:
         raise RuntimeError("No inline or suppressed CCR findings were recovered from the review.")
+    review_comments = _filter_unresolved_review_comments(
+        provider=provider,
+        pr_number=pr_number,
+        comments=review_comments,
+    )
+    if not review_comments:
+        return 0
     head_sha = _load_head_sha(
         owner=owner,
         repo=repo,
@@ -1547,14 +1587,6 @@ def main() -> int:
         supplied=args.head_sha,
     )
     args.output_dir.mkdir(parents=True, exist_ok=True)
-    if args.post_task_link and not args.dispatch_task:
-        raise ValueError("--post-task-link requires --dispatch-task")
-    if args.monitor and not args.post_task_link:
-        raise ValueError("--monitor requires --post-task-link")
-    if args.resume and not args.post_task_link:
-        raise ValueError("--resume requires --post-task-link")
-    if args.poll_interval_seconds < 0:
-        raise ValueError("--poll-interval-seconds must be non-negative")
     dispatch_config: tuple[str, str, str] | None = None
     if args.dispatch_task:
         base_ref = _load_pr_ref(
@@ -1603,6 +1635,9 @@ def main() -> int:
                 marker=marker,
                 provider=provider,
             )
+            thread_states = fetch_review_thread_states(_fresh_provider(), pr_number)
+            if not thread_states.degraded and thread_states.states.get(comment_id, (False, False))[0]:
+                continue
             dispatch_comment = replace(comment, id=comment_id, html_url=html_url, is_suppressed=False)
             record.update(
                 {
@@ -1613,7 +1648,16 @@ def main() -> int:
             )
         prepared_comments.append((dispatch_comment, record))
 
-    def _render_and_dispatch(ordinal: int, comment: Any, record: dict[str, Any]) -> None:
+    def _render_and_dispatch(ordinal: int, comment: Any, record: dict[str, Any]) -> bool:
+        if dispatch_config is not None:
+            thread_states = fetch_review_thread_states(_fresh_provider(), pr_number)
+            if (
+                not thread_states.degraded
+                and isinstance(getattr(comment, "id", None), int)
+                and comment.id >= 0
+                and thread_states.states.get(comment.id, (False, False))[0]
+            ):
+                return False
         body = _build_repair_comment(
             head_sha=head_sha,
             repair_type="review",
@@ -1657,6 +1701,7 @@ def main() -> int:
                 }
             )
             task_records.append(record)
+        return True
 
     def _write_initial_state() -> Path:
         state_path = _state_path(args.output_dir, pr_number, review_id)
@@ -1676,7 +1721,8 @@ def main() -> int:
             resume_record = _find_resume_record(resume_records, prepared_record) if args.resume else None
             if resume_record is None:
                 record = prepared_record
-                _render_and_dispatch(ordinal, comment, record)
+                if not _render_and_dispatch(ordinal, comment, record):
+                    continue
             else:
                 record = resume_record
                 record["ordinal"] = ordinal
@@ -1709,7 +1755,7 @@ def main() -> int:
                         completion_records=[record],
                     )
         if args.monitor and _tasks_ready_for_takeover(task_records):
-            _run_takeover_after_monitor(owner=owner, repo=repo, pr_number=pr_number, provider=provider)
+            _run_takeover_after_monitor(owner=owner, repo=repo, pr_number=pr_number, provider=_fresh_provider())
     else:
         for ordinal, (comment, prepared_record) in enumerate(prepared_comments, start=1):
             resume_record = _find_resume_record(resume_records, prepared_record) if args.resume else None
@@ -1754,7 +1800,7 @@ def main() -> int:
                 provider=provider,
             )
             if _tasks_ready_for_takeover(task_records):
-                _run_takeover_after_monitor(owner=owner, repo=repo, pr_number=pr_number, provider=provider)
+                _run_takeover_after_monitor(owner=owner, repo=repo, pr_number=pr_number, provider=_fresh_provider())
     return 0
 
 

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.485'
-__version_tuple__ = version_tuple = (0, 2, 485)
+__version__ = version = '0.2.487'
+__version_tuple__ = version_tuple = (0, 2, 487)
 
 __commit_id__ = commit_id = None

agentic_devtools/cli/azure_devops/helpers.py

@@ -15,7 +15,7 @@ from pathlib import Path
 from typing import Any
 from urllib.parse import quote, unquote
 
-from ...state import get_state_dir, is_safe_dir_segment, read_modify_write_state
+from ...state import get_state_dir, get_value, is_safe_dir_segment, read_modify_write_state
 from ..subprocess_utils import run_safe
 from .auth import get_auth_headers, get_pat
 from .config import (
@@ -1149,6 +1149,12 @@ def resolve_review_artifact_dir_name(
                     file=sys.stderr,
                 )
             return fallback
+        configured_dir_name = get_value("review.artifact_dir_name")
+        configured_pr_id = get_value("review.artifact_dir_pr_id")
+        if isinstance(configured_dir_name, str) and str(configured_pr_id) == str(pull_request_id):
+            configured_dir_name = configured_dir_name.strip()
+            if configured_dir_name and is_safe_dir_segment(configured_dir_name):
+                return configured_dir_name
         candidate = _discover_commit_hash_short_fallback(pull_request_id, get_state_dir())
         if candidate is not None:
             resolved = candidate

agentic_devtools/cli/azure_devops/pull_request_details_commands.py

@@ -454,6 +454,8 @@ def get_pull_request_details() -> None:
     Raises:
         SystemExit: On validation or execution errors.
     """
+    from ..workflows.base import _safe_print
+
     config = AzureDevOpsConfig.from_state()
     dry_run = is_dry_run()
     pull_request_id = get_pull_request_id(required=True)
@@ -466,17 +468,17 @@ def get_pull_request_details() -> None:
     temp_dir.mkdir(parents=True, exist_ok=True)
 
     if dry_run:
-        print(f"DRY-RUN: Would retrieve pull request details for PR {pull_request_id}")
-        print(f"  Organization: {config.organization}")
-        print(f"  Project: {config.project}")
-        print(f"  Repository: {config.repository}")
-        print(f"  Output: {output_file}")
+        _safe_print(f"DRY-RUN: Would retrieve pull request details for PR {pull_request_id}")
+        _safe_print(f"  Organization: {config.organization}")
+        _safe_print(f"  Project: {config.project}")
+        _safe_print(f"  Repository: {config.repository}")
+        _safe_print(f"  Output: {output_file}")
         return
 
     pat = get_pat()
     headers = get_auth_headers(pat)
 
-    print(f"Retrieving pull request details for PR {pull_request_id}...")
+    _safe_print(f"Retrieving pull request details for PR {pull_request_id}...")
 
     # Fetch the PR object via the project-scoped Azure DevOps REST API.
     #
@@ -491,7 +493,7 @@ def get_pull_request_details() -> None:
     config = replace(config, organization=_normalize_org_url(config.organization))
     pr_data = fetch_pull_request_via_rest(pull_request_id, config, headers)
     if not pr_data:
-        print(
+        _safe_print(
             f"Error: Failed to get pull request details for PR {pull_request_id}. "
             "Verify the PAT (AZURE_DEV_OPS_COPILOT_PAT / AZURE_DEVOPS_EXT_PAT) is "
             f"valid and can access {config.organization}/{config.project}/{config.repository}.",
@@ -499,16 +501,16 @@ def get_pull_request_details() -> None:
         )
         sys.exit(1)
 
-    print("Pull request details retrieved successfully:")
-    print(f"  PR ID: {pr_data.get('pullRequestId')}")
-    print(f"  Title: {pr_data.get('title')}")
-    print(f"  Is Draft: {pr_data.get('isDraft')}")
-    print(f"  Status: {pr_data.get('status')}")
+    _safe_print("Pull request details retrieved successfully:")
+    _safe_print(f"  PR ID: {pr_data.get('pullRequestId')}")
+    _safe_print(f"  Title: {pr_data.get('title')}")
+    _safe_print(f"  Is Draft: {pr_data.get('isDraft')}")
+    _safe_print(f"  Status: {pr_data.get('status')}")
 
     if pr_data.get("autoCompleteSetBy"):
-        print(f"  Auto-Complete: Set by {pr_data['autoCompleteSetBy'].get('displayName')}")
+        _safe_print(f"  Auto-Complete: Set by {pr_data['autoCompleteSetBy'].get('displayName')}")
     else:
-        print("  Auto-Complete: Not set")
+        _safe_print("  Auto-Complete: Not set")
 
     # Extract branch info
     target_branch = normalize_ref_name(pr_data.get("targetRefName"))
@@ -592,9 +594,9 @@ def get_pull_request_details() -> None:
     if reviewer_payload and reviewer_payload.get("reviewedFiles"):  # pragma: no cover
         reviewed_count = len(reviewer_payload["reviewedFiles"])
 
-    print(f"Captured {len(files_details)} file entries for comparison.")
+    _safe_print(f"Captured {len(files_details)} file entries for comparison.")
     if reviewed_count > 0:  # pragma: no cover
-        print(f"Found {reviewed_count} files already reviewed on latest iteration.")
+        _safe_print(f"Found {reviewed_count} files already reviewed on latest iteration.")
 
     # Build output payload
     output_payload = {
@@ -611,4 +613,4 @@ def get_pull_request_details() -> None:
     with open(output_file, "w", encoding="utf-8") as f:
         json.dump(output_payload, f, indent=2)
 
-    print(f"\nResponse (with file diffs) saved to: {output_file}")
+    _safe_print(f"\nResponse (with file diffs) saved to: {output_file}")