Cannot open PR: null review comment commit causes TypeError (reading 'oid')
Mantenedores costumam responder em até 1 dia
Ninguém assumiu esta issue ainda.
Avaliação
- Dificuldade
- 2/5
- Tempo estimado
- 1-3 horas
- Facilidade para iniciantes
- 78/100
- Tipo de issue
- Bug
- Clareza
- Claramente especificada
- Status de atividade
- Ativa
- Stack de tecnologia
- github, typescript, vscode
- Domínio
- devtools
Direção de pesquisa
Comece em src/github/utils.ts, em parseGraphQLComment, onde a issue identifica comment.commit.oid como o acesso que falha. Verifique como a resposta GraphQL representa comentários com um commit null e revise os testes de parsing próximos. Trate esse caso para que abrir o pull request não gere mais uma exceção e, em seguida, verifique com um teste que cubra um commit null.
Escrita pelo modelo de indexação a partir do texto da issue.
Descrição
Environment
GitHub Pull Requests 0.174.0; VS Code 1.144.0 x64; Linux; GitHub.com.
Reproduction
With the extension installed and signed in to GitHub, run:
git clone --filter=blob:none --branch ranges_chunk https://github.com/shyeyian/clang.git clang-pr-171234 &&
cd clang-pr-171234 &&
git remote add upstream https://github.com/llvm/llvm-project.git &&
git fetch --filter=blob:none upstream main &&
git config branch.ranges_chunk.github-pr-owner-number 'llvm#llvm-project#171234' &&
code .
Click the GitHub Pull Requests activity-bar icon. In Changes in Pull Request #171234, click the header action View Pull Request Description.
PR contents fail to load normally. The GitHub Pull Request output channel shows:
branch ranges_chunk. Repo: llvm/llvm-project PR: 171234
2026-11-04 16:06:57.386 [info] [PullRequestGitHelper] associate ranges_chunk with Pull Request #171234
2026-11-04 16:06:57.394 [info] [Review+0] Found matching pull request metadata for current branch ranges_chunk. Repo: llvm/llvm-project PR: 171234
2026-11-04 16:06:57.394 [info] [Review+0] current branch ranges_chunk is associated with pull request #171234
2026-11-04 16:06:57.394 [info] [Review+0] Resolving pull request
2026-11-04 16:07:03.093 [info] [GitHubRepository+0.1] Fetch branch main - enter
2026-11-04 16:07:04.321 [info] [GitHubRepository+0.1] Fetch branch main - done: true
2026-11-04 16:07:04.329 [info] [Review+0] Resolved PR #171234, state is OPEN
2026-11-04 16:07:04.329 [info] [Review+0] Fetching pull request data
2026-11-04 16:07:04.329 [info] [PullRequestTree] Adding PR #171234 to tree
2026-11-04 16:07:04.349 [info] [Review+0] Register comments provider
2026-11-04 16:07:04.349 [info] [Review+0] Register in memory content provider
2026-11-04 16:07:07.281 [info] [Review+0] Display pull request status bar indicator.
2026-11-04 16:07:07.281 [info] [Review+0] Using focus mode = true.
2026-11-04 16:07:07.281 [info] [Review+0] State validation silent = true.
2026-11-04 16:07:07.281 [info] [Review+0] PR show should show = false.
2026-11-04 16:07:18.702 [error] [Review+0] Failed to initialize PR data TypeError: Cannot read properties of null (reading 'oid'): Cannot read properties of null (reading 'oid')
2026-11-04 16:08:03.163 [info] [PullRequestTree] Creating file and commit nodes for PR #171234
2026-11-04 16:08:03.164 [info] [PullRequestTree] Getting children for Files node
2026-11-04 16:08:03.164 [info] [PullRequestTree] Got all children for Files node
2026-11-04 16:08:13.623 [info] [FolderRepositoryManager+0] Using in-memory cached assignable users.
2026-11-04 16:08:29.564 [error] [PullRequestOverviewPanel] Failed to update deferred timeline: Cannot read properties of null (reading 'oid')
Failing Source
src/github/utils.ts, line 648, parseGraphQLComment: commitId: comment.commit.oid throws when comment.commit is null. GraphQL currently returns null commits for 33 comments in this PR, including this comment.
- Linguagem predominante
- TypeScript
- Estrelas
- 2.6k
- Forks
- 811
- Merge médio
- 1d 9h
- PRs com merge (30d)
- 69
Preparar o ambiente
Este projeto não oferece contêiner de desenvolvimento, Dockerfile nem guia de contribuição, então a configuração fica por sua conta: comece pelo README e veja nosso guia da primeira contribuição para os passos gerais.
Primeiros passos
- Leia a issue inteira e depois o guia de contribuição do projeto.
- Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
- Faça um fork do repositório e trabalhe em uma branch.
- Abra um pull request que referencie o número da issue.
Mais de microsoft/vscode-pull-request-github
-
Merging from the PR description tab shows "Error refreshing pull request description: Webview is disposed"Talvez já em andamento @alexr00 assumiu há 2 dias. Aberta
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 74/100
microsoft/vscode-pull-request-github#9143 · 1 reação · 2 responsáveis ·
Mantenedores costumam responder em até 1 dia
-
Honor open-to-side placement when opening GitHub PR and issue linksTalvez já em andamento @alexr00 assumiu há 2 dias. Abertafeature-request
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 82/100
microsoft/vscode-pull-request-github#9072 · 1 reação · 2 responsáveis ·
Mantenedores costumam responder em até 1 dia
-
Resolve Conflicts doesn’t open in VS Code Desktop with a GitHub Remote RepositoryTalvez já em andamento @alexr00 assumiu há 1 dia. Aberta
microsoft/vscode-pull-request-github#9158 · 1 comentário · 1 responsável ·
Mantenedores costumam responder em até 1 dia
-
Checkout Pull Request in Worktree still fetches fork head from base remote in 0.174.0Talvez já em andamento @alexr00 assumiu há 2 dias. Aberta
microsoft/vscode-pull-request-github#9156 · 1 responsável ·
Mantenedores costumam responder em até 1 dia
-
Show loading feedback while Comment and Resolve is resolving a threadTalvez já em andamento @alexr00 assumiu há 2 dias. Aberta
microsoft/vscode-pull-request-github#9155 · 1 responsável ·
Mantenedores costumam responder em até 1 dia
Todas as issues de microsoft/vscode-pull-request-github
Issues semelhantes
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 78/100
Open-Earth-Foundation/CityCatalyst#3399 ·
Mantenedores costumam responder em até 2 dias
-
userReport
Dificuldade 1/5 Menos de uma hora Facilidade para iniciantes 72/100
nkprasad12/morcus-net#631 ·
Mantenedores costumam responder em até 1 dia
-
fix(app): fail closed if GetGeneralSetting fails during initializationTalvez já em andamento @kalavt assumiu hoje. Abertabug
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 76/100
Mantenedores costumam responder em até 1 dia
-
Dificuldade 1/5 Menos de uma hora Facilidade para iniciantes 85/100
nekko-labs/nekko-agent#1756 ·
Mantenedores costumam responder em até 1 dia
-
needs-area-label
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 66/100
Mantenedores costumam responder em até 2 dias