agentic-devtools 0.2.137__tar.gz → 0.2.139__tar.gz

{agentic_devtools-0.2.137 → agentic_devtools-0.2.139}/PKG-INFO

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

{agentic_devtools-0.2.137 → agentic_devtools-0.2.139}/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.137'
-__version_tuple__ = version_tuple = (0, 2, 137)
+__version__ = version = '0.2.139'
+__version_tuple__ = version_tuple = (0, 2, 139)
 
 __commit_id__ = commit_id = None

{agentic_devtools-0.2.137 → agentic_devtools-0.2.139}/agentic_devtools/cli/azure_devops/review_scaffold.py

@@ -37,7 +37,7 @@ from .review_state import (
     normalize_file_path,
     save_review_state,
 )
-from .review_templates import render_file_summary, render_overall_summary
+from .review_templates import render_file_summary, render_overall_summary, rewrite_header_for_subsequent
 from .suggestion_verification import (
     categorize_all_suggestions,
     fetch_threads_lookup,
@@ -283,12 +283,16 @@ def _demote_main_comment(
     thread_id: int,
     comment_id: int,
     new_main_content: str,
+    commit_hash: str | None = None,
+    commit_url: str | None = None,
 ) -> int:
     """Read current main comment, post it as reply, PATCH main with new content.
 
     Steps:
       1. GET the thread to read current main comment content.
       2. POST current content as a reply (preserving it as history).
+         The reply header is rewritten from ``## ... Summary`` to
+         ``### Commit: ...`` using ``rewrite_header_for_subsequent()``.
       3. PATCH the main comment with ``new_main_content``.
 
     Args:
@@ -298,6 +302,9 @@ def _demote_main_comment(
         thread_id: Thread whose main comment is being demoted.
         comment_id: The main comment ID (usually 1).
         new_main_content: New content for the main comment.
+        commit_hash: Commit hash for the reply header. When provided,
+            the demoted reply uses a compact commit-scoped heading.
+        commit_url: Commit URL for the reply header link.
 
     Returns:
         The comment ID of the newly-created reply (the demoted content).
@@ -310,8 +317,9 @@ def _demote_main_comment(
             old_content = comment.get("content", "")
             break
 
-    # Step 2: Post old content as a reply
-    reply_id = _post_reply(requests_module, headers, threads_url, thread_id, old_content)
+    # Step 2: Post old content as a reply (with header rewritten for subsequent format)
+    reply_content = rewrite_header_for_subsequent(old_content, commit_hash, commit_url)
+    reply_id = _post_reply(requests_module, headers, threads_url, thread_id, reply_content)
 
     # Step 3: PATCH main comment with new content
     _patch_comment_content(requests_module, headers, threads_url, thread_id, comment_id, new_main_content)
@@ -1673,6 +1681,12 @@ def _incremental_rescaffold(
         f"{n_new} new, {n_mod} modified, {n_del} deleted, {n_unch} unchanged files."
     )
 
+    commit_url_pr = (
+        build_commit_pr_url(config.organization, config.project, repo_name, pull_request_id, latest_iteration_id)
+        if commit_hash and latest_iteration_id
+        else None
+    )
+
     # -------------------------------------------------------------------
     # Suggestion verification gate
     # -------------------------------------------------------------------
@@ -1719,6 +1733,8 @@ def _incremental_rescaffold(
                                 overall.threadId,
                                 overall.commentId,
                                 abort_summary,
+                                commit_hash=commit_hash,
+                                commit_url=commit_url_pr,
                             )
                         except Exception as exc:
                             print(f"Warning: Could not post abort summary: {exc}", file=sys.stderr)
@@ -1788,11 +1804,6 @@ def _incremental_rescaffold(
         return None
 
     # Process new files
-    commit_url_pr = (
-        build_commit_pr_url(config.organization, config.project, repo_name, pull_request_id, latest_iteration_id)
-        if commit_hash and latest_iteration_id
-        else None
-    )
     for file_path in changes.new_files:
         folder = _get_folder_for_path(file_path)
         file_name = _get_file_name(file_path)
@@ -1868,6 +1879,8 @@ def _incremental_rescaffold(
                         commit_hash=commit_hash,
                         commit_url=commit_url_file,
                     ),
+                    commit_hash=commit_hash,
+                    commit_url=commit_url_file,
                 )
             except Exception as exc:
                 print(f"Warning: Could not demote comment for {file_path}: {exc}", file=sys.stderr)
@@ -1894,6 +1907,7 @@ def _incremental_rescaffold(
                     fe.threadId,
                     fe.commentId,
                     removed_msg,
+                    commit_hash=commit_hash,
                 )
             except Exception as exc:
                 print(f"Warning: Could not demote comment for deleted {file_path}: {exc}", file=sys.stderr)
@@ -1934,6 +1948,8 @@ def _incremental_rescaffold(
                     overall.threadId,
                     overall.commentId,
                     new_summary,
+                    commit_hash=commit_hash,
+                    commit_url=commit_url_pr,
                 )
             except Exception as exc:
                 print(f"Warning: Could not update overall summary: {exc}", file=sys.stderr)
@@ -1952,6 +1968,8 @@ def _incremental_rescaffold(
                     overall.threadId,
                     overall.commentId,
                     new_summary,
+                    commit_hash=commit_hash,
+                    commit_url=commit_url_pr,
                 )
             except Exception as exc:
                 print(f"Warning: Could not update overall summary: {exc}", file=sys.stderr)

{agentic_devtools-0.2.137 → agentic_devtools-0.2.139}/agentic_devtools/cli/azure_devops/review_templates.py

@@ -4,7 +4,9 @@ Provides functions to generate and regenerate full markdown content for
 file summaries and the overall PR summary at each status.
 """
 
-from .review_attribution import format_status, render_attribution_line
+import re
+
+from .review_attribution import SHORT_HASH_LENGTH, format_status, render_attribution_line
 from .review_state import (
     ConsolidationStatus,
     FileEntry,
@@ -132,6 +134,7 @@ def render_file_summary(
     commit_hash: str | None = None,
     commit_url: str | None = None,
     boss_model: str | None = None,
+    is_subsequent: bool = False,
 ) -> str:
     """Render a file review summary in markdown format.
 
@@ -148,6 +151,8 @@ def render_file_summary(
             attribution line link.
         boss_model: Boss/consolidator model name. Passed through to
             ``render_model_review_progress_table()`` for consolidation attribution.
+        is_subsequent: When True, emit a compact ``### Commit:`` header instead
+            of the full ``## File Review Summary:`` title. Used for reply comments.
 
     Returns:
         Markdown string for the file review summary.
@@ -155,8 +160,13 @@ def render_file_summary(
     status = file_entry.status
     status_display = format_status(status, use_emoji=True)
 
+    if is_subsequent:
+        header = _build_subsequent_header(commit_hash, commit_url)
+    else:
+        header = f"## File Review Summary: {file_entry.fileName}"
+
     lines: list[str] = [
-        f"## File Review Summary: {file_entry.fileName}",
+        header,
         "",
     ]
 
@@ -223,6 +233,7 @@ def render_overall_summary(
     model_icon: str | None = None,
     commit_hash: str | None = None,
     commit_url: str | None = None,
+    is_subsequent: bool = False,
 ) -> str:
     """Render the overall PR review summary in markdown format.
 
@@ -240,6 +251,9 @@ def render_overall_summary(
             ``model_name``, an attribution line is prepended.
         commit_url: URL to the PR files tab at the reviewed commit. Used in the
             attribution line link.
+        is_subsequent: When True, emit a compact ``### Commit:`` header instead
+            of the full ``## Overall PR Review Summary`` title. Used for reply
+            comments.
 
     Returns:
         Markdown string for the overall PR review summary.
@@ -270,8 +284,13 @@ def render_overall_summary(
         use_emoji=True,
     )
 
+    if is_subsequent:
+        header = _build_subsequent_header(commit_hash, commit_url)
+    else:
+        header = "## Overall PR Review Summary"
+
     lines: list[str] = [
-        "## Overall PR Review Summary",
+        header,
         "",
     ]
 
@@ -334,3 +353,115 @@ def render_overall_summary(
     lines.append(narrative if narrative else "Awaiting review...")
 
     return "\n".join(lines)
+
+
+# ---------------------------------------------------------------------------
+# Subsequent-comment header utilities
+# ---------------------------------------------------------------------------
+
+# Regex matching the top-level summary heading lines that should be rewritten
+_SUMMARY_HEADING_RE = re.compile(r"^## (?:File Review Summary: .+|Overall PR Review Summary)\s*$")
+
+
+def _build_subsequent_header(commit_hash: str | None, commit_url: str | None) -> str:
+    """Build the compact ``### Commit:`` header for subsequent/reply comments.
+
+    Fallback chain (FR-008):
+      - Both hash and URL → ``### Commit: [<short_hash>](<commit_url>)``
+      - Hash only (no URL) → ``### Commit: <short_hash>``
+      - No hash → ``### Commit: unknown``
+    """
+    if commit_hash and commit_url:
+        short_hash = commit_hash[:SHORT_HASH_LENGTH]
+        return f"### Commit: [{short_hash}]({commit_url})"
+    elif commit_hash:
+        short_hash = commit_hash[:SHORT_HASH_LENGTH]
+        return f"### Commit: {short_hash}"
+    else:
+        return "### Commit: unknown"
+
+
+def rewrite_header_for_subsequent(
+    content: str,
+    commit_hash: str | None,
+    commit_url: str | None,
+) -> str:
+    """Rewrite the top heading of previously rendered summary content for use as a reply.
+
+    Replaces the first line matching ``## File Review Summary: ...`` or
+    ``## Overall PR Review Summary`` with the compact ``### Commit:`` header.
+    If the first line does not match a known summary heading, the content is
+    returned unchanged.
+
+    Args:
+        content: Previously rendered summary markdown content.
+        commit_hash: Full commit hash (first 7 chars used for display).
+        commit_url: URL to the commit. When ``None``, falls back per FR-008.
+
+    Returns:
+        Content with the heading replaced (or unchanged if no match).
+    """
+    if not content:
+        return content
+
+    lines = content.split("\n", 1)
+    first_line = lines[0]
+
+    if not _SUMMARY_HEADING_RE.match(first_line):
+        return content
+
+    new_header = _build_subsequent_header(commit_hash, commit_url)
+    if len(lines) > 1:
+        return new_header + "\n" + lines[1]
+    return new_header
+
+
+def validate_comment_header(content: str, is_subsequent: bool) -> bool:
+    """Validate whether a comment's header matches the expected format for its position.
+
+    Args:
+        content: The full markdown content of the comment.
+        is_subsequent: Whether this is a subsequent (reply) comment or a top-level comment.
+
+    Returns:
+        True if the header format is valid for the given position, False otherwise.
+    """
+    if not content:
+        return False
+
+    first_line = content.split("\n", 1)[0]
+
+    if is_subsequent:
+        # Subsequent comments must start with ### Commit:
+        return first_line.startswith("### Commit:")
+    else:
+        # Top-level comments must start with ## File Review Summary: or ## Overall PR Review Summary
+        return bool(_SUMMARY_HEADING_RE.match(first_line))
+
+
+def repair_subsequent_header(content: str, review_state: ReviewState) -> str:
+    """Repair an invalid subsequent-comment header using ReviewState metadata.
+
+    If the content starts with a top-level summary heading (``## ...``), replaces
+    it with the compact ``### Commit:`` format using commit metadata from
+    ``review_state``.
+
+    Args:
+        content: The markdown content with a potentially invalid header.
+        review_state: ReviewState providing ``commitHash`` as the source of truth.
+
+    Returns:
+        Content with the header repaired (or unchanged if already valid).
+    """
+    if not content:
+        return content
+
+    first_line = content.split("\n", 1)[0]
+
+    # Only repair if the header is a known top-level summary heading
+    if not _SUMMARY_HEADING_RE.match(first_line):
+        return content
+
+    commit_hash = review_state.commitHash if review_state.commitHash else None
+    # ReviewState does not carry a commit URL; use hash-only or unknown fallback
+    return rewrite_header_for_subsequent(content, commit_hash, None)

{agentic_devtools-0.2.137 → agentic_devtools-0.2.139}/agentic_devtools/cli/cert_utils.py

@@ -27,19 +27,57 @@ from agentic_devtools.cli.subprocess_utils import run_safe
 _CERTS_DIR = Path.home() / ".agdt" / "certs"
 
 
+def normalize_pem_block(pem: str) -> str:
+    """Normalize a single PEM certificate block.
+
+    Removes blank/whitespace-only lines between markers, strips
+    leading/trailing whitespace from base64 content lines, emits canonical
+    marker lines, and converts ``\\r\\n`` to ``\\n``.
+
+    This is the shared normalization function (FR-005) called by both
+    :func:`fetch_certificate_chain_openssl` and ``_build_unified_ca_bundle``.
+
+    Args:
+        pem: A single PEM certificate block string (from BEGIN to END marker).
+
+    Returns:
+        Canonically normalized PEM block with Unix line endings.
+    """
+    # Normalize line endings first
+    pem = pem.replace("\r\n", "\n").replace("\r", "\n")
+
+    lines = pem.split("\n")
+    content_lines: list[str] = []
+    for line in lines:
+        stripped = line.strip()
+        if not stripped:
+            continue
+        if stripped == "-----BEGIN CERTIFICATE-----" or stripped == "-----END CERTIFICATE-----":
+            continue
+        content_lines.append(stripped)
+
+    return "-----BEGIN CERTIFICATE-----\n" + "\n".join(content_lines) + "\n-----END CERTIFICATE-----"
+
+
 def normalize_pem(pem_content: str) -> str:
-    """Remove blank lines within PEM certificate bodies.
+    """Normalize all PEM certificate blocks in *pem_content*.
 
-    Some corporate proxies and openssl builds insert empty lines between
-    base64 data lines in PEM certificates.  Python's ``ssl`` module rejects
-    such malformed PEM with ``[X509] PEM lib``.  This function strips those
-    blank lines while preserving the ``BEGIN``/``END`` markers.
+    Delegates to :func:`normalize_pem_block` for each block, which:
+
+    * Removes blank/whitespace-only lines between markers.
+    * Strips leading/trailing whitespace from base64 content lines.
+    * Rewrites ``BEGIN``/``END`` marker lines canonically (surrounding
+      whitespace trimmed).
+    * Converts ``\\r\\n`` to ``\\n``.
+
+    Non-certificate content outside the ``BEGIN``/``END`` markers is
+    preserved unchanged.
 
     Args:
         pem_content: Raw PEM string (may contain multiple certificates).
 
     Returns:
-        Normalized PEM string with blank lines removed from certificate bodies.
+        Normalized PEM string with all certificate blocks canonicalized.
     """
     cert_pattern = r"-----BEGIN CERTIFICATE-----.*?-----END CERTIFICATE-----"
     certs = re.findall(cert_pattern, pem_content, re.DOTALL)
@@ -48,10 +86,9 @@ def normalize_pem(pem_content: str) -> str:
 
     result = pem_content
     for cert in certs:
-        lines = cert.split("\n")
-        clean_lines = [line for line in lines if line.strip()]
-        if len(clean_lines) < len(lines):
-            result = result.replace(cert, "\n".join(clean_lines))
+        normalized = normalize_pem_block(cert)
+        if normalized != cert:
+            result = result.replace(cert, normalized)
     return result
 
 
@@ -85,10 +122,15 @@ def fetch_certificate_chain_openssl(hostname: str, port: int = 443) -> str | Non
             shell=False,
         )
         output = result.stdout.decode("utf-8", errors="ignore")
-        cert_pattern = r"-----BEGIN CERTIFICATE-----.*?-----END CERTIFICATE-----"
-        certs = re.findall(cert_pattern, output, re.DOTALL)
+        cert_pattern = (
+            r"^\s*-----BEGIN CERTIFICATE-----\s*$"
+            r".*?"
+            r"^\s*-----END CERTIFICATE-----\s*$"
+        )
+        certs = re.findall(cert_pattern, output, re.DOTALL | re.MULTILINE)
         if certs:
-            return normalize_pem("\n".join(certs))
+            normalized = [normalize_pem_block(c) for c in certs]
+            return "\n".join(normalized)
     except (subprocess.TimeoutExpired, FileNotFoundError, OSError, subprocess.SubprocessError):
         pass
     except Exception:  # noqa: BLE001 — cert fetch must not crash setup

{agentic_devtools-0.2.137 → agentic_devtools-0.2.139}/agentic_devtools/cli/setup/commands.py

@@ -20,7 +20,7 @@ from pathlib import Path
 from urllib.parse import urlparse
 
 from agentic_devtools.cli.cert_utils import ensure_ca_bundle as _ensure_ca_bundle
-from agentic_devtools.cli.cert_utils import normalize_pem as _normalize_pem
+from agentic_devtools.cli.cert_utils import normalize_pem_block as _normalize_pem_block
 
 from .copilot_cli_installer import install_copilot_cli
 from .dependency_checker import check_all_dependencies, print_dependency_report
@@ -105,7 +105,7 @@ def _build_unified_ca_bundle(per_host_pem_paths: list[str]) -> Path | None:
         chain = re.findall(cert_pattern, content, re.DOTALL)
         # Skip index 0 (leaf/server cert); only add intermediates and roots
         for cert in chain[1:]:
-            cert = _normalize_pem(cert)
+            cert = _normalize_pem_block(cert)
             if cert not in system_certs:
                 system_certs.add(cert)
                 extra_certs.append(cert)
@@ -117,7 +117,7 @@ def _build_unified_ca_bundle(per_host_pem_paths: list[str]) -> Path | None:
     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")
+        unified_path.write_text(unified_content, encoding="utf-8", newline="\n")
     except OSError as exc:
         print(f"  ⚠ Could not write unified CA bundle {unified_path}: {exc}", file=sys.stderr)
         return None

{agentic_devtools-0.2.137 → agentic_devtools-0.2.139}/specs/1518-thread-title-formatting-subsequent/tasks.md

@@ -20,57 +20,57 @@
 
 ## Phase 1: Setup
 
-- [ ] T001 Create test directory structure with `__init__.py` files for new test modules under `tests/unit/cli/azure_devops/review_templates/` (FR-010)
+- [x] T001 Create test directory structure with `__init__.py` files for new test modules under `tests/unit/cli/azure_devops/review_templates/` (FR-010)
 
 ---
 
 ## Phase 2: Foundational — Core Rendering Changes
 
-- [ ] T002 Add `is_subsequent: bool = False` parameter to `render_file_summary()` in `agentic_devtools/cli/azure_devops/review_templates.py` (FR-003)
-- [ ] T003 Add `is_subsequent: bool = False` parameter to `render_overall_summary()` in `agentic_devtools/cli/azure_devops/review_templates.py` (FR-003)
-- [ ] T004 Implement header selection logic in `render_file_summary()`: when `is_subsequent=True`, emit `### Commit: [<short_hash>](<commit_url>)` instead of `## File Review Summary: {fileName}`, with
+- [x] T002 Add `is_subsequent: bool = False` parameter to `render_file_summary()` in `agentic_devtools/cli/azure_devops/review_templates.py` (FR-003)
+- [x] T003 Add `is_subsequent: bool = False` parameter to `render_overall_summary()` in `agentic_devtools/cli/azure_devops/review_templates.py` (FR-003)
+- [x] T004 Implement header selection logic in `render_file_summary()`: when `is_subsequent=True`, emit `### Commit: [<short_hash>](<commit_url>)` instead of `## File Review Summary: {fileName}`, with
   FR-008 fallbacks (FR-001, FR-002, FR-004)
-- [ ] T005 Implement header selection logic in `render_overall_summary()`: when `is_subsequent=True`, emit `### Commit: [<short_hash>](<commit_url>)` instead of `## Overall PR Review Summary`, with
+- [x] T005 Implement header selection logic in `render_overall_summary()`: when `is_subsequent=True`, emit `### Commit: [<short_hash>](<commit_url>)` instead of `## Overall PR Review Summary`, with
   FR-008 fallbacks (FR-001, FR-002, FR-004)
-- [ ] T006 Add `rewrite_header_for_subsequent(content: str, commit_hash: str | None, commit_url: str | None) -> str` utility to
+- [x] T006 Add `rewrite_header_for_subsequent(content: str, commit_hash: str | None, commit_url: str | None) -> str` utility to
   `agentic_devtools/cli/azure_devops/review_templates.py` implementing FR-008 fallback chain
-- [ ] T007 Add `validate_comment_header(content: str, is_subsequent: bool) -> bool` to `agentic_devtools/cli/azure_devops/review_templates.py` (FR-005, FR-006)
-- [ ] T008 Add `repair_subsequent_header(content: str, review_state: ReviewState) -> str` to `agentic_devtools/cli/azure_devops/review_templates.py` (FR-007, FR-008)
+- [x] T007 Add `validate_comment_header(content: str, is_subsequent: bool) -> bool` to `agentic_devtools/cli/azure_devops/review_templates.py` (FR-005, FR-006)
+- [x] T008 Add `repair_subsequent_header(content: str, review_state: ReviewState) -> str` to `agentic_devtools/cli/azure_devops/review_templates.py` (FR-007, FR-008)
 
 ---
 
 ## Phase 3: User Story 1 — Compact Follow-Up Headers (P1)
 
-- [ ] T009 [P] [US1] Write happy-path tests for `render_file_summary(is_subsequent=True)` with full commit hash+URL (primary success scenario), plus hash-only and missing-hash fallback variants in
+- [x] T009 [P] [US1] Write happy-path tests for `render_file_summary(is_subsequent=True)` with full commit hash+URL (primary success scenario), plus hash-only and missing-hash fallback variants in
   `tests/unit/cli/azure_devops/review_templates/test_render_file_summary.py` (FR-003, FR-010)
-- [ ] T010 [P] [US1] Write happy-path tests for `render_file_summary(is_subsequent=False)` confirming existing `## File Review Summary:` title is unchanged
+- [x] T010 [P] [US1] Write happy-path tests for `render_file_summary(is_subsequent=False)` confirming existing `## File Review Summary:` title is unchanged
   (primary backward-compat success scenario) in
   `tests/unit/cli/azure_devops/review_templates/test_render_file_summary.py` (FR-001, FR-010)
-- [ ] T011 [P] [US1] Write happy-path tests for `render_overall_summary(is_subsequent=True)` with full commit hash+URL (primary success scenario) and all fallback variants in
+- [x] T011 [P] [US1] Write happy-path tests for `render_overall_summary(is_subsequent=True)` with full commit hash+URL (primary success scenario) and all fallback variants in
   `tests/unit/cli/azure_devops/review_templates/test_render_overall_summary.py` (FR-010)
-- [ ] T012 [P] [US1] Write tests for `render_overall_summary(is_subsequent=False)` confirming existing `## Overall PR Review Summary` title is unchanged in
+- [x] T012 [P] [US1] Write tests for `render_overall_summary(is_subsequent=False)` confirming existing `## Overall PR Review Summary` title is unchanged in
   `tests/unit/cli/azure_devops/review_templates/test_render_overall_summary.py` (FR-001, FR-010)
-- [ ] T013 [US1] Write tests for `rewrite_header_for_subsequent()` in `tests/unit/cli/azure_devops/review_templates/test_rewrite_header_for_subsequent.py` covering full-link, hash-only, and unknown
+- [x] T013 [US1] Write tests for `rewrite_header_for_subsequent()` in `tests/unit/cli/azure_devops/review_templates/test_rewrite_header_for_subsequent.py` covering full-link, hash-only, and unknown
   fallbacks (FR-002, FR-010)
-- [ ] T014 [US1] Update `_demote_main_comment()` in `agentic_devtools/cli/azure_devops/review_scaffold.py` to call `rewrite_header_for_subsequent()` on the old content before posting it as a reply
+- [x] T014 [US1] Update `_demote_main_comment()` in `agentic_devtools/cli/azure_devops/review_scaffold.py` to call `rewrite_header_for_subsequent()` on the old content before posting it as a reply
   (FR-002, FR-004)
-- [ ] T015 [US1] Verify that non-header body content is preserved unchanged when `is_subsequent=True` in both render functions (FR-004) — add assertion in existing test files
-- [ ] T016 [US1] Verify NFR-001 determinism: same inputs to render functions always produce same output — add parametrized test in
+- [x] T015 [US1] Verify that non-header body content is preserved unchanged when `is_subsequent=True` in both render functions (FR-004) — add assertion in existing test files
+- [x] T016 [US1] Verify NFR-001 determinism: same inputs to render functions always produce same output — add parametrized test in
   `tests/unit/cli/azure_devops/review_templates/test_render_file_summary.py` (FR-010)
 
 ---
 
 ## Phase 4: User Story 2 — Format Validation & Repair (P2)
 
-- [ ] T017 [P] [US2] Write tests for `validate_comment_header()` in `tests/unit/cli/azure_devops/review_templates/test_validate_comment_header.py` — top-level valid, subsequent with `## <title>`
+- [x] T017 [P] [US2] Write tests for `validate_comment_header()` in `tests/unit/cli/azure_devops/review_templates/test_validate_comment_header.py` — top-level valid, subsequent with `## <title>`
   invalid, subsequent with `### Commit:` valid (FR-005, FR-006, FR-010)
-- [ ] T018 [P] [US2] Write tests for `repair_subsequent_header()` in `tests/unit/cli/azure_devops/review_templates/test_repair_subsequent_header.py` — full repair, hash-only fallback, unknown fallback
+- [x] T018 [P] [US2] Write tests for `repair_subsequent_header()` in `tests/unit/cli/azure_devops/review_templates/test_repair_subsequent_header.py` — full repair, hash-only fallback, unknown fallback
   (FR-007, FR-008, FR-010)
 - [ ] T019 [US2] Add eligibility rules in `agentic_devtools/cli/azure_devops/finalization/classification.py` to identify demoted-summary replies in file-summary/overall-summary threads
 - [ ] T020 [US2] Integrate `validate_comment_header()` and `repair_subsequent_header()` into `agentic_devtools/cli/azure_devops/finalization/convergence.py` for eligible demoted-summary reply comments
   (FR-007)
-- [ ] T021 [US2] Ensure `_compute_file_summary_content()` and `_compute_overall_summary_content()` in convergence remain `is_subsequent=False` for top-level comment expectations (FR-001)
-- [ ] T022 [US2] Write integration tests verifying repair sources values from `ReviewState` (not comment content) in
+- [x] T021 [US2] Ensure `_compute_file_summary_content()` and `_compute_overall_summary_content()` in convergence remain `is_subsequent=False` for top-level comment expectations (FR-001)
+- [x] T022 [US2] Write integration tests verifying repair sources values from `ReviewState` (not comment content) in
   `tests/unit/cli/azure_devops/review_templates/test_repair_subsequent_header.py` (FR-007, FR-010)
 
 ---