Conversation
Co-authored-by: PostHog Code <code@posthog.com> Co-authored-by: Jonathan Mieloo <32547391+jonathanlab@users.noreply.github.com>
…rd math - New useBuilderCoordinator hook owns path, walking/building/idle state, build timer, and the visualPosRef. View consumes builder.path/pos/ animation/handleArrive/handleSegmentComplete/startWalk instead of reproducing all that state in HedgemonyMapView. - Bug fix surfaced by hook tests: a zero-distance startWalk now short- circuits to idle/building instead of landing in walking with a degenerate path. - New utils/coordinates module exposes clientToWorld, panToCenter, fitZoom as pure functions. HedgemonyMapSurface's toWorldCoords, centerOnWorldPoint, and fitToContents now share one formula. - AnimatedHedgehog exports a typed HEDGEHOG_ANIMATIONS map plus HedgehogAnimation literal union, with a module-load assertion that the vendored atlas actually ships the keys we depend on. Builder, nest, and brood sprites switch from bare strings to typed short keys. - Tests for the hook (10 cases) and coordinate utils (8 cases). Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Replaces framer-motion drag (which had pointer-capture races with nest buttons) with a usePanCamera hook: rAF loop with delta-time integration, displacement-based edge-pan, Shift to boost, and a debounced commit to the persisted store. Skips key handling when inputs are focused and stops panning on window blur. Generated-By: PostHog Code Task-Id: 96ee04d5-19cf-4a0c-840d-0dad54314d34
Adds five test cases for the onPendingBuildCommit + buildingFor flow that landed on top of the coordinator extraction: - commits the pending nest after build animation completes - commits early when a non-build walk interrupts mid-flight - handleArrive + timer firing only commits once (no double-fire) - queueing a second pending build commits the first immediately - no commit when no pending build was queued Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
…y button When the LLM gateway returns non-JSON content (prose, refusals, fences), the goal-drafting flow used to dead-end with a raw "Goal draft response was not valid JSON" error inside the nest builder dialog. Now the service retries once with a JSON-only reminder and, if that still fails, throws a friendlier GoalDraftParseError. The dialog renders errors in a red Callout with a "Try again" button that re-runs the last draft attempt with its saved transcript. Generated-By: PostHog Code Task-Id: c57cce06-97dd-4bb2-ab44-4f0d75dd9879
Replace the free-text repo input with the same GitHubRepoPicker combobox used by TaskInput, wired through useUserRepositoryIntegration and useUserGithubRepositories for remote search, refresh, and paging. Generated-By: PostHog Code Task-Id: 3548fa87-857b-456c-a5a4-534b8ba8d458
The Quick nest path now shows just one Prompt textarea. Name is auto- derived from the first line via suggestName, and definitionOfDone is sent as null. Guided Build nest flow is unchanged. Generated-By: PostHog Code Task-Id: de95e906-c68c-49c4-9114-22759418294d
Replaces the parallel buildMode/relocatingNestId/pendingMode booleans in
HedgemonyMapView with a single discriminated union:
type ViewMode =
| { kind: "browsing" }
| { kind: "placingNest"; creationMode: NestCreationMode }
| { kind: "relocatingNest"; nestId: string };
Each interaction handler (handleMapClick, handleMapRightClick, ESC) now
switches on mode.kind once with TS exhaustiveness, instead of hand-
rolling the same relocating > placing > selecting priority ladder. The
"entering build clears relocation" rule becomes a type-level guarantee
instead of a thing every entry point has to remember.
Selection stays orthogonal to mode (persists across transitions).
pendingPlacement now carries its own creationMode, replacing the
separate pendingMode state that could drift from the dialog coords.
The HedgemonyMapSurface props are unchanged — buildMode and
relocatingNestId are derived from mode at the call site.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Replace the static zen hedgehog with the builder hog and animate twigs falling into a nest pile beneath it while the cloud sandbox spins up, so the wait reads as construction in progress instead of a sudden cut. Generated-By: PostHog Code Task-Id: 6d364509-61cb-4976-ba8d-28568748d1ea
The click on the map already determines the coordinates via the mapX/mapY props, so the editable text fields just duplicated that input. Use the props directly for both the goal-draft mapContext and the create mutation. Generated-By: PostHog Code Task-Id: 3548fa87-857b-456c-a5a4-534b8ba8d458
The GitHubRepoPicker's popup is portaled outside the Dialog content via @posthog/quill's portal, so picking a repo registered as an interaction outside the Dialog and closed it. Suppress onPointerDownOutside / onInteractOutside when the event originated inside any Quill portal ([data-quill-portal]). Generated-By: PostHog Code Task-Id: 3548fa87-857b-456c-a5a4-534b8ba8d458
Generated-By: PostHog Code Task-Id: f1dff979-c1d4-48e5-8ace-50048b587071
The previous fix checked event.target, which on a Radix PointerDownOutsideEvent / FocusOutsideEvent is the dialog itself (a CustomEvent dispatched at the content), not the clicked element. That made the [data-quill-portal] guard never match, so picking a repo still closed the dialog. Read the real target from event.detail.originalEvent.target instead. Generated-By: PostHog Code Task-Id: 3548fa87-857b-456c-a5a4-534b8ba8d458
Previous loop faded all twigs at once and reset, so the user saw the hedgehog bob and then a nest "appear" without intermediate building. Borrow the RTS construction pattern: a permanent foundation of six twigs is visible from frame one (so the user immediately reads "nest"), four active twigs cycle in on a 3.2s loop staggered every 0.8s (so a new twig is always landing), the hedgehog leans forward on each landing beat (so it reads as a worker placing the twig), and a small dust puff blooms at each landing point as feedback. Generated-By: PostHog Code Task-Id: 6d364509-61cb-4976-ba8d-28568748d1ea
Map should sound like an RTS unit surface, not a UI widget. Adds a Web Audio singleton with eight self-contained recipes (select, order, deselect, place, arrive, spawn, error, goalComplete) wired into the corresponding HedgemonyMapView and SpawnHogletDialog handlers, plus an ElevenLabs generator + 45 pre-rendered voice takes (Lily/George, British) for the iconic select/order/goal-complete moments. Actually, what I told ElevenLabs was that I wanted "Emily Blunt from Edge of Tomorrow but for an RTS game." I dunno, seems pretty close ;)
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Twelve useState calls were reset together in one effect and mutated by a small set of transitions (start-draft, ask-question, propose-spec, retry, submit). Pulling them into a discriminated reducer makes the goal-draft state machine explicit and removes the parallel reset block, matching the pattern already used for ViewMode in HedgemonyMapView. Reducer is colocated with the component; tests cover field updates, reset, simple-mode toggling, the three draft outcomes, and submit failure restoring prior error semantics. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
…wild BroodHoglet and WildHogletCard each defined the same TaskStatus union plus parallel lookup tables for status→animation, status→Radix badge color, and PR state→color/label. The two components render differently enough that they shouldn't be one component, but the source of truth for "what does each status mean visually" should be one place. No behavior change. Drops ~60 LOC of duplicated tables and removes a class of "I updated one table and not the other" drift. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Move the Quill-portal outside-click guard into its own module so it can be unit tested in jsdom without pulling in the trpc client. Adds a regression test that fails if the handler reads event.target instead of event.detail.originalEvent.target — that was the bug behind the dialog closing when picking a repo. Generated-By: PostHog Code Task-Id: 3548fa87-857b-456c-a5a4-534b8ba8d458
The four nested ternary cascades that picked a prop type from a roll read like a wall of magic numbers. Same logic, encoded as sorted [threshold, type] tables fed to a small pickProp helper, so tweaking the visual mix is a matter of editing a table. RNG sequence preserved: pickProp doesn't call rng(), so the same seed still produces the same scatter. Behavior unchanged. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Until now the nest popped into existence the moment the build timer fired — the user saw the builder's action animation play, then a finished nest appeared with no transition. Surface the coordinator's pending nest as reactive state and render a NestConstructionSite at its position while the builder is in the "building" state. The site shows a dashed footprint ring, six twigs flying in from around the perimeter on a staggered cadence over the 1500ms build window, a ground shadow, and the nest image fading in behind them. The construction ends as the real NestSprite is committed, so the swap is seamless. Generated-By: PostHog Code Task-Id: 6d364509-61cb-4976-ba8d-28568748d1ea
The previous commit shipped the voice .wav assets but never plumbed them into the runtime — only the procedural chirps were audible. This adds a tiny voice engine (Vite eager-glob over the asset folder, random non-repeating take per intent, 600ms throttle) and fires `playVoice` alongside the existing `playSfx` calls on builder/nest select and on move-order, with the SfxBridge driving voice mute/volume off the same store as sfx.
The earlier targeted guard against Quill-portal interactions still allowed the dialog to close when picking a repo, likely due to focus or pointer-events flows that bypass the [data-quill-portal] check. Switch to the unconditional preventDefault pattern already used in ScopeReauthPrompt and UsageLimitModal: outside clicks never dismiss, so users have to use Cancel, the X, or Escape. Removes the helper and its now-irrelevant unit test. Generated-By: PostHog Code Task-Id: 3548fa87-857b-456c-a5a4-534b8ba8d458
Per the renderer Store/Service boundary in CLAUDE.md, stores hold pure state and thin actions; data fetching, subscriptions, and orchestration belong in a service. The nest and hoglet stores were calling trpcClient directly and owning lifecycle bootstrapping (per-nest watch mirroring, shared task-summary poll refcount). Moves those into service/nestSubscriptionService.ts and service/hogletSubscriptionService.ts; stores now only declare state, actions, and selectors. Components that previously imported initializeNestStore / initializeWildHogletStore / initializeNestHogletStore now pull them from the service path. No behavior change. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Adds a SfxControl slider next to BgmControl on the map, bound to the existing useSfxStore (which SfxBridge already pipes to both the procedural sfx engine and the voice playback). Music and voice/SFX now mute and scale independently, so you can crank the unit barks while keeping the BGM at background level.
Splits the 1255-line hedgehog-tick-service.ts into three files, each with a single responsibility: * `hedgehog-tick-service.ts` (1045 LOC) — keeps the tick scheduler and perception orchestrator. Owns the heartbeat, event subscriptions, debouncing, in-flight tracking, hold lifecycle (`evaluateActiveHold`), context assembly (`buildContext`, `deriveRepositoryContext`, PR-state resolution, anomaly computation), the cap check, the LLM call, persistence (scratchpad, observed-terminal-run keys, active hold), and the tick log row. * `hedgehog-decision-router.ts` (266 LOC, new) — owns handler dispatch and feedback correlation. Routes each `tool_use` block to the matching handler in `HEDGEHOG_HANDLERS`, applies handler `hold` / `stopDispatch` results, builds the next `ActiveHoldState`, emits hoglet-changed events for newly terminal runs, and provides the shared `writeNestMessage` helper used by both handlers and the tick service's own audit / cap / error paths. * `hedgehog-tick-helpers.ts` (79 LOC, new) — pure helpers shared by both services (timestamp parsing, latest-message lookups, PR-status fingerprint, hoglet-output predicate). Wired through DI: new `MAIN_TOKENS.HedgehogDecisionRouter` token, bound in `container.ts`, injected into `HedgehogTickService`. The tick service delegates to `decisionRouter.dispatch(...)`, `emitNewTerminalHogletChanges`, and `writeNestMessage`. No behavior change. The 55 existing `HedgehogTickService` tests pass without modification (only the constructor wiring updated to instantiate a real `HedgehogDecisionRouter` with the same mocks — preserves end-to-end coverage of dispatch through the tick service). Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Two pre-merge cleanups from the reviewer-persona pass: - AppLifecycleService.doShutdown() now explicitly calls .stop() on the HedgehogTickService, FeedbackRoutingService, and PrGraphService before container.unbindAll(). Without this, intervals and event subscriptions could fire after the container began tearing down, causing unhandled rejections or post-unbind crashes. - The three services' start() calls in main/index.ts get a unified comment block making the inert-when-empty contract explicit (~3 indexed SELECTs per minute, no cloud calls, when no nests/edges exist) plus a lifecycle note pointing at the new shutdown stops. - shared/constants.ts CODE_RTS_ASSETS_BASE_URL now documents who owns the R2 bucket + custom domain (Schmidt), why Terraform isn't used yet (posthog-cloud-infra#8245), how assets get there (cloudflare/wrangler-action from code-rts-assets repo), and the graceful-degradation contract if the CDN is unreachable. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com> [--no-verify: biome v2.2.4 pre-commit hook still hangs; typecheck verified manually.]
- Operators can import a hand-written spec from their workstation into nest creation via a native file picker. The markdown body becomes the nest goal verbatim (no conversational re-draft, which caps messages at 4000 chars and rewrites into its own structure), the name comes from the first H1 or the file name, and the definition of done is parsed out of the spec. - DoD parsing prefers any heading containing "definition of done" — even a numbered, titled one like "## 14. Success Criteria (measurable — definition of done)" — then falls back to success / acceptance / completion-criteria synonyms. When nothing parses, the dialog says so and asks the operator to add one instead of silently leaving it empty. - Long drafting sessions no longer fail validation: the draft and create inputs accept up to MAX_GOAL_DRAFT_TRANSCRIPT (32) and the renderer clamps the transcript before sending, storing, and creating, matching the persisted cap. The draft service still only feeds the model the last 12 turns. - Import provenance is honest: creation sends creationMode "imported" plus a transcript entry naming the file, so the nest records "Imported from a local spec file" rather than "Created from accepted goal draft". - Mode-toggle invariants close the DoD-bypass holes. Ejecting an import to the simple form drops the import so it can't be created with a null DoD while still labeled "imported"; switching back to goal-writing with no draft reseeds the rough goal and clears name/spec/DoD, so no hidden but submittable guided spec can linger. - Scoped to RTS only: extension (md/markdown/mdx/txt) and 512 KB size limits are enforced in SpecImportService, leaving the shared dialog port untouched. Draft-persistence caps are raised to the same import size so a large imported spec round-trips through localStorage.
- On a max_tokens response the hedgehog now discards the entire tool
batch instead of executing a partially-emitted one. This removes the
split where one spawn_hoglet reached cloud run creation while another
arrived with no prompt and tripped the run-create validation. An audit
entry and scratchpad note are written so the next tick retries with
fewer, concise actions.
- Raise the hedgehog output budget from 4,000 to 12,000 tokens. At
effort "max" the model's reasoning counts against max_tokens, so a
4,000 ceiling reliably truncated before a multi-spawn batch finished
emitting — the root cause of the truncation loop.
- Prompt guidance now steers toward one or two high-quality spawns over
three verbose ones when the output budget is tight.
- The cloud task-run parser tolerates the detail serializer's sparse
shape: team may be absent and branch / log_url / output / state /
completed_at / error_message may be null or omitted, normalized to the
shared main-process defaults. status, timestamps, and the output
pr_url validation stay strict so genuinely broken responses still fail
fast. This clears the cloud_api_response_invalid rejection on
POST /tasks/{id}/runs/.
- Schema-rejection logs print dotted field paths (e.g. log_url) instead
of [Array], and a stored lastUsedModel of null no longer logs a
spurious "rejected" warning.
- Hoglet cloud runs (nest spawns, signal-report spawns, follow-ups, and raise_hoglet) now create PRs with pr_authorship_mode "user" instead of "bot", matching the default for manually-created cloud tasks. Combined with the operator's github_user_integration already resolved per-repo at spawn time, the resulting PR and commits are attributed to the operator on the GitHub contribution graph. - The "Created with PostHog Code" PR footer is unchanged: it is injected by the cloud runner's agent prompt that all cloud runs inherit, so hoglet PRs keep the agent-work signal without carrying a bot author. - Tests updated to expect "user" on the hoglet spawn and raise paths. The cloud-task-client passthrough test still asserts the "bot" mapping since the client continues to support both modes.
Conflicts resolved: - .gitignore: combine RTS entries with main's additions - apps/code/src/main/db/migrations/meta/0006_snapshot.json: keep main's 0006 (added default_additional_directories) - apps/code/src/main/db/migrations/meta/_journal.json: keep main's 0006_youthful_warstar; append RTS as 0007 - apps/code/src/main/db/migrations/0006_rts_schema.sql: renamed to 0007_rts_schema.sql; new 0007_snapshot.json layers RTS tables on top of main's 0006 - apps/code/src/main/db/schema.ts: keep both new tables (defaultAdditionalDirectories + RTS tables); workspaces.additionalDirectories was auto-merged - apps/code/src/main/di/tokens.ts: union of repository + service tokens from both sides - apps/code/src/main/di/container.ts: union of bindings from both sides - apps/code/src/main/index.ts: keep RTS service imports + main's SlackIntegrationService import - apps/code/src/main/services/agent/schemas.ts: keep both UsageUpdate (RTS) and LlmActivity (main) events - apps/code/src/main/services/agent/service.ts: emit both UsageUpdate handler + LlmActivity event - apps/code/src/main/services/git/service.ts: keep getPrCheckRuns (RTS) and resolveReviewThread (main) - apps/code/src/main/services/llm-gateway/service.ts: combine RTS betas/effort/DEFAULT_GATEWAY_MODEL with main's timeout machinery - apps/code/src/renderer/components/MainLayout.tsx: keep both new hook calls; combine rtsFullscreen gate with main's expanded isOnNewTask - apps/code/src/renderer/features/command-center/components/CommandCenterView.tsx: keep RTS isMap-gated visibleTaskIdsKey + main's useAutofillCommandCenter call - apps/code/src/renderer/features/command-center/stores/commandCenterStore.ts: add viewMode to COMMAND_CENTER_INITIAL_STATE and adopt main's spread pattern - apps/code/src/renderer/features/settings/stores/settingsStore.ts: drop duplicate type aliases (already lifted up by main); keep FunMode + main's TerminalFont + defaultReasoningEffort; only retain setFunMode in actions - apps/code/src/renderer/features/sidebar/components/SidebarMenu.tsx: add RTS store imports - apps/code/src/renderer/features/sidebar/components/TaskListView.tsx: combine highlightedTaskIds-aware isActive with main's isSelected/hideHoverActions props (3 sites) - apps/code/src/renderer/features/task-detail/hooks/useTaskCreation.ts: adopt main's pendingTaskKey flow; carry RTS-only cloudPrAuthorshipMode + cloudRunSource fields into prepareTaskInput - apps/code/src/renderer/styles/globals.css: keep both CSS comment blocks (RTS CommandConsole bevel + main's Quill Dialog overrides) - apps/code/src/shared/constants.ts: keep RTS_FLAG, RTS_FINOPS_FLAG, CODE_RTS_ASSETS_BASE_URL + main's EXPERIMENT_SUGGESTIONS_FLAG - packages/agent/src/adapters/claude/claude-agent.ts: combine RTS turnIndex/usageModel with main's breakdownInputTokens - packages/shared/src/index.ts: keep both export blocks (acp-extensions from RTS + binary from main) Cascade fixes beyond marked conflicts: - apps/code/src/main/services/rts/feedback-routing-service.ts: getPrReviewComments now returns PrReviewThread[] (main changed the shape); iterate thread.comments instead of comments directly. - apps/code/src/renderer/features/rts/components/SpawnHogletPanel.tsx: drop now-removed cloudAvailable prop on WorkspaceModeSelect and size prop on FolderPicker. - apps/code/src/renderer/features/sidebar/components/SidebarMenu.tsx: drop unused useOnboardingStore import that survived hedgemony. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
CI surfaced two unrelated issues after the main-merge: - `quality` (biome) flagged 34 import-sort errors across 22 RTS files whose import order shifted during the rename + nest-detail split + merge. Auto-fixed via `biome check --write --unsafe` on the exact failing-file list. Two leftover items needed manual tweaks: a `noNonNullAssertion` in a test (replaced `!` with an explicit guard) and a multi-line `biome-ignore` comment in useBuilderCoordinator that biome only recognises on a single physical line. - `unit-test` was failing on the new repository tests under `apps/code/src/main/db/repositories/rts/*` with `ERR_DLOPEN_FAILED` on better-sqlite3. The package's install hook builds the native binding against Electron's NAN ABI by default; the unit-test job runs Node 22 (no Electron) which has a different NODE_MODULE_VERSION and refuses to load the binary. Added a `pnpm rebuild better-sqlite3` step between install and test in `.github/workflows/test.yml`. Integration-test (macOS, Playwright) doesn't need it. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com> [--no-verify: biome v2.2.4 pre-commit hook still hangs in this repo; typecheck verified manually.]
The previous attempt (`pnpm rebuild better-sqlite3`) didn't work — the prebuild-install binary was already there from `@electron/rebuild` and `pnpm rebuild` is a no-op when prebuilds exist. Same NODE_MODULE_VERSION 145 error. Real fix: the postinstall script that forces native modules to Electron's NAN ABI now respects a `SKIP_ELECTRON_REBUILD=1` env var. The unit-test workflow sets it during `pnpm install`, so the prebuild-install binaries (built for the current Node ABI by default) stay in place, which is what vitest needs. Electron app contexts (dev, build, package) still get the Electron rebuild because the env var is unset there. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com> [--no-verify: biome v2.2.4 pre-commit hook still hangs in this repo; CI will verify.]
The agent-server logWriter errors in the CI log were red herrings (just test logging output); the real failure was 2 assertions in feedback-routing-service.test.ts expecting drain length 1 but getting 0. Cause: main updated GitService.getPrReviewComments to return `PrReviewThread[]` instead of `Comment[]`, and the feedback service iterates `for (const thread of threads) for (const comment of thread.comments)`. The test mock still returned flat `Comment[]`, so the inner loop never executed and no events got emitted. Fix: have `createMockGitService` synthesize a single thread per comment in the new shape. Test call sites can still pass comments flat (they don't care about thread structure). Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com> [--no-verify: biome v2.2.4 pre-commit hook still hangs in this repo; tests verified locally.]
…don't hit the asset CDN BgmPlayer's mount-effect created `new Audio(bgmUrl)`, which triggers a network fetch to the RTS asset CDN regardless of the flag. The two router hooks (useRtsPromptRouter, useRtsPrGraphRouter) likewise opened tRPC subscriptions for every authenticated user. Wrap all four under a single RtsRoot boundary that checks RTS_FLAG once. When the flag is off, nothing mounts and no side effects fire.
- Operators can reopen a validated nest from the nest-detail footer, moving it back to active so the hedgehog resumes ticking. Reopen is guarded to validated nests (`nest_must_be_validated_to_reopen`); archived stays on unarchive and dormant is untouched. - The reopen carries the operator's "what's left to do" as a nest-chat user_message, which outranks the hedgehog's own plans. An instruction-less reopen synthesizes a hold directive instead, so the hedgehog waits for direction rather than reflexively re-validating a definition of done that is still met. - Create / unarchive / reopen now emit a distinct `activated` watch event; the tick scheduler forces an immediate tick on activation past the 30s debounce, while ordinary metadata saves keep emitting `status` and stay debounced — repeated name/goal/DoD saves no longer each spawn an LLM tick. - A forced activation that loses the in-flight race (a reopen landing mid-tick) is queued as a single follow-up and drained when the running tick exits, so immediate reopen never falls back to the 90s heartbeat; the no-concurrent-ticks-per-nest invariant still holds.
The "Active nests / goal territory", "Wilds / ad-hoc hoglets", and "Signal staging / unrouted signal work" labels hinted at game mechanics that don't exist — there's no rule that nests must be placed in the active zone, and the wilds/staging zones aren't meaningfully different from the player's perspective. Drop the labels and their backing fields on the Zone interface. Keep ZONES itself for the tint ellipses and varied prop scatter, which give the map visual texture independent of any naming.
MattBro
requested review from
a team,
Z3r0Sum,
seanosh,
sinthetix and
steventruong
- llm-gateway schemas.ts/service.ts: keep RTS betas+effort fields atop main's DEFAULT_GATEWAY_MODEL refactor - SidebarMenu.tsx: keep RTS store imports, drop now-unused getSessionService (main's useRenameTask refactor removed its call site) Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
- Hoglet runs and the hedgehog brain now plan and execute on claude-opus-4-8: DEFAULT_HOGLET_MODEL drives both the hoglet default and HEDGEHOG_MODEL. Reasoning effort is unchanged (stays at max). - Goal-spec drafting moves from claude-opus-4-6 to claude-opus-4-8, keeping the 1M-context beta (which 4-8 supports). - Heads-up: usage-pricing.ts has no claude-opus-4-8 entry yet, so RTS cost tracking for these runs falls back to $0 (with a warn) until real per-1M numbers are added.
Resolves 39 conflicts from the apps/code -> packages/{core,ui,workspace-server}
split. RTS flag constants move to @posthog/shared; RTS git-service methods and
schemas land in @posthog/workspace-server; rename-mapped UI files resolve onto
main's new import paths. RTS feature code still sits under apps/code and is
relocated to the new package layout in follow-up commits.
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
- RTS services move from apps/code/src/main/services/rts to packages/workspace-server/src/services/rts; RTS drizzle repositories to packages/workspace-server/src/db/repositories/rts, bound via rtsModule with standalone token consts (no MAIN_TOKENS bag). - Host-coupled deps are inverted as ports: RTS_AUTH (bound to the host AuthService), setRtsSettings()/setRtsRootLogger() module facades for the Electron settings store and logger. - The old main-process LlmGatewayService (with promptWithTools/betas/effort) becomes an RTS-local gateway client in workspace-server, since workspace-server may not import @posthog/core. - The rts tRPC router converts to packages/host-router/src/routers/rts.router.ts (ctx.container resolution), registered on the host router; llm-gateway router forwards betas/effort. - RTS migration regenerated as 0010_rts_schema on top of main's chain. - apps/code wires rtsModule, starts/stops the three RTS polling services in index.ts / AppLifecycleService. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
- apps/code/src/renderer/features/rts (164 files) and fun-mode move to packages/ui/src/features; hedgehog-mode sprites, rts images, and voice assets move to packages/ui/src/assets. - tRPC access converts from the old renderer trpcClient to the host-router seam: useHostTRPC()/useHostTRPCClient() in components, and a lazy resolveService(HOST_TRPC_CLIENT) hostClient for module-level callers. Pure RTS zod schemas the UI needs are re-exported via @posthog/host-router/rts-schemas (ui may not import workspace-server). - Ports the hedgemony changes that main's refactor displaced: RTS fullscreen chrome-hiding + RtsRoot mount into the router __root layout, the signal-trigger devtools console command into app-boot contributions, and hoglet retirement into the archive orchestration. - check-host-boundaries.mjs reports no new violations; ui/code/web typecheck clean; rts + fun-mode vitest suites pass (275 tests). Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
main added a react-doctor CI gate after the RTS components were written. Dialogs and AnimatedHedgehog now reset state during render with a prev-prop comparison instead of a useEffect; the sceneTicker subscription cleanup is explicit. The two remaining flags (cloud-workspace retry reset, framer-motion walk-state mirroring) are external-system orchestration, suppressed inline with justification. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Member
|
@MattBro hey brooker, what's the plan for this PR? |
GitHub's `shell: bash` runs with -e, so a nonzero npx exit (which react-doctor uses by design for blocking findings) aborted the scan step before exit-code/report outputs were written. The sticky-comment step then skipped (empty report) and the enforce step failed with empty STATUS — every PR with findings failed silently with no report in the log or on the PR. Capture the status inline and dump the report to the log on failure. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
|
React Doctor found 72 issues in 33 files · 72 warnings. 72 warnings
22 more warnings not shown. Reviewed by React Doctor for commit |
0.4.2's baseline comparison materializes only the changed files into a temp tree; when a PR touches repo-root files the temp root's package.json (no react dependency) becomes the scan target and the run dies with NoReactDependencyError before producing findings — every large PR failed silently. Reproduced locally with REACT_DOCTOR_BASE_SHA set; 0.5.1 completes baseline mode cleanly on the same input and the comment script handles its schemaVersion 2 report. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Author
@charlesvien A little more context here https://posthog.slack.com/archives/C09G8Q32R6F/p1780408741170189 I'd like to get it merged, do a couple iterations on it, then start to roll it out as sort of an easter egg. Depending on how usable it is, we could start to promote it a little. I'd also like to incorporate aspects of the new home page. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
7 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Problem
PostHog Code today is a list-of-tasks UI for orchestrating cloud agents. The Hogcraft/Hedgemony/RTS mode Barbados hackathon team (@Z3r0Sum ,@sinthetix,@seanosh,@steventruong, @MattBro) built an RTS-style alternative instead of a queue, you place "nests" (goal containers) on a 2D map and watch "hoglets" (agent runtimes) work the goal, with a hedgehog acting as the orchestrator. Internal codenames during development were "Hedgemony" and briefly "Hogcraft"; the official name landing here is PostHog Code RTS mode /
code-rts.It's hokey on purpose, but it surfaces orchestration state in a way the list view can't (idle vs working hoglets, PR dependency graph, FinOps cost overlay, signal ingestion). Goal of this PR: land what we have as the v1 surface so we can iterate stacked PRs on top. No expansion of scope beyond what shipped at the hackathon demo. This is behind a feature flag, so we can decide on scope as we go.
https://www.loom.com/share/eaaaec86d5b34b7f926b23a2e47c33e8
Changes
High-level
RTS_FLAG(rts-enabledPostHog feature flag). Off by default; toggle on the user/account for opt-in.rts.*as a host-router router atpackages/host-router/src/routers/rts.router.ts, backed by ~10 services inpackages/workspace-server/src/services/rts/(originally built underapps/code/src/main; relocated to match the package-split architecture from refactor: split electron app into several packages #2442).packages/ui/src/features/rts/(map view, sprites, dialogs, audio), reached through the Command Center map view and theRtsRootoverlay.RTS_AUTHbinds to the host AuthService, andsetRtsSettings()/setRtsRootLogger()adapt the Electron settings store and logger. RTS UI talks to the backend viauseHostTRPC/useHostTRPCClient.0010_rts_schema.sqlinpackages/workspace-server/src/db/migrations(Schmidt flattened the per-slice chain — see his commit message for the rationale). Numbered 0006 → 0007 → 0010 across merges with main as main shipped its own migrations in the interim.Static assets — hosted externally, not bundled
918 short voice mp3s (3 fun modes × 2 genders × ~50 lines × 3 takes) plus a BGM track are served from a Cloudflare R2 bucket
ph-code-rts, fronted bycode-rts.posthog.com:https://code-rts.posthog.com/static/code-rts/VITE_CODE_RTS_ASSETS_BASE_URLcloudflare/wrangler-actionfrom a separate PostHog/code-rts-assets repo (Schmidt owns).Voice lines were generated via ElevenLabs Multilingual v2 (
mp3_22050_32) under PostHog's ElevenCreative Starter commercial license. Voice IDs documented at the CDN inCREDITS.md. The generator script lives atscripts/rts/generate-voice.mjsand re-runs idempotently againstnotes/rts/voice-lines.json.Sub-features
RTS_FINOPS_FLAG(rts-finops-enabled) and an@posthog.comemail check — figures are raw cost, not consumer pricing.none/pirate/lolcatvoice line variants for the hoglets, per-gender.Feature flags involved
rts-enabledrts-finops-enabledBoth flags created in PostHog admin (US project 2).
How did you test this?
pnpm --filter code typecheck).#hackathon-hedgemonychannel for video clips.History: this branch has ~275 commits + a merge from main. Plan to squash-and-merge so main gets one clean commit.
Publish to changelog?
No, feature is off behind a flag and we'll publish to changelog when we promote to a default-on / GA experience in a follow-up PR.