feat(init): Seed the initial active annotation id (#461)

* feat(init): Seed the initial active annotation id

* chore: add unit tests

* chore: adding some ??

* chore: remove default value for fileOptions

* chore: pr comments

* chore: remove unused type

Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>

Author: Conrad Chan

src/common/BaseAnnotator.ts

@@ -9,6 +9,14 @@ import './BaseAnnotator.scss';
 
 export type Container = string | HTMLElement;
 
+export type FileOptions = {
+    [key: string]: {
+        annotations?: {
+            activeId?: string;
+        };
+    };
+};
+
 export type Options = {
     apiHost: string;
     container: Container;
@@ -19,6 +27,7 @@ export type Options = {
         };
         permissions: Permissions;
     };
+    fileOptions?: FileOptions;
     hasTouch?: boolean;
     intl: IntlOptions;
     locale?: string;
@@ -36,8 +45,12 @@ export default class BaseAnnotator {
 
     store: store.AppStore;
 
-    constructor({ apiHost, container, file, intl, token }: Options) {
+    constructor({ apiHost, container, file, fileOptions, intl, token }: Options) {
+        const activeId = fileOptions?.[file.id]?.annotations?.activeId ?? null;
         const initialState = {
+            annotations: {
+                activeId,
+            },
             options: {
                 fileId: file.id,
                 fileVersionId: file.file_version.id,

src/common/__tests__/BaseAnnotator-test.ts

@@ -4,6 +4,7 @@ import BaseAnnotator from '../BaseAnnotator';
 import eventManager from '../EventManager';
 import { ANNOTATOR_EVENT } from '../../constants';
 import { Mode } from '../../store/common';
+import APIFactory from '../../api';
 
 jest.mock('../EventManager');
 jest.mock('../../api');
@@ -47,6 +48,32 @@ describe('BaseAnnotator', () => {
         }
     });
 
+    describe('constructor()', () => {
+        test.each`
+            optionsFileId | expectedActiveId
+            ${'12345'}    | ${'123'}
+            ${'987'}      | ${null}
+        `(
+            'should parse fileOptions for initial activeId and set it to $expectedActiveId',
+            ({ optionsFileId, expectedActiveId }) => {
+                const fileOptions = {
+                    [optionsFileId]: {
+                        annotations: { activeId: '123' },
+                    },
+                };
+                annotator = getAnnotator({ fileOptions });
+
+                expect(store.createStore).toHaveBeenLastCalledWith(
+                    {
+                        annotations: { activeId: expectedActiveId },
+                        options: { fileId: '12345', fileVersionId: '98765' },
+                    },
+                    { api: expect.any(APIFactory) },
+                );
+            },
+        );
+    });
+
     describe('destroy()', () => {
         test('should remove the base class name from the root element', () => {
             const rootEl = document.createElement('div');

src/store/createStore.ts

@@ -8,7 +8,11 @@ export type Options = {
     api?: API;
 };
 
-export default function createStore(preloadedState?: Partial<AppState>, { api }: Options = {}): AppStore {
+type RecursivePartial<T> = {
+    [P in keyof T]?: RecursivePartial<T[P]>;
+};
+
+export default function createStore(preloadedState?: RecursivePartial<AppState>, { api }: Options = {}): AppStore {
     const thunkOptions = {
         extraArgument: { api },
     };