-
-
Notifications
You must be signed in to change notification settings - Fork 791
test: fix quality gates and skip flaky tests #607
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Changes from all commits
c81bdb0
593b465
b319199
3156a62
d14fd68
f9538b9
3203670
ad0ff47
49651a6
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Large diffs are not rendered by default.
This file was deleted.
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -53,6 +53,21 @@ Use for dependency graphs and circular dependency detection. | |
| - **Dependency tree:** Generate graph for a package with configurable depth | ||
| - **Circular check:** Detect circular dependency chains in the project | ||
|
|
||
| ### jcodemunch — Code Intelligence & Token Reducer (Essential) | ||
| Use INSTEAD OF Read/Grep para navegar código. Redução de 99% em tokens via AST indexing. | ||
| **FLUXO OBRIGATÓRIO antes de usar:** verifique se o projeto está indexado com `list_repos`. | ||
| Se não indexado: `index_folder` com o path do projeto (executar apenas uma vez). | ||
| - **Indexar projeto:** `index_folder({ "folder_path": "D:/aiox-genesis/aiox-core/projeto-one" })` | ||
| - **Verificar índice:** `list_repos({})` — lista projetos indexados | ||
| - **Visão geral do projeto:** `get_repo_outline({ "repo": "projeto-one" })` | ||
| - **Buscar função/classe:** `search_symbols({ "repo": "projeto-one", "query": "useAioxState" })` | ||
| - **Obter símbolo completo:** `get_symbol({ "repo": "projeto-one", "symbol_id": "lib/hooks/use-aiox-state.ts::useAioxState" })` | ||
| - **Trecho de arquivo:** `get_file_content({ "repo": "projeto-one", "file_path": "server/index.js", "start_line": 1, "end_line": 50 })` | ||
| - **Busca full-text:** `search_text({ "repo": "projeto-one", "query": "SSE event-stream" })` | ||
| - **Reindexar após refactor:** `invalidate_cache({ "repo": "projeto-one" })` → `index_folder` | ||
|
|
||
| **Regra de prioridade:** jcodemunch > Grep > Read (para arquivos de código) | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Resolve policy conflict with existing tool-priority rules. This priority rule contradicts 🤖 Prompt for AI Agents |
||
|
|
||
| ### docker-gateway — MCP Infrastructure | ||
| Use for managing Docker-based MCP servers. `@devops` manages infrastructure. | ||
| - **Health check:** `curl http://localhost:8080/health` | ||
|
|
||
| Original file line number | Diff line number | Diff line change | ||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
|
|
@@ -8,7 +8,9 @@ | |||||||||||||
| "Bash(git add:*)", | ||||||||||||||
| "Bash(git commit:*)", | ||||||||||||||
| "Bash(git push:*)", | ||||||||||||||
| "Bash(node -e:*)" | ||||||||||||||
| "Bash(node -e:*)", | ||||||||||||||
| "Bash(git config:*)", | ||||||||||||||
| "Bash(npm run:*)" | ||||||||||||||
|
Comment on lines
+11
to
+13
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Permission expansion is overly broad and increases command-execution risk. Line 12-Line 13 grants very wide shell capability ( Suggested tightening- "Bash(git config:*)",
- "Bash(npm run:*)"
+ "Bash(git config --local:*)",
+ "Bash(npm run build:*)"📝 Committable suggestion
Suggested change
🤖 Prompt for AI Agents |
||||||||||||||
| ] | ||||||||||||||
| } | ||||||||||||||
| } | ||||||||||||||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,3 @@ | ||
| # SYNAPSE runtime data (auto-generated) | ||
| sessions/ | ||
| cache/ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🧩 Analysis chain
🏁 Script executed:
Repository: SynkraAI/aiox-core
Length of output: 168
🏁 Script executed:
Repository: SynkraAI/aiox-core
Length of output: 25986
Skipping the entire suite eliminates coverage of deterministic tests.
The suite contains several tests that don't depend on git operations (constructor options, formatList, formatAge, and the first getMergeHistory test), yet all are skipped. The timeout issue stems from the beforeEach hook (lines 16–42), not all tests. Consider gating only the git-dependent tests instead of the entire suite using an environment variable.
Suggested approach
Alternatively, move deterministic tests (constructor, formatList, formatAge) outside the flaky beforeEach context into a separate describe block without the skip.
🤖 Prompt for AI Agents