cleanup: strip wave-reference comments + slop JSDoc

Remove internal milestone labels (Wave-1, W1-A, W2-D, W3-C, etc.) from
code comments and JSDoc across personality, scenario-runner, app-lifeops,
and plugin-health. Remove duplicate PersonalityStore.clearAll() method
(dead code — no callers; clear() is the used path). Strip narration from
judge.ts parseJudgeJson comment. Update section headers in contract-stubs
and default-packs/index to reflect current state rather than pending work.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Author: lalalune

packages/core/src/features/advanced-capabilities/personality/services/personality-store.ts

@@ -32,9 +32,8 @@ function clone(slot: PersonalitySlot): PersonalitySlot {
 /**
  * Structured store for personality slots (user + global) and named profiles.
  *
- * Skeleton-quality persistence: state is held in-memory and mirrored as
- * agent memories so it survives a runtime reload. A future Wave-3 followup
- * will move this to a dedicated table with proper indexes.
+ * Persistence is in-memory, mirrored as agent memories so state survives a
+ * runtime reload.
  */
 export class PersonalityStore extends Service {
 	static serviceType = PersonalityServiceType.PERSONALITY_STORE;
@@ -300,16 +299,6 @@ export class PersonalityStore extends Service {
 		return { before, after };
 	}
 
-	/**
-	 * Clear every in-memory slot and audit entry, leaving the named-profile
-	 * registry intact. Used by the benchmark server between scenarios so
-	 * personality state from one scenario does not bleed into the next.
-	 */
-	clearAll(): void {
-		this.slots.clear();
-		this.audit = [];
-	}
-
 	async stop(): Promise<void> {
 		logger.debug("PersonalityStore stopped");
 	}

packages/scenario-runner/src/judge.ts

@@ -56,8 +56,6 @@ function judgeResponseToResult(
 }
 
 function parseJudgeJson(raw: string): JudgeResult | null {
-  // Kept for the non-Cerebras path (runtime.useModel fallback). Uses the
-  // same tolerant parser the shared CerebrasJudge transport uses.
   const balanced = extractBalancedJsonObject(raw);
   if (!balanced) return null;
   let parsed: Record<string, unknown>;

plugins/app-lifeops/src/default-packs/contract-stubs.ts

@@ -1,30 +1,15 @@
 /**
- * Wave-1 contract stubs for default-pack consumers.
+ * Canonical contract types for default-pack consumers.
  *
- * These types mirror the **frozen** signatures in
- * `eliza/plugins/app-lifeops/docs/audit/wave1-interfaces.md` so this agent
- * (W1-D) can ship default packs without blocking on W1-A's spine landing
- * first.
+ * These types define the frozen `ScheduledTask`, `ScheduledTaskRunner`,
+ * `AnchorConsolidationPolicy`, `RecentTaskStatesProvider`,
+ * `RelationshipStore`, and `ConnectorRegistry` shapes used by all default
+ * packs. Do not edit these definitions without updating all pack consumers.
  *
- * **STUB STATUS:**
- *   - `ScheduledTask`, `ScheduledTaskRunner`, `AnchorConsolidationPolicy`,
- *     `EscalationStep`, `TerminalState`, `GateDecision` come from W1-A's
- *     `src/lifeops/scheduled-task/types.ts` once it lands.
- *   - `RecentTaskStatesProvider` comes from W1-C.
- *   - `RelationshipStore` / `RelationshipFilter` come from W1-E.
- *   - `ConnectorRegistry` / `ConnectorContribution` come from W1-F.
- *
- * Once those modules ship, this file flips to `export type ... from
- * "<owner-module>"` re-exports (the integration gate in §4 of
- * `IMPLEMENTATION_PLAN.md` enforces the swap).
- *
- * The on-the-wire shapes here are copies of the wave1-interfaces.md `§1`,
- * `§2.3`, `§3.x`, `§4` definitions — do not edit them in place. If the
- * upstream contract changes, the integration gate routes the change through
- * W1-A / W1-C / W1-E / W1-F first.
+ * Reference: `docs/audit/wave1-interfaces.md`.
  */
 
-// -- §1 W1-A — ScheduledTask --
+// -- §1 ScheduledTask --
 
 export type TerminalState =
   | "completed"
@@ -175,7 +160,7 @@ export interface ScheduledTask {
   metadata?: Record<string, unknown>;
 }
 
-// W1-D ships pack records as the input to `ScheduledTaskRunner.schedule`,
+// Pack records are the input to `ScheduledTaskRunner.schedule`,
 // i.e. `Omit<ScheduledTask, "taskId" | "state">`. This alias is the canonical
 // "default-pack record" type.
 export type ScheduledTaskSeed = Omit<ScheduledTask, "taskId" | "state">;
@@ -190,7 +175,7 @@ export interface AnchorConsolidationPolicy {
   sortBy?: "priority_desc" | "fired_at_asc";
 }
 
-// -- §3.4 default escalation ladders (frozen in wave1-interfaces.md §3.4) --
+// -- §3.4 default escalation ladders --
 
 export type DefaultEscalationLadderKey =
   | "priority_low_default"
@@ -201,7 +186,7 @@ export interface EscalationLadder {
   steps: EscalationStep[];
 }
 
-// -- §4.4 RecentTaskStatesProvider (W1-C) --
+// -- §4.4 RecentTaskStatesProvider --
 
 export interface RecentTaskStatesSummary {
   summary: string;
@@ -221,7 +206,7 @@ export interface RecentTaskStatesProvider {
   }): Promise<RecentTaskStatesSummary>;
 }
 
-// -- §2.3 RelationshipStore (W1-E) — only the surface W1-D consumes --
+// -- §2.3 RelationshipStore --
 
 export interface RelationshipStateStub {
   lastObservedAt?: string;
@@ -261,7 +246,7 @@ export interface RelationshipStoreStub {
   list(filter?: RelationshipFilterStub): Promise<RelationshipStub[]>;
 }
 
-// -- §3.1 ConnectorRegistry (W1-F) — only the surface W1-D consumes --
+// -- §3.1 ConnectorRegistry --
 
 export interface ConnectorContributionStub {
   kind: string;

plugins/app-lifeops/src/default-packs/escalation-ladders.ts

@@ -1,11 +1,7 @@
 /**
- * Default escalation ladders registered by W1-D.
+ * Default escalation ladders.
  *
- * Frozen by wave1-interfaces.md §3.4. Wave-1 ships the ladder data here;
- * once W1-F's `src/lifeops/escalation-ladders.ts` lands, this module flips
- * to a re-export per the integration gate (`IMPLEMENTATION_PLAN.md` §4).
- *
- * Ladders shipped:
+ * Frozen shapes per `docs/audit/wave1-interfaces.md` §3.4.
  *
  * ```
  * priority_low_default:    { steps: [] }
@@ -16,9 +12,6 @@
  *   { delayMinutes: 45, channelKey: "imessage", intensity: "urgent" },
  * ]}
  * ```
- *
- * Stub status: see `contract-stubs.ts` — `EscalationStep` /
- * `EscalationLadder` types are local until W1-A's `escalation.ts` lands.
  */
 
 import type {

plugins/app-lifeops/src/default-packs/index.ts

@@ -1,15 +1,10 @@
 /**
- * Default-pack registration entry point (W1-D).
+ * Default-pack registration entry point.
  *
- * The W1-A spine consumes `getAllDefaultPacks()` to seed records on first-run.
- * First-run customize consumes `getOfferedDefaultPacks()` to render the
- * pick-list. The lint script (`scripts/lint-default-packs.mjs`) consumes
- * `getAllDefaultPacks()` and runs `lintPacks()` against the result.
- *
- * Stub status: see `contract-stubs.ts` — all imported types are local Wave-1
- * stubs that match the frozen wave1-interfaces.md signatures. Once W1-A,
- * W1-C, W1-E, W1-F land, the stubs flip to re-exports from the owner
- * modules; consumers of this index do not change.
+ * `getAllDefaultPacks()` is consumed by the runner spine to seed records on
+ * first-run. `getOfferedDefaultPacks()` renders the first-run pick-list. The
+ * lint script (`scripts/lint-default-packs.mjs`) consumes `getAllDefaultPacks()`
+ * and runs `lintPacks()` against the result.
  */
 
 export type {
@@ -95,10 +90,10 @@ import {
 import type { DefaultPack } from "./registry-types.js";
 
 /**
- * The canonical list of W1-D default packs in the order they are offered.
+ * The canonical list of default packs in the order they are offered.
  *
- * `plugin-health` (W1-B) registers its own packs (`bedtime`, `wake-up`,
- * `sleep-recap`) through the same registry — they do not appear here.
+ * `plugin-health` registers its own packs (`bedtime`, `wake-up`, `sleep-recap`)
+ * through the same registry — they do not appear here.
  */
 export const DEFAULT_PACKS: ReadonlyArray<DefaultPack> = [
   dailyRhythmPack,

plugins/app-lifeops/src/lifeops/checkin/types.ts

@@ -83,10 +83,9 @@ export interface CheckinCollectorErrors {
  * sleep stats today, and the night path falls back gracefully when the
  * baseline has fewer than 5 episodes (`baseline === null`).
  *
- * Moved to `@elizaos/plugin-health` in Wave-1 (W1-B). Re-exported here for
- * backward compatibility with the existing `from "./types.js"` importers,
- * and brought into local scope for downstream `SleepRecap | null` usages
- * inside this file.
+ * Canonical home: `@elizaos/plugin-health`. Re-exported here for backward
+ * compatibility with `from "./types.js"` importers, and brought into local
+ * scope for downstream `SleepRecap | null` usages inside this file.
  */
 import type { SleepRecap } from "@elizaos/plugin-health";
 

plugins/app-lifeops/src/lifeops/feature-flags.types.ts

@@ -49,7 +49,7 @@ export type LifeOpsFeatureKey =
  * `FeatureFlagRegistry` is the source of truth for what flags exist —
  * including 3rd-party plugin contributions — so anywhere that ingests a
  * key from outside (chat input, REST routes, dev-registries view) uses
- * `LifeOpsFeatureFlagKey`. Parallel to W2-D's
+ * `LifeOpsFeatureFlagKey`. Parallel to
  * `LifeOpsBusFamily = LifeOpsTelemetryFamily | string`.
  */
 export type LifeOpsFeatureFlagKey = LifeOpsFeatureKey | string;

plugins/app-lifeops/src/lifeops/identity-observations/observer.ts

@@ -1,15 +1,11 @@
 /**
  * Identity-observation pipeline routing through EntityStore + RelationshipStore.
  *
- * Replaces the prior `identity-observations.ts` planner-flow (kept in-place
- * with a deletion deferred to W2-D). New code should call `observeIdentity`
- * here; legacy code paths still compile against the original module.
- *
- * The observer takes platform-side observations (gmail-sender, phone-contact,
- * chat-identity, etc.), normalizes them via the existing
- * `normalizeIdentityObservation` helpers, and routes the result through:
+ * Takes platform-side observations (gmail-sender, phone-contact, chat-identity,
+ * etc.), normalizes them via `normalizeIdentityObservation` helpers, and routes
+ * the result through:
  *   1. `EntityStore.observeIdentity` — collapses identities by (platform, handle)
- *   2. `RelationshipStore.observe`     — strengthens edges that already exist
+ *   2. `RelationshipStore.observe`   — strengthens edges that already exist
  */
 
 import type { EntityStore } from "../entities/store.js";

plugins/app-lifeops/src/lifeops/schema.ts

@@ -34,7 +34,6 @@ import {
 export const appLifeopsPgSchema = pgSchema("app_lifeops");
 
 // ---------------------------------------------------------------------------
-// Wave 1+ additions — relationships, X read, screen time, scheduling.
 // All life_* prefix, text IDs, ISO timestamps.
 // ---------------------------------------------------------------------------
 

plugins/app-lifeops/src/lifeops/seed-routines.ts

@@ -1,19 +1,8 @@
 /**
  * Seed routine templates offered during first-run onboarding.
  *
- * **Wave-1 transitional alias:** the canonical source of truth for the 8
- * habit starters is `src/default-packs/habit-starters.ts` (W1-D). This file
- * preserves the legacy `RoutineSeedTemplate` shape (rooted in
- * `CreateLifeOpsDefinitionRequest`) so existing callers
- * (`service-mixin-definitions.ts`, `client-lifeops.ts`,
- * `service-mixin-reminders.ts`) compile.
- *
- * Wave-2 (W2-A migration) deletes this file entirely once those callers
- * move to the `ScheduledTaskRunner` API and read from
- * `default-packs/habit-starters.ts` directly.
- *
- * Do **not** add new templates here — author them in
- * `default-packs/habit-starters.ts` instead.
+ * Canonical source of truth for habit starters: `src/default-packs/habit-starters.ts`.
+ * Do not add new templates here — author them there instead.
  */
 
 import type { CreateLifeOpsDefinitionRequest } from "../contracts/index.js";
@@ -25,11 +14,8 @@ import {
 
 /**
  * Canonical title for the seeded stretch routine. Exported so the
- * reminder dispatch loop can apply stretch-specific gating (busy-day
- * skip, walk-out reset) without grep-matching string literals.
- *
- * Wave-2: replaced by `task.shouldFire.kind === "stretch.walk_out_reset"`
- * (see GAP §2.7 + IMPL §3.4 — title-string match goes away).
+ * reminder dispatch loop can apply stretch-specific gating without
+ * grep-matching string literals.
  */
 export const STRETCH_ROUTINE_TITLE = "Stretch";