Visitar URL original
JS: Fix parsing of Member[num] token in summaries by asgerf · Pull Request #22784 · github/codeql · GitHub
Skip to content

JS: Fix parsing of Member[num] token in summaries - #22784

Draft
asgerf wants to merge 3 commits into
github:mainfrom
asgerf:js/fix-member-token
Draft

asgerf wants to merge 3 commits into
github:mainfrom
asgerf:js/fix-member-token

Conversation

@asgerf

@asgerf asgerf commented Oct 8, 2026

Copy link
Copy Markdown
Contributor

Fixes the root cause of the issue identified in #22781.

When Member[n] appeared as the input or output in a flow summary, and n is a number in the "max precise array index" range (0..9) it wasn't parsed correctly.

The canonical representative for this content is ArrayElement[n!] (! to exclude the unknownArrayElement content), which is why reverse-encoding fails to parse Member[n].

MathiasVP and others added 3 commits October 8, 2026 11:06
The canonical representative for this content is `ArrayElement[n!]`,
which is why reverse-encoding fails to parse `Member[n]`, but this
token needs to work for consistency with API graphs.
@github-actions github-actions Bot added the JS label Oct 8, 2026
@asgerf asgerf added the no-change-note-required This PR does not need a change note label Oct 8, 2026

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

JS no-change-note-required This PR does not need a change note

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants