Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
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
23 changes: 8 additions & 15 deletions .agents/skills/triage/diagnose.md
Original file line number Diff line number Diff line change
Expand Up @@ -18,11 +18,10 @@ These variables are referenced throughout this skill. They may be passed as args
## Overview

1. Review the reproduction and error details from `report.md`
2. Load any domain-specific skills relevant to the bug category
3. Locate the relevant source files in `packages/`
4. Add instrumentation to understand the code path
5. Identify the root cause
6. Append diagnosis findings to `report.md`
2. Locate the relevant source files in `packages/`
3. Add instrumentation to understand the code path
4. Identify the root cause
5. Append diagnosis findings to `report.md`

## Step 1: Review the Reproduction

Expand All @@ -36,17 +35,11 @@ Re-run the reproduction if needed to see the error firsthand:
pnpm -C <triageDir> run build # or dev/preview
```

## Step 2: Load Domain-Specific Skills

Before diving into the source, check whether any skills in `.agents/skills/` are relevant to this bug category. Use `ls .agents/skills/` to see what's available, then read the `---` frontmatter of any that look relevant to get their description.

If a skill covers the category of bug you're investigating, load it with the `skill` tool and follow its guidance during diagnosis. Domain skills contain deep knowledge about specific subsystems that can significantly speed up root cause identification.

## Step 3: Locate Relevant Source Files
## Step 2: Locate Relevant Source Files

Using the error messages, stack traces, and any other reproduction details from Step 1, identify the source files in `packages/` that are likely involved.

## Step 4: Investigate with Instrumentation
## Step 3: Investigate with Instrumentation

Add `console.log` statements to understand the code path:

Expand All @@ -68,7 +61,7 @@ Iterate until you understand:
- What data is being passed
- Where the logic diverges from expected behavior

## Step 5: Identify Root Cause
## Step 4: Identify Root Cause

Once you understand the issue, document:

Expand All @@ -83,7 +76,7 @@ Consider:
- Does this affect other similar use cases?
- Are there edge cases to consider?

## Step 6: Write Output
## Step 5: Write Output

Append your diagnosis findings to the existing `report.md` (written by the reproduce skill).

Expand Down
39 changes: 16 additions & 23 deletions .agents/skills/triage/fix.md
Original file line number Diff line number Diff line change
Expand Up @@ -18,15 +18,14 @@ These variables are referenced throughout this skill. They may be passed as args
## Overview

1. Review the diagnosis from `report.md`
2. Load any domain-specific skills relevant to the bug category
3. Verify fix feasibility (browser/runtime compatibility)
4. Implement a minimal fix in `packages/`
5. Rebuild the affected package(s)
6. Verify the fix resolves the reproduction
7. Ensure no regressions
8. Generate git diff
9. Append fix details to `report.md`
10. Clean up the working directory
2. Verify fix feasibility (browser/runtime compatibility)
3. Implement a minimal fix in `packages/`
4. Rebuild the affected package(s)
5. Verify the fix resolves the reproduction
6. Ensure no regressions
7. Generate git diff
8. Append fix details to `report.md`
9. Clean up the working directory

## Step 1: Review the Diagnosis

Expand All @@ -40,20 +39,14 @@ Read `report.md` from the `triageDir` directory to understand:

**Note:** The repo may be messy from previous steps. Check `git status` and either work from the current state or `git reset --hard` to start clean.

## Step 2: Load Domain-Specific Skills

Before implementing, check whether any skills in `.agents/skills/` are relevant to this bug category. Use `ls .agents/skills/` to see what's available, then read the `---` frontmatter of any that look relevant to get their description.

If a skill covers the subsystem you're fixing, load it with the `skill` tool. Domain skills often contain fix approaches, key files, and caveats that aren't captured in `report.md`.

## Step 3: Verify Fix Feasibility
## Step 2: Verify Fix Feasibility

Consider your potential fixes and verify that any modern features you plan to use are supported:

- **Node.js:** When writing code for the runtime (server, build logic, integrations, etc.), target Node.js version `>=22.12.0`.
- **Browsers:** If your fix relies on browser support for any web platform feature, check the browser compatibility table on MDN to confirm it is supported across our browser targets. Do not treat specification compliance as proof of browser support. If the feature lacks sufficient support, choose a different approach.

## Step 4: Implement the Fix
## Step 3: Implement the Fix

Make changes in `packages/` source files. Follow these principles:

Expand Down Expand Up @@ -89,7 +82,7 @@ export function renderComponent(component: AstroComponent, props: Props) {
}
```

