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: is still a server module and still
tests the backend. Agents following this summary will route HBase work away as
unsupported/legacy, which is not the current project state.
##########
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:
does not exist in this repository. The OpenCypher sources live under , so
this shortcut currently sends agents to a dead path instead of the actual
implementation.
##########
.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 , still runs in server CI, still
contains , and explicitly requires updating the license and files when adding
third-party dependencies. With this config, Serena-based agents will miss or be
unable to complete those tasks.
--
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]
