Skip to content

generate delegates.txt from the @tc39/data-delegates package - #422

Merged
ljharb merged 1 commit into
mainfrom
structured-data
Sep 18, 2026
Merged

ljharb merged 1 commit into
mainfrom
structured-data

Conversation

@michaelficarra

Copy link
Copy Markdown
Member

I had to update eslint in order to use the JSON import syntax.

Disclaimer: I used an LLM in the process of creating this PR.

Comment thread delegates.txt
Daniel Veditz (DVE)
Daniele Bonetta (DBA)
Danilo Françoso Tedeschi (DFT)
Danilo Francoso Tedeschi (DFT)

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

for this one, the ç is probably correct?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's just what they've listed on their GitHub account. If the ç was preferred, I imagine that's what they would've listed.

@rkirsling rkirsling Sep 18, 2026

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems like the suspect commit would instead be tc39/data@af1da8f then? If delegates.txt has been a manually maintained file, then it should be the (basis for the new) source of truth, right? There's been no foregoing expectation for these names to be auto-updated via GH; it's conceivable for people to want to represent themselves differently wrt Ecma. 🤔

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The names in delegates.txt are never updated, even when someone's preferred name changes, or when their legal name changes, each of which has happened multiple times. With few exceptions (all easily resolved manually), the data from GitHub is both more accurate and more respectful of their wishes.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Only four of seven delegates from my company are actually putting their full name in their GitHub profile, but all of them have been willing to provide it when registering as a delegate. It would be a problem if their provided, actual name got replaced with a potentially silly display name...yet evidently that didn't happen?

@michaelficarra michaelficarra Sep 18, 2026

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, that is what I was referring to above: the data is sourced from delegates.txt and from the delegate's GitHub profile. I manually reviewed where they differed, and went with the GitHub profile except where the profile name fell into one of the following 3 categories:

  1. Incomplete or abbreviated profiles: Aditi Singh (Aditi), Aki Braun (Aki), Alissa Renz (alissa), Angela Wang (Angela), Aurèle Barrière (Aurèle), Boshen Chen (Boshen), Cam Tenny (Cam), Dave Poole (Dave), Franziska Hinkelmann (F. Hinkelmann), Henry Zhu (Henry), Ilias Tsangaris (Ilias), Jenna Zeigen (Jenna), Jessica Sachs (Jess), Kristen Hewell Garrett (Kristen), Michael Z Goddard (Z Goddard), Sebastian McKenzie (Sebastian), Sigrid Huemer (Sigrid), Stephen Murphy (Stephen M), Steven Salat (Steven), Titus Wormer (Titus), and Yorkie Liu (Yorkie)
  2. Handles, aliases, or pseudonyms: Aaron Davis (kumavis), Guanyu Liu (liuguanyu), Gus Caplan (snek), Isaac Schlueter (isaacs), Kai Gong (kai.gongk), Liu ZiJian (Lxxyx), Luna Pfeiffer (Shark), Qiang Tu (Yuri), Ryuichi Hayashida (Linda_pp), Shi Jiang Yu (yusj), Song Yang Pu (pusongyang), Takuya Kawasaki (Kawa), Weiwen Liang (exoticknight), Weixuan Lin (Linbudu), Yuanyan Cao (Y2), Yulia Startsev (yulia), and Zhi Jie Li (zeldajay)
  3. Styling or materially different forms: Ashley Williams (ashley williams), Bradford Smith (Bradford C. Smith), Corey Frang (Mx Corey Frang), Hubert Manilla (Hubert Boma Manilla), István Sebestyén (Istvan SEBESTYEN), Josh Goldberg (Josh Goldberg ✨), Mary Marchini (mary marchini), Samina Husain (SaminaHusain), and Subo Zheng (Super Zheng)

Without asking every delegate their individual preference, it's impossible to know for sure which way to resolve them, but I feel for most cases it was pretty clear. Of course, delegates are free to PR the structured data to change it, just as they were free to PR delegates.txt before. Given that most delegates are no longer active and that most of the decisions seemed pretty straightforward to me, I didn't feel it was necessary or practical to reach out for clarification.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fair enough! Appreciate all the thought you've put into this. Sorry to be a backseat driver. 😅

Comment thread delegates.txt
Frank Yung-Fong Tang (FYT)
Franziska Hinkelmann (FHN)
Frederik Braun (FDY)
Frivalszky-Mayer Péter (PFM)

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

similarly, i think this one should probably stay in given + surname order?

@michaelficarra michaelficarra Sep 18, 2026

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is how they've written their name on their GitHub profile. Without consulting them, I don't think it'd be appropriate to write their name any way other than the 2 samples we have of them writing their own name: exactly as listed on their GitHub profile or exactly as they've previously added to delegates.txt. If you have another source (such as social media, publications, personal website, etc.) showing them referring to themselves in that way, I think it would be fine. Same goes for all of your other comments.

Comment thread delegates.txt
Simon Zünd (SZD)
Song Yang Pu (SYP)
Songfeng Li (SLI)
Songfeng Li(李松峰) (SLI)

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it might be confusing to have two parenthesized character sequences?

Comment thread delegates.txt
Willian Martins (WMS)
Xian Kun Wang (WXK)
Xuan Huang (HUX)
Xuan Huang (黄玄) (HUX)

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

also here

Comment thread delegates.txt
Comment on lines +582 to +583
小船 (HZO)
沧浪 (SNS)

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it might be better to stick with the roman character versions here (or have both)?

@ljharb
ljharb merged commit e8015bc into main Sep 18, 2026
2 checks passed
@ljharb
ljharb deleted the structured-data branch September 18, 2026 03:59
Comment thread delegates.txt
Zalim Bashorov (ZBV)
Zbigniew Tenerowicz (ZTZ)
Zeimin Lei (LZM)
Zbyszek Tenerowicz (ZTZ)

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@naugtur I know that you go in some places by one name and in others by the other; which name should be used here?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

(i believe we went with the github profile name when there were conflicts, so updating that might be the simplest way to change it)

@ctcpip

ctcpip commented Sep 21, 2026

Copy link
Copy Markdown
Member

I think this should have gotten broader committee awareness and feedback before merging.

@naugtur

naugtur commented Sep 21, 2026 via email

Copy link
Copy Markdown
Contributor

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants