Skip to content

feat(utils): add parseCommands utility to format custom command configs#555

Open
HamedHaghani wants to merge 1 commit into
jwu910:mainfrom
HamedHaghani:issue553-add-parse-functionality
Open

feat(utils): add parseCommands utility to format custom command configs#555
HamedHaghani wants to merge 1 commit into
jwu910:mainfrom
HamedHaghani:issue553-add-parse-functionality

Conversation

@HamedHaghani

Copy link
Copy Markdown

Description

This PR adds a utility function parseCommands in src/utils/parseCommands.ts.

It takes a commands object (e.g., from a config file) and parses it into an array of command objects with name, key, and description properties. This sets up the project to better support custom command configurations.

Closes #553

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update

Checklist:

  • I have read the CONTRIBUTING document.
  • My commits follow the commitizen commit convention
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.

@HamedHaghani
HamedHaghani requested review from jwu910 and kienD as code owners May 27, 2025 23:10
@jwu910

jwu910 commented May 28, 2025

Copy link
Copy Markdown
Owner

Hi @HamedHaghani thanks so much for being interested in contributing to this project! It's definitely been a long time since we have had any time to try to improve and update it.

I skimmed over the PR and the issue associated, and I had a few questions. The PR doesn't quite seem complete as I only see one file add which includes an exported function. I was thinking maybe there was some implementation work that didn't get pushed up.

Do you have any extra information on how this is getting implemented into the current workflow of the application and/or how a user would take advantage of the feature?

Happy to try to help answer any questions, but please have some patience, too, as I may not be able to respond right away.

@HamedHaghani

HamedHaghani commented May 29, 2025 via email

Copy link
Copy Markdown
Author

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.

Create parse functionality for commands

2 participants