Skip to content

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

Merged
wan9chi merged 4 commits into
mainfrom
02-05-test_add_comments_in_vite-task.json
Feb 5, 2026
Merged

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

Conversation

@wan9chi

@wan9chi wan9chi commented Feb 5, 2026

Copy link
Copy Markdown
Member

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

wan9chi commented Feb 5, 2026

Copy link
Copy Markdown
Member Author

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

@wan9chi
wan9chi marked this pull request as ready for review February 5, 2026 03:52
@wan9chi
wan9chi 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
wan9chi 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>
@wan9chi
wan9chi force-pushed the 02-05-test_add_comments_in_vite-task.json branch from 938ea44 to 4a56d3e Compare February 5, 2026 06:38
@wan9chi
wan9chi merged commit d5b543d into main Feb 5, 2026
7 checks passed
@wan9chi
wan9chi 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