@impeccable/reviews 0.1.31 → 0.1.32

package/dist/hosting.js

@@ -33,7 +33,24 @@ export function hostingOf(state, repo) {
         return [{ key, liveUrl: address(a.liveUrl), previews }];
     });
     const unread = Array.isArray(hosting.unread) ? hosting.unread.flatMap((u) => text(u) ?? []) : [];
-    return { apps, readAt: text(hosting.readAt), unread };
+    return { apps, readAt: text(hosting.readAt), unread, ...(hosting.readPending === true ? { readPending: true } : {}) };
+}
+/**
+ * The record once it is whole: while the state says a read of it is pending, the state is read again every few seconds,
+ * up to a minute by the clock, each read bounded by what is left of it. Past that, its previews are left out and it
+ * counts as never read, so each host tool reads its previews itself, as it does without a record; the live addresses stay.
+ */
+export async function settledHosting(first, repo, readState, opts = {}) {
+    const { everyMs = 3_000, upToMs = 60_000, sleep = (ms) => new Promise((resolve) => setTimeout(resolve, ms)), now = Date.now } = opts;
+    const deadline = now() + upToMs;
+    let hosting = hostingOf(first, repo);
+    while (hosting?.readPending && now() + everyMs < deadline) {
+        await sleep(everyMs);
+        const state = await readState(deadline - now());
+        if (state.ok)
+            hosting = hostingOf(state.data, repo) ?? hosting;
+    }
+    return hosting?.readPending ? { apps: hosting.apps.map((a) => ({ ...a, previews: [] })), readAt: null, unread: hosting.unread } : hosting;
 }
 /** The reports each host's previews are read from, besides the deployments every read takes. */
 const REPORTS = { netlify: ['statuses', 'comments'], cloudflare: ['check-runs', 'comments'], render: [] };

package/dist/main.js

@@ -7,7 +7,7 @@ import { runAgent } from './agent.js';
 import { runSkillCommand } from './skill.js';
 import { fenceExtension, guardedFileOperations, hostsAgreed, placeOf, previewRetryState } from './fence.js';
 import { accountsSaved, focusInstructions, focusState, severalApps, signInItem } from './focus.js';
-import { hostingOf } from './hosting.js';
+import { settledHosting } from './hosting.js';
 import { requestJson } from './http.js';
 import { copyText, letTabsHear, openBrowser, progressPage, screenPrompt, setupAuth, signIn } from './login.js';
 import { checkoutJournal } from './leftovers.js';
@@ -234,6 +234,8 @@ export async function main(argv) {
         ]);
         if (!instructions.ok || !state.ok)
             throw new UserError(`Impeccable's setup API did not answer: ${instructions.error ?? state.error}.`);
+        // A hosting record behind GitHub is read again while the questions below are asked (hosting.ts `settledHosting`).
+        const hosting = settledHosting(state.data, token.repository.name, (timeoutMs) => requestJson(`${setupApi}/state`, { token: token.access_token, timeoutMs }, log));
         // When the checkout cannot tell, the setup state's own verdict decides. When it has a web front end Impeccable skipped, the person adds it first.
         if (stateSaysNoFrontEnd(state.data, token.repository.name)) {
             if (frontEnd === 'unknown')
@@ -395,7 +397,7 @@ export async function main(argv) {
                 sandbox,
                 pushAccess: push.outcome,
                 severalApps: severalApps(state.data),
-                hosting: hostingOf(state.data, token.repository.name),
+                hosting: await hosting,
                 ...(flags.app ? { app: flags.app } : {}),
                 start: checkout.start,
                 ...(flags.signedInTestDrive ? { signedInTestDrive: { accountsSaved: accountsSaved(state.data, token.repository.name) } } : {}),

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@impeccable/reviews",
-  "version": "0.1.31",
+  "version": "0.1.32",
   "description": "Set up a repository for Impeccable for PRs from your terminal.",
   "license": "SEE LICENSE IN LICENSE",
   "private": false,