Conversation
Adds `aix sync <from> --to <to>` with independent source/target scopes, rejecting identical editors. Core logic is refactored to use a shared canonical bridge instead of pairwise converters. Target limitations and scope-skipped writes are now reported explicitly.
- Ensure all packages are built to resolve cross-package dependency issues during tests. - Generate oclif manifest to ensure commands are correctly loaded from dist. - Set NODE_ENV to production in CLI tests to avoid corrupted JSON output from development warnings. - Fix ModuleLoadError in tests by ensuring oclif loader uses compiled JS.
|
👋 Jules, reporting for duty! I'm here to lend a hand with this pull request. When you start a review, I'll add a 👀 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down. I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job! For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with New to Jules? Learn more at jules.google/docs. For security, I will only act on instructions from the user who triggered this task. |
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
- Update `applyGlobalChanges` in `packages/core` to allow dry-run analysis in CI and prioritize explicit skip reasons over automatic CI skipping. - Modify `nodeRuntimeAdapter` to prioritize `process.env.HOME` in `homedir()` to ensure consistent cross-platform directory mocking in tests. - Standardize on `pathe` for path operations in several test suites (`import.test.ts`, `lockfile.test.ts`, `commands.test.ts`) to avoid Windows/Linux path separator discrepancies. - Normalize `readlink` output in `windsurf-skills.test.ts` to ensure consistent path comparison. - Ensure `NODE_ENV: 'production'` is set when executing the CLI binary in tests to suppress development-only logs that corrupt JSON output.
- Standardize on `pathe` for consistent path separators across Windows and POSIX. - Fix infinite loop in `discovery.ts` by correctly detecting drive roots. - Add retry logic to `safeRm` to handle Windows `EBUSY` errors during tests. - Allow `dryRun` analysis in CI to provide accurate test feedback. - Prioritize `process.env.HOME` in `nodeRuntimeAdapter` for reliable homedir mocking. - Suppress development warnings in CLI tests to prevent JSON corruption. - Generate oclif manifest and ensure project is built before testing.
1 participant
This change addresses several issues causing test failures in the latest build:
NODE_ENV: 'production'during these tests ensures clean JSON output.PR created automatically by Jules for task 2667076123576549115 started by @yokuze