-
-
Notifications
You must be signed in to change notification settings - Fork 3.5k
Add Reference First rule to Build Tools and Development #334
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Open
southyang99
wants to merge
1
commit into
PatrickJS:main
Choose a base branch
from
southyang99:main
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
+26
−0
Open
Changes from all commits
Commits
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,25 @@ | ||
| --- | ||
| description: "Scout a pair of references (near-domain + far-domain) and extract transferable structure before building anything new" | ||
| globs: ["**/*"] | ||
| alwaysApply: false | ||
| --- | ||
| # Reference First | ||
|
|
||
| Before building anything new, scout references first, then build. Force the step to happen and point the search in a useful direction; which references to pick and what to conclude stays with the user. | ||
|
|
||
| ## When to trigger | ||
|
|
||
| Output a short "Reference Scouting Brief" BEFORE proposing a solution or writing code when: starting a new project or product, adding a new feature, interaction paradigm, or page type, or entering an unfamiliar domain (new algorithm, category, or platform convention). | ||
|
|
||
| Do NOT trigger for bug fixes, changes that follow an existing in-project pattern, or pure copy tweaks. When unsure, ask "should we run the reference step this time?" — do not trigger by default. | ||
|
|
||
| ## The Reference Scouting Brief (5–8 lines, no longer) | ||
|
|
||
| 1. What type to look for — pick 1–2 based on the core uncertainty: interaction pattern, information architecture, visual style, data structures and algorithms, copy tone, or pricing. | ||
| 2. Where to look — comparable products, design libraries, open-source implementations, far-domain analogs. | ||
| 3. Find a pair, not one: one near-domain reference plus one far-domain reference; state the analogy logic for the far one ("Y in domain X is structurally the same problem because…"). | ||
| 4. Comparison questions — what structure do both get right? which parts are transferable principle versus surface style? how have similar products actually fared? | ||
|
|
||
| ## Discipline | ||
|
|
||
| Cap scouting at 2–3 searches; verify references with tools rather than memory; if nothing solid turns up, say so instead of padding. The user can say "skip references" at any time. Before building, compress the findings into 3–5 lines: what to steal, what to avoid, what to deliberately not copy. | ||
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🎯 Functional Correctness | 🟠 Major | ⚡ Quick win
Clarify the rule’s trigger precedence.
“Force the step to happen” conflicts with “do not trigger by default” and “the user can say skip references.” State one policy explicitly: trigger automatically only when the listed conditions match, ask when applicability is ambiguous, and always honor an explicit skip.
Also applies to: 23-25
🤖 Prompt for AI Agents