fix: use ty-native ignore syntax for add_cookies

[stickerdaniel]

Summary

Replace # type: ignore[arg-type] (mypy syntax) with # ty: ignore[invalid-argument-type] (ty-native syntax) on the single suppression in core/browser.py. The mypy-style comment was inert under ty.

Why

The pending lockfile maintenance PR (#227) bumps ty to 0.0.33, which adds stricter checks. With the mypy-style # type: ignore[arg-type] comment that ty doesn't honor, ty 0.0.33 flags the BrowserContext.add_cookies(cookies) call as a dict[str, Any] vs Sequence[SetCookieParam] mismatch, breaking lint-and-check.

The mismatch is intentional and runtime-safe: cookies is a hand-built list of dicts loaded from JSON, and patchright validates the shape at runtime. Only the suppression syntax needs updating.

Test plan

• Reproduced the failure: uvx --from 'ty==0.0.33' ty check linkedin_mcp_server/core/browser.py → error[invalid-argument-type] before the fix.
• Verified silent after fix on ty 0.0.33.
• Verified ty 0.0.20 (currently pinned in main) still passes after fix — backwards-compatible with the current lock.
• Full pytest passes locally on Python 3.13.13 — 389 passed in 5.11s.
• pre-commit run passes (ruff, ruff-format, ty all green).
• CI green on this PR.
• After merge, re-rebase chore(deps): lock file maintenance #227 — its lint-and-check should pass.

[stickerdaniel]

• fix: use ty-native ignore syntax for add_cookies #394 👈 (View in Graphite)
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

[greptile-apps]

Greptile Summary

This PR updates a single type-checker suppression comment in core/browser.py from mypy syntax (# type: ignore[arg-type]) to ty-native syntax (# ty: ignore[invalid-argument-type]), making it effective under ty 0.0.33 which will be introduced by the pending lockfile PR (#227). The runtime behaviour is unchanged — no logic is modified.

Confidence Score: 5/5

Safe to merge — one-line comment change with no runtime impact.

The only change is swapping a type-checker suppression comment syntax; no logic, imports, or data flow are affected.

No files require special attention.

Important Files Changed

Filename	Overview
linkedin_mcp_server/core/browser.py	Replaced mypy-style # type: ignore[arg-type] with ty-native # ty: ignore[invalid-argument-type] on the add_cookies call; no logic change.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[Load cookies from JSON file] --> B{li_at cookie present?}
    B -- No --> C[Return False]
    B -- Yes --> D["await context.add_cookies(cookies)\n# ty: ignore[invalid-argument-type]"]
    D --> E[Log import success]
    E --> F[Return True]

Loading

_{Reviews (1): Last reviewed commit: "fix: use ty-native ignore syntax for add..." | Re-trigger Greptile}

[Copilot]

Pull request overview

Updates the type-check suppression in import_cookies() to use ty’s native ignore comment syntax so the existing runtime-safe add_cookies(cookies) call remains accepted under newer ty versions.

Changes:

• Replaced inert mypy-style # type: ignore[arg-type] with # ty: ignore[invalid-argument-type].
• Adjusted formatting of the add_cookies call to attach the ty ignore to the argument line.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.