## Step 5: Rebuild the Package
## Step 4: Rebuild the Package

After making changes, rebuild the affected package:

Expand All @@ -99,15 +92,15 @@ pnpm -C packages/astro build # or packages/integrations/<name>

Watch for build errors — fix any TypeScript issues before proceeding.

## Step 6: Verify the Fix
## Step 5: Verify the Fix

Re-run the reproduction, often using `pnpm run build`/`astro build` or `pnpm run dev`/`astro dev`.

## Step 7: Check for Regressions
## Step 6: Check for Regressions

Test that you didn't break anything new, and that normal cases still work. If you find regressions, refine the fix to handle all cases.

## Step 8: Generate Git Diff
## Step 7: Generate Git Diff

From the repository root, generate the diff:

Expand All @@ -117,7 +110,7 @@ git diff packages/

This captures all your changes for the report.

## Step 9: Write Output
## Step 8: Write Output

Append your fix details to the existing `report.md` (written by reproduce and diagnose skills).

Expand All @@ -132,7 +125,7 @@ The report must include all information needed for a final GitHub comment to be
- Any alternative approaches considered and their tradeoffs
- If the fix failed: what was tried and why it didn't work

## Step 10: Clean Up the Working Directory
## Step 9: Clean Up the Working Directory

1. Run `git status` and review all changed files
2. Revert any changes that are NOT part of the fix:
Expand Down
9 changes: 0 additions & 9 deletions .changeset/calm-coats-approve.md

This file was deleted.

5 changes: 0 additions & 5 deletions .changeset/common-cats-travel.md

This file was deleted.

5 changes: 0 additions & 5 deletions .changeset/common-symbols-teach.md

This file was deleted.

5 changes: 0 additions & 5 deletions .changeset/every-lights-shave.md

This file was deleted.

5 changes: 0 additions & 5 deletions .changeset/fast-hats-laugh.md

This file was deleted.

8 changes: 0 additions & 8 deletions .changeset/five-kids-double.md

This file was deleted.

5 changes: 0 additions & 5 deletions .changeset/fix-emitfile-serve-warning.md

This file was deleted.

5 changes: 0 additions & 5 deletions .changeset/fix-menuitemradio-non-interactive-role.md

This file was deleted.

5 changes: 0 additions & 5 deletions .changeset/fix-queued-rendering-html-pages.md

This file was deleted.

5 changes: 0 additions & 5 deletions .changeset/fix-server-island-namemap-prerender-env.md

This file was deleted.

5 changes: 0 additions & 5 deletions .changeset/fix-sitemap-i18n-fallback-routes.md

This file was deleted.

19 changes: 0 additions & 19 deletions .changeset/fix-sitemap-i18n-fallback.md

This file was deleted.

6 changes: 0 additions & 6 deletions .changeset/fix-stream-leaks.md

This file was deleted.

5 changes: 0 additions & 5 deletions .changeset/fix-svelte-empty-class.md

This file was deleted.

5 changes: 0 additions & 5 deletions .changeset/full-pillows-greet.md

This file was deleted.

5 changes: 0 additions & 5 deletions .changeset/giant-islands-stare.md

This file was deleted.

5 changes: 0 additions & 5 deletions .changeset/green-lies-tan.md

This file was deleted.

5 changes: 0 additions & 5 deletions .changeset/itchy-tires-speak.md

This file was deleted.

5 changes: 0 additions & 5 deletions .changeset/kind-lark-slim.md

This file was deleted.

24 changes: 0 additions & 24 deletions .changeset/legal-rings-rhyme.md

This file was deleted.

5 changes: 0 additions & 5 deletions .changeset/little-mice-peel.md

This file was deleted.

5 changes: 0 additions & 5 deletions .changeset/lucky-teams-play.md

This file was deleted.

5 changes: 0 additions & 5 deletions .changeset/nine-lights-rest.md

This file was deleted.

5 changes: 0 additions & 5 deletions .changeset/nine-points-dress.md

This file was deleted.

23 changes: 0 additions & 23 deletions .changeset/plain-kiwis-nail.md

This file was deleted.

27 changes: 0 additions & 27 deletions .changeset/red-heads-stare.md

This file was deleted.

5 changes: 0 additions & 5 deletions .changeset/short-bats-judge.md

This file was deleted.

5 changes: 0 additions & 5 deletions .changeset/ten-jobs-bathe.md

This file was deleted.

Loading
Loading