Skip to content

feat(command): add command tracker - #70

Merged
JunkyDeveloper merged 5 commits into
Steel-Foundation:masterfrom
SomeYellowGuy:command-tracker
Sep 3, 2026
Merged

feat(command): add command tracker#70
JunkyDeveloper merged 5 commits into
Steel-Foundation:masterfrom
SomeYellowGuy:command-tracker

Conversation

@SomeYellowGuy

@SomeYellowGuy SomeYellowGuy commented Aug 26, 2026

Copy link
Copy Markdown
Contributor

Expands the tracker to add support for commands.

This also requires the Extractor PR Steel-Foundation/SteelExtractor#26.

@JunkyDeveloper

Copy link
Copy Markdown
Contributor

please link the extractor PR, @Steel-Foundation/documentation-writers short message if you merge

@DarkMrMelther DarkMrMelther left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

It works and looks good, but you need to add the commands to the iterIssues function from ImplementationTracker.tsx in the line 69 (NICE).

Also, I'm not sure if the place for the extracted data for commands is the main repo since it's not used there anyway...?

@DarkMrMelther DarkMrMelther left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Seems good, I'll add a comment in the extractor PR and ask about where to locate the new extracted data

@SomeYellowGuy

Copy link
Copy Markdown
Contributor Author

Seems good, I'll add a comment in the extractor PR and ask about where to locate the new extracted data

I modified the code to use a slightly different regex for commands. This would fix something like /test being detected in Steel-Foundation/SteelMC#394

@SomeYellowGuy

Copy link
Copy Markdown
Contributor Author

I added the command JSON in the repository. What about now?

@DarkMrMelther

Copy link
Copy Markdown
Contributor

I added the command JSON in the repository. What about now?

Wait till the other PR is merged 👍

@tn-lorenz

Copy link
Copy Markdown
Contributor

I added the command JSON in the repository. What about now?

Wait till the other PR is merged 👍

Which one?

@SomeYellowGuy

Copy link
Copy Markdown
Contributor Author

I added the command JSON in the repository. What about now?

Wait till the other PR is merged 👍

Which one?

I think he meant the Extractor PR for commands.

@JunkyDeveloper

Copy link
Copy Markdown
Contributor

so we deadlock now...

@JunkyDeveloper
JunkyDeveloper merged commit ba259c5 into Steel-Foundation:master Sep 3, 2026
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.

4 participants