office-addin-debugging 'stop' doesn't work on MacOS

[andrewconnell]

On macOS 10.14.5, starting the debugger with npm run start which runs office-addin-debugging start manifest.xml spins up a debugging server but in a detached process. Running npm run stop or office-addin-debugging stop manifest.xml does not stop the process.

The only way I was able to get it to stop was to find the node process that had port 3000 open and kill it with Activity Monitor.

[akrantz]

You can avoid this by starting it with npm run dev-server in a separate Terminal window tab, or if using VS Code, use Terminal, Run Task..., Dev Server to run it inside VS Code. You can then close the Terminal window tab or close VS Code to stop the task.

We've already done some work to know the process id of the detached process which is started. We need office-addin-debugging start to write this to disk so that office-addin-debugging stop can know to stop the process.

[andrewconnell]

Good to know... would have helped me this week. However as it stands now, the stop command doesn't work when it says it successfully completed it's task.

[akrantz]

Yes, I agree. I'm not disputing that. We will work to make improvements here. Sorry if this caused problems for you.

[akrantz]

This is now fixed. I have published office-addin-debugging using the "next" dist-tag, so you install the updated package using "npm install -D office-addin-debugging@next".

[jpshankle]

MacOS still appears to have the problem of getting
$ office-addin-debugging stop manifest.xml env: node\r: No such file or directory error Command failed with exit code 127.
The way to resolve the issue locally is to change the line ending in .bin -> office-addin-debugging from CRLF to LF.

[akrantz]

@jpshankle I opened a separate issue about that: #168.

[akrantz]

@jpshankle Please see comments in #168. I wasn't able to mention your username there. Not sure why, but please reply there and we will follow up. Thanks.

[suuunly]

This is now fixed. I have published office-addin-debugging using the "next" dist-tag, so you install the updated package using "npm install -D office-addin-debugging@next".

I tried doing so, updating from 5.0.12, to 5.1.0. However, running
office-addin-debugging stop manifest.xml
still shows webpack in my Activity Monitor. What is more worrying is that it is slowly creeping upwards - suggesting a memory leak.

[suuunly]

@akrantz, I found the solution! I need to run the command with sudo!
So in my react based office add-in, I have to run sudo npm run stop.
Which is the equivalent to sudo office-addin-debugging stop manifest.xml 😁

@andrewconnell, if you haven't found the solution yet, then hopefully this helps :)