chore(devex): stop instructing agents to run hogli review - #90039
Merged
Conversation
The local Greptile review needs a per-person seat on PostHog's Greptile workspace, and seats are granted by hand today. Instructing every agent to run it before opening a PR sends most people through a sign-up flow that creates their own workspace instead. Drop the always-invoke wiring (AGENTS.md section, mandatory-skill entry, PR template bullet, ci:preflight nudge) and make the skill opt-in. The command itself and the PR bot are unchanged. Generated-By: PostHog Desktop Task-Id: 67296b75-291f-4056-bc95-d9f93e4785f6
|
😎 Merged successfully - details. |
webjunkie
approved these changes
Aug 27, 2026
Contributor
🤖 CI report
|
gantoine
marked this pull request as ready for review
August 27, 2026 12:19
Contributor
|
Reviews (1): Last reviewed commit: "chore(devex): stop instructing agents to..." | Re-trigger Greptile |
This was referenced Aug 27, 2026
gantoine
added a commit
that referenced
this pull request
Aug 28, 2026
A task that only said to open a PR matched the skill's description, which then pointed at the harness code review. That fans out parallel agents and cost one person over a hundred dollars on a 2,400-line diff. #90039 and #90504 already removed the mandatory rules and the hogli review command. Nothing else references the skill. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
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
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
hogli reviewneeds a per-person seat on PostHog's Greptile workspace, and seats are granted by hand right now (SSO is an enterprise feature we don't have). AGENTS.md told every agent to run it before opening a PR, so anyone without a seat gets walked through a sign-up flow that creates their own personal Greptile workspace rather than joining PostHog's — which is what happened to the first person who hit it.Changes
hogli reviewwhen told to open a PR. The Greptile bot reviews PRs as it did before this wiring landed.reviewing-before-prskill stays, but is now opt-in: it fires only when someone asks for a local Greptile review by name, and says up front that a seat is needed.hogli ci:preflightno longer nudges towardhogli reviewon code diffs.The
hogli reviewcommand, the flox install of the Greptile CLI, and the.greptile/config are untouched — people with a seat can still run it.How did you test this code?
uv run pytest tools/hogli-commands/hogli_commands/tests/test_ci_preflight.py— 32 passed. The nudge test now covers mypy only.ruff checkandruff format --checkclean on both changed Python files.Automatic notifications
Docs update
None — all surfaces here are agent guidance in AGENTS.md and
.agents/skills/.🤖 Agent context
Autonomy: Human-driven (agent-assisted)
Created with PostHog from a Slack thread