Repository navigation
docs: add shared AI agent instructions (#863) - #886
Conversation
|
👋 Welcome, @LaimaWu, and thanks for opening your first PR on AnythingMCP! A few quick pointers:
Someone from the core team will look at this within ~48h. If you don't hear back, please ping us in Discussions / Q&A. ⭐ While you wait — if you find AnythingMCP useful, a star helps others discover it. |
|
All contributors have signed the CLA ✍️ ✅ |
be0920b to
e972ed6
Compare
|
I have read the CLA Document and I hereby sign the CLA |
keysersoft
left a comment
There was a problem hiding this comment.
Went through it against main: every path and command you reference exists and matches what CI runs, and the ee/ and CLA bits are right at the top where an agent will actually read them. Good catch on test:e2e pointing at a config that isn't there, I'll clean that script up separately so it doesn't stay a trap.
Thanks @LaimaWu, nice work. I'll merge it shortly.
Summary
Agents currently guess workspace commands and miss contribution boundaries. Add shared root instructions for setup, checks, adapter changes, deployment modes, and PR rules, with the commercial
ee/boundary and one-time CLA requirement at the top.Fixes #863.
Changes
AGENTS.mdwith commands tested on a fresh upstream clone.CLAUDE.mdcontaining exactly@AGENTS.mdand a newline.Type
Testing
Fresh clone:
fefac941e21b16eb84ddd8b0c0d7d1814f9f0fdf; Node 24.19.0, npm 11.9.0. Results below apply to that exact base. All commands in the guide were executed; failures are reported rather than labeled as passing. Root means the fresh clone's root; backend/frontend meanpackages/backend/packages/frontend../setup.shdocker compose -f docker-compose.yml -f docker-compose.dev.yml up -d postgresnpm ciset -a,. ../../.env,set +anpx prisma migrate deploynpx prisma generate.env; bare invocation without exportedDATABASE_URLfailed (1).npm run dev/healthon :4000 and frontend on :3000 returned HTTP 200. Deliberately stopped with SIGTERM after readiness probes.npm run lint--fixrewrote two non-EE spec files. Saved evidence and restored both.npx tsc --noEmit -p tsconfig.jsonnpm testnpm run lintnpx tsc --noEmit -p tsconfig.jsonnpx playwright install --with-deps chromiumsu.npx playwright install chromiumnpm run test:e2enpm run adapter:new -- my-service --region intl --auth API_KEYnode scripts/validate-adapters.mjs --warnnode scripts/regenerate-catalog.mjsnode scripts/adapter-count.mjs --checknpm testtestscript; backend portion also had 3 failing assertions, 6,661 passed / 206 skipped.npm run test:e2etest/jest-e2e.jsonis missing.npm test -- --runInBand adapters/catalog.spec.tsnode --test scripts/validate-adapters.test.mjs scripts/adapter-new.test.mjsgit diff --checkExisting documentation discrepancies and verification notes
npm testas "All tests" and uses it for the catalog/PR requirements, but the frontend lacks atestscript. The guide specifies the backend working directory. Root scripts are unchanged.--warnshould be "clean"; the fresh base already has 1,247 warnings despite exit zero. The guide requires addressing warnings introduced by a new adapter.engines.noderequires >=22.12. The guide uses the stricter requirement.DATABASE_URL; a symlink alone did not make bare generation work. The guide shows exporting the trusted local.env. For manual setup, configure localhost/port 5433 instead of the example Docker database hostname.useEdition()returns null on Cloud; it actually returns an object whoseeditionfield is null. The guide describes the implementation.npm installchanged lockfile metadata under npm 11, and Next dev generated frontend instruction files and changednext-env.d.ts. All incidental changes/artifacts were removed. The guide flags generated Next files.Checklist
CLAUDE.mdbytes are exactly@AGENTS.md\nand CONTRIBUTING has one added line.Self-hosted and Cloud behavior are unchanged. Complete the one-time CLA check when the PR is opened.
Upstream advanced during verification to
df46b68e5c6cbb718fb128b6ec04cbb3e1d0ba67. None of the three PR paths changed there, so this documentation diff remains conflict-free; the command results above are explicitly for the recorded tested base.Publication update: the documentation commit was rebased without conflicts onto
39a868cdfa36c8a2ce803ac8208438614be74383. The matrix above remains unchanged and applies to the original fresh-clone test baseline. Since upstream extended the adapter validator metadata fields, supplemental checks on the updated branch passed:node scripts/validate-adapters.mjs --warn(exit 0; 268 passed, 1,247 warnings) andnode --test scripts/validate-adapters.test.mjs scripts/adapter-new.test.mjs(exit 0; 37 passed). Final scope, exact CLAUDE bytes, the single CONTRIBUTING link, unchanged EE contents, clean worktree, andgit diff --checkall passed. The full command matrix was not rerun.