fix(js-sandbox): improve scripting value handling and serialization

- Fix null/undefined environment variable handling across namespaces
- Fix pm.request console.log output to display properly
- Add pm.request.id and pm.request.name type definitions
- Fix assertion error messages to show actual values
- Strip `export {};` from collection exports and legacy sandbox editor display

Author: jamesgeorge007

packages/hoppscotch-common/src/components/collections/index.vue

@@ -2831,9 +2831,12 @@ const exportData = async (collection: HoppCollection | TeamCollection) => {
   if (collectionsType.value.type === "my-collections") {
     const collectionJSON = JSON.stringify(collection, null, 2)
 
+    // Strip `export {};\n` from `testScript` and `preRequestScript` fields
+    const cleanedCollectionJSON = collectionJSON.replace(/export \{\};\\n/g, "")
+
     const name = (collection as HoppCollection).name
 
-    initializeDownloadCollection(collectionJSON, name)
+    initializeDownloadCollection(cleanedCollectionJSON, name)
   } else {
     if (!collection.id) return
     exportLoading.value = true
@@ -2850,8 +2853,14 @@ const exportData = async (collection: HoppCollection | TeamCollection) => {
           const hoppColl = teamCollToHoppRESTColl(coll)
           const collectionJSONString = JSON.stringify(hoppColl, null, 2)
 
+          // Strip `export {};\n` from `testScript` and `preRequestScript` fields
+          const cleanedCollectionJSON = collectionJSONString.replace(
+            /export \{\};\\n/g,
+            ""
+          )
+
           await initializeDownloadCollection(
-            collectionJSONString,
+            cleanedCollectionJSON,
             hoppColl.name
           )
           exportLoading.value = false

packages/hoppscotch-common/src/composables/codemirror.ts

@@ -268,6 +268,19 @@ const getEditorLanguage = (
   completer: Completer | undefined
 ): Extension => hoppLang(getLanguage(langMime) ?? undefined, linter, completer)
 
+const MODULE_PREFIX = "export {};\n" as const
+
+/**
+ * Strips the `export {};\n` prefix from the value for display in the editor.
+ * The above is only used internally for Monaco editor's module scope,
+ * and should not be visible in the CodeMirror editor.
+ */
+const stripModulePrefix = (value?: string): string | undefined => {
+  return value?.startsWith(MODULE_PREFIX)
+    ? value.slice(MODULE_PREFIX.length)
+    : value
+}
+
 export function useCodemirror(
   el: Ref<any | null>,
   value: Ref<string | undefined>,
@@ -474,7 +487,10 @@ export function useCodemirror(
     view.value = new EditorView({
       parent: el,
       state: EditorState.create({
-        doc: parseDoc(value.value, options.extendedEditorConfig.mode ?? ""),
+        doc: parseDoc(
+          stripModulePrefix(value.value),
+          options.extendedEditorConfig.mode ?? ""
+        ),
         extensions,
       }),
       // scroll to top when mounting
@@ -514,13 +530,17 @@ export function useCodemirror(
     if (!view.value && el.value) {
       initView(el.value)
     }
+
+    // Strip `export {};\n` before displaying in CodeMirror
+    const displayValue = stripModulePrefix(newVal) ?? ""
+
     if (cachedValue.value !== newVal) {
       view.value?.dispatch({
         filter: false,
         changes: {
           from: 0,
           to: view.value.state.doc.length,
-          insert: newVal,
+          insert: displayValue,
         },
       })
     }

packages/hoppscotch-common/src/types/post-request.d.ts

@@ -652,6 +652,8 @@ declare namespace pm {
   }>
 
   const request: Readonly<{
+    readonly id: string
+    readonly name: string
     readonly url: Readonly<{
       toString(): string
       readonly protocol: string

packages/hoppscotch-js-sandbox/src/__tests__/combined/env-fallback-behavior.spec.ts

@@ -106,9 +106,8 @@ describe("Environment Variable Fallback Behavior - All Namespaces", () => {
     "$name - currentValue undefined/null fallback",
     ({ get }) => {
       // Note: These tests check GET behavior with null/undefined in initial state.
-      // Setting null via pm.environment.set(key, null) is currently NOT supported.
-      // These tests validate fallback logic when null/undefined exists in the env state,
-      // which can occur through other means (e.g., test setup, external sources).
+      // Setting null via pm.environment.set(key, null) IS supported (via NULL_MARKER).
+      // These tests validate fallback logic when null/undefined exists in the env state.
 
       test("should fallback to initialValue when currentValue is undefined", async () => {
         const envs = {