🏷️ types: move to ty (#47)

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: SigureMo <38436475+SigureMo@users.noreply.github.com>

Author: Copilot

.vscode/extensions.json

@@ -1,7 +1,7 @@
 {
   "recommendations": [
     "ms-python.python",
-    "ms-python.vscode-pylance",
+    "astral-sh.ty",
     "charliermarsh.ruff",
     "esbenp.prettier-vscode",
     "EditorConfig.EditorConfig",

README.md

@@ -327,7 +327,7 @@ All output follows consistent formatting rules so both humans and LLMs can parse
 
 ```bash
 uv run ruff check
-uv run pyright
+uv run ty check --error-on-warning src/gh_llm tests
 uv run pytest -q
 ```
 

justfile

@@ -12,7 +12,7 @@ fmt:
   prettier --write '**/*.md'
 
 lint:
-  uv run pyright src/gh_llm tests
+  uv run ty check --error-on-warning src/gh_llm tests
   uv run ruff check .
 
 fmt-docs:

pyproject.toml

@@ -30,17 +30,12 @@ gh-llm = "gh_llm.__main__:main"
 
 [dependency-groups]
 dev = [
-  "pyright>=1.1.407",
+  "ty>=0.0.32",
   "ruff>=0.14.10",
   "pytest>=9.0.2",
   "pytest-rerunfailures>=16.1",
 ]
 
-[tool.pyright]
-include = ["src/gh_llm", "tests"]
-pythonVersion = "3.14"
-typeCheckingMode = "strict"
-
 [tool.ruff]
 line-length = 120
 target-version = "py314"
@@ -85,8 +80,6 @@ select = [
 ]
 ignore = [
   "E501", # line too long, duplicate with ruff fmt
-  "F401", # imported but unused, duplicate with pyright
-  "F841", # local variable is assigned to but never used, duplicate with pyright
 ]
 future-annotations = true
 

src/gh_llm/pr_body.py

@@ -79,7 +79,7 @@ def build_pull_request_body_scaffold(
 
     existing_titles = {normalize_section_title(title) for title in extract_markdown_section_titles(cleaned_template)}
     added_sections: list[str] = []
-    blocks = [cleaned_template]
+    blocks: list[str] = [cleaned_template]
 
     for section in required_sections:
         normalized = normalize_section_title(section)

tests/test_cli.py

@@ -1576,7 +1576,7 @@ def test_extract_diff_hunks_prefers_first_added_line_for_right_side() -> None:
         ]
     )
 
-    hunks = pr_commands._extract_diff_hunks(diff)  # pyright: ignore[reportPrivateUsage]
+    hunks = pr_commands._extract_diff_hunks(diff)
 
     assert len(hunks) == 1
     assert hunks[0].path == "paddle/phi/kernels/funcs/abs.h"
@@ -1602,7 +1602,7 @@ def test_extract_diff_hunks_uses_real_new_file_line_numbers_on_right_side() -> N
         ]
     )
 
-    hunks = pr_commands._extract_diff_hunks(diff)  # pyright: ignore[reportPrivateUsage]
+    hunks = pr_commands._extract_diff_hunks(diff)
 
     assert len(hunks) == 1
     assert hunks[0].path == "src/gh_llm/commands/pr.py"
@@ -1612,7 +1612,7 @@ def test_extract_diff_hunks_uses_real_new_file_line_numbers_on_right_side() -> N
 
 
 def test_render_numbered_hunk_lines_preserves_real_right_side_line_numbers() -> None:
-    hunk = pr_commands._DiffHunk(  # pyright: ignore[reportPrivateUsage]
+    hunk = pr_commands._DiffHunk(
         path="src/gh_llm/commands/pr.py",
         header="@@ -890,6 +890,7 @@ def _extract_diff_hunks(diff: str) -> list[_DiffHunk]:",
         anchor_line=893,
@@ -1631,7 +1631,7 @@ def test_render_numbered_hunk_lines_preserves_real_right_side_line_numbers() ->
         match_paths={"src/gh_llm/commands/pr.py"},
     )
 
-    rendered = pr_commands._render_numbered_hunk_lines(hunk)  # pyright: ignore[reportPrivateUsage]
+    rendered = pr_commands._render_numbered_hunk_lines(hunk)
 
     assert "L 890 R 890 |      current_hunk_lines: list[str] = []" in rendered
     assert "L 891 R 891 |      current_old_line = 0" in rendered
@@ -1640,7 +1640,7 @@ def test_render_numbered_hunk_lines_preserves_real_right_side_line_numbers() ->
 
 
 def test_inline_review_thread_blocks_do_not_fallback_from_current_right_anchor_to_original_left_line() -> None:
-    current_hunk = pr_commands._DiffHunk(  # pyright: ignore[reportPrivateUsage]
+    current_hunk = pr_commands._DiffHunk(
         path="paddle/phi/api/include/compat/ATen/ops/from_blob.h",
         header="@@ -18,3 +80,4 @@",
         anchor_line=81,
@@ -1654,7 +1654,7 @@ def test_inline_review_thread_blocks_do_not_fallback_from_current_right_anchor_t
         right_commentable_lines={80, 81, 82},
         match_paths={"paddle/phi/api/include/compat/ATen/ops/from_blob.h"},
     )
-    stale_hunk = pr_commands._DiffHunk(  # pyright: ignore[reportPrivateUsage]
+    stale_hunk = pr_commands._DiffHunk(
         path="paddle/phi/api/include/compat/ATen/ops/from_blob.h",
         header="@@ -80,4 +210,1 @@",
         anchor_line=210,
@@ -1684,7 +1684,7 @@ def test_inline_review_thread_blocks_do_not_fallback_from_current_right_anchor_t
         comments=(),
     )
 
-    blocks_by_hunk = pr_commands._build_inline_review_thread_blocks_for_file(  # pyright: ignore[reportPrivateUsage]
+    blocks_by_hunk = pr_commands._build_inline_review_thread_blocks_for_file(
         hunks=[current_hunk, stale_hunk],
         summaries=[summary],
         extra_contexts=[None, None],
@@ -2897,7 +2897,7 @@ def test_pr_body_template_finds_docs_template_directory(
 
 def test_decode_repository_contents_text_returns_none_for_invalid_base64() -> None:
     payload: dict[str, object] = {"encoding": "base64", "content": "A"}
-    assert github_api._decode_repository_contents_text(payload) is None  # pyright: ignore[reportPrivateUsage]
+    assert github_api._decode_repository_contents_text(payload) is None
 
 
 def test_pr_body_template_surfaces_non_404_lookup_failures(