playthrough 0.4.3 → 0.5.0

package/src/playwright.config.js

@@ -1,4 +1,4 @@
-import { d as waitsFor } from "./flows-BkG0TgHA.js";
+import { f as waitsFor } from "./flows-DDB6AdQ-.js";
 import { availableParallelism } from "node:os";
 import { defineConfig } from "@playwright/test";
 import { fileURLToPath } from "node:url";

package/src/{reporter-DE8aueAO.js → reporter-Dr7dWwSf.js}

@@ -4,6 +4,7 @@ import { createHash } from "node:crypto";
 import { join, relative } from "node:path";
 // ../review/src/model.ts
 const text = (error) => z.string({ error }).refine((value) => value.trim() !== "", { error });
+const id = (error) => z.string({ error }).regex(/^[\w-]+$/, { error });
 const seconds$1 = (error) => z.number({ error }).min(0, { error });
 const share = (error) => z.number({ error }).min(0, { error }).max(1, { error });
 const fileName = (error) => z.string({ error }).regex(/^[\w.-]+(\/[\w.-]+)*$/, { error }).refine((name) => !name.split("/").some((part) => part === "." || part === ".."), { error });
@@ -95,7 +96,7 @@ const Reply = z.strictObject({
 	watch: text("needs a step's title").optional().describe("For an answer that moves: the step of its flow it's seen in, which the note then plays")
 });
 const Note = z.strictObject({
-	id: text("needs an id"),
+	id: id("needs an id of letters, digits and dashes"),
 	on: Place.optional().describe("Without it, about the work in general"),
 	words: text("needs words")
 });
