✨ More built-in commands

[Ph0enixKM]

This issue is about implementing the following commands:

• ~~cd (cd path: Text -> cd path) failable: directory does not exist~~
• ~~mv (mv a: Text, b: Text -> mv a b) failable: no permissions~~
• rm (rm path: Text -> rm -rf path) failable: no permissions / file does not exits
• exit (exit c: Num exit c)

[Mte90]

On https://github.com/Ph0enixKM/Amber/pull/185 I added is_root and is_command that I think that should be built-in commands. The second one just check if a command exists.

[Ph0enixKM]

I'm still thinking about that ORM style you suggested. To make such a nice way to compose commands would be a game changer

[Mte90]

I think that we need to define built-in should be one shot actions, instead ORM style should be more complex commands with various parameters. The ORM style requires a Class engine in Amber but for that we can discuss in the other ticket.

[b1ek]

we should also add [[ and [ as builtins for use like if [[ ... ]]

[Ph0enixKM]

we should also add [[ and [ as builtins for use like if [[ ... ]]

I think that the [[ and [ commands are the workaround to Bash. We can implement something that makes more sense than this

[Mte90]

The ticket is not yet closed as the topic was to add more builtin commands and there was a PR linked that doesn't involve that.

[Ph0enixKM]

Oh sorry @Mte90, I mistook it for a different issue

[Mte90]

RIght now we are missing:

• rm
• exit

I think that we can add touch too. In this way we have the majority of builtin for our tests.

Maybe @MuhamedMagdi do you want to look on these?

[MuhamedMagdi]

Maybe @MuhamedMagdi do you want to look on these?

I'm currently working on rm implementation, but wanted to know how we would approach this feature #369 before opening a pr.

[Mte90]

For that one I think that we have to wait for @Ph0enixKM.