feat: add autocomplete feature in cli #1724
Conversation
AayushSaini101
requested review from
Souvikns,
Amzani,
Shurtu-gal and
asyncapi-bot-eve
as code owners
|
|
/u |
|
/u |
|
@AayushSaini101 I don't think this is ths way autocomplete should be done can we run it as part of installation process? |
Sure. Let me check during the installation process |
|
Awesome feature in my opinion, I am always forgetting commands 😆 |
|
1 test case is failing, checking the cause |
| @@ -163,6 +165,7 @@ | |||
| "pretest": "npm run build", | |||
| "pretest:coverage": "npm run build", | |||
| "posttest": "rimraf ./test.asyncapi-cli", | |||
| "postinstall": "node ./scripts/enableAutoComplete.js", | |||
There was a problem hiding this comment.
Does this script runs after packaging as well?
Also what about docker image. Does this still work there?
Same question for chocolatey.
| const fs = require('fs'); | ||
| const path = require('path'); | ||
|
|
||
| const shellConfigs = { |
There was a problem hiding this comment.
What about their derivatives?
| }, | ||
| fish: { | ||
| rcFile: path.join(os.homedir(), '.config', 'fish', 'completions', 'asyncapi.fish'), | ||
| detectFile: path.join(os.homedir(), '.config', 'fish', 'config.fish'), |
There was a problem hiding this comment.
Detection using $SHELL or some other way would be better. Not everyone has their config like this.
There was a problem hiding this comment.
After running the npm install
Here is the result
The process.env.SHELL returning /bin/sh the because many system use by default to run these kind of scripts.
but if we use the same concept as in the PR, we can easily detect the default shell of the os. Result
|
/u |
|
Proof of working:
Resolves: #1685