Skip to content

Commit fea35ef

Browse files
author
Sapir Schneider
committed
Merge remote-tracking branch 'fork/agentsinfo-ai-agents-queries' into agentsinfo-ai-agents-queries
2 parents 52f6e61 + 237da8f commit fea35ef

4,346 files changed

Lines changed: 706233 additions & 593386 deletions

File tree

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

.github/copilot-instructions.md

Lines changed: 38 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -39,6 +39,38 @@ When reviewing pull requests, follow these rules strictly.
3939
3. Verify all required fields are present and properly formatted
4040
4. Do NOT use general instructions for these files - only use the huntingqueries custom instructions
4141

42+
### For Playbooks Files (`Playbooks/**/*.json`, `Playbooks/**/README.md`, `Solutions/**/Playbooks/**/*.json`, `Solutions/**/Playbooks/**/README.md`)
43+
1. **LOAD and USE:** `.github/instructions/playbooks.instructions.md`
44+
2. Apply ALL guidelines from that file
45+
3. Validate ARM template structure, metadata fields, parameters, and README requirements from that custom instruction file
46+
4. Check all required sections in README.md and ARM template metadata
47+
5. Do NOT use general instructions for these files - only use the playbooks custom instructions
48+
49+
### For Workbooks Files (`Workbooks/*.json`, `Solutions/**/Workbooks/*.json`)
50+
1. **LOAD and USE:** `.github/instructions/workbook.instructions.md`
51+
2. Apply ALL guidelines from that file
52+
3. Validate workbook JSON structure, required fields, items array, and metadata requirements from that custom instruction file
53+
4. Check all required top-level fields and item structure validation
54+
5. Do NOT use general instructions for these files - only use the workbook custom instructions
55+
56+
### For Parser Files (`Parsers/**/*.yaml`, `Parsers/**/*.yml`, `Solutions/**/Parsers/**/*.yaml`, `Solutions/**/Parsers/**/*.yml`)
57+
1. **LOAD and USE:** `.github/instructions/parsers.instructions.md`
58+
2. Apply ALL guidelines from that file
59+
3. Validate parser syntax, KQL accuracy, YAML structure, and all required fields from that custom instruction file
60+
4. Do NOT use general instructions for these files - only use the parsers custom instructions
61+
62+
### For Building, Packaging, or Validating Solutions
63+
When the user asks to **build**, **package**, **validate**, or **run validations** on a solution:
64+
1. **LOAD and USE:** `.github/instructions/packaging.instructions.md`
65+
2. Apply ALL guidelines from that file — it contains the script path, parameters, fuzzy matching behavior, and report display rules
66+
3. **Always use the build script** at `.script/local-validation/build-and-validate.ps1` — do NOT run validators manually or write your own validation logic
67+
4. **BEFORE running the script, tell the user:** "🔄 Running full build & validation suite for {SolutionName}. This typically takes 3-5 minutes — I'll present the complete report when it finishes." This message MUST appear in chat before the script executes so the user knows work is happening.
68+
5. **The script takes 3-10 minutes.** NEVER tell the user to "check the terminal."
69+
6. Present the full report — the text between `[REPORT_START]` and `[REPORT_END]` — in your chat response
70+
7. This applies to ALL solutions, including newly created ones that may not have a Package/ folder yet
71+
72+
**Trigger phrases:** "build solution", "package solution", "validate solution", "run validations", "run CI checks", "check if solution passes", "create V3 package"
73+
4274
---
4375

4476
## Files and folders to ignore
@@ -51,9 +83,12 @@ If files from these paths appear in the PR, completely skip them and do not gene
5183

5284
## Solutions Analyzer Tools
5385

54-
When working with the Solutions Analyzer tools in `Tools/Solutions Analyzer/`:
86+
When working with the Solutions Analyzer tools in `Tools/Solutions Analyzer/`, follow the dedicated skills under `.github/skills/`:
87+
88+
- **`run-solution-analyzer`** — running the mapper, doc generator, ASIM browser, and `upload_to_kusto`; output locations; force-refresh and caching.
89+
- **`update-solution-analyzer`** — modifying scripts, updating `script-docs/`, README changelog rules, CSV output sync with `upload_to_kusto.py`, and static/interactive index plus markdown/HTML entity page synchronization.
5590

56-
### Output Locations
91+
**MANDATORY:** Before editing ANY file under `Tools/Solutions Analyzer/` — including small fixes such as renames, regex/pattern tweaks, escaping changes, override additions, or one-line behavior changes — first load `.github/skills/update-solution-analyzer/SKILL.md` and follow it. A `## Version History` changelog entry in `Tools/Solutions Analyzer/README.md` is **required for feature additions and behavior changes** (new features, changed analysis logic, new/renamed/removed CSV columns, parameter changes); pure bug fixes — such as typo corrections, crash fixes, escape/regex tweaks, or one-line fixes that restore intended behavior — may be logged at your discretion but are not required.
5792

5893
There are THREE different output scenarios - **never confuse them**:
5994

@@ -128,4 +163,4 @@ Use `--force-refresh` with these types when modifying analysis logic:
128163
- Any script parameters added or changed
129164
- Any output file changes, including changes to CSV files (new columns, renamed columns, removed columns)
130165
- Any changes to analysis methods or logic
131-
- Update the primary readme.md if needed and add the change to the change log. Do not add a version if the previous version as manifested by the changelog, was not committed yet.
166+
- Update the primary readme.md if needed and add the change to the change log. Do not add a version if the previous version as manifested by the changelog, was not committed yet.

0 commit comments

Comments
 (0)