Skip to content

fix(create-expert): clarify demo sample data must be embedded in instruction#372

Merged
FL4TLiN3 merged 1 commit intomainfrom
fix/361-clarify-demo-template
Jan 3, 2026
Merged

fix(create-expert): clarify demo sample data must be embedded in instruction#372
FL4TLiN3 merged 1 commit intomainfrom
fix/361-clarify-demo-template

Conversation

@FL4TLiN3
Copy link
Contributor

@FL4TLiN3 FL4TLiN3 commented Jan 3, 2026

Summary

Clarify in the Demo Expert Template that sample data must be embedded directly in the instruction, not as placeholders.

Changes

  • Renamed "Sample Data" section to "Embedded Sample Data" with IMPORTANT note
  • Added concrete example showing what embedded sample data looks like (JSON format)
  • Emphasized "do NOT make API calls" in demo mode instructions

Rationale

The previous template had [Include relevant sample data for demonstration] which was ambiguous. This update makes it explicit that the ecosystem-builder must generate actual sample data, not placeholder text.

Closes #361

🤖 Generated with Claude Code


Note

Strengthens the Demo Expert template to ensure zero-config demos use real embedded data.

  • Updates apps/create-expert/src/lib/create-expert-toml.ts Demo Expert instructions: require embedded sample data (renamed to Embedded Sample Data), add an IMPORTANT note and a concrete JSON example, and emphasize “do NOT make API calls” in demo mode
  • Adds .changeset/clarify-demo-sample-data.md for a patch release

Written by Cursor Bugbot for commit 9c496e5. This will update automatically on new commits. Configure here.

…ruction

Updated Demo Expert Template to:
- Rename 'Sample Data' to 'Embedded Sample Data' with IMPORTANT note
- Add example showing what embedded sample data looks like
- Emphasize 'do NOT make API calls' in demo mode

Closes #361
@codecov
Copy link

codecov bot commented Jan 3, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@FL4TLiN3 FL4TLiN3 merged commit c4364f1 into main Jan 3, 2026
7 checks passed
@FL4TLiN3 FL4TLiN3 deleted the fix/361-clarify-demo-template branch January 3, 2026 06:07
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.

Fix: Demo Expert template includes readTextFile which may not be needed

1 participant