Skip to content

Commit 0de2f88

Browse files
committed
disable cosmos
1 parent 4b3dd8d commit 0de2f88

File tree

4 files changed

+5
-3
lines changed

4 files changed

+5
-3
lines changed

.github/workflows/ci.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@ env:
1010
DOTNET_VERSION: '10.0.x'
1111
SOLUTION_FILE: GraphRag.slnx
1212
GRAPH_RAG_ENABLE_JANUS: 'true'
13-
GRAPH_RAG_ENABLE_COSMOS: 'true'
13+
GRAPH_RAG_ENABLE_COSMOS: 'false'
1414

1515
jobs:
1616
build:

.github/workflows/release.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ env:
99
DOTNET_VERSION: '10.0.x'
1010
SOLUTION_FILE: GraphRag.slnx
1111
GRAPH_RAG_ENABLE_JANUS: 'true'
12-
GRAPH_RAG_ENABLE_COSMOS: 'true'
12+
GRAPH_RAG_ENABLE_COSMOS: 'false'
1313

1414
jobs:
1515
build:

AGENTS.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -19,6 +19,7 @@
1919
- When registering graph stores (especially Postgres) use `AddKeyedSingleton<T>(key)` without bespoke lambdas unless strictly required; rely on DI to construct services.
2020
- All integration tests must run against real dependencies via Testcontainers modules (Neo4j, Postgres/AGE, Cosmos, Janus, etc.); do not fall back to Aspire seeding or mock containers.
2121
- Keep integration tests provider-agnostic; avoid adding Postgres-only scenarios when the same flow should apply to every backend.
22+
- Disable Cosmos DB integration tests by default; only enable the emulator when explicitly requested (set `GRAPH_RAG_ENABLE_COSMOS=true`).
2223

2324
# Conversations
2425
any resulting updates to agents.md should go under the section "## Rules to follow"

README.md

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -222,7 +222,8 @@ If the graph is sparse, the pipeline falls back to connected components to ensur
222222
## Integration Testing Strategy
223223

224224
- **Real services only.** All graph operations run against containerised Neo4j and Apache AGE/PostgreSQL instances provisioned by Testcontainers.
225-
- **Extended startup window.** The integration fixture raises `TestcontainersSettings.WaitStrategyTimeout` to 10 minutes so first-time Docker pulls have enough time to complete before readiness checks fail.
225+
- **Extended startup window.** The integration fixture raises `TestcontainersSettings.WaitStrategyTimeout` to 30 minutes (override via `GRAPH_RAG_CONTAINER_TIMEOUT_MINUTES`) so first-time Docker pulls for Cosmos, Janus, or AGE have enough time to complete before readiness checks fail.
226+
- **Cosmos tests opt-in.** The Cosmos DB emulator only starts when `GRAPH_RAG_ENABLE_COSMOS=true`; otherwise the keyed Cosmos services are skipped so CI runs faster.
226227
- **Deterministic heuristics.** `StubEmbeddingGenerator` guarantees stable embeddings so semantic-dedup and token-budget assertions remain reliable.
227228
- **Cross-store validation.** Shared integration fixtures verify that workflows succeed against each adapter (Cosmos tests activate when the emulator connection string is present).
228229
- **Prompt precedence.** Tests validate that manual prompt overrides win over auto-tuned variants while still cascading correctly to the default templates.

0 commit comments

Comments
 (0)