fix(self-driving): Rename "canonical" scouts to "built-in" in self-driving prompt#717
Merged
Merged
Conversation
Mirrors the context-mill rename (PostHog/context-mill#199): swaps the jargon word "canonical" for the plainer "built-in" in the self-driving program prompt, and updates the two stale decline-label quotes in ARCHITECTURE.md to match the new label. Leaves backend mechanism names (sync_canonical_skills, the seeded_by discriminator) unchanged. Generated-By: PostHog Code Task-Id: ed014c8d-4c01-499b-879e-de953dc34179
🧙 Wizard CIRun the Wizard CI and test your changes against wizard-workbench example apps by replying with a GitHub comment using one of the following commands: Test all apps:
Test all apps in a directory:
Test an individual app:
Show more apps
Results will be posted here when complete. |
sortafreel
changed the title
Contributor
There was a problem hiding this comment.
Pull request overview
Updates self-driving program copy to replace the user-confusing term “canonical” with “built-in”, aligning the wizard’s prompt language with the corresponding context-mill skill vocabulary.
Changes:
- Updated the self-driving agent prompt to refer to the “built-in troop” and “built-in scout bodies”.
- Updated self-driving architecture documentation quotes of the decline label to use “built-in”.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
src/lib/programs/self-driving/prompt.ts |
Renames “canonical” to “built-in” in STEP 7 instructions so the agent’s guidance matches updated UX terminology. |
src/lib/programs/self-driving/ARCHITECTURE.md |
Updates documentation references to the decline label to use “built-in” for consistency with the prompt/skill language. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Comment on lines
67
to
+68
| 6. **Configure scout troop** — materialize the canonical troop, then enable a deliberately small set: `general` (always) + the **1–2 specialists** for the products this project uses most; never `error-tracking`/`session-replay` (consumed as native sources); disable the rest. The enabled troop lands at **2–5** (general + 1–2 specialists + 0–2 custom). | ||
| 7. **Design custom scouts** — gap-analyze the repo against the troop, propose **at most 2** candidates in one ask (each a plain-language `label` + a dimmed `description`, behind a leading "None — keep the canonical troop" default option), create the approved subset (the only place custom scouts are made). | ||
| 7. **Design custom scouts** — gap-analyze the repo against the troop, propose **at most 2** candidates in one ask (each a plain-language `label` + a dimmed `description`, behind a leading "None — keep the built-in troop" default option), create the approved subset (the only place custom scouts are made). |
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
During cloud onboarding testing, "canonical" was flagged as confusing UX — it is not a word product users know. The self-driving program prompt and the user-facing decline label used it. The user-facing label fix lives in PostHog/context-mill#199; this mirrors the vocabulary in the wizard so the agent's instructions match the skill it drives.
Changes
prompt.ts: "canonical troop" / "canonical scout bodies" → "built-in".ARCHITECTURE.md: updated the two stale quotes of the decline label (None — keep the canonical troop→built-in) to match.sync_canonical_skills, theseeded_bycanonical-vs-custom discriminator) unchanged — those are code-level identifiers, not user-facing copy.Test plan
Docs/prompt-text only; no behavior change. Covered by CI.
Created with PostHog Code from a Slack thread