Fixme Todo Cleanup

FIXME/TODO Cleanup Skill

When to Use

• User requests cleanup of FIXME/TODO items across a codebase
• Technical debt reduction initiative
• Pre-release code quality sweep
• Issue tracking consolidation

Verified Workflow

1. Discovery Phase

# Find all FIXME/TODO items
grep -rn "FIXME\|TODO" src/odyssey/ --include="*.mojo"

2. Categorization

Group items by:

• Stale references: FIXME pointing to closed issues
• Missing implementations: Placeholder code needing real logic
• External blockers: Items blocked by language/compiler features
• Documentation: TODO in docs describing future work

3. Execution Pattern

For each actionable item:

# 1. Create branch from latest main
git checkout main && git pull origin main
git checkout -b <issue-number>-<description>

# 2. Make the fix (one FIXME per commit)

# 3. Validate
just pre-commit-all
uv run mojo test tests/

# 4. Commit with issue reference
git add . && git commit -m "fix(scope): description

Closes #<issue>"

# 5. Push and create PR with auto-merge
git push -u origin <branch-name>
gh pr create --body "Closes #<issue>"
gh pr merge --auto --rebase

4. Handle Flaky CI

# Retry failed jobs
gh run rerun <run-id> --failed

Failed Attempts

1. @value to @fieldwise_init Migration

What was tried: Replace deprecated @value decorator with @fieldwise_init

Error:

error: 'TrainingCallbacks' has an explicitly declared fieldwise initializer

Why it failed: @fieldwise_init generates an __init__ method, conflicting with custom __init__

Solution: Remove decorator entirely, keep explicit Copyable, Movable conformances:

# Before (broken)
@fieldwise_init
struct TrainingCallbacks(Copyable, Movable):
    fn __init__(out self, verbose: Bool = True):
        self.verbose = verbose

# After (working)
struct TrainingCallbacks(Copyable, Movable):
    var verbose: Bool

    fn __init__(out self, verbose: Bool = True):
        self.verbose = verbose

2. Ralph-Loop Hook Stuck

What was tried: Set active: false in ralph-loop.local.md

Why it failed: Hook continued firing despite deactivation flag

Solution: Delete the file entirely:

rm .claude/ralph-loop.local.md

Results & Parameters

Items Resolved

Items NOT Resolved (External Blockers)

Git Configuration

branch_naming: "<issue-number>-<description>"
commit_format: "type(scope): description\n\nCloses #<issue>"
merge_strategy: "rebase"
auto_merge: true

Key Learnings

1. Categorize first: Not all FIXME/TODO items are actionable
2. One PR per fix: Keeps changes atomic and reviewable
3. Auto-merge is essential: Reduces manual intervention
4. Retry flaky tests: Some CI tests are non-deterministic
5. Mojo decorator conflicts: @fieldwise_init cannot coexist with custom __init__