Commit Graph

12629 Commits

Author SHA1 Message Date
Jordan Ritter 6e34ea836e fix(showcase): pydantic-ai gen-ui forwards the real conversation to its secondary call 2026-06-07 10:00:40 -07:00
Jordan Ritter 4920128332 fix(showcase): forward x-aimock-context across the AG-UI SSE-pump boundary in ms-agent-dotnet 2026-06-07 10:00:35 -07:00
Jordan Ritter e1cf704e13 fix(showcase): propagate x-aimock-context to off-thread gen-ui secondary call across Python backends 2026-06-07 10:00:29 -07:00
Jordan Ritter 46e66d8d4b fix(showcase): add missing per-pill copilotkit routes for agno/langroid/llamaindex/spring-ai/strands 2026-06-07 10:00:24 -07:00
Jordan Ritter 7bde1eef3a ci(showcase): pull Git LFS assets during build so demo images ship real binaries 2026-06-07 10:00:16 -07:00
Jordan Ritter 79d61b68ee docs(shell-docs): add shared-state/prebuilt guides, sidebar wiring, css, model-selection, whats-new 2026-06-06 16:10:13 -07:00
Jordan Ritter aade9d0e86 docs(shell-docs): add troubleshooting, migration, and concepts guides 2026-06-06 16:10:09 -07:00
Jordan Ritter 1c33e99f49 docs(shell-docs): add reference hook pages and retire orphaned v2 useAgent 2026-06-06 16:10:04 -07:00
Jordan Ritter e529593650 docs(shell-docs): add MS Agent Framework + Mastra integration samples 2026-06-06 16:10:00 -07:00
Jordan Ritter 58887baeda docs(shell-docs): add backend runtime/runner/self-managed docs 2026-06-06 16:09:56 -07:00
Jordan Ritter 19f8ef89c8 fix(react-core): correct useCoAgent JSDoc link and stale run description 2026-06-06 16:09:52 -07:00
Jordan Ritter 6f360b9f66 diag(showcase): ungated x-diag-probe (thread+ctx) to localize native-tool context loss (#5305)
TEMPORARY DIAGNOSTIC (to be reverted after reading). Adds one ungated
header `x-diag-probe: thread=<name>;ctx=<present|EMPTY>;keys=<n>` on
every outbound LLM call (before the context gate) across the 7 backends
whose native tool calls drop x-aimock-context. The aimock journal
(HTTP-readable) will then show, per dropping call, which thread it ran
on and whether the forwarded-headers contextvar was empty — pinning the
context-losing run-path. Additive, try/except-wrapped (never throws), no
change to existing forwarding/injection/gating. py_compile clean.
2026-06-06 12:35:58 -07:00
github-actions[bot] c73afd1e3d style: auto-fix formatting 2026-06-06 19:26:25 +00:00
Jordan Ritter 469d79b029 diag(showcase): ungated x-diag-probe (thread+ctx) on outbound LLM calls to localize native-tool context loss 2026-06-06 12:24:44 -07:00
Jordan Ritter ffb8af32b0 fix(showcase): restore sandboxed-UI jsFunctions in D6 gen-ui-open-advanced fixtures (#5294)
## Summary

The `generateSandboxedUi` calculator (and the ping-host /
inline-evaluator pills) rendered static HTML with no interactivity — the
`=` and keypad buttons were no-ops — because the affected
per-integration fixtures omitted the `jsFunctions` key that the
`OpenGenerativeUIRenderer` injects into the iframe to wire click
handlers to the host bridge
(`Websandbox.connection.remote.evaluateExpression` / `notifyHost`).

This copies the `jsFunctions` VERBATIM from the canonical
langgraph-typescript fixture (apples-to-apples) into agno, crewai-crews,
langgraph-fastapi, and mastra, and creates the missing langgraph-python
advanced fixture (Calculator + Ping pills; the inline evaluator already
lives in `gen-ui-open.json`). Fixture-only change; the renderer is
correct.

## Verification

Proven in a prior session against the D6 path.
2026-06-06 12:09:42 -07:00
Jordan Ritter 23b6fef129 fix(showcase): gate LGP AAPL tool-rendering fixture on toolName not hasToolResult (#5292)
## Summary

The D6 cell `tool-rendering-custom-catchall` for langgraph-python
deterministically timed out (30s no-response). The probe sends two turns
in one thread: turn 1 "weather in Tokyo" -> `get_weather`, turn 2
"What's the price of AAPL?" -> `get_stock_price`. The AAPL first-leg
fixture was gated on `hasToolResult:false`, but aimock implements that
gate as `messages.some(m => m.role === "tool")` — i.e. ANY tool message
anywhere in the thread. Turn 1's `get_weather` leaves a tool result, so
on turn 2 `hasToolResult` is permanently true and the false gate can
never match -> `no_fixture_match` -> 503 -> 30s timeout.

This replaces the broken `hasToolResult:false` gate with
`toolName:"get_stock_price"` (the same pattern the sibling weather/chain
first-leg fixtures already use). A `toolName` gate fires whenever the
tool is registered, regardless of prior tool results; the `toolCallId`
follow-up fixture still wins on iteration 2.

Fixture-only change.

## Verification

Proven in a prior session via the D6 Docker path:
`tool-rendering-custom-catchall` now green (turn 1 `get_weather` + turn
2 `get_stock_price` both render, no 503/timeout);
`tool-rendering-default-catchall` still green (no regression).
2026-06-06 12:09:38 -07:00
Jordan Ritter 1b50eacc98 feat(showcase): CVDIAG diagnostic instrumentation for x-aimock-context / CV-rung propagation (#5304)
## Summary
Diagnostic instrumentation to localize **where `x-aimock-context` is
dropped on the CV/D5 rung** (sustained fleet-wide aimock 503
`no_fixture_match`). Adds a uniform, durable, HTTP-readable per-hop
trace. **Instrumentation-only** — on non-diagnostic traffic every
forwarder is byte-identical to before (verified across all 22 breadcrumb
sites).

- **Shared contract + sink:** single-line `CVDIAG` log format (12-char
redacted header prefix), `x-diag-run-id`/`x-diag-hops` correlation +
breadcrumb headers, and a best-effort PocketBase `diag_events`
collection (anonymously HTTP-readable, so it's queryable mid-incident
without Railway logs).
- **Harness:** mint a run_id per D5/D6 run, inject the correlation
headers alongside `x-aimock-context`, and a timeout-bounded post-run
aimock-journal join that records a `cv-verdict` row tagged
`harness-d5`/`harness-d6` (localizing whether the header reached
aimock). Plus claim/worker_id/snapshot/pool-condition breadcrumbs and
surfaced previously-swallowed catches.
- **Per-framework forwarders** (LangGraph py/ts/fastapi, google-adk,
self-contained Node + 9 Python shims, spring-ai Java, ms-agent .NET ×2):
CVDIAG at each hop + a `route-<fw>`/`backend-<fw>` `x-diag-hops`
breadcrumb, gated on diagnostic-header presence. Surfaces
previously-silent forwarding misses (empty LangGraph `configurable`,
missing httpx event-hooks target, swallowed hook-install errors).

## Why
CV/D5 has been red ~42h fleet-wide; the aimock journal shows ~95% of
503s arrive with **no** `x-aimock-context`. Forwarding is divergent per
framework (LangGraph `configurable` SPOF; `claude-sdk-python` middleware
not wired; google-adk async path uses aiohttp, bypassing the httpx
hook). This makes the drop point visible per hop and durable, so the fix
can be made once — and it preserves apples-to-apples by adding a
*uniform* trace layer rather than more bespoke per-framework code.

## Test plan
- [x] Harness typecheck green; 2105/2105 harness tests pass
- [x] 7-agent CR converged (2 fix rounds); instrumentation-only
invariant verified across every outbound forwarder
- [ ] CI green (integration builds incl. Java/.NET)
- [ ] Post-deploy: read `/__aimock/journal` + PB `diag_events` to
localize the drop hop per framework and settle the open D5-vs-D6
question with data

🤖 Generated with [Claude Code](https://claude.com/claude-code)
2026-06-06 10:58:04 -07:00
github-actions[bot] 1e5a9b77cd style: auto-fix formatting 2026-06-06 17:44:07 +00:00
Jordan Ritter bd77954ab5 feat(showcase): instrument per-framework x-aimock-context forwarding with gated CVDIAG breadcrumb
Add CVDIAG logging + x-diag-hops breadcrumb (route-<fw>/backend-<fw>) at each forwarding hop across LangGraph (py/ts/fastapi), google-adk, the self-contained Node + Python shims, spring-ai (Java) and ms-agent (.NET). Breadcrumb append is gated on diagnostic-header presence so non-diagnostic traffic stays byte-identical; surfaces previously-silent forwarding misses (empty configurable, missing httpx event-hooks target, swallowed hook-install errors).
2026-06-06 10:40:25 -07:00
Jordan Ritter 32d0afaba0 feat(showcase-harness): instrument D5/D6 drivers, orchestrator, pool & scheduler with CVDIAG
Mint a run_id per D5/D6 feature run, inject x-diag-run-id + seed x-diag-hops=harness alongside x-aimock-context, and do a post-run aimock-journal join (timeout-bounded) that records a cv-verdict row (tagged harness-d5/harness-d6) localizing whether the context header reached aimock. Add claim/worker_id/snapshot/pool-condition breadcrumbs and surface previously-swallowed catches.
2026-06-06 10:40:25 -07:00
Jordan Ritter acf9f6fe2c feat(showcase-harness): add CVDIAG diagnostic contract + durable diag_events sink
Shared single-line CVDIAG log format (redacted 12-char header prefix), x-diag-run-id/x-diag-hops correlation header constants, and a best-effort PocketBase diag_events sink (anonymously HTTP-readable) so the CV/x-aimock-context propagation chain can be traced mid-incident without Railway log access.
2026-06-06 10:40:24 -07:00
Ben Taylor 0acc358fb6 fix(examples): correct Threads locked-state license command to copilotkit license (#5301)
## Summary

The Threads locked-state card in every integration example tells users
to add a CopilotKit Intelligence license with `copilotkit
add-intelligence`. That's the wrong command:

1. `add-intelligence` is intentionally not wired into the CLI dispatch
yet.
2. Even when implemented, it only drops the Intelligence overlay — it
does not issue a license (licensed behavior is deferred to ENT-725).

The command that actually issues a license key and writes it to `.env`
is **`copilotkit license`**.

## Changes

- `locked-state.tsx` in all 18 examples: `copilotkit add-intelligence` →
`copilotkit license` (oxfmt collapsed the now-shorter `<code>` element
onto one line)
- `threads-drawer.module.css` descriptive comments (11 examples):
"add-intelligence command" → "license command"
- `examples/integrations/_intelligence/` references to `copilotkit
add-intelligence` are untouched — those describe the overlay scaffold
command, not licensing.

## Verification

- `git grep add-intelligence -- 'examples/**'` only matches
`_intelligence/` scaffold docs
- `pnpm parity:check` — no new findings (threads-drawer/** is
allowedDivergence; only pre-existing next-env.d.ts warns)

Closes ENT-804

🤖 Generated with [Claude Code](https://claude.com/claude-code)
2026-06-06 12:09:59 -05:00
Benjamin Taylor 3a7f546fbd fix(examples): correct Threads locked-state license command to copilotkit license (ENT-804)
The locked-state card told users to add an Intelligence license with
`copilotkit add-intelligence`, but that command only drops the
Intelligence overlay and does not issue a license (and is not yet wired
into the CLI dispatch). The command that issues a license key is
`copilotkit license`.
2026-06-06 10:29:41 -05:00
Jordan Ritter da03e1626a fix(showcase): reconcile harness-workers SSOT key with the Railway service name
The pool-fleet worker's Railway service is named `harness-workers` (PLURAL),
but the SSOT keyed it `showcase-harness-worker` (singular). The image-ref gate
matches SSOT keys to Railway service names verbatim, so the gate reported
`harness-workers` as an untracked Railway service AND the stale singular key
matched nothing. Rename the SSOT key (and every test/fixture reference) to the
exact Railway name `harness-workers`.

It stays the staging-only, domainless, probe-disabled worker that runs the
shared `showcase-harness` image: serviceId c2aa8a0b-…, staging instance
362c1e37-…, ciBuilt:false, gateIgnore:true, no build slot (so no dispatchName),
single `staging` env with no domain. Add a focused test pinning that shape.
Counts are unchanged (29 services / 26 CI_BUILT) — this is a rename, not an
addition; both harness workers already existed on main.

Verified LOCALLY against Railway: verify-railway-image-refs reports
`54 env-scoped instances verified (2 skipped)` — 0 violations, 0 missing, 0
untracked (harness-workers reconciled, harness-workers + harness-legacy the 2
gateIgnore'd skips). emit --check zero drift, Ruby parity green (borrowed
.up.railway.app host is parity-excluded), full scripts suite + typecheck green.
2026-06-06 07:51:00 -07:00
Jordan Ritter 9b76a2d2a4 refactor(showcase): unify railway-envs SSOT into per-env environments map (Option C)
Replace ServiceEntry's parallel prodInstanceId/stagingInstanceId/domains/probe/
repoNameOverride fields with a single environments: Record<string, {instanceId,
domain?, probe?, repoName?}> map plus a hoisted env-independent probeDriver.
EnvName becomes an open string backed by an ENV_ID_BY_NAME registry so
accessors resolve arbitrary env names; a single-env service (the staging-only
showcase-harness-worker) now simply omits the absent env instead of carrying a
placeholder ID/borrowed host.

Accessors instanceIdFor/domainFor/repoNameFor index environments[env]
(domainFor still throws on missing/scheme); add envsFor(name),
serviceEnvPairs(), and probeEnabled(name, env). Generalize the image-ref gate
(iterate each service's declared environments, resolve env-id via the
registry, sum/iterate missingByEnv over registry env names) and verify-deploy's
host->env reverse-map (envForTarget iterates environments).

Pure TS-internal: emit-railway-envs-json.ts projects the env-map back onto the
FROZEN legacy JSON shape (prodInstanceId/stagingInstanceId/domains/probe/
repoNameOverride) via a documented legacyJsonCompat shim for the two domainless
harness workers, so railway-envs.generated.json stays byte-identical and Ruby
(bin/railway) + workflow jq + the parity test are untouched. Verified: emit
--check zero drift, Ruby test_expected_domains_parity green, golden snapshot
toEqual proves byte-identical resolution for every real (service, env) pair,
full scripts vitest suite green, showcase scripts typecheck clean.
2026-06-06 07:48:14 -07:00
Jordan Ritter a8b5214b5f test(showcase): add railway-envs golden snapshot (behavior-preservation guard)
Serializes the fully-resolved {service -> env -> {instanceId, domain, probe,
driver, repoName}} projection for all 29 services x 2 envs via the public
accessors (instanceIdFor/domainFor/repoNameFor) + per-entry probe config,
frozen as a fixture. This is the behavior-preservation guard for the
forthcoming env-map (Option C) refactor: resolved values must stay
byte-identical before and after.
2026-06-06 07:32:10 -07:00
Jordan Ritter 5387e4de38 fix(showcase): add D6 entry to coverage dashboard legend 2026-06-06 02:30:36 -07:00
Jordan Ritter d408766cce fix(harness): balance fleet claim distribution to reduce settle-timeout contention 2026-06-06 02:30:36 -07:00
Jordan Ritter d6b0e895ae fix(harness): restore resource_snapshots in the fleet worker + per-replica attribution
Wire the resource-snapshot writer back into the fleet worker path with
per-replica attribution, add the worker_id column migration, and stamp
the legacy single-process path's worker_id as "" (empty string) to match
PocketBase storage and the surrounding codebase convention.
2026-06-06 02:30:32 -07:00
Jordan Ritter 5240ba813c fix(showcase): quarantine gen-ui-interrupt/interrupt-headless pills via not_supported_features
Move gen-ui-interrupt + interrupt-headless from features: to
not_supported_features: across affected integration manifests, and align
the generate-registry/generate-catalog scripts tests to the resulting
wired-feature counts (derive expected lengths from the parsed manifest
rather than hardcoding pre-quarantine numbers).
2026-06-06 02:30:26 -07:00
Jordan Ritter 182e3e2fd9 fix(showcase): add follow-up-turn aimock fixtures for multi-pill D6 sessions 2026-06-06 02:30:17 -07:00
Jordan Ritter 0cbc990e42 fix(showcase): forward x-aimock-context on all built-in-agent demo routes 2026-06-06 02:30:17 -07:00
Jordan Ritter 5c72d1addd fix(showcase): propagate x-aimock-context across AG-UI SDK pooled-thread hop in spring-ai 2026-06-06 02:30:13 -07:00
Jordan Ritter 5812739b5e feat(showcase): drive bin/showcase test d5/d6 through the fleet control-plane + add dev hot-reload mode
Make the showcase dev tool faithful to staging by construction. Two changes:

1. `showcase test --d5/--d6` now drives the fleet CONTROL-PLANE (producer ->
   probe_jobs queue -> worker -> result-aggregator) instead of the legacy
   in-process runLevel() driver. The new cli/control-plane-run.ts replicates
   the deep/full producer tick exactly as runControlPlane wires it
   (createE2eDeepServiceEnumerator / createServiceEnumerator over
   createJobProducer + createFleetQueueClient), enqueues one operator-triggered
   tick, and polls local PocketBase for the run's terminal cells. The running
   worker fleet claims + runs the driver + the aggregator writes the d5/d6
   status cells, so the dev tool exercises the IDENTICAL wiring + concurrency
   as staging. The old in-process path stays available behind `--direct`.

2. `showcase up --dev` adds a docker-compose.dev.yml overlay that bind-mounts
   each integration's source and overrides the run command with a stack-aware
   hot-reload entrypoint (shared/dev/dev-entrypoint.sh: uvicorn --reload for
   FastAPI agents, langgraph dev for graphs, next dev for the frontend). Edit a
   source file and the component reloads in place with no image rebuild. The
   built-image mode remains the faithful/staging-equivalent default.
2026-06-06 00:06:29 -07:00
Jordan Ritter 9e9e20429f fix(showcase): run D5 as "D6 take-one" to fix systemic D5 dashboard reds (#5291)
## Problem

D5 dashboard cells were systemically RED while D6 was green. The D5
assistant-no-response failure traced to: the backend LLM call to aimock
arrived without `x-aimock-context`, so aimock's strict mode returned a
503, which surfaced as an `agent_run_error_event`, which led to a 30s
timeout and a red cell.

## Root cause

NOT fixtures / keying / aimock — all verified good; D6 greens the same
cells on the same V1 route. The actual cause: D5 ran as a **separate
driver** (`e2e_deep` / `d5-single-pill`) whose code path lost
`x-aimock-context` on the shared fleet, whereas D6's path keeps it. This
operational divergence was exposed when the pool migration put the
duplicate D5 path onto the fleet.

## Fix

**D5 = D6-take-one.** D5 now runs the `d6-all-pills` driver scoped to
`D5_REPRESENTATIVES` via `representativeOnly` + `rowPrefix: "d5"`:
- enumerator supplies `extraDriverInputs`; CLI builds inputs through
`buildDeepInputs`
- worker reads `rowPrefix` to filter the `d5:<slug>` aggregate
- deleted the `e2e_deep` kind and `d5-single-pill.ts`

## Verification

- Full harness typecheck clean (except the pre-existing
`resource-snapshot-writer.test.ts` `toReversed` lib-target error already
on main)
- ~2097 harness tests green
- 7-agent CR + confirmation round converged (zero bucket-a findings)

## NOTE

Merge auto-deploys to staging (mutable `:latest`, no rollback). This fix
is operational — real validation is watching the d5 dashboard cells go
green post-deploy. A clean local single-run cannot reproduce the
concurrency-driven staging failure.

## Deferred follow-ups (non-blocking)

- gate worker `rowPrefix`-override on `driverKind`
- `X-Test-Id` `d5-` for D5
- deploy-churn `incapable[]` (pre-existing)
- tests for `representativeOnly` + deploy-churn and the demos-path
2026-06-05 22:11:34 -07:00
Jordan Ritter 3948629576 docs(showcase): remove stale e2e_deep/d5-single-pill references after D5=D6-take-one 2026-06-05 21:54:42 -07:00
Jordan Ritter a6a28f276e fix(showcase): keep D5 CLI error-path key consistent with success path 2026-06-05 21:42:50 -07:00
Jordan Ritter f723bc27da docs(showcase): correct payload-mapper to three browser driver families
The e2e_deep kind was removed (D5 now runs the D6 driver), leaving three
browser driver families: e2e_d6, e2e_demos, e2e_smoke. Update the stale
"four browser driver families" docstring and its test comment.
2026-06-05 21:36:16 -07:00
Jordan Ritter db2bf3e2ce test(showcase): cover composed representativeOnly+rowPrefix:d5 D5 invocation
Asserts the real D5 invocation shape buildDeepInputs stamps (both knobs
together): only D5_REPRESENTATIVES featureTypes run AND every emitted key
(per-cell d5:<slug>/<ft> + aggregate d5:<slug>) uses the d5: prefix. The
existing tests cover the knobs in isolation only.
2026-06-05 21:36:09 -07:00
Jordan Ritter 27c5217d1c fix(showcase): forward notSupportedFeatures + use d5: error key in D5 CLI path
buildDeepInputs now forwards manifest.not_supported_features (matching D6's
buildFullInputs) so local CLI D5 runs don't false-red architecturally-
unsupported features. The D5 thrown-error terminal key changes from
d5-single-pill-e2e:<slug> to d5:<slug> (the driver's own emitAggregate key
shape) so a hard driver throw surfaces as a RED D5 cell, not a blank row.
Also corrects D5-scope docs: representativeOnly keeps the representative
featureTypes per D5_REPRESENTATIVES, not "one pill per category".
2026-06-05 21:36:02 -07:00
Jordan Ritter 04a7ee703a fix(showcase): repoint d6-capture-references import off deleted d5-single-pill 2026-06-05 21:35:47 -07:00
Jordan Ritter 703985ec39 refactor(showcase): drop the e2e_deep kind constant + stale D5-driver references
Removes E2E_DEEP_DRIVER_KIND and "e2e_deep" from the worker-internal
closed driver-kind set (D5 runs the e2e_d6 driver now), updates the
x-test-id-headers guard to assert on the surviving d6-all-pills driver
(d5-single-pill.ts was deleted), and repoints a stale doc comment.
2026-06-05 21:23:21 -07:00
Jordan Ritter 1829c3f580 feat(showcase): run D5 as "D6 take-one" via driver inputs, not a separate kind
Repoints the D5 probe at the unified D6 driver. The fleet D5
enumerator now stamps driverKind=e2e_d6 with representativeOnly + a
"d5" rowPrefix; the CLI's buildDeepInputs carries the same inputs;
config/probes/e2e-deep.yml declares kind=e2e_d6. Drops the e2e_deep
driver registration (orchestrator + worker registry + BROWSER_KINDS +
the worker-internal kind set), keeping the D5 producer schedule/cadence
intact. The worker now honors driverInputs.rowPrefix when filtering the
aggregate side-row out of captured cells so a "d5:<slug>" aggregate
doesn't leak into the D6 entry's "d6:<slug>" cell capture. This
eliminates the separate D5 launcher path whose own launcher instance +
cadence systematically dropped x-aimock-context against the shared fleet
pool (aimock strict 503 -> red).
2026-06-05 21:22:45 -07:00
Jordan Ritter 8db67ca064 feat(showcase): add representativeOnly + rowPrefix knobs to the D6 driver
Adds two input knobs to the d6-all-pills driver so it can run as "D5
take-one": `representativeOnly` filters the feature matrix to the
D5_REPRESENTATIVES set, and `rowPrefix` ("d5" | "d6", default "d6")
threads the dashboard key prefix through every emitted per-cell and
aggregate PB row. The representatives map is injectable for testing.
Everything else (route, headers, conversation, pooled launcher) is
unchanged. Red-green unit tests cover both knobs.
2026-06-05 21:21:03 -07:00
Jordan Ritter 07fe6aeaf8 feat(showcase): mount /api/probes trigger endpoint on the fleet control-plane (#5288)
## Summary

The fleet **control-plane** runs the 8 in-process HTTP probe families
(smoke, starter_smoke, image_drift, qa, aimock_wiring, version_drift,
pin_drift, redirect_decommission) on its scheduler, but the on-demand
trigger endpoint `POST /api/probes/:id/trigger` was only mounted on the
legacy `boot()` path. There was no way to fire a family immediately on
the control-plane — operators had to wait on the slow cron.

This wires the **same** `registerProbesRoutes` onto the control-plane's
`buildServer` call in `runControlPlane`, using the control-plane's
already-built `httpProbeRegistry` / `httpProbeConfigs` / `scheduler` /
`httpRunWriter` and `OPS_TRIGGER_TOKEN`. Enables instant verification
instead of waiting on crons.

- **Triggerable ids** (control-plane): the prefixed in-process HTTP
probe scheduler ids — `probe:smoke`, `probe:image_drift`, `probe:qa`,
etc. (whatever HTTP families are loaded). The router's `isProbeId` guard
resolves against `httpProbeConfigs` (keyed `probe:<cfg.id>`).
- **404'd**: browser-only families (`probe:e2e_smoke`,
`probe:e2e_demos`, `probe:e2e_deep`, `probe:e2e_d6`) — they are
worker-routed, never run in-process — plus unknown ids and the
producer's own scheduler entries.
- **Fail-safe token handling mirrors `boot()` exactly**:
`OPS_TRIGGER_TOKEN` unset → router omitted (route 404s); set-but-empty /
whitespace-only → fail-loud at boot (refuses to mount an insecure
route); missing bearer token → 401.

## Scope

Minimal: only the `buildServer({ ..., probes })` wiring + the
boot()-equivalent token resolution in `runControlPlane`. No changes to
`probes.ts`, the in-process runner, or the worker/boot paths.

## Test plan

- [x] RED→GREEN: with the impl reverted, the GET-list / trigger-runs /
no-token-401 / empty-token-fail-loud tests fail (route absent → 404);
with the impl they pass.
- [x] New tests in `orchestrator.test.ts` (real `runControlPlane` boot
on a live port): GET `/api/probes` lists the 3 HTTP families (not the
browser kind); POST `probe:smoke/trigger` with bearer runs it; 401
without token; 404 for `probe:e2e_smoke` and unknown ids; router omitted
when token unset; fail-loud on empty token.
- [x] Full harness suite green: 120 files / 2135 tests.
- [x] `tsc -p tsconfig.build.json` clean.
- [x] oxlint / oxfmt clean on touched files.

🤖 Generated with [Claude Code](https://claude.com/claude-code)
2026-06-05 16:50:49 -07:00
Jordan Ritter 723090e24e feat(showcase): mount /api/probes trigger endpoint on the fleet control-plane
The control-plane runs the 8 in-process HTTP probe families but the
on-demand trigger endpoint (POST /api/probes/:id/trigger) was only mounted
on the legacy boot() path, so operators had no way to fire a family
immediately and had to wait on the slow cron. Wire the same
registerProbesRoutes onto the control-plane's buildServer using its own
httpProbeRegistry/httpProbeConfigs/scheduler/httpRunWriter and
OPS_TRIGGER_TOKEN. Only the prefixed in-process probe ids (probe:<id>) are
triggerable; browser-only and unknown ids 404. Token handling mirrors
boot() (unset -> router omitted; set-but-empty -> fail-loud).
2026-06-05 16:44:43 -07:00
Jordan Ritter 23b1412e27 feat(showcase): run e2e_smoke/e2e_demos/e2e_deep browser families on the fleet (#5287)
## Summary

Phase 2 of the harness pool-fleet migration: make the three remaining
BROWSER probe families (`e2e_smoke`, `e2e_demos`, `e2e_deep`) actually
run on the fleet by wiring producers into the control-plane. The worker
`DriverRegistry` for all four browser kinds already landed in #5283;
this PR is purely the PRODUCER side. Builds on #5283/#5284/#5285/#5286
(all merged).

### Unit A — three catalog-enumerator factories
`createE2eSmokeServiceEnumerator` / `createE2eDemosServiceEnumerator` /
`createE2eDeepServiceEnumerator`, each a thin specialization of the
generic `createServiceEnumerator` (the #5285 seam) with its own
`driverKind` + dashboard `probeKey` prefix and the shared
`D6_DISCOVERY_FILTER`:

| family | driverKind | probeKey prefix | verified against |
|---|---|---|---|
| smoke | `e2e_smoke` | `d4:<slug>` | `src/cli/targets.ts` `d4:${slug}`
|
| demos | `e2e_demos` | `e2e-demos:<slug>` |
`config/probes/e2e-demos.yml` `id: e2e-demos` |
| deep | `e2e_deep` | `d5-single-pill-e2e:<slug>` | `src/cli/targets.ts`
`d5-single-pill-e2e:${slug}` |

### Unit B — multi-schedule wiring in `runControlPlane`
`runControlPlane` now builds four producers and passes a `schedules`
array to `createControlPlane` (the #5285 multi-schedule API) via a new
pure `buildProducerSchedules` helper. Crons are read **literally from
the config YAMLs** — the deliberate offsets stagger the four families'
Playwright fan-outs on the shared `BrowserPool`:

| scheduleId | cron | source |
|---|---|---|
| `fleet-job-producer` | `40 * * * *` | d6 (unchanged; still honors
`FLEET_PRODUCER_CRON`) |
| `fleet-producer-e2e-smoke` | `*/15 * * * *` | `e2e-smoke.yml` |
| `fleet-producer-e2e-demos` | `10 * * * *` | `e2e-demos.yml` |
| `fleet-producer-e2e-deep` | `5,20,35,50 * * * *` | `e2e-deep.yml` |

The in-process HTTP probe runner (#5284) and the d6 producer's REQ-B
sweep leg are left intact (additive). The worker registry is **not**
touched.

### R-timeout (demos) — addressed
The demos driver's 20-min outer cap is threaded in-process via the
legacy `E2E_DEMOS_TIMEOUT_MS` env, which the **fleet worker never sets**
— so without a fix the 38-demo service would blow the driver's 5-min
`DEFAULT_TIMEOUT_MS` and go all-red. Fix:
- New `E2E_DEMOS_TIMEOUT_MS` SSOT const (mirrors `e2e-demos.yml`
`timeout_ms`) in the enumerator module.
- `createE2eDemosServiceEnumerator` conveys the cap per-job in
`driverInputs.timeout_ms` (smoke/deep convey nothing — verified no
`timeout_ms` on their specs, and d6's spec shape is unchanged).
- The demos driver now reads `input.timeout_ms` as a resolution source:
`ctx.env.E2E_DEMOS_TIMEOUT_MS` (legacy) > `input.timeout_ms` (fleet) >
`deps.timeoutMs` > `DEFAULT_TIMEOUT_MS`. Schema gains `timeout_ms:
z.number().int().positive().optional()`.

### Risks honored
- **R3 pool-contention (deploy-time):** all four families' jobs are
claimed by the same pooled worker(s) drawing from ONE `BrowserPool`
under `BROWSER_POOL_MAX_CONTEXTS` (24). Per-family `max_concurrency`
governs producer ENQUEUE width, not worker execution. The guard is the
offset crons + the 24-cap — preserved faithfully here. **Verify at
deploy time by triggering each family's producer and confirming the
BrowserPool does not starve.**
- **R1 context-headers (verified, no change):** both
`createPooledE2eSmokeLauncher` and `createPooledE2eDeepLauncher` thread
`contextOpts.extraHTTPHeaders`, so smoke + deep set their per-slug
`X-AIMock-Context` themselves.

## Test plan
- [x] Unit A: `createE2eSmoke/Demos/Deep` factories stamp the right
driverKind + probeKey shape and carry driverInputs; demos conveys
`timeout_ms` (default + override); d6 equivalence (no `timeout_ms`)
asserted — red→green verified.
- [x] Demos driver reads `input.timeout_ms` when the env is absent
(fleet path), env still wins over input (precedence) — red→green
verified.
- [x] Unit B: `buildProducerSchedules` emits 4 schedules with exact ids
+ crons; `FLEET_PRODUCER_CRON` override applies to d6 only;
`runControlPlane` registers all 4 producer schedules on the live
scheduler alongside the `probe:*` HTTP entries — red→green verified.
- [x] Full harness suite green: **120 files / 2128 tests passed**.
- [x] `tsc --noEmit` clean except the lone pre-existing `toReversed`
error.
- [ ] Deploy-time: trigger each family's producer and verify BrowserPool
does not starve under co-firing (R3).

🤖 Generated with [Claude Code](https://claude.com/claude-code)
2026-06-05 16:18:06 -07:00
Jordan Ritter 3d940a3c74 feat(showcase): run e2e_smoke/e2e_demos/e2e_deep browser families on the fleet
Phase 2 of the harness pool-fleet migration — wire the three remaining BROWSER
probe families into the control-plane PRODUCER side so they actually run on the
fleet (the worker DriverRegistry for all four kinds already landed in #5283).

Unit A — three catalog-enumerator factories mirroring createD6ServiceEnumerator,
each delegating to the generic createServiceEnumerator with its own driverKind +
dashboard probeKey prefix and the shared D6_DISCOVERY_FILTER:
  - createE2eSmokeServiceEnumerator → e2e_smoke / d4:<slug>
  - createE2eDemosServiceEnumerator → e2e_demos / e2e-demos:<slug>
  - createE2eDeepServiceEnumerator  → e2e_deep  / d5-single-pill-e2e:<slug>

R-timeout (demos): the demos driver's 20-min outer cap is threaded in-process via
the legacy E2E_DEMOS_TIMEOUT_MS env, which the fleet worker never sets. The demos
enumerator now conveys the YAML timeout_ms per-job in driverInputs.timeout_ms
(new E2E_DEMOS_TIMEOUT_MS SSOT const mirroring e2e-demos.yml), and the demos
driver reads input.timeout_ms as a resolution source (env > input > deps >
default) so the 38-demo service no longer blows the 5-min default to all-red.

Unit B — runControlPlane now builds four producers and passes a multi-schedule
manifest (buildProducerSchedules) to createControlPlane, each family on its own
cron read literally from the config YAMLs (the deliberate offsets stagger the
families' Playwright fan-outs on the shared BrowserPool):
  - fleet-job-producer       40 * * * *          (d6, unchanged; honors FLEET_PRODUCER_CRON)
  - fleet-producer-e2e-smoke  */15 * * * *
  - fleet-producer-e2e-demos  10 * * * *
  - fleet-producer-e2e-deep   5,20,35,50 * * * *
The in-process HTTP probe runner and the d6 producer's REQ-B sweep leg are left
intact (additive). The worker registry is untouched.

R1 (verified, no change): both createPooledE2eSmokeLauncher and
createPooledE2eDeepLauncher thread contextOpts.extraHTTPHeaders, so smoke + deep
set their per-slug X-AIMock-Context themselves.
2026-06-05 16:11:08 -07:00
Jordan Ritter 54de90d65f feat(showcase): run HTTP-only probe families in-process on the fleet control-plane (#5284)
## Summary

Make the fleet control-plane ALSO run the 8 HTTP-only probe families
in-process (they don't need the BrowserPool/worker), by lifting the
legacy `boot()` probe-loader machinery into `runControlPlane`. Before
this, the control-plane only ran the d6 producer and the 8 HTTP families
were dark on the fleet.

HTTP-only families now run in-process: `smoke`, `starter_smoke`,
`image_drift`, `qa`, `aimock_wiring`, `version_drift`, `pin_drift`,
`redirect_decommission`. The browser families (`e2e_d6` / `e2e_smoke` /
`e2e_demos` / `e2e_deep`) are deliberately NOT run in-process — d6 goes
via the worker producer path; the rest need a BrowserPool the
control-plane does not own. This is independent of the worker-registry /
worker-loop work.

## Design

- **`BROWSER_KINDS`** (exported) = `{e2e_d6, e2e_smoke, e2e_demos,
e2e_deep}`. HTTP = every kind NOT in this set. Single source of truth
for the partition.
- **`registerHttpProbeDrivers`** (exported) registers only the 8 HTTP
drivers (no BrowserPool drivers) — kept separate from
`registerAllProbeDrivers` so the control-plane's `probeRegistry` is
HTTP-only.
- In **`runControlPlane`**: build an HTTP-only `probeRegistry` + a
discovery registry wiring the SAME sources `boot()` uses
(`railway-services` cached 24h with an auth tracker, `pnpm-packages`), a
`createProbeLoader` scoped to HTTP kinds, and the same
`diffProbeSchedules`/`buildProbeInvoker` loop `boot()` runs —
registering one `probe:<id>` scheduler entry per YAML config on the
control-plane's scheduler. **Crons are driven FROM the YAML**
(`cfg.schedule`), never hardcoded. Each tick flows through the same
`statusWriter` pipeline as the worker-result aggregator. Hot-reload via
`probeLoader.watch` mirrors `boot()`.
- **`includeKind` predicate** added to `createProbeLoader`: a browser
YAML on disk is SKIPPED (not rejected) against the HTTP-only registry,
so a present `e2e_*` YAML never surfaces a spurious
`probes.reload.failed`.

## /health handling

The control-plane now owns in-process probe rules, so `ruleCount`
reflects the real in-process HTTP probe count (was a hardcoded `0`). The
`role: "control-plane"` rules>0 gate-drop is retained (liveness is
governed by the scheduler signals already folded into `loopOk`), but the
real count means a silent probe-loader failure (zero HTTP probes loaded)
is now VISIBLE on `/health` rather than masked behind the role
short-circuit. `schedulerJobCount` already counts the new `probe:`
entries (covers BOTH the producer entry and the probe entries).

Teardown: the HTTP-probe file watcher is torn down on `stop()` and on
bind failure.

## Red→Green evidence

Wrote failing tests first, confirmed RED against baseline (stashed the
impl):
- scheduling test: `probe:smoke` / `probe:image_drift` not registered
(FAIL)
- /health test: `rules` < 2 (FAIL)
- `BROWSER_KINDS is not iterable` (FAIL)
- loader `includeKind` skip behavior

After implementing, all GREEN.

## Test plan

- [x] New `runControlPlane` in-process HTTP probe tests (scheduling
partition + /health rule count + BROWSER_KINDS) — green, RED-verified
- [x] New `createProbeLoader` `includeKind` skip-not-reject test — green
- [x] Full harness vitest suite: 119 files / 2076 tests passed
- [x] Production build `tsc -p tsconfig.build.json`: clean (the only
`--noEmit` error is the pre-existing `toReversed` lib-target issue in an
unrelated test file, excluded from the build config)

🤖 Generated with [Claude Code](https://claude.com/claude-code)
2026-06-05 15:51:32 -07:00
github-actions[bot] 12a6c18049 style: auto-fix formatting 2026-06-05 22:49:28 +00:00