Skip to content

Commit a0403cf

Browse files
earayuclaude
andauthored
ci(compat): trigger Backend-Compat-Test on real graph_storage path (#1926)
The pull_request paths filter on compat-test.yml only watched ``aperag/domains/knowledge_graph/graphindex/**`` and ``aperag/vectorstore/**``, but the actual graph store backends were moved to ``aperag/indexing/graph_storage/{neo4j,nebula,postgres}.py``. Any PR touching those files therefore bypassed Backend-Compat-Test — exactly the workflow that exists to catch cross-adapter regressions (e.g. the ``GET /graphs/labels`` 500 on Neo4j vs. pass on Nebula/PG that 冬柏 surfaced as a P0 candidate, or today's neo4j.py edits). Add ``aperag/indexing/graph_storage/**`` to the trigger list. Keep the legacy ``aperag/domains/knowledge_graph/graphindex/**`` entry as a defensive fallback for any follow-up commits that still touch the old tree. Pure CI configuration change; no test logic, no production code. This is task #61 P0 surfaced by chenyexuan's workflow-gate audit slice (cite trail in #indexing优化 msg=f298011e). Co-authored-by: Claude Opus 4.7 <noreply@anthropic.com>
1 parent 722d13b commit a0403cf

1 file changed

Lines changed: 13 additions & 1 deletion

File tree

.github/workflows/compat-test.yml

Lines changed: 13 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,8 +4,20 @@ on:
44
pull_request:
55
branches: [main]
66
paths:
7-
- 'aperag/domains/knowledge_graph/graphindex/**'
7+
# Real graph store implementations (Neo4j / Nebula / PG).
8+
# The legacy ``aperag/domains/knowledge_graph/graphindex`` tree
9+
# has been gradually migrated under ``aperag/indexing/graph_storage``;
10+
# without this entry, Backend-Compat-Test does NOT trigger when a
11+
# PR modifies neo4j.py / nebula.py / postgres.py — exactly the
12+
# invariant this workflow exists to defend.
13+
- 'aperag/indexing/graph_storage/**'
14+
# Vector store adapters (PGVector / Qdrant).
815
- 'aperag/vectorstore/**'
16+
# Legacy graphindex shim path kept as a defensive trigger; the
17+
# active backend code now lives under aperag/indexing/graph_storage/
18+
# but follow-up commits may still touch this tree, and removing
19+
# the entry could miss those.
20+
- 'aperag/domains/knowledge_graph/graphindex/**'
921
- 'tests/integration/compat/**'
1022
- '.github/workflows/compat-test.yml'
1123
workflow_dispatch:

0 commit comments

Comments
 (0)