docs: add AGENTS.md and reorganize project documentation - #6871
SeriousCoding789 wants to merge 33 commits into
Conversation
|
Two issues and one optional naming nit as below, please consider whether modifications are required.:
|
very good suggestion, will fix all. |
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
- Note the build fails fast on a JDK/arch mismatch - Scope the lint helper comment to framework main - Fix the generated-protobuf source path (core/, api/ subdirs) - Add the platform module (arch-specific source sets) to the layout - List both plugins jars (Toolkit, ArchiveManifest)
download() now prints a notice and returns non-zero; release files must be fetched and verified manually. It returns instead of exiting so that restart() still completes when rebuildManifest reaches it.
This reverts commit 800f084.
start.sh and start.sh.simple are not changed on this branch, so keep their guide and their rows in the README Executables table.
- Move the java.lang.Math check into .github/scripts/check_math_usage.sh and run it from both math-check.yml and AGENTS.md, so the self-check also flags fully qualified calls and imports. - Document the protoLint rule that the zero value of a new enum starts with UNKNOWN_. - Use an existing test method in the single-test example and drop protocol from the Checkstyle module list.
- Add the fields and enum values missing from Account, AccountResource, Transaction.Result, TransactionInfo, ResourceReceipt, InternalTransaction, SmartContract, ReasonCode and HelloMessage, with descriptions, and drop TransactionSign, which no longer exists. - Document the six Stake 2.0 contracts and the FreezeV2 / UnFreezeV2 messages. - State that the .proto files are the source of truth in the document, README, AGENTS.md and the outdated copies under protocol/src/main/protos.
- README: list p2p-standalone.jar in the executables table. - AGENTS.md and the modular introduction (en/zh): drop the libp2p version, the DNS providers, the tree:// URL format and how common exposes the module, since those can change.
What does this PR do?
AGENTS.md, a build, test and architecture guide for AI coding assistants and new contributors.p2pmodule: adds it to the module layout inAGENTS.mdand to the modular introduction (en / zh), links its README from the Documentation index, and listsp2p-standalone.jarunder Executables.docs/, and marks the two 2022 protocol copies as superseded..protofiles: missing fields and enum values, the six Stake 2.0 contracts, and the removal ofTransactionSign.docs/configuration.md.java.lang.Mathcheck into.github/scripts/check_math_usage.sh, run by bothmath-check.ymland theAGENTS.mdself-check..dockerignorefile.Why are these changes required?
p2pmodule..protofiles since 2023, mostly around Stake 2.0..codeclimate.ymlandsonar-project.propertiesare no longer used by CI.This PR has been tested by:
AGENTS.mdagainst the build configuration and sources, ran the pre-commit checklist commands, and confirmed all moved paths and links resolve.check_math_usage.shreports nothing on the repository and flags sample files usingjava.lang.Math.abs(-1),import java.lang.Math;and a static import. A script comparing every message and enum in the protocol document with the.protofiles reports no differences. Thep2pentries were checked against the build configuration, and the:p2pCheckstyle tasks referenced inAGENTS.mdpass.Follow up
None.
Extra details
None.