Skip to content

Conversation

@ocavue
Copy link
Collaborator

@ocavue ocavue commented Oct 30, 2025

Moves some code to fix the following "Circular dependencies" build warning. No runtime logic change.

Before:

❯ pnpm run build

src/index.ts → dist/index.js, dist/index.mjs...
(!) Circular dependencies
src/sync-plugin.ts -> src/cursor-plugin.ts -> src/sync-plugin.ts
src/sync-plugin.ts -> src/undo-plugin.ts -> src/sync-plugin.ts
created dist/index.js, dist/index.mjs in 127ms

After:

❯ pnpm run build

src/index.ts → dist/index.js, dist/index.mjs...
created dist/index.js, dist/index.mjs in 129ms

@github-actions
Copy link
Contributor

github-actions bot commented Oct 30, 2025

Coverage Report

Status Category Percentage Covered / Total
🔵 Lines 11.83% 612 / 5170
🔵 Statements 11.83% 612 / 5170
🔵 Functions 48.78% 20 / 41
🔵 Branches 64.74% 90 / 139
File Coverage
File Stmts Branches Functions Lines Uncovered Lines
Changed Files
src/cursor-plugin.ts 0% 0% 0% 0% 1-395
src/index.ts 0% 0% 0% 0% 1-25
src/sync-plugin-key.ts 0% 0% 0% 0% 1-23
src/sync-plugin.ts 0% 0% 0% 0% 1-237
src/undo-plugin-key.ts 0% 0% 0% 0% 1-18
src/undo-plugin.ts 0% 0% 0% 0% 1-180
Generated in workflow #138 for commit 09788ae by the Vitest Coverage Report Action

@ocavue ocavue marked this pull request as ready for review October 30, 2025 04:05
@ocavue ocavue requested a review from zxch3n October 30, 2025 04:05
@ocavue ocavue merged commit e228ca5 into main Oct 30, 2025
4 of 5 checks passed
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.

3 participants