agentic-devtools 0.2.23__tar.gz → 0.2.25__tar.gz

{agentic_devtools-0.2.23 → agentic_devtools-0.2.25}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: agentic-devtools
-Version: 0.2.23
+Version: 0.2.25
 Summary: AI assistant helper commands for the Dragonfly platform
 Author: Dragonfly Team
 License-Expression: MIT

{agentic_devtools-0.2.23 → agentic_devtools-0.2.25}/agentic_devtools/_version.py

@@ -28,7 +28,7 @@ version_tuple: VERSION_TUPLE
 commit_id: COMMIT_ID
 __commit_id__: COMMIT_ID
 
-__version__ = version = '0.2.23'
-__version_tuple__ = version_tuple = (0, 2, 23)
+__version__ = version = '0.2.25'
+__version_tuple__ = version_tuple = (0, 2, 25)
 
 __commit_id__ = commit_id = None

{agentic_devtools-0.2.23 → agentic_devtools-0.2.25}/agentic_devtools/cli/azure_devops/review_commands.py

@@ -731,27 +731,11 @@ def setup_pull_request_review() -> None:
         target_branch = pr_info.get("targetRefName", "").replace("refs/heads/", "")
         file_count = prompts_generated
 
-        workflow_context = {
-            "pull_request_id": pull_request_id,
-            "jira_issue_key": jira_issue_key or "",
-            "pr_title": pr_title,
-            "pr_author": pr_author,
-            "source_branch": source_branch,
-            "target_branch": target_branch,
-            "file_count": file_count,
-        }
-
-        set_workflow_state(
-            name="pull-request-review",
-            status="initiated",
-            step="initiate",
-            context=workflow_context,
+        config = AzureDevOpsConfig.from_state()
+        pr_url = (
+            f"{config.organization.rstrip('/')}/{config.project}/_git/{config.repository}/pullrequest/{pull_request_id}"
         )
 
-        print("\n" + "=" * 60)
-        print("WORKFLOW INITIALIZED: pull-request-review")
-        print("=" * 60)
-
         # Load repo-specific review focus areas (optional — None if not configured)
         from ...config import load_review_focus_areas
 
@@ -773,6 +757,30 @@ def setup_pull_request_review() -> None:
 
         repo_review_focus_areas = load_review_focus_areas(repo_root)
 
+        workflow_context = {
+            "pull_request_id": pull_request_id,
+            "jira_issue_key": jira_issue_key or "",
+            "pr_title": pr_title,
+            "pr_author": pr_author,
+            "source_branch": source_branch,
+            "target_branch": target_branch,
+            "file_count": file_count,
+            "pr_url": pr_url,
+            "source_code_platform": "AzureDevOps",
+            "repo_review_focus_areas": repo_review_focus_areas or "",
+        }
+
+        set_workflow_state(
+            name="pull-request-review",
+            status="initiated",
+            step="initiate",
+            context=workflow_context,
+        )
+
+        print("\n" + "=" * 60)
+        print("WORKFLOW INITIALIZED: pull-request-review")
+        print("=" * 60)
+
         variables = {
             "pull_request_id": pull_request_id,
             "jira_issue_key": jira_issue_key or "",
@@ -782,6 +790,8 @@ def setup_pull_request_review() -> None:
             "target_branch": target_branch,
             "file_count": file_count,
             "repo_review_focus_areas": repo_review_focus_areas or "",
+            "pr_url": pr_url,
+            "source_code_platform": "AzureDevOps",
         }
 
         load_and_render_prompt(

{agentic_devtools-0.2.23 → agentic_devtools-0.2.25}/agentic_devtools/cli/azure_devops/review_jira.py

@@ -6,7 +6,7 @@ Functions for fetching Jira issues and extracting linked PRs.
 
 import os
 import re
-from typing import Dict, List, Optional, Tuple
+from typing import Dict, List, Optional, Tuple, Union
 
 import requests
 
@@ -16,6 +16,25 @@ JIRA_BASE_URL_ENV = "JIRA_BASE_URL"
 JIRA_BASE_URL_DEFAULT = "https://jira.swica.ch"
 
 
+def _get_jira_ssl_verify() -> Union[bool, str]:
+    """Get SSL verification setting for Jira API calls.
+
+    Delegates to the Jira helpers module which handles the full priority chain
+    (env vars, state, repo-committed PEM, auto-fetch).  Falls back to ``True``
+    (system CA) if the helpers module is unavailable.
+
+    Returns:
+        Path to a CA bundle file, ``False`` to skip verification, or ``True``
+        to use the system CA bundle.
+    """
+    try:
+        from ..jira.helpers import _get_ssl_verify
+
+        return _get_ssl_verify()
+    except Exception:  # noqa: BLE001
+        return True
+
+
 def get_jira_credentials() -> Tuple[Optional[str], str]:
     """
     Get Jira credentials from environment.
@@ -53,6 +72,7 @@ def fetch_jira_issue(issue_key: str, verbose: bool = False) -> Optional[Dict]:
             url,
             headers={"Authorization": f"Bearer {pat}", "Accept": "application/json"},
             timeout=30,
+            verify=_get_jira_ssl_verify(),
         )
 
         if response.status_code == 200:
@@ -92,11 +112,15 @@ def fetch_development_panel_prs(issue_key: str, verbose: bool = False) -> List[D
     # First fetch the issue to get the ID
     issue_url = f"{base_url}/rest/api/2/issue/{issue_key}?fields=id"
 
+    # Compute SSL verify once to avoid repeated filesystem checks / cert auto-fetches
+    ssl_verify = _get_jira_ssl_verify()
+
     try:
         issue_response = requests.get(
             issue_url,
             headers={"Authorization": f"Bearer {pat}", "Accept": "application/json"},
             timeout=30,
+            verify=ssl_verify,
         )
 
         if issue_response.status_code != 200:
@@ -119,6 +143,7 @@ def fetch_development_panel_prs(issue_key: str, verbose: bool = False) -> List[D
             dev_url,
             headers={"Authorization": f"Bearer {pat}", "Accept": "application/json"},
             timeout=30,
+            verify=ssl_verify,
         )
 
         if dev_response.status_code != 200:

{agentic_devtools-0.2.23 → agentic_devtools-0.2.25}/agentic_devtools/cli/copilot/session.py

@@ -83,6 +83,16 @@ _MANAGED_COPILOT = Path.home() / ".agdt" / "bin" / ("copilot.exe" if sys.platfor
 # for the command prefix and OS overhead.
 _MAX_GH_COPILOT_ARGV_LENGTH = 30_000
 
+# Safe length limit for the pre-processed inline prompt passed via argv.
+# Truncation is applied to the prompt content (before ``_build_copilot_args``
+# checks ``_MAX_GH_COPILOT_ARGV_LENGTH``) so that the final single-line
+# version produced by ``_inline_prompt`` (after ``\n`` → ``<br>`` replacement
+# and appending the backup file-reference suffix) still sits at least 100
+# characters below the hard cap.  This gap is a conservative safety margin
+# for the rest of the command line (flags, executable path, OS overhead) and
+# guards against minor miscounts in edge-case content.
+_SAFE_ARGV_LENGTH = 29_900
+
 
 # ---------------------------------------------------------------------------
 # Public dataclass
@@ -294,6 +304,70 @@ def _print_fallback_prompt(prompt: str) -> None:
     print(prompt)
 
 
+def _inline_prompt(prompt_text: str, prompt_file_path: str) -> str:
+    """Convert multi-line prompt to single-line with ``<br>`` separators and backup reference.
+
+    If the result exceeds :data:`_SAFE_ARGV_LENGTH`, the ``Repo-Specific
+    Review Focus Areas`` content is truncated first (with ``...`` appended).
+    If that is insufficient or the section is absent, a short
+    file-reference-only prompt is returned instead so the backup path is
+    always preserved.  A :func:`warnings.warn` is emitted whenever
+    truncation occurs.
+    """
+    suffix = f"   <br>   The full prompt is also saved at: {prompt_file_path}"
+    single_line = prompt_text.replace("\n", "   <br>   ") + suffix
+
+    if len(single_line) <= _SAFE_ARGV_LENGTH:
+        return single_line
+
+    # --- Truncation path: prefer trimming Repo-Specific Focus Areas ---
+    focus_marker = "## Repo-Specific Review Focus Areas"
+    next_section_prefix = "\n## "
+
+    focus_start = prompt_text.find(focus_marker)
+    if focus_start != -1:
+        content_start = prompt_text.find("\n", focus_start) + 1
+        next_section = prompt_text.find(next_section_prefix, content_start)
+        if next_section != -1:
+            before = prompt_text[:content_start]
+            after = prompt_text[next_section:]
+            # Try with focus areas fully removed first
+            stripped = before + "...\n" + after
+            stripped_line = stripped.replace("\n", "   <br>   ") + suffix
+            if len(stripped_line) <= _SAFE_ARGV_LENGTH:
+                # Partially include focus areas to use available space
+                focus_content = prompt_text[content_start:next_section]
+                overhead = len(stripped_line)
+                available = _SAFE_ARGV_LENGTH - overhead
+                if available > 0:
+                    keep = focus_content[:available]
+                    last_newline = keep.rfind("\n")
+                    if last_newline > 0:
+                        keep = keep[:last_newline]
+                    partial = before + keep + "\n...\n" + after
+                    partial_line = partial.replace("\n", "   <br>   ") + suffix
+                    if len(partial_line) <= _SAFE_ARGV_LENGTH:
+                        warnings.warn(
+                            "Prompt truncated: Repo-Specific Review Focus Areas was trimmed to fit argv limit.",
+                            stacklevel=2,
+                        )
+                        return partial_line
+                # Fall through to fully removed version
+                warnings.warn(
+                    "Prompt truncated: Repo-Specific Review Focus Areas fully removed to fit argv limit.",
+                    stacklevel=2,
+                )
+                return stripped_line
+
+    # No focus areas section or still too long — fall back to a short
+    # file-reference-only prompt so the backup path is always preserved.
+    warnings.warn(
+        "Prompt too large for inline use; falling back to file-reference-only prompt.",
+        stacklevel=2,
+    )
+    return f"Prompt too large to pass inline safely. The full prompt is also saved at: {prompt_file_path}"
+
+
 # ---------------------------------------------------------------------------
 # Public API
 # ---------------------------------------------------------------------------
@@ -310,14 +384,18 @@ def start_copilot_session(
     Behaviour:
     - Generates (or reuses) a unique session ID.
     - Writes *prompt* to a temporary file for persistence and manual
-      reuse.  In **interactive** mode the full prompt text is also passed
-      as a CLI argument to the Copilot process.  In **non-interactive**
-      mode a short file-reference instruction is passed via ``-p`` instead
-      of the raw prompt content; this avoids Windows command-line
-      reliability issues with multiline / special-character prompts and
-      ensures the agent reads the instruction file as its first action.
-      When the prompt exceeds safe argv-length limits it is printed to
-      stdout instead.
+      reuse.  The full prompt text is inlined as a single line (newlines
+      replaced with ``   <br>   ``) and passed as a CLI argument to the
+      Copilot process in both interactive and non-interactive modes.
+      A backup file reference is appended to the inlined prompt.
+    - If the inlined prompt exceeds the safe argv-length limit
+      (``_SAFE_ARGV_LENGTH``), the ``Repo-Specific Review Focus Areas``
+      section is truncated first.  If that is insufficient or absent,
+      a short file-reference-only prompt is used instead.  A warning is
+      emitted whenever truncation occurs.
+    - If ``_build_copilot_args`` still returns ``None`` (prompt exceeds
+      ``_MAX_GH_COPILOT_ARGV_LENGTH``), the full prompt is printed to
+      stdout as a fallback.
     - Starts ``copilot -i/-p <prompt>`` (standalone binary) or
       ``gh copilot suggest <prompt>`` (extension fallback).
     - In **interactive** mode the child process inherits the current
@@ -379,16 +457,10 @@ def start_copilot_session(
         return result
 
     # --- Build command -------------------------------------------------------
-    # For non-interactive mode, pass a short file-reference instruction via
-    # -p rather than the full prompt text.  The full prompt is already on
-    # disk at prompt_file; this avoids Windows command-line reliability
-    # issues with multiline / special-character content and ensures the
-    # agent reads the instruction file as its very first action.
-    argv_prompt = (
-        (f"Your task instructions are in this file: {prompt_file}. Read that file before doing anything else.")
-        if not interactive
-        else prompt
-    )
+    # Inline the full prompt as a single line (newlines replaced with <br>
+    # separators) for both interactive and non-interactive modes.  The file
+    # on disk still contains the multi-line version for manual reuse.
+    argv_prompt = _inline_prompt(prompt, prompt_file)
     args = _build_copilot_args(argv_prompt, interactive=interactive)
 
     # When the prompt is too large for safe argv passing, fall back to

{agentic_devtools-0.2.23 → agentic_devtools-0.2.25}/agentic_devtools/cli/setup/commands.py

@@ -14,8 +14,11 @@ on whatever is available on the system ``PATH``.
 
 import argparse
 import os
+import re
 import sys
 from pathlib import Path
+from typing import List, Optional
+from urllib.parse import urlparse
 
 from agentic_devtools.cli.cert_utils import ensure_ca_bundle as _ensure_ca_bundle
 
@@ -41,6 +44,78 @@ _PATH_INSTRUCTIONS = (
 )
 
 
+_SETUP_HOSTS = (
+    "api.github.com",
+    "github.com",
+    "dev.azure.com",
+    "release-assets.githubusercontent.com",
+)
+
+
+def _build_unified_ca_bundle(per_host_pem_paths: List[str]) -> Optional[Path]:
+    """Build a unified CA bundle combining certifi's system CAs and fetched corporate CAs.
+
+    Reads the system certifi CA bundle, appends all non-leaf certificates
+    (index > 0 in each chain, i.e. intermediates and roots) from the
+    per-host PEM files, de-duplicates, and writes the result to
+    ``~/.agdt/certs/unified-ca-bundle.pem``.
+
+    Args:
+        per_host_pem_paths: List of paths to per-host PEM files.
+
+    Returns:
+        Path to the unified bundle file, or ``None`` if certifi is unavailable,
+        if no additional corporate CA certificates are found in the provided
+        PEM files, or if the certifi bundle cannot be read / the unified file
+        cannot be written.
+    """
+    try:
+        import certifi
+    except ImportError:
+        return None
+
+    cert_pattern = r"-----BEGIN CERTIFICATE-----.*?-----END CERTIFICATE-----"
+
+    # Start with certifi system CAs
+    certifi_path = Path(certifi.where())
+    try:
+        system_pem = certifi_path.read_text(encoding="utf-8", errors="ignore")
+    except OSError as exc:
+        print(f"  ⚠ Could not read certifi CA bundle {certifi_path}: {exc}", file=sys.stderr)
+        return None
+    system_certs = set(re.findall(cert_pattern, system_pem, re.DOTALL))
+
+    extra_certs: List[str] = []
+    for pem_path in per_host_pem_paths:
+        try:
+            content = Path(pem_path).read_text(encoding="utf-8", errors="ignore")
+        except OSError as exc:
+            print(f"  ⚠ Could not read CA bundle {pem_path}: {exc}", file=sys.stderr)
+            continue
+        chain = re.findall(cert_pattern, content, re.DOTALL)
+        # Skip index 0 (leaf/server cert); only add intermediates and roots
+        for cert in chain[1:]:
+            if cert not in system_certs:
+                system_certs.add(cert)
+                extra_certs.append(cert)
+
+    if not extra_certs:
+        # No additional corporate CAs found — no point writing a copy of the
+        # system bundle.  Return None so the caller can skip the REQUESTS_CA_BUNDLE
+        # hint (pointing users at a file identical to certifi is misleading).
+        return None
+
+    unified_content = system_pem.rstrip("\n") + "\n" + "\n".join(extra_certs) + "\n"
+    unified_path = Path.home() / ".agdt" / "certs" / "unified-ca-bundle.pem"
+    try:
+        unified_path.parent.mkdir(parents=True, exist_ok=True)
+        unified_path.write_text(unified_content, encoding="utf-8")
+    except OSError as exc:
+        print(f"  ⚠ Could not write unified CA bundle {unified_path}: {exc}", file=sys.stderr)
+        return None
+    return unified_path
+
+
 def _prefetch_certs() -> None:
     """Pre-fetch and cache corporate CA certificates for common setup hosts.
 
@@ -49,18 +124,51 @@ def _prefetch_certs() -> None:
     ``~/.agdt/npmrc`` file that configures npm to use the cached CA bundle
     for ``registry.npmjs.org``, enabling npm installs on corporate networks.
 
+    After fetching all per-host bundles a unified CA bundle is built at
+    ``~/.agdt/certs/unified-ca-bundle.pem`` by combining the system certifi
+    CA store with any extra intermediate/root CAs found in the per-host chains.
+    This single file can be pointed to by ``REQUESTS_CA_BUNDLE`` so that
+    all HTTPS calls (GitHub, Azure DevOps, Jira, etc.) work on corporate networks.
+
     The cert cache only needs to be refreshed infrequently (e.g. yearly).
     To force a refresh, delete ``~/.agdt/certs/``.
     """
     print("Fetching CA certificates for external hosts...")
 
-    # Hosts used by the CLI installers — store api.github.com result for pip hint below
-    gh_pem = None
-    for hostname in ("api.github.com", "github.com"):
+    # Determine Jira hostname dynamically
+    extra_hosts: List[str] = []
+    try:
+        from ..jira.config import get_jira_base_url
+
+        jira_url = get_jira_base_url()
+        # Use urlparse to correctly strip port numbers (e.g. jira.example.com:8443).
+        # Scheme-less URLs like "jira.example.com" need a "//" prefix so urlparse
+        # treats the first component as a network location rather than a path.
+        parsed = urlparse(jira_url)
+        jira_hostname = parsed.hostname
+        if not jira_hostname:
+            jira_hostname = urlparse("//" + jira_url).hostname
+        if jira_hostname:
+            extra_hosts.append(jira_hostname)
+    except Exception as exc:  # noqa: BLE001
+        print(f"  ⚠ Could not determine Jira hostname (skipping Jira cert): {exc}", file=sys.stderr)
+
+    all_pem_paths: List[str] = []
+
+    # Fetch certs for fixed setup hosts
+    for hostname in _SETUP_HOSTS:
+        pem = _ensure_ca_bundle(hostname)
+        if pem:
+            all_pem_paths.append(pem)
+            print(f"  ✓ CA bundle cached for {hostname}")
+        else:
+            print(f"  ⚠ Could not cache CA bundle for {hostname}; will try system CA")
+
+    # Fetch certs for dynamically determined hosts (e.g. Jira)
+    for hostname in extra_hosts:
         pem = _ensure_ca_bundle(hostname)
-        if hostname == "api.github.com":
-            gh_pem = pem
         if pem:
+            all_pem_paths.append(pem)
             print(f"  ✓ CA bundle cached for {hostname}")
         else:
             print(f"  ⚠ Could not cache CA bundle for {hostname}; will try system CA")
@@ -68,6 +176,7 @@ def _prefetch_certs() -> None:
     # npm registry — write cafile to ~/.agdt/npmrc so npm works on corporate networks
     npm_pem = _ensure_ca_bundle("registry.npmjs.org")
     if npm_pem:
+        all_pem_paths.append(npm_pem)
         npmrc_path = Path.home() / ".agdt" / "npmrc"
         npmrc_path.parent.mkdir(parents=True, exist_ok=True)
         npmrc_path.write_text(f"cafile={npm_pem}\n", encoding="utf-8")
@@ -81,13 +190,18 @@ def _prefetch_certs() -> None:
     else:
         print("  ⚠ Could not cache CA bundle for registry.npmjs.org; will try system CA")
 
-    # Print pip CA instructions (pip respects REQUESTS_CA_BUNDLE or PIP_CERT)
-    if gh_pem:
-        print("  ℹ For pip/requests:")
+    # Build unified CA bundle combining certifi + fetched corporate CAs
+    unified_path = _build_unified_ca_bundle(all_pem_paths)
+
+    # Print pip/requests CA instructions pointing at the unified bundle
+    if unified_path:
+        unified_str = str(unified_path)
+        print("  ✓ Unified CA bundle written to ~/.agdt/certs/unified-ca-bundle.pem")
+        print("  ℹ For pip/requests/az CLI (covers GitHub, Azure DevOps, Jira, etc.):")
         print("    # bash/zsh:")
-        print(f'    export REQUESTS_CA_BUNDLE="{gh_pem}"')
+        print(f'    export REQUESTS_CA_BUNDLE="{unified_str}"')
         print("    # PowerShell:")
-        print(f'    $env:REQUESTS_CA_BUNDLE = "{gh_pem}"')
+        print(f'    $env:REQUESTS_CA_BUNDLE = "{unified_str}"')
 
 
 def _print_path_instructions_if_needed() -> None:

{agentic_devtools-0.2.23 → agentic_devtools-0.2.25}/agentic_devtools/cli/workflows/commands.py

@@ -573,7 +573,7 @@ def advance_pull_request_review_workflow(step: Optional[str] = None) -> None:
 
     Usage: agdt-advance-workflow <step>
 
-    Steps: file-review, decision, completion
+    Steps: pull-request-overview, file-review, decision, completion
 
     Args:
         step: The step to advance to (optional, auto-detects next step if not provided)
@@ -583,7 +583,7 @@ def advance_pull_request_review_workflow(step: Optional[str] = None) -> None:
 
     if not is_workflow_active("pull-request-review"):
         print("ERROR: pull-request-review workflow is not active.", file=sys.stderr)
-        print("Start it with: agdt-review-pull-request", file=sys.stderr)
+        print("Start it with: agdt-initiate-pull-request-review-workflow", file=sys.stderr)
         sys.exit(1)
 
     workflow = get_workflow_state()
@@ -610,7 +610,7 @@ def advance_pull_request_review_workflow(step: Optional[str] = None) -> None:
     # Get queue status for file-review step
     queue_status = get_queue_status(pr_id_int)
 
-    _VALID_STEPS = {"file-review", "decision", "completion"}
+    _VALID_STEPS = {"pull-request-overview", "file-review", "decision", "completion"}
 
     if step is not None and step not in _VALID_STEPS:
         print(
@@ -622,6 +622,8 @@ def advance_pull_request_review_workflow(step: Optional[str] = None) -> None:
     if step is None:
         # Auto-detect next step based on current step and queue status
         if current_step == "initiate":
+            step = "pull-request-overview"
+        elif current_step == "pull-request-overview":
             step = "file-review"
         elif current_step == "file-review":
             # If all files are complete and no submissions are still pending,
@@ -668,6 +670,10 @@ def advance_pull_request_review_workflow(step: Optional[str] = None) -> None:
         "source_branch": source_branch,
         "target_branch": target_branch,
         "file_count": file_count,
+        # PR context variables (for pull-request-overview step)
+        "pr_url": context.get("pr_url", ""),
+        "source_code_platform": context.get("source_code_platform", ""),
+        "repo_review_focus_areas": context.get("repo_review_focus_areas", ""),
         # Queue status variables (for file-review step)
         "completed_count": queue_status["completed_count"],
         "pending_count": queue_status["pending_count"],

{agentic_devtools-0.2.23 → agentic_devtools-0.2.25}/agentic_devtools/cli/workflows/worktree_setup.py

@@ -19,6 +19,18 @@ from typing import Optional, Tuple
 # Exported for dynamic invocation by run_function_in_background
 __all__ = ["_setup_worktree_from_state"]
 
+# Static single-line prompt used when starting the Copilot CLI session for PR
+# review.  It contains *only* the instruction to run the first advance command
+# so the agent is forced into the workflow system before receiving any context.
+# MUST remain a single line (no ``\n``).  Use ``---`` as visual separators.
+COPILOT_SESSION_START_PROMPT = (
+    "You are a senior software engineer reviewing a Pull Request. "
+    "--- CRITICAL: Execute this command now: `agdt-advance-workflow pull-request-overview` "
+    "--- This command will provide you with all PR details, review criteria, and instructions. "
+    "Do NOT attempt to review any code or gather any information until you have run this command. "
+    "The agentic-devtools workflow will guide you through each step."
+)
+
 
 def is_vscode_available() -> bool:
     """Check if VS Code CLI is available on PATH.
@@ -967,14 +979,15 @@ def _start_copilot_session_for_pr_review(
     interactive: bool = True,
 ) -> None:
     """
-    Read the initiate prompt, load focus areas, compose the integrated
-    prompt, and start a ``gh copilot`` session.
+    Wait for the workflow setup to complete, then start a ``gh copilot``
+    session with a static single-line prompt that instructs the agent to
+    run ``agdt-advance-workflow pull-request-overview``.
 
-    The prompt file is expected at
-    ``{worktree_path}/scripts/temp/temp-pull-request-review-initiate-prompt.md``
-    (written by ``load_and_render_prompt()`` during the PR review setup).
-    Focus areas are loaded from ``{worktree_path}/.github/agdt-config.json``
-    if present.
+    The function still waits for the rendered initiate prompt file to
+    appear on disk (proving that the background setup has finished) but
+    does **not** read its content.  Instead it uses the constant
+    :data:`COPILOT_SESSION_START_PROMPT` so that the AI agent is forced
+    into the workflow step system before receiving any PR context.
 
     In interactive mode the Copilot session inherits the terminal so the
     user can interact with it; in non-interactive mode it runs in the
@@ -987,7 +1000,6 @@ def _start_copilot_session_for_pr_review(
         worktree_path: Absolute path to the worktree root.
         interactive: Whether to start the Copilot session interactively.
     """
-    from ...config import load_review_focus_areas
     from ..copilot.session import start_copilot_session
 
     prompt_file = Path(worktree_path) / "scripts" / "temp" / "temp-pull-request-review-initiate-prompt.md"
@@ -996,20 +1008,10 @@ def _start_copilot_session_for_pr_review(
     if not _wait_for_prompt_file(prompt_file):
         print("WARNING: Initiate prompt file not found after waiting. Skipping Copilot session.")
         return
-
-    try:
-        initiate_prompt = prompt_file.read_text(encoding="utf-8")
-    except OSError as exc:
-        print(f"WARNING: Could not read initiate prompt file: {exc}")
+    if not prompt_file.is_file():
+        print("WARNING: Initiate prompt path exists but is not a regular file. Skipping Copilot session.")
         return
 
-    # Append repo-specific focus areas when available
-    focus_areas = load_review_focus_areas(worktree_path)
-    if focus_areas:
-        integrated_prompt = initiate_prompt + "\n\n" + focus_areas
-    else:
-        integrated_prompt = initiate_prompt
-
     # Non-interactive mode when VS Code is not available (pipeline scenario),
     # or when there is no TTY attached (e.g. running inside run_function_in_background
     # where stdin/stdout are redirected to DEVNULL/log files).
@@ -1030,7 +1032,7 @@ def _start_copilot_session_for_pr_review(
     os.chdir(worktree_path)
     try:
         start_copilot_session(
-            prompt=integrated_prompt,
+            prompt=COPILOT_SESSION_START_PROMPT,
             working_directory=worktree_path,
             interactive=effective_interactive,
         )

{agentic_devtools-0.2.23 → agentic_devtools-0.2.25}/agentic_devtools/prompts/pull-request-review/default-file-review-prompt.md

@@ -57,7 +57,7 @@ agdt-request-changes-with-suggestion --file-path "{{current_file}}" --summary "O
 Run `agdt-task-wait` to:
 
 - Wait for the review to post
-- Complete the file review workflow and proceed to summary
+- Complete the file review workflow and proceed to decision
 
 {% else %}
 **No wait required** - proceed directly to the next file after submitting.

agentic_devtools-0.2.25/agentic_devtools/prompts/pull-request-review/default-initiate-prompt.md

@@ -0,0 +1,17 @@
+# Pull Request Review Workflow - Initiate
+
+The pull-request-review workflow has been initialized for PR **#{{pull_request_id}}**.
+
+**⚠️ CRITICAL: Run the following command now to view PR details and review criteria:**
+
+```bash
+agdt-advance-workflow pull-request-overview
+```
+
+Do NOT attempt to review any code or gather any information until you have run this command.
+The agentic-devtools workflow will guide you through each step.
+
+## Instructions File Reference
+
+The full prompt for this workflow step is saved at:
+scripts/temp/temp-pull-request-review-initiate-prompt.md