chore(format): organize imports via prettier-plugin-organize-imports#488
Merged
Conversation
@workglow/cli
@workglow/ai
@workglow/job-queue
@workglow/knowledge-base
@workglow/storage
@workglow/task-graph
@workglow/tasks
@workglow/util
workglow
commit: |
Coverage Report
File Coverage
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
sroussey
force-pushed
the
claude/add-import-reorganization-JjEUS
branch
from
723ac6b to
2960609
Compare
Adds prettier-plugin-organize-imports so newly-edited files get the same "Organize Imports" reorganization as VS Code's TS Language Service (sorts, dedupes, removes unused). Wires it into a husky pre-commit hook via lint-staged so only staged source files get reformatted on commit — no bulk pass across the existing codebase. To keep barrel re-export order stable (alphabetizing `export *` lines breaks runtime init order in the bundled outputs), every file containing top-level `export * from` is marked `// organize-imports-ignore`. The plugin will leave their order alone, while still organizing `import` statements inside every other file. Also fixes the format glob: `integrations` (non-existent) → `providers`.
sroussey
force-pushed
the
claude/add-import-reorganization-JjEUS
branch
from
2960609 to
8c7789a
Compare
sroussey
added a commit
that referenced
this pull request
May 13, 2026
…main Rebased onto main (#488 added the prettier-organize-imports plugin + husky hook). With -X theirs winning conflicts in our favor, three `export * from` barrels lost main's `// organize-imports-ignore` header. Re-add the comment so the plugin doesn't alphabetize the barrel export order (which would break runtime init). - packages/ai/src/common.ts (existed pre-PR, comment lost in rebase) - packages/ai/src/task/index.ts (existed pre-PR, comment lost in rebase) - packages/ai/src/capability/index.ts (new barrel introduced by this PR) Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Adds prettier-plugin-organize-imports so
bun run formatperformsthe same "Organize Imports" reorganization as VS Code's TS Language
Service (sorts, dedupes, removes unused).
Also fixes the format glob: replace the non-existent
integrationsdirectory with
providers, which is an actual workspace.