smithery/elliotjlt

trace-it

Before modifying shared code (utilities, types, configs, base classes), traces all callers and dependents first. Activates when editing files in shared/, utils/, lib/, or anything imported by 3+ files. Prevents "fixed one thing, broke three others."

Installation

$ npx skills add smithery/elliotjlt --skill trace-it

Similar popular skills

Related neighbors and high-traction skills in the same topics — useful to compare before installing.

Also in this package

Other skills from smithery/elliotjlt · top by installs.

npx skills add smithery/elliotjlt

Browse all from smithery/elliotjlt

More details

Agent compatibility

Declared targets from SKILL.md / docs. Unmarked agents are not listed — the skill may still install via the CLI.

Claude Code Not declared
Cursor Not declared
Codex Not declared
GitHub Copilot Not declared
Windsurf Not declared
Gemini CLI Not declared
Cline Not declared
OpenCode Not declared

Skill metadata

Parsed from SKILL.md frontmatter.

Allowed toolsbash: grep, find, git file: read

Package contents

Files included with this skill beyond the listing page.

  • skill md SKILL.md 4,691 B
  • docs SUMMARY.md 268 B

History

  1. First recorded snapshot · 0 installs

SKILL.md

Trace It

<purpose> Shared code is high-leverage but high-risk. Change a utility function and you might break 10 callers. Change a type definition and you invalidate assumptions across the codebase. This skill forces you to trace impact before editing. </purpose>

When To Activate

<triggers>

  • Editing files in: utils/, lib/, shared/, common/, core/
  • Modifying exported functions, classes, or types
  • Changing function signatures (parameters, return types)
  • Editing config files or constants
  • Renaming anything that might be imported elsewhere
  • Modifying base classes or interfaces
  • User says "change how X works" for foundational code

</triggers>

Instructions

Before Modifying Shared Code

<trace_callers>

Step 1: Find All Callers

# Find imports/requires of this file
grep -r "from.*['\"].*filename" src/
grep -r "import.*filename" src/
grep -r "require.*filename" src/

# Find usages of specific function/class
grep -r "functionName" src/ --include="*.ts"

Document what you find:

## Dependency Trace: [file or function name]

**Direct callers:** [count]
- `src/api/handler.ts:23` - uses for validation
- `src/services/user.ts:45` - uses for formatting
- `src/utils/helpers.ts:12` - re-exports it

**Indirect dependents:** [count]
- Files that import the direct callers

</trace_callers>

<assess_impact>

Step 2: Assess Impact

For each caller, determine:

Caller How it uses this Will change break it?
handler.ts Calls validate(input) Yes - signature changes
user.ts Uses return value No - return type same

Impact level:

  • Low: 1-2 callers, simple usages
  • Medium: 3-5 callers, varied usages
  • High: 6+ callers OR complex/varied usages

</assess_impact>

<plan_migration>

Step 3: Plan the Change

For Low impact: Proceed, update callers inline.

For Medium impact:

  1. Make change backward-compatible if possible
  2. Update all callers in same commit
  3. Test each caller's behavior

For High impact:

  1. Consider: Is this change necessary?
  2. Create migration plan
  3. Consider deprecation period
  4. Update incrementally with tests

</plan_migration>

Signature Changes

When changing function signatures:

## Signature Change

**Before:** `function process(data: string): Result`
**After:** `function process(data: string, options?: Options): Result`

**Breaking:** No (new param is optional)
**Callers to update:** 0 required, 5 could benefit

## OR

**Before:** `function process(data: string): Result`
**After:** `function process(data: ProcessInput): Result`

**Breaking:** Yes
**Callers to update:** 8 files
**Migration:** [list each file and required change]

Output Format

## Trace: [what you're changing]

**Callers found:** [count]
**Impact level:** [Low/Medium/High]

**Changes required:**
- [ ] `file1.ts:line` - [what needs updating]
- [ ] `file2.ts:line` - [what needs updating]

**Verified after change:**
- [ ] All callers updated
- [ ] No new type errors
- [ ] Tests pass

NEVER

  • Modify shared code without tracing callers first
  • Change function signatures assuming "it's probably fine"
  • Update shared code and only test one caller
  • Forget to check re-exports (file A exports from file B)
  • Assume grep found everything (check for dynamic imports)

ALWAYS

  • Trace before touching shared code
  • Document the caller count and impact level
  • Update ALL callers in the same change
  • Verify no type errors after the change
  • Check for string-based references (dynamic imports, configs)

Example

User: "Rename the formatDate function to formatDateTime"

Trace:

grep -r "formatDate" src/

Found:

src/utils/date.ts:15 - definition
src/components/Header.tsx:8 - import
src/components/EventCard.tsx:12 - import
src/pages/Dashboard.tsx:45 - import
src/api/events.ts:23 - import
src/utils/index.ts:5 - re-export

Impact: 6 files reference this. High impact.

Plan:

  1. Rename in date.ts
  2. Update re-export in utils/index.ts
  3. Update imports in all 4 consumer files
  4. Verify: grep -r "formatDate" src/ returns nothing

After completion:

Renamed formatDate → formatDateTime across 6 files.
Verified: no remaining references to old name.