Repository navigation
Use built-in token for team-memory dispatch - #1107
Closed
Changyong Gong (chagong) wants to merge 1 commit into
Closed
Changyong Gong (chagong) wants to merge 1 commit into
Changyong Gong (chagong) wants to merge 1 commit into
Conversation
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
|
Azure Pipelines: There may be pipelines that require an authorized user to comment /azp run to run. |
Changyong Gong (chagong)
marked this pull request as ready for review
October 12, 2026 02:49
Changyong Gong (chagong)
requested review from
Jinbo Wang (testforstephen) and
wenyt (wenytang-ms)
as code owners
October 12, 2026 02:49
wenyt (wenytang-ms)
approved these changes
Oct 12, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Blocked: not merge-ready
This is a user-directed draft, not a working cross-repository dispatch replacement. The built-in
github.token(GITHUB_TOKEN) is scoped tomicrosoft/vscode-java-dependencyand has no Actions-write access to the coordinator repository,microsoft/vscode-java-pack. Granting job-levelactions: writedoes not expand that repository scope. If merged as-is, opted-in pushes would fail to queue the Java Pack coordinator.Java Pack's reference workflow dispatches within its own repository. Neither that same-repository configuration nor a public read diagnostic proves this caller can write to Java Pack. Keep this PR draft and do not merge or enable auto-merge; cross-repository authorization needs a separately agreed solution. This PR does not add credentials, alter the coordinator architecture, or disable the opt-in gate to conceal the blocker.
Requested change
actions/create-github-app-tokendispatcher-token step and its App credential references.${{ github.token }}to the existing pinnedqueue-team-memoryaction.contents: readandactions: write, matching Java Pack's main workflow; retain top-levelpermissions: {}.Only
.github/workflows/team-memory-post-merge.ymlchanges. The canonical source, fixedmainpush trigger, actual-default-branch gate, workflow-head gate, non-created/non-deleted/non-forced gates, opt-in, concurrency, runtime limits, immutable action pin, explicit Java Pack repository/workflow/main target, and exact five-string JSON payload are unchanged.Offline checks
Passed YAML parsing with cached
js-yaml@4.1.0, exact parsed-baseline comparison, pinned action input-schema checks, permissions/reference comparison, credential-reference removal checks, exact payload comparison, all eight gate clauses, and 15 gate cases (one allowed and 14 rejected).git diff --checkalso passed.Reference: Java Pack main at
ea2dd0e56e4c5a4710913a8b0fda5197b7e67587; pinned queue action ata81d2d96167fc0e69ac631c2edc85f858e693289. These offline checks do not prove cross-repository dispatch authorization.No live token minting, coordinator dispatch, workflow rerun, or changes to secrets, variables, Apps, repository settings, Azure, IssueLens, deployments, or public documentation were performed. Production remains unchanged while this PR stays unmerged.