Skip to content

Commit fcd196d

Browse files
d-csclaude
andcommitted
docs(mollifier): strip internal planning labels from comments
Remove plan-tracking shorthand (Q2/Q3 design, _plans/) from mutations-layer mollifier comments; reword to plain English. Comment-only; no behaviour change. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
1 parent 29ae2fa commit fcd196d

2 files changed

Lines changed: 4 additions & 5 deletions

File tree

apps/webapp/app/routes/api.v1.runs.$runParam.replay.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -61,7 +61,7 @@ export async function action({ request, params }: ActionFunctionArgs) {
6161

6262
try {
6363
const env = authenticationResult.environment;
64-
// PG-first. Replay works on any status per audit (Q2 design) — no
64+
// PG-first. Replay works on any status per audit — no
6565
// filter beyond friendlyId is the existing semantic; findFirst with
6666
// env scoping tightens it minimally without changing behaviour for
6767
// a correctly-authed caller.
@@ -73,7 +73,7 @@ export async function action({ request, params }: ActionFunctionArgs) {
7373
});
7474

7575
if (!taskRun) {
76-
// Buffered fallback (Q2). The SyntheticRun shape was extended in
76+
// Buffered fallback. The SyntheticRun shape was extended in
7777
// Phase B4 to carry every field ReplayTaskRunService reads from a
7878
// TaskRun. Validate the subset of fields the service consumes
7979
// (BufferedReplayInputSchema above) before casting; a schema

apps/webapp/app/v3/mollifier/mutateWithFallback.server.ts

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ import { prisma, $replica } from "~/db.server";
88
import { logger } from "~/services/logger.server";
99
import { getMollifierBuffer } from "./mollifierBuffer.server";
1010

11-
// Wait/retry knobs per Q3 design. Exported for tests.
11+
// Wait/retry knobs. Exported for tests.
1212
export const DEFAULT_SAFETY_NET_MS = 2_000;
1313
// Initial gap between buffer polls; grows by BACKOFF_FACTOR up to
1414
// DEFAULT_MAX_POLL_STEP_MS so a slow drain doesn't poll at a tight fixed
@@ -50,8 +50,7 @@ export type MutateWithFallbackOutcome<TResponse> =
5050
| { kind: "not_found" }
5151
| { kind: "timed_out" };
5252

53-
// PG-first → buffer mutateSnapshot → wait-and-bounce. Implements the Q3
54-
// design (`_plans/2026-05-19-mollifier-mutation-race-design.md`). The
53+
// PG-first → buffer mutateSnapshot → wait-and-bounce. The
5554
// caller decides how to translate the outcome into an HTTP response —
5655
// this helper never throws Response objects so it remains route-agnostic
5756
// and unit-testable in isolation.

0 commit comments

Comments
 (0)