fixup har recording

Author: nedtwigg

browser-extension/README.md

@@ -84,9 +84,9 @@ We maintain a corpus of test pages in two formats for testing the browser extens
   - `npx playwright codegen https://github.com/login --save-storage=playwright/.auth/gh.json` will store new auth tokens
     - login manually, then close the browser
     - ***these cookies are very sensitive! we only run this script using a test account that has no permissions or memberships to anything, recommend you do the same!***
-  - `pnpm run corpus:record:har` records new HAR files using those auth tokens (it needs args, run it with no args for docs)
+  - `pnpm run corpus:har:record` records new HAR files using those auth tokens (it needs args, run it with no args for docs)
     - DO NOT COMMIT AND PUSH NEW OR CHANGED HAR files!
-    - we try to sanitize these (see `har-record.ts` for details) but there may be important PII in them
+    - we try to sanitize these (see `corpus-har-record.ts` for details) but there may be important PII in them
     - if you need new HAR files for something, let us know and we will generate them ourselves using a dummy account
     - IF YOUR PR CHANGES OR ADDS HAR FILES WE WILL CLOSE IT. Ask for HAR files and we'll be happy to generate clean ones you can test against.
 

browser-extension/package.json

@@ -62,7 +62,7 @@
     "test": "vitest run",
     "playground": "vite --config vite.playground.config.ts",
     "playground:build": "vite build --config vite.playground.config.ts",
-    "corpus:record:har": "tsx tests/har-record.ts",
+    "corpus:har:record": "tsx tests/corpus-har-record.ts",
     "corpus:view": "tsx tests/corpus-view.ts"
   },
   "type": "module",

browser-extension/tests/har-record.ts renamed to browser-extension/tests/corpus-har-record.ts

@@ -65,7 +65,7 @@ function stripHeaders(headers?: any[]) {
 async function sanitize(filename: string) {
   console.log('Sanitizing:', filename)
 
-  const p = path.join('tests/corpus/har', filename)
+  const p = path.join('tests/corpus', filename)
   const har = JSON.parse(await fs.readFile(p, 'utf8'))
 
   for (const e of har.log?.entries ?? []) {
@@ -84,7 +84,7 @@ async function sanitize(filename: string) {
   await fs.writeFile(p, JSON.stringify(har, null, 2))
 }
 
-;(async () => {
+; (async () => {
   const pattern = process.argv[2]
 
   // If no argument provided, show available keys