Repository navigation
docs: add AGENTS.md and reorganize project documentation - #6871
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)
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.
* release_v4.8.3: feat(framework): remove --keystore-factory support from FullNode (tronprotocol#6978) refactor(proto): remove unused proto discover definition (tronprotocol#7014) fix(config): close resource streams (tronprotocol#7002) refactor(crypto): remove unused SM2 and SM3 (tronprotocol#6999) feat(plugins): backfill section bloom (tronprotocol#6973) refactor(api): remove dead WalletExtension gRPC service and config (tronprotocol#6975) refactor(framework): decouple Manager from TronJsonRpcImpl (tronprotocol#6990) refactor(api): replace response copy wrappers with Jetty content count (tronprotocol#6982) update a new version. version name:GreatVoyage-v4.8.2.2-1-gf3e81404fe,version code:18830 (tronprotocol#7008) feat: improve node stability and execution efficiency (tronprotocol#7007)
* release_v4.8.3: feat(crypto): harden ECKey validation (tronprotocol#6996) fix(config): fail startup on invalid DNS publish parameters (tronprotocol#7019) fix(jsonrpc): normalize error responses and fatal handling (tronprotocol#6985) fix(jsonrpc): handle null parameters consistently (tronprotocol#6984) refactor(trie): remove unused RLP utilities (tronprotocol#6991)
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.