Skip to content

refactor: move fix command utilities to separate module#442

Open
Kushaal-k wants to merge 2 commits into
OWASP:mainfrom
Kushaal-k:refactor-fix-runner-utils
Open

refactor: move fix command utilities to separate module#442
Kushaal-k wants to merge 2 commits into
OWASP:mainfrom
Kushaal-k:refactor-fix-runner-utils

Conversation

@Kushaal-k
Copy link
Copy Markdown
Contributor

Closes #437

Summary

Extracted fix command utility functions from src/index.ts into a dedicated utility module.

What changed

  • Created src/utils/fix-runner.ts
  • Moved fix command helper functions into the new module
  • Updated src/index.ts to import the extracted utilities
  • Removed unwanted imports from src/index.ts

Validation

  • Ran npm run build
  • Ran npm test
  • Confirmed no behavior/output changes

Notes

This is a pure refactor with no intended behavior changes.

Copy link
Copy Markdown
Collaborator

@sonukapoor sonukapoor left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice refactor - spawn moves to where it belongs and the orphaned imports are cleaned up too. One fix needed, plus a rebase before this can merge.

Comment thread src/utils/fix-runner.ts Outdated
@Kushaal-k Kushaal-k force-pushed the refactor-fix-runner-utils branch from 39c96e8 to 51f62d7 Compare May 25, 2026 15:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

refactor: extract fix command utilities from src/index.ts to src/utils/fix-runner.ts

2 participants