Skip to content

Conversation

@pcarleton
Copy link
Member

Summary

  • Import BadName from git.exc directly instead of accessing it via git.exc.BadName

The recent security patches (GHSA merges a37158b and 9e5d5b8) introduced uses of git.exc.BadName which pyright doesn't recognize as a valid attribute access on the git module. This causes CI failures:

error: "exc" is not a known attribute of module "git" (reportAttributeAccessIssue)

The fix is to import BadName directly from git.exc at the module level.

Test plan

  • uv run --frozen pyright passes locally
  • All 35 tests pass locally
  • CI passes after merge

🤖 Generated with Claude Code

Import `BadName` from `git.exc` directly instead of accessing it via
`git.exc.BadName`, which pyright doesn't recognize as a valid attribute
access on the `git` module.

This fixes the pyright CI failures introduced by the recent security
patches (GHSA merges).

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
@felixweinberger felixweinberger merged commit 4750df4 into main Dec 18, 2025
23 checks passed
@felixweinberger felixweinberger deleted the fix-git-server-pyright-errors branch December 18, 2025 14:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants