update cli.md, cli.md and TODO.md

Author: MichaelCurrin

docs/cli.md

@@ -12,8 +12,14 @@ _This should work on Windows too but has not been tested yet._
 
 Requires Git and Node.
 
-<!-- TODO replace with instructions for downloading from a release using manual steps and install with curl or npm
+<!-- TODO replace with instructions for **downloading** from a release using manual steps and install with curl or npm
 Note you do not need Git or Node, you can download the pre-packaged CLI tool as a binary.
+
+For now:
+
+npm run build:install
+cp build-cli/acm-macos ~/.local/bin/acm
+
 -->
 
 ```sh

docs/development/cli.md

@@ -1,12 +1,15 @@
 # CLI Development
 > Maintaining code around the CLI tool
 
-The code is in [src/cli/](/src/cli). The CLI commands are built using steps set in [package.json](/package.json) - see the `cli` command and `bin` section.
-
+The code is in [src/cli/](/src/cli). The CLI commands are built using steps set in [package.json](/package.json).
 
+- See `cli:install` and `bin` section, which are both related and work with `npm link`.
+- See `make cli-build` which is for building binaries with `pkg` for distribution.
 
 ## Run directly
 
+For testing:
+
 ```sh
 $ npx ts-node src/cli/diffIndexGenerate.ts
 ```
@@ -45,4 +48,6 @@ Check:
 ls -l $(realpath /opt/homebrew/bin/acm)
 ```
 
-See [package.json](/package.json). Supposedly you should be able to leave out the project name when running `npm link` via an `npm run ...` command but I found this causes issues, so decided to always use the full name in the configuration. And to _always_ unlink then link in one go because of permissions issues.
+See [package.json](/package.json).
+
+Supposedly you should be able to leave out the project name when running `npm link` via an `npm run ...` command but I found this causes issues, so decided to always use the full name in the configuration. But removed it later . And to _always_ unlink then link in one go because of permissions issues.

shell/TODO.md

@@ -13,5 +13,6 @@ of template, so it that can be passed on.
     there's a problem - commit does not take cached flag - but it will use implied staged or not. so do not pass through.
     BUT could look at advanced mode of pass through names of files to commit IF they are handled by status and commit, but using manual git add is good enough probably. esp for untracked files which need add anyway.
 - [ ] See if it is possible to make the shell script here shorter and use the logic in acm instead and a minimal .sh hook file.
-- [ ] Fix issue where if using the `cli:install` command then doing a build that removes everything means the commands don't work. Perhaps a **different output directory**.
+- [ ] Fix issue where if using the `cli:install` command then doing a build that removes everything means the commands don't work. Perhaps a **different output directory**. Note `cli-build` can work though it is used pkg and renames (sometimes locally and also on CLI), so maybe different.
 - [ ] Rename - this doesn't have to be shell anymore. Like `hooks`. Just fix any references to shell directory.
+- [ ] Build all scripts.