Skip to content

Commit 5bb3c9d

Browse files
committed
Clean up skill docs from review
- Remove stale scratchpad references in cli-reference.md - Remove redundant "Key Learnings" header in SKILL.md - Remove redundant curl/grep commands from failure-patterns.md 🤖 Co-Authored-By: Claude Code <noreply@anthropic.com>
1 parent 92be27f commit 5bb3c9d

3 files changed

Lines changed: 7 additions & 31 deletions

File tree

.claude/skills/troubleshoot-ci-build/SKILL.md

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -260,8 +260,6 @@ What would you like to investigate?
260260
- **Run Tests Locally**: `docs/development/CI/RunSmokeTestsLocally.md` - Reproduce failures locally
261261
- **Azure DevOps Pipeline**: `.azure-pipelines.yml` - Pipeline configuration
262262

263-
## Key Learnings
264-
265263
### Timeline Record Structure
266264

267265
The Azure DevOps timeline contains a hierarchy:

.claude/skills/troubleshoot-ci-build/failure-patterns.md

Lines changed: 0 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -532,27 +532,6 @@ Are there canceled jobs?
532532

533533
---
534534

535-
## Useful Commands for Investigation
536-
537-
### Download Specific Test Logs
538-
```bash
539-
# Get timeline to find log ID
540-
curl -s "https://dev.azure.com/datadoghq/.../builds/<BUILD_ID>/timeline" | jq '.records[] | select(.result == "failed")'
541-
542-
# Download log
543-
curl -s "https://dev.azure.com/datadoghq/.../builds/<BUILD_ID>/logs/<LOG_ID>" > test.log
544-
```
545-
546-
### Search for Common Patterns
547-
```bash
548-
# In downloaded log
549-
grep -i "\[FAIL\]" test.log
550-
grep -i "Expected" test.log
551-
grep -i "error\|exception" test.log -A 5
552-
```
553-
554-
---
555-
556535
## Historical Context
557536

558537
### Known Issues

.claude/skills/troubleshoot-ci-build/references/cli-reference.md

Lines changed: 7 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -51,19 +51,18 @@ Available fields: `bucket`, `completedAt`, `description`, `event`, `link`, `name
5151

5252
## Windows CLI Pitfalls (Lessons Learned)
5353

54-
### 0. Always Use Scratchpad Directory
54+
### 0. Use Temporary Directory for Output Files
5555

56-
**Problem**: Using `/tmp` or `$TEMP` can cause issues on Windows
56+
**Problem**: Using `/tmp` or hardcoded paths can cause issues on Windows
5757

58-
**Solution**: Always use the scratchpad directory provided in the system prompt
58+
**Solution**: Use `$TEMP` (PowerShell/Windows) or a user-specified output directory
5959

6060
```bash
6161
# Bad - Don't hardcode /tmp
6262
az devops invoke ... > /tmp/timeline.json
6363

64-
# Good - Use scratchpad from system prompt
65-
SCRATCHPAD="<path provided in system prompt>"
66-
az devops invoke ... > "$SCRATCHPAD/timeline.json"
64+
# Good - Use $TEMP or a specific output directory
65+
az devops invoke ... > "$TEMP/timeline.json"
6766
```
6867

6968
### 1. Complex jq Filters Fail on Windows
@@ -77,8 +76,8 @@ az devops invoke ... > "$SCRATCHPAD/timeline.json"
7776
az devops invoke ... | jq '.records[] | select(.issues != null and .issues != [])'
7877

7978
# Good - Save first, then query
80-
az devops invoke ... --output json > "$SCRATCHPAD/timeline.json"
81-
cat "$SCRATCHPAD/timeline.json" | jq '.records[] | select(.issues)'
79+
az devops invoke ... --output json > "$TEMP/timeline.json"
80+
cat "$TEMP/timeline.json" | jq '.records[] | select(.issues)'
8281
```
8382

8483
### 2. API 500 Errors for Logs - USE TIMELINE URLs INSTEAD

0 commit comments

Comments
 (0)