Skip to content
Open
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
17 commits
Select commit Hold shift + click to select a range
222902a
Update the Peach analysis document with Peach upload timeouts
francois-mora-sonarsource Mar 13, 2026
ce6cefb
Merge remote-tracking branch 'origin/master' into fix/skill-peach-check
francois-mora-sonarsource Mar 17, 2026
c0e5fe2
Improve peach-check skill based on observed session execution
francois-mora-sonarsource Mar 17, 2026
e2fd0da
Merge branch 'master' into fix/skill-peach-check
francois-mora-sonarsource Mar 19, 2026
d5a71da
Merge remote-tracking branch 'origin/master' into fix/skill-peach-check
francois-mora-sonarsource Mar 19, 2026
f5cd71f
Improve peach-check skill: mass failure detection, grep-based triage,…
francois-mora-sonarsource Mar 19, 2026
9b3fc08
Refine peach-check skill: improve grep filters, fix step numbering, m…
francois-mora-sonarsource Mar 20, 2026
2b25cb2
Merge branch 'master' into fix/skill-peach-check
francois-mora-sonarsource Mar 20, 2026
df561fd
Fix stale step reference and incomplete flowchart for plugin init fai…
francois-mora-sonarsource Mar 20, 2026
a0aa56f
Merge remote-tracking branch 'origin/master' into fix/skill-peach-check
francois-mora-sonarsource Mar 23, 2026
cab24e3
Improve peach-check skill: save logs to disk, inline jq filtering, ad…
francois-mora-sonarsource Mar 23, 2026
1c01c14
Improve peach-check skill: clarify parallelism rule and filter script…
francois-mora-sonarsource Mar 26, 2026
f4e2d98
Merge branch 'master' into fix/skill-peach-check
francois-mora-sonarsource Mar 27, 2026
1472109
Add .codex and claude mcp config files to ignored files
francois-mora-sonarsource Mar 27, 2026
207fe71
Resolve merge conflicts in peach-check skill and docs
francois-mora-sonarsource Mar 27, 2026
d55f56f
Merge remote-tracking branch 'origin/master' into fix/skill-peach-check
francois-mora-sonarsource Mar 30, 2026
d126b5b
Add relative path to peach analysis document
francois-mora-sonarsource Mar 30, 2026
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
12 changes: 10 additions & 2 deletions .claude/skills/peach-check/SKILL.md
Original file line number Diff line number Diff line change
Expand Up @@ -29,6 +29,12 @@ Before running this skill, ensure:
separate Bash call. Chaining bypasses the per-tool permission prompts that allow the user to
review each action individually.

**Parallel execution is separate from chaining.** Issuing multiple independent Bash calls in
the same response message is the correct way to run jobs concurrently — it does not violate the
no-chaining rule. The no-chaining rule is about what goes *inside* a single Bash call; parallel
execution is about how many Bash calls appear in a single response. Both rules apply together:
separate calls, issued at the same time.

## Invocation

```
Expand Down Expand Up @@ -131,7 +137,7 @@ Instead:

**Step 5 — Read the classification guide and triage all logs**

Read `docs/peach-main-analysis.md` once to load the failure categories and decision flowchart.
Read `docs/peach-main-analysis.md` (at the repository root) once to load the failure categories and decision flowchart.

Create the work directory where logs will be stored for inspection:

Expand All @@ -155,6 +161,7 @@ via the `e` command, which is a risk when processing untrusted log content:
gh api "repos/SonarSource/peachee-js/actions/jobs/JOB_ID/logs" \
> target/peach-logs/JOB_ID.log
sed --sandbox -n '
/\[36;1m/b
/Process completed with exit code/p
/EXECUTION FAILURE/p
/OutOfMemoryError/p
Expand Down Expand Up @@ -183,6 +190,7 @@ SonarJS plugin stack trace. The log is already on disk from Phase 1 — no re-do

```bash
sed --sandbox -n '
/\[36;1m/b
/Sensor /p
/EXECUTION FAILURE/p
/OutOfMemoryError/p
Expand Down Expand Up @@ -271,5 +279,5 @@ The release recommendation is:
**Step 9 — Update docs if a new failure pattern was found**

If any job was classified as NEEDS-MANUAL-REVIEW and you identified its root cause during this
session, update `docs/peach-main-analysis.md` with a new category entry. This keeps the
session, update `docs/peach-main-analysis.md` (at the repository root) with a new category entry. This keeps the
classification guide current for future runs.
3 changes: 3 additions & 0 deletions .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -78,3 +78,6 @@ lcov.info
.claude/*
!.claude/*.md
!.claude/skills/

.codex/
.mcp.json
2 changes: 2 additions & 0 deletions docs/peach-main-analysis.md
Original file line number Diff line number Diff line change
Expand Up @@ -364,6 +364,7 @@ Identifies which step failed and what exit code was produced. Run on every faile

```bash
sed --sandbox -n '
/\[36;1m/b # skip GitHub Actions script-preview lines (ANSI-colored)
/Process completed with exit code/p # universal — exit code value drives the flowchart
/EXECUTION FAILURE/p # scanner ran and failed (exit code 3)
/OutOfMemoryError/p # OOM / Runner Killed
Expand All @@ -390,6 +391,7 @@ jobs where Phase 1 showed exit code 3 without a clear misconfiguration signal.

```bash
sed --sandbox -n '
/\[36;1m/b # skip GitHub Actions script-preview lines (ANSI-colored)
/Sensor /p # last sensor name — is it a SonarJS sensor?
/EXECUTION FAILURE/p # scanner failure marker
/OutOfMemoryError/p # OOM inside scanner
Expand Down
Loading