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 ✍️ ✅ |
LaimaWu
force-pushed
the
docs/863-agent-guidance
branch
from
October 5, 2026 12:14
be0920b to
e972ed6
Compare
Author
|
I have read the CLA Document and I hereby sign the CLA |
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.
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.