Dev#568
Merged
Merged
Conversation
There was a problem hiding this comment.
Pull request overview
This PR migrates InvisiProxy’s Wisp server implementation from @mercuryworkshop/wisp-js to mrrowisp, updates related configuration/documentation, and bumps the project version to 6.9.8.
Changes:
- Replace
wisp-jsserver integration with a newMrrowispinstance and updated request routing/shutdown handling. - Update dependencies (remove
@mercuryworkshop/wisp-js, addmrrowisp) and bump versions inpackage.json,package-lock.json, andsrc/data.json. - Adjust partner/external routing references (remove Freedom Project preset wiring from client JS; add
freedomprojectexternal route on the server) and update docs/TODO.
Reviewed changes
Copilot reviewed 7 out of 8 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
| views/assets/js/common-1778310233.js | Removes Freedom Project proxy preset + UI wiring from goProx/prSet. |
| TODO.md | Updates roadmap item indicating the switch to mrrowisp. |
| src/server.mjs | Replaces wisp-js with mrrowisp configuration, routing, and stop hook. |
| src/routes.mjs | Adds freedomproject as an externalPages redirect target. |
| src/data.json | Bumps embedded version string to 6.9.8. |
| README.md | Updates reverse-proxy warning text to reference mrrowisp. |
| package.json | Bumps version and swaps dependency from wisp-js to mrrowisp. |
| package-lock.json | Locks new dependency tree for mrrowisp and bumps version. |
Comments suppressed due to low confidence (1)
TODO.md:25
- TODO now has two checked items that conflict: one says the project swapped from wisp-js to mrrowisp (line 19) while a later item still claims the project swapped to wisp-js over wisp-server-node (line 25). Consider updating/removing the older wisp-js item (or clarifying timeline) to avoid misleading future contributors.
- [ ] Allow custom Wisp urls from the settings menu (not config side)
- [ ] Setting to open multiple stealth frames; basically about:blank but using our system. Pops out in another tab
- [x] Swap to mrrowisp over wisp-js for security and performance - done
- [x] Hide local browser history on launch - done
- [x] Setting to open about:blank and blob frames - done
- [x] Transport Options Swapping on Frame (Settings Menu doesn't swap) - done
- [x] Update respective site branding - done
- [x] Fix GeForce Now support - done
- [x] Swap to wisp-js over wisp-server-node for security and performance - done
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| freedomproject: urlHandler(sjUrl('https://0xdc.icu')), | ||
|
|
||
| chatgpt: urlHandler(sjUrl('https://chat.openai.com/chat')), | ||
|
|
Comment on lines
808
to
816
| prSet('pr-uv', 'ultraviolet'); | ||
| prSet('pr-sj', 'scramjet'); | ||
| prSet('pr-rh', 'rammerhead'); | ||
| prSet('pr-yt', 'youtube'); | ||
| prSet('pr-iv', 'invidious'); | ||
| prSet('pr-trl', 'tru'); | ||
| prSet('pr-fe', 'freedomproject'); | ||
| prSet('pr-cg', 'chatgpt'); | ||
| prSet('pr-fm', 'fmhy'); | ||
| prSet('pr-dc', 'discord'); |
Comment on lines
+25
to
+27
|
|
||
| const wisp = new Mrrowisp({ | ||
| port: 6001, |
Comment on lines
+81
to
+85
| reputation: { | ||
| enabled: true, | ||
| storePath: './data/mrrowisp-reputation.json', | ||
| saveIntervalSeconds: 30, | ||
| scoreDecayPerHour: 2, |
| import Fastify from 'fastify'; | ||
| import { createServer } from 'node:http'; | ||
| import { server as wisp, logging } from "@mercuryworkshop/wisp-js/server"; | ||
| import { Mrrowisp } from "mrrowisp"; |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
@QuiteAFancyEmerald do not merge before looking over the config, I think it's good but it definitely could use review