Bumps [@sentry/browser](https://github.com/getsentry/sentry-javascript) from 6.3.5 to 6.3.6.
- [Release notes](https://github.com/getsentry/sentry-javascript/releases)
- [Changelog](https://github.com/getsentry/sentry-javascript/blob/master/CHANGELOG.md)
- [Commits](https://github.com/getsentry/sentry-javascript/compare/6.3.5...6.3.6)
Signed-off-by: dependabot[bot] <support@github.com>

Bumps [prettier](https://github.com/prettier/prettier) from 2.2.1 to 2.3.0.
- [Release notes](https://github.com/prettier/prettier/releases)
- [Changelog](https://github.com/prettier/prettier/blob/main/CHANGELOG.md)
- [Commits](https://github.com/prettier/prettier/compare/2.2.1...2.3.0)
Signed-off-by: dependabot[bot] <support@github.com>

Bumps [katex](https://github.com/KaTeX/KaTeX) from 0.13.5 to 0.13.9.
- [Release notes](https://github.com/KaTeX/KaTeX/releases)
- [Changelog](https://github.com/KaTeX/KaTeX/blob/master/CHANGELOG.md)
- [Commits](https://github.com/KaTeX/KaTeX/compare/v0.13.5...v0.13.9)
Signed-off-by: dependabot[bot] <support@github.com>

Bumps [codecov](https://github.com/codecov/codecov-node) from 3.8.1 to 3.8.2.
- [Release notes](https://github.com/codecov/codecov-node/releases)
- [Changelog](https://github.com/codecov/codecov-node/blob/master/CHANGELOG.md)
- [Commits](https://github.com/codecov/codecov-node/compare/v3.8.1...v3.8.2)
Signed-off-by: dependabot[bot] <support@github.com>

Bumps [@babel/preset-env](https://github.com/babel/babel/tree/HEAD/packages/babel-preset-env) from 7.14.0 to 7.14.1.
- [Release notes](https://github.com/babel/babel/releases)
- [Changelog](https://github.com/babel/babel/blob/main/CHANGELOG.md)
- [Commits](https://github.com/babel/babel/commits/v7.14.1/packages/babel-preset-env)
Signed-off-by: dependabot[bot] <support@github.com>

Bumps [postcss](https://github.com/postcss/postcss) from 8.2.13 to 8.2.14.
- [Release notes](https://github.com/postcss/postcss/releases)
- [Changelog](https://github.com/postcss/postcss/blob/main/CHANGELOG.md)
- [Commits](https://github.com/postcss/postcss/compare/8.2.13...8.2.14)
Signed-off-by: dependabot[bot] <support@github.com>

Bumps [aws-sdk](https://github.com/aws/aws-sdk-js) from 2.897.0 to 2.903.0.
- [Release notes](https://github.com/aws/aws-sdk-js/releases)
- [Changelog](https://github.com/aws/aws-sdk-js/blob/master/CHANGELOG.md)
- [Commits](https://github.com/aws/aws-sdk-js/compare/v2.897.0...v2.903.0)
Signed-off-by: dependabot[bot] <support@github.com>

Bumps [eslint-config-hypothesis](https://github.com/hypothesis/frontend-toolkit) from 2.4.0 to 2.5.0.
- [Release notes](https://github.com/hypothesis/frontend-toolkit/releases)
- [Commits](https://github.com/hypothesis/frontend-toolkit/commits)
Signed-off-by: dependabot[bot] <support@github.com>

Bumps [gulp-replace](https://github.com/lazd/gulp-replace) from 1.1.2 to 1.1.3.
- [Release notes](https://github.com/lazd/gulp-replace/releases)
- [Commits](https://github.com/lazd/gulp-replace/compare/v1.1.2...v1.1.3)
Signed-off-by: dependabot[bot] <support@github.com>

Bumps [mocha](https://github.com/mochajs/mocha) from 8.3.2 to 8.4.0.
- [Release notes](https://github.com/mochajs/mocha/releases)
- [Changelog](https://github.com/mochajs/mocha/blob/master/CHANGELOG.md)
- [Commits](https://github.com/mochajs/mocha/compare/v8.3.2...v8.4.0)
Signed-off-by: dependabot[bot] <support@github.com>

Bumps [eslint](https://github.com/eslint/eslint) from 7.25.0 to 7.26.0.
- [Release notes](https://github.com/eslint/eslint/releases)
- [Changelog](https://github.com/eslint/eslint/blob/master/CHANGELOG.md)
- [Commits](https://github.com/eslint/eslint/compare/v7.25.0...v7.26.0)
Signed-off-by: dependabot[bot] <support@github.com>

Bumps [puppeteer](https://github.com/puppeteer/puppeteer) from 9.0.0 to 9.1.1.
- [Release notes](https://github.com/puppeteer/puppeteer/releases)
- [Changelog](https://github.com/puppeteer/puppeteer/blob/main/CHANGELOG.md)
- [Commits](https://github.com/puppeteer/puppeteer/compare/v9.0.0...v9.1.1)
Signed-off-by: dependabot[bot] <support@github.com>

Bumps [npm-packlist](https://github.com/npm/npm-packlist) from 2.1.5 to 2.2.2.
- [Release notes](https://github.com/npm/npm-packlist/releases)
- [Commits](https://github.com/npm/npm-packlist/compare/v2.1.5...v2.2.2)
Signed-off-by: dependabot[bot] <support@github.com>

Bumps [core-js](https://github.com/zloirock/core-js/tree/HEAD/packages/core-js) from 3.11.2 to 3.12.1.
- [Release notes](https://github.com/zloirock/core-js/releases)
- [Changelog](https://github.com/zloirock/core-js/blob/master/CHANGELOG.md)
- [Commits](https://github.com/zloirock/core-js/commits/v3.12.1/packages/core-js)
Signed-off-by: dependabot[bot] <support@github.com>

I removed a @ts-expect-error and tight some types.

I remove the returned `provider` property from the
`authStateFromProfile` function as it was not used.

Bumps [ua-parser-js](https://github.com/faisalman/ua-parser-js) from 0.7.23 to 0.7.28.
- [Release notes](https://github.com/faisalman/ua-parser-js/releases)
- [Commits](https://github.com/faisalman/ua-parser-js/compare/0.7.23...0.7.28)
Signed-off-by: dependabot[bot] <support@github.com>

Bumps [hosted-git-info](https://github.com/npm/hosted-git-info) from 2.8.8 to 2.8.9.
- [Release notes](https://github.com/npm/hosted-git-info/releases)
- [Changelog](https://github.com/npm/hosted-git-info/blob/v2.8.9/CHANGELOG.md)
- [Commits](https://github.com/npm/hosted-git-info/compare/v2.8.8...v2.8.9)
Signed-off-by: dependabot[bot] <support@github.com>

This ensures that the component doesn't attempt to call the prop and use
the result.

As noted in PR review, checking for a valid annotation first seems more
sensible.

Previously all components that used injected services did not have their
usage type-checked because `withServices` returned `any`.

This commit refactors the API of `withServices` to make it easier to
typecheck and adds types. The major API change is that the list of injected
services is now passed as a second argument to `withServices` rather
than by setting an `injectedProps` property on the component. This makes
it easy to infer that the component returned by `withServices(Widget,
['serviceA', 'serviceB'])` has the same props as `Widget` but without
`serviceA` or `serviceB`.

Making these changes also turned up a handful of mistakes in existing
types which are fixed here:

 - Correct `auth` type used by `HelpPanel`
 - Correct type of `thread` prop used by `ThreadCard`
 - Correct optionality of boolean arguments in callbacks to `Excerpt`
 - Change `showActions` logic in `Annotation` to make it more obvious
   to TS that the actions are only shown when there is an annotation
 - Remove unused props passed to `GroupList`

Additionally there was a type error with TopBar's `auth` prop which is
less trivial to fix, so I have punted it to a future commit.

The type of the `props` argument to `TopBar` was not specified, so its
props were not type-checked. Specifying a type for `props` turned up
several mistakes in other types.

 - Specify the type of the `props` object so that the props are actually
   type-checked
 - Add missing `status` field to `AuthState` type
 - Correctly set the optionality of various props

This reverts commit df0084c0.

Switch the markdown parser to the one recommended by the current Sphinx
documentation [1].

This fixes an error parsing Markdown code blocks in `adr-003.md` when
running `make checkdocs` and makes this file show up under "Architecture
Decision Records" in the generated docs.

The `requirements/docs.txt` file was updated by directly running
`.tox/docs/bin/pip-compile -r requirements/docs.in`.

[1] https://www.sphinx-doc.org/en/master/usage/markdown.html

 * This adds pinned versions of the docs dependencies, so they don't
 drift between local and CI
 * This is managed with `hdev requirements` so support for that has
 been added
 * Using the pyenv integration we have elsewhere so tox tracks the
 versions in the version file

9bec434d moved the logic for scrolling
to an anchor into the document integrations, in order to enable
custom logic depending on the document type. The bucket bar however
was still using a generic approach.

Change the bucket bar to use the same `scrollToAnchor` implementation as
when scrolling is triggered by clicking an annotation card in the
sidebar.

 - Add `scrollToAnchor` method to `Guest`
 - Modify bucket bar controller to use this method

Related to https://github.com/hypothesis/client/issues/3269

 - Rename `reanchoringWait` to `reanchoringMaxWait` to make it clear
   that this is the maximum time that we'll wait
 - Rename {initialOffset, targetOffset} => {startOffset, endOffset} for
   clarity
 - Add comment to explain a sleep

The scenario this function tests for is an anchor's highlight being
_inside_ a placeholder element rather than _being_ a placeholder
element.

When scrolling to an anchor in a page that has not been rendered by
PDF.js, the anchor actually references a placeholder element in the
middle of the page. In order to scroll to the correct location in the
page scrolling for such anchors needs to happen in three phases:

 1. Scroll to the approximate location of the final anchor, given by the
    placeholder anchor. This will trigger PDF.js to re-render the target
    page.
 2. Wait for PDF.js to finish re-rendering the page and for the client
    to finish re-anchoring the annotation.
 3. Scroll to the real/non-placeholder anchor.

Change `PDFIntegration#scrollToAnchor` to implement the above steps.

Fixes https://github.com/hypothesis/client/issues/3269

This uses the real `Intl.DateTimeFormat` API but with a fixed locale to
ensure consistent output across systems. This is a more complete test
than the previous one.

The test assumes that output is consistent across browsers for the date
given. In testing with Safari, Firefox and Chrome this was the case for
the en-US locale. We accept some small risk that the exact format might change
slightly in future releases of one of these browsers.

Previously `decayingInterval` exhibited reasonable behavior if the date
was invalid by never invoking the callback. However this was not easy to
see from the code and not tested. Add explicit checks in `nextFuzzyUpdate` to
make this more obvious.