Skip to content

fix(tasks): accept pi model configuration - #75898

Open
jonathanlab wants to merge 1 commit into
masterfrom
fix/tasks-pi-model-bootstrap
Open

fix(tasks): accept pi model configuration#75898
jonathanlab wants to merge 1 commit into
masterfrom
fix/tasks-pi-model-bootstrap

Conversation

@jonathanlab

@jonathanlab jonathanlab commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

Problem

Cloud Pi runs did not accept a model or thinking level at start. The API required an ACP adapter. New Pi tasks then used the default model.

Changes

  • Accept Pi model and thinking configuration without an ACP adapter.
  • Save this configuration in the run state before the sandbox starts.
  • Reject thinking levels that Pi does not support.

How did you test this code?

  • Ran Ruff, formatting, and Python compilation.
  • Added API tests for Pi model configuration and invalid Pi thinking levels.
  • The Django API tests did not start because this environment does not have sqlx.

👉 Stay up-to-date with PostHog coding conventions for a smoother review.

Automatic notifications

  • Publish to changelog?
  • Alert Sales and Marketing teams?

Docs update

No documentation update is needed.

🤖 Agent context

Autonomy: Human-driven (agent-assisted)

I used Pi coding agent tools. I used /improving-drf-endpoints and /writing-tests. The task-run serializer sets initial Pi configuration. The client uses Pi RPC for live and deferred changes.

@jonathanlab jonathanlab self-assigned this Jul 31, 2026
@trunk-io

trunk-io Bot commented Jul 31, 2026

Copy link
Copy Markdown

✨ Submitted to Merge by @jonathanlab. It will be added to the merge queue once all branch protection rules pass and there are no merge conflicts with the target branch. See more details here.

@jonathanlab jonathanlab added the skip-inkeep-docs Use this label to skip an Inkeep docs PR in posthog.com label Jul 31, 2026
@jonathanlab
jonathanlab marked this pull request as ready for review July 31, 2026 09:35
@pr-assigner-resolver-posthog
pr-assigner-resolver-posthog Bot requested a review from a team July 31, 2026 09:36
@greptile-apps

greptile-apps Bot commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

Reviews (1): Last reviewed commit: "fix(tasks): accept pi model configuratio..." | Re-trigger Greptile

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip-inkeep-docs Use this label to skip an Inkeep docs PR in posthog.com

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants