Add Markstream installation skill 🤖🤖🤖#2357
Conversation
🔒 PR Risk Scan ResultsScanned 3 changed file(s).
|
|
🔴 Contributor Reputation Check: HIGH risk
Maintainers: please review this contributor before merging. |
🔍 Vally Lint Results✅ All checks passed
Summary
Full linter output |
There was a problem hiding this comment.
Pull request overview
Adds a Markstream installation skill covering framework selection, dependencies, styling, SSR, streaming, and security.
Changes:
- Adds the installation workflow and examples.
- Adds framework-specific scenario guidance.
- Registers the skill in generated documentation.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 5 comments.
| File | Description |
|---|---|
skills/markstream-install/SKILL.md |
Defines the installation workflow. |
skills/markstream-install/references/scenarios.md |
Maps frameworks, peers, CSS, and input modes. |
docs/README.skills.md |
Adds the skill to the catalog. |
| | React 18+ / Next.js | `markstream-react` | | ||
| | Angular 20+ | `markstream-angular` | | ||
| | Svelte 5 | `markstream-svelte` | |
| - Auto pacing activates when `typewriter=true` or `maxLiveNodes <= 0` / `max-live-nodes <= 0`. | ||
| - `typewriter` controls the cursor and defaults to `false`. | ||
| - `fade` controls node-entry and streamed-text fade effects. | ||
| - `nodes` plus `final`: worker-preparsed content, shared AST stores, custom AST transforms, or cases where another layer already owns parsing. |
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.
Comments suppressed due to low confidence (1)
skills/markstream-install/references/scenarios.md:42
- The input matrix also omits the documented Vue 2 exception: high-frequency SSE, long conversations, and large code/table/math blocks should use externally parsed
nodesto avoid reparsing the entirecontentvalue for every token. Without that case, this mandatory reference can select the expensive input path for the Vue 2 package.
- `nodes` plus `final`: worker-preparsed content, shared AST stores, custom AST transforms, or cases where another layer already owns parsing.
|
|
||
| In React, Svelte, and Angular, use the equivalent camelCase or framework binding syntax. Keep `smoothStreaming="auto"`, `fade=false`, and `typewriter=true` while streaming; use `smoothStreaming=false` and `typewriter=false` for completed history. | ||
|
|
||
| Use `nodes` plus `final` only when a worker, shared AST store, custom transform, or another application layer already owns parsing. |
Pull Request Checklist
npm startand verified thatREADME.mdis up to date.mainbranch for this pull request.Description
Adds a focused
markstream-installskill for GitHub Copilot users who need to select, install, and safely configure the appropriate Markstream package in an existing frontend project. It covers framework package selection, optional peers, CSS ordering, SSR boundaries, and streaming-renderer setup without requiring product sign-up links.Type of Contribution
Additional Notes
The source project is MIT-licensed and the contribution is maintained by its author.
npm run skill:validate,npm run build, andgit diff --checkcompleted successfully. The GitHub Copilot interactive test and the template-specificnpm startcommand have not been run.By submitting this pull request, I confirm that my contribution abides by the Code of Conduct and will be licensed under the MIT License.