Skip to content

feat: allow comments (JSONC) in vite-task.json#139

Merged
branchseer merged 4 commits intomainfrom
02-05-test_add_comments_in_vite-task.json
Feb 5, 2026
Merged

feat: allow comments (JSONC) in vite-task.json#139
branchseer merged 4 commits intomainfrom
02-05-test_add_comments_in_vite-task.json

Conversation

@branchseer
Copy link
Copy Markdown
Member

@branchseer branchseer commented Feb 5, 2026

Add support for JSONC comments in vite-task.json

Use jsonc-parser to parse vite-task.json, enabling // line comments,
/* */ block comments, and trailing commas. Add comments to several
test fixture files to exercise this.

Co-Authored-By: Claude Opus 4.5 noreply@anthropic.com

Copy link
Copy Markdown
Member Author

branchseer commented Feb 5, 2026

This stack of pull requests is managed by Graphite. Learn more about stacking.

@branchseer branchseer marked this pull request as ready for review February 5, 2026 03:52
@branchseer branchseer requested a review from fengmk2 February 5, 2026 03:53
Base automatically changed from feat_tasks_-_run_with_cacheScripts_ to main February 5, 2026 05:07
branchseer and others added 3 commits February 5, 2026 13:08
Use jsonc-parser to parse vite-task.json, enabling // line comments,
/* */ block comments, and trailing commas. Add comments to several
test fixture files to exercise this.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
JsonUserConfigLoader is only instantiated in vite_task_bin, so it
belongs there rather than in the core graph crate. This also removes
the tokio dependency from vite_task_graph.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@branchseer branchseer force-pushed the 02-05-test_add_comments_in_vite-task.json branch from 938ea44 to 4a56d3e Compare February 5, 2026 06:38
@branchseer branchseer merged commit d5b543d into main Feb 5, 2026
7 checks passed
@branchseer branchseer deleted the 02-05-test_add_comments_in_vite-task.json branch February 5, 2026 16:23
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