Skip to content

Commit 1c71611

Browse files
committed
Clarify why cortex/snow CLIs and Snowflake connection are optional warnings in test coverage table
1 parent 10434e9 commit 1c71611

1 file changed

Lines changed: 3 additions & 3 deletions

File tree

AGENTS.md

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -58,13 +58,13 @@ bash tests/run-tests.sh # Mac/Linux
5858

5959
| Section | Checks | What it validates |
6060
|---------|--------|-------------------|
61-
| CLIs | 5 | python3, git required; cortex, snow optional (warn) |
61+
| CLIs | 5 | python3, git required; cortex, snow optional (warn — unit tests are pure Python and never invoke these CLIs, so contributors and CI can run the full suite without them installed) |
6262
| Plugin structure | 7 | plugin.json, 3 skill dirs, 9 router scripts, test files |
6363
| Content sanity | 8 | Valid JSON, `decide()` defined, `--permission-prompt-tool` usage, session_state functions |
6464
| Unit tests | 75 | `test_envelope_policy.py` (33): RO/RW/RESEARCH/DEPLOY envelope allow/deny decisions. `test_plugin_units.py` (42): session state CRUD, credential path blocking, envelope prompt building |
65-
| Snowflake connection | 1 | connections.toml or env vars exist (warn) |
65+
| Snowflake connection | 1 | connections.toml or env vars exist (warn — only needed for live integration, not for unit tests) |
6666

67-
Tests exit non-zero on any FAIL. Missing CLIs and missing Snowflake config are non-blocking warnings.
67+
Tests exit non-zero on any FAIL. Warnings (marked "warn") are informational only — they flag missing tools that aren't needed for the core test suite.
6868

6969
## Installers
7070

0 commit comments

Comments
 (0)