/ia-triage

name: ia-triage
description: Triage and categorize findings for the CLI todo system
argument-hint: "[findings list or source type]"
disable-model-invocation: true

**Input:** #$ARGUMENTS

• First set the /model to Haiku
• If input specifies a source or filter, use it. Otherwise read all pending todos in the todos/ directory.

Present all findings, decisions, or issues here one by one for triage. The goal is to go through each item and decide whether to add it to the CLI todo system.

**IMPORTANT: DO NOT CODE ANYTHING DURING TRIAGE!**

This command is for:

• Triaging code review findings
• Processing security audit results
• Reviewing performance analysis
• Handling any other categorized findings that need tracking

Workflow

Step 1: Present Each Finding

For each finding, present in this format:

---
Issue #X: [Brief Title]

Severity: 🔴 P1 (CRITICAL) / 🟡 P2 (IMPORTANT) / 🔵 P3 (NICE-TO-HAVE)

Category: [Security/Performance/Architecture/Bug/Feature/etc.]

Description:
[Detailed explanation of the issue or improvement]

Location: [file_path:line_number]

Problem Scenario:
[Step by step what's wrong or could happen]

Proposed Solution:
[How to fix it]

Estimated Effort: [Small (< 2 hours) / Medium (2-8 hours) / Large (> 8 hours)]

---
Do you want to add this to the todo list?
1. yes - create todo file
2. next - skip this item
3. custom - modify before creating

Severity bridges to the `ia-code-review` skill's four tiers: **P1 = Critical**, **P2 = Important**, **P3 = Medium + Minor** (P3/`p3` covers both lower tiers).

Step 2: Handle User Decision

**When user says "yes":**

1. **Update or create todo file** using the `ia-file-todos` skill for all naming, frontmatter, and status conventions. Change status from `pending` to `ready` in both filename and YAML frontmatter.

3. **Confirm approval:** "Approved: `{new_filename}` (Issue #{issue_id}) - Status: **ready**"

**When user says "next":**

• **Delete the todo file** - Remove it from todos/ directory since it's not relevant
• Skip to the next item
• Track skipped items for summary

**When user says "custom":**

• Ask what to modify (priority, description, details)
• Update the information
• Present revised version
• Ask again: yes/next/custom

Step 3: Continue Until All Processed

• Process all items one by one
• Track using TodoWrite for visibility
• Don't wait for approval between items - keep moving

Step 4: Final Summary

After all items processed:

## Triage Complete

**Total Items:** [X] **Todos Approved (ready):** [Y] **Skipped:** [Z]

### Approved Todos (Ready for Work):

- `042-ready-p1-transaction-boundaries.md` - Transaction boundary issue
- `043-ready-p2-cache-optimization.md` - Cache performance improvement ...

### Skipped Items (Deleted):

- Item #5: [reason] - Removed from todos/
- Item #12: [reason] - Removed from todos/

### Summary of Changes Made:

During triage, the following status updates occurred:

- **Pending → Ready:** Filenames and frontmatter updated to reflect approved status
- **Deleted:** Todo files for skipped findings removed from todos/ directory
- Each approved file now has `status: ready` in YAML frontmatter

### Next Steps:

1. View approved todos ready for work:
   ```bash
   ls todos/*-ready-*.md

2. Start work on approved items:

   ```bash
   /ia-resolve-todo-parallel  # Work on multiple approved items efficiently

3. Or pick individual items to work on

4. As you work, update todo status:

• Ready → In Progress (in your local context as you work)
• In Progress → Complete (rename file: ready → complete, update frontmatter)

Example Response Format

---

Issue #5: Missing Transaction Boundaries for Multi-Step Operations

Severity: 🔴 P1 (CRITICAL)

Category: Data Integrity / Security

Description: The google_oauth2_connected callback in GoogleOauthCallbacks concern performs multiple database operations without transaction protection. If any step fails midway, the database is left in an inconsistent state.

Location: src/controllers/OAuthController.ts:13-50

Problem Scenario:

1. User.update succeeds (email changed)
2. Account.save! fails (validation error)
3. Result: User has changed email but no associated Account
4. Next login attempt fails completely

Operations Without Transaction:

- User confirmation (line 13)
- Waitlist removal (line 14)
- User profile update (line 21-23)
- Account creation (line 28-37)
- Avatar attachment (line 39-45)
- Journey creation (line 47)

Proposed Solution: Wrap all operations in ApplicationRecord.transaction do ... end block

Estimated Effort: Small (30 minutes)

---

Do you want to add this to the todo list?

1. yes - create todo file
2. next - skip this item
3. custom - modify before creating

Progress Tracking

Show progress with each item: "X/Y completed". Follow the `ia-file-todos` skill for all file naming, frontmatter structure, and status transitions.

When done give these options

What would you like to do next?

1. run /ia-resolve-todo-parallel to resolve the todos
2. commit the todos
3. nothing, go chill