Skip to content

Conversation

@pyramation
Copy link
Contributor

Summary

Applies automatic import sorting fixes across the codebase using the simple-import-sort ESLint plugin configured in the base PR. This PR is intended to be merged after the config-only PR (feat/eslint-config-12-25).

Changes include:

  • Alphabetical sorting of import statements (204 files)
  • Reordering of export statements to match sorting rules
  • Removal of now-unnecessary eslint-disable comments for rules no longer enabled

Review & Testing Checklist for Human

  • Verify the whitespace-only lines are acceptable: In files like functions/send-email-link/src/index.ts and functions/simple-email/src/index.ts, // eslint-disable-next-line no-console comments were replaced with blank lines containing only spaces. This looks like unintended formatting - consider running Prettier after this to clean up
  • Spot-check import ordering in a few key files: Verify imports still resolve correctly, especially in files with complex relative imports like postgres/pgsql-test/src/test-client.ts
  • Confirm CI passes: Build and tests should verify no circular dependency issues were introduced by reordering

Test plan: Run pnpm build and pnpm test in a few packages to verify imports resolve correctly after sorting.

Notes

This PR targets feat/eslint-config-12-25 (the config PR), not main. Merge the config PR first, then this one.

Link to Devin run: https://app.devin.ai/sessions/1a95a9e3b5994325aeffe5ae1289aa0a
Requested by: Dan Lynch (pyramation@gmail.com) / @pyramation

@devin-ai-integration
Copy link
Contributor

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

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.

2 participants