fix(microsoft-excel): decide SharePoint site discovery consent model

Aberta
  • Algo errado?
#7,197 1 comentário 0 reações 0 responsáveis Ver no GitHub

Mantenedores costumam responder em até 1 dia

@Zoymusk já está trabalhando nisso.

Desde 19/9/2026.

Avaliação

Dificuldade
5/5
Tempo estimado
Mais de uma semana
Facilidade para iniciantes
35/100
Tipo de issue
Funcionalidade
Clareza
Razoavelmente clara
Status de atividade
Ativa
Stack de tecnologia
typescript
Domínio
api, authentication, authorization

Direção de pesquisa

Start by tracing the Excel OAuth consent and SharePoint site selector credential-binding paths, then review the focused permission/binding tests and Microsoft Graph site-search requirements. Done means the supported credential model is documented and implemented consistently, unsupported credentials show a clear product-level error, live work-or-school validation is covered, and existing Excel file operations still work.

Escrita pelo modelo de indexação a partir do texto da issue.

Descrição

Summary

Microsoft Excel credentials can now be recognized by the SharePoint site selector, but the existing Excel OAuth consent does not authorize Microsoft Graph site search.

Graph site search requires Sites.Read.All. Sim’s Excel connection currently requests file permissions only, so an ordinary existing Excel credential should be expected to receive a 403. The endpoint also does not support delegated personal Microsoft accounts.

This is a pre-existing consent-model limitation, not a selector transport or credential-binding bug.

Decision needed

Choose and implement one supported model:

  1. Add Sites.Read.All to the standard Excel consent and require existing users to reconnect/reconsent.
  2. Add conditional or incremental consent when SharePoint site discovery is requested.
  3. Require a separate SharePoint credential for site discovery and narrow the Excel selector compatibility accordingly.

The decision should account for least privilege, reconnect UX, existing credentials, and personal-account behavior.

Acceptance criteria

  • The supported Excel/SharePoint credential model is documented.
  • OAuth scopes, reconnect/reconsent behavior, and selector credential binding agree.
  • Unsupported credentials receive a clear product-level error.
  • Focused permission/binding tests and live work-or-school tenant validation are added.
  • Existing Excel file operations continue to work.

Reference

Microsoft Graph site search requires Sites.Read.All and does not support delegated personal Microsoft accounts:

https://learn.microsoft.com/en-us/graph/api/site-search?tabs=http&view=graph-rest-1.0

Linguagem predominante
TypeScript
Estrelas
29.8k
Forks
3.9k
Merge médio
1h 44min
PRs com merge (30d)
841

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

  1. Leia a issue inteira e depois o guia de contribuição do projeto.
  2. Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
  3. Faça um fork do repositório e trabalhe em uma branch.
  4. Abra um pull request que referencie o número da issue.

Mais de simstudioai/sim

Todas as issues de simstudioai/sim

Issues semelhantes

Mais issues de TypeScript

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.