@@ -115,7 +116,7 @@ const Verdict = z.strictObject({
 	words: z.string().optional()
 }).refine((verdict) => verdict.verdict !== "not quite" || verdict.words?.trim(), { error: "is not quite, so it needs words saying what's still wrong" });
 const Send = z.strictObject({
-	id: text("needs an id").describe("Chosen by the sender, so sending twice sends once"),
+	id: id("needs an id of letters, digits and dashes").describe("Chosen by the sender, so sending twice sends once"),
 	round: z.number().int().min(0),
 	by: text("needs to say who sent it"),
 	sent: z.iso.datetime({ error: "needs to say when it was sent, as an ISO date" }),
@@ -485,7 +486,10 @@ var Reviews = class {
 	async get(id) {
 		if (problemsWithId(id).length) throw new Refused(problemsWithId(id));
 		const about = await this.#readJson(`${id}/review.json`);
-		if (!about) throw new Refused([`There's no review called “${id}”.`]);
+		if (!about) {
+			const others = await this.list();
+			throw new Refused([`There's no review called “${id}”.${others.length ? ` The reviews here: ${list(others.map(quote))}.` : ""}`]);
+		}
 		const numbers = (await this.storage.list(`${id}/rounds`)).map(Number).filter(Number.isInteger).sort((a, b) => a - b);
 		const rounds = await Promise.all(numbers.map((n) => this.#readJson(`${id}/rounds/${n}/round.json`)));
 		const comparisons = await Promise.all(numbers.map(async (n) => {
@@ -673,7 +677,12 @@ var Reviews = class {
 	}
 	async #readJson(path) {
 		const bytes = await this.storage.read(path);
-		return bytes && JSON.parse(new TextDecoder().decode(bytes));
+		if (!bytes) return void 0;
+		try {
+			return JSON.parse(new TextDecoder().decode(bytes));
+		} catch (error) {
+			throw new Refused([`The review's file ${path} isn't JSON: ${error.message}. Put it right, then run this again.`]);
+		}
 	}
 	#writeJson(path, value) {
 		return this.storage.write(path, new TextEncoder().encode(`${JSON.stringify(value, null, 2)}\n`));
@@ -918,4 +927,4 @@ var Flows = class {
 	}
 };
 const seconds = (ms) => Math.round(Math.max(0, ms)) / 1e3;
-export { list as A, conversation as C, problemsFinishing as D, notesToAnswer as E, time as M, roundInProgress as O, allNotes as S, lastSeen as T, readingsOn as _, GLIMPSES as a, Refused as b, PERSON as c, RECORDING as d, WROTE as f, readingsOf as g, hashOf as h, Flows as i, quote as j, toAccountFor as k, PICTURE as l, glimpsesOf as m, ASSETS as n, HANDS as o, firstLine as p, DONE_ON as r, HOLD as s, ACTIONS as t, READINGS as u, seconds as v, inOrderJoined as w, Reviews as x, keepAssets as y };
+export { quote as A, inOrderJoined as C, roundInProgress as D, problemsFinishing as E, toAccountFor as O, conversation as S, notesToAnswer as T, seconds as _, GLIMPSES as a, Reviews as b, PERSON as c, RECORDING as d, WROTE as f, readingsOn as g, readingsOf as h, Flows as i, time as j, list as k, PICTURE as l, glimpsesOf as m, ASSETS as n, HANDS as o, firstLine as p, DONE_ON as r, HOLD as s, ACTIONS as t, READINGS as u, keepAssets as v, lastSeen as w, allNotes as x, Refused as y };

package/src/reporter.js

@@ -1 +1 @@
-export { t as ACTIONS, n as ASSETS, r as DONE_ON, a as GLIMPSES, o as HANDS, s as HOLD, c as PERSON, l as PICTURE, u as READINGS, d as RECORDING, f as WROTE, i as default, p as firstLine, m as glimpsesOf, h as hashOf, g as readingsOf, _ as readingsOn, v as seconds } from "./reporter-DE8aueAO.js";
+export { t as ACTIONS, n as ASSETS, r as DONE_ON, a as GLIMPSES, o as HANDS, s as HOLD, c as PERSON, l as PICTURE, u as READINGS, d as RECORDING, f as WROTE, i as default, p as firstLine, m as glimpsesOf, h as readingsOf, g as readingsOn, _ as seconds } from "./reporter-Dr7dWwSf.js";

package/src/test.js

@@ -1,5 +1,5 @@
-import { a as GLIMPSES, c as PERSON, d as RECORDING, f as WROTE, l as PICTURE, n as ASSETS, o as HANDS, p as firstLine, r as DONE_ON, s as HOLD, t as ACTIONS, u as READINGS, v as seconds, y as keepAssets } from "./reporter-DE8aueAO.js";
-import { C as firstOf, D as setupIn, E as personIn, O as repo, S as changes, T as make, n as REACHES_OUTSIDE, r as WALKED, t as ALONGSIDE } from "./flows-BkG0TgHA.js";
+import { _ as seconds, a as GLIMPSES, c as PERSON, d as RECORDING, f as WROTE, l as PICTURE, n as ASSETS, o as HANDS, p as firstLine, r as DONE_ON, s as HOLD, t as ACTIONS, u as READINGS, v as keepAssets } from "./reporter-Dr7dWwSf.js";
+import { A as repo, C as changes, D as personIn, E as make, O as setupIn, n as REACHES_OUTSIDE, r as WALKED, t as ALONGSIDE, w as firstOf } from "./flows-DDB6AdQ-.js";
 import { n as readPage, r as unhovered } from "./read-Bg9XVTtn.js";
 import { readFileSync, writeFileSync } from "node:fs";
 import { join } from "node:path";

package/types/cli/src/flows.d.ts

@@ -1,6 +1,7 @@
 import { type Did, type Flow } from "../../review/src/index.js";
 import { type Failed, type Written } from "./reporter.js";
 import { type App, type Repo } from "./repo.js";
+export declare const recorder: string;
 export type Run = {
     flows: string;
     only?: string[];

package/types/cli/src/repo.d.ts

@@ -3,6 +3,7 @@ export type App = {
     start: string;
     url: string;
     oneAtATime?: boolean;
+    port?: number;
 };
 export type Repo = ReturnType<typeof repo>;
 export declare function repo(root?: string): {
@@ -21,3 +22,6 @@ export declare function repo(root?: string): {
     keepOutOfGit(): Promise<void>;
     app(): Promise<App>;
 };
+export declare const PORTED = "{ \"start\": \"npm run dev -- --port {port}\", \"url\": \"http://localhost:{port}\" }";
+export declare function copies(root: string): string[];
+export declare const inWorktree: (root: string) => boolean;

package/types/cli/src/reporter.d.ts

@@ -37,7 +37,6 @@ export type Written = {
     file: string;
     hash: string;
 };
-export declare const hashOf: (file: string) => string | undefined;
 export default class Flows implements Reporter {
     #private;
     constructor({ folder, flows, before, taken }: {