Skip to content

fix(ci): build harness before publishing agent#3618

Merged
trunk-io[bot] merged 1 commit into
mainfrom
posthog-code/fix-agent-release-harness
Jul 21, 2026
Merged

fix(ci): build harness before publishing agent#3618
trunk-io[bot] merged 1 commit into
mainfrom
posthog-code/fix-agent-release-harness

Conversation

@tatoalo

@tatoalo tatoalo commented Jul 21, 2026

Copy link
Copy Markdown
Contributor

Problem

Agent releases fail while building the bundled Pi RPC host because @posthog/harness has not been built. The package is also listed as a runtime dependency even though it is bundled and is not published separately.

Changes

Build harness before the agent and keep it as a build-time dependency.

@trunk-io

trunk-io Bot commented Jul 21, 2026

Copy link
Copy Markdown

😎 Merged successfully - details.

@tatoalo
tatoalo force-pushed the posthog-code/fix-agent-release-harness branch from 88c2bde to 100f30e Compare July 21, 2026 12:22
@github-actions

Copy link
Copy Markdown

React Doctor found no issues in the changed files. 🎉

Reviewed by React Doctor for commit 100f30e.

@tatoalo tatoalo self-assigned this Jul 21, 2026
@tatoalo tatoalo added the Stamphog This will request an autostamp by stamphog on small changes label Jul 21, 2026
@tatoalo
tatoalo requested a review from a team July 21, 2026 12:24
@tatoalo
tatoalo marked this pull request as ready for review July 21, 2026 12:24

@stamphog stamphog Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Gates denied this PR because it modifies CI/CD workflows and dependency toolchain files, which are in the deny-list. These changes require human review — the CI workflow edit and the dependency reclassification from devDependencies to dependencies both need a team member to verify correctness and intent.

@stamphog stamphog Bot removed the Stamphog This will request an autostamp by stamphog on small changes label Jul 21, 2026
@greptile-apps

greptile-apps Bot commented Jul 21, 2026

Copy link
Copy Markdown
Contributor

Reviews (1): Last reviewed commit: "fix(ci): build harness before publishing..." | Re-trigger Greptile

@tatoalo
tatoalo enabled auto-merge (squash) July 21, 2026 12:27
@trunk-io
trunk-io Bot merged commit 9222c50 into main Jul 21, 2026
42 checks passed
@trunk-io
trunk-io Bot deleted the posthog-code/fix-agent-release-harness branch July 21, 2026 12:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants