Add action row support by solareon · Pull Request #1484 · Tyrrrz/DiscordChatExporter

solareon · 2026-02-25T11:00:27Z

this adds support for exporting the action rows sometimes found in embeds. Only was able to test with single button action rows. I used copilot to draft up a proof of concept and tweaked to get it working 100% and fixed the colors to match discord styling

No issues closed by this. This adds support for action rows in exports for html and json

solareon · 2026-02-25T11:04:27Z

Preview in HTML format

Copilot

Pull request overview

This pull request adds support for exporting Discord action rows with button components to both HTML and JSON export formats. The implementation follows Discord's component structure and styling conventions, allowing exported chat logs to display interactive buttons (though they're rendered as static or disabled elements in the export).

Changes:

New data models for button components and action rows with JSON parsing support
HTML rendering with Discord-style button CSS including 6 button styles (Primary, Secondary, Success, Danger, Link, Premium)
JSON export serialization for components structure
Basic test coverage for component parsing and JSON array existence

Reviewed changes

Copilot reviewed 9 out of 9 changed files in this pull request and generated 7 comments.

Show a summary per file

File	Description
DiscordChatExporter.Core/Discord/Data/Components/ButtonStyle.cs	Defines enum for 6 Discord button styles matching API specification
DiscordChatExporter.Core/Discord/Data/Components/ButtonComponent.cs	Button component model with parsing logic for style, label, emoji, URL, and other properties
DiscordChatExporter.Core/Discord/Data/Components/ActionRowComponent.cs	Action row container for button components with filtering for button type
DiscordChatExporter.Core/Discord/Data/Message.cs	Integrates components into Message model and IsEmpty logic
DiscordChatExporter.Core/Exporting/PreambleTemplate.cshtml	Adds CSS styles for button components and action rows
DiscordChatExporter.Core/Exporting/MessageGroupTemplate.cshtml	Renders buttons as anchor tags (for URL buttons) or disabled button elements
DiscordChatExporter.Core/Exporting/JsonMessageWriter.cs	Serializes components to JSON format with button and action row structures
DiscordChatExporter.Cli.Tests/Specs/ComponentParsingSpecs.cs	Tests parsing of link button from message payload
DiscordChatExporter.Cli.Tests/Specs/JsonContentSpecs.cs	Verifies components array exists in JSON export

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

this adds support for exporting the action rows sometimes found in embeds. Only was able to test with single button action rows. I used copilot to draft up a proof of concept and tweaked to get it working 100% and fixed the colors to match discord styling Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Copilot

Pull request overview

Copilot reviewed 8 out of 8 changed files in this pull request and generated 4 comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Tyrrrz · 2026-04-07T14:44:24Z

@solareon thank you. Please see updated review comments above.

This should improve buffer performance for the JsonWriter. I guess that the others in this file are probably redundant as well but out of scope for this PR.

This respects the disabled state of buttons and fixes a few styling and performance tweaks to the loop.

Tyrrrz · 2026-04-21T08:01:06Z

