VGalaxies commented on code in PR #2998: URL: https://github.com/apache/hugegraph/pull/2998#discussion_r3107042988
########## AGENTS.md: ########## @@ -1,261 +1,103 @@ # AGENTS.md -This file provides guidance to an AI coding tool when working with code in this repository. +Development reference for AI coding agents. See [README.md](README.md) if the task really needs project overview or architecture context. -## Project Overview +## Key Architectural Patterns -Apache HugeGraph is a fast-speed and highly-scalable graph database that supports billions of vertices and edges. It is compliant with Apache TinkerPop 3 and supports both Gremlin and Cypher query languages. +1. **Pluggable Backend Architecture**: Storage backends implement the `BackendStore` interface — new backends require no core changes. Active backends: RocksDB (default/embedded), HStore (distributed) Review Comment: The condensed guide now presents RocksDB/HStore as the only active backends, which conflicts with the current tree: `hugegraph-hbase` is still a server module and `server-ci.yml` still tests the `hbase` backend. Agents following this summary will route HBase work away as unsupported/legacy, which is not the current project state. ########## .serena/project.yml: ########## @@ -20,22 +28,40 @@ languages: # For a list of possible encodings, see https://docs.python.org/3.11/library/codecs.html#standard-encodings encoding: "utf-8" -# whether to use the project's gitignore file to ignore files -# Added on 2025-04-07 +# whether to use project's .gitignore files to ignore files ignore_all_files_in_gitignore: true -# list of additional paths to ignore -# same syntax as gitignore, so you can use * and ** -# Was previously called `ignored_dirs`, please update your config if you are using that. -# Added (renamed) on 2025-04-07 -ignored_paths: [] +# list of additional paths to ignore in this project. +# Same syntax as gitignore, so you can use * and **. +# Note: global ignored_paths from serena_config.yml are also applied additively. +ignored_paths: + # --- Deprecated backends (focus on RocksDB/HStore only) --- + - "hugegraph-server/hugegraph-cassandra/**" + - "hugegraph-server/hugegraph-scylladb/**" + - "hugegraph-server/hugegraph-mysql/**" + - "hugegraph-server/hugegraph-postgresql/**" + - "hugegraph-server/hugegraph-palo/**" + - "hugegraph-server/hugegraph-hbase/**" + # --- gRPC generated Java (235k lines, never hand-edited, regenerated by mvn compile) --- + - "hugegraph-pd/hg-pd-grpc/src/main/java/**" + - "hugegraph-store/hg-store-grpc/src/main/java/**" + # --- License/legal files (585 txt files, only touched when adding dependencies) --- + - "install-dist/release-docs/licenses/**" + - "install-dist/scripts/dependency/known-dependencies.txt" + # --- Rarely modified tests/examples --- + - "hugegraph-server/hugegraph-test/**/tinkerpop/**" + - "hugegraph-server/hugegraph-example/**" + - "hugegraph-cluster-test/**" Review Comment: This ignore list now removes HBase, cluster tests, and the dependency-compliance paths from Serena's project view. That is broader than 'dead/generated code': the repo still ships `hugegraph-hbase`, still runs `hbase` in server CI, still contains `hugegraph-cluster-test`, and `AGENTS.md` explicitly requires updating the license and `known-dependencies.txt` files when adding third-party dependencies. With this config, Serena-based agents will miss or be unable to complete those tasks. ########## AGENTS.md: ########## @@ -1,261 +1,103 @@ # AGENTS.md -This file provides guidance to an AI coding tool when working with code in this repository. +Development reference for AI coding agents. See [README.md](README.md) if the task really needs project overview or architecture context. -## Project Overview +## Key Architectural Patterns -Apache HugeGraph is a fast-speed and highly-scalable graph database that supports billions of vertices and edges. It is compliant with Apache TinkerPop 3 and supports both Gremlin and Cypher query languages. +1. **Pluggable Backend Architecture**: Storage backends implement the `BackendStore` interface — new backends require no core changes. Active backends: RocksDB (default/embedded), HStore (distributed) +2. **gRPC Communication**: All distributed components (PD, Store, Server) communicate via gRPC. Proto definitions in `*/grpc/` directories +3. **Multi-Language Queries**: Native Gremlin support + OpenCypher implementation in `hugegraph-api/opencypher/` Review Comment: `hugegraph-api/opencypher/` does not exist in this repository. The OpenCypher sources live under `hugegraph-server/hugegraph-api/src/main/java/org/apache/hugegraph/opencypher/`, so this shortcut currently sends agents to a dead path instead of the actual implementation. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
