Summary

• Deleted packages/server/lib/util/chrome_policy_check.js and replaced it with a fully-typed chrome_policy_check.ts
• Converted the test file test/unit/util/chrome_policy_check.js → .ts, switching from require() to ES module imports and describe in place of context
• Updated the import in packages/server/lib/modes/run.ts to use the named run export from the new module

Changes

chrome_policy_check.ts (new, replaces .js)

• Uses ES module imports instead of require()
• Adds a local RegistryValueWithPath intersection type (RegistryValue & { fullPath: string }) to properly extend the readonly registry-js union type instead of mutating it in place
• Types POLICY_KEYS as string[] and POLICY_HKEYS as HKEY[]
• Types warnIfPolicyMatches parameters — warningName uses Parameters<typeof errors.get>[0] to satisfy the keyof AllCypressErrorObj constraint without importing the large union directly
• Replaces the lodash chain .find().get() (which TS cannot type-check correctly) with _.find()?.fullPath and a type-predicate .filter()
• getAllPolicies now spreads each RegistryValue into a new RegistryValueWithPath object instead of mutating the readonly entry
• The platform guard (os.platform() !== 'win32') is expressed as an early return inside the exported run() function rather than a top-level if block with a try/catch around a dynamic require()

modes/run.ts

• Updates the import to use the named run export and calls it directly as runChromePolicyCheck() (the onWarning callback is now wired internally via the module's run export)

test/unit/util/chrome_policy_check.ts

• Converted from CommonJS to ES module imports
• context → describe (consistent with the rest of the test suite)

Test plan

• Existing unit tests in packages/server/test/unit/util/chrome_policy_check.ts cover all three branches (policy matched, no policy, enumerateValues throws)
• Run yarn workspace @packages/server test-unit -- packages/server/test/unit/util/chrome_policy_check.ts to verify

Made with Cursor