Repository navigation
docs(ai): add lessons migrated from claude memory - #6806
caseyisonit wants to merge 1 commit into
Conversation
|
📚 Branch Preview Links🔍 Gen1 Visual Regression Test ResultsWhen a visual regression test fails (or has previously failed while working on this branch), its results can be found in the following URLs:
Deployed to Azure Blob Storage: If the changes are expected, update the |
05a7487 to
e13fc38
Compare
e13fc38 to
86cdac4
Compare
86cdac4 to
c2afadd
Compare
c2afadd to
1e1981a
Compare
1e1981a to
275a23e
Compare
0ccb690 to
d1eccd9
Compare
d1eccd9 to
81cc95f
Compare
Description
Adds three lessons from my Claude memory to
.ai/memory/agnostic-lessons.md, so every agent that reads.ai/gets them, not only Claude.feedback-swc-styling-api-prefix): gen2 components expose--swc-*custom properties as their public styling API, for example--swc-badge-heightinBadge.ts.--mod-*and--spectrum-*are 1st-gen and Spectrum CSS conventions and belong in gen2 docs only in migration-guide comparisons. The lesson notes that Stylelint'scustom-property-patternaccepts any lowercase kebab-case name, so the prefix isn't linted.feedback-update-stale-generated-diffs): whenyarn buildrewrites generated output, such asgen2/packages/swc/stylesheets/global/global-*.cssfromgen2/packages/tools/vite-global-elements-css, commit the regenerated file even if the diff looks unrelated. Revert it only if it contradicts its source.feedback-no-hardwrap-markdown-docs): don't hard-wrap prose in.mdand.mdx..prettierrc.yamlleavesproseWrapat its default,preserve, so Prettier never reflows hand-wrapped lines.The
--swc-*lesson lives inagnostic-lessonsrather thancss-styling-lessonsbecause it governs MDX docs,@csspropJSDoc in.tsfiles, and examples.css-styling-lessonsloads only for**/*.css, so it would never be in context while editing those files.The frontmatter
descriptionnow names the two new categories.yarn ai:syncregenerated the Copilot instruction, the Cursor rule, and the.ai/README.mdcatalog entry. No other files were edited by hand.Not migrated:
@pwrs/cemisn't used in gen2.icon-rfc.mdno longer exists.Motivation and context
These lessons came from review corrections, but they lived only in one person's Claude memory, so Copilot, Cursor, and other contributors' agents didn't get them. Moving them into
.ai/memory/makes them path-scoped instructions that every supported tool loads.Related issue(s)
Screenshots (if appropriate)
Not applicable: no visual changes.
Author's checklist
yarn lint:aichecks the lessons frontmatter and generated-file drift.Reviewer's checklist
patch,minor, ormajorfeaturesManual review test cases
Generated files are in sync
yarn ai:sync --check, thenyarn lint:ai.Copilot loads the lessons
copilot instruction list.memory-agnostic-lessons.instructions.mdfrom.github/instructions/.Each lesson is in the generated instruction
.github/instructions/memory-agnostic-lessons.instructions.mdfor## Styling API,## Generated files, andDon't hard-wrap prose.Verified by the author:
yarn ai:sync --check,yarn lint:ai, Prettier, andcopilot instruction listpass, and each lesson appears once in the generated instruction.Device review
Accessibility testing checklist
This PR changes only agent instruction files. It adds and changes no components, stories, or rendered UI.