[PR #1836] [MERGED] Move to JS code execution sandbox (HOPP-67, HOPP-66) #3512

New issue

Closed

opened 2026-03-17 01:07:17 +03:00 by kerem · 0 comments

kerem commented

2026-03-17 01:07:17 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/hoppscotch/hoppscotch/pull/1836
Author: @AndrewBastin
Created: 9/24/2021
Status: ✅ Merged
Merged: 9/25/2021
Merged by: @AndrewBastin

Base: main ← Head: refactor/js-sandbox

📝 Commits (7)

d2865c6 refactor: bring js-sandbox project to the monorepo
166f9e8 feat: project level testing + add fp-ts and sandbox as deps
9454d8c feat: move testing code to js-sandbox
0c38970 chore(docs): updated readme
6751c50 fix: build
680937e feat: fix issue with the pre-request envs
de2d336 refactor: bump js-sandbox version to 1.0.0

📊 Changes

32 files changed (+2091 additions, -508 deletions)

View changed files

📝 README.md (+8 -6)
📝 package.json (+2 -1)
📝 packages/hoppscotch-app/components/http/Request.vue (+23 -16)
📝 packages/hoppscotch-app/helpers/RequestRunner.ts (+63 -132)
📝 packages/hoppscotch-app/helpers/__tests__/postwomanTesting.sample (+0 -0)
➖ packages/hoppscotch-app/helpers/postwomanTesting.ts (+0 -321)
📝 packages/hoppscotch-app/helpers/preRequest.ts (+18 -31)
📝 packages/hoppscotch-app/package.json (+4 -1)
➕ packages/hoppscotch-js-sandbox/.eslintrc.js (+29 -0)
➕ packages/hoppscotch-js-sandbox/.gitignore (+5 -0)
➕ packages/hoppscotch-js-sandbox/.prettierignore (+8 -0)
➕ packages/hoppscotch-js-sandbox/.prettierrc.js (+3 -0)
➕ packages/hoppscotch-js-sandbox/LICENSE (+21 -0)
➕ packages/hoppscotch-js-sandbox/README.md (+65 -0)
➕ packages/hoppscotch-js-sandbox/jest.config.js (+6 -0)
➕ packages/hoppscotch-js-sandbox/jest.setup.ts (+1 -0)
➕ packages/hoppscotch-js-sandbox/package.json (+59 -0)
➕ packages/hoppscotch-js-sandbox/src/__tests__/preRequest.spec.ts (+64 -0)
➕ packages/hoppscotch-js-sandbox/src/__tests__/testing/expect/toBe.spec.ts (+98 -0)
➕ packages/hoppscotch-js-sandbox/src/__tests__/testing/expect/toBeLevelxxx.spec.ts (+361 -0)

...and 12 more files

📄 Description

This PR brings forth the following updates.

Move @hoppscotch/js-sandbox to the new monorepo
Introduce fp-ts as a dependency to hoppscotch-app
Migrate the current pre-request and test script execution to the newly introduced sandbox.
Introduce a test command handler in root project.

(HOPP-67)
(HOPP-66)

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/hoppscotch/hoppscotch/pull/1836 **Author:** [@AndrewBastin](https://github.com/AndrewBastin) **Created:** 9/24/2021 **Status:** ✅ Merged **Merged:** 9/25/2021 **Merged by:** [@AndrewBastin](https://github.com/AndrewBastin) **Base:** `main` ← **Head:** `refactor/js-sandbox` --- ### 📝 Commits (7) - [`d2865c6`](https://github.com/hoppscotch/hoppscotch/commit/d2865c637c2bf82d01b63ca1b62da87a39f1b059) refactor: bring js-sandbox project to the monorepo - [`166f9e8`](https://github.com/hoppscotch/hoppscotch/commit/166f9e817b56657d7b139b39513152a3f90a3296) feat: project level testing + add fp-ts and sandbox as deps - [`9454d8c`](https://github.com/hoppscotch/hoppscotch/commit/9454d8c100bbd1fd98927dd18afa744bacf3f8d9) feat: move testing code to js-sandbox - [`0c38970`](https://github.com/hoppscotch/hoppscotch/commit/0c389701fe14bd406ea1dfbe248a98a059a6a3ad) chore(docs): updated readme - [`6751c50`](https://github.com/hoppscotch/hoppscotch/commit/6751c50514ea01733d9f9d6b791617057660c684) fix: build - [`680937e`](https://github.com/hoppscotch/hoppscotch/commit/680937e50b69ba3a08cfa7c49336e6bd11cb9e3b) feat: fix issue with the pre-request envs - [`de2d336`](https://github.com/hoppscotch/hoppscotch/commit/de2d3361a703b422b5e25b7cc7df4b9eea4518df) refactor: bump js-sandbox version to 1.0.0 ### 📊 Changes **32 files changed** (+2091 additions, -508 deletions) <details> <summary>View changed files</summary> 📝 `README.md` (+8 -6) 📝 `package.json` (+2 -1) 📝 `packages/hoppscotch-app/components/http/Request.vue` (+23 -16) 📝 `packages/hoppscotch-app/helpers/RequestRunner.ts` (+63 -132) 📝 `packages/hoppscotch-app/helpers/__tests__/postwomanTesting.sample` (+0 -0) ➖ `packages/hoppscotch-app/helpers/postwomanTesting.ts` (+0 -321) 📝 `packages/hoppscotch-app/helpers/preRequest.ts` (+18 -31) 📝 `packages/hoppscotch-app/package.json` (+4 -1) ➕ `packages/hoppscotch-js-sandbox/.eslintrc.js` (+29 -0) ➕ `packages/hoppscotch-js-sandbox/.gitignore` (+5 -0) ➕ `packages/hoppscotch-js-sandbox/.prettierignore` (+8 -0) ➕ `packages/hoppscotch-js-sandbox/.prettierrc.js` (+3 -0) ➕ `packages/hoppscotch-js-sandbox/LICENSE` (+21 -0) ➕ `packages/hoppscotch-js-sandbox/README.md` (+65 -0) ➕ `packages/hoppscotch-js-sandbox/jest.config.js` (+6 -0) ➕ `packages/hoppscotch-js-sandbox/jest.setup.ts` (+1 -0) ➕ `packages/hoppscotch-js-sandbox/package.json` (+59 -0) ➕ `packages/hoppscotch-js-sandbox/src/__tests__/preRequest.spec.ts` (+64 -0) ➕ `packages/hoppscotch-js-sandbox/src/__tests__/testing/expect/toBe.spec.ts` (+98 -0) ➕ `packages/hoppscotch-js-sandbox/src/__tests__/testing/expect/toBeLevelxxx.spec.ts` (+361 -0) _...and 12 more files_ </details> ### 📄 Description This PR brings forth the following updates. - Move [@hoppscotch/js-sandbox](https://github.com/hoppscotch/hopp-js-sandbox) to the new monorepo - Introduce [fp-ts](https://github.com/gcanti/fp-ts) as a dependency to `hoppscotch-app` - Migrate the current pre-request and test script execution to the newly introduced sandbox. - Introduce a test command handler in root project. (HOPP-67) (HOPP-66) --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>