feat(harness): integrate pi runtime into desktop app by jonathanlab · Pull Request #3469 · PostHog/code

jonathanlab · 2026-07-15T11:27:27Z

This sets up all the wiring needed to run Pi in the desktop app. We use a dummy conversation view, which is replaced by the ChatThread in the next PR.

it also alters taskmetadata to store pi session files, for local dev. this is probably a temporary solution

ALTER TABLE `task_metadata` ADD `pi_session_file` text;

jonathanlab · 2026-07-15T11:27:45Z

feat: hook up pi runtime to ChatThread #3474
feat(harness): integrate pi runtime into desktop app #3469 👈 (View in Graphite)
feat(harness): expose native Pi runtime #3391
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

github-actions · 2026-07-15T11:28:05Z

React Doctor found no issues in the changed files. 🎉

_{Reviewed by React Doctor for commit 92aa2f9.}

greptile-apps · 2026-07-15T11:36:51Z

Security Review

The new Pi-session routes do not validate task ownership before controlling sessions or returning events. The child process also receives a reusable OAuth credential snapshot through its environment, which can become invalid when refresh tokens rotate.

_{Reviews (1): Last reviewed commit: "feat(harness): integrate pi runtime into..." | Re-trigger Greptile}

greptile-apps · 2026-07-15T11:36:57Z

+  const oauthEnvironment: Record<string, string> = {
+    POSTHOG_OAUTH_CREDENTIALS: JSON.stringify(oauthCredentials),
+  };
+  const childEnvironment: Record<string, string> = {
+    ...env,
+    ...oauthEnvironment,
+  };



Child Credentials Diverge After Refresh

The child receives a one-time copy of the desktop access and refresh tokens. If the provider rotates refresh tokens and the parent refreshes first, the long-running child retains the invalidated token and its next refresh fails even though the desktop remains signed in, interrupting the Pi session.

veria-ai · 2026-07-17T09:19:40Z

PR overview

All previously flagged issues have been addressed. No open security concerns remain on this pull request.

Security review

No open security issues remain on this pull request.

Fixed/addressed: 3 · PR risk: 0/10

tatoalo

can we end up in a situation in which Pi tasks can start a competing ACP session and thus create some problems? If we are starting from the "simple" task view that's fine as well for now I guess

tatoalo · 2026-07-17T13:19:55Z

