Moved takeSnapshot into test-case-recorder package (#2514)

## Checklist

- [/] I have added
[tests](https://www.cursorless.org/docs/contributing/test-case-recorder/)
- [/] I have updated the
[docs](https://github.com/cursorless-dev/cursorless/tree/main/docs) and
[cheatsheet](https://github.com/cursorless-dev/cursorless/tree/main/cursorless-talon/src/cheatsheet)
- [/] I have not broken the cheatsheet

packages/common/src/index.ts

@@ -9,7 +9,6 @@ export { HatStability } from "./ide/types/HatStability";
 export * from "./util";
 export * from "./ide/util/messages";
 export { getKey, splitKey } from "./util/splitKey";
-export { hrtimeBigintToSeconds } from "./util/timeUtils";
 export * from "./util/disposableFrom";
 export * from "./util/camelCaseToAllDown";
 export { Notifier } from "./util/Notifier";

packages/cursorless-engine/src/index.ts

@@ -1,5 +1,4 @@
 export * from "./testUtil/plainObjectToTarget";
-export * from "./testUtil/takeSnapshot";
 export * from "./core/StoredTargets";
 export * from "./cursorlessEngine";
 export * from "./generateSpokenForm/defaultSpokenForms/surroundingPairsDelimiters";

packages/cursorless-vscode/src/constructTestHelpers.ts

@@ -15,10 +15,10 @@ import {
 import {
   StoredTargetMap,
   plainObjectToTarget,
-  takeSnapshot,
 } from "@cursorless/cursorless-engine";
 import { VscodeTestHelpers } from "@cursorless/vscode-common";
 import * as vscode from "vscode";
+import { takeSnapshot } from "@cursorless/test-case-recorder";
 import { VscodeFileSystem } from "./ide/vscode/VscodeFileSystem";
 import { VscodeIDE } from "./ide/vscode/VscodeIDE";
 import { toVscodeEditor } from "./ide/vscode/toVscodeEditor";

packages/test-case-recorder/src/TestCase.ts

@@ -24,10 +24,10 @@ import {
 import {
   extractTargetKeys,
   getPartialTargetDescriptors,
-  takeSnapshot,
   type StoredTargetMap,
 } from "@cursorless/cursorless-engine";
 import { pick } from "lodash-es";
+import { takeSnapshot } from "./takeSnapshot";
 
 export class TestCase {
   private languageId: string;

packages/test-case-recorder/src/TestCaseRecorder.ts

@@ -28,7 +28,6 @@ import {
   injectIde,
   SpokenFormGenerator,
   StoredTargetMap,
-  takeSnapshot,
 } from "@cursorless/cursorless-engine";
 import { getRecordedTestsDirPath, walkDirsSync } from "@cursorless/node-common";
 import { invariant } from "immutability-helper";
@@ -37,6 +36,7 @@ import * as fs from "node:fs";
 import { access, readFile } from "node:fs/promises";
 import * as path from "node:path";
 import { RecordTestCaseCommandOptions } from "./RecordTestCaseCommandOptions";
+import { takeSnapshot } from "./takeSnapshot";
 import { TestCase } from "./TestCase";
 
 const CALIBRATION_DISPLAY_DURATION_MS = 50;

packages/test-case-recorder/src/index.ts

@@ -1,2 +1,3 @@
-export * from "./TestCaseRecorder";
 export * from "./ScopeTestRecorder";
+export * from "./takeSnapshot";
+export * from "./TestCaseRecorder";

packages/cursorless-engine/src/testUtil/takeSnapshot.ts → packages/test-case-recorder/src/takeSnapshot.ts

@@ -2,16 +2,16 @@ import {
   ExcludableSnapshotField,
   ExtraContext,
   ExtraSnapshotField,
-  hrtimeBigintToSeconds,
   IDE,
   rangeToPlainObject,
   selectionToPlainObject,
   SerializedMarks,
+  storedTargetKeys,
   TestCaseSnapshot,
   TextEditor,
 } from "@cursorless/common";
-import { type StoredTargetMap } from "../core/StoredTargets";
-import { storedTargetKeys } from "@cursorless/common";
+import type { StoredTargetMap } from "@cursorless/cursorless-engine";
+import { hrtimeBigintToSeconds } from "./timeUtils";
 
 export async function takeSnapshot(
   storedTargets: StoredTargetMap | undefined,