-
Notifications
You must be signed in to change notification settings - Fork 3
Agents-md-setup #106
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
YehonatanSaharof
wants to merge
8
commits into
master
Choose a base branch
from
agents-md-setup
base: master
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.
Open
Agents-md-setup #106
Changes from 6 commits
Commits
Show all changes
8 commits
Select commit
Hold shift + click to select a range
edbf9cc
chore: add AGENTS.md and CLAUDE.md
YehonatanSaharof f22d1f1
chore(interact): add non-standard pattern warnings to AGENTS.md
YehonatanSaharof 6f695c6
chore(interact): add API patterns from stress test
YehonatanSaharof 47e5995
Merge branch 'master' into agents-md-setup
YehonatanSaharof 7df46ed
lint
YehonatanSaharof b24945c
docs(interact): add llms.txt standard support
YehonatanSaharof cd659bb
fix
YehonatanSaharof f0fa3e0
Update AGENTS.md
YehonatanSaharof 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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,78 @@ | ||
| # AGENTS.md | ||
|
|
||
| Declarative web animation library — trigger-based interactions using the Web Animations API. | ||
|
|
||
| ## Commands | ||
|
|
||
| - **Install**: `yarn install` (Yarn 4 via Corepack) | ||
| - **Build**: `yarn build` (topological — packages first, then apps) | ||
| - **Test**: `yarn test` (runs Vitest across all packages) | ||
| - **Lint**: `yarn lint` (ESLint, zero warnings allowed) | ||
| - **Format**: `yarn format` (Prettier) / `yarn format:check` | ||
| - **Dev docs**: `yarn dev:docs` | ||
| - **Dev demo**: `yarn dev:demo` | ||
| - **Single workspace**: `yarn workspace @wix/<name> <script>` | ||
|
|
||
| ## Tech Stack | ||
|
|
||
| - TypeScript 5.9 (strict mode), Vite 7 (library mode, dual CJS/ESM), Vitest 4 | ||
| - React 18 (optional peer dep in `@wix/interact`) | ||
| - Package manager: Yarn 4.10.3 | ||
| - Node: >=18 | ||
|
|
||
| ## Structure | ||
|
|
||
| | Path | What it is | | ||
| | -------------------------- | ------------------------------------------------------------------------------------- | | ||
| | `packages/interact/` | Core interaction library (`@wix/interact`) — triggers, handlers, React/Web Components | | ||
| | `packages/motion/` | Animation engine (`@wix/motion`) — Web Animations API wrapper | | ||
| | `packages/motion-presets/` | 82+ animation presets for `@wix/motion` | | ||
| | `apps/demo/` | Demo app (Vite + React) | | ||
| | `apps/docs/` | Documentation site (Vite + React + React Router) | | ||
| | `packages/interact/rules/` | **Library interaction rules** (markdown specs for triggers) — NOT AI/cursor rules | | ||
| | `packages/interact/dev/` | Internal spec/plan documents — not shipped code | | ||
| | `packages/interact/docs/` | Package API docs, guides, and examples | | ||
|
|
||
| ## Conventions | ||
|
|
||
| - Use `fastdom` for all DOM reads/writes in animation code — never raw DOM access | ||
| - Use `??` for default values, not `||` (falsy values like `0` are valid in animation params) | ||
| - Never mutate function parameters — create copies when modifications are needed | ||
| - Prefer `export *` for barrel re-exports; be explicit about default vs named exports | ||
| - Update doc examples when changing implementation — reviewers catch stale docs | ||
| - Commit messages: `<type>(<scope>): <description>` — types: feat, fix, docs, refactor, test, chore — scopes: interact, motion, docs, demo | ||
| - Tests go in `packages/*/test/` as `*.spec.ts` / `*.spec.tsx` | ||
| - `@wix/interact` has three entry points: main (types/core), `/react` (React components), `/web` (Web Components) | ||
| - Presets require `Interact.registerEffects({ FadeIn })` before use — reference via `namedEffect` in configs; use `keyframeEffect` for inline keyframes | ||
| - `@wix/motion` wraps Web Animations API — use its API (`getWebAnimation`, `getScrubScene`), not native WAAPI | ||
| - In tests, use dynamic `import()` after `vi.resetModules()` — static imports won't pick up mocked modules | ||
| - `@wix/motion-presets` uses `export * as Namespace` — imports are namespace objects, not direct exports | ||
|
|
||
| ## Boundaries | ||
|
|
||
| **Always:** | ||
|
|
||
| - Run `yarn build && yarn test && yarn lint` before committing | ||
| - Add tests for new functions/features — reviewers always request this | ||
| - Keep PRs focused on one logical change | ||
|
|
||
| **Ask first:** | ||
|
|
||
| - Before adding new dependencies | ||
| - Before architectural changes or new APIs | ||
|
|
||
| **Never:** | ||
|
|
||
| - Edit `dist/` directories — they are build output | ||
| - Confuse `packages/interact/rules/` with AI rules — those are library interaction rule specs | ||
| - Import between apps — packages are the shared layer | ||
| - Use native Web Animations API directly — always go through `@wix/motion` | ||
|
|
||
| ## Documentation Index | ||
|
|
||
| | Doc | Contents | | ||
| | ---------------------------------------------------------------- | --------------------------------------------- | | ||
| | [CONTRIBUTING.md](CONTRIBUTING.md) | Dev setup, PR process, code standards | | ||
| | [packages/interact/docs/guides/](packages/interact/docs/guides/) | Getting started, config shape, usage patterns | | ||
| | [packages/interact/docs/](packages/interact/docs/) | API reference, guides, examples | | ||
| | [packages/motion/docs/](packages/motion/) | Motion API, animation categories | | ||
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,5 @@ | ||
| # CLAUDE.md | ||
|
|
||
| See [AGENTS.md](AGENTS.md) for guidance on working with this repository. | ||
|
|
||
| <!-- This file exists for Claude Code compatibility. AGENTS.md is the canonical source. --> |
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
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.
Uh oh!
There was an error while loading. Please reload this page.