+        await this.deps.piRunner.create({
+          taskId: task.id,
+          cwd,
+          prompt: input.content ?? "",
+          model: input.model,


I believe we are missing the exec modes here?

jonathanlab · 2026-07-20T10:49:00Z

Merge activity

Jul 20, 10:49 AM UTC: A user started a stack merge that includes this pull request via Graphite.
Jul 20, 10:50 AM UTC: Graphite rebased this pull request as part of a merge.
Jul 20, 10:54 AM UTC: Graphite couldn't merge this PR because it was not satisfying all requirements (Failed CI: 'unit-test', 'typecheck', 'upload').
Jul 20, 2:16 PM UTC: A user started a stack merge that includes this pull request via Graphite.
Jul 20, 2:16 PM UTC: @jonathanlab merged this pull request with Graphite.

trunk-io · 2026-07-20T10:49:22Z

Merging to main in this repository is managed by Trunk.

To merge this pull request, check the box to the left or comment /trunk merge below.

After your PR is submitted to the merge queue, this comment will be automatically updated with its status. If the PR fails, failure details will also be posted here

posthog · 2026-07-20T10:55:19Z

👋 Visual changes detected for this PR.

Review and approve in PostHog Visual Review

If these changes are unexpected, they may be caused by a flaky test or a broken snapshot on master. Don't approve — rerun the job or wait for a fix.

jonathanlab mentioned this pull request Jul 15, 2026

feat(harness): expose native Pi runtime #3391

Merged

jonathanlab marked this pull request as ready for review July 15, 2026 11:34

greptile-apps Bot reviewed Jul 15, 2026

View reviewed changes

jonathanlab changed the base branch from pi-harness-runtime to graphite-base/3469 July 15, 2026 11:45

jonathanlab force-pushed the graphite-base/3469 branch from a4f25c4 to cd593d5 Compare July 15, 2026 14:00

jonathanlab force-pushed the 07-15-feat_harness_integrate_pi_runtime_into_desktop_app branch from 745cd4b to b8845b7 Compare July 15, 2026 14:00

jonathanlab changed the base branch from graphite-base/3469 to pi-harness-runtime July 15, 2026 14:00

jonathanlab mentioned this pull request Jul 15, 2026

feat: hook up pi runtime to ChatThread #3474

Merged

2 tasks

jonathanlab changed the base branch from pi-harness-runtime to graphite-base/3469 July 15, 2026 14:15

jonathanlab force-pushed the 07-15-feat_harness_integrate_pi_runtime_into_desktop_app branch from b8845b7 to 9485f63 Compare July 15, 2026 14:23

jonathanlab force-pushed the graphite-base/3469 branch from cd593d5 to 84dccb9 Compare July 15, 2026 14:23

jonathanlab changed the base branch from graphite-base/3469 to pi-harness-runtime July 15, 2026 14:23

jonathanlab changed the base branch from pi-harness-runtime to graphite-base/3469 July 15, 2026 14:37

jonathanlab force-pushed the graphite-base/3469 branch from 84dccb9 to 0e925bf Compare July 15, 2026 14:42

jonathanlab force-pushed the 07-15-feat_harness_integrate_pi_runtime_into_desktop_app branch from 9485f63 to 401dd8d Compare July 15, 2026 14:42

jonathanlab changed the base branch from graphite-base/3469 to pi-harness-runtime July 15, 2026 14:42

jonathanlab force-pushed the 07-15-feat_harness_integrate_pi_runtime_into_desktop_app branch 2 times, most recently from ed3507d to 3185f38 Compare July 17, 2026 09:03

veria-ai Bot reviewed Jul 17, 2026

View reviewed changes

Comment thread packages/agent/src/pi/rpc-client.ts Outdated

jonathanlab force-pushed the 07-15-feat_harness_integrate_pi_runtime_into_desktop_app branch 2 times, most recently from fc88402 to 32ed95f Compare July 17, 2026 09:43

veria-ai Bot reviewed Jul 17, 2026

View reviewed changes

Comment thread apps/code/src/main/platform-adapters/desktop-pi-rpc-client-factory.ts

jonathanlab force-pushed the 07-15-feat_harness_integrate_pi_runtime_into_desktop_app branch 2 times, most recently from 8c9355a to 8af4881 Compare July 17, 2026 10:01

jonathanlab force-pushed the pi-harness-runtime branch from 0e925bf to 3c236f3 Compare July 17, 2026 10:01

veria-ai Bot reviewed Jul 17, 2026

View reviewed changes

Comment thread packages/harness/src/runtime.ts

jonathanlab changed the base branch from pi-harness-runtime to graphite-base/3469 July 17, 2026 10:39

jonathanlab force-pushed the 07-15-feat_harness_integrate_pi_runtime_into_desktop_app branch from 8af4881 to 75ec760 Compare July 17, 2026 10:43

jonathanlab force-pushed the graphite-base/3469 branch from 3c236f3 to 7388b0d Compare July 17, 2026 10:43

jonathanlab changed the base branch from graphite-base/3469 to pi-harness-runtime July 17, 2026 10:43

jonathanlab force-pushed the 07-15-feat_harness_integrate_pi_runtime_into_desktop_app branch from 75ec760 to dfb6df0 Compare July 17, 2026 10:44

jonathanlab changed the base branch from pi-harness-runtime to graphite-base/3469 July 17, 2026 11:16

jonathanlab force-pushed the graphite-base/3469 branch from 7388b0d to 1e4cd6d Compare July 17, 2026 11:18

jonathanlab force-pushed the 07-15-feat_harness_integrate_pi_runtime_into_desktop_app branch from dfb6df0 to 98059aa Compare July 17, 2026 11:18

jonathanlab changed the base branch from graphite-base/3469 to pi-harness-runtime July 17, 2026 11:18

tatoalo reviewed Jul 17, 2026

View reviewed changes

tatoalo approved these changes Jul 17, 2026

View reviewed changes

jonathanlab changed the base branch from pi-harness-runtime to graphite-base/3469 July 20, 2026 10:49

jonathanlab changed the base branch from graphite-base/3469 to main July 20, 2026 10:49

feat(harness): integrate pi runtime into desktop app

fb86dd5

jonathanlab force-pushed the 07-15-feat_harness_integrate_pi_runtime_into_desktop_app branch from 98059aa to fb86dd5 Compare July 20, 2026 10:49

fix(core): pass PiRunner to TaskService in test

92aa2f9

jonathanlab merged commit 4f0acfb into main Jul 20, 2026
29 of 32 checks passed

jonathanlab deleted the 07-15-feat_harness_integrate_pi_runtime_into_desktop_app branch July 20, 2026 14:17

Uh oh!

Conversation

jonathanlab commented Jul 15, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

jonathanlab commented Jul 15, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

github-actions Bot commented Jul 15, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

greptile-apps Bot commented Jul 15, 2026

Security Review

Uh oh!

Uh oh!

Uh oh!

Uh oh!

greptile-apps Bot Jul 15, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

veria-ai Bot commented Jul 17, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

PR overview

Security review

Uh oh!

Uh oh!

Uh oh!

tatoalo left a comment

Choose a reason for hiding this comment

Uh oh!

tatoalo Jul 17, 2026

Choose a reason for hiding this comment

Uh oh!

jonathanlab commented Jul 20, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Merge activity

Uh oh!

trunk-io Bot commented Jul 20, 2026

Uh oh!

posthog Bot commented Jul 20, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

jonathanlab commented Jul 15, 2026 •

edited

Loading

jonathanlab commented Jul 15, 2026 •

edited

Loading

github-actions Bot commented Jul 15, 2026 •

edited

Loading

veria-ai Bot commented Jul 17, 2026 •

edited

Loading

jonathanlab commented Jul 20, 2026 •

edited

Loading

posthog Bot commented Jul 20, 2026 •

edited

Loading