feat(harness): integrate pi runtime into desktop app#3469
Conversation
|
Warning This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
This stack of pull requests is managed by Graphite. Learn more about stacking. |
|
React Doctor found no issues in the changed files. 🎉 Reviewed by React Doctor for commit |
|
| const oauthEnvironment: Record<string, string> = { | ||
| POSTHOG_OAUTH_CREDENTIALS: JSON.stringify(oauthCredentials), | ||
| }; | ||
| const childEnvironment: Record<string, string> = { | ||
| ...env, | ||
| ...oauthEnvironment, | ||
| }; | ||
|
|
There was a problem hiding this comment.
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.
a4f25c4 to
cd593d5
Compare
745cd4b to
b8845b7
Compare
b8845b7 to
9485f63
Compare
cd593d5 to
84dccb9
Compare
84dccb9 to
0e925bf
Compare
9485f63 to
401dd8d
Compare
401dd8d to
ed3507d
Compare

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
ChatThreadin the next PR.it also alters
taskmetadatato temporarily include these two:Until we distinguish by runtime via posthog django models