+    bool IsDisabled
+)
+{
+    public bool IsUrlButton => !string.IsNullOrWhiteSpace(Url);


Suggested change

public bool IsUrlButton => !string.IsNullOrWhiteSpace(Url);

public bool IsLink => !string.IsNullOrWhiteSpace(Url);

Tyrrrz · 2026-04-21T08:02:58Z

+        if (!Enum.IsDefined(typeof(MessageComponentType), type))
+            return null;


Are you sure this check is necessary? If that value is not defined in the enum, your current logic would flow through to ParseDefault anyway. Is ParseDefault not meant to act as a fallback?

Tyrrrz · 2026-04-21T08:04:45Z

+            json.GetPropertyOrNull("style")
+                ?.GetInt32OrNull()
+                ?.Pipe(s =>
+                    Enum.IsDefined(typeof(ButtonStyle), s) ? (ButtonStyle)s : (ButtonStyle?)null


I think we can just do Enum.ParseOrNull<ButtonStyle>(s) here (you may need to merge latest prime to get access to this method via PowerKit.Extensions). We will handle unknown styles during rendering (which your code already does by falling back to secondary style).

Tyrrrz · 2026-04-21T09:14:25Z

+namespace DiscordChatExporter.Core.Discord.Data.Components;
+
+// https://docs.discord.com/developers/components/reference#component-object
+public partial record MessageComponent(


The design I was thinking about was more like this:

MessageComponent: abstract class, essentially a marker (basically a union in other languages). Has a Parse(JsonElement) method that checks the rawType and calls actual components' own Parse(JsonElement) methods.

ButtonMessageComponent: inherits from MessageComponent, has its own Parse(JsonElement), and the properties that are inherent to a button.

ActionRowComponent: inherits from MessageComponent, has its own Parse(JsonElement) (that recursively calls into MessageComponent.Parse to parse children), and the properties that are inherent to an action row (i.e. Children).

The rendering code would then pattern match via:

component switch { ButtonMessageComponent button => ... ActionRowComponent actionRow => ... }

Tyrrrz reviewed Feb 26, 2026

View reviewed changes

Comment thread DiscordChatExporter.Cli.Tests/Specs/ComponentParsingSpecs.cs Outdated

Comment thread DiscordChatExporter.Cli.Tests/Specs/JsonContentSpecs.cs Outdated

Comment thread DiscordChatExporter.Core/Exporting/JsonMessageWriter.cs

Tyrrrz added the enhancement label Feb 26, 2026

Tyrrrz requested a review from Copilot February 26, 2026 13:29

Tyrrrz changed the title ~~feat: add action row support~~ Add action row support Feb 26, 2026

Copilot started reviewing on behalf of Tyrrrz February 26, 2026 13:30 View session

Tyrrrz added format/html format/json labels Feb 26, 2026

Copilot AI reviewed Feb 26, 2026

View reviewed changes

solareon requested a review from Tyrrrz February 27, 2026 13:33

Tyrrrz reviewed Mar 2, 2026

View reviewed changes

Comment thread DiscordChatExporter.Core/Discord/Data/Message.cs Outdated

Comment thread DiscordChatExporter.Core/Discord/Data/Message.cs Outdated

Comment thread DiscordChatExporter.Core/Exporting/JsonMessageWriter.cs Outdated

solareon and others added 5 commits March 2, 2026 14:46

refactor: remove excessive test

231857e

fix: add themeing support for dark/light mode for buttons

a54e011

chore: add todo for other component types

09f9f38

fix: remove unused import from test

1a671b3

solareon force-pushed the add-buttons branch 2 times, most recently from cbeec1a to 1a671b3 Compare March 2, 2026 13:59

refactor: pt 1 of redesign for generic components structure

0414da2

Tyrrrz reviewed Mar 13, 2026

View reviewed changes

Comment thread DiscordChatExporter.Core/Discord/Data/Components/MessageComponent.cs

Tyrrrz reviewed Mar 13, 2026

View reviewed changes

Comment thread DiscordChatExporter.Core/Discord/Data/Components/MessageComponent.cs Outdated

refactor: make MessageComponent accept type

2bf8b86

Tyrrrz requested a review from Copilot April 6, 2026 20:12

Copilot started reviewing on behalf of Tyrrrz April 6, 2026 20:12 View session

Copilot AI reviewed Apr 6, 2026

View reviewed changes

solareon added 2 commits April 8, 2026 09:32

fix: remove extra FlushAsync

08524e9

This should improve buffer performance for the JsonWriter. I guess that the others in this file are probably redundant as well but out of scope for this PR.

refactor: respect disabled state

b889603

This respects the disabled state of buttons and fixes a few styling and performance tweaks to the loop.

solareon requested a review from Tyrrrz April 8, 2026 07:47

Tyrrrz reviewed Apr 21, 2026

View reviewed changes

	public bool IsUrlButton => !string.IsNullOrWhiteSpace(Url);
	public bool IsLink => !string.IsNullOrWhiteSpace(Url);

		if (!Enum.IsDefined(typeof(MessageComponentType), type))
		return null;

Uh oh!

Conversation

solareon commented Feb 25, 2026

Uh oh!

solareon commented Feb 25, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Tyrrrz commented Apr 7, 2026

Uh oh!

Tyrrrz Apr 21, 2026

Choose a reason for hiding this comment

Uh oh!

Tyrrrz Apr 21, 2026

Choose a reason for hiding this comment

Uh oh!

Tyrrrz Apr 21, 2026

Choose a reason for hiding this comment

Uh oh!

Tyrrrz Apr 21, 2026

Choose a reason for hiding this comment

Uh oh!

Tyrrrz Apr 21, 2026

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

solareon commented Feb 25, 2026 •

edited

Loading