refactor: typescript support

components/realtime/Log.vue

@@ -17,14 +17,13 @@
         {{ title }}
       </label>
     </div>
-    <div ref="log" name="log" class="realtime-log">
+    <div name="log" class="realtime-log">
       <span v-if="log" class="space-y-2">
         <span
           v-for="(entry, index) in log"
           :key="`entry-${index}`"
           :style="{ color: entry.color }"
-          >{{ entry.ts }}{{ getSourcePrefix(entry.source)
-          }}{{ entry.payload }}</span
+          >{{ entry.ts }}{{ source(entry.source) }}{{ entry.payload }}</span
         >
       </span>
       <span v-else>{{ $t("response.waiting_for_connection") }}</span>
@@ -32,27 +31,14 @@
   </div>
 </template>
 
-<script>
-import { defineComponent } from "@nuxtjs/composition-api"
-import { getSourcePrefix } from "~/helpers/utils/string"
+<script setup lang="ts">
+import { getSourcePrefix as source } from "~/helpers/utils/string"
 
-export default defineComponent({
-  props: {
-    log: { type: Array, default: () => [] },
-    title: {
-      type: String,
-      default: "",
-    },
-  },
-  updated() {
-    this.$nextTick(function () {
-      if (this.$refs.log) {
-        this.$refs.log.scrollBy(0, this.$refs.log.scrollHeight + 100)
-      }
-    })
-  },
-  methods: {
-    getSourcePrefix,
+defineProps({
+  log: { type: Array, default: () => [] },
+  title: {
+    type: String,
+    default: "",
   },
 })
 </script>

components/realtime/Mqtt.vue

@@ -145,7 +145,7 @@ import { defineComponent } from "@nuxtjs/composition-api"
 import { Splitpanes, Pane } from "splitpanes"
 import "splitpanes/dist/splitpanes.css"
 import Paho from "paho-mqtt"
-import debounce from "~/helpers/utils/debounce"
+import debounce from "lodash/debounce"
 import { logHoppRequestRunToAnalytics } from "~/helpers/fb/analytics"
 import { useSetting } from "~/newstore/settings"
 import useWindowSize from "~/helpers/utils/useWindowSize"

components/realtime/Socketio.vue

@@ -165,7 +165,7 @@ import { Splitpanes, Pane } from "splitpanes"
 import "splitpanes/dist/splitpanes.css"
 import { io as Client } from "socket.io-client"
 import wildcard from "socketio-wildcard"
-import debounce from "~/helpers/utils/debounce"
+import debounce from "lodash/debounce"
 import { logHoppRequestRunToAnalytics } from "~/helpers/fb/analytics"
 import { useSetting } from "~/newstore/settings"
 import useWindowSize from "~/helpers/utils/useWindowSize"

components/realtime/Sse.vue

@@ -89,8 +89,8 @@
 import { defineComponent } from "@nuxtjs/composition-api"
 import { Splitpanes, Pane } from "splitpanes"
 import "splitpanes/dist/splitpanes.css"
+import debounce from "lodash/debounce"
 import { logHoppRequestRunToAnalytics } from "~/helpers/fb/analytics"
-import debounce from "~/helpers/utils/debounce"
 
 export default defineComponent({
   components: { Splitpanes, Pane },

components/realtime/Websocket.vue

@@ -205,8 +205,8 @@
 import { defineComponent } from "@nuxtjs/composition-api"
 import { Splitpanes, Pane } from "splitpanes"
 import "splitpanes/dist/splitpanes.css"
+import debounce from "lodash/debounce"
 import { logHoppRequestRunToAnalytics } from "~/helpers/fb/analytics"
-import debounce from "~/helpers/utils/debounce"
 import useWindowSize from "~/helpers/utils/useWindowSize"
 import { useSetting } from "~/newstore/settings"
 

helpers/lenses/htmlLens.ts

@@ -1,10 +1,12 @@
-const htmlLens = {
+import { Lens } from "./lenses"
+
+const htmlLens: Lens = {
   lensName: "response.html",
   isSupportedContentType: (contentType) =>
     /\btext\/html|application\/xhtml\+xml\b/i.test(contentType),
   renderer: "htmlres",
   rendererImport: () =>
-    import("~/components/lenses/renderers/HTMLLensRenderer"),
+    import("~/components/lenses/renderers/HTMLLensRenderer.vue"),
 }
 
 export default htmlLens

helpers/lenses/imageLens.ts

@@ -1,4 +1,6 @@
-const imageLens = {
+import { Lens } from "./lenses"
+
+const imageLens: Lens = {
   lensName: "response.image",
   isSupportedContentType: (contentType) =>
     /\bimage\/(?:gif|jpeg|png|bmp|svg\+xml|x-icon|vnd\.microsoft\.icon)\b/i.test(
@@ -6,7 +8,7 @@ const imageLens = {
     ),
   renderer: "imageres",
   rendererImport: () =>
-    import("~/components/lenses/renderers/ImageLensRenderer"),
+    import("~/components/lenses/renderers/ImageLensRenderer.vue"),
 }
 
 export default imageLens

helpers/lenses/jsonLens.ts

@@ -1,11 +1,12 @@
 import { isJSONContentType } from "../utils/contenttypes"
+import { Lens } from "./lenses"
 
-const jsonLens = {
+const jsonLens: Lens = {
   lensName: "response.json",
   isSupportedContentType: isJSONContentType,
   renderer: "json",
   rendererImport: () =>
-    import("~/components/lenses/renderers/JSONLensRenderer"),
+    import("~/components/lenses/renderers/JSONLensRenderer.vue"),
 }
 
 export default jsonLens

helpers/lenses/lenses.js

@@ -1,28 +0,0 @@
-import jsonLens from "./jsonLens"
-import rawLens from "./rawLens"
-import imageLens from "./imageLens"
-import htmlLens from "./htmlLens"
-import xmlLens from "./xmlLens"
-
-export const lenses = [jsonLens, imageLens, htmlLens, xmlLens, rawLens]
-
-export function getSuitableLenses(response) {
-  const contentType = response.headers.find((h) => h.key === "content-type")
-
-  if (!contentType) return [rawLens]
-
-  const result = []
-  for (const lens of lenses) {
-    if (lens.isSupportedContentType(contentType.value)) result.push(lens)
-  }
-
-  return result
-}
-
-export function getLensRenderers() {
-  const response = {}
-  for (const lens of lenses) {
-    response[lens.renderer] = lens.rendererImport
-  }
-  return response
-}

helpers/lenses/lenses.ts

@@ -0,0 +1,42 @@
+import { HoppRESTResponse } from "../types/HoppRESTResponse"
+import jsonLens from "./jsonLens"
+import rawLens from "./rawLens"
+import imageLens from "./imageLens"
+import htmlLens from "./htmlLens"
+import xmlLens from "./xmlLens"
+
+export type Lens = {
+  lensName: string
+  isSupportedContentType: (contentType: string) => boolean
+  renderer: string
+  rendererImport: () => Promise<typeof import("*.vue")>
+}
+
+export const lenses: Lens[] = [jsonLens, imageLens, htmlLens, xmlLens, rawLens]
+
+export function getSuitableLenses(response: HoppRESTResponse): Lens[] {
+  // return empty array if response is loading or error
+  if (response.type === "loading" || response.type === "network_fail") return []
+
+  const contentType = response.headers.find((h) => h.key === "content-type")
+
+  if (!contentType) return [rawLens]
+
+  const result = []
+  for (const lens of lenses) {
+    if (lens.isSupportedContentType(contentType.value)) result.push(lens)
+  }
+  return result
+}
+
+type LensRenderers = {
+  [key: string]: Lens["rendererImport"]
+}
+
+export function getLensRenderers(): LensRenderers {
+  const response: LensRenderers = {}
+  for (const lens of lenses) {
+    response[lens.renderer] = lens.rendererImport
+  }
+  return response
+}

helpers/lenses/rawLens.js

@@ -1,8 +0,0 @@
-const rawLens = {
-  lensName: "response.raw",
-  isSupportedContentType: () => true,
-  renderer: "raw",
-  rendererImport: () => import("~/components/lenses/renderers/RawLensRenderer"),
-}
-
-export default rawLens

helpers/lenses/rawLens.ts

@@ -0,0 +1,11 @@
+import { Lens } from "./lenses"
+
+const rawLens: Lens = {
+  lensName: "response.raw",
+  isSupportedContentType: () => true,
+  renderer: "raw",
+  rendererImport: () =>
+    import("~/components/lenses/renderers/RawLensRenderer.vue"),
+}
+
+export default rawLens

helpers/lenses/xmlLens.ts

@@ -1,8 +1,11 @@
-const xmlLens = {
+import { Lens } from "./lenses"
+
+const xmlLens: Lens = {
   lensName: "response.xml",
   isSupportedContentType: (contentType) => /\bxml\b/i.test(contentType),
   renderer: "xmlres",
-  rendererImport: () => import("~/components/lenses/renderers/XMLLensRenderer"),
+  rendererImport: () =>
+    import("~/components/lenses/renderers/XMLLensRenderer.vue"),
 }
 
 export default xmlLens

helpers/utils/StreamUtils.ts

@@ -8,9 +8,9 @@ import { map } from "rxjs/operators"
  *
  * @returns The constructed object observable
  */
-export function constructFromStreams<T>(
-  streamObj: { [key in keyof T]: Observable<T[key]> }
-): Observable<T> {
+export function constructFromStreams<T>(streamObj: {
+  [key in keyof T]: Observable<T[key]>
+}): Observable<T> {
   return combineLatest(Object.values<Observable<T[keyof T]>>(streamObj)).pipe(
     map((streams) => {
       const keys = Object.keys(streamObj) as (keyof T)[]

helpers/utils/__tests__/debounce.spec.js

@@ -1,38 +0,0 @@
-import debounce from "../debounce"
-
-describe("debounce", () => {
-  test("doesn't call function right after calling", () => {
-    const fn = jest.fn()
-
-    const debFunc = debounce(fn, 100)
-    debFunc()
-
-    expect(fn).not.toHaveBeenCalled()
-  })
-
-  test("calls the function after the given timeout", () => {
-    const fn = jest.fn()
-
-    jest.useFakeTimers()
-
-    const debFunc = debounce(fn, 100)
-    debFunc()
-
-    jest.runAllTimers()
-
-    expect(fn).toHaveBeenCalled()
-    // expect(setTimeout).toHaveBeenCalledWith(expect.any(Function), 100)
-  })
-
-  test("calls the function only one time within the timeframe", () => {
-    const fn = jest.fn()
-
-    const debFunc = debounce(fn, 1000)
-
-    for (let i = 0; i < 100; i++) debFunc()
-
-    jest.runAllTimers()
-
-    expect(fn).toHaveBeenCalledTimes(1)
-  })
-})

helpers/utils/__tests__/uri.spec.js

@@ -1,21 +0,0 @@
-import { parseUrlAndPath } from "../uri"
-
-describe("parseUrlAndPath", () => {
-  test("has url and path fields", () => {
-    const result = parseUrlAndPath("https://hoppscotch.io/")
-
-    expect(result).toHaveProperty("url")
-    expect(result).toHaveProperty("path")
-  })
-
-  test("parses out URL correctly", () => {
-    const result = parseUrlAndPath("https://hoppscotch.io/test/page")
-
-    expect(result.url).toBe("https://hoppscotch.io")
-  })
-  test("parses out Path correctly", () => {
-    const result = parseUrlAndPath("https://hoppscotch.io/test/page")
-
-    expect(result.path).toBe("/test/page")
-  })
-})

helpers/utils/b64.ts

@@ -1,4 +1,4 @@
-export const decodeB64StringToArrayBuffer = (input) => {
+export function decodeB64StringToArrayBuffer(input: string): ArrayBuffer {
   const bytes = Math.floor((input.length / 4) * 3)
   const ab = new ArrayBuffer(bytes)
   const uarray = new Uint8Array(ab)

helpers/utils/debounce.js

@@ -1,15 +0,0 @@
-// Debounce is a higher order function which makes its enclosed function be executed
-// only if the function wasn't called again till 'delay' time has passed, this helps reduce impact of heavy working
-// functions which might be called frequently
-// NOTE : Don't use lambda functions as this doesn't get bound properly in them, use the 'function (args) {}' format
-const debounce = (func, delay) => {
-  let inDebounce
-  return function () {
-    const context = this
-    const args = arguments
-    clearTimeout(inDebounce)
-    inDebounce = setTimeout(() => func.apply(context, args), delay)
-  }
-}
-
-export default debounce

helpers/utils/string.js

@@ -1,12 +0,0 @@
-export function getSourcePrefix(source) {
-  const sourceEmojis = {
-    // Source used for info messages.
-    info: "\tℹ️ [INFO]:\t",
-    // Source used for client to server messages.
-    client: "\t⬅️ [SENT]:\t",
-    // Source used for server to client messages.
-    server: "\t➡️ [RECEIVED]:\t",
-  }
-  if (Object.keys(sourceEmojis).includes(source)) return sourceEmojis[source]
-  return ""
-}

helpers/utils/string.ts

@@ -0,0 +1,12 @@
+const sourceEmojis = {
+  // Source used for info messages.
+  info: "\tℹ️ [INFO]:\t",
+  // Source used for client to server messages.
+  client: "\t⬅️ [SENT]:\t",
+  // Source used for server to client messages.
+  server: "\t➡️ [RECEIVED]:\t",
+}
+
+export function getSourcePrefix(source: keyof typeof sourceEmojis) {
+  return sourceEmojis[source]
+}

helpers/utils/uri.js

@@ -1,15 +0,0 @@
-export function parseUrlAndPath(value) {
-  const result = {}
-  try {
-    const url = new URL(value)
-    result.url = url.origin
-    result.path = url.pathname
-  } catch (e) {
-    const uriRegex = value.match(
-      /^((http[s]?:\/\/)?(<<[^/]+>>)?[^/]*|)(\/?.*)$/
-    )
-    result.url = uriRegex[1]
-    result.path = uriRegex[4]
-  }
-  return result
-}