Skip to content

Conversation

@camchenry
Copy link
Member

@camchenry camchenry commented Oct 24, 2025

Adds a -fixand -fix-suggestions option to headless mode to enable generating fixes and suggestions which will be returned in the payload. If there are no fixes or suggestions, we omit the field entirely instead of returning an empty array.

Copy link
Member Author

camchenry commented Oct 24, 2025

@camchenry camchenry changed the base branch from 10-22-refactor_allow_suggestions_fixes_to_be_generated_lazily to graphite-base/317 October 24, 2025 14:38
@camchenry camchenry force-pushed the 10-23-feat_make_fixes_suggestions_opt-in branch from 5177d4a to fe274cb Compare October 24, 2025 14:39
@camchenry camchenry changed the base branch from graphite-base/317 to 10-22-refactor_allow_suggestions_fixes_to_be_generated_lazily October 24, 2025 14:39
@camchenry camchenry force-pushed the 10-23-feat_make_fixes_suggestions_opt-in branch 2 times, most recently from 83aede2 to b8bdfdf Compare October 24, 2025 15:27
@camchenry camchenry marked this pull request as ready for review October 24, 2025 15:49
@camchenry camchenry changed the title feat: make fixes/suggestions opt-in feat!: make fixes/suggestions opt-in Oct 24, 2025
@camchenry camchenry requested a review from camc314 October 27, 2025 17:20
@camchenry camchenry changed the base branch from 10-22-refactor_allow_suggestions_fixes_to_be_generated_lazily to graphite-base/317 October 27, 2025 23:32
@camchenry camchenry force-pushed the 10-23-feat_make_fixes_suggestions_opt-in branch from fa558f6 to 4c4dace Compare October 27, 2025 23:32
@camchenry camchenry changed the base branch from graphite-base/317 to 10-22-refactor_allow_suggestions_fixes_to_be_generated_lazily October 27, 2025 23:32
@camchenry camchenry force-pushed the 10-22-refactor_allow_suggestions_fixes_to_be_generated_lazily branch from e38f138 to 11f985e Compare October 29, 2025 01:29
@camchenry camchenry force-pushed the 10-23-feat_make_fixes_suggestions_opt-in branch from 4c4dace to a0e26f3 Compare October 29, 2025 01:29
@graphite-app graphite-app bot changed the base branch from 10-22-refactor_allow_suggestions_fixes_to_be_generated_lazily to graphite-base/317 October 29, 2025 01:52
@graphite-app graphite-app bot force-pushed the graphite-base/317 branch from 11f985e to ea27947 Compare October 29, 2025 02:02
@graphite-app graphite-app bot force-pushed the 10-23-feat_make_fixes_suggestions_opt-in branch from a0e26f3 to 0c58d3f Compare October 29, 2025 02:02
@graphite-app graphite-app bot changed the base branch from graphite-base/317 to main October 29, 2025 02:03
@graphite-app graphite-app bot force-pushed the 10-23-feat_make_fixes_suggestions_opt-in branch from 0c58d3f to d8a7d52 Compare October 29, 2025 02:03
@camchenry camchenry force-pushed the 10-23-feat_make_fixes_suggestions_opt-in branch from d8a7d52 to e857d26 Compare October 29, 2025 02:07
graphite-app bot pushed a commit to oxc-project/oxc that referenced this pull request Oct 29, 2025
In preparation for oxc-project/tsgolint#317. Enables oxlint to support missing fixes/suggestions in the payload from tsgolint.
Copy link
Contributor

@camc314 camc314 left a comment

Choose a reason for hiding this comment

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

thanks for working on this, apologies have been OOO since beginning of this week

Copy link
Contributor

camc314 commented Oct 29, 2025

Merge activity

Adds a `-fix`and `-fix-suggestions` option to headless mode to enable generating fixes and suggestions which will be returned in the payload. If there are no fixes or suggestions, we omit the field entirely instead of returning an empty array.
@graphite-app graphite-app bot force-pushed the 10-23-feat_make_fixes_suggestions_opt-in branch from e857d26 to f430beb Compare October 29, 2025 21:36
@graphite-app graphite-app bot merged commit f430beb into main Oct 29, 2025
6 checks passed
@graphite-app graphite-app bot deleted the 10-23-feat_make_fixes_suggestions_opt-in branch October 29, 2025 21:46
@graphite-app graphite-app bot removed the 0-merge label Oct 29, 2025
graphite-app bot pushed a commit to oxc-project/oxc that referenced this pull request Oct 30, 2025
With oxc-project/tsgolint#317, tsgolint fixes will become opt-in. So, we need to pass the `-fix` and `-fix-suggestions` when fixes are enabled in oxlint.

@camc314 This will need to be added in oxlint with, or before switching to the next version of tsgolint.
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