fix(ci): correct pyinstaller test assertion and ruff format; add changelog

- copy_metadata() returns one directory entry per package (the whole dist-info
  dir), not individual file entries. The test was asserting p.endswith("/METADATA")
  which can never match a directory path — fix to assert the dest dir name matches.
- Inline the asyncio.to_thread lambda to satisfy ruff line-length check.
- Add changelog entries for the mcp add fix and gitignore auto-exclusion feature.

Author: elkaix

CHANGELOG.md

@@ -15,6 +15,8 @@ GitHub Releases page; `0.8.0` is the new starting line.
 
 ## Unreleased
 
+- **`pythinker mcp add` no longer crashes on Windows and Linux native builds.** The PyInstaller specs were using `collect_data_files()` which silently omits the `fastmcp-*.dist-info/` sibling directory; fastmcp calls `importlib.metadata.version("fastmcp")` at import time, so every `mcp add` / `mcp list` invocation raised `PackageNotFoundError`. Switched all three specs (Windows installer, Linux installer, macOS/tarball) to `copy_metadata()` — the PyInstaller-standard hook for bundling dist-info.
+- **Pythinker work directories are automatically gitignored on startup.** When the agent starts inside a git repository, `.pythinker/`, `.pythinker-review/`, and `.pythinker-review-flow/` are silently appended to the project's `.gitignore` if missing, preventing local agent state from making the working tree dirty.
 - **Windows upgrade version display fix.** In-place upgrades no longer show a stale version number or re-trigger the update prompt. Inno Setup now wipes `_internal` before installing new files, preventing old `dist-info` directories from accumulating and causing `importlib.metadata` to report the previous version.
 
 ## 0.31.0 (2026-06-02)

src/pythinker_code/scratchpad.py

@@ -445,9 +445,7 @@ async def _append_gitignore_entries(work_dir: HostPath) -> None:
     """
     gitignore_path = Path(str(work_dir)) / ".gitignore"
     try:
-        await with_retries(
-            lambda: asyncio.to_thread(_write_gitignore_entries, gitignore_path)
-        )
+        await with_retries(lambda: asyncio.to_thread(_write_gitignore_entries, gitignore_path))
     except Exception:
         logger.debug("scratchpad .gitignore update failed")
 

tests/utils/test_pyinstaller_utils.py

@@ -46,10 +46,12 @@ def test_pyinstaller_datas():
     # justext stoplists), so assert METADATA presence rather than pinning every
     # file. copy_metadata() produces clean paths ({dist-info}/FILE) — no
     # fastmcp/../ prefix that the old collect_data_files workaround created.
+    # copy_metadata() returns a single directory entry per package (the whole
+    # dist-info dir), so check that the dest dir name matches — not a file inside it.
     for pkg in ("fastmcp", "mcp"):
         pkg_dist = f"{pkg}-{version(pkg)}.dist-info"
-        assert any(d == pkg_dist and p.endswith("/METADATA") for p, d in datas), (
-            f"{pkg_dist}/METADATA must be bundled so importlib.metadata works in the frozen app"
+        assert any(d == pkg_dist for p, d in datas), (
+            f"{pkg_dist} must be bundled so importlib.metadata works in the frozen app"
         )
 
     dist_info_dirs = {f"{pkg}-{version(pkg)}.dist-info" for pkg in ("fastmcp", "mcp")}