refactor(CollectorInvite): drop manual #accessInfo short-circuit, delegate to lib cache

pryv@3.1.0 introduces a per-Connection accessInfo cache with the same
`forceRefresh` parameter the wrapper already exposed. The local
short-circuit `if (!forceRefresh && this.#accessInfo) return this.#accessInfo`
became redundant.

Kept:
- `#accessInfo` field as a sync-read mirror for the `patientUsername` getter
- `forceRefresh` parameter passed through to the lib (Phase 2 test fix
  shows real callers need this after revoke/delete)
- `invalid-access-token` → revokeInvite side effect

Plan 58 Phase 4a. 488/488 tests pass.

Author: perki

ts/appTemplates/CollectorInvite.ts

@@ -160,12 +160,16 @@ export class CollectorInvite {
   /**
    * Check if connection is valid. (only if active)
    * If result is "forbidden" update and set as revoked
-   * @returns accessInfo if valid.
+   *
+   * Caching is delegated to `pryv.Connection.accessInfo()` (per-Connection cache
+   * introduced in pryv@3.1.0). Pass `forceRefresh=true` to bust the cache —
+   * required after operations that invalidate the access (revoke, delete).
+   *
+   * The `#accessInfo` field mirrors the result for the sync `patientUsername` getter.
    */
   async checkAndGetAccessInfo (forceRefresh: boolean = false): Promise<any> {
-    if (!forceRefresh && this.#accessInfo) return this.#accessInfo;
     try {
-      this.#accessInfo = await this.connection.accessInfo();
+      this.#accessInfo = await this.connection.accessInfo(forceRefresh);
       return this.#accessInfo;
     } catch (e: any) {
       this.#accessInfo = null;