Repository navigation
feat: expose current user access token - #24
Conversation
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: df634c5f03
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 02dee93412
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| /// Gets the access token used to authenticate the current request, | ||
| /// or null when no access token is available. | ||
| /// </summary> | ||
| string? AccessToken { get; } |
There was a problem hiding this comment.
Document the new access-token contract
For consumers following the README to implement an ICurrentUser transport adapter, this new required member is absent from the Handler Authorization guidance, so the shipped documentation does not explain that adapters must populate it or may return null. Update that guidance or its example alongside the public interface change.
AGENTS.md reference: AGENTS.md:L59-L63
Useful? React with 👍 / 👎.

0 New Issues
0 Fixed Issues
0 Accepted Issues
No data about coverage (100.00% Estimated after merge)
Что изменено
ICurrentUserдобавлено nullable-свойствоstring? AccessToken { get; }с XML-документацией.5.2.Зачем
Дать прикладным сценариям доступ к токену текущего вызывающего через общий контекст вместо передачи токена в каждой команде. В этом PR добавлен только контракт; HTTP-реализация будет отдельным изменением.
Проверка локально
Прошли 309 тестов. Сборки без предупреждений; форматирование и создание NuGet-пакета прошли. Предварительная сборка нужна форматтеру для source generator.
Совместимость
Добавлен обязательный член публичного интерфейса: реализации и тестовые заглушки
ICurrentUserдолжны добавитьAccessToken; при отсутствии токена можно возвращатьnull. HTTP-адаптер необходимо обновить отдельно перед переходом потребителей на новую версию.Миграций, новых зависимостей и изменений конфигурации нет.