refactor(match2): esports match summary too

[Rathoz]

Summary

refactor esports wiki match summary too

How did you test this change?

trivial

[Copilot]

🔵 Needs a closer look

The new createGame implementation can error by dereferencing game.extradata.link even though extradata is optional (table?).

Pull request overview

Refactors the esports wiki MatchSummary implementation to align with the newer MatchSummary/Base customization surface by removing the custom body builder and switching to a createGame hook.

Changes:

• Removes the custom createBody implementation (and its related imports).
• Renames/replaces the per-game row generator with CustomMatchSummary.createGame.
• Updates type annotations to Renderable/Renderable?.

File summaries

File	Description
lua/wikis/esports/MatchSummary.lua	Drops custom body creation and migrates to a createGame implementation used by MatchSummary/Base.

Review details

Suppressed comments (1)

lua/wikis/esports/MatchSummary.lua:41

• MatchGroupUtilGame.extradata is typed as optional (table?), but this code dereferences game.extradata.link unconditionally when the game is not finished. If extradata is absent, this will throw a Lua error. Safer to normalize with local extradata = game.extradata or {} and read extradata.link.

	if gamePhase ~= 'finished' then
		if game.extradata.link == nil then
			return MatchSummaryWidgets.Row{
				children = MatchSummaryWidgets.GameCenter{children = 'Voting opening soon!'}
			}
		end

		return MatchSummaryWidgets.Row{
			children = MatchSummaryWidgets.GameCenter{children = Button{
				link = game.extradata.link,

• Files reviewed: 1/1 changed files
• Comments generated: 0
• Review effort level: Lite

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.