Include and install patched pi-diff-review extension

[knocte]

With skynot setup (running Pi with a pi user, which doesn't have GUI access), pi-diff-review extension (https://github.com/badlogic/pi-diff-review) doesn't work because this extension tries to open a browser instance.

We should create a PR for skynot that:

• Adds https://github.com/badlogic/pi-diff-review as a submodule.
• As part of its npm run exec routine (or npx after it has been published in npm), it clones the submodule.
• After cloning the submodule, it makes a few-lines modification of its sources (i.e. having a .patch file inside the repo and using patch -i on it), to just print the URL of the website that it would have opened in the browser (so that the user of Pi can copy the link from the terminal and open the browser manually).
• It should install the modified extension inside the pi user's $HOME.

[webwarrior-ws]

From what I've understood reading code of pi-diff-review, it doesn't open a browser, it uses glimpseui to open specialized webview window and communicate with it.

[knocte]

it doesn't open a browser, it uses glimpseui to open specialized webview window and communicate with it.

Ok, whatever, does that difference matter? Given how we would launch Pi with skynot, I don't think this would work either. Test it?

[webwarrior-ws]

it doesn't open a browser, it uses glimpseui to open specialized webview window and communicate with it.

Ok, whatever, does that difference matter? Given how we would launch Pi with skynot, I don't think this would work either. Test it?

Indeed it does not work. And my point was that "print the URL of the website that it would have opened in the browser" wouldn't work either.

[webwarrior-ws]

Also mere presence of awto-pi-lot breaks pi-diff-review by writing to console.

[knocte]

my point was that "print the URL of the website that it would have opened in the browser" wouldn't work either.

CAN YOU BE MORE SPECIFIC PLEASE???? why wouldn't that work???? it is just printing stuff into the console

[knocte]

Also mere presence of awto-pi-lot breaks pi-diff-review by writing to console.

Yes I noticed that, we need to fix that in awto-pi-lot or in the pi-diff-review extension itself.

[webwarrior-ws]

my point was that "print the URL of the website that it would have opened in the browser" wouldn't work either.

CAN YOU BE MORE SPECIFIC PLEASE???? why wouldn't that work???? it is just printing stuff into the console

Because the extension has to talk to the browser it opens.