Skip to content

docs(dataset): fix V2 --type examples to match CLI validation (#51) - #57

Open
linhongyu510 wants to merge 1 commit into
volcengine:mainfrom
linhongyu510:fix/v2-dataset-type-docs-51
Open

linhongyu510 wants to merge 1 commit into
volcengine:mainfrom
linhongyu510:fix/v2-dataset-type-docs-51

Conversation

@linhongyu510

@linhongyu510 linhongyu510 commented Aug 28, 2026 •

Copy link
Copy Markdown

Summary

Fixes #51.

Recent upstream changes independently corrected most V2 dataset type examples and acceptance fixtures. This PR is now rebased onto the latest main and narrowed to the remaining root-help inconsistency.

Changes

  • Update the V2 quick-start dataset infer-schema example from rejected --type item to valid --type multi_modal --theme e_commerce.
  • Replace the stale root-help item assertion with deterministic positive and negative assertions for the supported example.

Verification

  • npm run validate:skills
  • npm run build
  • npm run test:acceptance:dist (54 passed, 0 failed)

@linhongyu510
linhongyu510 force-pushed the fix/v2-dataset-type-docs-51 branch from e28a78e to 68313cc Compare September 10, 2026 02:09
@linhongyu510

Copy link
Copy Markdown
Author

Rebased onto the latest main and force-pushed. Upstream had independently covered most of the original patch, so I narrowed this PR to the remaining invalid root-help example and its deterministic assertions.

Verification: npm run validate:skills, npm run build, and npm run test:acceptance:dist (54 passed, 0 failed).

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.

V2 dataset type examples are inconsistent with current CLI validation

1 participant